Skip to content

feat: register additional CC hook types (#571)#945

Merged
OneStepAt4time merged 1 commit intomainfrom
feat/571-1775198856
Apr 3, 2026
Merged

feat: register additional CC hook types (#571)#945
OneStepAt4time merged 1 commit intomainfrom
feat/571-1775198856

Conversation

@OneStepAt4time
Copy link
Copy Markdown
Owner

Summary

Register additional Claude Code hook types:

  • PreCompact, PostCompact — context management
  • FileChanged, CwdChanged — file and directory change tracking

Changes

  • src/hook-settings.ts: Add 4 new hook events to HTTP_HOOK_EVENTS array
  • src/tests/hook-settings.test.ts: Update test count 14→18, add assertions for new events

Tests

npx tsc --noEmit && npm run build && npm test → 2159 passed

Fixes #571

Copy link
Copy Markdown
Contributor

@aegis-gh-agent aegis-gh-agent bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

4 new hook types correctly added to HTTP_HOOK_EVENTS, tests updated. Approved.

@aegis-gh-agent
Copy link
Copy Markdown
Contributor

aegis-gh-agent bot commented Apr 3, 2026

Self-rebase failed — conflicts with #944 (merged) across hook-settings.ts, hook-settings.test.ts, release.yml, session.ts. Please rebase onto current main and resolve. CI must pass before merge.

@aegis-gh-agent
Copy link
Copy Markdown
Contributor

aegis-gh-agent bot commented Apr 3, 2026

Self-rebase failed — conflicts with merged PRs across release.yml, hook-settings.ts, hook-settings.test.ts, session.ts. Please rebase onto current main and force-push.

@aegis-gh-agent
Copy link
Copy Markdown
Contributor

aegis-gh-agent bot commented Apr 3, 2026

Rebased branch conflicts with #940 (just merged). Please rebase onto current main and force-push. Approved — will merge immediately on green CI.

@aegis-gh-agent
Copy link
Copy Markdown
Contributor

aegis-gh-agent bot commented Apr 3, 2026

Rebased again but #943 merge added new conflicts (hook-settings.ts, hook-settings.test.ts, server.ts, session.ts, validation.ts). Please rebase once more onto current main and push. Approved — will merge immediately.

- Add PreCompact, PostCompact (context management)
- Add FileChanged, CwdChanged (file/directory changes)
- Update hook-settings test count from 14 to 18
@OneStepAt4time OneStepAt4time merged commit 8cd5ab8 into main Apr 3, 2026
5 checks passed
@OneStepAt4time OneStepAt4time deleted the feat/571-1775198856 branch April 3, 2026 08:14
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

feat: register additional CC hook types (FileChanged, CwdChanged, PreCompact, PostCompact)

1 participant