mixdog 0.9.3 → 0.9.5

package/scripts/fix-format-tool-surface.mjs

@@ -1,24 +0,0 @@
-import fs from "node:fs";
-const p = "src/runtime/shared/tool-surface.mjs";
-let s = fs.readFileSync(p, "utf8");
-const block = `export function formatToolSurface(name, args, opts = {}) {
-  const parsed = parseToolArgs(args);
-  return {
-    label: displayToolName(name, parsed),
-    summary: summarizeToolArgs(name, parsed, opts),
-    normalizedName: normalizeToolName(name),
-    args: parsed,
-  };
-}
-
-`;
-while (s.includes(block)) {
-  s = s.replace(block, "");
-}
-const anchor = `/** Collapsed agent/worker surface card: one-line brief under the header (30-40 chars). */`;
-if (!s.includes(anchor)) throw new Error("anchor missing");
-if (!s.includes("export function formatToolSurface")) {
-  s = s.replace(anchor, block + anchor);
-}
-fs.writeFileSync(p, s);
-console.log("fixed formatToolSurface");

package/scripts/fix-tool-exec-visible.mjs

@@ -1,42 +0,0 @@
-import fs from "node:fs";
-const p = "src/tui/components/ToolExecution.jsx";
-let s = fs.readFileSync(p, "utf8");
-const bad = `  let visibleDetailLines = detailLines;
-  if (isSkillSurface && !showRawResult) {
-    visibleDetailLines = [];
-  } else if (isAgentSurfaceCard && !showRawResult) {
-    if (agentSurfaceBrief) {
-      visibleDetailLines = [agentSurfaceBrief];
-    } else if (pending) {
-      visibleDetailLines = [pendingDetailPlaceholder || 'Running'];
-    } else {
-      visibleDetailLines = [];
-    }
-  }
-  } else if (isAgentSurfaceCard && !showRawResult) {
-  } else if (isAgentSurfaceCard && !showRawResult) {
-    const agentDetailFallback = collapsedDetail
-      || (pending ? (pendingDetailPlaceholder || 'Running') : 'Finished');
-    const agentDetailLine = agentSurfaceBrief
-      || truncateToWidth(String(agentDetailFallback), Math.min(AGENT_SURFACE_BRIEF_MAX, maxResultChars));
-    visibleDetailLines = [agentDetailLine];
-  }`;
-const good = `  let visibleDetailLines = detailLines;
-  if (isSkillSurface && !showRawResult) {
-    visibleDetailLines = [];
-  } else if (isAgentSurfaceCard && !showRawResult) {
-    const agentDetailFallback = collapsedDetail
-      || (pending ? (pendingDetailPlaceholder || 'Running') : 'Finished');
-    const agentDetailLine = agentSurfaceBrief
-      || truncateToWidth(String(agentDetailFallback), Math.min(AGENT_SURFACE_BRIEF_MAX, maxResultChars));
-    visibleDetailLines = [agentDetailLine];
-  }`;
-if (s.includes(bad)) {
-  s = s.replace(bad, good);
-  fs.writeFileSync(p, s);
-  console.log("ToolExecution fixed");
-} else if (s.includes("agentDetailFallback")) {
-  console.log("ToolExecution already ok");
-} else {
-  throw new Error("ToolExecution pattern not found");
-}

package/scripts/patch-agent-brief.mjs

