Skip to content

fix: make on-create hook defaults sane - #68

Merged
c-h- merged 1 commit into
mainfrom
fix/hook-defaults
Feb 25, 2026
Merged

c-h- merged 1 commit into
mainfrom
fix/hook-defaults

Conversation

@c-h-

@c-h- c-h- commented Feb 25, 2026

Copy link
Copy Markdown
Collaborator

Summary

  • Increase default --on-create hook timeout from 60s to 300s (5 minutes) for large monorepo installs
  • Make hook failures throw instead of being silently swallowed, so a failed on-create hook aborts the launch rather than starting the agent in a broken worktree
  • Update test to expect throw on hook failure

Fixes #55

Test plan

  • npm test — all 388 tests pass
  • npm run typecheck — clean
  • Pre-push hooks pass (lint, typecheck, build, test)

🤖 Generated with Claude Code

Increase default hook timeout from 60s to 300s for large monorepo
installs. Make hook failures throw instead of being silently swallowed,
so a failed on-create hook aborts the launch rather than starting the
agent in a broken worktree.

Fixes #55

Co-Authored-By: Charlie Hulcher <[email protected]>
Co-Authored-By: Claude Opus 4.6 <[email protected]>
@c-h-
c-h- merged commit 36c2df9 into main Feb 25, 2026
1 check passed
c-h- added a commit that referenced this pull request Feb 26, 2026
## What's New
- fix: resolve pending- session IDs so peek/stop/resume work (#63, fixes #60)
- feat: add `logs` command as alias for `peek` with 50-line default (#62, fixes #57)
- fix: omit undefined fields in lock error messages (#66, fixes #58)
- docs: document default --cwd behavior (#64, fixes #59)
- fix: make on-create hook defaults sane — 300s timeout, abort on failure (#68, fixes #55)

Co-Authored-By: Charlie Hulcher <[email protected]>
@c-h- c-h- mentioned this pull request Feb 26, 2026
c-h- added a commit that referenced this pull request Feb 26, 2026
## What's New
- fix: resolve pending- session IDs so peek/stop/resume work (#63, fixes #60)
- feat: add `logs` command as alias for `peek` with 50-line default (#62, fixes #57)
- fix: omit undefined fields in lock error messages (#66, fixes #58)
- docs: document default --cwd behavior (#64, fixes #59)
- fix: make on-create hook defaults sane — 300s timeout, abort on failure (#68, fixes #55)

Co-authored-by: Doink (OpenClaw) <[email protected]>
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.

Enhancement: Make --on-create hook timeout configurable (60s too short for yarn install)

1 participant