@opsee/mcp-server 0.9.34 → 0.9.35

package/src/__tests__/initiatives.test.ts

@@ -1,188 +0,0 @@
-import { describe, test, expect, beforeAll, afterAll, beforeEach } from "vitest";
-import { Client } from "@modelcontextprotocol/sdk/client/index.js";
-import { InMemoryTransport } from "@modelcontextprotocol/sdk/inMemory.js";
-import { createServer } from "../server.js";
-import type { ApiClients } from "../client/api.js";
-
-const mockInitiative = {
-  id: 2,
-  uuid: "u-2",
-  title: "Async brain",
-  status: "active",
-  anchorType: "none",
-  coreIdea: "",
-  projectId: 1,
-  createdByUserId: 7,
-};
-
-// Records the last addInitiativeMemoryEntry request so checkpoint tests can assert the mapping.
-export let lastMemoryReq: any = null;
-
-// Records the last getInitiatives request so list tests can assert the mapping.
-export let lastListReq: any = null;
-
-// Records the last reconcileInitiative request so reconcile tests can assert pass-through.
-export let lastReconcileReq: any = null;
-
-const mockListInitiative = {
-  id: 2, uuid: "u-2", title: "Auth refactor", status: "active",
-  anchorType: "none", coreIdea: "", projectId: 1, createdByUserId: 7,
-};
-
-function createMockClients(): ApiClients {
-  lastMemoryReq = null;
-  lastListReq = null;
-  lastReconcileReq = null;
-  return {
-    initiatives: {
-      getInitiatives: async (req: any) => {
-        lastListReq = req;
-        return { initiatives: [mockListInitiative], pagination: {} };
-      },
-      getInitiativeContext: async () => ({
-        initiative: mockInitiative,
-        graph: undefined,
-        memoryEntries: [],
-        pullRequests: [],
-        comments: [],
-      }),
-      reconcileInitiative: async (req: any) => {
-        lastReconcileReq = req;
-        return {
-          changeset: {
-            added: [{ taskId: 0, identifier: "", title: "New", reason: "" }],
-            removed: [],
-            flagged: [{ taskId: 3, identifier: "T-3", title: "InFlight", reason: "has linked PR" }],
-            edgesAdded: [],
-            edgesRemoved: [],
-          },
-        };
-      },
-      addInitiativeMemoryEntry: async (req: any) => {
-        lastMemoryReq = req;
-        return {
-          memoryEntry: {
-            id: 99,
-            initiativeId: req.initiativeId,
-            kind: req.kind,
-            body: req.body,
-            contentType: req.contentType,
-            sourceType: req.sourceType,
-            sourceId: req.sourceId,
-            sourceUrl: req.sourceUrl,
-          },
-        };
-      },
-    },
-  } as unknown as ApiClients;
-}
-
-describe("Initiative write-back discipline", () => {
-  let client: Client;
-  let cleanup: () => Promise<void>;
-
-  beforeAll(async () => {
-    const [ct, st] = InMemoryTransport.createLinkedPair();
-    const server = createServer(() => createMockClients());
-    await server.connect(st);
-    client = new Client({ name: "init-test", version: "1.0.0" });
-    await client.connect(ct);
-    cleanup = async () => { await client.close(); await server.close(); };
-  });
-
-  afterAll(async () => { await cleanup(); });
-
-  beforeEach(() => { lastMemoryReq = null; lastListReq = null; lastReconcileReq = null; });
-
-  test("get_initiative_context appends the working agreement", async () => {
-    const result = await client.callTool({
-      name: "opsee_get_initiative_context",
-      arguments: { initiativeId: 2 },
-    });
-    const text = (result.content as Array<{ type: string; text: string }>)[0].text;
-    expect(text).toContain("Working agreement");
-    expect(text).toContain("opsee_checkpoint");
-    expect(text).toContain("opsee_add_initiative_memory");
-  });
-
-  test("checkpoint writes an outcome entry by default with no source", async () => {
-    const result = await client.callTool({
-      name: "opsee_checkpoint",
-      arguments: { initiativeId: 2, summary: "Wired the parser" },
-    });
-    const text = (result.content as Array<{ type: string; text: string }>)[0].text;
-    expect(text).toContain("Checkpoint recorded");
-    expect(lastMemoryReq.kind).toBe("outcome");
-    expect(lastMemoryReq.body).toBe("Wired the parser");
-    expect(lastMemoryReq.sourceType).toBe("none");
-    // next-obligations nudge
-    expect(text).toContain("Link the PR");
-  });
-
-  test("checkpoint maps taskId to a task source", async () => {
-    await client.callTool({
-      name: "opsee_checkpoint",
-      arguments: { initiativeId: 2, summary: "done", taskId: 918 },
-    });
-    expect(lastMemoryReq.sourceType).toBe("task");
-    expect(lastMemoryReq.sourceId).toBe(918);
-    expect(lastMemoryReq.sourceUrl).toBeUndefined();
-  });
-
-  test("checkpoint maps prUrl to a pull_request source", async () => {
-    await client.callTool({
-      name: "opsee_checkpoint",
-      arguments: { initiativeId: 2, summary: "shipped", prUrl: "https://example.com/mr/94" },
-    });
-    expect(lastMemoryReq.sourceType).toBe("pull_request");
-    expect(lastMemoryReq.sourceUrl).toBe("https://example.com/mr/94");
-    expect(lastMemoryReq.sourceId).toBeUndefined();
-  });
-
-  test("checkpoint with both taskId and prUrl prefers task source and keeps the url", async () => {
-    await client.callTool({
-      name: "opsee_checkpoint",
-      arguments: { initiativeId: 2, summary: "both", taskId: 918, prUrl: "https://example.com/mr/94" },
-    });
-    expect(lastMemoryReq.sourceType).toBe("task");
-    expect(lastMemoryReq.sourceId).toBe(918);
-    expect(lastMemoryReq.sourceUrl).toBe("https://example.com/mr/94");
-  });
-
-  test("checkpoint honors an explicit kind", async () => {
-    await client.callTool({
-      name: "opsee_checkpoint",
-      arguments: { initiativeId: 2, summary: "learned a thing", kind: "learning" },
-    });
-    expect(lastMemoryReq.kind).toBe("learning");
-  });
-
-  test("list_initiatives passes search through filterOptions", async () => {
-    await client.callTool({
-      name: "opsee_list_initiatives",
-      arguments: { projectId: 1, search: "auth" },
-    });
-    expect(lastListReq.filterOptions?.search).toBe("auth");
-  });
-
-  test("list_initiatives omits filterOptions when no search", async () => {
-    await client.callTool({
-      name: "opsee_list_initiatives",
-      arguments: { projectId: 1 },
-    });
-    expect(lastListReq.filterOptions).toBeUndefined();
-  });
-
-  test("reconcile passes dryRun + existingTaskId and renders flagged drops", async () => {
-    const result = await client.callTool({
-      name: "opsee_reconcile_initiative",
-      arguments: { initiativeId: 2, dryRun: true, tasks: [{ ref: "a", title: "Keep", existingTaskId: 1 }], edges: [] },
-    });
-    expect(lastReconcileReq.dryRun).toBe(true);
-    expect(lastReconcileReq.tasks[0].existingTaskId).toBe(1);
-    const text = (result.content as Array<{ type: string; text: string }>)[0].text;
-    expect(text).toContain("[DRY RUN");
-    expect(text).toContain("flagged");
-    expect(text).toContain("has linked PR");
-  });
-});

