@stigmer/ink 3.34.0 → 3.36.0

package/src/__tests__/data-pickers.test.tsx

@@ -1,147 +0,0 @@
-import React from "react";
-import { describe, it, expect, vi } from "vitest";
-import { render } from "ink-testing-library";
-import { create } from "@bufbuild/protobuf";
-import { SearchResultSchema } from "@stigmer/protos/ai/stigmer/search/v1/io_pb";
-import { SessionSchema } from "@stigmer/protos/ai/stigmer/agentic/session/v1/api_pb";
-import { InkStigmerProvider } from "../provider.js";
-import { AgentPicker } from "../components/AgentPicker.js";
-import { SessionPicker } from "../components/SessionPicker.js";
-import { fakeClient } from "./test-support.js";
-
-const KEY = { enter: "\r" } as const;
-
-// Allow the hooks' async fetch + a re-render to settle.
-function settle(): Promise<void> {
-  return new Promise((resolve) => setTimeout(resolve, 60));
-}
-
-function agentResult(id: string, slug: string, description: string) {
-  return create(SearchResultSchema, { id, qualifiedSlug: slug, description });
-}
-
-function session(id: string, subject: string, agentInstanceId: string) {
-  return create(SessionSchema, {
-    metadata: { id },
-    spec: { subject, agentInstanceId },
-  });
-}
-
-describe("AgentPicker", () => {
-  it("renders search results and returns the chosen agent on Enter", async () => {
-    const results = [
-      agentResult("agt_1", "acme/alpha", "first"),
-      agentResult("agt_2", "acme/beta", "second"),
-    ];
-    const client = fakeClient({
-      agent: { list: vi.fn(async () => ({ entries: results, totalPages: 1, totalCount: 2 })) },
-    });
-    const onSelect = vi.fn();
-
-    const { lastFrame, stdin } = render(
-      <InkStigmerProvider client={client}>
-        <AgentPicker org="acme" onSelect={onSelect} onCancel={vi.fn()} />
-      </InkStigmerProvider>,
-    );
-
-    await settle();
-    expect(lastFrame() ?? "").toContain("acme/alpha");
-    expect(lastFrame() ?? "").toContain("first");
-
-    stdin.write(KEY.enter);
-    await settle();
-    expect(onSelect).toHaveBeenCalledTimes(1);
-    expect(onSelect.mock.calls[0][0].id).toBe("agt_1");
-  });
-
-  it("offers the built-in assistant first when asked, and picks it without an agent", async () => {
-    const results = [agentResult("agt_1", "acme/alpha", "first")];
-    const client = fakeClient({
-      agent: { list: vi.fn(async () => ({ entries: results, totalPages: 1, totalCount: 1 })) },
-    });
-    const onSelect = vi.fn();
-    const onSelectBuiltInAssistant = vi.fn();
-
-    const { lastFrame, stdin } = render(
-      <InkStigmerProvider client={client}>
-        <AgentPicker
-          org="acme"
-          onSelect={onSelect}
-          onSelectBuiltInAssistant={onSelectBuiltInAssistant}
-          onCancel={vi.fn()}
-        />
-      </InkStigmerProvider>,
-    );
-
-    await settle();
-    const frame = lastFrame() ?? "";
-    expect(frame).toContain("Assistant (built-in)");
-    // First row: above every agent the search found.
-    expect(frame.indexOf("Assistant (built-in)")).toBeLessThan(frame.indexOf("acme/alpha"));
-
-    stdin.write(KEY.enter);
-    await settle();
-    expect(onSelectBuiltInAssistant).toHaveBeenCalledTimes(1);
-    expect(onSelect).not.toHaveBeenCalled();
-  });
-
-  it("offers agents only when no built-in handler is given", async () => {
-    const client = fakeClient({
-      agent: { list: vi.fn(async () => ({ entries: [], totalPages: 0, totalCount: 0 })) },
-    });
-    const { lastFrame } = render(
-      <InkStigmerProvider client={client}>
-        <AgentPicker org="acme" onSelect={vi.fn()} onCancel={vi.fn()} />
-      </InkStigmerProvider>,
-    );
-    await settle();
-    expect(lastFrame() ?? "").not.toContain("Assistant (built-in)");
-  });
-
-  it("seeds the search box from initialQuery", async () => {
-    const list = vi.fn(async () => ({ entries: [], totalPages: 0, totalCount: 0 }));
-    const client = fakeClient({ agent: { list } });
-
-    const { lastFrame } = render(
-      <InkStigmerProvider client={client}>
-        <AgentPicker org="acme" initialQuery="deploy" onSelect={vi.fn()} onCancel={vi.fn()} />
-      </InkStigmerProvider>,
-    );
-
-    await settle();
-    // The picker pre-fills the input with the unresolved query the user
-    // already typed; useAgentSearch then re-searches it (debounced).
-    expect(lastFrame() ?? "").toContain("deploy");
-  });
-});
-
-describe("SessionPicker", () => {
-  it("client-side filters the session list and returns the chosen session", async () => {
-    const sessions = [
-      session("ses_1", "deploy staging", "ai_1"),
-      session("ses_2", "review PR", "ai_2"),
-      session("ses_3", "deploy prod", "ai_3"),
-    ];
-    const client = fakeClient({
-      session: { list: vi.fn(async () => ({ entries: sessions })) },
-    });
-    const onSelect = vi.fn();
-
-    const { lastFrame, stdin } = render(
-      <InkStigmerProvider client={client}>
-        <SessionPicker initialQuery="deploy" onSelect={onSelect} onCancel={vi.fn()} />
-      </InkStigmerProvider>,
-    );
-
-    await settle();
-    const frame = lastFrame() ?? "";
-    expect(frame).toContain("deploy staging");
-    expect(frame).toContain("deploy prod");
-    expect(frame).not.toContain("review PR");
-
-    stdin.write(KEY.enter);
-    await settle();
-    expect(onSelect).toHaveBeenCalledTimes(1);
-    expect(onSelect.mock.calls[0][0].metadata.id).toBe("ses_1");
-  });
-});

