@qwen-code/qwen-code 0.21.14-preview.0 → 0.21.14

package/chunks/{chunk-KSGWAYNE.js → chunk-XYRBK56E.js}

@@ -2,13 +2,13 @@
 "use strict";
 import {
   listCommand
-} from "./chunk-K3CECR5C.js";
+} from "./chunk-5IJSFWQO.js";
 import {
   getErrorMessage as getErrorMessage2
-} from "./chunk-LRDB2OLB.js";
+} from "./chunk-XITLNXLX.js";
 import {
   serveCommand
-} from "./chunk-RAQBBGS6.js";
+} from "./chunk-Y7QY6DKW.js";
 import {
   OBSERVED_CONTACT_MAX_FRESH_WITHIN_SECONDS,
   ObservedChannelContactStore
@@ -35,7 +35,7 @@ import {
   collectProviderModelsForProtocol,
   getAuthTypeFromEnv,
   resolveCliGenerationConfig
-} from "./chunk-ISRHAMNL.js";
+} from "./chunk-EOPBFLA5.js";
 import {
   resolveSkillSettings
 } from "./chunk-Z4X5MTRT.js";
@@ -55,14 +55,14 @@ import {
   registerToolCallDispatch,
   selectFirstModel,
   sessionsPath
-} from "./chunk-2JXDFSVE.js";
+} from "./chunk-55VKMB7X.js";
 import {
   getExtensionManager,
   requestChoicePluginNonInteractive,
   requestConsentNonInteractive,
   requestConsentOrFail,
   resolveExtensionCommandScope
-} from "./chunk-HGPMOXLZ.js";
+} from "./chunk-U657UZJN.js";
 import {
   parseChannelConfig
 } from "./chunk-NRPXRNN4.js";
@@ -135,33 +135,33 @@ import {
   assembleMcpServers,
   getPendingGatedMcpServers,
   mcpCommand
-} from "./chunk-QECSDJET.js";
+} from "./chunk-S7N3HUIB.js";
 import {
   loadSettings
-} from "./chunk-OLZSWUZR.js";
+} from "./chunk-VFP5YOHJ.js";
 import {
   detectSystemLanguage,
   getCurrentLanguage,
   initializeI18n,
   resolveLanguageSetting,
   t
-} from "./chunk-IOTPI425.js";
+} from "./chunk-VTOLYFWM.js";
 import {
   loadEnvironment
-} from "./chunk-SGLPDKRT.js";
+} from "./chunk-MVHJZ3FZ.js";
 import {
   resolveAcpChannelFallback,
   scrubAndReportInheritedLoaderEnv
 } from "./chunk-CXJHQVEK.js";
 import {
   getCliVersion
-} from "./chunk-6DFPR7CS.js";
+} from "./chunk-PCETXVVV.js";
 import {
   getPackageJson
 } from "./chunk-WZMH7II3.js";
 import {
   isWorkspaceTrusted
-} from "./chunk-YXC7JYUI.js";
+} from "./chunk-C3T6M24E.js";
 import {
   require_strip_json_comments
 } from "./chunk-NPQ5ZKRP.js";
@@ -203,7 +203,7 @@ import {
   unquoteCStylePath,
   updateChannelMemoryEntry,
   updateSetting
-} from "./chunk-XC377TLN.js";
+} from "./chunk-OFTPCOGR.js";
 import {
   parsePositiveIntegerEnv
 } from "./chunk-6PVPNMXU.js";
