feat: register additional CC hook types (#571)#939
feat: register additional CC hook types (#571)#939OneStepAt4time wants to merge 2 commits intomainfrom
Conversation
TOCTOU fix: the test job now uploads dist/ and dashboard/dist/ as artifacts. The publish-npm job downloads these instead of rebuilding from scratch, ensuring the exact same artifacts that passed tests are published to npm. Fixes #649
FileChanged, CwdChanged, PreCompact, PostCompact added to HTTP_HOOK_EVENTS. These hooks provide telemetry for real-time file tracking and context compaction detection. Quality gate: tsc ✅ build ✅ 2173 tests ✅
![aegis-gh-agent[bot]](https://avatars.githubusercontent.com/u/106186915?s=60&v=4){kind=small}
There was a problem hiding this comment.
Minor note: feat: is used for registering 4 new hook types (FileChanged, CwdChanged, PreCompact, PostCompact) but these are not yet wired to any handler — they are just accepted registrations. Consider using chore: or refactor: for incomplete declarations. Since the PR is otherwise correct and CI green, approving.
|
Self-rebase failed — conflicts with merged PR #937 (removed unused sessionId props) across ApprovalBanner.tsx, PanePreview.tsx, SessionDetailPage.tsx, hook-settings.ts, hook-settings.test.ts. Please rebase onto current main, resolve conflicts (keep only the HTTP_HOOK_EVENTS additions), and force-push. CI must pass after rebase. |
|
This PR is superseded by #945 (register additional CC hook types) which is now merged. Please close this PR. |
1 participant
Summary
Registers 4 additional CC hook types in
HTTP_HOOK_EVENTS:FileChanged— Track which files an agent is modifying in real-timeCwdChanged— Know when agent changes directoryPreCompact/PostCompact— Detect when CC compacts contextChanges
src/hook-settings.ts: Added 4 events toHTTP_HOOK_EVENTSsrc/__tests__/hook-settings.test.ts: Updated test to useHTTP_HOOK_EVENTS.lengthinstead of hardcoded valueTest Plan
Developed with: v2.6.3
Tested with: v2.6.3