Skip to content
Open
Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
28 commits
Select commit Hold shift + click to select a range
541408c
feat(review): swap re-review rounds to a fix-audit shape under the cr…
wenshao Aug 26, 2026
b8611dd
Merge branch 'main' into feat/review-critical-round-shape
wenshao Aug 26, 2026
8e1fd85
fix(review): align fix-audit shape readers and key the deferral claim…
qwen-code-dev-bot Aug 26, 2026
bd704ce
Merge branch 'main' into feat/review-critical-round-shape
qwen-code-dev-bot Aug 26, 2026
a20cb48
Merge branch 'main' into feat/review-critical-round-shape
qwen-code-dev-bot Aug 27, 2026
d0b2db7
Merge branch 'main' into feat/review-critical-round-shape
wenshao Aug 27, 2026
774531b
fix(review): plug the fix-audit shape gaps the round findings named (…
qwen-code-dev-bot Aug 27, 2026
f85bf21
Merge branch 'main' into feat/review-critical-round-shape
qwen-code-dev-bot Aug 27, 2026
7e42977
fix(review): align fix-audit prose with the roster and floor code (#1…
qwen-code-dev-bot Aug 27, 2026
72903be
Merge branch 'main' into feat/review-critical-round-shape
qwen-code-dev-bot Aug 28, 2026
ae1cf2f
Merge branch 'main' into feat/review-critical-round-shape
qwen-code-dev-bot Aug 28, 2026
81b1749
Merge remote-tracking branch 'origin/main' into feat/review-critical-…
qwen-code-dev-bot Aug 28, 2026
fe8944c
fix(review): close the round-shape Criticals on the posture machinery…
qwen-code-dev-bot Aug 28, 2026
08bfeca
Merge branch 'main' into feat/review-critical-round-shape
qwen-code-dev-bot Aug 28, 2026
0be9a32
fix(review): close the round-11 Criticals on the seam bound and round…
qwen-code-dev-bot Aug 29, 2026
06b944c
Merge branch 'main' into feat/review-critical-round-shape
wenshao Aug 29, 2026
18ce3b3
Merge branch 'main' into feat/review-critical-round-shape
qwen-code-dev-bot Aug 29, 2026
eb018fb
Merge branch 'main' into feat/review-critical-round-shape
qwen-code-dev-bot Aug 29, 2026
9590c5a
Merge branch 'main' into feat/review-critical-round-shape
qwen-code-dev-bot Aug 30, 2026
8edaa56
Merge branch 'main' into feat/review-critical-round-shape
qwen-code-dev-bot Aug 30, 2026
e9d5a24
Merge branch 'main' into feat/review-critical-round-shape
qwen-code-dev-bot Aug 31, 2026
98f9506
Merge branch 'main' into feat/review-critical-round-shape
qwen-code-dev-bot Aug 31, 2026
9f7fc90
Merge branch 'main' into feat/review-critical-round-shape
wenshao Sep 3, 2026
66bc461
fix(review): lex the seam oracle, state the wave rule, and close the …
wenshao Sep 3, 2026
3e35d01
Merge branch 'main' into feat/review-critical-round-shape
wenshao Sep 3, 2026
b58e4b3
fix(review): read the seam oracle through TypeScript's parser and clo…
wenshao Sep 5, 2026
00380a7
test(review): pin the seam parser's default base order and its older-…
wenshao Sep 5, 2026
6dd917b
Merge branch 'main' into feat/review-critical-round-shape
wenshao Sep 5, 2026
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
2 changes: 2 additions & 0 deletions docs/users/features/code-review.md
Original file line number Diff line number Diff line change
Expand Up @@ -367,6 +367,8 @@ If you switch models (via `/model`) and re-review the same PR, `/review` detects

The model match also gates incremental scoping, not just the skip: "clean up to the cached commit" is the previous model's verdict, so when new commits have landed since the cached review, a model mismatch never scopes to `lastCommitSha..HEAD` — the range is the full diff, noting "Previous round was reviewed by qwen3-coder. Running full review with gpt-4o." — unless an anchor certified by the model now running is recovered from the last posted review (below), which scopes the range instead. The previous round's findings still carry over to be re-ruled; only the anchor does not. The same gate binds the anchor recovered from the last posted review's machine-ledger marker when the cache is absent or its anchor is unusable (CI, another clone): it scopes the incremental range only if the model now running certified it — a marker certified by a different model, or carrying no model (a review posted with `review.attribution` off, or one from before the field), falls back to the full diff. A round that did not close cleanly posts its marker without an anchor (it cannot certify a range), but that loss is not sticky when the round's work list survived whole: recovery grafts the anchor forward from the most recent earlier marker your own account posted with one, so a single non-clean round no longer forces every later round to re-read the full diff — the next round scopes `anchor..HEAD`, which re-covers the range the non-clean round could not certify. A size-capped round's graft is refused (dropped findings would fall outside the grafted scope and retire silently), so later rounds keep re-reading the full diff until a complete marker lands.

**Critical-posture re-reviews run a narrower round (fix-audit shape).** Once a PR's re-review resolves the round-adaptive posting floor to critical-only (from round 6, or earlier when the flat first-time-finding trend latches — see the convergence posture) _and_ a usable incremental anchor exists, the round stops re-running round 1's shape. The capture command predicts the floor from the previous round's posted marker and records `incremental.posture: "critical"` in the plan, and the round then: fans out one territory agent per chunk of the delta (whatever its size — no 3A dimension set, but keeping the issue-fidelity agent: it re-checks head against the issue whatever the delta displays, and is the one auditor that can see a fix commit removing behaviour the issue required when the removal appears on neither side of the narrowed range); republishes each still-clean _interaction_ file seam-bounded — only the hunks displaying a line that imports or uses what changed, with a `seam: {kept, total}` census in the plan and a header-only section when nothing qualifies, so the file still gets an agent briefed to check the seam from the worktree (a file whose full-range slice classifies as heavy is exempt and republishes whole, so its invariant agents still launch; so is every read TypeScript's parser cannot certify — the parser is resolved at run time from the reviewed repository, never bundled into the CLI, and no parser, a syntax error, a computed `require`/`import(` specifier or a required value that escapes into an expression each republish the file in full with no census); and narrows the reverse-audit waves instead of capping them — from wave 3, a chunk holding no delta file leaves the schedule after one substantive dry audit, while delta territories keep the full retirement rules and any non-delta chunk the waves could not certify dry stays in (a stale dry receipt returns it to the ordinary rules), so the late waves (where fix-induced Criticals measurably surface) keep running over a shrinking front. Severities are unchanged — the posture governs what posts, never what is found — and every reduction is disclosed: in the plan, in the agents' briefs, in the round's `posture narrowing:` note, and as a one-sentence "Round shape" disclosure in the posted review body. The plan's posture record is itself an arm of the compose-time floor resolution, so a fix-audit round's posting bar cannot disagree with the shape it ran (sub-Critical findings defer even if the floor's usual inputs are unavailable at compose time). An explicit `--severity-floor suggestion` turns the shape off along with the posture (and outranks a stale plan record at posting time); an explicit `--severity-floor critical` enables it from the first anchored re-review.

Cache is stored in `.qwen/review-cache/` and tracks both the commit SHA and model ID. Make sure this directory is in your `.gitignore` (a broader rule like `.qwen/*` also works). On GitHub, if the cached commit was rebased or force-pushed away, it falls back to a full review; Aone rules the cached anchor differently — see its paragraph below. Only high-effort reviews consult or write the cache — a `--effort low|medium` quick pass never counts as "already reviewed".

## Review Reports
Expand Down
Loading
Loading