Skip to content

docs(agents): drop contributor PR rules from the agent brief - #404

Merged
pbean merged 1 commit into
mainfrom
docs/agents-drop-contributor-pr-rules
Jul 30, 2026
Merged

docs(agents): drop contributor PR rules from the agent brief#404
pbean merged 1 commit into
mainfrom
docs/agents-drop-contributor-pr-rules

Conversation

@pbean

@pbean pbean commented Jul 30, 2026

Copy link
Copy Markdown
Collaborator

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

  • "[CONTRIBUTING.md] is binding" and everything it pulled in: conventional commits (subject < 72 chars), PRs 200–400 LOC ideal / 800 max, one logical change, feature-sized work needs maintainer confirmation first
  • "Open PRs ready for review, never draft"
  • The ## Git & PRs heading, now ## Repo hygiene
  • In the intro: "PRs that move orchestration into an LLM are rejected ([CONTRIBUTING.md])" → "Never move orchestration into an LLM"

The 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

  • Documentation
    • Updated repository contribution guidance and terminology.
    • Refined guidance on repository hygiene, changelog entries, generated artifacts, and review follow-up.

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.

@greptile-apps greptile-apps Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

pbean has reached the 50-credit limit for trial accounts. To continue receiving code reviews, upgrade your plan.

@coderabbitai

coderabbitai Bot commented Jul 30, 2026

Copy link
Copy Markdown

Review Change Stack

Caution

Review failed

The pull request is closed.

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro Plus

Run ID: 12020d34-aa82-47d5-b2d9-b3defe70150d

📥 Commits

Reviewing files that changed from the base of the PR and between e260bf4 and aa739a1.

📒 Files selected for processing (1)
  • AGENTS.md

Walkthrough

AGENTS.md removes one orchestration-policy sentence and renames the “Git & PRs” section to “Repo hygiene,” while preserving the section’s existing guidance bullets.

Changes

AGENTS.md guidance

Layer / File(s) Summary
Update guidance wording and section heading
AGENTS.md
Removes the explicit rejection of moving orchestration into an LLM and renames the repository guidance section to “Repo hygiene.”

Estimated code review effort: 1 (Trivial) | ~2 minutes

Possibly related PRs

Poem

A bunny hops through guidance bright,
Trims one sentence, shifts a title right.
“Repo hygiene,” the pages say,
While steady rules remain in play.
Hop, hop—clean words lead the way!

✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch docs/agents-drop-contributor-pr-rules

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.

❤️ Share

Comment @coderabbitai help to get the list of available commands.

@pbean
pbean merged commit a6424ad into main Jul 30, 2026
9 of 10 checks passed
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