mixdog 1.0.1 → 1.0.2

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "mixdog",
-  "version": "1.0.1",
+  "version": "1.0.2",
   "private": false,
   "type": "module",
   "description": "Standalone mixdog coding-agent CLI/TUI workspace.",
@@ -31,7 +31,7 @@
     "!scripts/bench/",
     "!scripts/recall-bench-*.txt",
     "!scripts/*-test.mjs",
-    "!scripts/*.test.mjs",
+    "!scripts/**/*.test.mjs",
     "!scripts/tool-contracts/",
     "!scripts/provider-toolcall/",
     "!scripts/session-transport/",

package/src/defaults/skills/setup/references/actions.md

@@ -64,11 +64,11 @@ Read this when choosing a `setup` mutation.
 | Delete an existing schedule/webhook | `delete_automation` |
 | Enable/disable a schedule/webhook | `set_automation_enabled` |
 | Webhook listener enabled state, port, or domain | `set_webhook_config` |
-| Desktop keep-awake, usage pin, or Computer observe-only state | `set_desktop_settings` |
+| Desktop keep-awake, run in background after window close, usage pin, or Computer observe-only state | `set_desktop_settings` |
 | Desktop theme, display language, side panels, or zoom | `set_appearance` |
 | Register a Project and optionally set its display alias | `save_project` |
 | Unregister a Project without deleting its files | `remove_project` |
-| Revoke one linked device; re-pairing restores access | `revoke_linked_device` |
+| Revoke one linked device by its client id from `status connection`; re-pairing restores access | `revoke_linked_device` |
 | Managed local model context allocation; null restores automatic | `set_local_context` |
 
 ## Routes and workflow
@@ -116,7 +116,10 @@ values through chat.
 ## Built-in features
 
 Read `status features` and distinguish installation, enabled state, and live
-bridge state.
+bridge state. `status features` covers Git, Memory, Office, Code Tidy, Local
+Provider and Web Search plus Browser/Computer bridge activity; voice status and
+the Browser Use / Computer Use installed and enabled markers are read with
+`status desktop`.
 
 - Git, Memory, Office, and Local Provider are install-first capabilities.
 - Git and Office installation run on the Desktop when this conversation is

package/src/defaults/skills/setup/references/surfaces.md

@@ -37,7 +37,8 @@ An attached Desktop or TUI may navigate and return `opened:true`.
 - Settings → Providers: API-key, OAuth, local providers, and usage sign-in.
 - Extensions → Plugin → Git & GitHub: Git/GitHub CLI installation and connection.
 - Settings → Connection: web-app pairing and linked devices.
-- Settings → System: update, keep-awake, and Doctor.
+- Settings → System: update, keep-awake, run in background after the window
+  closes, and Doctor.
 - Projects: Project registration, name, Project Memories, and Common Memory.
 - Workflows: workflow packs, Main and Web Search defaults, and agent editing.
 - Extensions → Plugin: Built-in cards and plugins.
@@ -48,12 +49,15 @@ the user to Desktop.
 
 ## Desktop-backed settings
 
-Setup can read and change Desktop appearance, keep-awake, usage pin,
-Computer observe-only, Browser/Computer/voice installation and toggles,
-Projects, and linked-device revocation through the attached
+Setup can read and change Desktop appearance, keep-awake, run in background,
+usage pin, Computer observe-only, Browser/Computer/voice installation and
+toggles, Projects, and linked-device revocation through the attached
 Desktop. A receipt identifies `scope:desktop-host`; appearance is not applied
 to the paired browser. No Desktop claimant means no confirmed change.
 
+Computer observe-only has no Desktop settings control; setup is its only
+supported route. Usage pin is the pin toggle on the activity rail usage card.
+
 Schedules/webhooks and workflow/agent/skill definitions use their runtime
 actions directly. These do not require clicking their Desktop rails.
 

package/src/runtime/agent/orchestrator/context/role-instructions.mjs