@@ -13226,7 +13226,7 @@ You are undirected on purpose. Do not restrict yourself to the list.`
 Read the JSON it prints:
 
 - \`toolchain: "npm"\` \u2192 use its \`build[]\` / \`test[]\` results. A failure in a file **the diff changed** is a **Critical** (\`Source: [build]\` or \`[test]\`); a failure in a file it did **not** touch is pre-existing \u2014 say so, do not file it against this PR. A non-empty \`timedOut\`, or a failed \`install\`, is environment/infrastructure \u2014 informational, never a Critical. On \`ok: true\`, name the workspaces built and the commands run; a return that names no command is a whiff. Report the TEST coverage from \`testScope\`, never from assumption. \`testScope.workspaces\` lists exactly the suites that ran \u2014 say "tests scoped to <list> \u2014 the changed workspaces and their declared dependents that define a test script". \`testScope.notRun\`, when present, names suites the whole-call budget stopped before they ran \u2014 say they did not run, never fold them into the coverage. When \`testScope.caveat\` is present, the scope may be incomplete \u2014 quote the caveat and say exactly that. A green run is a claim about those suites only \u2014 do not phrase it as the whole suite passing.
-- **A suite left unrun is not a suite that passed \u2014 continue the run.** \`testScope.notRun\` names suites the whole-call budget could not reach, and a \`test[]\` entry with \`"clamped": true\` is a suite the budget started too late and killed (its deadline was shortened, so its timeout says nothing about the suite). A third shape carries no field at all \u2014 a single-package repo whose budget ran out before its one suite has an empty \`test[]\` and no \`testScope\`, and only the \`note\` says so; read it before calling the dimension finished. That third shape cannot be continued \u2014 a continuation has no recorded scope to read, and answers "ended before its test phase" without running anything \u2014 so report the dimension UNFINISHED and do not spend a continuation on it. The first two mean the dimension is unfinished AND continuable: re-run the SAME \`build-test\` command with \`--resume\` \u2014 it skips install and build, runs only what is left, and merges into the same report file. The ${SHELL_TOOL_MAX_TIMEOUT_MS / 1e3}-second ceiling is per CALL, so this is the only way a repo whose suites do not fit one call ever finishes them (measured on this repo: \`packages/cli\` alone needs 401s, and install + builds + \`packages/core\` had already spent 285s). Keep resuming while work is left, up to ${MAX_RESUME_CALLS} continuations; then report what the run has, with \`notRun\` disclosed.
+- **A suite left unrun is not a suite that passed \u2014 continue the run.** \`testScope.notRun\` names suites the whole-call budget could not reach, and a \`test[]\` entry with \`"clamped": true\` is a suite the budget started too late and killed (its deadline was shortened, so its timeout says nothing about the suite). A third shape ends before any suite \u2014 a single-package repo whose budget ran out before its one suite has an empty \`test[]\`, no \`testScope\`, and \`"endedBeforeTests": true\` (the report's own stamp), with the \`note\` naming the unrun suite; read them before calling the dimension finished. That third shape cannot be continued \u2014 a continuation has no recorded scope to read, and a \`--resume\` on it answers "ended before its test phase" and points at a fresh run \u2014 so report the dimension UNFINISHED and do not spend a continuation on it. The first two mean the dimension is unfinished AND continuable: re-run the SAME \`build-test\` command with \`--resume\` \u2014 it skips install and build, runs only what is left, and merges into the same report file. The ${SHELL_TOOL_MAX_TIMEOUT_MS / 1e3}-second ceiling is per CALL, so this is the only way a repo whose suites do not fit one call ever finishes them (measured on this repo: \`packages/cli\` alone needs 401s, and install + builds + \`packages/core\` had already spent 285s). Keep resuming while work is left, up to ${MAX_RESUME_CALLS} continuations; then report what the run has, with \`notRun\` disclosed.
 - **When any \`test[]\` command failed (exit non-zero, not a timeout), MEASURE which failures are the PR's before ruling by path.** The path rule above misclassifies in both directions \u2014 an environment-flaky test in a touched file gets filed as a Critical it did not cause, and a PR that breaks a test in an UNTOUCHED file gets waved through as pre-existing. The measurement is two commands: \`qwen review base-tree --plan <plan> --worktree <worktree> --out <plan dir>/qwen-review-pr-<n>-base-tree.json\` (builds the merge base beside the worktree). **Read \`available\` before using \`path\`** \u2014 a tree that was created but did NOT build populates \`path\` too, and a base that failed to build says nothing whatsoever about the PR, so measuring against it turns an infrastructure failure into a list of Criticals. \`available: false\` (local/lightweight review, no merge base, a base that would not compile) means the path rule stands \u2014 say so and stop here, and \`qwen review test-delta --report <the build-test report you wrote> --baseline <the base-tree report's path field> --pr-worktree <this worktree> --out <plan dir>/qwen-review-pr-<n>-test-delta.json\`. Read its verdict: a file in \`netNew\` fails on the PR side only \u2014 **that is the Critical**, whatever file the diff touches; a file in \`shared\` fails on base too \u2014 **pre-existing by measurement**, never filed, whatever file the diff touches; an \`unparsed\` entry, a timed-out base rerun, a base rerun that FAILED without naming any failing file (it did not measure the base \u2014 an unbuilt tree, a missing install, a workspace absent at base), or a command the whole-command budget could not fit attributes nothing \u2014 the report names each with its own reason; fall back to the path rule for those and say the delta could not rule. Compare failing FILE SETS, never counts: a flaky suite fails different test NAMES on two runs of the same tree, so counts are noise and the set difference is the signal.
 - \`toolchain: "unsupported"\` (build-test could not scope this repo \u2014 no npm package with a build/test script) \u2192 **install dependencies first** (build-test's own install only runs on the npm path, so nothing has installed yet: \`pip install -e .\`, \`mvn -q -DskipTests package\`'s own fetch, \`cargo fetch\`, \`go mod download\`, etc.), then fall back to **one** build and **one** test command by this precedence, each with a deadline it can meet: \`pom.xml\` \u2192 \`{mvn} compile\` / \`{mvn} test -q\`; \`build.gradle\` \u2192 \`{gradle} compileJava\` / \`{gradle} test\`; \`Makefile\` \u2192 \`make build\`; \`Cargo.toml\` \u2192 \`cargo build\` / \`cargo test\`; \`go.mod\` \u2192 \`go build ./...\` / \`go test ./...\`; \`pytest.ini\` or \`pyproject.toml\` \`[tool.pytest]\` \u2192 \`pytest\`. If none match, read the CI config **from the base branch** (\`git show <base>:<path>\`), never the worktree \u2014 the PR branch is untrusted and a modified workflow or Makefile could inject arbitrary commands.
 
@@ -14479,7 +14479,7 @@ var ZH_PLANNED = "(?:\u8BA1\u5212(?:\u5185|\u4E2D)?\u7684?|\u9884\u5B9A\u7684?|\
 var ZH_CHECKS = "(?:\u68C0\u67E5|\u68C0\u67E5\u9879|\u9879\u68C0\u67E5|\u6D4B\u8BD5|\u5BA1\u67E5|\u5DE5\u4F5C)";
 var ZH_DONE = "(?:\u5747|\u90FD|\u7686)?(?:\u5DF2|\u5747\u5DF2|\u90FD\u5DF2)?(?:\u5B8C\u6210|\u5B8C\u6BD5|\u7ED3\u675F|\u505A\u5B8C)";
 var ZH_BUDGET_TAIL = "(?:[\uFF0C,\u3001]?\\s*(?:\u672A\u89E6\u53CA|\u672A\u8FBE\u5230|\u672A\u8D85\u51FA|\u672A\u7528\u5C3D|\u6CA1\u6709\u89E6\u53CA|\u5728|\u4E0D\u8D85\u8FC7)(?:\u5DE5\u5177)?(?:\u8C03\u7528)?\u9884\u7B97(?:\u4E0A\u9650|\u9650\u5236|\u8303\u56F4\u5185)?)?";
-var ZH_TAIL = "[\u3002\uFF0E.!\uFF01\u2026,\uFF0C;\uFF1B:\uFF1A\\s]*$";
+var ZH_TAIL = "[\u3002\uFF0E.!\uFF01\u2026,\uFF0C;\uFF1B:\uFF1A\uFF1F\u3001\\s]*$";
 var ZH_COMPLETION_CLAUSE = `${ZH_ALL}?${ZH_PLANNED}?${ZH_CHECKS}?${ZH_DONE}${ZH_BUDGET_TAIL}`;
 var ZH_PLACEHOLDER = `${ZH_TOKEN}(?:${ZH_TAIL}|\\s*(?:${ZH_SEPARATOR})\\s*${ZH_COMPLETION_CLAUSE}${ZH_TAIL})`;
 var PLACEHOLDER_GAP_RE = new RegExp(
@@ -14515,7 +14515,7 @@ function stripWrappers(s) {
   return out;
 }
 __name(stripWrappers, "stripWrappers");
-var TRAILING_GAP_CHAR_RE = /[.!…,;:\s]/;
+var TRAILING_GAP_CHAR_RE = /[.!…,;:\s。，；：！？、．]/;
 function stripTrailingGapChars(s) {
   let end = s.length;
   while (end > 0 && TRAILING_GAP_CHAR_RE.test(s.charAt(end - 1))) end--;
@@ -14549,11 +14549,13 @@ function budgetGapDisclosures(finalText) {
     }
     raw = stripWrappers(raw.trim()).trim();
     const normalized = stripTrailingGapChars(raw).trim();
-    const unparenthesized = normalized.startsWith("(") && normalized.endsWith(")") ? normalized.slice(1, -1).trim() : normalized;
+    const unparenthesized = stripTrailingGapChars(
+      normalized.startsWith("(") && normalized.endsWith(")") || normalized.startsWith("\uFF08") && normalized.endsWith("\uFF09") ? normalized.slice(1, -1).trim() : normalized
+    );
     if (normalized.length === 0 || PLACEHOLDER_GAP_RE.test(unparenthesized)) {
       continue;
     }
-    const key = stripTrailingGapChars(unparenthesized).toLowerCase();
+    const key = unparenthesized.toLowerCase();
     if (seen.has(key)) continue;
     seen.add(key);
     gaps.push(truncateGap(raw));
@@ -14920,7 +14922,7 @@ function budgetStopDisclosure(round) {
     subject: "reverse audit",
     reason: `stopped before ${which} by the ${BUDGET_STOP_PHRASE}`,
     subjectZh: "\u53CD\u5411\u5BA1\u8BA1",
-    reasonZh: `\u8BC4\u5BA1\u65F6\u95F4\u9884\u7B97\u4E0D\u8DB3\uFF0C\u672A\u80FD\u5F00\u59CB${whichZh}`
+    reasonZh: `${BUDGET_STOP_PHRASE_ZH}\u4E0D\u8DB3\uFF0C\u672A\u80FD\u5F00\u59CB${whichZh}`
   };
 }
 __name(budgetStopDisclosure, "budgetStopDisclosure");
@@ -14941,7 +14943,7 @@ function roundCapStopDisclosure(cap) {
     subject: "reverse audit",
     reason: `did not converge within the ${ROUND_CAP_PHRASE} of ${cap}`,
     subjectZh: "\u53CD\u5411\u5BA1\u8BA1",
-    reasonZh: `\u5728 ${cap} \u8F6E\u7684\u53CD\u5BA1\u8F6E\u6570\u4E0A\u9650\u5185\u672A\u6536\u655B`
+    reasonZh: `\u5728 ${cap} \u8F6E\u7684${ROUND_CAP_PHRASE_ZH}\u5185\u672A\u6536\u655B`
   };
 }
 __name(roundCapStopDisclosure, "roundCapStopDisclosure");
@@ -17901,11 +17903,23 @@ function floorEnforcedReroute(severityFloor, contextUnavailable, prevRound, draf
   return { indices, entries };
 }
 __name(floorEnforcedReroute, "floorEnforcedReroute");
+function canonicalDimensionHead(s) {
+  return s.toLowerCase().replace(/&/g, " and ").replace(/[-\s]+/g, "-").replace(/^the-/, "").replace(/-(?:check|verification)$/, "");
+}
+__name(canonicalDimensionHead, "canonicalDimensionHead");
+function squashedDimensionHead(s) {
+  return canonicalDimensionHead(s).replace(/-/g, "");
+}
+__name(squashedDimensionHead, "squashedDimensionHead");
+var NON_DIFF_DIMENSION_HEADS = new Set(
+  Object.values(BRIEFS).filter((b) => !b.readsDiff).map((b) => canonicalDimensionHead(b.publicLabel))
+);
+var NON_DIFF_DIMENSION_HEADS_SQUASHED = new Set(
+  [...NON_DIFF_DIMENSION_HEADS].map((h) => h.replace(/-/g, ""))
+);
 function isNonDiffDimensionGap(entry) {
-  const head = entry.split(/[—–-]{1,2}\s/)[0].trim().toLowerCase();
-  return /^(?:the\s+)?build[-\s]?(?:and|&)[-\s]?test(?:\s+check|\s+verification)?$/.test(
-    head
-  );
+  const head = entry.split(/[—–-]{1,2}\s/)[0].trim();
+  return NON_DIFF_DIMENSION_HEADS.has(canonicalDimensionHead(head)) || NON_DIFF_DIMENSION_HEADS_SQUASHED.has(squashedDimensionHead(head));
 }
 __name(isNonDiffDimensionGap, "isNonDiffDimensionGap");
 var UNVERIFIED_FINDING_TAG_RE = /—\s*\[unverified\]/gi;
@@ -18240,9 +18254,9 @@ function composeReviewBody(input, cliVersion, attribution, prevRound, reroute =
         budgetStopEntryZh(stop.round ?? void 0)
       ]);
       const isRoundCap = stop.cause === "round-cap";
-      const phrases = isRoundCap ? [ROUND_CAP_PHRASE, ROUND_CAP_PHRASE_ZH] : [BUDGET_STOP_PHRASE, BUDGET_STOP_PHRASE_ZH];
+      const entries = [...canonicalStopEntries];
       for (let i = unreviewed.length - 1; i >= 0; i--) {
-        if (phrases.some((ph) => unreviewed[i].includes(ph))) {
+        if (entries.some((c2) => unreviewed[i].includes(c2))) {
           splicedForBudgetPhrase.push(unreviewed[i]);
           unreviewed.splice(i, 1);
         }
@@ -18887,7 +18901,7 @@ ${footer}`;
   ] : [];
   const deferredShown = deferredSuggestions.slice(0, MAX_DEFERRED_SUGGESTION_LINES).map(renderDeferredEntry).map(boundDeferredLine);
   const deferredMore = deferredSuggestions.length - deferredShown.length;
