Skip to content

Add StrandsCurator.hook() and end-to-end tests#12

Merged
SheaHawkins merged 1 commit into
mainfrom
claude/curator-hook
May 28, 2026
Merged

Add StrandsCurator.hook() and end-to-end tests#12
SheaHawkins merged 1 commit into
mainfrom
claude/curator-hook

Conversation

@skillos-bot

Copy link
Copy Markdown
Collaborator

StrandsCurator.hook() returns a Strands HookProvider that can be passed
directly to Agent(hooks=[curator.hook()]). The hook registers an
AfterInvocationEvent callback that feeds agent.messages to the curator
after every invocation. Empty message lists are skipped.

Tests:

  • test_hook.py: hook provider creation, registry registration, fires
    after invocation, skips empty messages.
  • test_end_to_end.py: full flow with mocked curator LLM — user agent
    conversation → curator extracts skill → skill lands in repo. Also
    covers mixed success/failure and no-op curator runs.

https://claude.ai/code/session_01K9ZQSP244HQWEobYU7euik

StrandsCurator.hook() returns a Strands HookProvider that can be passed
directly to Agent(hooks=[curator.hook()]). The hook registers an
AfterInvocationEvent callback that feeds agent.messages to the curator
after every invocation. Empty message lists are skipped.

Tests:
- test_hook.py: hook provider creation, registry registration, fires
  after invocation, skips empty messages.
- test_end_to_end.py: full flow with mocked curator LLM — user agent
  conversation → curator extracts skill → skill lands in repo. Also
  covers mixed success/failure and no-op curator runs.

https://claude.ai/code/session_01K9ZQSP244HQWEobYU7euik
@SheaHawkins SheaHawkins merged commit 760352c into main May 28, 2026
4 checks passed
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.

3 participants