@@ -281,7 +281,6 @@ export function composeSystemPrompt(opts) {
   // pre-tool preamble toward English.
   const environmentParts = [
     opts.sessionStartContext,
-    opts.projectInstructionsContext,
     opts.environmentContext,
     opts.languageContext,
   ].map(trimmedPromptSlice);

package/src/runtime/agent/orchestrator/session/manager/session-prompt-composition.mjs

@@ -86,7 +86,7 @@ export function composeSessionSystem(opts, { profile, providerName, modelName, s
   const languageContext = leadOnly ? _buildLeadLanguageContext() : '';
   const shellEnvironmentContext = buildShellEnvironmentContext(opts, ownerIsAgent, surface.toolsForRouting);
   // Persisted env tail: refreshSessionBp3Environment rebuilds the env block
-  // as [session, project, bp3EnvironmentContext], so the language block must
+  // as [session, bp3EnvironmentContext], so the language block must
   // already sit at the end of this string to stay last after a refresh.
   const environmentTailContext = [shellEnvironmentContext, languageContext].filter(Boolean).join('\n\n---\n\n');
   const composed = composeSystemPrompt({

package/src/session-runtime/cwd-plugins/plugin-status.mjs

@@ -1,6 +1,7 @@
 // cwd-plugins/plugin-status.mjs — the registered-plugin status list with its
 // per-root caches (manifest keyed on mtime, MCP discovery and skill-file count
 // on a short TTL).
+import { readFileSync } from 'node:fs';
 import { discoverPluginMcp } from '../../runtime/agent/orchestrator/runtime-core/plugin-mcp.mjs';
 import { pluginMetadata } from '../../runtime/shared/plugin-metadata.mjs';
 
@@ -35,11 +36,25 @@ export function createPluginStatus({
     }
     return key;
   }
+  // A manifest file that exists but is not a JSON object is breakage.
+  function manifestInvalid(root) {
+    for (const rel of ['.codex-plugin/plugin.json', 'plugin.json']) {
+      const file = resolve(root, rel);
+      if (!existsSync(file)) continue;
+      try {
+        const parsed = JSON.parse(readFileSync(file, 'utf8'));
+        if (!parsed || typeof parsed !== 'object' || Array.isArray(parsed)) return true;
+      } catch {
+        return true;
+      }
+    }
+    return false;
+  }
   function cachedPluginData(root) {
     const key = manifestMtimeKey(root);
     const hit = pluginRootCache.get(root);
     if (hit && hit.key === key) return hit;
-    const entry = { key, manifest: pluginManifest(root) };
+    const entry = { key, manifest: pluginManifest(root), invalid: manifestInvalid(root) };
     pluginRootCache.set(root, entry);
     return entry;
   }
@@ -65,8 +80,9 @@ export function createPluginStatus({
 
   function registeredPlugin(entry, configuredMcp) {
     const root = clean(entry.root);
-    if (!root || !existsSync(root)) return null;
-    const manifest = cachedPluginData(root).manifest;
+    if (!root) return null;
+    if (!existsSync(root)) return { missing: true, id: clean(entry.id) || clean(entry.name) || root, name: clean(entry.name) || clean(entry.id) || root.split(/[\\/]/).pop() || root };
+    const { manifest, invalid } = cachedPluginData(root);
     const name = clean(manifest.name) || clean(manifest.id) || clean(entry.name) || root.split(/[\\/]/).pop() || root;
     const { mcpScript, mcpInline } = cachedMcpDiscovery(root);
     const plugin = {
@@ -89,6 +105,7 @@ export function createPluginStatus({
       mcpScript,
       mcpInline,
     };
+    if (invalid) plugin.invalid = true;
     plugin.mcpServerName = pluginMcpServerName(plugin);
     plugin.mcpEnabled =
       Object.hasOwn(configuredMcp, plugin.mcpServerName) ||
@@ -101,9 +118,11 @@ export function createPluginStatus({
     const dataDir = cfgMod.getPluginData?.();
     const configuredMcp = config?.mcpServers && typeof config.mcpServers === 'object' ? config.mcpServers : {};
     const plugins = [];
+    const missing = [];
     for (const entry of listRegisteredPlugins({ dataDir })) {
       const plugin = registeredPlugin(entry, configuredMcp);
-      if (plugin) plugins.push(plugin);
+      if (plugin?.missing) missing.push({ id: plugin.id, name: plugin.name, enabled: entry.enabled !== false });
+      else if (plugin) plugins.push(plugin);
     }
     plugins.sort((a, b) => {
       if (a.source !== b.source) return a.source.localeCompare(b.source);
@@ -113,6 +132,7 @@ export function createPluginStatus({
     return {
       count: plugins.length,
       plugins,
+      missing,
       roots: {
         registry: admin.registryPath,
         installed: admin.installRoot,

package/src/session-runtime/setup-tool/tool-defs.mjs

@@ -204,7 +204,7 @@ export const SETUP_TOOL_DEFS = Object.freeze([
         },
         name: {
           type: 'string',
-          description: `set_builtin_enabled: ${SETUP_BUILTIN_TOGGLE_FEATURES.join('|')}; install_builtin also accepts memory. Memory toggles use set_memory_enabled. set_first_use_approval: browser|computer; else MCP/plugin/provider id.`,
+          description: `set_builtin_enabled: ${SETUP_BUILTIN_TOGGLE_FEATURES.join('|')}; install_builtin also accepts memory. Memory toggles use set_memory_enabled. set_first_use_approval: browser|computer; revoke_linked_device: client id from status connection, not its display name; else MCP/plugin/provider id.`,
         },
         modelId: {
           type: 'string',

package/src/session-runtime/workflow-agents-api/agent-route.mjs

@@ -68,8 +68,10 @@ export function createAgentRouteApi(deps) {
     const requested = { ...(next || {}) };
     const stored = agentRouteFromConfig(getConfig(), id) || {};
     // Off is an explicit state, stored apart from the route so the model the
-    // user picked survives and comes back when the agent is switched on.
-    if (requested.disabled === true) {
+    // user picked survives and comes back when the agent is switched on. An
+    // off request that also carries a model saves that model and stays off.
+    const keepOff = requested.disabled === true;
+    if (keepOff && !(clean(requested.provider) && clean(requested.model))) {
       saveDisabled(id, true);
       return { ...stored, id, disabled: true };
     }
@@ -77,6 +79,7 @@ export function createAgentRouteApi(deps) {
       saveDisabled(id, false);
       return { ...stored, id, disabled: false, inherited: !stored.provider };
     }
+    delete requested.disabled;
     // Tuning-only edits must preserve the selected agent model, not silently inherit Main.
     const current = stored.provider ? stored : normalizeWorkflowRoute(resolveRoute(getConfig(), {})) || {};
     if (!hasOwn(requested, 'provider')) requested.provider = current.provider;
@@ -87,11 +90,11 @@ export function createAgentRouteApi(deps) {
     if (!clean(requested.model)) throw new Error('agent route requires provider and model');
     const routeToSave = normalizeWorkflowRoute(await resolveSelectedRoute(requested, stored));
     if (!routeToSave) throw new Error('agent route requires provider and model');
-    // Picking a model is also the "on" switch — the two states are exclusive.
-    const nextConfig = withAgentDisabled(getConfig(), id, false);
+    // Picking a model is also the "on" switch unless the request keeps it off.
+    const nextConfig = withAgentDisabled(getConfig(), id, keepOff);
     nextConfig.agents = { ...(nextConfig.agents || {}), [id]: routeToSave };
     saveConfigAndAdopt(canonicalizeAgentRouteStorage(nextConfig));
-    return routeToSave;
+    return keepOff ? { ...routeToSave, id, disabled: true } : routeToSave;
   }
 
   return { setAgentRoute };

package/src/tui/app/doctor.mjs

@@ -15,6 +15,7 @@
 import { compareSemver } from '../../runtime/shared/update-checker.mjs';
 import { resolvePluginData } from '../../runtime/shared/plugin-paths.mjs';
 import { hasEnabledAutomation } from '../../runtime/shared/automation-presence.mjs';
+import { builtinFeatureActive } from '../../runtime/agent/orchestrator/runtime-core/builtin-features.mjs';
 import { providerRowUsable } from './provider-usable.mjs';
 import { constants as fsConstants, readFileSync } from 'node:fs';
 import { access, readdir, readFile } from 'node:fs/promises';
@@ -247,16 +248,78 @@ function reportRegistry(label) {
     const broken = active.filter(
       (entry) => entry.broken || entry.error || entry.invalid || entry.dependencyIssues?.length
     );
+    const missing = Array.isArray(status.missing)
+      ? status.missing.filter((entry) => entry.enabled !== false)
+      : [];
     let detail = `${active.length}/${entries.length} active${scopeDetail}`;
-    if (!broken.length) {
+    const issues = [...broken, ...(label === 'plugins' ? missing : [])];
+    if (!issues.length) {
       row('ok', detail);
       return;
     }
-    detail += ` · issues: ${broken.map((entry) => entry.name || entry.id).join(', ')}`;
+    detail += ` · issues: ${issues.map((entry) => entry.name || entry.id).join(', ')}`;
     row('warn', detail, { command: `/${label}` });
   };
 }
 
+function reportLocalProvider(local, row) {
+  if (typeof local.installed !== 'boolean') {
+    row('warn', 'status unavailable');
+    return;
+  }
+  if (!local.installed) {
+    row('ok', 'not installed');
+    return;
+  }
+  if (local.enabled !== true) {
+    row('ok', 'installed · disabled');
+    return;
+  }
+  const fix = { command: '/providers' };
+  const models = Array.isArray(local.models) ? local.models.filter((model) => model.installed === true).length : 0;
+  const jobs = Array.isArray(local.installations) ? local.installations : [];
+  const latest = jobs.reduce((a, b) => (a && (a.updatedAt || 0) > (b.updatedAt || 0) ? a : b), null);
+  if (latest?.state === 'failed') {
+    row('warn', `installed · enabled · ${models} models · latest installation failed`, fix);
+  } else if (!models) {
+    row('warn', 'installed · enabled · no model installed', fix);
+  } else row('ok', `installed · enabled · ${models} models installed`);
+}
+
+function reportBuiltins(settings, row) {
+  const groups = { enabled: [], disabled: [], 'not installed': [] };
+  for (const id of ['git', 'office', 'tidy']) {
+    const entry = settings[id];
+    if (!entry || typeof entry.installed !== 'boolean') {
+      row('warn', 'status unavailable');
+      return;
+    }
+    if (!entry.installed) groups['not installed'].push(id);
+    else groups[entry.enabled === false ? 'disabled' : 'enabled'].push(id);
+  }
+  const parts = Object.entries(groups)
+    .filter(([, ids]) => ids.length)
+    .map(([state, ids]) => `${state}: ${ids.join(', ')}`);
+  const bridge = (id) => (builtinFeatureActive(undefined, id) ? 'bridge active' : 'no bridge');
+  row('ok', `${parts.join(' · ')} · browser use ${bridge('browser')} · computer use ${bridge('computer')}`);
+}
+
+function reportVoice(voice, row) {
+  if (typeof voice.enabled !== 'boolean') {
+    row('warn', 'status unavailable');
+    return;
+  }
+  if (!voice.enabled) {
+    row('ok', 'disabled');
+    return;
+  }
+  if (voice.installed !== true || voice.components?.model === false) {
+    row('warn', 'enabled · runtime or model not installed', { hint: 'reinstall voice from Extensions' });
+    return;
+  }
+  row('ok', 'enabled · runtime and model installed');
+}
+
 function reportHooks(hooks, row) {
   if (
     typeof hooks.enabled !== 'boolean' ||
@@ -333,6 +396,19 @@ export async function runDoctorChecks(runtime = {}, getState = () => ({}), optio
     ['providers', 'providers', () => runtime.getProviderSetup?.(), reportProviders(getState)],
     ['mcp', 'mcp', () => runtime.mcpStatus?.(), reportMcp],
     ['memory', 'memory', async () => (await runtime.getToolModuleSettings?.())?.memory, reportMemory(runtime)],
+    [
+      'localProvider',
+      'local provider',
+      async () => (await runtime.getToolModuleSettings?.())?.localProvider,
+      reportLocalProvider,
+    ],
+    ['builtins', 'built-ins', async () => runtime.getToolModuleSettings?.(), reportBuiltins],
+    [
+      'voice',
+      'voice',
+      async () => (runtime.getVoiceStatus || (await import('../lib/voice-setup.mjs')).getVoiceStatus)(),
+      reportVoice,
+    ],
     [
       'channels',
       'channels',

package/scripts/lib/bench-sandbox.test.mjs

@@ -1,54 +0,0 @@
-import assert from 'node:assert/strict';
-import { existsSync, mkdirSync, mkdtempSync, readFileSync, rmSync, writeFileSync } from 'node:fs';
-import { tmpdir } from 'node:os';
-import { join } from 'node:path';
-import test from 'node:test';
-import {
-  authArtifactNamesForSandbox,
-  copyAuthArtifacts,
-  extractSessionId,
-  readUnifiedConfig,
-} from './bench-sandbox.mjs';
-
-function withDir(run) {
-  const dir = mkdtempSync(join(tmpdir(), 'bench-sandbox-'));
-  try {
-    return run(dir);
-  } finally {
-    rmSync(dir, { recursive: true, force: true });
-  }
-}
-
-test('extractSessionId prefers the labelled id, then any sess_ token', () => {
-  assert.equal(extractSessionId('sessionId: sess_abc123 other sess_zzz'), 'sess_abc123');
-  assert.equal(extractSessionId('noise sess_q9 end'), 'sess_q9');
-  assert.equal(extractSessionId('nothing here'), null);
-  assert.equal(extractSessionId(undefined), null);
-});
-
-test('readUnifiedConfig returns an object, and {} for a missing or non-object file', () => {
-  withDir((dir) => {
-    assert.deepEqual(readUnifiedConfig(dir), {});
-    writeFileSync(join(dir, 'mixdog-config.json'), '{"a":1}');
-    assert.deepEqual(readUnifiedConfig(dir), { a: 1 });
-    writeFileSync(join(dir, 'mixdog-config.json'), 'null');
-    assert.deepEqual(readUnifiedConfig(dir), {});
-  });
-});
-
-test('auth artifacts: provider files plus every oauth/credentials json in the real dir', () => {
-  withDir((real) => {
-    writeFileSync(join(real, 'grok-oauth.json'), 'g');
-    writeFileSync(join(real, 'extra-credentials.json'), 'c');
-    writeFileSync(join(real, 'unrelated.json'), 'u');
-    const names = authArtifactNamesForSandbox(real, 'grok-oauth').sort();
-    assert.deepEqual(names, ['extra-credentials.json', 'grok-oauth-models.json', 'grok-oauth.json']);
-    const sandbox = join(real, 'sandbox');
-    mkdirSync(sandbox);
-    const { copied, skipped } = copyAuthArtifacts(real, sandbox, 'grok-oauth');
-    assert.deepEqual(copied.sort(), ['extra-credentials.json', 'grok-oauth.json']);
-    assert.deepEqual(skipped, ['grok-oauth-models.json']);
-    assert.equal(readFileSync(join(sandbox, 'grok-oauth.json'), 'utf8'), 'g');
-    assert.equal(existsSync(join(sandbox, 'unrelated.json')), false);
-  });
-});

package/scripts/lib/cli-args.test.mjs

@@ -1,64 +0,0 @@
-import assert from 'node:assert/strict';
-import test from 'node:test';
-import { argValue, hasFlag, intArg, numArg, positionalArgs } from './cli-args.mjs';
-
-test('argValue reads both the separated and the joined option form', () => {
-  const argv = ['node', 'bench.mjs', '--model', 'opus', '--round=3', '--trailing'];
-  assert.equal(argValue('--model', null, argv), 'opus');
-  assert.equal(argValue('--round', null, argv), '3');
-  assert.equal(argValue('--missing', null, argv), null);
-  assert.equal(argValue('--missing', 'fallback', argv), 'fallback');
-  // An explicit `undefined` fallback takes the parameter default, so a
-  // missing option always reads as null rather than undefined.
-  assert.equal(argValue('--missing', undefined, argv), null);
-});
-
-test('argValue falls back when the separated form has no value left', () => {
-  assert.equal(argValue('--save', 'default', ['node', 'bench.mjs', '--save']), 'default');
-  // An empty joined value is a value, not a missing option.
-  assert.equal(argValue('--save', 'default', ['node', 'bench.mjs', '--save=']), '');
-  // The next argument is taken verbatim, even when it looks like a flag.
-  assert.equal(argValue('--save', null, ['node', 'bench.mjs', '--save', '--json']), '--json');
-});
-
-test('hasFlag reports presence only', () => {
-  const argv = ['node', 'bench.mjs', '--json', '--limit=5'];
-  assert.equal(hasFlag('--json', argv), true);
-  assert.equal(hasFlag('--limit', argv), false);
-  assert.equal(hasFlag('--quiet', argv), false);
-});
-
-test('intArg accepts positive integers and keeps the fallback otherwise', () => {
-  assert.equal(intArg('--limit', 30, ['node', 'diag.mjs', '--limit', '5']), 5);
-  assert.equal(intArg('--limit', 30, ['node', 'diag.mjs', '--limit=12']), 12);
-  assert.equal(intArg('--limit', 30, ['node', 'diag.mjs']), 30);
-  assert.equal(intArg('--limit', 30, ['node', 'diag.mjs', '--limit', 'abc']), 30);
-  assert.equal(intArg('--limit', 30, ['node', 'diag.mjs', '--limit', '0']), 30);
-  assert.equal(intArg('--limit', 30, ['node', 'diag.mjs', '--limit', '-4']), 30);
-  // parseInt stops at the first non-digit, as the CLIs have always accepted.
-  assert.equal(intArg('--limit', 30, ['node', 'diag.mjs', '--limit', '7x']), 7);
-});
-
-test('numArg keeps a deliberate 0 and falls back only on missing or non-numeric values', () => {
-  assert.equal(numArg('--budget', 16000, ['node', 'bench.mjs', '--budget', '0']), 0);
-  assert.equal(numArg('--budget', 16000, ['node', 'bench.mjs', '--budget=0']), 0);
-  assert.equal(numArg('--budget', 16000, ['node', 'bench.mjs', '--budget', '2500']), 2500);
-  assert.equal(numArg('--budget', 16000, ['node', 'bench.mjs', '--budget', '0.5']), 0.5);
-  assert.equal(numArg('--budget', 16000, ['node', 'bench.mjs']), 16000);
-  assert.equal(numArg('--budget', 16000, ['node', 'bench.mjs', '--budget=']), 16000);
-  assert.equal(numArg('--budget', 16000, ['node', 'bench.mjs', '--budget', 'lots']), 16000);
-  assert.equal(numArg('--budget', 16000, ['node', 'bench.mjs', '--budget']), 16000);
-});
-
-test('positionalArgs never lets a boolean flag or a joined value swallow a positional', () => {
-  const booleans = new Set(['--render']);
-  assert.deepEqual(positionalArgs(['--render', 'deck.js', 'deck.pptx', '--mode', 'auto'], booleans), [
-    'deck.js',
-    'deck.pptx',
-  ]);
-  assert.deepEqual(positionalArgs(['--mode=auto', 'deck.js', 'deck.pptx'], booleans), ['deck.js', 'deck.pptx']);
-  assert.deepEqual(positionalArgs(['deck.js', '--out', 'result.json', 'deck.pptx', '--render'], booleans), [
-    'deck.js',
-    'deck.pptx',
-  ]);
-});

package/scripts/lib/embedding-model-bench-core.test.mjs

@@ -1,47 +0,0 @@
-import assert from 'node:assert/strict';
-import test from 'node:test';
-import { selectDeterministicCorpus } from './embedding-model-bench-core.mjs';
-
-const documents = Object.freeze([6, 3, 2, 5, 4, 1, 3].map((id) => Object.freeze({ id })));
-
-test('negative sampling preserves SHA-256 order, document order, and duplicate rows', () => {
-  // SHA-256("4") starts 4b22, then "3" starts 4e07; the other IDs sort after both.
-  const selection = selectDeterministicCorpus(documents, [{ positiveIdsByTarget: [[6]] }], 3);
-  assert.deepEqual(selection, {
-    documents: [{ id: 6 }, { id: 3 }, { id: 4 }, { id: 3 }],
-    sourceDocuments: 7,
-    selectedDocuments: 3,
-    positiveDocuments: 1,
-    method: 'all-positive-roots + sha256(document-id) negatives',
-  });
-});
-
-test('positive caps use the same hash ordering without counting duplicate IDs twice', () => {
-  const selection = selectDeterministicCorpus(documents, [{ positiveIdsByTarget: [[1, 2, 3, 4, 5, 6, 3]] }], 2, 2);
-  assert.deepEqual(selection, {
-    documents: [{ id: 3 }, { id: 4 }, { id: 3 }],
-    sourceDocuments: 7,
-    selectedDocuments: 2,
-    positiveDocuments: 2,
-    method: 'up-to-2-positive-roots-per-target + sha256(document-id) negatives',
-  });
-});
-
-test('full-corpus selections retain the original array and capped positive count', () => {
-  for (const limit of [0, 1.5, 7, 9]) {
-    const selection = selectDeterministicCorpus(
-      documents,
-      [{ positiveIdsByTarget: [[1, 2, 3, 4, 5, 6, 3]] }],
-      limit,
-      2
-    );
-    assert.equal(selection.documents, documents);
-    assert.deepEqual(selection, {
-      documents,
-      sourceDocuments: 7,
-      selectedDocuments: 7,
-      positiveDocuments: 2,
-      method: 'full-corpus',
-    });
-  }
-});

package/scripts/lib/smoke-loop-shared.test.mjs

@@ -1,22 +0,0 @@
-import assert from 'node:assert/strict';
-import test from 'node:test';
-import { parseDuration, summarize } from './smoke-loop-shared.mjs';
-
-test('parseDuration reads bare milliseconds and unit suffixes', () => {
-  assert.equal(parseDuration('1500'), 1500);
-  assert.equal(parseDuration('30s'), 30_000);
-  assert.equal(parseDuration('5M'), 300_000);
-  assert.equal(parseDuration('1.5h'), 5_400_000);
-  assert.equal(parseDuration('0ms'), 1);
-});
-
-test('parseDuration returns the fallback when empty and throws on garbage', () => {
-  assert.equal(parseDuration('', 7), 7);
-  assert.equal(parseDuration(undefined), null);
-  assert.throws(() => parseDuration('soon'), /invalid duration: soon/);
-});
-
-test('summarize rounds min/max/avg to one decimal, zeros when empty', () => {
-  assert.deepEqual(summarize([]), { min: 0, max: 0, avg: 0 });
-  assert.deepEqual(summarize([1.04, 2.06, 3.31]), { min: 1, max: 3.3, avg: 2.1 });
-});

package/scripts/lib/trace-row.test.mjs

@@ -1,47 +0,0 @@
-import assert from 'node:assert/strict';
-import test from 'node:test';
-import { resolve } from 'node:path';
-import { defaultTraceFiles, field, num, payload, sessionId } from './trace-row.mjs';
-
-test('defaultTraceFiles: an explicit path wins, a data dir yields its rotated pair', () => {
-  assert.deepEqual(defaultTraceFiles({ pathArg: 'x.jsonl' }), [resolve('x.jsonl')]);
-  assert.deepEqual(defaultTraceFiles({ dataDir: 'd' }), [
-    resolve('d', 'history', 'agent-trace.jsonl.1'),
-    resolve('d', 'history', 'agent-trace.jsonl'),
-  ]);
-});
-
-test('defaultTraceFiles without arguments covers the project and user data dirs, deduplicated', () => {
-  const files = defaultTraceFiles();
-  assert.equal(files.length, new Set(files).size);
-  assert.ok(files.includes(resolve(process.cwd(), '.mixdog', 'data', 'history', 'agent-trace.jsonl')));
-});
-
-test('payload is the row payload object, else an empty object', () => {
-  assert.deepEqual(payload({ payload: { a: 1 } }), { a: 1 });
-  assert.deepEqual(payload({ payload: 'text' }), {});
-  assert.deepEqual(payload(null), {});
-});
-
-test('field prefers the row itself, falls back to the payload, else null', () => {
-  assert.equal(field({ a: 1, payload: { a: 2 } }, 'a'), 1);
-  assert.equal(field({ payload: { a: 2 } }, 'a'), 2);
-  assert.equal(field({ a: null, payload: { a: 3 } }, 'a'), 3);
-  assert.equal(field({}, 'a'), null);
-  assert.equal(field(undefined, 'a'), null);
-});
-
-test('num coerces the field and drops non-finite values', () => {
-  assert.equal(num({ payload: { n: '12' } }, 'n'), 12);
-  assert.equal(num({ n: 'x' }, 'n'), null);
-  // A missing field reads as Number(null) === 0, not null (existing behavior
-  // of every trace CLI that used its own copy).
-  assert.equal(num({}, 'n'), 0);
-});
-
-test('sessionId reads either spelling, then the payload, as a string', () => {
-  assert.equal(sessionId({ session_id: 's1' }), 's1');
-  assert.equal(sessionId({ sessionId: 's2' }), 's2');
-  assert.equal(sessionId({ payload: { session_id: 's3' } }), 's3');
-  assert.equal(sessionId({}), '');
-});

package/scripts/lib/trace-stats.test.mjs

@@ -1,62 +0,0 @@
-import assert from 'node:assert/strict';
-import test from 'node:test';
-import { distribution, median, percentile, sortedFinite, stats } from './trace-stats.mjs';
-
-test('distribution coerces samples with Number() and reports nearest-rank p50/p95', () => {
-  assert.deepEqual(distribution(['10', 20, '30', 40]), { p50: 20, p95: 40 });
-  assert.deepEqual(distribution([]), { p50: null, p95: null });
-  assert.deepEqual(distribution([undefined, 'x']), { p50: null, p95: null });
-});
-
-test('percentile picks the nearest rank of an ascending sample', () => {
-  const sorted = [10, 20, 30, 40];
-  assert.equal(percentile(sorted, 50), 20);
-  assert.equal(percentile(sorted, 90), 40);
-  assert.equal(percentile(sorted, 99), 40);
-  assert.equal(percentile([5], 50), 5);
-  assert.equal(percentile([], 50), null);
-  // Rank 0 and below clamp to the first sample, never to undefined.
-  assert.equal(percentile(sorted, 0), 10);
-});
-
-test('median is the nearest-rank p50: the lower middle sample, null when empty', () => {
-  assert.equal(median([10, 20, 30, 40]), 20);
-  assert.equal(median([10, 20, 30]), 20);
-  assert.equal(median([7]), 7);
-  assert.equal(median([]), null);
-});
-
-test('percentile is nearest-rank, not floor(n*p) index truncation', () => {
-  // floor(n*p) indexing reports 20 for p50 of [10,20] and 20 for p95 of 1..20.
-  assert.equal(percentile([10, 20], 50), 10);
-  assert.equal(
-    percentile(
-      Array.from({ length: 20 }, (_, i) => i + 1),
-      95
-    ),
-    19
-  );
-});
-
-test('sortedFinite sorts a copy and drops non-finite entries without coercing them', () => {
-  const input = [3, null, '2', Number.NaN, 1, undefined, Number.NEGATIVE_INFINITY];
-  assert.deepEqual(sortedFinite(input), [1, 3]);
-  assert.equal(input[0], 3);
-});
-
-test('stats summarizes finite samples and reports null for an empty set', () => {
-  assert.deepEqual(stats([30, 10, 20]), { n: 3, sum: 60, avg: 20, p50: 20, p90: 30, p99: 30, max: 30 });
-  assert.equal(stats([]), null);
-  assert.equal(stats([Number.NaN, Number.POSITIVE_INFINITY]), null);
-});
-
-test('stats drops non-finite samples and leaves the caller array untouched', () => {
-  const input = [3, Number.NaN, 1, Number.POSITIVE_INFINITY, 2];
-  assert.deepEqual(stats(input), { n: 3, sum: 6, avg: 2, p50: 2, p90: 3, p99: 3, max: 3 });
-  assert.deepEqual(input, [3, Number.NaN, 1, Number.POSITIVE_INFINITY, 2]);
-});
-
-test('stats rounds the average, matching the reported millisecond columns', () => {
-  assert.equal(stats([1, 2]).avg, 2);
-  assert.equal(stats([1, 1, 2]).avg, 1);
-});