-  const deferredRound = deferredSuggestions.length ? prevRound + 1 : 0;
+  const deferredRound = deferredSuggestions.length ? Math.min(prevRound + 1, LEDGER_MAX_ROUND) : 0;
   const unlicensedDeferralBlock = unlicensedDeferral === null ? [] : [
     {
       keep: 1,
@@ -22277,7 +22291,9 @@ function recoverLedger(reviews, login) {
   }
   if (!best) return { recovered: null, sawOwnReview };
   let ledger = best.foreign ? stripAnchor(best.ledger) : best.ledger;
-  if (best.foreign && bestOwn) {
+  let mergedOverOwn = false;
+  if (best.foreign && bestOwn && bestOwn.ledger.findings.length > 0) {
+    mergedOverOwn = true;
     const ownIds = new Set(bestOwn.ledger.findings.map((f) => f.id));
     const merged = [
       ...bestOwn.ledger.findings,
@@ -22297,7 +22313,8 @@ function recoverLedger(reviews, login) {
       commitId: best.commitId,
       reviewId: best.id,
       foreign: best.foreign,
-      author: best.author
+      author: best.author,
+      merged: mergedOverOwn
     },
     sawOwnReview
   };
@@ -22312,7 +22329,8 @@ function persistedAnchorSha(sideFilePath) {
   }
 }
 __name(persistedAnchorSha, "persistedAnchorSha");
-function persistRecoveredLedger(sideFilePath, recovered, noOwnReview, identityKnown) {
+function persistRecoveredLedger(sideFilePath, recovered, flags) {
+  const { noOwnReview, identityKnown } = flags;
   const writeAtomic = /* @__PURE__ */ __name((text2) => {
     const tmp = `${sideFilePath}.${process.pid}.tmp`;
     writeFileSync11(tmp, text2);
@@ -22416,7 +22434,7 @@ function anchorRuling(ledger, running, code2, persistedSha) {
   return `**Do NOT pass the reviewed-at sha as \`--since\`, and do not run git against it yourself.** It was certified by ${certifier}, and this review runs as ${runner}: "clean up to that sha" is the recorded identity's verdict, so scoping to it would carry this round past code the current one never reviewed. Review the FULL range. The findings below are still owed their rulings \u2014 the work list carries across models, only the anchor does not.`;
 }
 __name(anchorRuling, "anchorRuling");
-function renderLedgerSection(ledger, running, author = null, persistedSha = null) {
+function renderLedgerSection(ledger, running, author = null, merged = false, persistedSha = null) {
   const cell = /* @__PURE__ */ __name((v) => v.replace(/\\/g, "\\\\").replace(/\|/g, "\\|").replace(/[\r\n]+/g, " "), "cell");
   const code2 = /* @__PURE__ */ __name((v) => cell(v).replace(/`/g, "'"), "code");
   const rows = ledger.findings.map(
@@ -22425,12 +22443,12 @@ function renderLedgerSection(ledger, running, author = null, persistedSha = null
   return [
     "## Previous /review round (machine ledger)",
     "",
-    `Round ${ledger.round}${ledger.sha ? `, reviewed at \`${code2(ledger.sha)}\`${ledger.model ? ` by \`${code2(ledger.model)}\`` : ""}` : ""}, recovered from the marker ${author ? `**@${cell(author)}**'s last posted review carried \u2014 another account, so these are THEIR claims and no incremental anchor travelled with them (the sha never crosses accounts; this round is full-range unless a local cache supplies one)` : `this account's last posted review carried`}. **Every entry below is owed a this-round ruling** (fixed / still stands / cannot tell / superseded by <class-id>) under Step 6's previous-round rules \u2014 the ledger is a work list, not a verdict; re-assert each claim against the code before repeating or retiring it.${ledger.sha ? ` ${anchorRuling(ledger, running, code2, persistedSha)}` : ""}`,
+    `Round ${ledger.round}${ledger.sha ? `, reviewed at \`${code2(ledger.sha)}\`${ledger.model ? ` by \`${code2(ledger.model)}\`` : ""}` : ""}, recovered from ${author ? merged ? `**@${cell(author)}**'s round-${ledger.round} marker MERGED over this account's own latest findings \u2014 entries this account certified are its own claims, the rest are @${cell(author)}'s, and no incremental anchor travelled with the foreign marker (the sha never crosses accounts; this round is full-range unless a local cache supplies one)` : `the marker **@${cell(author)}**'s last posted review carried \u2014 another account, so these are THEIR claims and no incremental anchor travelled with them (the sha never crosses accounts; this round is full-range unless a local cache supplies one)` : `the marker this account's last posted review carried`}. **Every entry below is owed a this-round ruling** (fixed / still stands / cannot tell / superseded by <class-id>) under Step 6's previous-round rules \u2014 the ledger is a work list, not a verdict; re-assert each claim against the code before repeating or retiring it.${ledger.sha ? ` ${anchorRuling(ledger, running, code2, persistedSha)}` : ""}`,
     // A truncated ledger must not read like a complete one. `dropped` exists
     // to draw that line, and this is the only place a reader sees the list.
     ...ledger.dropped ? [
       "",
-      `**This list is PARTIAL**: ${ledger.dropped} further finding(s) from round ${ledger.round} did not fit the marker's size cap and are not here. Absence below is not evidence a finding was fixed \u2014 say so rather than reporting the missing ones as retired.`
+      merged ? `**This list is PARTIAL**: ${ledger.dropped} further finding(s) did not survive into this merged list \u2014 lost to a source marker's size cap or to the merge's own re-cap, and not attributable to any single round's marker. Absence below is not evidence a finding was fixed \u2014 say so rather than reporting the missing ones as retired.` : `**This list is PARTIAL**: ${ledger.dropped} further finding(s) from round ${ledger.round} did not fit the marker's size cap and are not here. Absence below is not evidence a finding was fixed \u2014 say so rather than reporting the missing ones as retired.`
     ] : [],
     "",
     "| id | severity | location | title |",
@@ -22440,7 +22458,7 @@ function renderLedgerSection(ledger, running, author = null, persistedSha = null
   ].join("\n");
 }
 __name(renderLedgerSection, "renderLedgerSection");
-function buildMarkdown(prNumber, ownerRepo, meta, inline2, issue, reviews, prevLedger = null, me = "", prevLedgerAuthor = null, host, persistedSha = null) {
+function buildMarkdown(prNumber, ownerRepo, meta, inline2, issue, reviews, prevLedger = null, me = "", prevLedgerAuthor = null, prevLedgerMerged = false, host, persistedSha = null) {
   const {
     openRoots,
     openBlockerRoots,
@@ -22485,6 +22503,7 @@ function buildMarkdown(prNumber, ownerRepo, meta, inline2, issue, reviews, prevL
         prevLedger,
         roundModelIdFrom(process.env),
         prevLedgerAuthor,
+        prevLedgerMerged,
         persistedSha
       )
     );
@@ -22640,23 +22659,27 @@ async function runPrContext(args) {
   }
   const prevLedger = prevRecovered?.ledger ?? null;
   const prevLedgerAuthor = prevRecovered?.foreign ? prevRecovered.author ?? null : null;
+  const prevLedgerMerged = prevRecovered?.merged ?? false;
   persistRecoveredLedger(
     join17(dirname12(out), `qwen-review-pr-${prNumber}-prev-ledger.json`),
     prevRecovered,
-    // Deletion is licensed ONLY by proof of true absence: a CONFIRMED
-    // identity, and a non-empty list this run walked in which no submitted
-    // review by that identity exists. An empty `reviews` may be an error
-    // envelope ghApiAll flattened to []; an own review whose marker fails to
-    // parse is a persistent state, not absence; and a failed identity lookup
-    // proves nothing about anyone — all take the conservative strip path.
-    // (A recovered foreign ledger also protects the file, but through the
-    // helper's own recovered-first branch, not through this flag.)
-    reviews.length > 0 && identityKnown && !recoveryThrew && !sawOwnReview,
-    // Separately from deletion: an ANONYMOUS recovery (identity unknown)
-    // must not replace the persisted work list — the helper's fourth
-    // outcome. Every marker walks as foreign without a `me`, so the union
-    // never protected the own list this run.
-    identityKnown
+    {
+      // Deletion is licensed ONLY by proof of true absence: a CONFIRMED
+      // identity, and a non-empty list this run walked in which no submitted
+      // review by that identity exists. An empty `reviews` may be an error
+      // envelope ghApiAll flattened to []; an own review whose marker fails
+      // to parse is a persistent state, not absence; and a failed identity
+      // lookup proves nothing about anyone — all take the conservative strip
+      // path. (A recovered foreign ledger also protects the file, but
+      // through the helper's own recovered-first branch, not through this
+      // flag.)
+      noOwnReview: reviews.length > 0 && identityKnown && !recoveryThrew && !sawOwnReview,
+      // Separately from deletion: an ANONYMOUS recovery (identity unknown)
+      // must not replace the persisted work list — the helper's fourth
+      // outcome. Every marker walks as foreign without a `me`, so the union
+      // never protected the own list this run.
+      identityKnown
+    }
   );
   const persistedSha = persistedAnchorSha(
     join17(dirname12(out), `qwen-review-pr-${prNumber}-prev-ledger.json`)
@@ -22673,6 +22696,7 @@ async function runPrContext(args) {
     prevLedger,
     me,
     prevLedgerAuthor,
+    prevLedgerMerged,
     bakeHost,
     persistedSha
   );
@@ -25348,7 +25372,7 @@ function buildRoleBrief(report, role, opts = {}) {
         `  --out ${resolve18(dirname18(opts.planPath), outName)}`,
         "```",
         "",
-        `**If the report says work is left, run it again with \`--resume\`.** The ${SHELL_TOOL_MAX_TIMEOUT_MS / 1e3}-second ceiling is per CALL, not per run: this repo needs more than one call to finish its suites (install, the builds, then \`packages/core\` at 106s and \`packages/cli\` at 401s, before the rest). Work is left when \`testScope.notRun\` is non-empty, or when any \`test[]\` entry has \`"clamped": true\` \u2014 a suite the budget started too late and killed, which says nothing about the suite. A third shape carries no field at all: a single-package repo whose budget ran out before its one suite has an empty \`test[]\` and no \`testScope\`, and only its \`note\` says so \u2014 read the note before calling the dimension finished. That shape cannot be continued (a continuation has no recorded scope to read, and answers "ended before its test phase" without running anything): report the dimension UNFINISHED and do not spend a continuation on it. A resumed call skips install and build and runs only what is left, merging into the SAME report file. Same \`timeout: ${SHELL_TOOL_MAX_TIMEOUT_MS}\`, and at most ${MAX_RESUME_CALLS} continuations \u2014 then report what the run has:`,
+        `**If the report says work is left, run it again with \`--resume\`.** The ${SHELL_TOOL_MAX_TIMEOUT_MS / 1e3}-second ceiling is per CALL, not per run: this repo needs more than one call to finish its suites (install, the builds, then \`packages/core\` at 106s and \`packages/cli\` at 401s, before the rest). Work is left when \`testScope.notRun\` is non-empty, or when any \`test[]\` entry has \`"clamped": true\` \u2014 a suite the budget started too late and killed, which says nothing about the suite. A third shape ends before any suite: a single-package repo whose budget ran out before its one suite has an empty \`test[]\`, no \`testScope\`, and \`"endedBeforeTests": true\` \u2014 the report's own stamp \u2014 with the note naming the unrun suite. That shape cannot be continued (a continuation has no recorded scope to read; a \`--resume\` on it answers "ended before its test phase" and points at a fresh run): report the dimension UNFINISHED and do not spend a continuation on it. A resumed call skips install and build and runs only what is left, merging into the SAME report file. Same \`timeout: ${SHELL_TOOL_MAX_TIMEOUT_MS}\`, and at most ${MAX_RESUME_CALLS} continuations \u2014 then report what the run has:`,
         "",
         "```bash",
         `"\${QWEN_CODE_CLI:-qwen}" review build-test \\`,
@@ -26516,8 +26540,9 @@ function resumeNpmToolchain(args, previous) {
   ];
   if (work.length === 0) {
     const neverTested = previous.test.length === 0 && !previous.testScope;
+    const endedEarly = neverTested && (previous.buildOnly === true || previous.ok === false || previous.endedBeforeTests === true);
     return withNote(
-      neverTested ? "Nothing to resume: the run being continued ended before its test phase, so it left no scope to continue \u2014 no suite ran. Re-run build-test without --resume." : "Nothing to resume: the run being continued reached every suite in scope."
+      endedEarly ? "Nothing to resume: the run being continued ended before its test phase, so it left no scope to continue \u2014 no suite ran. Re-run build-test without --resume." : neverTested ? "Nothing to resume: the run being continued completed with no suite to run." : "Nothing to resume: the run being continued reached every suite in scope."
     );
   }
   const ranDirs = [];
@@ -26559,9 +26584,13 @@ function resumeNpmToolchain(args, previous) {
       liveSegments.push(
         `a --resume call ran ${ranDirs.length + replaced.size} more command(s), ${outstanding.length} still to run: ` + outstanding.join(", ")
       );
-    } else if (stillClamped.length > 0) {
+    }
+    const provisionalOnly = stillClamped.filter(
+      (c) => !outstanding.includes(c)
+    );
+    if (provisionalOnly.length > 0) {
       liveSegments.push(
-        `a --resume call left ${stillClamped.length} command(s) provisional (killed on a budget-shortened deadline): ${stillClamped.join(", ")}`
+        `a --resume call left ${provisionalOnly.length} command(s) provisional (killed on a budget-shortened deadline): ${provisionalOnly.join(", ")}`
       );
     }
     const caveat = liveSegments.join("; ");
@@ -26585,6 +26614,11 @@ function resumeNpmToolchain(args, previous) {
     // Recomputed, not inherited: the previous `false` may have been nothing
     // but the clamped timeout this call just replaced with a pass.
     ok: previous.build.every(succeeded) && mergedTest.every(succeeded),
+    // Recomputed for the same staleness reason: the stamp claims the test
+    // phase ENTERED and ran nothing, which any suite in `mergedTest`
+    // falsifies. A continuation that ran none keeps it — the claim still
+    // holds then. JSON.stringify drops the undefined key.
+    endedBeforeTests: mergedTest.length === 0 ? previous.endedBeforeTests : void 0,
     timedOut,
     // REPLACED, not appended. The note being continued says things that were
     // true when it was written and are not now — "the whole-call budget was
@@ -26633,9 +26667,10 @@ function resumedNote(previous, test2, timedOut, stillPending, stillClamped) {
       `The budget was spent with ${stillPending.length} command(s) still to run \u2014 not run: ${stillPending.join(", ")}. Resume again to reach them.`
     );
   }
-  if (stillClamped.length > 0) {
+  const provisionalOnly = stillClamped.filter((c) => !stillPending.includes(c));
+  if (provisionalOnly.length > 0) {
     parts.push(
-      `${stillClamped.length} command(s) are still provisional \u2014 killed on a deadline the budget shortened, not on their own: ${stillClamped.join(", ")}. Resume again to give them a full one.`
+      `${provisionalOnly.length} command(s) are still provisional \u2014 killed on a deadline the budget shortened, not on their own: ${provisionalOnly.join(", ")}. Resume again to give them a full one.`
     );
   }
   if (stillPending.length === 0 && stillClamped.length === 0) {
@@ -26700,6 +26735,10 @@ function runNpmToolchain(args) {
       build: [],
       test: [],
       ...testScope ? { testScope } : {},
+      // The probe stamp rides EVERY producer path: this return already
+      // branches on args.buildOnly for its note, and a probe report without
+      // the stamp lost its probe answer on --resume.
+      ...args.buildOnly ? { buildOnly: true } : {},
       ok: true,
       timedOut: [],
       note: args.buildOnly ? `The diff changes ${changed.length} file(s), none of them inside a workspace. There is no package to build, and tests are out of scope for a build-only probe.` : testScope?.caveat ? `The diff changes ${changed.length} file(s), none of them inside a workspace. There is no package to build and no test to run, but the scope decision recorded a caveat: ${testScope.caveat}.` : `The diff changes ${changed.length} file(s), none of them inside a workspace (nothing the workspaces' tests can feel). There is no package to build and no test to run \u2014 this is a complete answer, not a skipped step.`
@@ -26722,7 +26761,12 @@ function runNpmToolchain(args) {
     test: [],
     ok: true,
     timedOut: [],
-    note: ""
+    note: "",
+    // Stamped structurally, not narrated: `--resume` on a report with no
+    // tests and no scope needs to tell a deliberate probe apart from a
+    // completed zero-suite run, and the note is prose an agent must never
+    // have to parse.
+    ...args.buildOnly ? { buildOnly: true } : {}
   };
   const npmLock = existsSync10(join20(root, "package-lock.json"));
   const installComplete = /* @__PURE__ */ __name(() => existsSync10(join20(root, "node_modules", ".package-lock.json")), "installComplete");
@@ -26876,6 +26920,9 @@ function runNpmToolchain(args) {
     if (r.timedOut) results.timedOut.push(r.command);
     if (r.exitCode !== 0) results.ok = false;
   }
+  if (results.test.length === 0 && notRun.length > 0) {
+    results.endedBeforeTests = true;
+  }
   notRun.sort();
   const partialNote = [
     notBuilt.length > 0 ? `the build phase reached the whole-call budget \u2014 not built: ` + notBuilt.join(", ") : "",
@@ -27098,11 +27145,16 @@ function previousReport(out) {
   })();
   const strings = /* @__PURE__ */ __name((v) => Array.isArray(v) && v.every((e) => typeof e === "string" && e.length > 0), "strings");
   const affectedOk = strings(parsed.affected);
+  const buildOnlyShape = parsed.buildOnly;
+  const buildOnlyOk = buildOnlyShape === void 0 || typeof buildOnlyShape === "boolean";
+  const endedBeforeShape = parsed.endedBeforeTests;
+  const endedBeforeOk = endedBeforeShape === void 0 || typeof endedBeforeShape === "boolean";
+  const okOk = typeof parsed.ok === "boolean";
   const notBuiltShape = parsed.notBuilt;
   const notBuiltOk = notBuiltShape === void 0 || strings(notBuiltShape);
   const scope = shape.testScope;
   const scopeOk = scope === void 0 || typeof scope === "object" && scope !== null && !Array.isArray(scope) && strings(scope.workspaces) && (scope.notRun === void 0 || strings(scope.notRun)) && (scope.caveat === void 0 || typeof scope.caveat === "string") && (scope.liveCaveat === void 0 || typeof scope.liveCaveat === "string");
-  if (!commandsOk(shape.test) || !commandsOk(shape.build) || !strings(shape.timedOut) || !affectedOk || !notBuiltOk || !scopeOk || !runOk) {
+  if (!commandsOk(shape.test) || !commandsOk(shape.build) || !strings(shape.timedOut) || !affectedOk || !buildOnlyOk || !endedBeforeOk || !okOk || !notBuiltOk || !scopeOk || !runOk) {
     throw new Error(
       `build-test: --resume expected a build-test report at ${out}, and that file is not one. Run build-test without --resume first.`
     );
@@ -27500,7 +27552,7 @@ init_esbuild_shims();
 import { spawnSync as spawnSync6 } from "node:child_process";
 import { existsSync as existsSync13, mkdirSync as mkdirSync22, readFileSync as readFileSync29, writeFileSync as writeFileSync20 } from "node:fs";
 import { dirname as dirname20, resolve as resolve21 } from "node:path";
-var RERUNNABLE_COMMAND_RE = /^npm test(?: --workspace="[\w@./-]+")?$/;
+var RERUNNABLE_COMMAND_RE = TEST_COMMAND_RE;
 var TRIM_MARKER_RE = /\.\.\. \[\d+ characters omitted/;
 function run2(command, cwd, timeoutMs) {
   const started = Date.now();
@@ -33226,10 +33278,10 @@ var updateCommand3 = {
       stdioHelpers,
       { updateEventEmitter }
     ] = await Promise.all([
-      import("./settings-WCVB34HW.js"),
-      import("./updateCheck-KYDCBLSF.js"),
-      import("./installationInfo-CO47QX3Q.js"),
-      import("./standalone-update-ALRIPWSM.js"),
+      import("./settings-ESZ3NWAP.js"),
+      import("./updateCheck-MTZIVGT5.js"),
+      import("./installationInfo-NMQIOUGY.js"),
+      import("./standalone-update-P42HDFXX.js"),
       import("./stdioHelpers-7A2EE64Z.js"),
       import("./updateEventEmitter-5MKC7BLQ.js")
     ]);
@@ -34420,7 +34472,7 @@ async function loadCliConfig(settings, argv, cwd = process.cwd(), overrideExtens
     disabledSlashCommands: disabledSlashCommands.length > 0 ? disabledSlashCommands : void 0,
     disabledSkillNamesProvider: bareMode || safeMode ? void 0 : disabledSkillNamesProvider,
     terminalImageRenderSupportProvider: interactive ? async () => {
-      const { getTerminalImageRenderSupport } = await import("./terminal-image-renderer-TEBO6ONY.js");
+      const { getTerminalImageRenderSupport } = await import("./terminal-image-renderer-6J2NPL32.js");
       return getTerminalImageRenderSupport();
     } : void 0,
     disabledSkillLevels: bareMode || safeMode || !Array.isArray(settings.skills?.disabledLevels) ? void 0 : settings.skills.disabledLevels.filter(isSkillLevel),

package/chunks/{chunk-6VDPDHIP.js → chunk-Y4RVDOXV.js}

@@ -2,16 +2,16 @@
 "use strict";
 import {
   functionResponsePartsToString
-} from "./chunk-XO24LIZC.js";
+} from "./chunk-6RDZLBSJ.js";
 import {
   observeHeadlessToolResultProjection,
   observeHeadlessToolResultWire,
   projectJsonStringToByteBudget
-} from "./chunk-EOCQCNV4.js";
+} from "./chunk-K2Y45XXS.js";
 import {
   parseAndFormatApiError,
   toolResultBoundaryArtifact
-} from "./chunk-XC377TLN.js";
+} from "./chunk-OFTPCOGR.js";
 import {
   formatVisionBridgeNoticeDisplay,
   isVisionBridgeNoticeDisplay

package/chunks/{chunk-RAQBBGS6.js → chunk-Y7QY6DKW.js}

@@ -27,10 +27,10 @@ import {
 } from "./chunk-3VUENPWF.js";
 import {
   loadSettings
-} from "./chunk-OLZSWUZR.js";
+} from "./chunk-VFP5YOHJ.js";
 import {
   MCP_BUDGET_WARN_FRACTION
-} from "./chunk-XC377TLN.js";
+} from "./chunk-OFTPCOGR.js";
 import {
   parsePositiveIntegerEnv
 } from "./chunk-6PVPNMXU.js";
@@ -422,7 +422,7 @@ var serveCommand = {
     }
     const externalToolGuardToken = process.env[EXTERNAL_TOOL_GUARD_TOKEN_ENV] ?? "";
     delete process.env[EXTERNAL_TOOL_GUARD_TOKEN_ENV];
-    const { runQwenServe } = await import("./run-qwen-serve-7RHU3NSV.js");
+    const { runQwenServe } = await import("./run-qwen-serve-UR3DYDAF.js");
     try {
       const handle = await runQwenServe({
         port: argv.port,

package/chunks/{chunk-4KQ2TI2K.js → chunk-YHXQK4SR.js}

@@ -5,10 +5,10 @@ import {
 } from "./chunk-2AUM35J5.js";
 import {
   getNpmCliPath
-} from "./chunk-DRKYT2TR.js";
+} from "./chunk-WAO53ZQF.js";
 import {
   t
-} from "./chunk-IOTPI425.js";
+} from "./chunk-VTOLYFWM.js";
 import {
   getPackageJson
 } from "./chunk-WZMH7II3.js";

package/chunks/{computer-use-ABECHPS5.js → computer-use-YORZJQE5.js}

@@ -4,7 +4,7 @@ import {
   Client,
   So,
   StdioClientTransport
-} from "./chunk-XC377TLN.js";
+} from "./chunk-OFTPCOGR.js";
 import "./chunk-GOFAQQZA.js";
 import "./chunk-5M6IDOMF.js";
 import "./chunk-TWPJO254.js";
@@ -14,7 +14,7 @@ import "./chunk-SLILKYV2.js";
 import "./chunk-NVRMCWTP.js";
 import "./chunk-AKBOLMUF.js";
 import "./chunk-6PVPNMXU.js";
-import "./chunk-SR2LYSMT.js";
+import "./chunk-544KT7X4.js";
 import "./chunk-IRH27ZC2.js";
 import "./chunk-QHWCP53L.js";
 import "./chunk-OBJYN4MX.js";
@@ -64,7 +64,7 @@ import {
   BaseDeclarativeTool,
   BaseToolInvocation
 } from "./chunk-6HY6IF3Z.js";
-import "./chunk-QIHHJKXT.js";
+import "./chunk-2A6VQJYK.js";
 import "./chunk-F6WFNA7U.js";
 import "./chunk-WZAD4ZNJ.js";
 import "./chunk-FPGTNKCP.js";

package/chunks/{contextCommand-4CD5HZQQ.js → contextCommand-PACHGIF4.js}

@@ -4,12 +4,12 @@ import {
   collectContextData,
   contextCommand,
   formatContextUsageText
-} from "./chunk-X3YA3YGG.js";
+} from "./chunk-7VNHO4MN.js";
 import "./chunk-4WGODMVR.js";
 import "./chunk-EKTCKZV5.js";
-import "./chunk-IOTPI425.js";
+import "./chunk-VTOLYFWM.js";
 import "./chunk-5EJZE3FP.js";
-import "./chunk-XC377TLN.js";
+import "./chunk-OFTPCOGR.js";
 import "./chunk-GOFAQQZA.js";
 import "./chunk-5M6IDOMF.js";
 import "./chunk-TWPJO254.js";
@@ -19,7 +19,7 @@ import "./chunk-SLILKYV2.js";
 import "./chunk-NVRMCWTP.js";
 import "./chunk-AKBOLMUF.js";
 import "./chunk-6PVPNMXU.js";
-import "./chunk-SR2LYSMT.js";
+import "./chunk-544KT7X4.js";
 import "./chunk-IRH27ZC2.js";
 import "./chunk-QHWCP53L.js";
 import "./chunk-OBJYN4MX.js";
@@ -66,7 +66,7 @@ import "./chunk-CHHABXPE.js";
 import "./chunk-MA2HEDVP.js";
 import "./chunk-XUJNK7Y6.js";
 import "./chunk-6HY6IF3Z.js";
-import "./chunk-QIHHJKXT.js";
+import "./chunk-2A6VQJYK.js";
 import "./chunk-F6WFNA7U.js";
 import "./chunk-WZAD4ZNJ.js";
 import "./chunk-FPGTNKCP.js";

package/chunks/{core-runtime-2MOVQ4VX.js → core-runtime-6HD5DSPX.js}

@@ -6,7 +6,7 @@ import {
   findProviderById,
   getDefaultModelIds,
   resolveBaseUrl
-} from "./chunk-XC377TLN.js";
+} from "./chunk-OFTPCOGR.js";
 import "./chunk-GOFAQQZA.js";
 import "./chunk-5M6IDOMF.js";
 import "./chunk-TWPJO254.js";
@@ -16,7 +16,7 @@ import "./chunk-SLILKYV2.js";
 import "./chunk-NVRMCWTP.js";
 import "./chunk-AKBOLMUF.js";
 import "./chunk-6PVPNMXU.js";
-import "./chunk-SR2LYSMT.js";
+import "./chunk-544KT7X4.js";
 import "./chunk-IRH27ZC2.js";
 import "./chunk-QHWCP53L.js";
 import "./chunk-OBJYN4MX.js";
@@ -80,7 +80,7 @@ import "./chunk-CHHABXPE.js";
 import "./chunk-MA2HEDVP.js";
 import "./chunk-XUJNK7Y6.js";
 import "./chunk-6HY6IF3Z.js";
-import "./chunk-QIHHJKXT.js";
+import "./chunk-2A6VQJYK.js";
 import "./chunk-F6WFNA7U.js";
 import "./chunk-WZAD4ZNJ.js";
 import "./chunk-FPGTNKCP.js";