@opsee/mcp-server 0.9.34 → 0.9.36

package/src/__tests__/tools.test.ts

@@ -1,913 +0,0 @@
-import { describe, test, expect, beforeAll, afterAll } 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 { Code, ConnectError } from "@connectrpc/connect";
-
-// --- Mock data ---
-
-const mockProject = {
-  id: 1,
-  name: "Test Project",
-  shortName: "TP",
-  slug: "test-project",
-  description: "A test project",
-  isActive: true,
-};
-
-const mockTask = {
-  id: 10,
-  identifier: "TP-1",
-  title: "Fix the bug",
-  description: "Something is broken",
-  displayOrder: 0,
-  storyPoints: 3,
-  aiModeEnabled: false,
-  boardColumn: { id: 1, name: "To Do" },
-  taskPriority: { id: 2, name: "High" },
-  taskType: { id: 1, name: "Bug" },
-  assignedUser: { id: 5, fullName: "Alice" },
-  cycle: { id: 1, name: "Sprint 1" },
-  reporterUser: { id: 5 },
-  board: { id: 1 },
-  project: { id: 1 },
-  taskRepositories: [],
-};
-
-const mockUser = {
-  id: 5,
-  fullName: "Alice Test",
-  email: "alice@test.com",
-  companyId: 1,
-  company: { name: "TestCo" },
-  flattenedRoles: ["User"],
-};
-
-const mockBoard = { id: 1, name: "Main Board", isActive: true };
-const mockBoardColumn = {
-  id: 1,
-  name: "To Do",
-  displayOrder: 0,
-  lifecycleState: "active",
-};
-
-const mockCycle = {
-  id: 1,
-  name: "Sprint 1",
-  description: "First sprint",
-  goal: "Ship v1",
-  startDate: "2026-03-01",
-  endDate: "2026-03-15",
-  isActive: true,
-};
-
-const mockDocSpace = {
-  id: 1,
-  title: "Engineering Docs",
-  description: "Technical documentation",
-  slug: "engineering-docs",
-  isDefault: true,
-};
-
-const mockDocPage = {
-  id: 1,
-  title: "Getting Started",
-  slug: "getting-started",
-  currentContent: "Welcome to the docs",
-  currentVersionNumber: 1,
-  createdByUser: { fullName: "Alice" },
-};
-
-const mockTaskType = { id: 1, name: "Bug" };
-const mockTaskPriority = { id: 2, name: "High", level: 2 };
-const mockVcsIntegration = {
-  id: 1,
-  provider: 2,
-  repoOwner: "acme",
-  repoName: "backend",
-  defaultBranch: "main",
-  isActive: true,
-};
-
-// --- Mock client factory ---
-
-function createMockClients(): ApiClients {
-  return {
-    projects: {
-      getProjects: async () => ({ projects: [mockProject] }),
-      getProject: async () => ({ project: mockProject }),
-    },
-    tasks: {
-      getTasks: async () => ({ tasks: [mockTask] }),
-      getTask: async () => ({ task: mockTask }),
-      addTask: async () => ({ task: mockTask }),
-      editTask: async () => ({ task: { ...mockTask, title: "Updated bug" } }),
-      bulkEditTasks: async (req: { taskIds: number[] }) => ({
-        updatedCount: req.taskIds.length,
-      }),
-      deleteTask: async () => ({}),
-      bulkDeleteTasks: async () => ({ deletedCount: 2 }),
-      addWorkLog: async (req: { taskId: number; hours: number }) => ({
-        workLog: { id: 42, taskId: req.taskId, userId: 5, hours: req.hours },
-      }),
-      deleteWorkLog: async () => ({}),
-    },
-    comments: {
-      getComment: async () => ({ comment: { id: 11, content: "old text", isInternal: false, taskId: 10, userId: 5 } }),
-      getComments: async () => ({ comments: [] }),
-      editComment: async () => ({ comment: { id: 11, content: "edited text", isInternal: true, taskId: 10, userId: 5 } }),
-    },
-    taskLabels: {
-      getTaskLabels: async () => ({ taskLabels: [] }),
-    },
-    taskDependencies: {
-      getTaskDependencies: async () => ({ taskDependencies: [] }),
-      getDependencyChain: async () => ({
-        nodes: [{ id: 10, identifier: "TP-1", title: "Fix the bug" }],
-        edges: [],
-      }),
-    },
-    acceptanceCriteria: {
-      getAcceptanceCriteriaByTask: async () => ({
-        criteria: [{ id: 1, taskId: 10, text: "endpoint returns 200", done: false, displayOrder: 0 }],
-      }),
-      addAcceptanceCriterion: async (req: { taskId: number; text: string; displayOrder?: number }) => ({
-        criterion: { id: 2, taskId: req.taskId, text: req.text, done: false, displayOrder: req.displayOrder ?? 0 },
-      }),
-      editAcceptanceCriterion: async (req: { id: number; text?: string; done?: boolean }) => ({
-        criterion: { id: req.id, taskId: 10, text: req.text ?? "old", done: req.done ?? true, displayOrder: 0 },
-      }),
-      deleteAcceptanceCriterion: async () => ({}),
-    },
-    notifications: {
-      getNotifications: async () => ({
-        notifications: [
-          {
-            id: 1,
-            title: "You were assigned to TP-1",
-            message: "Alice assigned you",
-            notificationType: "task_assigned",
-            entityType: "task",
-            entityId: 10,
-            isRead: false,
-            sentViaEmail: false,
-            userId: 5,
-          },
-        ],
-      }),
-      getNotification: async () => ({
-        notification: {
-          id: 1,
-          title: "You were assigned to TP-1",
-          message: "Alice assigned you",
-          notificationType: "task_assigned",
-          isRead: false,
-          sentViaEmail: false,
-          userId: 5,
-        },
-      }),
-      editNotification: async () => ({
-        notification: { id: 1, title: "x", message: "y", notificationType: "task_assigned", isRead: true, sentViaEmail: false, userId: 5 },
-      }),
-    },
-    boards: {
-      getBoards: async () => ({ boards: [mockBoard] }),
-    },
-    boardColumns: {
-      getBoardColumns: async () => ({ boardColumns: [mockBoardColumn] }),
-    },
-    cycles: {
-      getCycles: async () => ({ cycles: [mockCycle] }),
-      getCycle: async () => ({ cycle: { ...mockCycle, projectId: 1 } }),
-      addCycle: async () => ({ cycle: { ...mockCycle, id: 2, name: "Sprint 2" } }),
-      editCycle: async () => ({ cycle: { ...mockCycle, name: "Sprint 1 renamed" } }),
-      deleteCycle: async () => ({}),
-    },
-    docSpaces: {
-      getDocSpaces: async () => ({ docSpaces: [mockDocSpace] }),
-    },
-    docPages: {
-      getDocPages: async () => ({ docPages: [mockDocPage] }),
-      getDocPage: async () => ({ docPage: mockDocPage }),
-      addDocPage: async () => ({ docPage: mockDocPage }),
-    },
-    taskTypes: {
-      getTaskTypes: async () => ({ taskTypes: [mockTaskType] }),
-    },
-    taskPriorities: {
-      getTaskPriorities: async () => ({
-        taskPriorities: [mockTaskPriority],
-      }),
-    },
-    users: {
-      getMe: async () => ({ user: mockUser }),
-    },
-    vcsIntegrations: {
-      getVCSIntegrations: async () => ({
-        vcsIntegrations: [mockVcsIntegration],
-      }),
-    },
-  } as unknown as ApiClients;
-}
-
-// --- Test suite ---
-
-describe("MCP Tools", () => {
-  let client: Client;
-  let cleanup: () => Promise<void>;
-
-  beforeAll(async () => {
-    const [clientTransport, serverTransport] =
-      InMemoryTransport.createLinkedPair();
-    const server = createServer(() => createMockClients());
-    await server.connect(serverTransport);
-
-    client = new Client({ name: "test-client", version: "1.0.0" });
-    await client.connect(clientTransport);
-
-    cleanup = async () => {
-      await client.close();
-      await server.close();
-    };
-  });
-
-  afterAll(async () => {
-    await cleanup();
-  });
-
-  // --- Project tools ---
-
-  test("opsee_list_projects returns projects", async () => {
-    const result = await client.callTool({
-      name: "opsee_list_projects",
-      arguments: {},
-    });
-    const text = (result.content as Array<{ type: string; text: string }>)[0]
-      .text;
-    expect(text).toContain("Test Project");
-    expect(text).toContain("TP");
-    expect(text).toContain("1 project(s)");
-  });
-
-  test("opsee_get_project returns project details", async () => {
-    const result = await client.callTool({
-      name: "opsee_get_project",
-      arguments: { projectId: 1 },
-    });
-    const text = (result.content as Array<{ type: string; text: string }>)[0]
-      .text;
-    expect(text).toContain("Test Project");
-    expect(text).toContain("test-project");
-    expect(text).toContain("ID: 1");
-  });
-
-  // --- Task tools ---
-
-  test("opsee_list_tasks returns tasks for a project", async () => {
-    const result = await client.callTool({
-      name: "opsee_list_tasks",
-      arguments: { projectId: 1 },
-    });
-    const text = (result.content as Array<{ type: string; text: string }>)[0]
-      .text;
-    expect(text).toContain("TP-1");
-    expect(text).toContain("Fix the bug");
-    expect(text).toContain("1 task(s)");
-  });
-
-  test("opsee_get_task returns task details", async () => {
-    const result = await client.callTool({
-      name: "opsee_get_task",
-      arguments: { taskId: 10 },
-    });
-    const text = (result.content as Array<{ type: string; text: string }>)[0]
-      .text;
-    expect(text).toContain("TP-1");
-    expect(text).toContain("Fix the bug");
-    expect(text).toContain("High");
-    expect(text).toContain("Bug");
-  });
-
-  test("opsee_get_task rejects a non-positive / non-integer taskId", async () => {
-    // The ID-validation sweep: an id must be a positive integer. Zero,
-    // negatives, and fractionals fail schema validation (isError) before the
-    // handler runs. A valid positive integer (taskId: 10) succeeds above.
-    for (const taskId of [0, 1.5, -3]) {
-      const result = await client.callTool({
-        name: "opsee_get_task",
-        arguments: { taskId },
-      });
-      expect(result.isError, `taskId=${taskId} should be rejected`).toBe(true);
-    }
-  });
-
-  test("opsee_create_task creates a task with defaults", async () => {
-    const result = await client.callTool({
-      name: "opsee_create_task",
-      arguments: { projectId: 1, title: "New task" },
-    });
-    const text = (result.content as Array<{ type: string; text: string }>)[0]
-      .text;
-    expect(text).toContain("Task created");
-    expect(text).toContain("TP-1");
-  });
-
-  test("opsee_update_task does read-modify-write", async () => {
-    const result = await client.callTool({
-      name: "opsee_update_task",
-      arguments: { taskId: 10, title: "Updated title" },
-    });
-    const text = (result.content as Array<{ type: string; text: string }>)[0]
-      .text;
-    expect(text).toContain("Task updated");
-  });
-
-  // --- User tools ---
-
-  test("opsee_get_me returns current user", async () => {
-    const result = await client.callTool({
-      name: "opsee_get_me",
-      arguments: {},
-    });
-    const text = (result.content as Array<{ type: string; text: string }>)[0]
-      .text;
-    expect(text).toContain("Alice Test");
-    expect(text).toContain("alice@test.com");
-    expect(text).toContain("TestCo");
-  });
-
-  // --- Board/metadata tools ---
-
-  test("opsee_list_boards returns boards", async () => {
-    const result = await client.callTool({
-      name: "opsee_list_boards",
-      arguments: { projectId: 1 },
-    });
-    const text = (result.content as Array<{ type: string; text: string }>)[0]
-      .text;
-    expect(text).toContain("Main Board");
-    expect(text).toContain("ID: 1");
-  });
-
-  test("opsee_list_board_columns returns columns", async () => {
-    const result = await client.callTool({
-      name: "opsee_list_board_columns",
-      arguments: { boardId: 1 },
-    });
-    const text = (result.content as Array<{ type: string; text: string }>)[0]
-      .text;
-    expect(text).toContain("To Do");
-    expect(text).toContain("ID: 1");
-  });
-
-  test("opsee_list_task_types returns types", async () => {
-    const result = await client.callTool({
-      name: "opsee_list_task_types",
-      arguments: { projectId: 1 },
-    });
-    const text = (result.content as Array<{ type: string; text: string }>)[0]
-      .text;
-    expect(text).toContain("Bug");
-    expect(text).toContain("ID: 1");
-  });
-
-  test("opsee_list_task_priorities returns priorities", async () => {
-    const result = await client.callTool({
-      name: "opsee_list_task_priorities",
-      arguments: { projectId: 1 },
-    });
-    const text = (result.content as Array<{ type: string; text: string }>)[0]
-      .text;
-    expect(text).toContain("High");
-    expect(text).toContain("level: 2");
-  });
-
-  // --- Cycle tools ---
-
-  test("opsee_list_cycles returns cycles", async () => {
-    const result = await client.callTool({
-      name: "opsee_list_cycles",
-      arguments: { projectId: 1 },
-    });
-    const text = (result.content as Array<{ type: string; text: string }>)[0]
-      .text;
-    expect(text).toContain("Sprint 1");
-    expect(text).toContain("1 cycle(s)");
-  });
-
-  test("opsee_get_cycle returns cycle details", async () => {
-    const result = await client.callTool({
-      name: "opsee_get_cycle",
-      arguments: { cycleId: 1 },
-    });
-    const text = (result.content as Array<{ type: string; text: string }>)[0]
-      .text;
-    expect(text).toContain("Sprint 1");
-    expect(text).toContain("Ship v1");
-    expect(text).toContain("ID: 1");
-  });
-
-  test("opsee_create_cycle creates a cycle", async () => {
-    const result = await client.callTool({
-      name: "opsee_create_cycle",
-      arguments: {
-        projectId: 1,
-        name: "Sprint 2",
-        startDate: "2026-03-25",
-        endDate: "2026-04-08",
-      },
-    });
-    const text = (result.content as Array<{ type: string; text: string }>)[0]
-      .text;
-    expect(text).toContain("Cycle created");
-    expect(text).toContain("Sprint 2");
-  });
-
-  // --- Doc tools ---
-
-  test("opsee_list_doc_spaces returns spaces", async () => {
-    const result = await client.callTool({
-      name: "opsee_list_doc_spaces",
-      arguments: { projectId: 1 },
-    });
-    const text = (result.content as Array<{ type: string; text: string }>)[0]
-      .text;
-    expect(text).toContain("Engineering Docs");
-    expect(text).toContain("1 doc space(s)");
-  });
-
-  test("opsee_list_doc_pages returns pages", async () => {
-    const result = await client.callTool({
-      name: "opsee_list_doc_pages",
-      arguments: { docSpaceId: 1 },
-    });
-    const text = (result.content as Array<{ type: string; text: string }>)[0]
-      .text;
-    expect(text).toContain("Getting Started");
-  });
-
-  test("opsee_get_doc_page returns page content", async () => {
-    const result = await client.callTool({
-      name: "opsee_get_doc_page",
-      arguments: { pageId: 1 },
-    });
-    const text = (result.content as Array<{ type: string; text: string }>)[0]
-      .text;
-    expect(text).toContain("Getting Started");
-    expect(text).toContain("Welcome to the docs");
-  });
-
-  test("opsee_create_doc_page creates a page", async () => {
-    const result = await client.callTool({
-      name: "opsee_create_doc_page",
-      arguments: {
-        projectId: 1,
-        title: "New Page",
-        content: "Hello world",
-      },
-    });
-    const text = (result.content as Array<{ type: string; text: string }>)[0]
-      .text;
-    expect(text).toContain("Doc page created");
-    expect(text).toContain("Getting Started");
-  });
-
-  // --- Repository tools ---
-
-  test("opsee_list_repositories returns connected repos", async () => {
-    const result = await client.callTool({
-      name: "opsee_list_repositories",
-      arguments: { projectId: 1 },
-    });
-    const text = (result.content as Array<{ type: string; text: string }>)[0]
-      .text;
-    expect(text).toContain("acme/backend");
-    expect(text).toContain("GitLab");
-  });
-
-  // --- Error handling ---
-
-  test("tool returns isError on API failure", async () => {
-    const errorClients = createMockClients();
-    (errorClients.projects as any).getProjects = async () => {
-      throw new Error("connection refused ECONNREFUSED");
-    };
-
-    const [ct, st] = InMemoryTransport.createLinkedPair();
-    const errorServer = createServer(() => errorClients);
-    await errorServer.connect(st);
-
-    const errorClient = new Client({
-      name: "error-test",
-      version: "1.0.0",
-    });
-    await errorClient.connect(ct);
-
-    const result = await errorClient.callTool({
-      name: "opsee_list_projects",
-      arguments: {},
-    });
-
-    expect(result.isError).toBe(true);
-    const text = (result.content as Array<{ type: string; text: string }>)[0]
-      .text;
-    expect(text).toContain("ECONNREFUSED");
-
-    await errorClient.close();
-    await errorServer.close();
-  });
-
-  // --- Track 3 bulk + cycle + label additions ---
-
-  test("opsee_create_task accepts labelIds and returns the task", async () => {
-    const result = await client.callTool({
-      name: "opsee_create_task",
-      arguments: { projectId: 1, title: "with labels", labelIds: [1, 2] },
-    });
-    const text = (result.content as Array<{ type: string; text: string }>)[0].text;
-    expect(text).toContain("Task created");
-    expect(text).toContain("TP-1");
-  });
-
-  test("opsee_update_task accepts labelIds replace-set", async () => {
-    const result = await client.callTool({
-      name: "opsee_update_task",
-      arguments: { taskId: 10, labelIds: [3] },
-    });
-    const text = (result.content as Array<{ type: string; text: string }>)[0].text;
-    expect(text).toContain("Task updated");
-  });
-
-  test("opsee_bulk_update_tasks reports per-task count", async () => {
-    const result = await client.callTool({
-      name: "opsee_bulk_update_tasks",
-      arguments: { projectId: 1, taskIds: [10, 11, 12], boardColumnId: 5 },
-    });
-    const text = (result.content as Array<{ type: string; text: string }>)[0].text;
-    expect(text).toContain("Updated 3 of 3");
-  });
-
-  test("opsee_bulk_update_tasks accepts storyPoints/estimatedHours/parentTaskId/labelIds", async () => {
-    const result = await client.callTool({
-      name: "opsee_bulk_update_tasks",
-      arguments: {
-        projectId: 1,
-        taskIds: [10, 11],
-        storyPoints: 5,
-        estimatedHours: 8,
-        parentTaskId: 3,
-        labelIds: [1, 2],
-      },
-    });
-    const text = (result.content as Array<{ type: string; text: string }>)[0].text;
-    expect(text).toContain("Updated 2 of 2");
-  });
-
-  test("opsee_bulk_move_to_cycle delegates to bulk edit", async () => {
-    const result = await client.callTool({
-      name: "opsee_bulk_move_to_cycle",
-      arguments: { projectId: 1, taskIds: [10, 11], cycleId: 7 },
-    });
-    const text = (result.content as Array<{ type: string; text: string }>)[0].text;
-    expect(text).toContain("Moved 2 of 2");
-    expect(text).toContain("cycle 7");
-  });
-
-  test("opsee_bulk_move_to_column delegates to bulk edit", async () => {
-    const result = await client.callTool({
-      name: "opsee_bulk_move_to_column",
-      arguments: { projectId: 1, taskIds: [10], boardColumnId: 5 },
-    });
-    const text = (result.content as Array<{ type: string; text: string }>)[0].text;
-    expect(text).toContain("Moved 1 of 1");
-    expect(text).toContain("column 5");
-  });
-
-  test("opsee_bulk_assign delegates to bulk edit", async () => {
-    const result = await client.callTool({
-      name: "opsee_bulk_assign",
-      arguments: { projectId: 1, taskIds: [10, 11, 12], assigneeId: 5 },
-    });
-    const text = (result.content as Array<{ type: string; text: string }>)[0].text;
-    expect(text).toContain("Assigned 3 of 3");
-    expect(text).toContain("user 5");
-  });
-
-  test("opsee_bulk_set_priority delegates to bulk edit", async () => {
-    const result = await client.callTool({
-      name: "opsee_bulk_set_priority",
-      arguments: { projectId: 1, taskIds: [10], priorityId: 2 },
-    });
-    const text = (result.content as Array<{ type: string; text: string }>)[0].text;
-    expect(text).toContain("Set priority 2");
-    expect(text).toContain("1 of 1");
-  });
-
-  test("opsee_replace_task_labels with a set returns count", async () => {
-    const result = await client.callTool({
-      name: "opsee_replace_task_labels",
-      arguments: { taskId: 10, labelIds: [1, 2, 3] },
-    });
-    const text = (result.content as Array<{ type: string; text: string }>)[0].text;
-    expect(text).toContain("Set 3 label(s)");
-  });
-
-  test("opsee_replace_task_labels with empty clears", async () => {
-    const result = await client.callTool({
-      name: "opsee_replace_task_labels",
-      arguments: { taskId: 10, labelIds: [] },
-    });
-    const text = (result.content as Array<{ type: string; text: string }>)[0].text;
-    expect(text).toContain("Cleared all labels");
-  });
-
-  test("opsee_update_cycle returns the renamed cycle", async () => {
-    const result = await client.callTool({
-      name: "opsee_update_cycle",
-      arguments: { cycleId: 1, name: "Sprint 1 renamed" },
-    });
-    const text = (result.content as Array<{ type: string; text: string }>)[0].text;
-    expect(text).toContain("Cycle updated");
-    expect(text).toContain("Sprint 1 renamed");
-  });
-
-  test("opsee_close_cycle closes and carries forward", async () => {
-    const result = await client.callTool({
-      name: "opsee_close_cycle",
-      arguments: { cycleId: 1, nextCycleId: 2 },
-    });
-    const text = (result.content as Array<{ type: string; text: string }>)[0].text;
-    expect(text).toContain("closed");
-    // mockTask is returned by getTasks, so we should see a carry-forward count.
-    expect(text).toContain("Carried 1 task(s) forward to cycle 2");
-  });
-
-  test("opsee_close_cycle rejects mutually exclusive args", async () => {
-    const result = await client.callTool({
-      name: "opsee_close_cycle",
-      arguments: { cycleId: 1, nextCycleId: 2, createNextCycle: true },
-    });
-    expect(result.isError).toBe(true);
-    const text = (result.content as Array<{ type: string; text: string }>)[0].text;
-    expect(text).toContain("mutually exclusive");
-  });
-
-  test("opsee_get_active_cycle returns the active one", async () => {
-    const result = await client.callTool({
-      name: "opsee_get_active_cycle",
-      arguments: { projectId: 1 },
-    });
-    const text = (result.content as Array<{ type: string; text: string }>)[0].text;
-    expect(text).toContain("Sprint 1");
-  });
-
-  test("opsee_delete_cycle succeeds", async () => {
-    const result = await client.callTool({
-      name: "opsee_delete_cycle",
-      arguments: { cycleId: 1 },
-    });
-    const text = (result.content as Array<{ type: string; text: string }>)[0].text;
-    expect(text).toContain("Cycle deleted");
-  });
-
-  // --- Track 1 defect fixes ---
-
-  test("opsee_delete_task succeeds", async () => {
-    const result = await client.callTool({
-      name: "opsee_delete_task",
-      arguments: { taskId: 10 },
-    });
-    const text = (result.content as Array<{ type: string; text: string }>)[0].text;
-    expect(text).toContain("Task deleted");
-  });
-
-  test("opsee_bulk_delete_tasks reports counts", async () => {
-    const result = await client.callTool({
-      name: "opsee_bulk_delete_tasks",
-      arguments: { projectId: 1, taskIds: [10, 11] },
-    });
-    const text = (result.content as Array<{ type: string; text: string }>)[0].text;
-    expect(text).toContain("Deleted 2 of 2");
-  });
-
-  test("opsee_edit_comment updates content", async () => {
-    const result = await client.callTool({
-      name: "opsee_edit_comment",
-      arguments: { commentId: 11, content: "edited text", isInternal: true },
-    });
-    const text = (result.content as Array<{ type: string; text: string }>)[0].text;
-    expect(text).toContain("Comment updated");
-    expect(text).toContain("edited text");
-  });
-
-  test("opsee_get_task_with_context composes related data", async () => {
-    const result = await client.callTool({
-      name: "opsee_get_task_with_context",
-      arguments: { taskId: 10 },
-    });
-    const text = (result.content as Array<{ type: string; text: string }>)[0].text;
-    expect(text).toContain("TP-1");
-    expect(text).toContain("--- Labels ---");
-    expect(text).toContain("--- Dependencies ---");
-    expect(text).toContain("--- Subtasks ---");
-    expect(text).toContain("--- Comments ---");
-  });
-
-  // --- Acceptance Criteria (§3.7) ---
-
-  test("opsee_list_task_acceptance_criteria lists items", async () => {
-    const result = await client.callTool({
-      name: "opsee_list_task_acceptance_criteria",
-      arguments: { taskId: 10 },
-    });
-    const text = (result.content as Array<{ type: string; text: string }>)[0].text;
-    expect(text).toContain("endpoint returns 200");
-    expect(text).toContain("[ ]");
-  });
-
-  test("opsee_add_acceptance_criterion returns the new criterion", async () => {
-    const result = await client.callTool({
-      name: "opsee_add_acceptance_criterion",
-      arguments: { taskId: 10, text: "deploy to staging" },
-    });
-    const text = (result.content as Array<{ type: string; text: string }>)[0].text;
-    expect(text).toContain("Criterion added");
-    expect(text).toContain("deploy to staging");
-  });
-
-  test("opsee_update_acceptance_criterion toggles done", async () => {
-    const result = await client.callTool({
-      name: "opsee_update_acceptance_criterion",
-      arguments: { criterionId: 1, done: true },
-    });
-    const text = (result.content as Array<{ type: string; text: string }>)[0].text;
-    expect(text).toContain("Criterion updated");
-    expect(text).toContain("[x]");
-  });
-
-  test("opsee_delete_acceptance_criterion succeeds", async () => {
-    const result = await client.callTool({
-      name: "opsee_delete_acceptance_criterion",
-      arguments: { criterionId: 1 },
-    });
-    const text = (result.content as Array<{ type: string; text: string }>)[0].text;
-    expect(text).toContain("Acceptance criterion deleted");
-  });
-
-  // --- Notifications (§15) ---
-
-  test("opsee_list_my_notifications returns notifications", async () => {
-    const result = await client.callTool({
-      name: "opsee_list_my_notifications",
-      arguments: {},
-    });
-    const text = (result.content as Array<{ type: string; text: string }>)[0].text;
-    expect(text).toContain("notification(s)");
-    expect(text).toContain("task_assigned");
-    expect(text).toContain("unread");
-  });
-
-  test("opsee_list_my_notifications unreadOnly returns unread header", async () => {
-    const result = await client.callTool({
-      name: "opsee_list_my_notifications",
-      arguments: { unreadOnly: true },
-    });
-    const text = (result.content as Array<{ type: string; text: string }>)[0].text;
-    expect(text).toContain("unread notification");
-  });
-
-  test("opsee_mark_notification_read flips is_read", async () => {
-    const result = await client.callTool({
-      name: "opsee_mark_notification_read",
-      arguments: { notificationId: 1 },
-    });
-    const text = (result.content as Array<{ type: string; text: string }>)[0].text;
-    expect(text).toContain("marked read");
-  });
-
-  test("opsee_mark_all_notifications_read summarizes", async () => {
-    const result = await client.callTool({
-      name: "opsee_mark_all_notifications_read",
-      arguments: {},
-    });
-    const text = (result.content as Array<{ type: string; text: string }>)[0].text;
-    expect(text).toContain("Marked 1 of 1");
-  });
-
-  test("opsee_get_dependency_chain returns nodes and edges summary", async () => {
-    const result = await client.callTool({
-      name: "opsee_get_dependency_chain",
-      arguments: { taskId: 10, direction: "both", depth: 3 },
-    });
-    const text = (result.content as Array<{ type: string; text: string }>)[0].text;
-    expect(text).toContain("Dependency chain from task 10");
-    expect(text).toContain("1 task(s), 0 edge(s)");
-    expect(text).toContain("[TP-1]");
-  });
-
-  // --- WorkLog tools (§12) ---
-
-  test("opsee_log_work logs hours", async () => {
-    const result = await client.callTool({
-      name: "opsee_log_work",
-      arguments: { taskId: 10, hours: 2.5, description: "implemented foo" },
-    });
-    const text = (result.content as Array<{ type: string; text: string }>)[0].text;
-    expect(text).toContain("Logged 2.5h on task 10");
-  });
-
-  test("opsee_delete_work_log succeeds", async () => {
-    const result = await client.callTool({
-      name: "opsee_delete_work_log",
-      arguments: { taskId: 10, workLogId: 42 },
-    });
-    const text = (result.content as Array<{ type: string; text: string }>)[0].text;
-    expect(text).toContain("Work log deleted");
-  });
-
-  test("opsee_list_work_logs reports empty for tasks without logs", async () => {
-    const result = await client.callTool({
-      name: "opsee_list_work_logs",
-      arguments: { taskId: 10 },
-    });
-    const text = (result.content as Array<{ type: string; text: string }>)[0].text;
-    expect(text).toContain("No work logs");
-  });
-
-  test("opsee_get_task_actual_hours sums to 0 with no logs", async () => {
-    const result = await client.callTool({
-      name: "opsee_get_task_actual_hours",
-      arguments: { taskId: 10 },
-    });
-    const text = (result.content as Array<{ type: string; text: string }>)[0].text;
-    expect(text).toContain("TP-1");
-    expect(text).toContain("0h logged");
-  });
-
-  test("formatTask surfaces storyPoints and estimatedHours", async () => {
-    // mockTask has storyPoints: 3 — confirms D4 fix.
-    const result = await client.callTool({
-      name: "opsee_get_task",
-      arguments: { taskId: 10 },
-    });
-    const text = (result.content as Array<{ type: string; text: string }>)[0].text;
-    expect(text).toContain("Story Points: 3");
-  });
-
-  test("opsee_get_task returns full description while lists truncate", async () => {
-    const longDesc = "x".repeat(250);
-    const preview = "x".repeat(200) + "...";
-    const clients = createMockClients();
-    (clients.tasks as any).getTask = async () => ({ task: { ...mockTask, description: longDesc } });
-    (clients.tasks as any).getTasks = async () => ({ tasks: [{ ...mockTask, description: longDesc }] });
-
-    const [ct, st] = InMemoryTransport.createLinkedPair();
-    const srv = createServer(() => clients);
-    await srv.connect(st);
-    const c = new Client({ name: "full-desc-test", version: "1.0.0" });
-    await c.connect(ct);
-
-    const full = await c.callTool({ name: "opsee_get_task", arguments: { taskId: 10 } });
-    const fullText = (full.content as Array<{ type: string; text: string }>)[0].text;
-    expect(fullText).toContain(longDesc);
-    expect(fullText).not.toContain(preview);
-
-    const list = await c.callTool({ name: "opsee_list_tasks", arguments: { projectId: 1 } });
-    const listText = (list.content as Array<{ type: string; text: string }>)[0].text;
-    expect(listText).toContain(preview);
-
-    await c.close();
-    await srv.close();
-  });
-
-  test("auth error returns login instruction", async () => {
-    const authClients = createMockClients();
-    // The connect client surfaces backend auth failures as a ConnectError with
-    // Code.Unauthenticated (not a plain Error), which is what formatError keys on.
-    (authClients.users as any).getMe = async () => {
-      throw new ConnectError("unauthenticated", Code.Unauthenticated);
-    };
-
-    const [ct, st] = InMemoryTransport.createLinkedPair();
-    const authServer = createServer(() => authClients);
-    await authServer.connect(st);
-
-    const authClient = new Client({ name: "auth-test", version: "1.0.0" });
-    await authClient.connect(ct);
-
-    const result = await authClient.callTool({
-      name: "opsee_get_me",
-      arguments: {},
-    });
-
-    expect(result.isError).toBe(true);
-    const text = (result.content as Array<{ type: string; text: string }>)[0]
-      .text;
-    expect(text).toContain("Not authenticated");
-    expect(text).toContain("login");
-
-    await authClient.close();
-    await authServer.close();
-  });
-});