package/src/__tests__/kv-store.test.ts

@@ -1,53 +0,0 @@
-import { describe, test, expect } from "vitest";
-import { InMemoryKVStore } from "../auth/kv-store.js";
-
-describe("InMemoryKVStore", () => {
-  test("stores and retrieves values", async () => {
-    const kv = new InMemoryKVStore();
-    await kv.set("a", "1");
-    expect(await kv.get("a")).toBe("1");
-    expect(await kv.get("missing")).toBeNull();
-  });
-
-  test("expires entries after their TTL", async () => {
-    const kv = new InMemoryKVStore();
-    await kv.set("a", "1", -1); // already expired
-    expect(await kv.get("a")).toBeNull();
-  });
-
-  test("bounds the store to maxEntries, evicting oldest first", async () => {
-    const kv = new InMemoryKVStore(3);
-    await kv.set("a", "1");
-    await kv.set("b", "2");
-    await kv.set("c", "3");
-    await kv.set("d", "4"); // exceeds cap -> "a" (oldest) evicted
-
-    expect(await kv.get("a")).toBeNull();
-    expect(await kv.get("b")).toBe("2");
-    expect(await kv.get("c")).toBe("3");
-    expect(await kv.get("d")).toBe("4");
-  });
-
-  test("re-setting a key refreshes its position so it is not evicted first", async () => {
-    const kv = new InMemoryKVStore(3);
-    await kv.set("a", "1");
-    await kv.set("b", "2");
-    await kv.set("a", "1-updated"); // refresh a -> b is now oldest
-    await kv.set("c", "3");
-    await kv.set("d", "4"); // evicts oldest live entry (b)
-
-    expect(await kv.get("b")).toBeNull();
-    expect(await kv.get("a")).toBe("1-updated");
-  });
-
-  test("evicts expired entries before live ones when over cap", async () => {
-    const kv = new InMemoryKVStore(2);
-    await kv.set("stale", "x", -1); // expired
-    await kv.set("a", "1");
-    await kv.set("b", "2"); // over cap -> expired "stale" dropped, a/b kept
-
-    expect(await kv.get("stale")).toBeNull();
-    expect(await kv.get("a")).toBe("1");
-    expect(await kv.get("b")).toBe("2");
-  });
-});

