@mpowr/nexus-mcp 0.5.0 → 0.6.0

package/src/__tests__/task-list-and-doc-list.test.ts

@@ -0,0 +1,226 @@
+/**
+ * Tests for task_list and doc_list tools.
+ */
+
+import { beforeEach, describe, expect, it, vi } from 'vitest'
+import { mockApiError, mockApiSuccess, parseToolResponse, TEST_IDS } from './helpers'
+
+vi.mock('../nexus-api.js', () => ({
+  nexusPost: vi.fn(),
+}))
+
+import { nexusPost } from '../nexus-api.js'
+
+// ---------------------------------------------------------------------------
+// task_list
+// ---------------------------------------------------------------------------
+
+describe('task_list', () => {
+  beforeEach(() => {
+    vi.restoreAllMocks()
+  })
+
+  it('should list tasks for a project', async () => {
+    vi.mocked(nexusPost).mockResolvedValue(
+      mockApiSuccess({
+        action: 'task_list',
+        project_id: TEST_IDS.projectId,
+        count: 2,
+        tasks: [
+          {
+            id: TEST_IDS.taskId,
+            title: 'Implement feature',
+            status: 'open',
+            priority: 'high',
+          },
+          {
+            id: TEST_IDS.noteId,
+            title: 'Fix bug',
+            status: 'in_progress',
+            priority: 'normal',
+          },
+        ],
+      }),
+    )
+
+    const { listTasks } = await import('../tools/list-tasks.js')
+    const result = await listTasks({
+      project_id: TEST_IDS.projectId,
+      user_id: TEST_IDS.userId,
+    })
+
+    expect(result.isError).toBeUndefined()
+    const parsed = parseToolResponse(result)
+    expect(parsed.action).toBe('task_list')
+    expect(parsed.count).toBe(2)
+    expect(parsed.tasks).toHaveLength(2)
+  })
+
+  it('should pass status_filter to API', async () => {
+    vi.mocked(nexusPost).mockResolvedValue(
+      mockApiSuccess({
+        action: 'task_list',
+        project_id: TEST_IDS.projectId,
+        count: 1,
+        tasks: [
+          { id: TEST_IDS.taskId, title: 'Open task', status: 'open' },
+        ],
+      }),
+    )
+
+    const { listTasks } = await import('../tools/list-tasks.js')
+    await listTasks({
+      project_id: TEST_IDS.projectId,
+      status_filter: ['open', 'in_progress'],
+      user_id: TEST_IDS.userId,
+    })
+
+    expect(vi.mocked(nexusPost)).toHaveBeenCalledWith('/api/mcp/tasks', {
+      action: 'task_list',
+      project_id: TEST_IDS.projectId,
+      status_filter: ['open', 'in_progress'],
+      limit: 50,
+    })
+  })
+
+  it('should return error on API failure', async () => {
+    vi.mocked(nexusPost).mockResolvedValue(mockApiError('Project not found', 404))
+
+    const { listTasks } = await import('../tools/list-tasks.js')
+    const result = await listTasks({
+      project_id: TEST_IDS.projectId,
+      user_id: TEST_IDS.userId,
+    })
+
+    expect(result.isError).toBe(true)
+  })
+
+  it('should handle empty task list', async () => {
+    vi.mocked(nexusPost).mockResolvedValue(
+      mockApiSuccess({
+        action: 'task_list',
+        project_id: TEST_IDS.projectId,
+        count: 0,
+        tasks: [],
+      }),
+    )
+
+    const { listTasks } = await import('../tools/list-tasks.js')
+    const result = await listTasks({
+      project_id: TEST_IDS.projectId,
+      user_id: TEST_IDS.userId,
+    })
+
+    expect(result.isError).toBeUndefined()
+    const parsed = parseToolResponse(result)
+    expect(parsed.count).toBe(0)
+    expect(parsed.tasks).toHaveLength(0)
+  })
+})
+
+// ---------------------------------------------------------------------------
+// doc_list
+// ---------------------------------------------------------------------------
+
+describe('doc_list', () => {
+  beforeEach(() => {
+    vi.restoreAllMocks()
+  })
+
+  it('should list documents for a project', async () => {
+    vi.mocked(nexusPost).mockResolvedValue(
+      mockApiSuccess({
+        action: 'doc_list',
+        project_id: TEST_IDS.projectId,
+        count: 2,
+        documents: [
+          {
+            id: TEST_IDS.documentId,
+            title: 'Research Findings',
+            source: 'agent:app-agent',
+            classification: 'unclassified',
+          },
+          {
+            id: TEST_IDS.noteId,
+            title: 'E2E Test Plan',
+            source: 'mcp',
+            classification: 'unclassified',
+          },
+        ],
+      }),
+    )
+
+    const { listDocuments } = await import('../tools/list-documents.js')
+    const result = await listDocuments({
+      project_id: TEST_IDS.projectId,
+      user_id: TEST_IDS.userId,
+    })
+
+    expect(result.isError).toBeUndefined()
+    const parsed = parseToolResponse(result)
+    expect(parsed.action).toBe('doc_list')
+    expect(parsed.count).toBe(2)
+    expect(parsed.documents).toHaveLength(2)
+  })
+
+  it('should pass source filter to API', async () => {
+    vi.mocked(nexusPost).mockResolvedValue(
+      mockApiSuccess({
+        action: 'doc_list',
+        project_id: TEST_IDS.projectId,
+        count: 1,
+        documents: [
+          { id: TEST_IDS.documentId, title: 'Agent doc', source: 'agent:app-agent' },
+        ],
+      }),
+    )
+
+    const { listDocuments } = await import('../tools/list-documents.js')
+    await listDocuments({
+      project_id: TEST_IDS.projectId,
+      source: 'agent:app-agent',
+      user_id: TEST_IDS.userId,
+    })
+
+    expect(vi.mocked(nexusPost)).toHaveBeenCalledWith('/api/mcp/documents', {
+      action: 'doc_list',
+      project_id: TEST_IDS.projectId,
+      source: 'agent:app-agent',
+      limit: 50,
+    })
+  })
+
+  it('should return error on API failure', async () => {
+    vi.mocked(nexusPost).mockResolvedValue(mockApiError('No access to this project', 403))
+
+    const { listDocuments } = await import('../tools/list-documents.js')
+    const result = await listDocuments({
+      project_id: TEST_IDS.projectId,
+      user_id: TEST_IDS.userId,
+    })
+
+    expect(result.isError).toBe(true)
+  })
+
+  it('should handle empty document list', async () => {
+    vi.mocked(nexusPost).mockResolvedValue(
+      mockApiSuccess({
+        action: 'doc_list',
+        project_id: TEST_IDS.projectId,
+        count: 0,
+        documents: [],
+      }),
+    )
+
+    const { listDocuments } = await import('../tools/list-documents.js')
+    const result = await listDocuments({
+      project_id: TEST_IDS.projectId,
+      user_id: TEST_IDS.userId,
+    })
+
+    expect(result.isError).toBeUndefined()
+    const parsed = parseToolResponse(result)
+    expect(parsed.count).toBe(0)
+    expect(parsed.documents).toHaveLength(0)
+  })
+})

