mixdog 1.4.1 → 1.4.2

package/src/runtime/tidy/process.mjs

@@ -108,8 +108,9 @@ function captureInto(state, key, chunk, cap) {
 
 /**
  * Spawn `bin` with `args` and collect its output.
- * Resolves `{code, signal, stdout, stderr, timedOut, error}` — a failed spawn is
- * reported, never thrown, so one broken engine cannot abort the multiplexer.
+ * Resolves `{code, signal, stdout, stderr, timedOut, error, errorCode}` — a
+ * failed spawn is reported (`errorCode`: its code, e.g. ENOENT), never thrown,
+ * so one broken engine cannot abort the multiplexer.
  */
 export function runProcess(
   bin,
@@ -151,6 +152,7 @@ export function runProcess(
         timedOut: false,
         truncated: false,
         error: error?.message || String(error),
+        errorCode: error?.code || '',
       });
       return;
     }
@@ -209,7 +211,7 @@ export function runProcess(
 
     // One exit path for both outcomes: whoever settles first reports the
     // captured output and clears the timers.
-    const settle = (code, exitSignal, error) => {
+    const settle = (code, exitSignal, error, errorCode = '') => {
       if (settled) return;
       settled = true;
       clearTimers();
@@ -221,12 +223,13 @@ export function runProcess(
         timedOut,
         truncated: Boolean(state.stdoutTruncated || state.stderrTruncated),
         error,
+        errorCode,
       });
     };
 
     child.stdout?.on('data', (chunk) => captureInto(state, 'stdout', chunk, cap));
     child.stderr?.on('data', (chunk) => captureInto(state, 'stderr', chunk, cap));
-    child.on('error', (error) => settle(-1, null, error?.message || String(error)));
+    child.on('error', (error) => settle(-1, null, error?.message || String(error), error?.code || ''));
     if (input != null && child.stdin) {
       child.stdin.on('error', () => {
         /* child may close stdin early */

package/src/runtime/tidy/resolve.mjs

@@ -135,7 +135,7 @@ function installableFlag(entry, manifest) {
   return entry.managed && manifestAsset(manifest, entry.id) ? { installable: true } : {};
 }
 
-function resolveOne({ cwd, entry, config, env, manifest, pluginData, pathIndex }) {
+async function resolveOne({ cwd, entry, config, env, manifest, pluginData, pathIndex }) {
   const base = {
     id: entry.id,
     kind: entry.kind,
@@ -169,7 +169,8 @@ function resolveOne({ cwd, entry, config, env, manifest, pluginData, pathIndex }
         installHint: entry.hostInstallHint || entry.installHint,
       };
     }
-    const managed = entry.managed && pluginData ? managedEngineBinary({ id: entry.id, manifest, pluginData }) : null;
+    const managed =
+      entry.managed && pluginData ? await managedEngineBinary({ id: entry.id, manifest, pluginData }) : null;
     if (managed) {
       return {
         ...base,
@@ -194,7 +195,7 @@ function resolveOne({ cwd, entry, config, env, manifest, pluginData, pathIndex }
   const onPath = pathBinary(entry, env, pathIndex);
   if (onPath) return { ...base, source: 'path', path: onPath.path, ...commandFor(entry, onPath.path) };
   if (entry.managed && pluginData) {
-    const managed = managedEngineBinary({ id: entry.id, manifest, pluginData });
+    const managed = await managedEngineBinary({ id: entry.id, manifest, pluginData });
     if (managed) {
       return {
         ...base,
@@ -282,20 +283,22 @@ export async function resolveEngines({
   else if (languages.length > 0) requested = enginesForLanguages(languages);
   const unknown = engineIds.filter((id) => !ENGINE_IDS.includes(id));
 
-  // One listing per PATH dir for the whole batch: this runs synchronously on
-  // the host loop (the desktop's Code Tidy status read at startup).
+  // One listing per PATH dir for the whole batch (the desktop's Code Tidy
+  // status read at startup runs this on the host loop).
   const pathIndex = createPathIndex(env);
-  const resolved = requested.map((id) => {
-    const entry = ENGINE_CATALOG[id];
-    const result = resolveOne({ cwd, entry, config, env, manifest: loadedManifest, pluginData, pathIndex });
-    const configFile = projectConfigFileFor(cwd, entry);
-    if (configFile) result.configFile = configFile;
-    if (entry.requiresConfig && !configFile && result.source !== 'project-config') {
-      result.skipped = 'no project config for this engine';
-    }
-    if (entry.toolchain) result.toolchain = true;
-    return result;
-  });
+  const resolved = await Promise.all(
+    requested.map(async (id) => {
+      const entry = ENGINE_CATALOG[id];
+      const result = await resolveOne({ cwd, entry, config, env, manifest: loadedManifest, pluginData, pathIndex });
+      const configFile = projectConfigFileFor(cwd, entry);
+      if (configFile) result.configFile = configFile;
+      if (entry.requiresConfig && !configFile && result.source !== 'project-config') {
+        result.skipped = 'no project config for this engine';
+      }
+      if (entry.toolchain) result.toolchain = true;
+      return result;
+    })
+  );
 
   // A project driving Prettier/ESLint from node_modules/.bin keeps them: Biome
   // stands down rather than reformatting against the project's own config.

package/src/runtime/tidy/runners/biome.mjs

@@ -1,5 +1,4 @@
 // Biome — JSON reporter for check, `--write` (1.x: `--apply`) for fix.
-import { availableParallelism } from 'node:os';
 import { mapLimit, runProcess } from '../process.mjs';
 import {
   FILES_PER_SPAWN,
@@ -19,8 +18,9 @@ export const BIOME_CHECK_ARGS = ['check', '--reporter=json', '--max-diagnostics=
 export const BIOME_TRUNCATED_NOTE =
   'biome JSON output was truncated; split the scope and re-run to get complete diagnostics';
 const EXPLAIN_TIMEOUT_MS = 8_000;
-// One short single-rule process each; scale the fan-out with the host.
-const EXPLAIN_CONCURRENCY = Math.max(4, Math.ceil(availableParallelism() / 2));
+// One short single-rule process each, a few at a time: the host is the
+// user's working machine, so the fan-out does not grow with its core count.
+const EXPLAIN_CONCURRENCY = 4;
 
 function severityOf(value) {
   const level = String(value || '').toLowerCase();

package/src/runtime/tidy/tool.mjs

@@ -214,7 +214,7 @@ async function satisfyMissingEngines({ resolution, cwd, engineFilter, approveDow
   if (policy === 'never') return { resolution, needsApproval: null, installed: null };
   const { installEngines, planInstall } = await import('./install.mjs');
   if (policy !== 'auto' && !approveDownloads) {
-    const plan = planInstall({
+    const plan = await planInstall({
       ids: installable,
       manifest: resolution.manifest,
       pluginData: resolution.pluginData,

package/src/runtime/tidy/worktree.mjs

@@ -7,6 +7,7 @@
 // already non-conforming in HEAD had to be reverted by hand. Scope still comes
 // from `paths` and nothing here derives one from a diff: the split is a report
 // the caller reads before writing, so they can pass a precise file list.
+import { startGit } from '../shared/git-launch.mjs';
 import { normalizeRel } from './languages.mjs';
 import { runProcess, which } from './process.mjs';
 
@@ -44,13 +45,16 @@ export async function listWorkingTreeChanges({
   env = process.env,
 } = {}) {
   if (!which('git', { env })) return { files: null, error: '', skipped: NO_GIT_REASON };
+  // The git the caller's own PATH names, as the lookup above found it.
+  const git = (args) =>
+    startGit(
+      env,
+      (command, childEnv) => runProcess(command, args, { cwd, signal, timeoutMs, env: childEnv }),
+      (result) => ({ code: result.errorCode })
+    );
   const [status, prefix] = await Promise.all([
-    runProcess(
-      'git',
-      ['-c', 'core.quotepath=false', '--literal-pathspecs', 'status', '--porcelain', '-z', '--untracked-files=all'],
-      { cwd, signal, timeoutMs, env }
-    ),
-    runProcess('git', ['rev-parse', '--show-prefix'], { cwd, signal, timeoutMs, env }),
+    git(['-c', 'core.quotepath=false', '--literal-pathspecs', 'status', '--porcelain', '-z', '--untracked-files=all']),
+    git(['rev-parse', '--show-prefix']),
   ]);
   const failed = [status, prefix].find((result) => result.code !== 0);
   if (failed) {

package/src/runtime/visualize-bridge/client.mjs

@@ -43,6 +43,9 @@ export function validateVisualizeArgs(args) {
     return { ok: false, error: 'action must be open, close, annotate, capture, export, check, or inspect' };
   const input = args.input ?? {};
   if (typeof input !== 'object' || Array.isArray(input)) return { ok: false, error: 'input must be an object' };
+  if (args.action === 'open' && input.verify != null && typeof input.verify !== 'boolean') {
+    return { ok: false, error: 'open input.verify must be a boolean' };
+  }
   if (args.action === 'open' && input.content != null) {
     const content = input.content;
     if (input.targets != null)
@@ -114,7 +117,7 @@ export function validateVisualizeArgs(args) {
   if (args.action === 'export') {
     if (input.format !== 'png' && input.format !== 'pdf')
       return { ok: false, error: 'export requires input.format png or pdf' };
-    if (typeof input.output !== 'string' || !new RegExp('\\.' + input.format + '$', 'i').test(input.output.trim())) {
+    if (typeof input.output !== 'string' || !new RegExp(`\\.${input.format}$`, 'i').test(input.output.trim())) {
       return { ok: false, error: `export requires input.output ending in .${input.format}` };
     }
     if (input.version != null && !(Number.isInteger(input.version) && input.version >= 1)) {

package/src/runtime/visualize-bridge/tool-defs.mjs

@@ -10,8 +10,8 @@ export const TOOL_DEFS = [
     name: 'visualize',
     title: 'Mixdog Visualize',
     description:
-      'Show the user any artifact you produced or changed (mockups, docs, decks, sheets, PDFs, images, video, local dev pages) in the desktop side panel beside the chat. Panel tabs auto-reload when a file changes, so re-opening after edits is unnecessary. ' +
-      'open: input.targets (1-3 file paths, relative to the working directory or absolute anywhere on this machine; 2-3 = side-by-side comparison; or a single http(s) URL alone, never mixed with files or other URLs), optional label, optional version (show that kept version of a file tab read-only; each open/reload that changes the files keeps a version). close: optional tab (omit = all). ' +
+      'Show the user a finished deliverable you produced or changed (mockups, docs, decks, sheets, PDFs, images, video, local dev pages) in the desktop side panel beside the chat. A summary, plan or report of your own work is not one either: it stays in the chat. Your own check screenshots are not deliverables: look at them with read; open with verify: true only to run check/inspect/capture/export. Panel tabs auto-reload when a file changes, so re-opening after edits is unnecessary. ' +
+      'open: input.targets (1-3 file paths, relative to the working directory or absolute anywhere on this machine; 2-3 = side-by-side comparison; or a single http(s) URL alone, never mixed with files or other URLs), optional label, optional version (show that kept version of a file tab read-only; each open/reload that changes the files keeps a version), optional verify (true = you open only to check your own work with check/inspect/capture/export: the user is not shown the tab and it closes when the turn ends; leave it off for anything the user should see). close: optional tab (omit = all). ' +
       'annotate: tab + add [{selector,label?}] | remove [selector] | clear; HTML pages only, never edits the file. capture: tab + output (.png path). ' +
       'export: tab + format (png|pdf) + output (project-relative path ending .png/.pdf) + optional version (a kept version of the tab) + optional overwrite (true replaces an existing file, only under design/; otherwise an existing file is refused). Writes a record of the approved page inside the project and returns its project-relative path, size/dimensions and the page problems seen while rendering. A single HTML page (file, or inline html/chart) is rendered whole in a hidden window, so the tab need not be on screen (PDF: HTML pages only; PNG has a pixel cap, over it export pdf or shorten the page); any other tab exports as capture does (the displayed tab, PNG only). Same project containment as capture: links and paths outside the working directory are refused. ' +
       'check: tab; HTML pages only; waits briefly for the page to settle and returns only its runtime problems (uncaught errors, unhandled rejections, CSP violations, console errors/warnings, missing local assets). ' +
@@ -61,6 +61,11 @@ export const TOOL_DEFS = [
               description:
                 'open (file tabs): kept version number to show read-only instead of the live files. export: kept version number to render instead of the live page.',
             },
+            verify: {
+              type: 'boolean',
+              description:
+                'open (files or content): true to check your own work only; the user is not shown the tab and it closes when the turn ends.',
+            },
             tab: { type: 'string', description: 'Tab id from open (close: omit for all).' },
             add: {
               type: 'array',

package/src/runtime/web-search/index.mjs

@@ -102,7 +102,7 @@ function normalizeUrlArgs(rawArgs) {
 function invalidArgsResponse(error) {
   if (!(error instanceof z.ZodError)) return null;
   return {
-    content: [{ type: 'text', text: JSON.stringify({ error: 'Invalid arguments', details: error.errors }) }],
+    content: [{ type: 'text', text: JSON.stringify({ error: 'Invalid arguments', details: error.issues }) }],
     isError: true,
   };
 }