package/src/__tests__/file-change-progress.test.tsx

@@ -1,295 +0,0 @@
-import React from "react";
-import { describe, it, expect } from "vitest";
-import { render } from "ink-testing-library";
-import { create } from "@bufbuild/protobuf";
-import {
-  FileChangeProgressSchema,
-  FileChangeProgressEntrySchema,
-  CapturedFileChangeSchema,
-} from "@stigmer/protos/ai/stigmer/agentic/agentexecution/v1/filereview_pb";
-import type {
-  FileChangeProgress,
-  FileChangeProgressEntry,
-} from "@stigmer/protos/ai/stigmer/agentic/agentexecution/v1/filereview_pb";
-import { FileChangeKind } from "@stigmer/protos/ai/stigmer/agentic/agentexecution/v1/enum_pb";
-import { FileChangeProgressBar } from "../components/FileChangeProgressBar.js";
-import { changeDisplayPath, progressEntryDisplayPath } from "../file-review.js";
-
-// --- fixtures -------------------------------------------------------------
-
-interface EntryOpts {
-  readonly pathBefore?: string;
-  readonly pathAfter?: string;
-  readonly kind?: FileChangeKind;
-  readonly linesAdded?: number;
-  readonly linesRemoved?: number;
-}
-
-function makeEntry(opts: EntryOpts): FileChangeProgressEntry {
-  return create(FileChangeProgressEntrySchema, {
-    pathBefore: opts.pathBefore ?? "",
-    pathAfter: opts.pathAfter ?? "",
-    kind: opts.kind ?? FileChangeKind.MODIFY,
-    linesAdded: opts.linesAdded ?? 0,
-    linesRemoved: opts.linesRemoved ?? 0,
-  });
-}
-
-interface ProgressOpts {
-  readonly filesChanged?: number;
-  readonly linesAdded?: number;
-  readonly linesRemoved?: number;
-  readonly entries?: FileChangeProgressEntry[];
-}
-
-function makeProgress(opts: ProgressOpts): FileChangeProgress {
-  const entries = opts.entries ?? [];
-  return create(FileChangeProgressSchema, {
-    changeSetId: "cs-1",
-    filesChanged: opts.filesChanged ?? entries.length,
-    linesAdded: opts.linesAdded ?? 0,
-    linesRemoved: opts.linesRemoved ?? 0,
-    entries,
-    capturedAt: "2026-07-05T00:00:00Z",
-  });
-}
-
-// --- FileChangeProgressBar: visibility ------------------------------------
-
-describe("FileChangeProgressBar — visibility", () => {
-  it("renders nothing when progress is undefined", () => {
-    const { lastFrame } = render(<FileChangeProgressBar progress={undefined} />);
-    expect((lastFrame() ?? "").trim()).toBe("");
-  });
-
-  it("renders nothing when no files have changed yet", () => {
-    const { lastFrame } = render(
-      <FileChangeProgressBar progress={makeProgress({ filesChanged: 0 })} />,
-    );
-    expect((lastFrame() ?? "").trim()).toBe("");
-  });
-});
-
-// --- FileChangeProgressBar: compact summary -------------------------------
-
-describe("FileChangeProgressBar — compact summary", () => {
-  it("uses the singular noun for a single file", () => {
-    const { lastFrame } = render(
-      <FileChangeProgressBar
-        progress={makeProgress({
-          filesChanged: 1,
-          entries: [makeEntry({ pathAfter: "one.ts" })],
-        })}
-      />,
-    );
-    expect(lastFrame() ?? "").toContain("1 file changing");
-  });
-
-  it("uses the plural noun for multiple files", () => {
-    const { lastFrame } = render(
-      <FileChangeProgressBar
-        progress={makeProgress({
-          filesChanged: 3,
-          entries: [makeEntry({ pathAfter: "one.ts" })],
-        })}
-      />,
-    );
-    expect(lastFrame() ?? "").toContain("3 files changing");
-  });
-
-  it("shows the aggregate +N -M when counts exist", () => {
-    const { lastFrame } = render(
-      <FileChangeProgressBar
-        progress={makeProgress({
-          filesChanged: 2,
-          linesAdded: 12,
-          linesRemoved: 4,
-          entries: [makeEntry({ pathAfter: "one.ts" })],
-        })}
-      />,
-    );
-    const out = lastFrame() ?? "";
-    expect(out).toContain("+12");
-    expect(out).toContain("-4");
-  });
-
-  it("hides the aggregate stat when both counts are zero (never +0 -0)", () => {
-    const { lastFrame } = render(
-      <FileChangeProgressBar
-        progress={makeProgress({
-          filesChanged: 1,
-          linesAdded: 0,
-          linesRemoved: 0,
-          entries: [makeEntry({ pathAfter: "one.ts" })],
-        })}
-      />,
-    );
-    // No "+" anywhere means no stat rendered; the count copy carries none.
-    expect(lastFrame() ?? "").not.toContain("+");
-  });
-});
-
-// --- FileChangeProgressBar: expansion -------------------------------------
-
-describe("FileChangeProgressBar — expansion", () => {
-  const progress = makeProgress({
-    filesChanged: 2,
-    linesAdded: 8,
-    linesRemoved: 1,
-    entries: [
-      makeEntry({ pathAfter: "src/app.ts", linesAdded: 5, linesRemoved: 1 }),
-      makeEntry({ pathAfter: "src/util.ts", linesAdded: 3, linesRemoved: 0 }),
-    ],
-  });
-
-  it("hides the per-file list when collapsed (default)", () => {
-    const { lastFrame } = render(<FileChangeProgressBar progress={progress} />);
-    const out = lastFrame() ?? "";
-    expect(out).toContain("2 files changing");
-    expect(out).not.toContain("src/app.ts");
-    expect(out).not.toContain("src/util.ts");
-  });
-
-  it("shows the per-file list with kind letter, path, and stats when expanded", () => {
-    const { lastFrame } = render(
-      <FileChangeProgressBar progress={progress} expanded />,
-    );
-    const out = lastFrame() ?? "";
-    expect(out).toContain("src/app.ts");
-    expect(out).toContain("src/util.ts");
-    expect(out).toContain("M"); // MODIFY kind letter
-    expect(out).toContain("+5");
-    expect(out).toContain("+3");
-  });
-
-  it("shows '… and K more' when the entry list is capped below the true total", () => {
-    const capped = makeProgress({
-      filesChanged: 5,
-      entries: [
-        makeEntry({ pathAfter: "a.ts" }),
-        makeEntry({ pathAfter: "b.ts" }),
-      ],
-    });
-    const { lastFrame } = render(
-      <FileChangeProgressBar progress={capped} expanded />,
-    );
-    expect(lastFrame() ?? "").toContain("and 3 more");
-  });
-
-  it("renders a rename entry as 'before → after'", () => {
-    const renamed = makeProgress({
-      filesChanged: 1,
-      entries: [
-        makeEntry({
-          pathBefore: "old/name.ts",
-          pathAfter: "new/name.ts",
-          kind: FileChangeKind.RENAME,
-        }),
-      ],
-    });
-    const { lastFrame } = render(
-      <FileChangeProgressBar progress={renamed} expanded />,
-    );
-    expect(lastFrame() ?? "").toContain("old/name.ts → new/name.ts");
-  });
-
-  it("hides per-file stats for a withheld entry (zero counts)", () => {
-    const secretish = makeProgress({
-      filesChanged: 1,
-      entries: [makeEntry({ pathAfter: ".env", linesAdded: 0, linesRemoved: 0 })],
-    });
-    const { lastFrame } = render(
-      <FileChangeProgressBar progress={secretish} expanded />,
-    );
-    const out = lastFrame() ?? "";
-    expect(out).toContain(".env");
-    expect(out).not.toContain("+");
-  });
-});
-
-// --- path helpers (direct unit coverage) ----------------------------------
-
-interface ChangeOpts {
-  readonly pathBefore?: string;
-  readonly pathAfter?: string;
-  readonly kind: FileChangeKind;
-}
-
-function makeChange(opts: ChangeOpts) {
-  return create(CapturedFileChangeSchema, {
-    id: "c-1",
-    pathBefore: opts.pathBefore ?? "",
-    pathAfter: opts.pathAfter ?? "",
-    kind: opts.kind,
-  });
-}
-
-describe("changeDisplayPath", () => {
-  it("shows the after-path for an add", () => {
-    expect(
-      changeDisplayPath(makeChange({ pathAfter: "new.ts", kind: FileChangeKind.ADD })),
-    ).toBe("new.ts");
-  });
-
-  it("shows the after-path for a modify", () => {
-    expect(
-      changeDisplayPath(
-        makeChange({ pathAfter: "app.ts", kind: FileChangeKind.MODIFY }),
-      ),
-    ).toBe("app.ts");
-  });
-
-  it("falls back to the before-path for a delete", () => {
-    expect(
-      changeDisplayPath(
-        makeChange({ pathBefore: "gone.ts", kind: FileChangeKind.DELETE }),
-      ),
-    ).toBe("gone.ts");
-  });
-
-  it("shows 'before → after' for a rename", () => {
-    expect(
-      changeDisplayPath(
-        makeChange({
-          pathBefore: "old.ts",
-          pathAfter: "new.ts",
-          kind: FileChangeKind.RENAME,
-        }),
-      ),
-    ).toBe("old.ts → new.ts");
-  });
-});
-
-describe("progressEntryDisplayPath", () => {
-  it("shows the after-path for an add", () => {
-    expect(
-      progressEntryDisplayPath(makeEntry({ pathAfter: "new.ts", kind: FileChangeKind.ADD })),
-    ).toBe("new.ts");
-  });
-
-  it("shows the after-path for a modify", () => {
-    expect(
-      progressEntryDisplayPath(makeEntry({ pathAfter: "app.ts" })),
-    ).toBe("app.ts");
-  });
-
-  it("falls back to the before-path for a delete", () => {
-    expect(
-      progressEntryDisplayPath(
-        makeEntry({ pathBefore: "gone.ts", kind: FileChangeKind.DELETE }),
-      ),
-    ).toBe("gone.ts");
-  });
-
-  it("shows 'before → after' for a rename — identical to changeDisplayPath", () => {
-    expect(
-      progressEntryDisplayPath(
-        makeEntry({
-          pathBefore: "old.ts",
-          pathAfter: "new.ts",
-          kind: FileChangeKind.RENAME,
-        }),
-      ),
-    ).toBe("old.ts → new.ts");
-  });
-});