@vellumai/assistant 0.8.7-dev.202606052232.2ddc989 → 0.8.8

package/src/acp/__tests__/helpers/acp-history-db.ts

@@ -1,82 +0,0 @@
-/**
- * Shared test helpers for the `acp_session_history` table.
- *
- * Several suites (session-manager resume/persistence, the ACP route
- * handlers) seed and read history rows directly via SQL; this module
- * consolidates the copy-pasted row shape + insert/read/clear helpers.
- * Tests must run `initializeDb()` themselves before calling these.
- */
-
-import { getSqlite } from "../../../memory/db-connection.js";
-
-/** Raw column snapshot of an `acp_session_history` row. */
-export interface HistoryRow {
-  id: string;
-  agent_id: string;
-  acp_session_id: string;
-  parent_conversation_id: string;
-  started_at: number;
-  completed_at: number | null;
-  status: string;
-  stop_reason: string | null;
-  error: string | null;
-  event_log_json: string;
-  cwd: string | null;
-}
-
-export function clearHistory(): void {
-  getSqlite().run("DELETE FROM acp_session_history");
-}
-
-/**
- * Inserts a history row. Every field except `id` defaults to the values
- * the resume suites were built around (a completed claude run in
- * /tmp/proj); pass explicit values to override, including `null` where the
- * column is nullable (e.g. `cwd: null` for legacy rows).
- */
-export function insertHistoryRow(row: {
-  id: string;
-  agentId?: string;
-  acpSessionId?: string;
-  parentConversationId?: string;
-  startedAt?: number;
-  completedAt?: number | null;
-  status?: string;
-  stopReason?: string | null;
-  error?: string | null;
-  eventLogJson?: string;
-  cwd?: string | null;
-}): void {
-  getSqlite()
-    .query(
-      `INSERT INTO acp_session_history (
-         id, agent_id, acp_session_id, parent_conversation_id,
-         started_at, completed_at, status, stop_reason, error,
-         event_log_json, cwd
-       ) VALUES (?, ?, ?, ?, ?, ?, ?, ?, ?, ?, ?)`,
-    )
-    .run(
-      row.id,
-      row.agentId ?? "claude",
-      row.acpSessionId ?? "proto-old",
-      row.parentConversationId ?? "conv-1",
-      row.startedAt ?? 1234,
-      row.completedAt === undefined ? 5678 : row.completedAt,
-      row.status ?? "completed",
-      row.stopReason === undefined ? "end_turn" : row.stopReason,
-      row.error ?? null,
-      row.eventLogJson ?? "[]",
-      row.cwd === undefined ? "/tmp/proj" : row.cwd,
-    );
-}
-
-export function readHistoryRow(id: string): HistoryRow | null {
-  return getSqlite()
-    .query(
-      `SELECT id, agent_id, acp_session_id, parent_conversation_id,
-              started_at, completed_at, status, stop_reason, error,
-              event_log_json, cwd
-       FROM acp_session_history WHERE id = ?`,
-    )
-    .get(id) as HistoryRow | null;
-}

package/src/acp/__tests__/helpers/exec-file-stub.ts

@@ -1,101 +0,0 @@
-/**
- * Shared test helper: stub `execFile` from `node:child_process` for ACP tests.
- *
- * Several ACP suites (the `acp_spawn` tool's version probes, the adapter
- * auto-installer, and the `/v1/acp/spawn` route) shell out via
- * `execFileWithTimeout`. Each test file used to duplicate the same
- * `mock.module("node:child_process", ...)` + scripted-responses boilerplate;
- * this helper consolidates it, mirroring `which-stub.ts`.
- *
- * Like the other helpers here, the hook is process-global by design (Bun's
- * `mock.module` is process-global). Each test file should call
- * `installExecFileStub()` once at the top level, script calls per test via
- * `execScripts`, and clear state in `beforeEach` via `reset()`.
- */
-
-import * as realChildProcess from "node:child_process";
-import { mock } from "bun:test";
-
-type ExecCallback = (
-  err: Error | null,
-  stdout: string | Buffer,
-  stderr: string | Buffer,
-) => void;
-
-export interface ExecScript {
-  /** When set, the call rejects with this error. */
-  error?: Error;
-  /** When set, the call resolves with this stdout. */
-  stdout?: string;
-  /** When set, runs as the call executes (e.g. flip a which-stub flag). */
-  onCall?: () => void;
-}
-
-type ExecFileMock = ReturnType<
-  typeof mock<
-    (
-      command: string,
-      args: string[],
-      options: unknown,
-      callback?: ExecCallback,
-    ) => ReturnType<typeof realChildProcess.execFile>
-  >
->;
-
-export interface ExecFileStubHandle {
-  /**
-   * Per-call scripted responses, keyed by `${command} ${args[0]}` so tests
-   * can target `npm ls`, `npm view`, and `npm i` independently. Calls with
-   * no script reject with a recognizable "No script for <key>" error.
-   */
-  execScripts: Map<string, ExecScript>;
-  execFileMock: ExecFileMock;
-  /** Clears scripts and recorded calls. Call from `beforeEach`. */
-  reset(): void;
-}
-
-/** Installs a process-global `execFile` mock. Returns handles to drive it. */
-export function installExecFileStub(): ExecFileStubHandle {
-  const execScripts: Map<string, ExecScript> = new Map();
-
-  const execFileMock: ExecFileMock = mock(
-    (
-      command: string,
-      args: string[],
-      _options: unknown,
-      callback?: ExecCallback,
-    ) => {
-      const key = `${command} ${args[0]}`;
-      const script = execScripts.get(key);
-      queueMicrotask(() => {
-        if (!callback) return;
-        if (!script) {
-          callback(new Error(`No script for ${key}`), "", "");
-          return;
-        }
-        script.onCall?.();
-        if (script.error) {
-          callback(script.error, "", "");
-          return;
-        }
-        callback(null, script.stdout ?? "", "");
-      });
-      // Return value is not used by execFileWithTimeout.
-      return {} as ReturnType<typeof realChildProcess.execFile>;
-    },
-  );
-
-  mock.module("node:child_process", () => ({
-    ...realChildProcess,
-    execFile: execFileMock,
-  }));
-
-  return {
-    execScripts,
-    execFileMock,
-    reset(): void {
-      execScripts.clear();
-      execFileMock.mockClear();
-    },
-  };
-}