mixdog 0.9.172 → 0.9.173

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "mixdog",
-  "version": "0.9.172",
+  "version": "0.9.173",
   "private": false,
   "type": "module",
   "description": "Standalone mixdog coding-agent CLI/TUI workspace.",
@@ -74,9 +74,9 @@
     "test:release-assets": "node --check scripts/verify-release-assets.mjs && node --check scripts/release-gate-test.mjs && node --check scripts/prepare-native-assets.mjs && node --test scripts/release-gate-test.mjs scripts/prepare-native-assets-test.mjs scripts/generate-voice-runtime-manifest.test.mjs",
     "test:native-edit-wire": "node scripts/native-edit-wire-test.mjs",
     "test:release-critical": "npm run test:release-assets && npm run smoke:patch && npm run test:providers",
-    "test:spec-advisory": "node scripts/advisory-spec-test.mjs && npm run test:spec-advisory --prefix apps/desktop",
+    "test:spec-advisory": "node scripts/run-advisory-spec.mjs && npm run test:spec-advisory --prefix apps/desktop",
     "test:session-transport": "node --test \"scripts/session-transport/*.test.mjs\" scripts/daemon-bootstrap-test.mjs src/runtime/shared/child-spawn-remote.test.mjs src/standalone/session-runtime-agent-control-client.test.mjs src/standalone/session-runtime-dispatch-cancel.test.mjs src/standalone/session-runtime-host-factory.test.mjs src/standalone/session-runtime-inline-host.test.mjs src/standalone/session-runtime-provider-cooldown.test.mjs src/standalone/session-runtime-shard-host.test.mjs src/standalone/session-runtime-shard-router.test.mjs",
-    "test:session": "node --test scripts/runtime-turn-contract-test.mjs scripts/session-save-fault-store-test.mjs scripts/turn-review-revert-test.mjs src/runtime/shared/tool-surface.test.mjs src/runtime/bridge-clients.test.mjs src/runtime/browser-bridge/action-schema.test.mjs src/runtime/computer-bridge/client-semantic-error.test.mjs src/runtime/agent/orchestrator/session/tool-envelope.test.mjs src/runtime/agent/orchestrator/session/eager-dispatch.test.mjs",
+    "test:session": "node --test scripts/runtime-turn-contract-test.mjs scripts/session-save-fault-store.slow.test.mjs scripts/turn-review-revert.slow.test.mjs src/runtime/shared/tool-surface.test.mjs src/runtime/bridge-clients.test.mjs src/runtime/browser-bridge/action-schema.test.mjs src/runtime/computer-bridge/client-semantic-error.test.mjs src/runtime/agent/orchestrator/session/tool-envelope.test.mjs src/runtime/agent/orchestrator/session/eager-dispatch.test.mjs",
     "test:quick": "npm run test:session && npm run test:session-transport && npm run test:desktop-main --prefix apps/desktop",
     "test:office": "node --test scripts/run-office-live-tests.test.mjs src/runtime/office/office-runtime-contract.test.mjs src/runtime/office/office-runtime-com.test.mjs src/runtime/office/office-runtime-portable.test.mjs src/runtime/office/xlsx-formula-audit.test.mjs src/runtime/office/xlsx-portable-snapshot.test.mjs src/runtime/office/office-runtime-pdf.test.mjs src/runtime/office/office-runtime-design.test.mjs src/runtime/office/office-assurance.test.mjs src/runtime/office/office-design-aesthetics.test.mjs src/runtime/office/portable-authoring.test.mjs src/runtime/office/slide-quality.test.mjs src/runtime/office/authoring/pptx-brief.test.mjs src/runtime/office/authoring/pptx-receipt.test.mjs src/runtime/office/quality/render-air.test.mjs src/runtime/office/quality/pptx-deck-rubric.test.mjs && npm run test:office-approval --prefix apps/desktop",
     "test:office:live": "node scripts/run-office-live-tests.mjs",

