mixdog 0.9.3 → 0.9.5

package/src/runtime/shared/task-notification-envelope.test.mjs

@@ -1,107 +0,0 @@
-import assert from 'node:assert/strict';
-import test from 'node:test';
-import {
-  renderShellCompletionEnvelope,
-  shellCompletionInstruction,
-  renderShellPromptStallEnvelope,
-  shellPromptStallInstruction,
-  isInternalRuntimeNotificationText,
-  isBracketedShellNotificationEnvelope,
-} from './task-notification-envelope.mjs';
-
-test('completion envelope matches historical assembly plus the corrected blank-line separator', () => {
-  const jobId = 'shell_job_1';
-  const status = 'completed';
-  const exitCode = 0;
-  const elapsedMs = 1234;
-  const detail = {
-    command: 'npm test',
-    summary: 'ok',
-    stdoutPreview: 'out',
-    stderrPreview: 'err',
-    mergeStderr: false,
-  };
-  const expected = [
-    `[task_id: ${jobId}]`,
-    `[status: ${status}]`,
-    `[exit: ${exitCode === null ? 'n/a' : exitCode}]`,
-    elapsedMs !== null ? `[elapsed: ${elapsedMs} ms]` : null,
-    detail.command ? `[command: ${detail.command}]` : null,
-    // The historical inline assembly filtered this '' out, gluing `Summary:`
-    // onto the header block; the corrected renderer keeps exactly one blank
-    // line between the bracket headers and the body sections.
-    '',
-    detail.summary ? `Summary: ${detail.summary}` : null,
-    detail.stdoutPreview ? `\n[stdout preview]\n${detail.stdoutPreview}` : null,
-    (detail.mergeStderr !== true && detail.stderrPreview) ? `\n[stderr preview]\n${detail.stderrPreview}` : null,
-  ].filter((l) => l !== null).join('\n');
-  const actual = renderShellCompletionEnvelope({ jobId, status, exitCode, elapsedMs, ...detail });
-  assert.equal(actual, expected);
-  assert.equal(isInternalRuntimeNotificationText(actual), true);
-  assert.equal(isBracketedShellNotificationEnvelope(actual), true);
-});
-
-test('summary-only completion envelope keeps one blank line before the body', () => {
-  const actual = renderShellCompletionEnvelope({
-    jobId: 'shell_job_1',
-    status: 'completed',
-    exitCode: 0,
-    summary: 'ok',
-  });
-  const expected = [
-    '[task_id: shell_job_1]',
-    '[status: completed]',
-    '[exit: 0]',
-    '',
-    'Summary: ok',
-  ].join('\n');
-  assert.equal(actual, expected);
-  // The blank-line separator must survive so body detectors see a body.
-  assert.match(actual, /\n\s*\n/);
-});
-
-test('bodyless completion envelope has no trailing blank line', () => {
-  const actual = renderShellCompletionEnvelope({
-    jobId: 'shell_job_1',
-    status: 'completed',
-    exitCode: 0,
-  });
-  assert.equal(actual, ['[task_id: shell_job_1]', '[status: completed]', '[exit: 0]'].join('\n'));
-  assert.doesNotMatch(actual, /\n\s*\n/);
-});
-
-test('completion instruction uses shared async wording + exit detail', () => {
-  assert.equal(
-    shellCompletionInstruction({ jobId: 'shell_job_1', status: 'completed', exitCode: 0 }),
-    'The async shell task shell_job_1 has finished (completed, exit 0) - review this result in your next step.',
-  );
-  assert.match(shellCompletionInstruction({ jobId: 'x', status: 'failed', exitCode: null }), /exit n\/a/);
-});
-
-test('prompt-stall envelope is byte-compatible with historical inline assembly', () => {
-  const jobId = 'shell_job_1';
-  const stalledMs = 5000;
-  const elapsedMs = 6000;
-  const tail = { text: 'Password:' };
-  const detail = { command: 'ssh host' };
-  const expected = [
-    `[task_id: ${jobId}]`,
-    '[status: running]',
-    `[stalled: no output growth for ${stalledMs} ms]`,
-    elapsedMs >= 0 ? `[elapsed: ${elapsedMs} ms]` : null,
-    detail.command ? `[command: ${detail.command}]` : null,
-    '',
-    'This background shell task appears to be waiting for interactive input. Background tasks cannot answer prompts automatically; cancel it or rerun with non-interactive flags/input.',
-    tail.text ? `\n${tail.text}` : null,
-  ].filter((line) => line !== null && line !== '').join('\n');
-  const actual = renderShellPromptStallEnvelope({ jobId, stalledMs, elapsedMs, command: detail.command, tailText: tail.text });
-  assert.equal(actual, expected);
-  assert.equal(isInternalRuntimeNotificationText(actual), true);
-});
-
-test('prompt-stall instruction wording unchanged', () => {
-  assert.equal(
-    shellPromptStallInstruction({ jobId: 'shell_job_1' }),
-    'The background shell task shell_job_1 appears to be waiting for interactive input; inspect the prompt, then cancel or rerun it non-interactively.',
-  );
-});

