Skip to content

fix: make on-create hook defaults sane#68

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

fix: make on-create hook defaults sane#68
c-h- merged 1 commit intomainfrom
fix/hook-defaults

Conversation

@c-h-
Copy link
Copy Markdown
Collaborator

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

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 <charlie@kindo.ai>
Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
@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 <charlie@kindo.ai>
@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) <charlie+doink@kindo.ai>
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