@@ -1,48 +0,0 @@
-import fs from "node:fs";
-const p = "src/runtime/shared/tool-surface.mjs";
-let s = fs.readFileSync(p, "utf8");
-if (s.includes("summarizeAgentSurfaceBrief")) { console.log("skip"); process.exit(0); }
-const insert = `/** Collapsed agent/worker surface card: one-line brief under the header (30-40 chars). */
-export const AGENT_SURFACE_BRIEF_MAX = 40;
-
-export function summarizeAgentSurfaceBrief(name, args, resultText, { isError = false, isResponse = false } = {}) {
-  const a = parseToolArgs(args);
-  const action = String(a?.type || a?.action || "").toLowerCase();
-  const text = String(resultText ?? "").trim();
-
-  if (isResponse && text) {
-    const fromResult = summarizeToolResult(name, args, text, isError);
-    if (fromResult) return truncateSingleLine(stripInlineMarkdown(fromResult), AGENT_SURFACE_BRIEF_MAX);
-    const line = firstAgentResultLine(text);
-    if (line) return truncateSingleLine(stripInlineMarkdown(line), AGENT_SURFACE_BRIEF_MAX);
-  }
-
-  const outbound = firstText(a?.prompt, a?.message);
-  if (outbound && (action === "spawn" || action === "send" || !action)) {
-    return truncateSingleLine(outbound, AGENT_SURFACE_BRIEF_MAX);
-  }
-
-  if ((action === "spawn" || action === "send") && text && !isResponse) {
-    const fromResult = summarizeToolResult(name, args, text, isError);
-    if (fromResult) return truncateSingleLine(stripInlineMarkdown(fromResult), AGENT_SURFACE_BRIEF_MAX);
-  }
-
-  return "";
-}
-
-`;
-const needle = `export function formatToolSurface(name, args, opts = {}) {
-  const parsed = parseToolArgs(args);
-  return {
-    label: displayToolName(name, parsed),
-    summary: summarizeToolArgs(name, parsed, opts),
-    normalizedName: normalizeToolName(name),
-    args: parsed,
-  };
-}
-
-function pluralize`;
-if (!s.includes(needle)) throw new Error("needle not found");
-s = s.replace(needle, insert + "function pluralize");
-fs.writeFileSync(p, s);
-console.log("tool-surface ok");

package/scripts/patch-app.mjs

@@ -1,21 +0,0 @@
-import fs from "node:fs";
-const p = "src/tui/App.jsx";
-let s = fs.readFileSync(p, "utf8");
-const old = `        // EVERY agent card is a single header row whether pending or completed —
-        // ToolExecution drops the ⎿ body for all of them AND the pending-delay
-        // placeholder reserves only 1 row for agent surfaces — so reserve
-        // exactly 1 row regardless of pending/hasResult. Otherwise the estimate
-        // (2) and the real render (1) diverge and the viewport jumps.
-        if (isSkillSurface || isAgentSurface) return 1;`;
-const neu = `        // Skill loads are a single header row; agent cards are header + brief.
-        if (isSkillSurface) return 1;
-        if (isAgentSurface) return 2;`;
-if (s.includes(old)) {
-  s = s.replace(old, neu);
-  fs.writeFileSync(p, s);
-  console.log("App ok");
-} else if (s.includes("if (isAgentSurface) return 2")) {
-  console.log("App already");
-} else {
-  throw new Error("App block not found");
-}

package/scripts/patch-app2.mjs

@@ -1,18 +0,0 @@
-import fs from "node:fs";
-const p = "src/tui/App.jsx";
-let s = fs.readFileSync(p, "utf8");
-const old = `        if (isSkillSurface || isAgentSurface) return 1;`;
-const neu = `        if (isSkillSurface) return 1;
-        if (isAgentSurface) return 2;`;
-if (s.includes(old) && !s.includes("if (isAgentSurface) return 2")) {
-  s = s.replace(old, neu);
-  const commentOld = `        // EVERY agent card is a single header row whether pending or completed —
-        // ToolExecution drops the ⎿ body for all of them AND the pending-delay
-        // placeholder reserves only 1 row for agent surfaces — so reserve
-        // exactly 1 row regardless of pending/hasResult. Otherwise the estimate
-        // (2) and the real render (1) diverge and the viewport jumps.`;
-  const commentNeu = `        // Skill loads are a single header row; agent cards are header + brief.`;
-  s = s.replace(commentOld, commentNeu);
-  fs.writeFileSync(p, s);
-  console.log("App ok");
-} else { console.log("App skip or done"); }