package/src/runtime/shared/tool-execution-contract.test.mjs

@@ -1,183 +0,0 @@
-import assert from 'node:assert/strict';
-import test from 'node:test';
-import {
-  isInternalRuntimeNotificationText,
-  isModelVisibleToolCompletionWrapper,
-  modelVisibleToolCompletionMessage,
-  shouldPersistModelVisibleToolCompletion,
-  toolCompletionMeta,
-  toolCompletionInstruction,
-} from './tool-execution-contract.mjs';
-import { _isInternalRuntimeNotificationText } from '../agent/orchestrator/session/manager.mjs';
-
-const completedBackgroundTask = [
-  'background task',
-  'task_id: task_agent_abc123',
-  'surface: agent',
-  'operation: run',
-  'status: completed',
-  'started: 2026-03-28T00:00:00.000Z',
-  'finished: 2026-03-28T00:00:05.000Z',
-  '',
-  'worker finished successfully',
-].join('\n');
-
-const runningBackgroundTask = [
-  'background task',
-  'task_id: task_agent_abc123',
-  'surface: agent',
-  'operation: run',
-  'status: running',
-  'started: 2026-03-28T00:00:00.000Z',
-  'notification: completion will be delivered to the owner session; use status/read only for manual recovery.',
-].join('\n');
-
-test('terminal background task with body should persist as model-visible wrapper', () => {
-  const meta = toolCompletionMeta({
-    surface: 'agent',
-    id: 'task_agent_abc123',
-    status: 'completed',
-    context: { callerSessionId: 'sess_owner' },
-  });
-  assert.equal(shouldPersistModelVisibleToolCompletion(completedBackgroundTask, meta), true);
-  const visible = modelVisibleToolCompletionMessage(completedBackgroundTask, meta);
-  assert.ok(visible);
-  assert.equal(isInternalRuntimeNotificationText(visible), false);
-  assert.equal(_isInternalRuntimeNotificationText(visible), false);
-  assert.equal(isModelVisibleToolCompletionWrapper(visible), true);
-  assert.match(visible, /worker finished successfully/);
-});
-
-test('model-visible completion wrapper is not a human transcript item', () => {
-  const meta = toolCompletionMeta({
-    surface: 'agent',
-    id: 'task_agent_abc123',
-    status: 'completed',
-    instruction: toolCompletionInstruction({
-      surface: 'agent',
-      id: 'task_agent_abc123',
-      status: 'completed',
-    }),
-    context: { callerSessionId: 'sess_owner' },
-  });
-  const visible = modelVisibleToolCompletionMessage(completedBackgroundTask, meta);
-  assert.ok(visible);
-  assert.match(visible, /^The async agent task/);
-  assert.match(visible, /\n\nResult:\n/);
-  assert.equal(isModelVisibleToolCompletionWrapper(visible), true);
-  assert.equal(isModelVisibleToolCompletionWrapper('please fix the leak in engine.mjs'), false);
-  assert.equal(isModelVisibleToolCompletionWrapper(completedBackgroundTask), false);
-});
-
-test('running background task stays hidden from model-visible pending drain', () => {
-  const meta = toolCompletionMeta({
-    surface: 'agent',
-    id: 'task_agent_abc123',
-    status: 'running',
-    context: { callerSessionId: 'sess_owner' },
-  });
-  assert.equal(shouldPersistModelVisibleToolCompletion(runningBackgroundTask, meta), false);
-  assert.equal(modelVisibleToolCompletionMessage(runningBackgroundTask, meta), '');
-});
-
-test('terminal background task persists when result body mentions task-notification', () => {
-  const taskWithTagInBody = [
-    ...completedBackgroundTask.split('\n').slice(0, -1),
-    'log line: saw <task-notification status="completed"> in worker output',
-  ].join('\n');
-  const meta = toolCompletionMeta({
-    surface: 'agent',
-    id: 'task_agent_abc123',
-    status: 'completed',
-    context: { callerSessionId: 'sess_owner' },
-  });
-  assert.equal(shouldPersistModelVisibleToolCompletion(taskWithTagInBody, meta), true);
-  const visible = modelVisibleToolCompletionMessage(taskWithTagInBody, meta);
-  assert.ok(visible);
-  assert.match(visible, /<task-notification/);
-  assert.equal(isInternalRuntimeNotificationText(visible), false);
-  assert.equal(_isInternalRuntimeNotificationText(visible), false);
-});
-
-test('internal task-notification envelope does not persist', () => {
-  const internalEnvelope = [
-    '<task-notification>',
-    '<status>completed</status>',
-    '<summary>Agent completed</summary>',
-    '<task-id>task_agent_internal</task-id>',
-    '</task-notification>',
-  ].join('\n');
-  const meta = toolCompletionMeta({
-    surface: 'agent',
-    id: 'task_agent_internal',
-    status: 'completed',
-    context: { callerSessionId: 'sess_owner' },
-  });
-  assert.equal(shouldPersistModelVisibleToolCompletion(internalEnvelope, meta), false);
-  assert.equal(modelVisibleToolCompletionMessage(internalEnvelope, meta), '');
-  assert.equal(isInternalRuntimeNotificationText(internalEnvelope), true);
-  assert.equal(_isInternalRuntimeNotificationText(internalEnvelope), true);
-});
-
-test('raw running background task remains internal for manager pending drain', () => {
-  assert.equal(isInternalRuntimeNotificationText(runningBackgroundTask), true);
-  assert.equal(_isInternalRuntimeNotificationText(runningBackgroundTask), true);
-});
-
-
-
-const bracketedShellCompleted = [
-  "[task_id: shell_job_1]",
-  "[status: completed]",
-  "[exit: 0]",
-  "",
-  "done output",
-].join("\n");
-
-const bracketedShellRunning = [
-  "[task_id: shell_job_1]",
-  "[status: running]",
-  "[command: npm test]",
-].join("\n");
-
-test("bracketed shell completed raw envelope is internal", () => {
-  assert.equal(isInternalRuntimeNotificationText(bracketedShellCompleted), true);
-  assert.equal(_isInternalRuntimeNotificationText(bracketedShellCompleted), true);
-});
-
-test("bracketed shell running raw envelope is internal", () => {
-  assert.equal(isInternalRuntimeNotificationText(bracketedShellRunning), true);
-  assert.equal(_isInternalRuntimeNotificationText(bracketedShellRunning), true);
-});
-
-test("wrapped shell completion stays model-visible and not internal", () => {
-  const meta = toolCompletionMeta({
-    surface: "shell",
-    id: "shell_job_1",
-    status: "completed",
-    context: { callerSessionId: "sess_owner" },
-  });
-  const visible = modelVisibleToolCompletionMessage(bracketedShellCompleted, meta);
-  assert.ok(visible);
-  assert.match(visible, /done output/);
-  assert.equal(isInternalRuntimeNotificationText(visible), false);
-  assert.equal(_isInternalRuntimeNotificationText(visible), false);
-});
-
-test("model-visible wrapper bounds large completion bodies", () => {
-  const hugeBody = "x".repeat(20_000);
-  const hugeTask = [
-    ...completedBackgroundTask.split("\n").slice(0, -1),
-    hugeBody,
-  ].join("\n");
-  const meta = toolCompletionMeta({
-    surface: "agent",
-    id: "task_agent_abc123",
-    status: "completed",
-    context: { callerSessionId: "sess_owner" },
-  });
-  const visible = modelVisibleToolCompletionMessage(hugeTask, meta);
-  assert.ok(visible);
-  assert.match(visible, /\[result truncated for model context\]/);
-  assert.ok(visible.length < hugeBody.length);
-});

