@gwdn/nexus-mcp 0.8.7 → 0.8.9

package/dist/server.js

@@ -121,7 +121,7 @@ function withIdentity(handler) {
 // ---------------------------------------------------------------------------
 const server = new McpServer({
     name: 'nexus-mcp',
-    version: '0.8.7',
+    version: '0.8.9',
 }, {
     capabilities: {
         tools: {},

package/dist/tools/create-task.d.ts

@@ -11,7 +11,7 @@ export declare const createTaskSchema: {
     description: z.ZodOptional<z.ZodString>;
     priority: z.ZodDefault<z.ZodEnum<{
         low: "low";
-        normal: "normal";
+        medium: "medium";
         high: "high";
         urgent: "urgent";
     }>>;

package/dist/tools/create-task.js

@@ -11,8 +11,8 @@ export const createTaskSchema = {
     title: z.string().max(500).describe('Task title'),
     description: z.string().max(100_000).optional().describe('Task description'),
     priority: z
-        .enum(['low', 'normal', 'high', 'urgent'])
-        .default('normal')
+        .enum(['low', 'medium', 'high', 'urgent'])
+        .default('medium')
         .describe('Task priority'),
     assignee: z.string().uuid().optional().describe('UUID of the assigned user'),
     status: z
@@ -27,7 +27,7 @@ export async function createTask(args) {
         project_id: args.project_id,
         title: args.title,
         description: args.description,
-        priority: args.priority ?? 'normal',
+        priority: args.priority ?? 'medium',
         assignee: args.assignee,
         status: args.status ?? 'open',
     });

package/dist/tools/directives.d.ts

@@ -43,8 +43,8 @@ export declare const pdCreateSchema: {
     category: z.ZodDefault<z.ZodString>;
     priority: z.ZodDefault<z.ZodEnum<{
         low: "low";
-        high: "high";
         medium: "medium";
+        high: "high";
     }>>;
     enabled: z.ZodDefault<z.ZodBoolean>;
 };
@@ -70,8 +70,8 @@ export declare const pdUpdateSchema: {
     category: z.ZodOptional<z.ZodString>;
     priority: z.ZodOptional<z.ZodEnum<{
         low: "low";
-        high: "high";
         medium: "medium";
+        high: "high";
     }>>;
     enabled: z.ZodOptional<z.ZodBoolean>;
 };

package/dist/tools/letters.d.ts

@@ -13,9 +13,9 @@ export declare const createLetterSchema: {
     body: z.ZodString;
     priority: z.ZodDefault<z.ZodEnum<{
         low: "low";
-        normal: "normal";
         high: "high";
         urgent: "urgent";
+        normal: "normal";
     }>>;
     blocking: z.ZodDefault<z.ZodBoolean>;
     thread_id: z.ZodOptional<z.ZodString>;

package/dist/tools/update-task-status.d.ts

@@ -16,9 +16,9 @@ export declare const updateTaskStatusSchema: {
     }>;
     priority: z.ZodOptional<z.ZodEnum<{
         low: "low";
+        medium: "medium";
         high: "high";
         urgent: "urgent";
-        medium: "medium";
     }>>;
     assignee: z.ZodOptional<z.ZodString>;
     agent_id: z.ZodOptional<z.ZodString>;

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@gwdn/nexus-mcp",
-  "version": "0.8.7",
+  "version": "0.8.9",
   "description": "MCP server for the Nexus platform",
   "type": "module",
   "main": "dist/server.js",
@@ -8,7 +8,8 @@
     "nexus-mcp": "dist/server.js"
   },
   "scripts": {
-    "build": "tsc",
+    "build": "tsc && chmod +x dist/server.js",
+    "prepublishOnly": "npm run build",
     "dev": "tsx src/server.ts",
     "start": "node dist/server.js",
     "test": "vitest run",
@@ -25,6 +26,11 @@
     "typescript": "^5.8.3",
     "vitest": "^4.1.4"
   },
+  "files": [
+    "dist",
+    "README.md",
+    "LICENSE"
+  ],
   "engines": {
     "node": ">=20"
   }

package/.github/workflows/ci.yml

@@ -1,38 +0,0 @@
-name: CI
-
-on:
-  push:
-    branches: [main]
-  pull_request:
-    branches: [main]
-
-jobs:
-  test:
-    name: Test (Node ${{ matrix.node-version }})
-    runs-on: blacksmith-2vcpu-ubuntu-2204
-    strategy:
-      fail-fast: false
-      matrix:
-        node-version: [20, 22]
-
-    steps:
-      - name: Checkout
-        uses: actions/checkout@v4
-
-      - name: Setup Node.js ${{ matrix.node-version }}
-        uses: actions/setup-node@v4
-        with:
-          node-version: ${{ matrix.node-version }}
-          cache: npm
-
-      - name: Install dependencies
-        run: npm ci
-
-      - name: Typecheck
-        run: npm run typecheck
-
-      - name: Test
-        run: npm test
-
-      - name: Build
-        run: npm run build

package/.github/workflows/release.yml

@@ -1,57 +0,0 @@
-name: Release
-
-on:
-  push:
-    tags:
-      - "v*"
-
-permissions:
-  contents: write
-
-jobs:
-  release:
-    name: Publish to npm
-    runs-on: blacksmith-2vcpu-ubuntu-2204
-
-    steps:
-      - name: Checkout
-        uses: actions/checkout@v4
-
-      - name: Setup Node.js
-        uses: actions/setup-node@v4
-        with:
-          node-version: 22
-          cache: npm
-          registry-url: https://registry.npmjs.org
-
-      - name: Install dependencies
-        run: npm ci
-
-      - name: Typecheck
-        run: npm run typecheck
-
-      - name: Test
-        run: npm test
-
-      - name: Build
-        run: npm run build
-
-      - name: Verify version matches tag
-        run: |
-          PKG_VERSION="v$(node -p "require('./package.json').version")"
-          TAG_VERSION="${GITHUB_REF#refs/tags/}"
-          if [ "$PKG_VERSION" != "$TAG_VERSION" ]; then
-            echo "ERROR: package.json version ($PKG_VERSION) does not match tag ($TAG_VERSION)"
-            exit 1
-          fi
-          echo "Version verified: $TAG_VERSION"
-
-      - name: Publish to npm
-        run: npm publish --access public
-        env:
-          NODE_AUTH_TOKEN: ${{ secrets.NPM_TOKEN }}
-
-      - name: Create GitHub Release
-        uses: softprops/action-gh-release@v2
-        with:
-          generate_release_notes: true

package/src/__tests__/auth.test.ts

@@ -1,162 +0,0 @@
-/**
- * Tests for src/auth.ts
- *
- * Covers:
- * - initIdentity() with valid/invalid tokens
- * - getIdentity() before/after init
- */
-
-import { beforeEach, describe, expect, it, vi } from 'vitest'
-
-// Mock the nexus-api module before importing auth
-vi.mock('../nexus-api.js', () => ({
-  nexusGet: vi.fn(),
-}))
-
-import { nexusGet } from '../nexus-api.js'
-import { TEST_IDS } from './helpers'
-
-describe('MCP Auth', () => {
-  beforeEach(() => {
-    vi.resetModules()
-    vi.restoreAllMocks()
-    // Suppress console.error
-    vi.spyOn(console, 'error').mockImplementation(() => {})
-  })
-
-  describe('initIdentity', () => {
-    it('should resolve identity from a valid token', async () => {
-      vi.mocked(nexusGet).mockResolvedValue({
-        ok: true,
-        status: 200,
-        data: {
-          userId: TEST_IDS.userId,
-          email: 'test@example.com',
-          displayName: 'Test User',
-          isPlatformAdmin: true,
-          isPlatformOwner: false,
-          tenantId: TEST_IDS.tenantId,
-          memberships: [],
-          agentAssignments: [],
-        },
-        error: null,
-      })
-
-      process.env.NEXUS_PRIVATE_TOKEN =
-        'nxs_pat_valid-test-token-1234567890abcde'
-
-      const { initIdentity } = await import('../auth.js')
-      const identity = await initIdentity()
-
-      expect(identity).toBeDefined()
-      expect(identity.userId).toBe(TEST_IDS.userId)
-      expect(identity.displayName).toBe('Test User')
-      expect(identity.email).toBe('test@example.com')
-      expect(identity.isPlatformAdmin).toBe(true)
-      expect(identity.isPlatformOwner).toBe(false)
-    })
-
-    it('should throw if NEXUS_PRIVATE_TOKEN is not set', async () => {
-      delete process.env.NEXUS_PRIVATE_TOKEN
-
-      const { initIdentity } = await import('../auth.js')
-      await expect(initIdentity()).rejects.toThrow(
-        'NEXUS_PRIVATE_TOKEN is not set',
-      )
-    })
-
-    it('should throw if API returns an error', async () => {
-      vi.mocked(nexusGet).mockResolvedValue({
-        ok: false,
-        status: 401,
-        data: null,
-        error: 'Invalid token',
-      })
-
-      process.env.NEXUS_PRIVATE_TOKEN = 'nxs_pat_invalid-token-123456789012345'
-
-      const { initIdentity } = await import('../auth.js')
-      await expect(initIdentity()).rejects.toThrow(
-        'Identity resolution failed',
-      )
-    })
-
-    it('should throw if API returns null data', async () => {
-      vi.mocked(nexusGet).mockResolvedValue({
-        ok: true,
-        status: 200,
-        data: null,
-        error: null,
-      })
-
-      process.env.NEXUS_PRIVATE_TOKEN = 'nxs_pat_null-data-token-12345678901234'
-
-      const { initIdentity } = await import('../auth.js')
-      await expect(initIdentity()).rejects.toThrow(
-        'Identity resolution failed',
-      )
-    })
-
-    it('should resolve platform_owner identity with both admin and owner flags', async () => {
-      vi.mocked(nexusGet).mockResolvedValue({
-        ok: true,
-        status: 200,
-        data: {
-          userId: TEST_IDS.userId,
-          email: 'owner@example.com',
-          displayName: 'Platform Owner',
-          isPlatformAdmin: true,
-          isPlatformOwner: true,
-          tenantId: TEST_IDS.tenantId,
-          memberships: [],
-          agentAssignments: [],
-        },
-        error: null,
-      })
-
-      process.env.NEXUS_PRIVATE_TOKEN = 'nxs_pat_owner-test-token-12345678901'
-
-      const { initIdentity } = await import('../auth.js')
-      const identity = await initIdentity()
-
-      expect(identity.isPlatformAdmin).toBe(true)
-      expect(identity.isPlatformOwner).toBe(true)
-      expect(identity.displayName).toBe('Platform Owner')
-      expect(identity.email).toBe('owner@example.com')
-    })
-  })
-
-  describe('getIdentity', () => {
-    it('should throw if called before initIdentity', async () => {
-      const { getIdentity } = await import('../auth.js')
-      expect(() => getIdentity()).toThrow('Identity not initialized')
-    })
-
-    it('should return cached identity after init', async () => {
-      vi.mocked(nexusGet).mockResolvedValue({
-        ok: true,
-        status: 200,
-        data: {
-          userId: TEST_IDS.userId,
-          email: 'cached@example.com',
-          displayName: 'Cached User',
-          isPlatformAdmin: false,
-          isPlatformOwner: false,
-          tenantId: TEST_IDS.tenantId,
-          memberships: [],
-          agentAssignments: [],
-        },
-        error: null,
-      })
-
-      process.env.NEXUS_PRIVATE_TOKEN = 'nxs_pat_cached-test-token-1234567890'
-
-      const { initIdentity, getIdentity } = await import('../auth.js')
-      await initIdentity()
-
-      const identity = getIdentity()
-      expect(identity.userId).toBe(TEST_IDS.userId)
-      expect(identity.displayName).toBe('Cached User')
-    })
-  })
-})

package/src/__tests__/decision-comments.test.ts

@@ -1,173 +0,0 @@
-/**
- * Tests for decision comment tools.
- * All tools delegate to the Nexus API via nexusPost().
- */
-
-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'
-
-describe('Decision Comments tools', () => {
-  beforeEach(() => {
-    vi.restoreAllMocks()
-  })
-
-  describe('addDecisionComment', () => {
-    it('should add a comment to an existing decision', async () => {
-      vi.mocked(nexusPost).mockResolvedValue(
-        mockApiSuccess({
-          action: 'dc_add',
-          comment_id: 'comment-id-1',
-          decision_id: TEST_IDS.adrId,
-          decision_title: 'Test ADR',
-          project_id: TEST_IDS.projectId,
-        }),
-      )
-
-      const { addDecisionComment } = await import('../tools/decision-comments.js')
-      const result = await addDecisionComment({
-        decision_id: TEST_IDS.adrId,
-        body: 'This is a comment',
-        user_id: TEST_IDS.userId,
-      })
-
-      expect(result.isError).toBeUndefined()
-      const parsed = parseToolResponse(result)
-      expect(parsed.action).toBe('dc_add')
-      expect(parsed.comment_id).toBe('comment-id-1')
-      expect(parsed.decision_id).toBe(TEST_IDS.adrId)
-      expect(parsed.decision_title).toBe('Test ADR')
-    })
-
-    it('should support agent_id in comment', async () => {
-      vi.mocked(nexusPost).mockResolvedValue(
-        mockApiSuccess({
-          action: 'dc_add',
-          comment_id: 'comment-id-2',
-          decision_id: TEST_IDS.adrId,
-          decision_title: 'ADR',
-          project_id: TEST_IDS.projectId,
-        }),
-      )
-
-      const { addDecisionComment } = await import('../tools/decision-comments.js')
-      const result = await addDecisionComment({
-        decision_id: TEST_IDS.adrId,
-        body: 'Agent comment',
-        agent_id: TEST_IDS.agentId,
-        user_id: TEST_IDS.userId,
-      })
-
-      expect(result.isError).toBeUndefined()
-      // Verify agent_id was passed to the API
-      expect(nexusPost).toHaveBeenCalledWith('/api/mcp/governance', {
-        action: 'dc_add',
-        decision_id: TEST_IDS.adrId,
-        body: 'Agent comment',
-        agent_id: TEST_IDS.agentId,
-      })
-    })
-
-    it('should return error if decision not found', async () => {
-      vi.mocked(nexusPost).mockResolvedValue(mockApiError('Decision not found', 404))
-
-      const { addDecisionComment } = await import('../tools/decision-comments.js')
-      const result = await addDecisionComment({
-        decision_id: TEST_IDS.adrId,
-        body: 'Comment',
-        user_id: TEST_IDS.userId,
-      })
-
-      expect(result.isError).toBe(true)
-    })
-
-    it('should return error on API failure', async () => {
-      vi.mocked(nexusPost).mockResolvedValue(mockApiError('Failed to add comment'))
-
-      const { addDecisionComment } = await import('../tools/decision-comments.js')
-      const result = await addDecisionComment({
-        decision_id: TEST_IDS.adrId,
-        body: 'Comment',
-        user_id: TEST_IDS.userId,
-      })
-
-      expect(result.isError).toBe(true)
-    })
-  })
-
-  describe('listDecisionComments', () => {
-    it('should list comments for a decision', async () => {
-      vi.mocked(nexusPost).mockResolvedValue(
-        mockApiSuccess({
-          decision_id: TEST_IDS.adrId,
-          decision_title: 'Test ADR',
-          total: 2,
-          comments: [
-            {
-              id: 'c1',
-              author_id: TEST_IDS.userId,
-              agent_id: null,
-              body: 'First comment',
-              created_at: '2026-01-01T00:00:00Z',
-            },
-            {
-              id: 'c2',
-              author_id: TEST_IDS.userId,
-              agent_id: TEST_IDS.agentId,
-              body: 'Second comment',
-              created_at: '2026-01-01T01:00:00Z',
-            },
-          ],
-        }),
-      )
-
-      const { listDecisionComments } = await import('../tools/decision-comments.js')
-      const result = await listDecisionComments({
-        decision_id: TEST_IDS.adrId,
-      })
-
-      expect(result.isError).toBeUndefined()
-      const parsed = parseToolResponse(result)
-      expect(parsed.decision_title).toBe('Test ADR')
-      expect(parsed.total).toBe(2)
-      expect(parsed.comments).toHaveLength(2)
-      expect(parsed.comments[0].body).toBe('First comment')
-    })
-
-    it('should return empty list for decision with no comments', async () => {
-      vi.mocked(nexusPost).mockResolvedValue(
-        mockApiSuccess({
-          decision_id: TEST_IDS.adrId,
-          decision_title: 'ADR',
-          total: 0,
-          comments: [],
-        }),
-      )
-
-      const { listDecisionComments } = await import('../tools/decision-comments.js')
-      const result = await listDecisionComments({
-        decision_id: TEST_IDS.adrId,
-      })
-
-      const parsed = parseToolResponse(result)
-      expect(parsed.total).toBe(0)
-      expect(parsed.comments).toHaveLength(0)
-    })
-
-    it('should return error if decision not found', async () => {
-      vi.mocked(nexusPost).mockResolvedValue(mockApiError('Decision not found', 404))
-
-      const { listDecisionComments } = await import('../tools/decision-comments.js')
-      const result = await listDecisionComments({
-        decision_id: TEST_IDS.adrId,
-      })
-
-      expect(result.isError).toBe(true)
-    })
-  })
-})