Skip to content

feat: add notify hook system with input_required and timeout support - #7672

Closed
xiaocang wants to merge 1 commit into
anomalyco:devfrom
xiaocang:feat/session-completed-hook
Closed

xiaocang wants to merge 1 commit into
anomalyco:devfrom
xiaocang:feat/session-completed-hook

Conversation

@xiaocang

@xiaocang xiaocang commented Jan 10, 2026 •

Copy link
Copy Markdown

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
Copy Markdown
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
Copy Markdown
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
@github-actions

Copy link
Copy Markdown
Contributor

Closing this pull request because it has had no updates for more than 60 days. If you plan to continue working on it, feel free to reopen or open a new PR.

@github-actions github-actions Bot closed this Mar 13, 2026
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