@ran-sh/dsh-crew 2.2.6 → 2.2.8

package/.claude-plugin/plugin.json

@@ -1,6 +1,6 @@
 {
   "name": "dsh-crew",
-  "version": "2.2.6",
+  "version": "2.2.8",
   "description": "Dispatch subtasks to DeepSeek Harness (DSH) agents as native subagents with live progress",
   "author": {
     "name": "ZSeven-W"

package/docs/readiness-matrix.md

@@ -12,10 +12,29 @@ The matrix is emitted by `hubStatus()` and is therefore visible inside the exist
 - `FAIL` — a check actually ran and produced an incompatible or failed result.
 - `BLOCKED` — the check could not run because required infrastructure or authorization was unavailable.
 - `SKIP` — the row is intentionally not applicable for the active policy/path.
+- `NOT_APPLICABLE` — the row asks about a route this machine does not use, and the
+  matrix says so instead of omitting it. The two built-in DeepSeek rows are the only
+  rows a policy may mark this way: they read `NOT_APPLICABLE` /
+  `WORKER_PROVIDER_FOLLOWS_DSH` whenever a known, other provider is selected, and keep
+  `NOT_RUN` when the selection is unknown.
 - `NOT_RUN` — no trusted evidence has been supplied for the row.
 
 `BLOCKED` and `SKIP` are not failures. `NOT_RUN` is not success.
 
+Applicability is decided when the row is built and it is final for that row:
+
+- Supplied evidence can neither create nor clear it. `NOT_APPLICABLE` is not part of
+  the evidence vocabulary (`PASS`, `FAIL`, `BLOCKED`, `SKIP`, `NOT_RUN`), so no
+  evidence source can make a required row disappear from readiness, and evidence
+  reported against a not-applicable row is kept as `reported_evidence` metadata
+  instead of flipping the status back.
+- A required row that arrives as `NOT_APPLICABLE` anyway is a contradiction, not a
+  pass: consumers that project such a row into readiness report
+  `UNAVAILABLE` / `CHECK_NOT_APPLICABLE_ON_REQUIRED_ROW`, which cannot leave the
+  aggregate READY.
+- The matrix declares `schema_version: 2`: a row may legally be `NOT_APPLICABLE` and
+  the summary carries a matching key, which is what a strict consumer needs to know.
+
 ## Evidence classes
 
 The matrix separates three kinds of evidence:
@@ -61,7 +80,7 @@ actually run" — `provider_health`, `reviewer_health`, `model_execution`,
 `worker_primary_callable`, `reviewer_primary_callable`, `reviewer_pipeline` — is
 filtered to whatever route is currently selected, so an operator on other
 providers is covered by those and loses nothing by the two DeepSeek rows
-reading `NOT_RUN`. A static row per provider was tried and replaced by these
+reading `NOT_APPLICABLE` under `follow-dsh`. A static row per provider was tried and replaced by these
 dynamic signals; `opencode_go_mimo_qwen` was that attempt's last trace in this
 document.
 

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@ran-sh/dsh-crew",
-  "version": "2.2.6",
+  "version": "2.2.8",
   "type": "module",
   "main": "./src/hub/entry.mjs",
   "bin": {

package/scripts/setup.mjs

@@ -384,11 +384,19 @@ export async function setupStatus({ log = console.log, root = ROOT, home = homed
     } catch { dshPlugin = 'unknown'; }
   }
   log(`DSH plugin: ${dshPlugin} (dedicated dsh-crew profile; official web profile ignored)`);
+  // The frontend is on-demand by design: 3210 is the service, and 3080 is started
+  // for a browser session. Rendering it as anything else reads as a missing piece
+  // of the installation, which it is not.
+  const frontendConfig = (installer.readGlobalConfig ?? realInstaller.readGlobalConfig)({ configFile: join(home, '.config', 'dsh-crew', 'config.json') });
+  const frontend = frontendConfig?.frontend_autostart === true
+    ? 'configured to auto-start at login (dsh-crew open for a session URL)'
+    : 'available on demand (dsh-crew open)';
+  log(`Frontend (3080): ${frontend}`);
   log(`Codex Desktop integration: ${codex}`);
   log(`ZCode integration: ${zcode}`);
   log(`Claude Code integration: ${claude}`);
   log(`Windows login startup: ${windowsStartup}`);
-  return { ok: true, dshPlugin, codex, zcode, claude, windowsStartup };
+  return { ok: true, dshPlugin, frontend, codex, zcode, claude, windowsStartup };
 }
 
 export async function runSetupCli({ argv = process.argv.slice(2), run: actions = {}, log = console.log } = {}) {

package/src/extension-contract.mjs

@@ -29,9 +29,19 @@ function workspaceComponent(workspace) {
     : { ...component('UNAVAILABLE', workspace?.code ?? 'WORKSPACE_NOT_CHECKED'), state: 'UNAVAILABLE' };
 }
 
-function readinessFromRow(entry, { pass = 'READY', notRun = 'DEGRADED' } = {}) {
+function readinessFromRow(entry, { pass = 'READY', notRun = 'DEGRADED', allowNotApplicable = false } = {}) {
   if (!entry) return component('UNAVAILABLE', 'NO_EVIDENCE');
   if (entry.status === 'PASS') return component(pass, entry.reason_code ?? 'CHECK_PASSED');
+  // Every row this function is called with asks a question that applies to any Crew
+  // machine (a reachable hub, a consistent provider lifecycle, a runnable route), so
+  // "not applicable" is a contradiction here, not a pass: it is reported as missing
+  // evidence and it cannot leave the aggregate READY. Only a caller that names a row
+  // which may legitimately not apply opts in.
+  if (entry.status === 'NOT_APPLICABLE') {
+    return allowNotApplicable
+      ? component('NOT_APPLICABLE', entry.reason_code ?? 'CHECK_NOT_APPLICABLE')
+      : component('UNAVAILABLE', 'CHECK_NOT_APPLICABLE_ON_REQUIRED_ROW');
+  }
   if (entry.status === 'NOT_RUN' || entry.status === 'SKIP') return component(notRun, entry.reason_code ?? 'CHECK_NOT_RUN');
   return component('UNAVAILABLE', entry.reason_code ?? 'CHECK_FAILED');
 }
@@ -50,7 +60,12 @@ function modelReadinessFromSnapshot(readinessSnapshot, matrix, runtime, expected
     });
     if (validation.ok && validation.state === 'CALLABLE') return component('READY', 'CURRENT_MODEL_CALLABLE', { captured_at: callability.captured_at, expires_at: callability.expires_at, runtime_id: callability.current_runtime_id });
     if (validation.ok && validation.state === 'NOT_CALLABLE') return component('UNAVAILABLE', callability.roles.worker?.state === 'NOT_CALLABLE' ? callability.roles.worker.reason_code : callability.roles.reviewer.reason_code ?? 'CURRENT_MODEL_NOT_CALLABLE');
+    // Aging evidence is reported as aging, not as a fault, and "nothing has proven
+    // this route yet" is reported as unknown. The validator's own success code is
+    // `MODEL_CALLABILITY_VALID`, and rendering that next to DEGRADED read as a
+    // contradiction: the projection was valid, the evidence was simply not current.
     if (validation.ok && validation.state === 'STALE') return component('DEGRADED', 'MODEL_EVIDENCE_STALE');
+    if (validation.ok) return component('DEGRADED', 'MODEL_CALLABILITY_UNKNOWN');
     return component('DEGRADED', validation.reason_code ?? 'MODEL_CALLABILITY_UNKNOWN');
   }
   if (providerHealthEvidence?.status === 'FAIL') return readinessFromRow(providerHealthEvidence);

package/src/hub/index.mjs

@@ -642,6 +642,10 @@ export class WorkerRegistry {  constructor(ctx) {
       delivery_complete: !!job.delivery_complete,
       allow_no_changes: job.allow_no_changes === true,
       task_status: job.outcome?.task_status ?? null,
+      // Whether the worker's execution itself completed, kept beside the task-level
+      // verdict: a `partial` task still had a provider response, and that is what
+      // model callability asks about.
+      execution_status: job.outcome?.execution_status ?? null,
       workspace_evidence_ok: job.outcome?.workspace_evidence_ok ?? null,
       review_verdict: job.review?.verdict ?? null,
       workspace_diff_available: !!job.workspaceDiff && ['git', 'filesystem-empty'].includes(job.workspaceDiff.kind),

package/src/install/install-legacy.mjs

@@ -17,6 +17,14 @@ import { integrationRoot } from './crew-paths.mjs';
 const ROOT = resolve(dirname(fileURLToPath(import.meta.url)), '..', '..');
 const MARKETPLACE_NAME = 'dsh-crew';
 const PLUGIN_KEY = `dsh-crew@${MARKETPLACE_NAME}`;
+// Every name this installer has ever registered Crew under in a Claude host, and
+// the only names its uninstall may remove. `dsh-workers` is the pre-rename
+// identity: an operator who upgraded keeps that marketplace, plugin entry and its
+// permission rules until the same uninstall cleans them, and nothing else in the
+// file is Crew's to touch.
+const CREW_CLAUDE_MARKETPLACES = [MARKETPLACE_NAME, 'dsh-workers'];
+const CREW_CLAUDE_PLUGIN_KEYS = [PLUGIN_KEY, 'dsh-workers@dsh-workers'];
+const CREW_CLAUDE_PERMISSION_PREFIXES = ['mcp__plugin_dsh-crew_', 'mcp__plugin_dsh-workers_'];
 // A Claude Code plugin refresh is a real copy of the plugin tree, and its cost
 // tracks machine load: 163s measured idle, ~6 minutes measured during an
 // activation. The install is the step that copies, so it gets the ceiling that
@@ -645,7 +653,7 @@ export function uninstallCodex({ home = homedir(), env = process.env } = {}) {
 export function claudeIntegrationLine(result) {
   if (result?.ok === false) return '✗ Claude Code integration failed';
   if (result?.detected === false) {
-    return '- Claude Code not detected; settings registered, CLI step skipped';
+    return '- Claude Code not installed (claude CLI not found); nothing written';
   }
   if (result?.degraded === true) {
     return `- Claude Code integration registered, but not loaded: ${result.reason ?? 'plugin snapshot not refreshed'}`;
@@ -906,9 +914,22 @@ export async function runClaudeStep(args, { timeoutMs = CLAUDE_STEP_TIMEOUT_MS,
   };
 }
 
-export async function installClaudeCode({ home = homedir(), statusline = false, root = ROOT } = {}) {
+export async function installClaudeCode({ home = homedir(), statusline = false, root = ROOT, resolveClaude = resolveClaudeCommand } = {}) {
   const actions = [];
 
+  // This registration exists only to make Crew callable FROM Claude Code, so a
+  // host without the `claude` CLI has nothing for it to serve. Writing it anyway
+  // left Crew's marketplace, plugin entry and permission rules in a file Crew
+  // could never exercise, next to a footprint that read back as an integration
+  // wanting a CLI the machine does not have. Detect once, up front, and write
+  // nothing when the host cannot use it: `host_detected` then reports the truth
+  // with no stale registration to explain away.
+  const claudeCommand = resolveClaude();
+  if (!claudeCommand) {
+    actions.push('claude CLI not found; no Claude configuration written');
+    return { ok: true, detected: false, actions };
+  }
+
   // The repository carries its own marketplace manifest (.claude-plugin/
   // marketplace.json with source "."), so the marketplace root IS the plugin
   // checkout itself — no parent-directory layout assumption. This keeps the
@@ -999,7 +1020,8 @@ export async function installClaudeCode({ home = homedir(), statusline = false,
   // Materialize the install through the claude CLI: registers the marketplace
   // in the plugin cache (settings alone leave a stale/absent cache entry) and
   // pulls the plugin so the next session loads it. Best-effort: settings are
-  // already correct, so a missing CLI just means one manual `plugin install`.
+  // already correct, so a CLI that fails on its own terms leaves the operator one
+  // manual `plugin install` short rather than an unregistered host.
   const registered = readJson(join(home, '.claude', 'plugins', 'known_marketplaces.json'), {})?.[MARKETPLACE_NAME];
   const installedRecord = readJson(join(home, '.claude', 'plugins', 'installed_plugins.json'), {})?.plugins?.[PLUGIN_KEY];
   const installedEntries = Array.isArray(installedRecord) ? installedRecord : [installedRecord];
@@ -1019,14 +1041,9 @@ export async function installClaudeCode({ home = homedir(), statusline = false,
     actions.push('cli: skipped (non-default home; test mode)');
     return { ok: true, actions };
   }
-  // The CLI is best-effort, but a host that does not have it must not be told to
-  // run it. The checkout entry already gates on this; doing it here too keeps the
-  // two entries describing the same machine the same way.
-  const claudeCommand = resolveClaudeCommand();
-  if (!claudeCommand) {
-    actions.push('cli: skipped (claude not found)');
-    return { ok: true, detected: false, actions };
-  }
+  // `claudeCommand` was resolved before anything was written: reaching this line
+  // already proves the CLI is there, so the step is best-effort rather than
+  // possibly-pointless.
   return refreshClaudePlugin({ home, root, actions, claudeCommand });
 }
 
@@ -1230,25 +1247,94 @@ export function installHudSegment({ home = homedir() } = {}) {
   return { ok: true, actions: [...(bak ? [`backup: ${bak}`] : []), 'statusLine: claude-hud now runs worker-segment.sh via --extra-cmd'] };
 }
 
+// Crew-owned directories, identified by path SEGMENT rather than by substring: the
+// payload tree lives under `.../dsh-crew/...` and the pre-rename identity used
+// `dsh-workers`. A user path that merely contains those words (`my-dsh-crew-notes`)
+// is not Crew's, and nothing here may treat it as if it were.
+function crewOwnedDirectory(value) {
+  if (typeof value !== 'string' || !value.trim()) return false;
+  return value.split(/[\\/]+/).filter(Boolean).some((segment) => {
+    const lower = segment.toLowerCase();
+    return lower === 'dsh-crew' || lower === 'dsh-workers';
+  });
+}
+
+// Only the status line `--statusline` installs is Crew's to remove: it points at
+// Crew's own script, in Crew's own directory, in the exact shape the installer
+// writes (`bash <root>/statusline/statusline.sh`). A command that merely mentions
+// these words belongs to whoever wrote it.
+function crewOwnedStatusLine(value) {
+  const command = typeof value?.command === 'string' ? value.command.trim() : '';
+  const match = /^bash\s+(.+[\\/])statusline[\\/]statusline\.sh$/i.exec(command);
+  return match ? crewOwnedDirectory(match[1]) : false;
+}
+
+// The claude CLI records, per marketplace and plugin, what it materialized, in
+// files of its own. Unregistering the settings and leaving those records behind
+// hands the next install a snapshot with no owner, so an explicit uninstall
+// clears them too. Crew-scoped and best-effort: a file that is absent, unreadable
+// or shaped differently than expected is left exactly as it is found.
+function removeCrewClaudeCacheRecords({ home, actions }) {
+  const dir = join(home, '.claude', 'plugins');
+  const marketsFile = join(dir, 'known_marketplaces.json');
+  const markets = readJson(marketsFile, null);
+  if (markets && typeof markets === 'object' && !Array.isArray(markets)) {
+    const owned = CREW_CLAUDE_MARKETPLACES.filter((name) => Object.hasOwn(markets, name));
+    if (owned.length) {
+      backup(marketsFile);
+      for (const name of owned) delete markets[name];
+      writeFileSync(marketsFile, JSON.stringify(markets, null, 2) + '\n');
+      actions.push(`marketplace cache: removed ${owned.join(', ')}`);
+    }
+  }
+  const pluginsFile = join(dir, 'installed_plugins.json');
+  const installed = readJson(pluginsFile, null);
+  if (installed && typeof installed === 'object' && !Array.isArray(installed)
+    && installed.plugins && typeof installed.plugins === 'object' && !Array.isArray(installed.plugins)) {
+    const owned = CREW_CLAUDE_PLUGIN_KEYS.filter((key) => Object.hasOwn(installed.plugins, key));
+    if (owned.length) {
+      backup(pluginsFile);
+      for (const key of owned) delete installed.plugins[key];
+      writeFileSync(pluginsFile, JSON.stringify(installed, null, 2) + '\n');
+      actions.push(`plugin cache: removed ${owned.join(', ')}`);
+    }
+  }
+}
+
 export function uninstallClaudeCode({ home = homedir() } = {}) {
+  const actions = [];
   const settingsFile = join(home, '.claude', 'settings.json');
   const settings = readJson(settingsFile, null);
-  if (!settings) return { ok: true, actions: ['settings.json not found'] };
-  backup(settingsFile);
-  const mpDir = join(home, '.config', 'dsh-crew', 'marketplace');
-  if (Array.isArray(settings.extraKnownMarketplaces)) {
-    settings.extraKnownMarketplaces = settings.extraKnownMarketplaces.filter((m) => m?.path !== mpDir);
-  } else if (settings.extraKnownMarketplaces) {
-    delete settings.extraKnownMarketplaces[MARKETPLACE_NAME];
-  }
-  if (Array.isArray(settings.enabledPlugins)) {
-    settings.enabledPlugins = settings.enabledPlugins.filter((p) => p !== PLUGIN_KEY);
-  } else if (settings.enabledPlugins) {
-    delete settings.enabledPlugins[PLUGIN_KEY];
-  }
-  if (settings.permissions?.allow) {
-    settings.permissions.allow = settings.permissions.allow.filter((r) => !r.startsWith('mcp__plugin_dsh-crew_'));
+  if (settings) {
+    backup(settingsFile);
+    const mpDir = join(home, '.config', 'dsh-crew', 'marketplace');
+    if (Array.isArray(settings.extraKnownMarketplaces)) {
+      // Legacy array shape: entries are `{ path }` records this installer wrote under
+      // one of its own names, including the pre-rename one. Anything else in that
+      // array belongs to another plugin and stays.
+      settings.extraKnownMarketplaces = settings.extraKnownMarketplaces
+        .filter((entry) => !crewOwnedDirectory(entry?.path) && entry?.path !== mpDir);
+    } else if (settings.extraKnownMarketplaces) {
+      for (const name of CREW_CLAUDE_MARKETPLACES) delete settings.extraKnownMarketplaces[name];
+    }
+    if (Array.isArray(settings.enabledPlugins)) {
+      settings.enabledPlugins = settings.enabledPlugins.filter((p) => !CREW_CLAUDE_PLUGIN_KEYS.includes(p));
+    } else if (settings.enabledPlugins) {
+      for (const key of CREW_CLAUDE_PLUGIN_KEYS) delete settings.enabledPlugins[key];
+    }
+    if (settings.permissions?.allow) {
+      settings.permissions.allow = settings.permissions.allow.filter((rule) => typeof rule !== 'string'
+        || !CREW_CLAUDE_PERMISSION_PREFIXES.some((prefix) => rule.startsWith(prefix)));
+    }
+    if (crewOwnedStatusLine(settings.statusLine)) {
+      delete settings.statusLine;
+      actions.push('statusLine: removed (Crew-installed)');
+    }
+    writeFileSync(settingsFile, JSON.stringify(settings, null, 2) + '\n');
+    actions.push('unregistered from settings.json (backup kept)');
+  } else {
+    actions.push('settings.json not found');
   }
-  writeFileSync(settingsFile, JSON.stringify(settings, null, 2) + '\n');
-  return { ok: true, actions: ['unregistered from settings.json (backup kept)'] };
+  removeCrewClaudeCacheRecords({ home, actions });
+  return { ok: true, actions };
 }

package/src/install/npx-lifecycle.mjs

@@ -3327,6 +3327,15 @@ export function npxStatus({
   }
   log(`DSH plugin: ${dshPlugin} (dedicated dsh-crew profile on 3210)`);
   log(`Official 3080 UI bridge: ${officialWeb}`);
+  // The frontend is not an install that can be missing: 3210 is the service, and
+  // 3080 is started on demand. Saying so is the difference between "this machine is
+  // incomplete" and "run `dsh-crew open` when you want the browser surface" — and
+  // whether 3080 happens to be up right now does not change either answer.
+  const frontendConfig = (installer.readGlobalConfig ?? realInstaller.readGlobalConfig)({ configFile: join(home, '.config', 'dsh-crew', 'config.json') });
+  const frontend = frontendConfig?.frontend_autostart === true
+    ? 'configured to auto-start at login (dsh-crew open for a session URL)'
+    : 'available on demand (dsh-crew open)';
+  log(`Frontend (3080): ${frontend}`);
   // The desktop app carries its own bridge entry, which pins a revision: after a
   // payload update that entry is stale until it is re-pointed, so say which of the
   // three states the machine is in instead of leaving the panel silently old.
@@ -3349,6 +3358,7 @@ export function npxStatus({
     installedPath: pointer?.path ?? null,
     dshPlugin,
     officialWeb,
+    frontend,
     codex,
     zcode,
     claude,

package/src/install/official-frontend-assets.mjs

@@ -19,9 +19,10 @@ function plan(home, root) {
   // `core.autocrlf=true` checkout hashes to a different revision than the npm
   // payload it installs, which made readiness look for a snapshot that does not
   // exist and reported "needs repair" on a correct install (and made every payload
-  // update look like a stale desktop revision).
+  // update look like a stale desktop revision). Lone CR collapses too: one line
+  // break is one line break in these sources.
   const canonicalBytes = (bytes) => (bytes.includes(0) ? bytes
-    : Buffer.from(bytes.toString('utf8').replace(/\r\n/g, '\n'), 'utf8'));
+    : Buffer.from(bytes.toString('utf8').replace(/\r\n?/g, '\n'), 'utf8'));
   for (const [name, bytes] of files) files.set(name, canonicalBytes(bytes));
   const hash = createHash('sha256').update(JSON.stringify(metadata));
   for (const [name, bytes] of files) hash.update(name).update(bytes);
@@ -43,10 +44,12 @@ function plan(home, root) {
 // snapshot was written from the npm payload (LF). The bridge sources are text that
 // either line ending serves, so a CRLF checkout must not read as a drifted snapshot;
 // bytes are still compared first, and the tolerant path only applies to text.
+// Lone CR is normalized as well, so a file saved with classic-Mac endings reads as
+// the same source rather than as content drift.
 function sameTextContent(installed, expected) {
   if (installed.equals(expected)) return true;
   if (installed.includes(0) || expected.includes(0)) return false;
-  const normalize = (buffer) => buffer.toString('utf8').replace(/\r\n/g, '\n');
+  const normalize = (buffer) => buffer.toString('utf8').replace(/\r\n?/g, '\n');
   return normalize(installed) === normalize(expected);
 }
 
@@ -61,7 +64,7 @@ export function officialFrontendAssetsReady({ home = homedir(), root } = {}) {
   try {
     const p = plan(home, root);
     const overlay = readFileSync(p.overlayFile, 'utf8');
-    return matches(p) && (overlay === p.overlay || overlay.replace(/\r\n/g, '\n') === p.overlay.replace(/\r\n/g, '\n'));
+    return matches(p) && (overlay === p.overlay || overlay.replace(/\r\n?/g, '\n') === p.overlay.replace(/\r\n?/g, '\n'));
   } catch { return false; }
 }
 

package/src/install/windows-startup.mjs

@@ -129,9 +129,12 @@ function sha256File(file) {
 // Git checks these scripts out as CRLF on a `core.autocrlf=true` machine and
 // PowerShell accepts either, so a line-ending difference is not a difference this
 // check may report: comparing raw bytes made every Windows dev checkout read as
-// "needs repair" while the npm payload (LF) was installed correctly.
+// "needs repair" while the npm payload (LF) was installed correctly. Lone CR is
+// collapsed too — an editor that saved one of these files with classic-Mac
+// endings is still the same script, and only a real character difference should
+// send the operator to a repair step.
 function normalizeEol(text) {
-  return String(text).replace(/\r\n/g, '\n');
+  return String(text).replace(/\r\n?/g, '\n');
 }
 
 function samePath(left, right) {

package/src/log-prune.mjs

@@ -10,14 +10,19 @@ import { readdirSync, rmSync, statSync } from 'node:fs';
 import { join } from 'node:path';
 import { tmpdir } from 'node:os';
 
-export const DEFAULT_KEEP_RUNS = 20;
+export const DEFAULT_KEEP_RUNS = 10;
 export const DEFAULT_MAX_AGE_DAYS = 14;
 const RECENT_GUARD_MS = 5 * 60 * 1000;
 
+// Per family. The hub-start pairs are what a crash-before-readiness leaves behind,
+// and ten of them are the post-mortem window the policy names; the frontend families
+// are launched far less often and keep the retention they already had, so one
+// family's bound is not silently imposed on the others. An explicit `keepRuns`
+// still applies to every family: that is an operator stating how many runs to keep.
 const LOG_FAMILIES = [
-  { prefix: 'dsh-crew-dsh-crew-3210-', suffix: '.out.log' },
-  { prefix: 'dsh-crew-web-', suffix: '.out.log' },
-  { prefix: 'dsh-official-web-', suffix: '.out.log' },
+  { prefix: 'dsh-crew-dsh-crew-3210-', suffix: '.out.log', keep: 10 },
+  { prefix: 'dsh-crew-web-', suffix: '.out.log', keep: 20 },
+  { prefix: 'dsh-official-web-', suffix: '.out.log', keep: 20 },
 ];
 
 // `dsh-crew-<profile>-<port>-<stamp>.out.log` -> the stamp identifies one run, and
@@ -28,8 +33,11 @@ function runStamp(fileName, family) {
   return stamp || null;
 }
 
-export function pruneCrewTempLogs({ tempDir = tmpdir(), keepRuns = DEFAULT_KEEP_RUNS, maxAgeDays = DEFAULT_MAX_AGE_DAYS, now = Date.now() } = {}) {
-  const keep = Number.isInteger(keepRuns) && keepRuns > 0 ? keepRuns : DEFAULT_KEEP_RUNS;
+export function pruneCrewTempLogs({ tempDir = tmpdir(), keepRuns = null, maxAgeDays = DEFAULT_MAX_AGE_DAYS, now = Date.now() } = {}) {
+  // `keepRuns` is an explicit operator override for every family; without one each
+  // family uses its own bound, so the default is "nothing was asked for" rather than
+  // a number that would flatten them all to the hub's ten.
+  const keepOverride = Number.isInteger(keepRuns) && keepRuns > 0 ? keepRuns : null;
   const maxAgeMs = Number.isFinite(maxAgeDays) && maxAgeDays > 0 ? maxAgeDays * 24 * 60 * 60 * 1000 : DEFAULT_MAX_AGE_DAYS * 24 * 60 * 60 * 1000;
   let names;
   try { names = readdirSync(tempDir); } catch { return { ok: true, removed: [], kept: [], skipped: 'unreadable' }; }
@@ -50,10 +58,16 @@ export function pruneCrewTempLogs({ tempDir = tmpdir(), keepRuns = DEFAULT_KEEP_
       runs.set(stamp, run);
     }
     const ordered = [...runs.values()].sort((left, right) => right.newest - left.newest);
+    const keep = keepOverride ?? family.keep;
     ordered.forEach((run, index) => {
       const tooOld = now - run.newest > maxAgeMs;
       const tooMany = index >= keep;
       const tooRecentToTouch = now - run.newest < RECENT_GUARD_MS;
+      // Delete when EITHER bound is exceeded. The newest `keep` runs are what an
+      // operator reads, and past that a run beyond the age bound is gone regardless
+      // of its rank: keeping an eleventh run merely because it is only hours old
+      // would let a burst of starts grow without bound, which is the growth this
+      // bound exists to stop. A burst of eleven young runs still keeps ten.
       if ((tooOld || tooMany) && !tooRecentToTouch) {
         for (const name of run.files) removed.push(name);
         const errName = run.files[0].replace(/\.out\.log$/, '.err.log');

package/src/readiness-matrix.mjs

@@ -5,7 +5,18 @@
 // (Hub handshake / catalog read) are intentionally separate from execution
 // verification (real worker, reviewer, cancellation, timeout, etc.).
 
-const READINESS_STATUSES = Object.freeze(['PASS', 'FAIL', 'BLOCKED', 'SKIP', 'NOT_RUN']);
+const READINESS_STATUSES = Object.freeze(['PASS', 'FAIL', 'BLOCKED', 'SKIP', 'NOT_APPLICABLE', 'NOT_RUN']);
+// Statuses a trusted evidence source may report. `NOT_APPLICABLE` is deliberately
+// absent: applicability is a fact about this machine's policy, decided when the row
+// is built, and letting evidence declare a required row "not applicable" would be a
+// way to make a check disappear while the matrix still reads healthy. Evidence can
+// neither create nor clear it.
+const EVIDENCE_STATUSES = Object.freeze(['PASS', 'FAIL', 'BLOCKED', 'SKIP', 'NOT_RUN']);
+// The rows this release may decide are not applicable, and only when the reason is
+// policy rather than observation: the two built-in DeepSeek routes exist to name a
+// provider the operator may not be using. Every other row asks a question that is
+// always applicable to a Crew machine.
+const POLICY_NOT_APPLICABLE_ROWS = Object.freeze(['deepseek_flash', 'deepseek_pro']);
 
 export const READINESS_REASON_CODES = Object.freeze({
   LIVE_CHECK_PASSED: 'LIVE_CHECK_PASSED',
@@ -22,7 +33,7 @@ export const READINESS_REASON_CODES = Object.freeze({
   NO_CI_EVIDENCE: 'NO_CI_EVIDENCE',
   NO_EXECUTION_EVIDENCE: 'NO_EXECUTION_EVIDENCE',
   CREDENTIAL_STATUS_NOT_PROBED: 'CREDENTIAL_STATUS_NOT_PROBED',
-  ROUTE_NOT_SELECTED: 'ROUTE_NOT_SELECTED',
+  WORKER_PROVIDER_FOLLOWS_DSH: 'WORKER_PROVIDER_FOLLOWS_DSH',
   EVIDENCE_REPORTED: 'EVIDENCE_REPORTED',
 });
 
@@ -59,7 +70,15 @@ function baseRow(id, category, status, reasonCode, evidenceSource = 'none', extr
 
 function normalizeEvidence(row, evidence) {
   if (!evidence || typeof evidence !== 'object') return row;
-  if (!READINESS_STATUSES.includes(evidence.status)) return row;
+  if (!EVIDENCE_STATUSES.includes(evidence.status)) return row;
+  // A row the policy marked not applicable keeps that status: reported evidence says
+  // something happened, not that the current worker route changed, and letting it
+  // flip the row is how "this route is unused" would turn into "this route passed".
+  // What was reported is kept as metadata rather than thrown away.
+  if (row.status === 'NOT_APPLICABLE') {
+    if (typeof evidence.status !== 'string') return row;
+    return { ...row, reported_evidence: { status: evidence.status, ...(typeof evidence.reason_code === 'string' && evidence.reason_code.trim() ? { reason_code: evidence.reason_code.trim() } : {}), ...(typeof evidence.evidence_source === 'string' && evidence.evidence_source.trim() ? { evidence_source: evidence.evidence_source.trim() } : {}) } };
+  }
   const reason = typeof evidence.reason_code === 'string' && evidence.reason_code.trim()
     ? evidence.reason_code.trim()
     : READINESS_REASON_CODES.EVIDENCE_REPORTED;
@@ -166,15 +185,19 @@ export function buildReadinessMatrix({
   // The two built-in DeepSeek rows describe a route only when the operator runs
   // it. Under follow-dsh with a KNOWN, other selected provider they can never
   // gather evidence, and NOT_RUN reads like a check that should have run rather
-  // than a route that was never chosen. An unknown selection stays NOT_RUN: this
-  // matrix is conservative, and "we could not tell" must not read as "not
-  // applicable". Reported evidence still wins, because it is applied below.
+  // than a route that was never chosen. They are reported NOT_APPLICABLE: the rows
+  // stay in the matrix (dropping them would hide that the question was asked and
+  // answered), and neither status can move the aggregate — only a FAIL, an
+  // UNPROVEN row or an UNAVAILABLE component does. An unknown selection stays
+  // NOT_RUN: this matrix is conservative, and "we could not tell" must not read as
+  // "not applicable". This policy decision is also final for the row: evidence
+  // applied below records what it saw without changing the status back.
   const selectionKnown = typeof workerSelection?.provider === 'string' && workerSelection.provider.length > 0;
   const deepseekRouteSelected = workerProviderMode === 'deepseek-official'
     || workerSelection?.provider === 'deepseek-official';
   if (workerProviderMode !== null && selectionKnown && !deepseekRouteSelected) {
-    for (const id of ['deepseek_flash', 'deepseek_pro']) {
-      rows[id] = baseRow(id, 'real-execution', 'SKIP', READINESS_REASON_CODES.ROUTE_NOT_SELECTED, 'runtime-policy');
+    for (const id of POLICY_NOT_APPLICABLE_ROWS) {
+      rows[id] = baseRow(id, 'real-execution', 'NOT_APPLICABLE', READINESS_REASON_CODES.WORKER_PROVIDER_FOLLOWS_DSH, 'runtime-policy');
     }
   }
 
@@ -189,7 +212,10 @@ export function buildReadinessMatrix({
   ]));
 
   return {
-    schema_version: 1,
+    // A row can now legally be NOT_APPLICABLE and the summary carries a matching key,
+    // so the vocabulary a strict consumer must accept changed: that is what the
+    // version means here, not a change in the rules that produce PASS.
+    schema_version: 2,
     platform,
     conservative: true,
     summary: counts,

package/src/runtime-identity.mjs

@@ -36,7 +36,7 @@ export {
 // included in the identity contract.
 const RUNTIME_ID = randomUUID();
 
-export const RUNTIME_VERSION = '2.2.6';
+export const RUNTIME_VERSION = '2.2.8';
 export const HUB_PROTOCOL_VERSION = 1;
 
 export const HUB_CAPABILITIES = Object.freeze([

package/src/runtime-readiness-snapshot.mjs

@@ -108,9 +108,15 @@ function projectModelRole({ role, selected, health, jobs, runtime, now, executio
   if (healthCurrent && matchingHealth.state === 'callable') {
     return { state: 'CALLABLE', reason_code: matchingHealth.reason_code ?? 'PROVIDER_CALLABLE', selected, observed_at: matchingHealth.observed_at ?? null, expires_at: expiresAt, source: 'provider_health', last_success: null };
   }
+  // What proves the route ran is a provider response, not a passing task contract:
+  // a job that reached `done` with its execution completed has demonstrably called
+  // the selected model even when the task itself came back `partial`. Failed and
+  // cancelled work never reaches this predicate, and neither does a job that errored
+  // before the model answered (its execution_status is `failed`, not `completed`).
+  const responded = (job) => job?.task_status === 'success' || job?.execution_status === 'completed';
   const completed = (Array.isArray(jobs) ? jobs : [])
     .filter((job) => job?.role === role && job?.provider === selected.provider && job?.model === selected.model
-      && job?.status === 'done' && job?.task_status === 'success'
+      && job?.status === 'done' && responded(job)
       && sameCompleteRuntimeIdentity(job.execution_context, runtime))
     .map((job) => ({ job, endedAt: Date.parse(job.endedAt ?? '') }))
     .filter(({ endedAt }) => Number.isFinite(endedAt))

package/windows/start-dsh-crew.cmd

@@ -32,6 +32,7 @@ if not "%~1"=="" goto :invalid_argument
 set "LAUNCH_HELPER=%LAUNCH_DIR%start-dsh-crew.ps1"
 set "LAUNCH_LOG=%TEMP%\dsh-crew-launcher.log"
 if not exist "%LAUNCH_HELPER%" (
+  call :rotate_launcher_log
   >>"%LAUNCH_LOG%" echo [%date% %time%] ERROR Managed launcher helper is missing: %LAUNCH_HELPER%
   echo ERROR: DSH Crew launcher helper is missing.
   echo Repair it with: dsh-crew update
@@ -51,6 +52,21 @@ echo ERROR: Unsupported launcher arguments: %LAUNCH_REQUEST%
 echo Use --open, --background, or --watch.
 exit /b 64
 
+:rotate_launcher_log
+rem The PowerShell launcher bounds this log at 5 MiB before its first line, but the
+rem emergency path that calls this is what runs when that helper is MISSING — so a
+rem bounded log cannot depend on the helper whose absence is one of the failures the
+rem log exists to diagnose. Same cap, same two generations, best-effort: a locked or
+rem unreadable log is left alone rather than failing the launch.
+set "LAUNCH_LOG_SIZE="
+for %%A in ("%LAUNCH_LOG%") do set "LAUNCH_LOG_SIZE=%%~zA"
+if not defined LAUNCH_LOG_SIZE goto :eof
+if %LAUNCH_LOG_SIZE% LSS 5242880 goto :eof
+if exist "%LAUNCH_LOG%.2" del /f /q "%LAUNCH_LOG%.2" >nul 2>&1
+if exist "%LAUNCH_LOG%.1" move /y "%LAUNCH_LOG%.1" "%LAUNCH_LOG%.2" >nul 2>&1
+move /y "%LAUNCH_LOG%" "%LAUNCH_LOG%.1" >nul 2>&1
+goto :eof
+
 :help
 echo Usage: %~nx0 [--open ^| --background ^| --watch]
 echo   --open        Open official Harness on 3080 and return once Crew is supervised;

package/windows/start-dsh-crew.ps1

@@ -127,6 +127,24 @@ function Write-LaunchLog {
   }
 }
 
+# Every launch appends to one file for the life of the machine, so it grows without
+# bound while only its tail is ever read. Rotation runs once per launch before the
+# first line is written and keeps two generations: a post-mortem needs the current
+# run and the one before it. Deliberately best-effort — a locked, unreadable or
+# huge log is a reason to leave it alone, never a reason to fail the launch.
+$launcherLogMaxBytes = 5MB
+function Rotate-LaunchLog {
+  try {
+    if (-not (Test-Path -LiteralPath $launcherLog -PathType Leaf)) { return }
+    if ((Get-Item -LiteralPath $launcherLog -ErrorAction Stop).Length -lt $launcherLogMaxBytes) { return }
+    $oldest = "$launcherLog.2"
+    $previous = "$launcherLog.1"
+    if (Test-Path -LiteralPath $oldest) { Remove-Item -LiteralPath $oldest -Force -ErrorAction Stop }
+    if (Test-Path -LiteralPath $previous) { Move-Item -LiteralPath $previous -Destination $oldest -Force -ErrorAction Stop }
+    Move-Item -LiteralPath $launcherLog -Destination $previous -Force -ErrorAction Stop
+  } catch { }
+}
+
 function Resolve-OfficialHarnessCommand {
   $shim = Get-Command dsh.cmd -CommandType Application -ErrorAction Stop | Select-Object -First 1
   $root = Join-Path (Split-Path -Parent $shim.Source) 'node_modules\@deepseek-ai\dsh'
@@ -1528,6 +1546,7 @@ if ($env:DSH_CREW_LAUNCHER_TEST_IMPORT -eq '1') { return }
 
 try {
   New-Item -ItemType Directory -Path $logRoot -Force | Out-Null
+  Rotate-LaunchLog
   Write-LaunchLog ('Launcher started; mode={0}; user={1}' -f $Mode, $env:USERNAME)
   if (-not (Test-Path -LiteralPath $dshCli -PathType Leaf)) {
     throw "DSH CLI was not found at $dshCli. Configure DSH_CREW_DSH_CLI to a Crew-owned official CLI entry, or run: npm install -g @ran-sh/dsh-crew@latest; dsh-crew update"