@aria-cli/tools 1.0.9 → 1.0.11

package/tests/executors/session-history.test.ts

@@ -1,340 +0,0 @@
-import { describe, it, expect, vi, beforeEach } from "vitest";
-import { executeSessionHistory } from "../../src/executors/session-history.js";
-import type {
-  ToolContext,
-  SessionHistoryRef,
-  SessionSummaryRef,
-  SessionContextRef,
-} from "../../src/types.js";
-
-// ── Helpers ──────────────────────────────────────────────────────────────────
-
-function makeSummary(overrides: Partial<SessionSummaryRef> = {}): SessionSummaryRef {
-  return {
-    id: "aaaaaaaa-bbbb-cccc-dddd-eeeeeeeeeeee",
-    createdAt: new Date("2026-03-01T10:00:00Z"),
-    updatedAt: new Date("2026-03-01T11:00:00Z"),
-    title: "Test session",
-    arion: "aria",
-    model: "claude-4-sonnet",
-    messageCount: 10,
-    preview: "Hello, how can I help?",
-    ...overrides,
-  };
-}
-
-function makeSessionContext(overrides: Partial<SessionContextRef> = {}): SessionContextRef {
-  return {
-    id: "aaaaaaaa-bbbb-cccc-dddd-eeeeeeeeeeee",
-    arion: "aria",
-    model: "claude-4-sonnet",
-    messages: [
-      {
-        id: 1,
-        role: "user",
-        content: "Hello",
-        createdAt: new Date("2026-03-01T10:00:00Z"),
-      },
-      {
-        id: 2,
-        role: "assistant",
-        content: "Hi! How can I help?",
-        createdAt: new Date("2026-03-01T10:00:01Z"),
-      },
-    ],
-    ...overrides,
-  };
-}
-
-function mockSessionHistory(overrides: Partial<SessionHistoryRef> = {}): SessionHistoryRef {
-  return {
-    listSessions: vi.fn().mockReturnValue([makeSummary()]),
-    searchSessionsFts: vi.fn().mockReturnValue([makeSummary()]),
-    getSession: vi.fn().mockReturnValue(makeSessionContext()),
-    getIncompleteSessions: vi.fn().mockReturnValue([]),
-    deleteSession: vi.fn(),
-    markCompleted: vi.fn(),
-    setSessionTitle: vi.fn(),
-    findSessionByPrefix: vi.fn().mockReturnValue("aaaaaaaa-bbbb-cccc-dddd-eeeeeeeeeeee"),
-    getSessionCount: vi.fn().mockReturnValue(42),
-    getRunMetrics: vi.fn().mockReturnValue([]),
-    ...overrides,
-  };
-}
-
-function makeCtx(sessionHistory?: SessionHistoryRef, currentSessionId?: string): ToolContext {
-  return {
-    workingDir: "/tmp",
-    env: {},
-    confirm: vi.fn().mockResolvedValue(true),
-    sessionHistory,
-    currentSessionId,
-  } as unknown as ToolContext;
-}
-
-// ── Tests ────────────────────────────────────────────────────────────────────
-
-describe("executeSessionHistory", () => {
-  let sh: SessionHistoryRef;
-  let ctx: ToolContext;
-
-  beforeEach(() => {
-    sh = mockSessionHistory();
-    ctx = makeCtx(sh, "current-session-id-1234");
-  });
-
-  // ── Missing prerequisites ──
-
-  it("fails when action is missing", async () => {
-    const result = await executeSessionHistory({} as any, ctx);
-    expect(result.success).toBe(false);
-    expect(result.message).toContain("action is required");
-  });
-
-  it("falls back to dynamic import when sessionHistory is not in context", async () => {
-    const ctxNoHistory = makeCtx(undefined);
-    const result = await executeSessionHistory({ action: "list" }, ctxNoHistory);
-    // In the monorepo, the fallback dynamic import of @aria/cli succeeds,
-    // so the tool works even without ctx.sessionHistory.  It either succeeds
-    // (with real data from the DB) or fails with "not available" in isolated
-    // environments where @aria/cli can't be located.
-    expect(typeof result.success).toBe("boolean");
-  });
-
-  it("fails on unknown action", async () => {
-    const result = await executeSessionHistory({ action: "foobar" as any }, ctx);
-    expect(result.success).toBe(false);
-    expect(result.message).toContain("Unknown action");
-  });
-
-  it("respects abort signal", async () => {
-    const controller = new AbortController();
-    controller.abort();
-    const ctxAborted = { ...ctx, abortSignal: controller.signal } as ToolContext;
-    const result = await executeSessionHistory({ action: "list" }, ctxAborted);
-    expect(result.success).toBe(false);
-    expect(result.message).toContain("cancelled");
-  });
-
-  // ── list ──
-
-  describe("list", () => {
-    it("lists sessions", async () => {
-      const result = await executeSessionHistory({ action: "list" }, ctx);
-      expect(result.success).toBe(true);
-      expect(sh.listSessions).toHaveBeenCalledWith(20, 0);
-      expect(result.data).toHaveProperty("sessions");
-      const data = result.data as any;
-      expect(data.sessions).toHaveLength(1);
-      expect(data.sessions[0].id).toBe("aaaaaaaa-bbbb-cccc-dddd-eeeeeeeeeeee");
-    });
-
-    it("passes limit and offset", async () => {
-      await executeSessionHistory({ action: "list", limit: 5, offset: 10 }, ctx);
-      expect(sh.listSessions).toHaveBeenCalledWith(5, 10);
-    });
-
-    it("returns empty message when no sessions", async () => {
-      (sh.listSessions as any).mockReturnValue([]);
-      const result = await executeSessionHistory({ action: "list" }, ctx);
-      expect(result.success).toBe(true);
-      expect(result.message).toContain("No sessions found");
-    });
-  });
-
-  // ── search ──
-
-  describe("search", () => {
-    it("searches sessions by query", async () => {
-      const result = await executeSessionHistory({ action: "search", query: "deployment" }, ctx);
-      expect(result.success).toBe(true);
-      expect(sh.searchSessionsFts).toHaveBeenCalledWith("deployment", 20, 0);
-      const data = result.data as any;
-      expect(data.sessions).toHaveLength(1);
-    });
-
-    it("fails without query", async () => {
-      const result = await executeSessionHistory({ action: "search" }, ctx);
-      expect(result.success).toBe(false);
-      expect(result.message).toContain("query is required");
-    });
-
-    it("fails with empty query", async () => {
-      const result = await executeSessionHistory({ action: "search", query: "  " }, ctx);
-      expect(result.success).toBe(false);
-      expect(result.message).toContain("query is required");
-    });
-
-    it("returns empty message when no results", async () => {
-      (sh.searchSessionsFts as any).mockReturnValue([]);
-      const result = await executeSessionHistory({ action: "search", query: "nonexistent" }, ctx);
-      expect(result.success).toBe(true);
-      expect(result.message).toContain("No sessions found matching");
-    });
-  });
-
-  // ── get ──
-
-  describe("get", () => {
-    it("gets a session by ID", async () => {
-      const result = await executeSessionHistory(
-        { action: "get", sessionId: "aaaaaaaa-bbbb-cccc-dddd-eeeeeeeeeeee" },
-        ctx,
-      );
-      expect(result.success).toBe(true);
-      const data = result.data as any;
-      expect(data.id).toBe("aaaaaaaa-bbbb-cccc-dddd-eeeeeeeeeeee");
-      expect(data.messages).toHaveLength(2);
-      expect(data.messages[0].role).toBe("user");
-    });
-
-    it("gets a session by prefix", async () => {
-      const result = await executeSessionHistory({ action: "get", sessionId: "aaaaaaaa-bb" }, ctx);
-      expect(result.success).toBe(true);
-      expect(sh.findSessionByPrefix).toHaveBeenCalledWith("aaaaaaaa-bb");
-    });
-
-    it("fails without sessionId", async () => {
-      const result = await executeSessionHistory({ action: "get" }, ctx);
-      expect(result.success).toBe(false);
-      expect(result.message).toContain("sessionId is required");
-    });
-
-    it("fails when session not found", async () => {
-      (sh.findSessionByPrefix as any).mockReturnValue(null);
-      const result = await executeSessionHistory({ action: "get", sessionId: "deadbeef" }, ctx);
-      expect(result.success).toBe(false);
-      expect(result.message).toContain("not found");
-    });
-
-    it("respects messageLimit", async () => {
-      const longSession = makeSessionContext({
-        messages: Array.from({ length: 100 }, (_, i) => ({
-          id: i,
-          role: "user" as const,
-          content: `Message ${i}`,
-          createdAt: new Date(),
-        })),
-      });
-      (sh.getSession as any).mockReturnValue(longSession);
-
-      const result = await executeSessionHistory(
-        { action: "get", sessionId: "aaaaaaaa-bbbb-cccc-dddd-eeeeeeeeeeee", messageLimit: 10 },
-        ctx,
-      );
-      expect(result.success).toBe(true);
-      const data = result.data as any;
-      expect(data.messages).toHaveLength(10);
-      // Should be the LAST 10 messages
-      expect(data.messages[0].content).toBe("Message 90");
-    });
-
-    it("truncates long message content", async () => {
-      const longContent = "x".repeat(5000);
-      const session = makeSessionContext({
-        messages: [{ id: 1, role: "user", content: longContent, createdAt: new Date() }],
-      });
-      (sh.getSession as any).mockReturnValue(session);
-
-      const result = await executeSessionHistory(
-        { action: "get", sessionId: "aaaaaaaa-bbbb-cccc-dddd-eeeeeeeeeeee" },
-        ctx,
-      );
-      const data = result.data as any;
-      expect(data.messages[0].content.length).toBeLessThan(longContent.length);
-      expect(data.messages[0].content).toMatch(/\.\.\.$/);
-    });
-  });
-
-  // ── current ──
-
-  describe("current", () => {
-    it("returns current session info", async () => {
-      const result = await executeSessionHistory({ action: "current" }, ctx);
-      expect(result.success).toBe(true);
-      const data = result.data as any;
-      expect(data.sessionId).toBe("current-session-id-1234");
-    });
-
-    it("fails without current session", async () => {
-      const ctxNoCurrent = makeCtx(sh, undefined);
-      const result = await executeSessionHistory({ action: "current" }, ctxNoCurrent);
-      expect(result.success).toBe(false);
-      expect(result.message).toContain("No current session");
-    });
-  });
-
-  // ── stats ──
-
-  describe("stats", () => {
-    it("returns session statistics", async () => {
-      (sh.getIncompleteSessions as any).mockReturnValue([makeSummary(), makeSummary()]);
-      const result = await executeSessionHistory({ action: "stats" }, ctx);
-      expect(result.success).toBe(true);
-      const data = result.data as any;
-      expect(data.totalSessions).toBe(42);
-      expect(data.incompleteSessions).toBe(2);
-    });
-  });
-
-  // ── delete ──
-
-  describe("delete", () => {
-    it("deletes a session", async () => {
-      const result = await executeSessionHistory(
-        { action: "delete", sessionId: "aaaaaaaa-bbbb-cccc-dddd-eeeeeeeeeeee" },
-        ctx,
-      );
-      expect(result.success).toBe(true);
-      expect(sh.deleteSession).toHaveBeenCalledWith("aaaaaaaa-bbbb-cccc-dddd-eeeeeeeeeeee");
-    });
-
-    it("fails without sessionId", async () => {
-      const result = await executeSessionHistory({ action: "delete" }, ctx);
-      expect(result.success).toBe(false);
-      expect(result.message).toContain("sessionId is required");
-    });
-
-    it("fails when session not found", async () => {
-      (sh.findSessionByPrefix as any).mockReturnValue(null);
-      const result = await executeSessionHistory({ action: "delete", sessionId: "deadbeef" }, ctx);
-      expect(result.success).toBe(false);
-      expect(result.message).toContain("not found");
-    });
-  });
-
-  // ── set_title ──
-
-  describe("set_title", () => {
-    it("sets session title", async () => {
-      const result = await executeSessionHistory(
-        {
-          action: "set_title",
-          sessionId: "aaaaaaaa-bbbb-cccc-dddd-eeeeeeeeeeee",
-          title: "New Title",
-        },
-        ctx,
-      );
-      expect(result.success).toBe(true);
-      expect(sh.setSessionTitle).toHaveBeenCalledWith(
-        "aaaaaaaa-bbbb-cccc-dddd-eeeeeeeeeeee",
-        "New Title",
-      );
-    });
-
-    it("fails without sessionId", async () => {
-      const result = await executeSessionHistory({ action: "set_title", title: "New Title" }, ctx);
-      expect(result.success).toBe(false);
-      expect(result.message).toContain("sessionId is required");
-    });
-
-    it("fails without title", async () => {
-      const result = await executeSessionHistory(
-        { action: "set_title", sessionId: "aaaaaaaa-bbbb-cccc-dddd-eeeeeeeeeeee" },
-        ctx,
-      );
-      expect(result.success).toBe(false);
-      expect(result.message).toContain("title is required");
-    });
-  });
-});