package/src/standalone/agent-task-status.test.mjs

@@ -1,76 +0,0 @@
-import assert from 'node:assert/strict';
-import test from 'node:test';
-import {
-  appendAgentProgressKv,
-  buildAgentTaskProgressFields,
-  resolveSilentForSeconds,
-} from './agent-task-status.mjs';
-
-test('waiting for first activity explains silence', () => {
-  const now = 1_000_000;
-  const fields = buildAgentTaskProgressFields({
-    now,
-    sessionStatus: 'running',
-    runtimeStage: 'requesting',
-    snapshot: {
-      stage: 'requesting',
-      waitingForFirstActivity: true,
-      modelRequestStartedAt: now - 12_000,
-      askStartedAt: now - 12_000,
-      lastProgressAt: now - 12_000,
-    },
-    policy: { firstResponseMs: 120_000, idleStaleMs: 1_800_000, toolRunningMs: 600_000 },
-    taskStatus: 'running',
-  });
-  assert.equal(fields.last_progress, 'awaiting first model response');
-  assert.match(fields.diagnostic, /waiting for first response/);
-  assert.match(fields.watchdog, /first=120s/);
-});
-
-test('streaming without recent deltas is called out', () => {
-  const now = 2_000_000;
-  const fields = buildAgentTaskProgressFields({
-    now,
-    sessionStatus: 'running',
-    runtimeStage: 'streaming',
-    snapshot: {
-      stage: 'streaming',
-      waitingForFirstActivity: false,
-      lastStreamDeltaAt: now - 23_000,
-      lastProgressAt: now - 23_000,
-    },
-    taskStatus: 'running',
-  });
-  assert.match(fields.last_progress, /no stream delta for 23s/);
-  assert.match(fields.diagnostic, /no visible output yet \(23s\)/);
-  assert.equal(fields.silent_for, 23);
-});
-
-test('queued follow-ups surface on idle worker', () => {
-  const fields = buildAgentTaskProgressFields({
-    sessionStatus: 'idle',
-    runtimeStage: 'streaming',
-    queuedFollowups: 2,
-    taskStatus: 'running',
-  });
-  assert.equal(fields.queued_followups, 2);
-  assert.match(fields.diagnostic, /2 follow-ups queued/);
-});
-
-test('resolveSilentForSeconds uses latest progress signal', () => {
-  const now = 10_000;
-  const silent = resolveSilentForSeconds(now, { lastProgressAt: now - 5_000 }, { lastStreamDeltaAt: now - 9_000 });
-  assert.equal(silent, 5);
-});
-
-test('appendAgentProgressKv keeps base line first', () => {
-  const line = appendAgentProgressKv('- job1 spawn running', {
-    worker_stage: 'streaming',
-    silent_for: 4,
-    diagnostic: 'streaming',
-  });
-  assert.match(line, /^- job1 spawn running /);
-  assert.match(line, /stage=streaming/);
-  assert.match(line, /hint=streaming/);
-});
-