Skip to content

docs(review): enforce rule against challenging PR's stated intent#3420

Open
kilo-code-bot[bot] wants to merge 1 commit into
mainfrom
review-respect-pr-intent
Open

docs(review): enforce rule against challenging PR's stated intent#3420
kilo-code-bot[bot] wants to merge 1 commit into
mainfrom
review-respect-pr-intent

Conversation

@kilo-code-bot
Copy link
Copy Markdown
Contributor

@kilo-code-bot kilo-code-bot Bot commented May 22, 2026

Summary

  • Adds a new top-level rule to REVIEW.md that explicitly prohibits the reviewer from raising issues about a PR's stated goals, scope decisions, or intentional trade-offs unless the approach is unsafe (data loss, security vulnerability, correctness bug).
  • The rule is placed at the top of the file and marked as the highest-priority, overriding all other review guidance.
  • Motivated by PR fix(code-reviews): bound cron pending drain #3396 where the bot flagged "old pending rows will become invisible after deploy" — a concern the PR description had already acknowledged as an intentional design decision ("Existing old idle pending reviews need manual retry or a one-time cleanup if we want to move them").

Verification

  • Read the updated REVIEW.md to confirm the new section is clear, prominent, and includes a concrete counter-example showing what a violation looks like.

Visual Changes

N/A

Reviewer Notes

The new rule has a single exception: if the described approach is objectively unsafe (data loss, security vuln, or incorrect behaviour contrary to the PR's own goals), it should still be flagged as CRITICAL. All other design/scope challenges are prohibited.


Built for alex-alecu by Kilo

@kilo-code-bot
Copy link
Copy Markdown
Contributor Author

kilo-code-bot Bot commented May 22, 2026

Code Review Summary

Status: No Issues Found | Recommendation: Merge

Files Reviewed (1 file)
  • REVIEW.md

Reviewed by claude-sonnet-4.6 · 161,109 tokens

Review guidance: REVIEW.md from base branch main

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.

0 participants