Repository navigation
chore: add Entire's StopFailure hook for Claude Code - #1601
Conversation
Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
|
No actionable comments were generated in the recent review.
|
| Layer / File(s) | Summary |
|---|---|
Configure StopFailure hook .claude/settings.json |
Adds a StopFailure hook that runs entire hooks claude-code stop-failure when entire is on PATH. Otherwise, the command exits successfully. |
Estimated code review effort: 1 (Trivial) | ~5 minutes
Change: Other
Merge Risk: 
Minimal · up to a167b
-
The configuration-only change adds the intended StopFailure hook without identified merge-blocking risk.
Pre-merge checks: 
5
-
Full details: Docstring Coverage
-
Explanation
No eligible functions found in the changed files. Skipping docstring coverage check. Docstring coverage is scoped to functions touched by this diff with cyclomatic complexity of at least 2. Documentation-only edits and deletions within surviving functions are included; unchanged diff context and fully deleted functions are excluded. Analyzed 0 functions across 0 files. (1 skipped: 1 unsupported.)
-
- Autofix · Keep fixing CodeRabbit findings and required CI, and resolving merge conflicts
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.
[Low impact] This PR appears safe to merge. SummaryAdds an Entire
Reviews (1) · Last reviewed commit: "chore: add Entire's StopFailure hook for..." · Reviewed by Greptile |
Instruction counts
No instruction-count regression above 1%. Only instruction counts gate. Wall clock is shown for context — on identical hardware it moves 4-20% run to run. Measured by tak — instruction-counted CLI benchmarks, stored in this repository's git notes. Shadow comparisonParsing
|
### 🐛 Bug Fixes - **(spec)** reject several targets in a relationship property by [@jdx](https://github.com/jdx) in [#1599](#1599) ### 🔍 Other Changes - add Entire's StopFailure hook for Claude Code by [@jdx](https://github.com/jdx) in [#1601](#1601)
entire statuswarns "Claude Code hooks out of date · run 'entire enable --force'" in this repository. Entire CLI 0.11.x added aStopFailurehook for Claude Code, which this repository's.claude/settings.jsondoesn't have yet.This adds that hook, exactly as
entire enable --force --agent claude-codewrites it. It's the samesh -c 'if ! command -v entire …; then exit 0; fi; exec entire hooks claude-code stop-failure'form as the existing Entire hooks, so it does nothing whereentireisn't installed.entire enable --forcealso re-sorts.entire/settings.jsonand.claude/settings.json; those reorderings are left out, so the diff is only the new hook. After the change,entire statuson this branch no longer shows the warning.AI-assisted — Tool: Claude Code; model: anthropic/claude-opus-5-5; version: claude-code_2-1-309_agent.
🤖 Generated with Claude Code
Summary by CodeRabbit