@gwdn/nexus-mcp 0.8.7 → 0.8.8

package/src/__tests__/layer3-governance.test.ts

@@ -1,205 +0,0 @@
-/**
- * Tests for Layer 3 Governance 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('Layer 3: ADR Governance tools', () => {
-  beforeEach(() => {
-    vi.restoreAllMocks()
-  })
-
-  describe('createAdrDraft', () => {
-    it('should create an ADR draft with auto-incremented number', async () => {
-      vi.mocked(nexusPost).mockResolvedValue(
-        mockApiSuccess({
-          action: 'adr_create',
-          adr_id: TEST_IDS.adrId,
-          adr_number: '0006',
-          project_id: TEST_IDS.projectId,
-          title: 'New ADR',
-          status: 'draft',
-        }),
-      )
-
-      const { createAdrDraft } = await import('../tools/governance.js')
-      const result = await createAdrDraft({
-        project_id: TEST_IDS.projectId,
-        title: 'New ADR',
-        context: 'ADR context',
-        decision: 'ADR decision content',
-        user_id: TEST_IDS.userId,
-      })
-
-      expect(result.isError).toBeUndefined()
-      const parsed = parseToolResponse(result)
-      expect(parsed.action).toBe('adr_create')
-      expect(parsed.adr_number).toBe('0006')
-      expect(parsed.status).toBe('draft')
-    })
-
-    it('should return error on API failure', async () => {
-      vi.mocked(nexusPost).mockResolvedValue(mockApiError('Failed to create ADR draft'))
-
-      const { createAdrDraft } = await import('../tools/governance.js')
-      const result = await createAdrDraft({
-        project_id: TEST_IDS.projectId,
-        title: 'Fail',
-        context: 'Context',
-        decision: 'Decision',
-        user_id: TEST_IDS.userId,
-      })
-
-      expect(result.isError).toBe(true)
-    })
-  })
-
-  describe('submitAdrReview', () => {
-    it('should transition a draft ADR to under_review', async () => {
-      vi.mocked(nexusPost).mockResolvedValue(
-        mockApiSuccess({
-          action: 'adr_submit',
-          adr_id: TEST_IDS.adrId,
-          title: 'ADR to review',
-          new_state: 'under_review',
-        }),
-      )
-
-      const { submitAdrReview } = await import('../tools/governance.js')
-      const result = await submitAdrReview({
-        adr_id: TEST_IDS.adrId,
-        user_id: TEST_IDS.userId,
-      })
-
-      expect(result.isError).toBeUndefined()
-      const parsed = parseToolResponse(result)
-      expect(parsed.action).toBe('adr_submit')
-      expect(parsed.new_state).toBe('under_review')
-    })
-
-    it('should return error for non-draft ADR', async () => {
-      vi.mocked(nexusPost).mockResolvedValue(
-        mockApiError('ADR must be in draft state to submit for review', 409),
-      )
-
-      const { submitAdrReview } = await import('../tools/governance.js')
-      const result = await submitAdrReview({
-        adr_id: TEST_IDS.adrId,
-        user_id: TEST_IDS.userId,
-      })
-
-      expect(result.isError).toBe(true)
-    })
-
-    it('should return error if ADR not found', async () => {
-      vi.mocked(nexusPost).mockResolvedValue(mockApiError('ADR not found', 404))
-
-      const { submitAdrReview } = await import('../tools/governance.js')
-      const result = await submitAdrReview({
-        adr_id: TEST_IDS.adrId,
-        user_id: TEST_IDS.userId,
-      })
-
-      expect(result.isError).toBe(true)
-    })
-  })
-
-  describe('recordAdrDecision', () => {
-    it('should accept an ADR under review', async () => {
-      vi.mocked(nexusPost).mockResolvedValue(
-        mockApiSuccess({
-          action: 'adr_decide',
-          adr_id: TEST_IDS.adrId,
-          title: 'Test ADR',
-          decision: 'accepted',
-          new_state: 'accepted',
-        }),
-      )
-
-      const { recordAdrDecision } = await import('../tools/governance.js')
-      const result = await recordAdrDecision({
-        adr_id: TEST_IDS.adrId,
-        decision: 'accepted',
-        user_id: TEST_IDS.userId,
-      })
-
-      expect(result.isError).toBeUndefined()
-      const parsed = parseToolResponse(result)
-      expect(parsed.action).toBe('adr_decide')
-      expect(parsed.decision).toBe('accepted')
-      expect(parsed.new_state).toBe('accepted')
-    })
-
-    it('should reject an ADR under review', async () => {
-      vi.mocked(nexusPost).mockResolvedValue(
-        mockApiSuccess({
-          action: 'adr_decide',
-          adr_id: TEST_IDS.adrId,
-          title: 'Test ADR',
-          decision: 'rejected',
-          new_state: 'rejected',
-        }),
-      )
-
-      const { recordAdrDecision } = await import('../tools/governance.js')
-      const result = await recordAdrDecision({
-        adr_id: TEST_IDS.adrId,
-        decision: 'rejected',
-        rationale: 'Does not align with platform direction',
-        user_id: TEST_IDS.userId,
-      })
-
-      expect(result.isError).toBeUndefined()
-      const parsed = parseToolResponse(result)
-      expect(parsed.decision).toBe('rejected')
-    })
-
-    it('should include superseded_adr in response when accepting with supersedes', async () => {
-      const supersededId = '11111111-1111-1111-1111-111111111111'
-      vi.mocked(nexusPost).mockResolvedValue(
-        mockApiSuccess({
-          action: 'adr_decide',
-          adr_id: TEST_IDS.adrId,
-          title: 'Superseding ADR',
-          decision: 'accepted',
-          new_state: 'accepted',
-          superseded_adr: supersededId,
-        }),
-      )
-
-      const { recordAdrDecision } = await import('../tools/governance.js')
-      const result = await recordAdrDecision({
-        adr_id: TEST_IDS.adrId,
-        decision: 'accepted',
-        user_id: TEST_IDS.userId,
-      })
-
-      expect(result.isError).toBeUndefined()
-      const parsed = parseToolResponse(result)
-      expect(parsed.superseded_adr).toBe(supersededId)
-    })
-
-    it('should return error for non-review ADR', async () => {
-      vi.mocked(nexusPost).mockResolvedValue(
-        mockApiError('ADR must be under review to record a decision', 409),
-      )
-
-      const { recordAdrDecision } = await import('../tools/governance.js')
-      const result = await recordAdrDecision({
-        adr_id: TEST_IDS.adrId,
-        decision: 'accepted',
-        user_id: TEST_IDS.userId,
-      })
-
-      expect(result.isError).toBe(true)
-    })
-  })
-})

package/src/__tests__/machine-id.test.ts

@@ -1,136 +0,0 @@
-/**
- * Tests for src/machine-id.ts
- *
- * Covers:
- * - Reading existing machine.toml
- * - Generating new machine ID when file doesn't exist
- * - Caching behavior
- * - Graceful failure on I/O errors
- */
-
-import { afterEach, beforeEach, describe, expect, it, vi } from 'vitest'
-import { existsSync, mkdirSync, readFileSync, writeFileSync } from 'node:fs'
-import { join } from 'node:path'
-import { homedir } from 'node:os'
-
-// Mock fs and os modules
-vi.mock('node:fs', () => ({
-  existsSync: vi.fn(),
-  mkdirSync: vi.fn(),
-  readFileSync: vi.fn(),
-  writeFileSync: vi.fn(),
-}))
-
-vi.mock('node:os', () => ({
-  homedir: vi.fn(() => '/mock-home'),
-}))
-
-const EXPECTED_TOML_PATH = join('/mock-home', '.config', 'nexus', 'machine.toml')
-
-describe('machine-id', () => {
-  beforeEach(() => {
-    vi.resetModules()
-    vi.restoreAllMocks()
-    vi.mocked(homedir).mockReturnValue('/mock-home')
-  })
-
-  afterEach(() => {
-    vi.restoreAllMocks()
-  })
-
-  it('should read machine_id from existing machine.toml', async () => {
-    vi.mocked(existsSync).mockReturnValue(true)
-    vi.mocked(readFileSync).mockReturnValue(
-      '# Nexus persistent machine identifier\nmachine_id = "aaaaaaaa-bbbb-cccc-dddd-eeeeeeeeeeee"\n',
-    )
-
-    const { getMachineId, resetMachineId } = await import('../machine-id.js')
-    resetMachineId()
-    const id = getMachineId()
-
-    expect(id).toBe('aaaaaaaa-bbbb-cccc-dddd-eeeeeeeeeeee')
-    expect(existsSync).toHaveBeenCalledWith(EXPECTED_TOML_PATH)
-    expect(readFileSync).toHaveBeenCalledWith(EXPECTED_TOML_PATH, 'utf-8')
-  })
-
-  it('should generate and persist machine_id when file does not exist', async () => {
-    vi.mocked(existsSync).mockReturnValue(false)
-
-    const { getMachineId, resetMachineId } = await import('../machine-id.js')
-    resetMachineId()
-    const id = getMachineId()
-
-    expect(id).toBeDefined()
-    expect(id).toMatch(/^[0-9a-f]{8}-[0-9a-f]{4}-[0-9a-f]{4}-[0-9a-f]{4}-[0-9a-f]{12}$/)
-    expect(mkdirSync).toHaveBeenCalledWith(
-      join('/mock-home', '.config', 'nexus'),
-      { recursive: true },
-    )
-    expect(writeFileSync).toHaveBeenCalledWith(
-      EXPECTED_TOML_PATH,
-      expect.stringContaining(`machine_id = "${id}"`),
-      'utf-8',
-    )
-  })
-
-  it('should cache the machine_id after first read', async () => {
-    vi.mocked(existsSync).mockReturnValue(true)
-    vi.mocked(readFileSync).mockReturnValue('machine_id = "cached-id-1234"\n')
-
-    const { getMachineId, resetMachineId } = await import('../machine-id.js')
-    resetMachineId()
-
-    const callsBefore = vi.mocked(readFileSync).mock.calls.length
-    const id1 = getMachineId()
-    const id2 = getMachineId()
-
-    expect(id1).toBe('cached-id-1234')
-    expect(id1).toBe(id2)
-    // readFileSync should only be called once more after reset (cached on second call)
-    expect(vi.mocked(readFileSync).mock.calls.length - callsBefore).toBe(1)
-  })
-
-  it('should return null on I/O error', async () => {
-    vi.mocked(existsSync).mockImplementation(() => {
-      throw new Error('Permission denied')
-    })
-
-    const { getMachineId, resetMachineId } = await import('../machine-id.js')
-    resetMachineId()
-    const id = getMachineId()
-
-    expect(id).toBeNull()
-  })
-
-  it('should generate new ID if toml exists but has no machine_id line', async () => {
-    vi.mocked(existsSync).mockReturnValue(true)
-    vi.mocked(readFileSync).mockReturnValue('# empty config\n')
-
-    const { getMachineId, resetMachineId } = await import('../machine-id.js')
-    resetMachineId()
-    const id = getMachineId()
-
-    expect(id).toBeDefined()
-    expect(id).toMatch(/^[0-9a-f]{8}-/)
-    expect(writeFileSync).toHaveBeenCalled()
-  })
-
-  it('should resetMachineId clear the cache', async () => {
-    vi.mocked(existsSync).mockReturnValue(true)
-    vi.mocked(readFileSync)
-      .mockReturnValueOnce('machine_id = "first-id"\n')
-      .mockReturnValueOnce('machine_id = "second-id"\n')
-
-    const { getMachineId, resetMachineId } = await import('../machine-id.js')
-    resetMachineId()
-
-    const id1 = getMachineId()
-    expect(id1).toBe('first-id')
-
-    resetMachineId()
-    const id2 = getMachineId()
-    expect(id2).toBe('second-id')
-
-    expect(id1).not.toBe(id2)
-  })
-})