package/src/nexus-api.ts

@@ -9,6 +9,15 @@
 let _baseUrl: string | null = null
 let _token: string | null = null
 
+/**
+ * Reset the cached API configuration. Used by E2E tests to switch
+ * from mocked env vars to real credentials.
+ */
+export function resetApiConfig(): void {
+  _baseUrl = null
+  _token = null
+}
+
 function getConfig(): { baseUrl: string; token: string } {
   if (_baseUrl && _token) return { baseUrl: _baseUrl, token: _token }
 

package/src/server.ts

@@ -27,6 +27,7 @@
  *   - task_create: Create a task in a project
  *   - task_update: Update task status, priority, or assignee
  *   - task_note: Append a note to a task (append-only)
+ *   - task_list: List tasks for a project with filters
  *   - dc_add: Add a comment to an ADR (append-only)
  *   - dc_list: List comments for an ADR
  *   - sk_list: List skills for the current tenant
@@ -38,6 +39,7 @@
  *   - sk_unassign: Remove a skill assignment from a project
  *   - sk_export: Export all skill assignments for a project
  *   - doc_ingest: Push text/markdown content into project knowledge base
+ *   - doc_list: List ingested documents for a project
  *   - session_append: Append to a session (write-isolated)
  *   - session_create: Start a new work session
  *   - session_close: End a session with summary and next entry point
@@ -114,6 +116,11 @@ import {
   replyLetter,
   replyLetterSchema,
 } from './tools/letters.js'
+import {
+  listDocuments,
+  listDocumentsSchema,
+} from './tools/list-documents.js'
+import { listTasks, listTasksSchema } from './tools/list-tasks.js'
 import {
   closeSession,
   closeSessionSchema,
@@ -195,7 +202,7 @@ function withIdentity(handler: (args: any) => Promise<any>) {
 const server = new McpServer(
   {
     name: 'nexus-mcp',
-    version: '0.5.0',
+    version: '0.6.0',
   },
   {
     capabilities: {
@@ -303,6 +310,13 @@ server.tool(
   withIdentity(addTaskNote),
 )
 
+server.tool(
+  'task_list',
+  'List tasks for a project with optional status filtering. Returns tasks ordered by creation date (newest first).',
+  listTasksSchema,
+  withIdentity(listTasks),
+)
+
 server.tool(
   'doc_ingest',
   'Push text or markdown content into a project knowledge base. Creates an ingest item that can later be classified. Useful for agents to persist research results, generated documents, or extracted knowledge.',
@@ -310,6 +324,13 @@ server.tool(
   withIdentity(ingestDocument),
 )
 
+server.tool(
+  'doc_list',
+  'List ingested documents for a project with optional source filtering. Returns documents ordered by creation date (newest first).',
+  listDocumentsSchema,
+  withIdentity(listDocuments),
+)
+
 server.tool(
   'session_append',
   'Append an entry to an existing session. Enforces session write isolation: only the session creator can append entries.',

package/src/tools/list-documents.ts

@@ -0,0 +1,57 @@
+/**
+ * doc_list -- Layer 2 Coordination
+ *
+ * List ingested documents for a project with optional source filtering.
+ * Delegates to POST /api/mcp/documents (action: doc_list).
+ */
+
+import { z } from 'zod'
+import { nexusPost } from '../nexus-api.js'
+
+export const listDocumentsSchema = {
+  project_id: z.string().uuid().describe('Project UUID'),
+  source: z
+    .string()
+    .optional()
+    .describe('Filter by source (e.g., "mcp-agent", "research", "session-extract")'),
+  limit: z
+    .number()
+    .min(1)
+    .max(100)
+    .default(50)
+    .describe('Maximum number of documents to return (default 50)'),
+}
+
+type ListDocumentsArgs = {
+  project_id: string
+  source?: string
+  limit?: number
+  user_id: string
+}
+
+export async function listDocuments(args: ListDocumentsArgs) {
+  const result = await nexusPost('/api/mcp/documents', {
+    action: 'doc_list',
+    project_id: args.project_id,
+    source: args.source,
+    limit: args.limit ?? 50,
+  })
+
+  if (!result.ok) {
+    return {
+      content: [
+        {
+          type: 'text' as const,
+          text: JSON.stringify({ error: result.error }, null, 2),
+        },
+      ],
+      isError: true,
+    }
+  }
+
+  return {
+    content: [
+      { type: 'text' as const, text: JSON.stringify(result.data, null, 2) },
+    ],
+  }
+}

package/src/tools/list-tasks.ts

@@ -0,0 +1,57 @@
+/**
+ * task_list -- Layer 2 Coordination
+ *
+ * List tasks for a project with optional status filtering.
+ * Delegates to POST /api/mcp/tasks (action: task_list).
+ */
+
+import { z } from 'zod'
+import { nexusPost } from '../nexus-api.js'
+
+export const listTasksSchema = {
+  project_id: z.string().uuid().describe('Project UUID'),
+  status_filter: z
+    .array(z.enum(['open', 'in_progress', 'blocked', 'done', 'cancelled']))
+    .optional()
+    .describe('Filter by task status (default: all statuses)'),
+  limit: z
+    .number()
+    .min(1)
+    .max(100)
+    .default(50)
+    .describe('Maximum number of tasks to return (default 50)'),
+}
+
+type ListTasksArgs = {
+  project_id: string
+  status_filter?: string[]
+  limit?: number
+  user_id: string
+}
+
+export async function listTasks(args: ListTasksArgs) {
+  const result = await nexusPost('/api/mcp/tasks', {
+    action: 'task_list',
+    project_id: args.project_id,
+    status_filter: args.status_filter,
+    limit: args.limit ?? 50,
+  })
+
+  if (!result.ok) {
+    return {
+      content: [
+        {
+          type: 'text' as const,
+          text: JSON.stringify({ error: result.error }, null, 2),
+        },
+      ],
+      isError: true,
+    }
+  }
+
+  return {
+    content: [
+      { type: 'text' as const, text: JSON.stringify(result.data, null, 2) },
+    ],
+  }
+}