package/scripts/code-graph-description-contract.mjs

@@ -20,8 +20,7 @@ const includesTerm = (text, term) => {
 const includesAll = (text, terms) => terms.every((term) => includesTerm(text, term));
 const hasPositiveClause = (text, terms) =>
   clauses(text).some((clause) => !NEGATION.test(clause) && includesAll(clause, terms));
-const hasModeClause = (text, label, modes, target = null) =>
-  hasPositiveClause(text, [label, ...modes, ...(target ? [target] : [])]);
+const hasModeClause = (text, label, modes) => hasPositiveClause(text, [label, ...modes]);
 const hasContradictoryTargetAssignment = (text) =>
   hasPositiveClause(text, [...FILE_MODES, 'symbols[]']) || hasPositiveClause(text, [...SYMBOL_MODES, 'files[]']);
 const removeIdentifier = (text, identifier) =>

package/scripts/context-renewal-live-check.mjs

@@ -58,7 +58,7 @@ if (child) {
     console.error(`${spec.name}: ${error.message}`);
     process.exitCode = 1;
   } finally {
-    await runtime?.stop?.('context-renewal-live-check');
+    await runtime?.close('context-renewal-live-check');
   }
   process.exit(process.exitCode || 0);
 }

package/scripts/coverage-query.mjs

@@ -0,0 +1,27 @@
+#!/usr/bin/env node
+// Ask the artifact written by `node scripts/test.mjs --coverage` whether the
+// suite executed the function a line belongs to. A file that changed since the
+// collection answers STALE instead of a verdict nobody should act on.
+//
+// Usage: node scripts/coverage-query.mjs <file> <line>
+//        node scripts/coverage-query.mjs --status
+import { relative, resolve } from 'node:path';
+import {
+  coverageStaleness,
+  formatCoverageAnswer,
+  formatStaleness,
+  loadCoverageArtifact,
+  queryCoverage,
+} from './lib/coverage.mjs';
+
+const [file, line] = process.argv.slice(2);
+if (file === '--status') {
+  console.log(formatStaleness(await coverageStaleness(await loadCoverageArtifact())));
+} else if (!file || !/^\d+$/.test(line ?? '')) {
+  console.error('usage: node scripts/coverage-query.mjs <file> <line> | --status');
+  process.exitCode = 1;
+} else {
+  const artifact = await loadCoverageArtifact();
+  const path = relative(process.cwd(), resolve(process.cwd(), file));
+  console.log(formatCoverageAnswer(await queryCoverage(artifact, path, Number(line))));
+}

package/scripts/generate-runtime-manifest.mjs