package/src/__tests__/task-dependencies.test.ts

@@ -1,173 +0,0 @@
-import { describe, test, expect, beforeAll, afterAll, beforeEach } from "vitest";
-import { Client } from "@modelcontextprotocol/sdk/client/index.js";
-import { InMemoryTransport } from "@modelcontextprotocol/sdk/inMemory.js";
-import { createServer } from "../server.js";
-import type { ApiClients } from "../client/api.js";
-import { TaskDependencyType } from "../../gen/api/v1/models_pb.js";
-
-const mockDependency = {
-  id: 7,
-  fromTaskId: 10,
-  toTaskId: 11,
-  type: TaskDependencyType.BLOCKS,
-  fromTask: { id: 10, identifier: "TP-1" },
-  toTask: { id: 11, identifier: "TP-2" },
-};
-
-// Records the last request each mock method received so tests can assert pass-through.
-let lastAddReq: any = null;
-let lastListReq: any = null;
-let lastChainReq: any = null;
-
-function createMockClients(): ApiClients {
-  lastAddReq = null;
-  lastListReq = null;
-  lastChainReq = null;
-  return {
-    taskDependencies: {
-      addTaskDependency: async (req: any) => {
-        lastAddReq = req;
-        return { taskDependency: mockDependency };
-      },
-      getTaskDependencies: async (req: any) => {
-        lastListReq = req;
-        return { taskDependencies: [mockDependency] };
-      },
-      deleteTaskDependency: async () => ({}),
-      getDependencyChain: async (req: any) => {
-        lastChainReq = req;
-        return {
-          nodes: [
-            { id: 10, identifier: "TP-1", title: "Fix the bug" },
-            { id: 11, identifier: "TP-2", title: "Write the test" },
-          ],
-          edges: [{ fromTaskId: 10, toTaskId: 11, type: TaskDependencyType.BLOCKS }],
-        };
-      },
-    },
-  } as unknown as ApiClients;
-}
-
-describe("Task dependency tools", () => {
-  let client: Client;
-  let cleanup: () => Promise<void>;
-
-  beforeAll(async () => {
-    const [ct, st] = InMemoryTransport.createLinkedPair();
-    const server = createServer(() => createMockClients());
-    await server.connect(st);
-    client = new Client({ name: "task-dependencies-test", version: "1.0.0" });
-    await client.connect(ct);
-    cleanup = async () => {
-      await client.close();
-      await server.close();
-    };
-  });
-
-  afterAll(async () => {
-    await cleanup();
-  });
-
-  beforeEach(() => {
-    lastAddReq = null;
-    lastListReq = null;
-    lastChainReq = null;
-  });
-
-  test("opsee_create_task_dependency maps type and returns the formatted dependency", async () => {
-    const result = await client.callTool({
-      name: "opsee_create_task_dependency",
-      arguments: { fromTaskId: 10, toTaskId: 11, type: "BLOCKS" },
-    });
-    const text = (result.content as Array<{ type: string; text: string }>)[0].text;
-    expect(text).toContain("Task dependency created");
-    expect(text).toContain("TP-1 BLOCKS TP-2");
-    expect(lastAddReq.type).toBe(TaskDependencyType.BLOCKS);
-  });
-
-  test("opsee_create_task_dependency reports failure when backend returns no dependency", async () => {
-    const emptyClients = createMockClients();
-    (emptyClients.taskDependencies as any).addTaskDependency = async () => ({});
-
-    const [ct, st] = InMemoryTransport.createLinkedPair();
-    const server = createServer(() => emptyClients);
-    await server.connect(st);
-    const c = new Client({ name: "empty-add-test", version: "1.0.0" });
-    await c.connect(ct);
-
-    const result = await c.callTool({
-      name: "opsee_create_task_dependency",
-      arguments: { fromTaskId: 10, toTaskId: 11, type: "RELATES_TO" },
-    });
-    expect(result.isError).toBe(true);
-    const text = (result.content as Array<{ type: string; text: string }>)[0].text;
-    expect(text).toContain("Failed to create task dependency");
-
-    await c.close();
-    await server.close();
-  });
-
-  test("opsee_list_task_dependencies returns the formatted list and default pagination", async () => {
-    const result = await client.callTool({
-      name: "opsee_list_task_dependencies",
-      arguments: { taskId: 10 },
-    });
-    const text = (result.content as Array<{ type: string; text: string }>)[0].text;
-    expect(text).toContain("Found 1 dependency(ies)");
-    expect(text).toContain("TP-1 BLOCKS TP-2");
-    expect(lastListReq.taskId).toBe(10);
-  });
-
-  test("opsee_list_task_dependencies reports none for an empty result", async () => {
-    const emptyClients = createMockClients();
-    (emptyClients.taskDependencies as any).getTaskDependencies = async () => ({ taskDependencies: [] });
-
-    const [ct, st] = InMemoryTransport.createLinkedPair();
-    const server = createServer(() => emptyClients);
-    await server.connect(st);
-    const c = new Client({ name: "empty-list-test", version: "1.0.0" });
-    await c.connect(ct);
-
-    const result = await c.callTool({
-      name: "opsee_list_task_dependencies",
-      arguments: { taskId: 10 },
-    });
-    const text = (result.content as Array<{ type: string; text: string }>)[0].text;
-    expect(text).toContain("No task dependencies found.");
-
-    await c.close();
-    await server.close();
-  });
-
-  test("opsee_delete_task_dependency succeeds", async () => {
-    const result = await client.callTool({
-      name: "opsee_delete_task_dependency",
-      arguments: { dependencyId: 7 },
-    });
-    const text = (result.content as Array<{ type: string; text: string }>)[0].text;
-    expect(text).toContain("Task dependency deleted.");
-  });
-
-  test("opsee_get_dependency_chain renders nodes and edges with defaults", async () => {
-    const result = await client.callTool({
-      name: "opsee_get_dependency_chain",
-      arguments: { taskId: 10 },
-    });
-    const text = (result.content as Array<{ type: string; text: string }>)[0].text;
-    expect(text).toContain("direction: both, depth: 5");
-    expect(text).toContain("2 task(s), 1 edge(s)");
-    expect(text).toContain("[TP-1] Fix the bug");
-    expect(text).toContain("task 10 → task 11 (type");
-    expect(lastChainReq.direction).toBe("both");
-    expect(lastChainReq.depth).toBe(5);
-  });
-
-  test("opsee_get_dependency_chain passes through an explicit direction and depth", async () => {
-    await client.callTool({
-      name: "opsee_get_dependency_chain",
-      arguments: { taskId: 10, direction: "blocked_by", depth: 2 },
-    });
-    expect(lastChainReq.direction).toBe("blocked_by");
-    expect(lastChainReq.depth).toBe(2);
-  });
-});