mixdog 0.8.1 → 0.9.1

package/src/tui/engine-runtime-notification.test.mjs

@@ -0,0 +1,115 @@
+import test from "node:test";
+import assert from "node:assert/strict";
+import { resolveTuiRuntimeNotificationDelivery } from "./engine.mjs";
+import { toolCompletionMeta } from "../runtime/shared/tool-execution-contract.mjs";
+
+const completedBackgroundTask = [
+  "background task",
+  "task_id: task_agent_abc123",
+  "surface: agent",
+  "operation: run",
+  "status: completed",
+  "",
+  "worker finished successfully",
+].join("\n");
+
+const runningBackgroundTask = [
+  "background task",
+  "task_id: task_agent_abc123",
+  "surface: agent",
+  "operation: run",
+  "status: running",
+].join("\n");
+
+const statusOnlyCompletedBackgroundTask = [
+  "background task",
+  "task_id: task_agent_abc123",
+  "surface: agent",
+  "operation: run",
+  "status: completed",
+].join("\n");
+
+const agentTaskEnvelopeWithBody = [
+  "agent task: task_agent_xyz",
+  "status: completed",
+  "",
+  "done",
+].join("\n");
+
+const nestedAgentTaskInBackgroundTask = [
+  "background task",
+  "task_id: task_agent_abc123",
+  "surface: agent",
+  "operation: run",
+  "status: completed",
+  "",
+  "agent task: task_agent_nested",
+  "status: completed",
+  "",
+  "nested worker result",
+].join("\n");
+
+test("execution completion uses wrapper model content not raw enqueue text", () => {
+  const meta = toolCompletionMeta({
+    surface: "agent",
+    id: "task_agent_abc123",
+    status: "completed",
+    context: { callerSessionId: "sess_owner" },
+  });
+  const delivery = resolveTuiRuntimeNotificationDelivery({ meta }, completedBackgroundTask);
+  assert.equal(delivery.action, "execution-ui");
+  assert.equal(delivery.displayText, completedBackgroundTask);
+  assert.ok(delivery.modelContent);
+  assert.match(delivery.modelContent, /worker finished successfully/);
+  assert.doesNotMatch(delivery.modelContent, /^background task\b/m);
+  assert.notEqual(delivery.modelContent, completedBackgroundTask);
+});
+
+test("running execution notification is UI-only with empty model content", () => {
+  const meta = toolCompletionMeta({
+    surface: "agent",
+    id: "task_agent_abc123",
+    status: "running",
+    context: { callerSessionId: "sess_owner" },
+  });
+  const delivery = resolveTuiRuntimeNotificationDelivery({ meta }, runningBackgroundTask);
+  assert.equal(delivery.action, "execution-ui");
+  assert.equal(delivery.modelContent, "");
+});
+
+test("nested agent task body inside completed background task is not status-only", () => {
+  const meta = toolCompletionMeta({
+    surface: "agent",
+    id: "task_agent_abc123",
+    status: "completed",
+    context: { callerSessionId: "sess_owner" },
+  });
+  const delivery = resolveTuiRuntimeNotificationDelivery({ meta }, nestedAgentTaskInBackgroundTask);
+  assert.equal(delivery.action, "execution-ui");
+  assert.ok(String(delivery.modelContent || "").trim());
+  assert.match(delivery.modelContent, /nested worker result/);
+});
+
+test("agent task envelope with blank-line body is not status-only", () => {
+  const meta = toolCompletionMeta({
+    surface: "agent",
+    id: "task_agent_xyz",
+    status: "completed",
+    context: { callerSessionId: "sess_owner" },
+  });
+  const delivery = resolveTuiRuntimeNotificationDelivery({ meta }, agentTaskEnvelopeWithBody);
+  assert.equal(delivery.action, "execution-ui");
+  assert.ok(String(delivery.modelContent || "").trim());
+});
+
+test("header-only completed background task is status-only", () => {
+  const meta = toolCompletionMeta({
+    surface: "agent",
+    id: "task_agent_abc123",
+    status: "completed",
+    context: { callerSessionId: "sess_owner" },
+  });
+  const delivery = resolveTuiRuntimeNotificationDelivery({ meta }, statusOnlyCompletedBackgroundTask);
+  assert.equal(delivery.action, "status-only");
+  assert.equal(delivery.modelContent, "");
+});

package/src/tui/engine-tool-result-text.test.mjs

@@ -0,0 +1,75 @@
+import test from 'node:test';
+import assert from 'node:assert/strict';
+import {
+  toolAggregateDetailFallback,
+  toolGroupedDisplayFallback,
+  toolResultText,
+} from './engine.mjs';
+
+test('toolResultText prefers nested content over empty scalar fields', () => {
+  assert.equal(
+    toolResultText({ text: '', content: [{ type: 'text', text: 'nested body' }] }),
+    'nested body',
+  );
+});
+
+test('toolResultText unwraps nested tool_result blocks', () => {
+  assert.equal(
+    toolResultText({
+      type: 'tool_result',
+      content: [{ type: 'tool_result', content: 'inner payload' }],
+    }),
+    'inner payload',
+  );
+});
+
+test('toolResultText preserves explicit empty tool_result clears', () => {
+  assert.equal(toolResultText({ type: 'tool_result', content: '' }), '');
+});
+
+test('toolResultText preserves empty tool_result content arrays', () => {
+  assert.equal(toolResultText({ type: 'tool_result', content: [] }), '');
+  assert.notEqual(toolResultText({ type: 'tool_result', content: [] }), '{"type":"tool_result","content":[]}');
+});
+
+test('toolResultText preserves empty typed text parts inside tool_result', () => {
+  assert.equal(
+    toolResultText({ type: 'tool_result', content: [{ type: 'text', text: '' }] }),
+    '',
+  );
+});
+
+test('toolResultText preserves empty object-shaped tool_result inner content', () => {
+  assert.equal(toolResultText({ type: 'tool_result', content: {} }), '');
+  assert.equal(
+    toolResultText({ type: 'tool_result', content: { type: 'text', text: '' } }),
+    '',
+  );
+});
+
+test('toolResultText still extracts plain strings and structured bodies', () => {
+  assert.equal(toolResultText('plain body'), 'plain body');
+  assert.equal(
+    toolResultText([{ type: 'text', text: 'line one' }, { type: 'text', text: 'line two' }]),
+    'line one\nline two',
+  );
+});
+
+test('toolResultText maps every top-level array element without using index as depth', () => {
+  const parts = Array.from({ length: 14 }, (_, i) => ({ type: 'text', text: `line-${i}` }));
+  const out = toolResultText(parts);
+  assert.match(out, /^line-0\n/);
+  assert.match(out, /\nline-13$/);
+  assert.equal(out.split('\n').length, 14);
+});
+
+test('toolAggregateDetailFallback uses raw body when summary detail is empty', () => {
+  assert.equal(toolAggregateDetailFallback('', 'file contents here'), 'file contents here');
+});
+
+test('toolGroupedDisplayFallback uses later extracted body when grouped text is empty', () => {
+  assert.equal(toolGroupedDisplayFallback('', '', 'recovered body'), 'recovered body');
+  assert.equal(toolGroupedDisplayFallback('group summary', '', 'later body'), 'group summary');
+});
+
+