package/scripts/patch-dist-brief.mjs

@@ -1,96 +0,0 @@
-import fs from "node:fs";
-const p = "src/tui/dist/index.mjs";
-let s = fs.readFileSync(p, "utf8");
-
-const helpers = `const AGENT_SURFACE_BRIEF_MAX = 40;
-function truncateAgentSurfaceBrief(value, max = AGENT_SURFACE_BRIEF_MAX) {
-  const text = String(value ?? "").replace(/\\s+/g, " ").trim();
-  if (!text) return "";
-  if (text.length <= max) return text;
-  return \`\${text.slice(0, Math.max(1, max - 1))}…\`;
-}
-function summarizeAgentSurfaceBrief(name, args, resultText, { isError = false, isResponse = false } = {}) {
-  const a = parseToolArgs(args);
-  const action = String(a?.type || a?.action || "").toLowerCase();
-  const text = String(resultText ?? "").trim();
-
-  if (isResponse && text) {
-    const fromResult = summarizeToolResult(name, args, text, isError);
-    if (fromResult) return truncateAgentSurfaceBrief(stripInlineMarkdown(fromResult));
-    const line = firstAgentResultLine(text);
-    if (line) return truncateAgentSurfaceBrief(stripInlineMarkdown(line));
-  }
-
-  const outbound = firstText(a?.prompt, a?.message);
-  if (outbound && (action === "spawn" || action === "send" || !action)) {
-    return truncateAgentSurfaceBrief(outbound);
-  }
-
-  if ((action === "spawn" || action === "send") && text && !isResponse) {
-    const fromResult = summarizeToolResult(name, args, text, isError);
-    if (fromResult) return truncateAgentSurfaceBrief(stripInlineMarkdown(fromResult));
-  }
-
-  return "";
-}
-
-`;
-
-if (!s.includes("function summarizeAgentSurfaceBrief")) {
-  const needle = `function formatToolSurface(name, args, opts = {}) {
-  const parsed = parseToolArgs(args);
-  return {
-    label: displayToolName(name, parsed),
-    summary: summarizeToolArgs(name, parsed, opts),
-    normalizedName: normalizeToolName(name),
-    args: parsed
-  };
-}
-function pluralize`;
-  if (!s.includes(needle)) throw new Error("formatToolSurface needle missing");
-  s = s.replace(needle, `function formatToolSurface(name, args, opts = {}) {
-  const parsed = parseToolArgs(args);
-  return {
-    label: displayToolName(name, parsed),
-    summary: summarizeToolArgs(name, parsed, opts),
-    normalizedName: normalizeToolName(name),
-    args: parsed
-  };
-}
-` + helpers + `function pluralize`);
-}
-
-const oldVisible = `  const isAgentSurfaceCard = isAgentTool(normalizedName);
-  const visibleDetailLines = (isAgentSurfaceCard || isSkillSurface) && !showRawResult ? [] : detailLines;`;
-const newVisible = `  const isAgentSurfaceCard = isAgentTool(normalizedName);
-  const agentSurfaceBriefRaw = isAgentSurfaceCard && !showRawResult
-    ? summarizeAgentSurfaceBrief(name, parsedArgs, displayedResultText || "", { isError, isResponse: isAgentResponse })
-    : "";
-  const agentSurfaceBrief = agentSurfaceBriefRaw
-    ? truncateToWidth(agentSurfaceBriefRaw, Math.min(AGENT_SURFACE_BRIEF_MAX, maxResultChars))
-    : "";
-  let visibleDetailLines = detailLines;
-  if (isSkillSurface && !showRawResult) {
-    visibleDetailLines = [];
-  } else if (isAgentSurfaceCard && !showRawResult) {
-    const agentDetailFallback = collapsedDetail
-      || (pending ? (pendingDetailPlaceholder || "Running") : "Finished");
-    const agentDetailLine = agentSurfaceBrief
-      || truncateToWidth(String(agentDetailFallback), Math.min(AGENT_SURFACE_BRIEF_MAX, maxResultChars));
-    visibleDetailLines = [agentDetailLine];
-  }`;
-if (s.includes(oldVisible)) s = s.replace(oldVisible, newVisible);
-else if (!s.includes("agentSurfaceBriefRaw")) throw new Error("visibleDetailLines block missing");
-
-s = s.replace(
-  "const placeholderSingleRow = !aggregate && (SKILL_SURFACE_NAMES.has(placeholderNormalizedName) || isAgentTool(placeholderNormalizedName));",
-  "const placeholderSingleRow = !aggregate && SKILL_SURFACE_NAMES.has(placeholderNormalizedName);"
-);
-
-s = s.replace(
-  "        if (isSkillSurface || isAgentSurface) return 1;",
-  "        if (isSkillSurface) return 1;\n        if (isAgentSurface) return 2;"
-);
-
-fs.writeFileSync(p, s);
-console.log("dist patched");

