Skip to content

Conversation

@xiaocang
Copy link

@xiaocang xiaocang commented Jan 10, 2026

Closes #5515
Related to #7640

Summary

  • Added new input_required hook type to the config schema for notifications when input is needed
  • Implemented configurable timeout option for all hooks (default 30000ms)
  • Integrated hook initialization during instance bootstrap

Test plan

  • Verify hook configuration is properly loaded from config
  • Test input_required hook fires when session requires user input
  • Test timeout configuration works as expected
  • Verify hooks don't block normal operation

🤖 Generated with Claude Code

@github-actions
Copy link
Contributor

Thanks for your contribution!

This PR doesn't have a linked issue. All PRs must reference an existing issue.

Please:

  1. Open an issue describing the bug/feature (if one doesn't exist)
  2. Add Fixes #<number> or Closes #<number> to this PR description

See CONTRIBUTING.md for details.

@github-actions
Copy link
Contributor

The following comment was made by an LLM, it may be inaccurate:

No duplicate PRs found

Added a new Hook module that manages lifecycle hooks with:
- New input_required hook type in config schema
- Configurable timeout option for all hooks (default 30000ms)
- Hook initialization during instance bootstrap
@xiaocang xiaocang force-pushed the feat/session-completed-hook branch from d197c58 to 80049b3 Compare January 12, 2026 01:58
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.

[FEATURE]: Finish hook, when LLM is ready to get new instructions

1 participant