Add operating-method, code-change-discipline, and ship-and-recover rules#319
Add operating-method, code-change-discipline, and ship-and-recover rules#319Sharrmavishal wants to merge 2 commits into
Conversation
Three project-agnostic Cursor rules from operating-kit: - operating-method: builder + strategic-vetting persona with hard gates, always-on - code-change-discipline: re-read before edit, bug-as-hypothesis, 3-edit stop rule - ship-and-recover: deploy discipline and production incident recovery Each rule references playbooks in docs/claude/ for the full context. Source: https://github.com/Sharrmavishal/operating-kit Co-authored-by: Cursor <cursoragent@cursor.com>
|
No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configurationConfiguration used: defaults Review profile: CHILL Plan: Pro Run ID: 📒 Files selected for processing (3)
✅ Files skipped from review due to trivial changes (3)
📝 WalkthroughWalkthroughThree new Cursor MDC rule files are added: ChangesCursor Operating Rules
Estimated code review effort🎯 1 (Trivial) | ⏱️ ~5 minutes Suggested reviewers
Poem
🚥 Pre-merge checks | ✅ 5✅ Passed checks (5 passed)
✏️ Tip: You can configure your own custom pre-merge checks in the settings. ✨ 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 |
There was a problem hiding this comment.
Actionable comments posted: 2
🤖 Prompt for all review comments with AI agents
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.
Inline comments:
In `@rules/operating-method.mdc`:
- Around line 28-36: The references to `@docs/claude/operating-principles.md`,
`@docs/claude/vigilance-protocol.md`, `@docs/claude/multi-model-collaboration.md`,
`@docs/claude/field-notes.md`, and `@docs/claude/incident-response.md` in the
operating-method.mdc file point to files that do not exist in the repository
since there is no docs/ directory. Either remove these broken references
entirely from the "Full playbooks" section, or replace them with references to
actual existing documentation or resources that are available in the repository.
Ensure all references in operating-method.mdc, ship-and-recover.mdc, and
code-change-discipline.mdc are consistent and point to valid locations.
In `@rules/ship-and-recover.mdc`:
- Around line 3-4: The rule file has an empty globs field while alwaysApply is
set to false, which violates the rule-frontmatter contract. Fix this by either
adding non-empty glob patterns to the globs field to specify when the rule
should trigger on-demand, or change alwaysApply to true if the rule is intended
to always be active.
🪄 Autofix (Beta)
Fix all unresolved CodeRabbit comments on this PR:
- Push a commit to this branch (recommended)
- Create a new PR with the fixes
ℹ️ Review info
⚙️ Run configuration
Configuration used: defaults
Review profile: CHILL
Plan: Pro
Run ID: 8e5fc498-d9e3-48dd-8439-99bfca896187
📒 Files selected for processing (3)
rules/code-change-discipline.mdcrules/operating-method.mdcrules/ship-and-recover.mdc
…dd globs to ship-and-recover - Replace all @docs/claude/* references with direct GitHub URLs to the operating-kit source repo, since those files don't exist in this repo. The local @docs/ references apply when operating-kit is installed in a project; standalone these rules work without them. - Add CI/deployment file globs to ship-and-recover.mdc to satisfy the frontmatter contract (alwaysApply: false requires non-empty globs). Globs cover GitHub Actions, Dockerfile, docker-compose, Makefile, and deploy/release scripts. Co-authored-by: Cursor <cursoragent@cursor.com>
Three project-agnostic Cursor rules from operating-kit, a portable self-installing operating method for Claude Code and Cursor.
Rules added:
operating-method.mdc(alwaysApply: true)Builder + strategic-vetting persona with hard gates: permission discipline, production-mutation pre-flight (Backup/Ownership/Blast-radius/Mechanism/Memory), scratch-test before prod, milestone gate, verify-don't-infer. References playbooks in
docs/claude/for teams that want the full context.code-change-discipline.mdc(auto-attached on code globs)Re-read the whole function before editing, treat bugs as hypotheses before fixing, stop and consolidate after 3 edits to one file, scan field-notes before deploying.
ship-and-recover.mdc(on-demand)Deploy discipline: test > build > deploy > verify-live. Incident recovery: stabilize > confirm recovery point > recover one change at a time > verify > root-cause > mechanical fix > postmortem.
All three are project-agnostic and work standalone or alongside the full kit. The
globsincode-change-discipline.mdccover common source extensions and should be tuned per project.Made with Cursor
Summary by CodeRabbit
New Features
Documentation