package/scripts/patch-tool-exec.mjs

@@ -1,70 +0,0 @@
-import fs from "node:fs";
-const p = "src/tui/components/ToolExecution.jsx";
-let s = fs.readFileSync(p, "utf8");
-if (!s.includes("summarizeAgentSurfaceBrief,")) {
-  s = s.replace(
-    "  displayModelName,\n} from '../../runtime/shared/tool-surface.mjs';",
-    "  displayModelName,\n  summarizeAgentSurfaceBrief,\n  AGENT_SURFACE_BRIEF_MAX,\n} from '../../runtime/shared/tool-surface.mjs';",
-  );
-}
-s = s.replace(
-  `    // Skill surfaces AND agent surfaces both collapse to a single header row in
-    // estimateTranscriptItemRows (agent cards drop their ⎿ body entirely), so
-    // the pending-delay placeholder must reserve exactly 1 row for them too —
-    // otherwise the card paints 2 blank rows for ~1s then snaps to 1 ("튐").
-    const placeholderSingleRow = !aggregate
-      && (SKILL_SURFACE_NAMES.has(placeholderNormalizedName) || isAgentTool(placeholderNormalizedName));`,
-  `    // Skill surfaces collapse to a single header row; agent surfaces reserve
-    // header + one brief detail row (see estimateTranscriptItemRows).
-    const placeholderSingleRow = !aggregate && SKILL_SURFACE_NAMES.has(placeholderNormalizedName);`,
-);
-const oldBlock = `  // Every agent card is a single header row when collapsed: spawn/send/response/
-  // status/list/cancel/close/cleanup all fold their context into the header
-  // label, so the ⎿ detail body (response summary, "agents: N …" worker list,
-  // status metadata) is dropped unless the user expands with ctrl+o.
-  const isAgentSurfaceCard = isAgentTool(normalizedName);
-  // Skill loads carry the skill name in the header already
-  // ("Loaded 1 skill (name)"); the collapsed detail row just repeats it, so
-  // drop it and keep the card a single line. Expanding (ctrl+o) still shows the
-  // full skill body via the raw-result path.
-  // Agent responses now identify the responder in the header itself
-  // ("Heavy Worker (Opus 4.8)"); the collapsed body just echoed the response
-  // summary ("All green. Done."), so drop it and keep the card a single line.
-  // ctrl+o expand (showRawResult) still surfaces the full response body.
-  // Suppression is COLLAPSED-ONLY: agent/skill cards hide their ⎿ body when
-  // collapsed, but once the user expands (showRawResult) the raw body — the
-  // response, the "agents: N …" worker list, the status metadata — must render.
-  const visibleDetailLines = ((isAgentSurfaceCard || isSkillSurface) && !showRawResult)
-    ? []
-    : detailLines;`;
-const newBlock = `  const isAgentSurfaceCard = isAgentTool(normalizedName);
-  const agentSurfaceBriefRaw = isAgentSurfaceCard && !showRawResult
-    ? summarizeAgentSurfaceBrief(name, parsedArgs, displayedResultText || '', { isError, isResponse: isAgentResponse })
-    : '';
-  const agentSurfaceBrief = agentSurfaceBriefRaw
-    ? truncateToWidth(agentSurfaceBriefRaw, Math.min(AGENT_SURFACE_BRIEF_MAX, maxResultChars))
-    : '';
-  // Skill loads carry the skill name in the header already
-  // ("Loaded 1 skill (name)"); the collapsed detail row just repeats it, so
-  // drop it and keep the card a single line. Expanding (ctrl+o) still shows the
-  // full skill body via the raw-result path.
-  // Agent spawn/send/response cards show a tight brief under the ⎿ gutter when
-  // collapsed; ctrl+o expand still surfaces the full body.
-  let visibleDetailLines = detailLines;
-  if (isSkillSurface && !showRawResult) {
-    visibleDetailLines = [];
-  } else if (isAgentSurfaceCard && !showRawResult) {
-    if (agentSurfaceBrief) {
-      visibleDetailLines = [agentSurfaceBrief];
-    } else if (pending) {
-      visibleDetailLines = [pendingDetailPlaceholder || 'Running'];
-    } else {
-      visibleDetailLines = [];
-    }
-  }`;
-if (!s.includes("agentSurfaceBriefRaw")) {
-  if (!s.includes(oldBlock)) throw new Error("ToolExecution block not found");
-  s = s.replace(oldBlock, newBlock);
-}
-fs.writeFileSync(p, s);
-console.log("ToolExecution ok");

