Skip to content

fix(hooks): extend session cleanup timeout - #160

Open
ketpatil77 wants to merge 1 commit into
vercel:mainfrom
ketpatil77:fix/session-end-cleanup-timeout
Open

fix(hooks): extend session cleanup timeout#160
ketpatil77 wants to merge 1 commit into
vercel:mainfrom
ketpatil77:fix/session-end-cleanup-timeout

Conversation

@ketpatil77

Copy link
Copy Markdown

Closes #122

Summary

  • give the SessionEnd cleanup hook a 15-second teardown budget
  • preserve fast normal exits while avoiding Claude Code's 1.5-second default cancellation
  • add a regression assertion for the hook configuration

Verification

  • npx --yes bun test tests/session-end-cleanup.test.ts (4 passed)
  • npx --yes bun test tests/hooks-json-structural.test.ts (6 passed)
  • git diff --check

The repository-wide structural validator still reports five pre-existing command/catalog failures unrelated to this change.

Signed-off-by: ketpatil77 <243740572+ketpatil77@users.noreply.github.com>
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.

SessionEnd cleanup hook gets cancelled on exit — hooks.json ships no timeout, and Claude Code's default for SessionEnd hooks is only 1.5s

1 participant