Repository navigation
feat(review): record per-file content verdicts that survive a rebase #9661
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Merged
Merged
Changes from 7 commits
Commits
Show all changes
98 commits
Select commit
Hold shift + click to select a range
dff2b78
feat(review): content-anchored incremental rounds for the local revie…
wenshao 8c1330a
feat(review): transfer per-file content verdicts across rebases
wenshao ee54f8b
fix(review): close nine Criticals, most of them my own last round's
wenshao b3cc1ca
fix(review): carry `source` through promotion, and sweep C1 too
wenshao 0810d06
fix(review): close round 2 — the cache rename's fallout, and the stop…
wenshao 53bfe15
fix(review): give inertText the full class its own rule sweeps
wenshao 91d93cb
fix(review): close round 3 — the cache records the gate's identity, a…
github-actions[bot] 87b769b
fix(review): close round 4 — the named-path flow's six dead ends
github-actions[bot] f0bdf53
fix(review): one control class, used by both the test and the escape
wenshao c661806
fix(review): close round 5 — including a gate of mine that could neve…
wenshao 357db85
merge: keep both path tests, and the completed fail-closed list
wenshao feb55b5
fix(review): close round 6 — the five Criticals the re-read of HEAD s…
github-actions[bot] b295232
fix(review): close round 7 — and stop enumerating the fail-closed list
wenshao ecf5fea
merge: keep both sets of SKILL guards
wenshao 2d7d49a
fix(review): record the rendering the reviewer gets, not the tree's
wenshao a2d46ec
merge: carry the positive fail-closed rule into the cache-commit para…
wenshao af708af
fix(review): close round 8 — the four shapes the last fixes left open
wenshao 575d136
fix(review): close round 9 — date a stop's blockers; key the local ca…
wenshao 8192762
fix(review): close round 10 — date each stop blocker against its own …
wenshao 9b89827
merge: take the base's one-definition fail-closed rule, drop my copy …
wenshao 518080c
fix(review): make the attrs component work, and stop the allowlist dr…
wenshao df533ff
fix(review): close round 11 — date blockers on bytes; certify vanishe…
wenshao 3b916d6
fix(review): close round 12 — attribute landed fixes, date by regime,…
github-actions[bot] 4142065
Merge origin/main into review-incremental-anchor
wenshao 39ca812
Merge review/local-content-anchor into review/rebase-surviving-verdicts
wenshao e6aa683
fix(review): close round 13 — pay the isolation test's import cost at…
github-actions[bot] 99fff42
Merge branch 'review/local-content-anchor' of https://github.com/Qwen…
qwen-code-dev-bot 1f0fb94
refactor(review): drop the stop-round blocker-dating chain
wenshao 7774eb3
Merge remote round-12/13 autofix commits, keeping the chain deleted
wenshao 6ae23b4
Merge review/local-content-anchor: blocker-dating chain removed
wenshao b13eb13
Merge remote-tracking branch 'origin/main' into p1
wenshao d3a2331
Merge remote-tracking branch 'origin/review/local-content-anchor' int…
wenshao 4a24ca5
fix(review): close round 14 — certify what git diff cannot see, split…
github-actions[bot] 5dba419
fix(review): close round 15 — withhold the candidate under visibility…
github-actions[bot] dc74e83
fix(review): close round 15 review — bracket the visibility oracle, g…
wenshao f49f82e
Merge review/local-content-anchor: round-15 review fixes
wenshao c723d5f
fix(review): close round 16 — withhold the incremental stops under --…
github-actions[bot] 9273aa1
Merge remote-tracking branch 'origin/review/local-content-anchor' int…
wenshao af521c5
fix(review): gate decided stops on a composed re-rule verdict (#9941)
wenshao 0d11241
Revert "fix(review): gate decided stops on a composed re-rule verdict…
wenshao 2f050a6
Merge remote-tracking branch 'origin/main' into p1
wenshao ec0d937
Merge remote-tracking branch 'origin/review/local-content-anchor' int…
wenshao 0cf1b07
fix(cli): drop duplicate getCurrentCustomTitle key that breaks npm ci
wenshao 2ba8fe7
Merge remote-tracking branch 'origin/review/local-content-anchor' int…
wenshao 8b81762
fix(cli): drop the second duplicate key too (getSessionDisplayName)
wenshao a7b28bb
Merge remote-tracking branch 'origin/review/local-content-anchor' int…
wenshao 2e03ecd
Merge branch 'main' into review/local-content-anchor
qwen-code-dev-bot 9443811
test(review): pin the visibility oracle's sample count
wenshao d04a69d
Merge remote-tracking branch 'origin/review/local-content-anchor' int…
wenshao 49c2db3
Merge remote-tracking branch 'origin/review/local-content-anchor' int…
wenshao 6abc9af
Merge remote-tracking branch 'origin/review/local-content-anchor' int…
wenshao 79dfd58
fix(review): close round 17 — restore the merged-away fake, publish t…
wenshao 09403b3
Merge review/local-content-anchor: round-17 fixes
wenshao eef3c34
fix(review): close round 18 — ask git for the cone, make the cap idem…
wenshao e8d8ade
Merge remote-tracking branch 'origin/review/local-content-anchor' int…
wenshao 0f70270
fix(review): close round 19 — only ENOENT is absence, undecodable is …
wenshao 729f57f
Merge remote-tracking branch 'origin/review/local-content-anchor' int…
wenshao 518d26f
fix(review): close round 20 — bool-typed fileMode, no directory in th…
wenshao 6fdbac2
Merge review/local-content-anchor: round-20 fixes
wenshao 146ab25
fix(review): close round 21 — measure the pointer, ride the skipped l…
wenshao ac97e65
Merge remote-tracking branch 'origin/review/local-content-anchor' int…
wenshao 60bb3ce
fix(review): close round 22 — a gitlink's identity is real, on both s…
wenshao 45be1ac
Merge remote-tracking branch 'origin/review/local-content-anchor' int…
wenshao e94892e
Merge remote-tracking branch 'origin/main' into p1
wenshao f1d9c62
fix(review): close round 23 — withhold over the dropped-out set, excl…
wenshao af381ed
Merge remote-tracking branch 'origin/review/local-content-anchor' int…
wenshao 3847817
test(review): adapt the withheld-candidate pin to this branch's contract
wenshao f425a8c
fix(review): keep this run's stop sidecar across its own cleanup
wenshao e9a7dfd
Merge remote-tracking branch 'origin/review/local-content-anchor' int…
wenshao 917e1d4
Merge remote-tracking branch 'origin/main' into p1
wenshao 3e3c800
Merge remote-tracking branch 'origin/review/local-content-anchor' int…
wenshao 348528b
Merge main after #9659's squash landed
wenshao 51fa283
Merge remote-tracking branch 'origin/main' into review/rebase-survivi…
wenshao 7f27178
Merge branch 'main' into review/rebase-surviving-verdicts
wenshao a47be01
Merge branch 'main' into review/rebase-surviving-verdicts
wenshao 95efd93
Merge branch 'main' into review/rebase-surviving-verdicts
wenshao f9342ff
fix(review): keep unlinkSync imported after merging main
wenshao 5267817
fix(review): drop two eslint-disable directives the rules no longer flag
wenshao dad260d
fix(review): sweep the whole promoted cache for control characters
wenshao 1918ddf
fix(review): never transfer a verdict over an UNHASHABLE rendering
wenshao e3391f3
fix(review): guard the withheld candidate's removal like its write
wenshao 73f7929
fix(review): say a failed blob-pair listing out loud in fetch-pr
wenshao b961b4d
docs(review): scope the candidate stateId check to local rounds
wenshao 3653b52
docs(review): state the verdict identity's rendering contract
wenshao c652a92
Merge branch 'main' into review/rebase-surviving-verdicts
wenshao 77b1acf
fix(review): refuse a redirected .qwen/tmp at a round's first write
wenshao 56357a7
fix(review): withhold the cache candidate when no identity is published
wenshao c339154
feat(review): bind cache-commit's promotion to the plan's stateId
wenshao 4336524
docs(review): close the same-target concurrency window in Step 8
wenshao 7a6a4b9
Merge branch 'main' into review/rebase-surviving-verdicts
wenshao c5e757a
fix(review): import atomicWriteFileSync from its core module in cache…
wenshao d1cc8bf
fix(review): run fetch-pr's scratch guard after the launch-directory …
wenshao 49e898a
fix(review): print the scratch-directory refusal under one command pr…
wenshao ad90c4d
test(review): reconcile the guard's tests with what main added
wenshao 1b10d44
Merge remote-tracking branch 'origin/main' into review/rebase-survivi…
2e674bd
fix(review): refuse a round-owned symlink under .qwen/tmp on disk, tr…
2467d8e
fix(review): refuse to resume a round under another model identity
f4468ad
fix(review): persist the local findings ledger on a fresh checkout an…
File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Oops, something went wrong.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Uh oh!
There was an error while loading. Please reload this page.