Uh oh!
There was an error while loading. Please reload this page.
Conversation
Warning Ignoring CodeRabbit configuration file changes. For security, only the configuration from the base branch is applied for open source repositories. No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configurationConfiguration used: Path: .coderabbit.yaml Review profile: ASSERTIVE Plan: Pro Run ID: 📒 Files selected for processing (3)
📝 WalkthroughWalkthroughAdds a strict “No Agent Marketing” policy and enforces it via pre-merge checks: documentation updated ( Changes
Estimated code review effort🎯 2 (Simple) | ⏱️ ~10 minutes Possibly related PRs
Suggested reviewers
🚥 Pre-merge checks | ✅ 4✅ Passed checks (4 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: 3
🤖 Prompt for all review comments with AI agents
Verify each finding against the current code and only fix it if needed.
Inline comments:
In @.coderabbit.yaml:
- Around line 54-57: The policy text for `.policies/no-agent-marketing.md` is
currently vague ("such as") and causes nondeterministic enforcement; update the
rule to list explicit, deterministic match patterns (e.g., exact phrases like
"Generated with", "Generated by", "Created with AI", "Co-Authored-By:
.*\\bAI\\b" and any specific agent names you want banned) and enumerated
exceptions (e.g., allow legitimate human Co-Authored-By trailers formatted as
"Co-Authored-By: Name <email@example.com>" or an explicit "Human co-author"
tag). Replace the vague wording at the rule definition (the policy referenced in
the diff) with these concrete phrases/patterns and a clear exception section so
the pre-merge check can evaluate deterministically at error level.
In @.policies/no-agent-marketing.md:
- Around line 30-32: Three fenced code blocks containing the commit message
"feat(worker): add request batching support" are missing language tags; update
the first commit-only block to use ```gitcommit, update the block that includes
"Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>" to use ```gitcommit as
well, and update the block that contains "(built with Copilot)" to use ```text
so markdownlint recognizes the languages.
- Line 89: Replace the lowercase word "markdown" with the proper noun "Markdown"
in the documentation text — specifically update the table cell containing
"Including AI tool attribution in a markdown link" to "Including AI tool
attribution in a Markdown link" so capitalization is consistent (look for the
exact string "Including AI tool attribution in a markdown link" to locate the
spot).
🪄 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: Path: .coderabbit.yaml
Review profile: ASSERTIVE
Plan: Pro
Run ID: 3770458a-bc39-43a5-9ca1-a46731855b99
📒 Files selected for processing (4)
.coderabbit.yaml.policies/index.md.policies/no-agent-marketing.mdAGENTS.md
Uh oh!
There was an error while loading. Please reload this page.
Uh oh!
There was an error while loading. Please reload this page.
Uh oh!
There was an error while loading. Please reload this page.
Uh oh!
There was an error while loading. Please reload this page.
Motivation
Agent tools often append marketing footers (e.g. "Generated with...",
"Co-Authored-By: ") to commits, PRs, issues, and comments. We are not
being paid for this free advertising, so it should not appear in our project
history.
Approach
.policies/no-agent-marketing.mdwith examplesof compliant and non-compliant content
.policies/index.md.coderabbit.yamlpre-merge checks so CodeRabbitenforces it automatically on PRs
Summary by CodeRabbit
Documentation
Chores