@creativeintelligence/abbie 0.1.4 → 0.1.5

package/src/lib/runner.test.ts

@@ -1,288 +0,0 @@
-import { afterEach, beforeEach, describe, expect, it } from "bun:test";
-import { spawn } from "node:child_process";
-import { existsSync, mkdtempSync, readFileSync, rmSync } from "node:fs";
-import { tmpdir } from "node:os";
-import { join } from "node:path";
-import { type ExitInfo, parseRunnerConfig } from "./runner.js";
-
-describe("runner", () => {
-  describe("parseRunnerConfig", () => {
-    it("returns null when required env vars are missing", () => {
-      const config = parseRunnerConfig({});
-      expect(config).toBeNull();
-    });
-
-    it("parses valid config from environment", () => {
-      const config = parseRunnerConfig({
-        AGENTS_RUNNER_SESSION_ID: "test-session-123",
-        AGENTS_RUNNER_SESSION_PATH: "/tmp/session.json",
-        AGENTS_RUNNER_PROMPT_PATH: "/tmp/prompt.md",
-        AGENTS_RUNNER_OUTPUT_PATH: "/tmp/output.json",
-        AGENTS_RUNNER_AGENT: "claude",
-      });
-      expect(config).not.toBeNull();
-      expect(config!.sessionId).toBe("test-session-123");
-      expect(config!.agent).toBe("claude");
-      expect(config!.outputPath).toBe("/tmp/output.json");
-      expect(config!.resultPath).toBe("/tmp/test-session-123-result.json");
-    });
-
-    it("derives stdout/stderr paths from output path", () => {
-      const config = parseRunnerConfig({
-        AGENTS_RUNNER_SESSION_ID: "test-session",
-        AGENTS_RUNNER_SESSION_PATH: "/tmp/session.json",
-        AGENTS_RUNNER_PROMPT_PATH: "/tmp/prompt.md",
-        AGENTS_RUNNER_OUTPUT_PATH: "/tmp/output.json",
-        AGENTS_RUNNER_AGENT: "codex",
-      });
-      expect(config).not.toBeNull();
-      expect(config!.stdoutPath).toBe("/tmp/output-stdout.log");
-      expect(config!.stderrPath).toBe("/tmp/output-stderr.log");
-    });
-
-    it("uses explicit stdout/stderr paths when provided", () => {
-      const config = parseRunnerConfig({
-        AGENTS_RUNNER_SESSION_ID: "test-session",
-        AGENTS_RUNNER_SESSION_PATH: "/tmp/session.json",
-        AGENTS_RUNNER_PROMPT_PATH: "/tmp/prompt.md",
-        AGENTS_RUNNER_OUTPUT_PATH: "/tmp/output.json",
-        AGENTS_RUNNER_STDOUT_PATH: "/tmp/custom-stdout.log",
-        AGENTS_RUNNER_STDERR_PATH: "/tmp/custom-stderr.log",
-        AGENTS_RUNNER_AGENT: "copilot",
-      });
-      expect(config).not.toBeNull();
-      expect(config!.stdoutPath).toBe("/tmp/custom-stdout.log");
-      expect(config!.stderrPath).toBe("/tmp/custom-stderr.log");
-    });
-
-    it("uses explicit result path when provided", () => {
-      const config = parseRunnerConfig({
-        AGENTS_RUNNER_SESSION_ID: "test-session",
-        AGENTS_RUNNER_SESSION_PATH: "/tmp/session.json",
-        AGENTS_RUNNER_PROMPT_PATH: "/tmp/prompt.md",
-        AGENTS_RUNNER_OUTPUT_PATH: "/tmp/output.json",
-        AGENTS_RUNNER_RESULT_PATH: "/tmp/custom-result.json",
-        AGENTS_RUNNER_AGENT: "codex",
-      });
-      expect(config).not.toBeNull();
-      expect(config!.resultPath).toBe("/tmp/custom-result.json");
-    });
-
-    it("includes model when provided", () => {
-      const config = parseRunnerConfig({
-        AGENTS_RUNNER_SESSION_ID: "test-session",
-        AGENTS_RUNNER_SESSION_PATH: "/tmp/session.json",
-        AGENTS_RUNNER_PROMPT_PATH: "/tmp/prompt.md",
-        AGENTS_RUNNER_OUTPUT_PATH: "/tmp/output.json",
-        AGENTS_RUNNER_AGENT: "gemini",
-        AGENTS_RUNNER_MODEL: "gemini-3-flash",
-      });
-      expect(config).not.toBeNull();
-      expect(config!.model).toBe("gemini-3-flash");
-    });
-  });
-
-  describe("spawn failure handling", () => {
-    let tempDir: string;
-
-    beforeEach(() => {
-      tempDir = mkdtempSync(join(tmpdir(), "runner-test-"));
-    });
-
-    afterEach(() => {
-      rmSync(tempDir, { recursive: true, force: true });
-    });
-
-    it("handles ENOENT when command does not exist", async () => {
-      // This tests the spawn error handling path
-      const child = spawn("/nonexistent/command", [], {
-        stdio: "ignore",
-      });
-
-      const errorPromise = new Promise<Error>((resolve) => {
-        child.on("error", resolve);
-      });
-
-      const error = await errorPromise;
-      expect(error.message).toContain("ENOENT");
-    });
-
-    it("handles spawn with invalid cwd", async () => {
-      const child = spawn("echo", ["hello"], {
-        cwd: "/nonexistent/directory",
-        stdio: "ignore",
-      });
-
-      const errorPromise = new Promise<Error>((resolve) => {
-        child.on("error", resolve);
-      });
-
-      const error = await errorPromise;
-      expect(error.message).toContain("ENOENT");
-    });
-  });
-
-  describe("signal forwarding", () => {
-    let tempDir: string;
-
-    beforeEach(() => {
-      tempDir = mkdtempSync(join(tmpdir(), "runner-signal-test-"));
-    });
-
-    afterEach(() => {
-      rmSync(tempDir, { recursive: true, force: true });
-    });
-
-    it("child process receives SIGTERM", async () => {
-      // Spawn a process that traps SIGTERM and writes to a file
-      const markerFile = join(tempDir, "received-signal");
-      const child = spawn(
-        "bash",
-        ["-c", `trap 'echo SIGTERM > ${markerFile}; exit 0' TERM; while true; do sleep 0.1; done`],
-        {
-          detached: true,
-          stdio: "ignore",
-        },
-      );
-
-      // Wait for process to start and trap to be installed
-      await new Promise((r) => setTimeout(r, 300));
-
-      // Send SIGTERM
-      process.kill(child.pid!, "SIGTERM");
-
-      // Wait for handler and exit (longer timeout for CI)
-      await new Promise((r) => setTimeout(r, 500));
-
-      // Check the marker file - if it exists, signal was received
-      if (existsSync(markerFile)) {
-        expect(readFileSync(markerFile, "utf-8").trim()).toBe("SIGTERM");
-      } else {
-        // Process may have been killed before trap installed - this is acceptable
-        // Just verify the process is no longer running
-        let isRunning = false;
-        try {
-          process.kill(child.pid!, 0);
-          isRunning = true;
-        } catch {
-          isRunning = false;
-        }
-        expect(isRunning).toBe(false);
-      }
-    });
-
-    it("process group kill terminates child tree", async () => {
-      // Spawn a parent that spawns a child
-      const markerFile = join(tempDir, "parent-pid");
-      const child = spawn("bash", ["-c", `echo $$ > ${markerFile}; sleep 10`], {
-        detached: true,
-        stdio: "ignore",
-      });
-
-      // Wait for process to start
-      await new Promise((r) => setTimeout(r, 100));
-
-      const pid = child.pid!;
-
-      // Kill the process group (negative PID)
-      try {
-        process.kill(-pid, "SIGKILL");
-      } catch {
-        // May already be dead or group not available on some platforms
-      }
-
-      // Wait for termination
-      await new Promise((r) => setTimeout(r, 100));
-
-      // Verify process is gone
-      let isRunning = false;
-      try {
-        process.kill(pid, 0); // Signal 0 checks if process exists
-        isRunning = true;
-      } catch {
-        isRunning = false;
-      }
-
-      expect(isRunning).toBe(false);
-    });
-  });
-
-  describe("ExitInfo structure", () => {
-    it("captures exit code and signal correctly", () => {
-      const exitInfo: ExitInfo = {
-        exitCode: 0,
-        signal: null,
-        endedAt: new Date().toISOString(),
-      };
-
-      expect(exitInfo.exitCode).toBe(0);
-      expect(exitInfo.signal).toBeNull();
-      expect(exitInfo.endedAt).toMatch(/^\d{4}-\d{2}-\d{2}T/);
-    });
-
-    it("captures signal when process is killed", () => {
-      const exitInfo: ExitInfo = {
-        exitCode: null,
-        signal: "SIGTERM",
-        endedAt: new Date().toISOString(),
-      };
-
-      expect(exitInfo.exitCode).toBeNull();
-      expect(exitInfo.signal).toBe("SIGTERM");
-    });
-  });
-
-  describe("output file handling", () => {
-    let tempDir: string;
-
-    beforeEach(() => {
-      tempDir = mkdtempSync(join(tmpdir(), "runner-output-test-"));
-    });
-
-    afterEach(() => {
-      rmSync(tempDir, { recursive: true, force: true });
-    });
-
-    it("captures stdout to output file", async () => {
-      const outputFile = join(tempDir, "output.txt");
-
-      const child = spawn("echo", ["hello world"], {
-        stdio: ["ignore", "pipe", "ignore"],
-      });
-
-      const { createWriteStream } = await import("node:fs");
-      const stream = createWriteStream(outputFile);
-      child.stdout?.pipe(stream);
-
-      await new Promise<void>((resolve) => {
-        child.on("close", () => {
-          stream.end(() => resolve());
-        });
-      });
-
-      expect(existsSync(outputFile)).toBe(true);
-      expect(readFileSync(outputFile, "utf-8").trim()).toBe("hello world");
-    });
-
-    it("captures stderr separately", async () => {
-      const stderrFile = join(tempDir, "stderr.txt");
-
-      const child = spawn("bash", ["-c", "echo error >&2"], {
-        stdio: ["ignore", "ignore", "pipe"],
-      });
-
-      const { createWriteStream } = await import("node:fs");
-      const stream = createWriteStream(stderrFile);
-      child.stderr?.pipe(stream);
-
-      await new Promise<void>((resolve) => {
-        child.on("close", () => {
-          stream.end(() => resolve());
-        });
-      });
-
-      expect(existsSync(stderrFile)).toBe(true);
-      expect(readFileSync(stderrFile, "utf-8").trim()).toBe("error");
-    });
-  });
-});