@@ -46,7 +46,6 @@ try {
 
 const { assets } = JSON.parse(assetsJson);
 
-// Filter tarball assets
 const tarballs = assets.filter((a) => /^mixdog-runtime-.+\.tar\.gz$/.test(a.name));
 console.log(
   `Found ${tarballs.length} tarball(s):`,
@@ -65,27 +64,24 @@ if (tarballs.length === 0) {
 const PG_RE = /pg(\d+)\.(\d+)/;
 const VEC_RE = /pgvector(\d+(?:\.\d+)+)/;
 
-let pgMajor = 16,
-  pgMinor = 4,
-  pgvectorVersion = '0.7.4';
-for (const t of tarballs) {
-  const pgM = PG_RE.exec(t.name);
-  const vM = VEC_RE.exec(t.name);
-  if (pgM) {
-    pgMajor = parseInt(pgM[1], 10);
-    pgMinor = parseInt(pgM[2], 10);
-  }
-  if (vM) {
-    pgvectorVersion = vM[1];
-  }
-  break; // all tarballs share the same version; just read the first
+// All tarballs of one release share the same versions, so the first asset
+// carrying both wins. A release whose names carry neither must fail here: a
+// manifest built from hardcoded defaults would publish the wrong runtime.
+const versioned = tarballs.find((t) => PG_RE.test(t.name) && VEC_RE.test(t.name));
+if (!versioned) {
+  console.error(
+    `Cannot read pg/pgvector versions from release asset name(s): ${tarballs.map((t) => t.name).join(', ')}`
+  );
+  process.exit(1);
 }
+const pgMajor = parseInt(PG_RE.exec(versioned.name)[1], 10);
+const pgMinor = parseInt(PG_RE.exec(versioned.name)[2], 10);
+const pgvectorVersion = VEC_RE.exec(versioned.name)[1];
 
 // ---------------------------------------------------------------------------
 // 3. Fetch sha256 sidecar content for each tarball
 // ---------------------------------------------------------------------------
 async function fetchSha256(sha256AssetName, _downloadUrl) {
-  // Try matching .sha256 sidecar from the asset list first
   const sidecar = assets.find((a) => a.name === sha256AssetName);
   if (!sidecar) {
     throw new Error(`Missing required checksum sidecar: ${sha256AssetName}`);

package/scripts/graph-parity.mjs

@@ -18,6 +18,53 @@ export function normalizeRel(rel) {
   return String(rel || '').replace(/\\/g, '/');
 }
 
+/** Flags whose next argv entry is stored verbatim on the named option field. */
+const STRING_FLAG_FIELDS = new Map([
+  ['--old', 'old'],
+  ['--new', 'new'],
+  ['--root', 'root'],
+  ['--json', 'json'],
+  ['--old-jsonl', 'oldJsonl'],
+  ['--new-jsonl', 'newJsonl'],
+  ['--kind-map', 'kindMap'],
+]);
+
+/** Flags whose next argv entry is stored as a Number on the named option field. */
+const NUMBER_FLAG_FIELDS = new Map([
+  ['--max-symbol-loss', 'maxSymbolLoss'],
+  ['--max-import-diff', 'maxImportDiff'],
+  ['--max-token-loss', 'maxTokenLoss'],
+  ['--max-time-ratio', 'maxTimeRatio'],
+  ['--runs', 'runs'],
+]);
+
+/** Consume the consecutive non-flag argv entries that follow index `start`. */
+function takeTrailingValues(argv, start) {
+  const values = [];
+  let index = start;
+  while (index + 1 < argv.length && !String(argv[index + 1]).startsWith('--')) {
+    values.push(argv[index + 1]);
+    index += 1;
+  }
+  return { values, index };
+}
+
+/** Cross-flag mode requirements and numeric threshold ranges. */
+function validateParityOptions(out) {
+  const jsonlMode = Boolean(out.oldJsonl || out.newJsonl);
+  if (jsonlMode && (!out.oldJsonl || !out.newJsonl)) {
+    throw new Error('both --old-jsonl and --new-jsonl are required');
+  }
+  if (!jsonlMode && (!out.old || !out.new)) {
+    throw new Error('--old and --new binaries are required (or --old-jsonl/--new-jsonl)');
+  }
+  if (!Number.isFinite(out.maxSymbolLoss) || out.maxSymbolLoss < 0) throw new Error('invalid --max-symbol-loss');
+  if (!Number.isFinite(out.maxImportDiff) || out.maxImportDiff < 0) throw new Error('invalid --max-import-diff');
+  if (!Number.isFinite(out.maxTokenLoss) || out.maxTokenLoss < 0) throw new Error('invalid --max-token-loss');
+  if (!Number.isFinite(out.maxTimeRatio) || out.maxTimeRatio <= 0) throw new Error('invalid --max-time-ratio');
+  if (!Number.isFinite(out.runs) || out.runs < 1 || !Number.isInteger(out.runs)) throw new Error('invalid --runs');
+}
+
 export function parseArgs(argv) {
   const out = {
     old: null,
@@ -43,55 +90,19 @@ export function parseArgs(argv) {
   };
   for (let i = 0; i < argv.length; i += 1) {
     const a = argv[i];
+    const stringField = STRING_FLAG_FIELDS.get(a);
+    if (stringField) {
+      out[stringField] = take(argv, i, a);
+      i += 1;
+      continue;
+    }
+    const numberField = NUMBER_FLAG_FIELDS.get(a);
+    if (numberField) {
+      out[numberField] = Number(take(argv, i, a));
+      i += 1;
+      continue;
+    }
     switch (a) {
-      case '--old':
-        out.old = take(argv, i, a);
-        i += 1;
-        break;
-      case '--new':
-        out.new = take(argv, i, a);
-        i += 1;
-        break;
-      case '--root':
-        out.root = take(argv, i, a);
-        i += 1;
-        break;
-      case '--json':
-        out.json = take(argv, i, a);
-        i += 1;
-        break;
-      case '--old-jsonl':
-        out.oldJsonl = take(argv, i, a);
-        i += 1;
-        break;
-      case '--new-jsonl':
-        out.newJsonl = take(argv, i, a);
-        i += 1;
-        break;
-      case '--max-symbol-loss':
-        out.maxSymbolLoss = Number(take(argv, i, a));
-        i += 1;
-        break;
-      case '--max-import-diff':
-        out.maxImportDiff = Number(take(argv, i, a));
-        i += 1;
-        break;
-      case '--max-token-loss':
-        out.maxTokenLoss = Number(take(argv, i, a));
-        i += 1;
-        break;
-      case '--max-time-ratio':
-        out.maxTimeRatio = Number(take(argv, i, a));
-        i += 1;
-        break;
-      case '--runs':
-        out.runs = Number(take(argv, i, a));
-        i += 1;
-        break;
-      case '--kind-map':
-        out.kindMap = take(argv, i, a);
-        i += 1;
-        break;
       case '--tokens':
         out.tokens = true;
         break;
@@ -99,18 +110,17 @@ export function parseArgs(argv) {
         const first = take(argv, i, a);
         i += 1;
         out.allowNewLanguages.push(...splitLangIds(first));
-        while (i + 1 < argv.length && !String(argv[i + 1]).startsWith('--')) {
-          out.allowNewLanguages.push(...splitLangIds(argv[i + 1]));
-          i += 1;
-        }
+        const trailing = takeTrailingValues(argv, i);
+        for (const value of trailing.values) out.allowNewLanguages.push(...splitLangIds(value));
+        i = trailing.index;
         break;
       }
-      case '--files':
-        while (i + 1 < argv.length && !String(argv[i + 1]).startsWith('--')) {
-          out.files.push(argv[i + 1]);
-          i += 1;
-        }
+      case '--files': {
+        const trailing = takeTrailingValues(argv, i);
+        out.files.push(...trailing.values);
+        i = trailing.index;
         break;
+      }
       case '--help':
       case '-h':
         throw new Error('help');
@@ -118,18 +128,7 @@ export function parseArgs(argv) {
         throw new Error(`unknown argument: ${a}`);
     }
   }
-  const jsonlMode = Boolean(out.oldJsonl || out.newJsonl);
-  if (jsonlMode && (!out.oldJsonl || !out.newJsonl)) {
-    throw new Error('both --old-jsonl and --new-jsonl are required');
-  }
-  if (!jsonlMode && (!out.old || !out.new)) {
-    throw new Error('--old and --new binaries are required (or --old-jsonl/--new-jsonl)');
-  }
-  if (!Number.isFinite(out.maxSymbolLoss) || out.maxSymbolLoss < 0) throw new Error('invalid --max-symbol-loss');
-  if (!Number.isFinite(out.maxImportDiff) || out.maxImportDiff < 0) throw new Error('invalid --max-import-diff');
-  if (!Number.isFinite(out.maxTokenLoss) || out.maxTokenLoss < 0) throw new Error('invalid --max-token-loss');
-  if (!Number.isFinite(out.maxTimeRatio) || out.maxTimeRatio <= 0) throw new Error('invalid --max-time-ratio');
-  if (!Number.isFinite(out.runs) || out.runs < 1 || !Number.isInteger(out.runs)) throw new Error('invalid --runs');
+  validateParityOptions(out);
   out.allowNewLanguages = [...new Set(out.allowNewLanguages)];
   return out;
 }
@@ -184,11 +183,9 @@ export function parseLangsStdout(text) {
     }
   };
   const whole = tryParse(src);
-  if (whole && Object.keys(whole).length) return whole;
   if (whole) return whole;
   for (const line of src.split(/\n/)) {
     const parsed = tryParse(line.trim());
-    if (parsed && Object.keys(parsed).length) return parsed;
     if (parsed) return parsed;
   }
   return {};
@@ -771,8 +768,9 @@ function formatExamples(examples) {
   return examples.map((ex) => `    - ${JSON.stringify(ex)}`).join('\n');
 }
 
-export function formatMarkdown(report) {
-  const lines = ['# mixdog-graph extraction parity', ''];
+/** Run inputs: compared binaries/root, file counts, language allow-list, kind-map source. */
+function formatRunInputLines(report) {
+  const lines = [];
   if (report.old) lines.push(`- old: \`${report.old}\``);
   if (report.new) lines.push(`- new: \`${report.new}\``);
   if (report.root) lines.push(`- root: \`${report.root}\``);
@@ -799,6 +797,12 @@ export function formatMarkdown(report) {
     lines.push('- kind-map: inactive');
   }
   if (report.tokens) lines.push('- tokens: on');
+  return lines;
+}
+
+/** Gate outcome: manifest parity, timing ratio, and the PASS/FAIL verdict. */
+function formatGateLines(report) {
+  const lines = [];
   if (report.manifestIdentical == null) lines.push('- manifest: (skipped)');
   else if (report.manifestIdentical) {
     lines.push(
@@ -822,7 +826,12 @@ export function formatMarkdown(report) {
   } else {
     lines.push('- result: **PASS**');
   }
-  lines.push('', '## Totals', '');
+  return lines;
+}
+
+/** Per-category totals table. */
+function formatTotalsLines(report) {
+  const lines = ['', '## Totals', ''];
   lines.push('| category | count |');
   lines.push('|---|---|');
   lines.push(`| LOSS | ${report.totals.loss} |`);
@@ -837,6 +846,12 @@ export function formatMarkdown(report) {
     lines.push(`| TOKEN_LOSS | ${report.totals.tokenLost || 0} |`);
     lines.push(`| TOKEN_ADD | ${report.totals.tokenAdded || 0} |`);
   }
+  return lines;
+}
+
+/** Optional sections that qualify the totals: unmapped kinds, additive fields, line moves. */
+function formatTotalsQualifierLines(report) {
+  const lines = [];
   if (report.unmappedKinds?.length) {
     lines.push('', '## Unmapped kinds', '');
     for (const item of report.unmappedKinds) {
@@ -862,7 +877,12 @@ export function formatMarkdown(report) {
       lines.push(`- ${item.rel}: \`${item.name}\``);
     }
   }
+  return lines;
+}
 
+/** One section per language with its non-empty diff buckets and examples. */
+function formatLanguageLines(report) {
+  const lines = [];
   const langs = Object.keys(report.byLanguage).sort();
   for (const lang of langs) {
     const b = report.byLanguage[lang];
@@ -881,7 +901,20 @@ export function formatMarkdown(report) {
       if (formatted) lines.push(formatted);
     }
   }
-  lines.push('');
+  return lines;
+}
+
+export function formatMarkdown(report) {
+  const lines = [
+    '# mixdog-graph extraction parity',
+    '',
+    ...formatRunInputLines(report),
+    ...formatGateLines(report),
+    ...formatTotalsLines(report),
+    ...formatTotalsQualifierLines(report),
+    ...formatLanguageLines(report),
+    '',
+  ];
   return lines.join('\n');
 }