package/src/__tests__/nexus-api.test.ts

@@ -1,127 +0,0 @@
-/**
- * Tests for src/nexus-api.ts
- *
- * Covers:
- * - X-Nexus-Machine-Id header is sent automatically when machine_id is available
- * - Custom headers parameter is forwarded
- * - nexusPost convenience works correctly
- */
-
-import { afterEach, beforeEach, describe, expect, it, vi } from 'vitest'
-
-// Mock machine-id module before importing nexus-api
-vi.mock('../machine-id.js', () => ({
-  getMachineId: vi.fn(() => 'test-machine-uuid-1234'),
-  resetMachineId: vi.fn(),
-}))
-
-describe('nexus-api: header injection', () => {
-  let nexusApi: typeof import('../nexus-api.js').nexusApi
-  let nexusPost: typeof import('../nexus-api.js').nexusPost
-  let fetchSpy: ReturnType<typeof vi.fn>
-
-  beforeEach(async () => {
-    vi.resetModules()
-
-    process.env.NEXUS_API_URL = 'https://test.nexus.example.com'
-    process.env.NEXUS_PRIVATE_TOKEN = 'nxs_pat_test-token-for-header-tests-abc'
-
-    fetchSpy = vi.fn().mockResolvedValue({
-      ok: true,
-      status: 200,
-      json: async () => ({ action: 'test' }),
-    })
-    vi.stubGlobal('fetch', fetchSpy)
-
-    const mod = await import('../nexus-api.js')
-    nexusApi = mod.nexusApi
-    nexusPost = mod.nexusPost
-    mod.resetApiConfig()
-  })
-
-  afterEach(() => {
-    vi.unstubAllGlobals()
-  })
-
-  it('should include X-Nexus-Machine-Id header in requests', async () => {
-    await nexusApi('/api/mcp/sessions', { method: 'POST', body: { action: 'test' } })
-
-    expect(fetchSpy).toHaveBeenCalledTimes(1)
-    const [, opts] = fetchSpy.mock.calls[0]
-    expect(opts.headers['X-Nexus-Machine-Id']).toBe('test-machine-uuid-1234')
-  })
-
-  it('should include Authorization and Content-Type headers', async () => {
-    await nexusApi('/api/mcp/sessions')
-
-    const [, opts] = fetchSpy.mock.calls[0]
-    expect(opts.headers['Authorization']).toBe(
-      'Bearer nxs_pat_test-token-for-header-tests-abc',
-    )
-    expect(opts.headers['Content-Type']).toBe('application/json')
-  })
-
-  it('should merge custom headers with defaults', async () => {
-    await nexusApi('/api/mcp/sessions', {
-      method: 'POST',
-      body: { action: 'test' },
-      headers: { 'X-Nexus-Toolstack': 'opencode v0.3.0' },
-    })
-
-    const [, opts] = fetchSpy.mock.calls[0]
-    expect(opts.headers['X-Nexus-Machine-Id']).toBe('test-machine-uuid-1234')
-    expect(opts.headers['X-Nexus-Toolstack']).toBe('opencode v0.3.0')
-  })
-
-  it('should allow custom headers to override machine-id', async () => {
-    await nexusApi('/api/mcp/sessions', {
-      headers: { 'X-Nexus-Machine-Id': 'override-id' },
-    })
-
-    const [, opts] = fetchSpy.mock.calls[0]
-    expect(opts.headers['X-Nexus-Machine-Id']).toBe('override-id')
-  })
-
-  it('nexusPost should also include machine-id header', async () => {
-    await nexusPost('/api/mcp/sessions', { action: 'session_create' })
-
-    const [, opts] = fetchSpy.mock.calls[0]
-    expect(opts.headers['X-Nexus-Machine-Id']).toBe('test-machine-uuid-1234')
-    expect(opts.method).toBe('POST')
-  })
-})
-
-describe('nexus-api: no machine-id available', () => {
-  beforeEach(async () => {
-    vi.resetModules()
-    // Re-mock with null return
-    vi.doMock('../machine-id.js', () => ({
-      getMachineId: vi.fn(() => null),
-      resetMachineId: vi.fn(),
-    }))
-
-    process.env.NEXUS_API_URL = 'https://test.nexus.example.com'
-    process.env.NEXUS_PRIVATE_TOKEN = 'nxs_pat_test-token-no-machine-id-abcde'
-  })
-
-  afterEach(() => {
-    vi.unstubAllGlobals()
-  })
-
-  it('should not include X-Nexus-Machine-Id header when machine_id is null', async () => {
-    const fetchSpy = vi.fn().mockResolvedValue({
-      ok: true,
-      status: 200,
-      json: async () => ({ action: 'test' }),
-    })
-    vi.stubGlobal('fetch', fetchSpy)
-
-    const mod = await import('../nexus-api.js')
-    mod.resetApiConfig()
-
-    await mod.nexusApi('/api/mcp/sessions')
-
-    const [, opts] = fetchSpy.mock.calls[0]
-    expect(opts.headers).not.toHaveProperty('X-Nexus-Machine-Id')
-  })
-})