vrr/qwen-code
2
0
Fork 0
mirror of https://github.com/QwenLM/qwen-code.git synced 2026-05-01 21:20:44 +00:00
Code Issues Projects Releases Packages Wiki Activity Actions 10

fix(review): 3 Copilot comments — conditional cleanup, italic format, cache SHA

Browse source 
1. Step 11: conditional worktree removal — skip if Step 8 flagged
   preservation (autofix commit/push failure)
2. Standardize model attribution to _italic_ (was mixed *italic*)
3. Cache stores pre-autofix headRefOid (not worktree HEAD which may
   include the autofix commit)

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
This commit is contained in:
wenshao 2026-04-07 08:58:13 +08:00
parent 226c9e2c19
commit b9d3602309
1 changed files with 6 additions and 4 deletions
Download patch file Download diff file Expand all files Collapse all files

10
packages/core/src/skills/bundled/review/SKILL.md
View file

@ -495,7 +495,7 @@ If reviewing a PR, update the review cache for incremental review support:
2. Write `.qwen/review-cache/pr-<number>.json` with:
```json
{
"lastCommitSha": "<current HEAD SHA>",
"lastCommitSha": "<pre-autofix HEAD SHA captured in Step 1>",
"lastModelId": "{{model}}",
"lastReviewDate": "<ISO timestamp>",
"findingsCount": <number>,
@ -508,10 +508,12 @@ If reviewing a PR, update the review cache for incremental review support:
Remove all temp files (`/tmp/qwen-review-{target}-context.md`, `/tmp/qwen-review-{target}-comment.txt`, `/tmp/qwen-review-{target}-summary.txt`).
If a PR worktree was created in Step 1, remove it and its local ref:
If a PR worktree was created in Step 1, **and Step 8 did NOT instruct to preserve it** (autofix commit/push failure), remove it and its local ref:
1. `git worktree remove .qwen/tmp/review-pr-<number> --force` (the `--force` flag handles cases where autofix left uncommitted changes)
2. `git branch -D qwen-review/pr-<number> 2>/dev/null || true` (clean up the local ref; ignore errors if already deleted)
1. `git worktree remove .qwen/tmp/review-pr-<number> --force`
2. `git branch -D qwen-review/pr-<number> 2>/dev/null || true`
If Step 8 flagged the worktree for preservation (autofix failure), skip worktree removal but still clean up temp files.
This step runs **after** Step 9 and Step 10 to ensure all review outputs are saved before cleanup.