package/src/examples/schedules/SCHEDULE.example.md

@@ -1,32 +0,0 @@
----
-time: 0 9 * * *
-timezone: Asia/Seoul
-days: weekday
-channel: main
-model: gpt-5
-enabled: true
----
-
-# Schedule instructions (prompt body)
-
-This whole markdown body is the prompt the scheduler runs when the cron fires.
-Everything above the second `---` is frontmatter metadata; everything below is
-the instructions.
-
-Frontmatter keys:
-
-- `time` — required. 5- or 6-field cron expression (e.g. `0 9 * * *`).
-- `timezone` — optional IANA tz (e.g. `Asia/Seoul`); host-local when omitted.
-- `days` — optional; omit for `daily`. Allowed values: `weekday`, `weekend`, or
-  a comma list of day abbreviations (`mon,tue,wed,thu,fri,sat,sun`). Written
-  only when not `daily`.
-- `channel` — optional channel label. WITH a channel the run dispatches to that
-  channel (non-interactive) and REQUIRES `model`; WITHOUT it the run injects
-  into the current session (interactive).
-- `model` — required when `channel` is set; the preset id used for dispatch.
-- `enabled` — optional; written `false` to disable. Missing/anything-else means
-  enabled. The reader casts this string to a boolean.
-
-Storage layout: `<data>/schedules/<name>/SCHEDULE.md` — one file per schedule.
-This `.example.md` is documentation only; the readers match `SCHEDULE.md`
-exactly, so this file is never loaded as a real schedule.

package/src/examples/webhooks/WEBHOOK.example.md

