docs: add root CLAUDE.md naming make test as the verification command - #83
Conversation
|
No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configurationConfiguration used: defaults Review profile: CHILL Plan: Pro Plus Run ID: 📒 Files selected for processing (1)
📝 WalkthroughWalkthroughAdded ChangesRepository operating rules
Estimated code review effort: 1 (Trivial) | ~2 minutes 🚥 Pre-merge checks | ✅ 5✅ Passed checks (5 passed)
✨ Finishing Touches🧪 Generate unit tests (beta)
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. Comment |
|
@coderabbitai review |
✅ Action performedReview finished.
|
What
Adds a root
CLAUDE.mdwith two sections:make testas the verification command and documents the two probes that fail in a way that reads as "pytest is unavailable" (uv run pytest,python3 -m pytest).main+pr-watch, config inconfig/dev-model.yaml, the living handoff / friction log, and the conservative closing-keyword rule (as settled in 42873d8).Why
The friction log (2026-07-27, H) records that no markdown file in the repo mentions
make testand there is no rootCLAUDE.md. The previous session concluded tests could not run locally and wrote a false "tests were not run locally — pytest is not installed" into the body of a merged PR (#80). This file is the discoverability precondition for #54 (#54 stays open — it proposes the rule itself; this change only makes the rule satisfiable).Verification
Run in this session, before writing the file:
make test→372 passed in 22.95suv run pytest→error: Failed to spawn: pytest — No such file or directorypython3 -m pytest --version→No module named pytestThe file's claims about the suite paths match
Makefile:31-32. Docs-only change; no engine or config touched.Summary by CodeRabbit