Skip to content

Commit 2c798db

Browse files
committed
ci(workflow): update review workflow to verify latest versions
Signed-off-by: Tommy Nguyen <tuannvm@hotmail.com>
1 parent 584e68c commit 2c798db

File tree

1 file changed

+11
-2
lines changed

1 file changed

+11
-2
lines changed

.github/workflows/cursor.yml

Lines changed: 11 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -51,12 +51,19 @@ jobs:
5151
Objectives:
5252
1) Re-check existing review comments and reply resolved when addressed.
5353
2) Review the current PR diff and flag only clear, high-severity issues.
54-
3) Leave very short inline comments (1-2 sentences) on changed lines only and a brief summary at the end.
54+
3) Verify language and library versions against latest releases using web search.
55+
4) Leave very short inline comments (1-2 sentences) on changed lines only and a brief summary at the end.
5556
5657
Procedure:
5758
- Get existing comments: gh pr view --json comments
5859
- Get diff: gh pr diff
5960
- Get changed files with patches to compute inline positions: gh api repos/${{ github.repository }}/pulls/${{ github.event.pull_request.number }}/files --paginate --jq '.[] | {filename,patch}'
61+
- IMPORTANT: When reviewing files that specify versions (go.mod, package.json, requirements.txt, Dockerfile, GitHub Actions workflows, etc.), MUST search the web for:
62+
- Latest stable versions of languages (Go, Node.js, Python, etc.)
63+
- Latest versions of libraries/dependencies
64+
- Latest versions of base images (Docker)
65+
- Latest versions of GitHub Actions
66+
- Compare found versions with what's in the PR and flag outdated versions
6067
- Compute exact inline anchors for each issue (file path + diff position). Comments MUST be placed inline on the changed line in the diff, not as top-level comments.
6168
- Detect prior top-level "no issues" style comments authored by this bot (match bodies like: "✅ no issues", "No issues found", "LGTM").
6269
- If CURRENT run finds issues and any prior "no issues" comments exist:
@@ -74,14 +81,16 @@ jobs:
7481
- Obvious logic errors with incorrect behavior
7582
- Clear performance anti-patterns with measurable impact
7683
- Definitive security vulnerabilities
84+
- Outdated language/library/dependency/action versions (use web search to verify latest)
85+
- Deprecated APIs or patterns (search documentation for current best practices)
7786
- Avoid duplicates: skip if similar feedback already exists on or near the same lines.
7887
7988
Commenting rules:
8089
- Max 10 inline comments total; prioritize the most critical issues
8190
- One issue per comment; place on the exact changed line
8291
- All issue comments MUST be inline (anchored to a file and line/position in the PR diff)
8392
- Natural tone, specific and actionable; do not mention automated or high-confidence
84-
- Use emojis: 🚨 Critical 🔒 Security ⚡ Performance ⚠️ Logic ✅ Resolved ✨ Improvement
93+
- Use emojis: 🚨 Critical 🔒 Security ⚡ Performance ⚠️ Logic 📦 Outdated Version 🔄 Deprecated ✅ Resolved ✨ Improvement
8594
8695
Submission:
8796
- If there are NO issues to report and an existing top-level comment indicating "no issues" already exists (e.g., "✅ no issues", "No issues found", "LGTM"), do NOT submit another comment. Skip submission to avoid redundancy.

0 commit comments

Comments
 (0)