Skip to content

Add after rules for PostToolUse feedback#32

Merged
ldayton merged 3 commits intomainfrom
lily/config-v1
Jan 14, 2026
Merged

Add after rules for PostToolUse feedback#32
ldayton merged 3 commits intomainfrom
lily/config-v1

Conversation

@ldayton
Copy link
Copy Markdown
Owner

@ldayton ldayton commented Jan 14, 2026

Summary

Implements the after directive from config-v1.md spec. After rules let users define messages sent to Claude after commands complete:

after git push * "Check CI status"
after npm install *      # silent override (bare pattern)
after npm install * ""   # silent override (explicit empty)
  • Last match wins semantics (consistent with other rules)
  • Bare pattern or empty message = silent (no output)
  • Detects PostToolUse hook event and routes to new handler
  • 17 new tests covering parsing, matching, and hook integration

Implements the `after` directive from config-v1.md spec. After rules
let users define messages sent to Claude after commands complete:

    after git push * "Check CI status"
    after npm install * ""  # silent override

Last match wins. Empty message = silent (no output).
Move assistant hook docs to docs/assistants/ and add:
- codex-cli.md: comprehensive Codex CLI reference
- _comparison.md: hook feature comparison across all four assistants
Fixes quoted argument handling in PostToolUse - patterns like
'git commit -m fix:*' now match 'git commit -m "fix: message"'.
@ldayton ldayton merged commit b8b259b into main Jan 14, 2026
2 checks passed
@ldayton ldayton deleted the lily/config-v1 branch January 14, 2026 16:10
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.

1 participant