Skip to content

Conversation

@mtpauly
Copy link
Contributor

@mtpauly mtpauly commented Aug 6, 2025

Update review request template to enforce precise anchoring for inline comments

This PR introduces a "Precise Anchoring" section to the code review prompt template, guiding reviewers to anchor comments by line + side rather than the patch-relative position field.

Key changes:

  • Instruct reviewers to anchor using line + side; avoid position.
  • Default to side="RIGHT" for added/modified lines; use side="LEFT" only for deleted lines.
  • Validate anchors before posting; if uncertain, prefer a single brief general comment over inline.
  • Allow suggestion blocks (```suggestion) only on side="RIGHT"; emphasize precise anchoring when proposing edits.
  • For multi-line ranges, set start_line/start_side and line/side consistently on the same side.

Impact:

  • Reduces off-by-one and hunk-position errors when anchoring comments.
  • Improves reliability of suggestion blocks and auto-applied edits.
  • No runtime or functional changes; documentation/prompt-only update.

🤖 This description was generated automatically. Please react with 👍 if it's helpful or 👎 if it needs improvement.

…ge and avoid position; emphasize validation and suggestions anchoring
Copy link
Contributor

@github-actions github-actions bot left a comment

Choose a reason for hiding this comment

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

Clear and helpful guidance on precise anchoring. I left one suggestion to fix an unmatched code-fence in the new section to avoid rendering issues.


🤖 Automated review complete. Please react with 👍 or 👎 on the individual review comments to provide feedback on their usefulness.

Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
@mtpauly mtpauly merged commit 3c189fe into main Aug 6, 2025
1 check passed
@nathro nathro deleted the chore/precise-anchoring branch August 6, 2025 19:41
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.

3 participants