Skip to content

Commit 2824bf6

Browse files
wenshaoclaude
andcommitted
fix(review): avoid #N notation in PR comments (GitHub auto-links)
GitHub renders #1, #2 as links to issues/PRs with those numbers. Review summaries using "#1 (logic error)" link to the wrong target. Added guideline: use (1), [1], or descriptive references instead. Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
1 parent ea2f8e1 commit 2824bf6

File tree

1 file changed

+1
-0
lines changed
  • packages/core/src/skills/bundled/review

1 file changed

+1
-0
lines changed

packages/core/src/skills/bundled/review/SKILL.md

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -560,4 +560,5 @@ These criteria apply to both Step 4 (review agents) and Step 5 (verification age
560560
- When suggesting a fix, show the actual code change.
561561
- Flag any exposed secrets, credentials, API keys, or tokens in the diff as **Critical**.
562562
- Silence is better than noise. If you have nothing important to say, say nothing.
563+
- **Do NOT use `#N` notation** (e.g., `#1`, `#2`) in PR comments or summaries — GitHub auto-links these to issues/PRs. Use `(1)`, `[1]`, or descriptive references instead.
563564
- **Match the language of the PR.** Write review comments, findings, and summaries in the same language as the PR title/description/code comments. If the PR is in English, write in English. If in Chinese, write in Chinese. Do NOT switch languages.

0 commit comments

Comments
 (0)