docs(agents): drop contributor PR rules from the agent brief - #404
Conversation
AGENTS.md carried CONTRIBUTING.md's contributor workflow as binding on agent sessions: PR size caps, one-logical-change, confirm-before-starting. Applied to maintainer work that produces one fix per finding, it split single tasks across many PRs. CONTRIBUTING.md still governs outside contributors; it no longer gates sessions in this repo. The no-LLM-in-the-control-loop invariant stays -- only its PR-rejection framing goes.
There was a problem hiding this comment.
pbean has reached the 50-credit limit for trial accounts. To continue receiving code reviews, upgrade your plan.
|
Caution Review failedThe pull request is closed. ℹ️ Recent review info⚙️ Run configurationConfiguration used: Organization UI Review profile: CHILL Plan: Pro Plus Run ID: 📒 Files selected for processing (1)
Walkthrough
ChangesAGENTS.md guidance
Estimated code review effort: 1 (Trivial) | ~2 minutes Possibly related PRs
Poem
✨ 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 |
AGENTS.md carried CONTRIBUTING.md's contributor workflow as binding on agent sessions. Applied to maintainer work — where one task routinely surfaces several related fixes — the PR size caps and one-logical-change rule split single tasks across many PRs, multiplying review load rather than reducing it. CONTRIBUTING.md is unchanged and still governs outside contributors.
Removed
## Git & PRsheading, now## Repo hygieneThe no-LLM-in-the-control-loop invariant is untouched — only its PR-rejection framing went.
Kept
The three bullets that are hygiene, not PR bureaucracy: CHANGELOG style, never commit session notes or run artifacts, and review non-convergence → escalate rather than grind.
Notes
Nothing enforced the removed conventions mechanically — no commitlint, no release-please, no installed git hook — so this is a docs-only change with no tooling impact. CONTRIBUTING.md, README.md, and docs/README.md still document the contributor guidelines unchanged.
Summary by CodeRabbit