@@ -1,40 +0,0 @@
----
-parser: github
-channel: main
-model: gpt-5
-role: webhook-handler
-enabled: true
----
-
-# Webhook instructions (prompt body)
-
-This markdown body is the handler prompt. On each verified delivery the webhook
-payload is fenced as untrusted data and appended after these instructions.
-Everything above the second `---` is frontmatter metadata; everything below is
-the instructions.
-
-Frontmatter keys:
-
-- `parser` — required. One of `github`, `generic`, `stripe`, `sentry`. Selects
-  the signature header + verification scheme.
-- `channel` — optional channel label. WITH a channel the delivery is dispatched
-  as a delegate (REQUIRES `model` + `role`) and reported to that channel;
-  WITHOUT it the payload is enqueued into the current session (interactive).
-- `model` — required when `channel` is set; the preset id used for delegate
-  dispatch.
-- `role` — user-workflow role for delegate mode; defaults to `webhook-handler`.
-- `enabled` — optional; written `false` to disable (delivery rejected with 404).
-  Missing/anything-else means enabled. The reader casts this string to a boolean.
-
-Secret storage: the HMAC secret is NOT a frontmatter key. It lives in a side
-file `<data>/webhooks/<name>/secret` (plaintext, one line), autogenerated
-(24 random bytes hex) when omitted at save time. Keeping it out of the lossy
-`key: value` frontmatter lets an arbitrary user secret (quotes/colons/newlines)
-round-trip losslessly and prevents enable/disable rewrites from corrupting it.
-The admin list API never exposes the value — only `secretSet: true/false`.
-
-Storage layout: `<data>/webhooks/<name>/WEBHOOK.md` — one file per endpoint.
-`secret` and `deliveries.jsonl` live alongside it and are managed separately.
-This
-`.example.md` is documentation only; the readers match `WEBHOOK.md` exactly, so
-this file is never loaded as a real endpoint.

package/src/runtime/agent/orchestrator/session/manager.reactive-persist.test.mjs

@@ -1,107 +0,0 @@
-import assert from 'node:assert/strict';
-import test from 'node:test';
-
-import {
-    _applyCompactFailurePersistToSession,
-    _sessionMessagesAdvancedBeyondCompactedOutgoing,
-} from './manager.mjs';
-
-const compacted = [
-    { role: 'user', content: 'summary turn' },
-    { role: 'user', content: 'current task' },
-];
-
-test('sessionMessagesAdvancedBeyondCompactedOutgoing: true for compacted prefix plus final assistant', () => {
-    const current = [
-        ...compacted,
-        { role: 'assistant', content: 'done' },
-    ];
-    assert.equal(_sessionMessagesAdvancedBeyondCompactedOutgoing(current, compacted), true);
-});
-
-test('sessionMessagesAdvancedBeyondCompactedOutgoing: true for compacted prefix plus empty-final assistant', () => {
-    const current = [
-        ...compacted,
-        { role: 'assistant', content: '', emptyFinal: true },
-    ];
-    assert.equal(_sessionMessagesAdvancedBeyondCompactedOutgoing(current, compacted), true);
-});
-
-test('sessionMessagesAdvancedBeyondCompactedOutgoing: false when transcript has not advanced', () => {
-    assert.equal(_sessionMessagesAdvancedBeyondCompactedOutgoing(compacted, compacted), false);
-});
-
-test('sessionMessagesAdvancedBeyondCompactedOutgoing: false when extra message is not a final assistant', () => {
-    const current = [
-        ...compacted,
-        { role: 'user', content: 'steering follow-up' },
-    ];
-    assert.equal(_sessionMessagesAdvancedBeyondCompactedOutgoing(current, compacted), false);
-});
-
-test('sessionMessagesAdvancedBeyondCompactedOutgoing: false when prefix diverges from compacted outgoing', () => {
-    const current = [
-        { role: 'user', content: 'different summary' },
-        { role: 'user', content: 'current task' },
-        { role: 'assistant', content: 'done' },
-    ];
-    assert.equal(_sessionMessagesAdvancedBeyondCompactedOutgoing(current, compacted), false);
-});
-
-test('applyCompactFailurePersistToSession: preserves providerState when transcript already advanced', () => {
-    const priorSanitized = [
-        ...compacted,
-        { role: 'assistant', content: 'done' },
-    ];
-    const session = {
-        messages: priorSanitized,
-        providerState: { continuation: 'fresh' },
-        compaction: { lastStage: 'compacting' },
-    };
-    const applied = _applyCompactFailurePersistToSession(session, {
-        priorSanitized,
-        sanitized: compacted,
-        messagesAdvanced: true,
-        error: new Error('ask failed'),
-    });
-    assert.equal(applied, true);
-    assert.deepEqual(session.messages, priorSanitized);
-    assert.deepEqual(session.providerState, { continuation: 'fresh' });
-    assert.equal(session.compaction.lastStage, 'failed');
-});
-
-test('applyCompactFailurePersistToSession: clears providerState when replacing messages with compacted outgoing', () => {
-    const priorSanitized = [
-        { role: 'user', content: 'old long history' },
-    ];
-    const session = {
-        messages: priorSanitized,
-        providerState: { stale: true },
-        compaction: { lastStage: 'compacting' },
-    };
-    const applied = _applyCompactFailurePersistToSession(session, {
-        priorSanitized,
-        sanitized: compacted,
-        messagesAdvanced: false,
-        error: { code: 'AGENT_CONTEXT_OVERFLOW' },
-    });
-    assert.equal(applied, true);
-    assert.deepEqual(session.messages, compacted);
-    assert.equal(session.providerState, undefined);
-    assert.equal(session.compaction.lastStage, 'overflow_failed');
-});
-
-test('applyCompactFailurePersistToSession: no-op when messages unchanged and not advanced', () => {
-    const session = {
-        messages: compacted,
-        providerState: { keep: true },
-    };
-    const applied = _applyCompactFailurePersistToSession(session, {
-        priorSanitized: compacted,
-        sanitized: compacted,
-        messagesAdvanced: false,
-    });
-    assert.equal(applied, false);
-    assert.deepEqual(session.providerState, { keep: true });
-});
-

package/src/runtime/agent/orchestrator/tools/builtin/arg-guard.test.mjs

@@ -1,143 +0,0 @@
-import { test } from 'node:test';
-import assert from 'node:assert/strict';
-import { validateBuiltinArgs } from './arg-guard.mjs';
-
-test('read: numeric-string offset/limit coerce losslessly', () => {
-    const a = { path: 'x.js', offset: '850', limit: '850.0' };
-    assert.equal(validateBuiltinArgs('read', a), null);
-    assert.equal(a.offset, 850);
-    assert.equal(a.limit, 850);
-});
-
-test('read: non-integer numeric string still rejected', () => {
-    const a = { path: 'x.js', offset: '3.5' };
-    assert.match(validateBuiltinArgs('read', a), /must be a finite integer/);
-});
-
-test('read: non-numeric string still rejected', () => {
-    const a = { path: 'x.js', offset: 'soon' };
-    assert.match(validateBuiltinArgs('read', a), /must be a finite integer/);
-});
-
-test('read: top-level line/context convert to offset/limit', () => {
-    const a = { path: 'x.js', line: 100, context: 10 };
-    assert.equal(validateBuiltinArgs('read', a), null);
-    assert.equal(a.offset, 89);
-    assert.equal(a.limit, 21);
-    assert.equal('line' in a, false);
-    assert.equal('context' in a, false);
-});
-
-test('read: line without context defaults to a single-line window', () => {
-    const a = { path: 'x.js', line: 1 };
-    assert.equal(validateBuiltinArgs('read', a), null);
-    assert.equal(a.offset, 0);
-    assert.equal(a.limit, 1);
-});
-
-test('read: explicit offset/limit wins over line/context (dropped)', () => {
-    const a = { path: 'x.js', line: 100, context: 10, offset: 5, limit: 20 };
-    assert.equal(validateBuiltinArgs('read', a), null);
-    assert.equal(a.offset, 5);
-    assert.equal(a.limit, 20);
-    assert.equal('line' in a, false);
-    assert.equal('context' in a, false);
-});
-
-test('read: path[] entry object line/context converts the same way', () => {
-    const a = { path: [{ path: 'x.js', line: 50, context: 5 }] };
-    assert.equal(validateBuiltinArgs('read', a), null);
-    assert.equal(a.path[0].offset, 44);
-    assert.equal(a.path[0].limit, 11);
-});
-
-test('read: path[] entry negative numeric-string offset is rejected after coercion', () => {
-    const a = { path: [{ path: 'x.js', offset: '-5' }] };
-    const err = validateBuiltinArgs('read', a);
-    assert.match(err, /path\[0\]\.offset/);
-    assert.match(err, />= 0/);
-});
-
-test('read: path[] entry negative numeric-string limit is rejected after coercion', () => {
-    const a = { path: [{ path: 'x.js', limit: '-1' }] };
-    const err = validateBuiltinArgs('read', a);
-    assert.match(err, /path\[0\]\.limit/);
-    assert.match(err, />= 0/);
-});
-
-test('read: path[] entry valid numeric-string offset/limit still coerces and passes', () => {
-    const a = { path: [{ path: 'x.js', offset: '10', limit: '5' }] };
-    assert.equal(validateBuiltinArgs('read', a), null);
-    assert.equal(a.path[0].offset, 10);
-    assert.equal(a.path[0].limit, 5);
-});
-
-test('read: JSON object string path is accepted as a single-entry batch', () => {
-    const a = { path: '{"path":"x.js","offset":10,"limit":5}' };
-    assert.equal(validateBuiltinArgs('read', a), null);
-    assert.equal(Array.isArray(a.path), true);
-    assert.equal(a.path[0].path, 'x.js');
-    assert.equal(a.path[0].offset, 10);
-});
-
-test('shell: win32 drive path with no shell set defaults to powershell', (t) => {
-    if (process.platform !== 'win32') { t.skip('win32 only'); return; }
-    const a = { command: 'dir C:\\Project\\mixdog' };
-    assert.equal(validateBuiltinArgs('shell', a), null);
-    assert.equal(a.shell, 'powershell');
-});
-
-test('glob: missing pattern with a path defaults pattern to "*"', () => {
-    const a = { path: 'src' };
-    assert.equal(validateBuiltinArgs('glob', a), null);
-    assert.equal(a.pattern, '*');
-});
-
-test('glob: path carrying glob magic is left for the path-magic fallback, not overridden', () => {
-    const a = { path: 'src/**/*.mjs' };
-    assert.equal(validateBuiltinArgs('glob', a), null);
-    assert.equal('pattern' in a, false);
-    assert.equal(a.path, 'src/**/*.mjs');
-});
-
-test('glob: missing pattern and path is still an error surface (guard passes through)', () => {
-    const a = {};
-    assert.equal(validateBuiltinArgs('glob', a), null);
-    assert.equal('pattern' in a, false);
-});
-
-test('grep: output_mode with newline-concatenated debris truncates to valid enum', () => {
-    const a = { pattern: 'foo', output_mode: 'content_with_context\ntrue' };
-    assert.equal(validateBuiltinArgs('grep', a), null);
-    assert.equal(a.output_mode, 'content_with_context');
-});
-
-test('grep: trailing literal \\n artifact stripped from pattern', () => {
-    const a = { pattern: 'foo bar\\n' };
-    assert.equal(validateBuiltinArgs('grep', a), null);
-    assert.equal(a.pattern, 'foo bar');
-});
-
-test('grep: trailing concatenation artifact ">\\n" stripped from pattern', () => {
-    const a = { pattern: 'foo bar">\n' };
-    assert.equal(validateBuiltinArgs('grep', a), null);
-    assert.equal(a.pattern, 'foo bar');
-});
-
-test('grep: mid-pattern \\n is left untouched', () => {
-    const a = { pattern: 'foo\\nbar' };
-    assert.equal(validateBuiltinArgs('grep', a), null);
-    assert.equal(a.pattern, 'foo\\nbar');
-});
-
-test('grep: bare trailing ">" with no newline artifact survives untouched', () => {
-    const a = { pattern: 'class="active">' };
-    assert.equal(validateBuiltinArgs('grep', a), null);
-    assert.equal(a.pattern, 'class="active">');
-});
-
-test('grep: numeric-string -C context coerces losslessly', () => {
-    const a = { pattern: 'foo', '-C': '5' };
-    assert.equal(validateBuiltinArgs('grep', a), null);
-    assert.equal(a['-C'], 5);
-});