@gwdn/nexus-mcp 0.8.6 → 0.8.8

package/README.md

@@ -89,12 +89,11 @@ terminals that were started before the update.
 
 **Symptoms:** MCP tools report an older version, or new tools/fixes are missing.
 
-**Fix:** Before starting OpenCode, clear stale npx caches:
+**Fix:** Before starting OpenCode, clear all npx caches for this package:
 
 ```bash
-find ~/.npm/_npx -path "*/node_modules/@gwdn/nexus-mcp/package.json" \
-  -exec node -e "if(require('{}').version!=='0.8.1') console.log(require('path').resolve('{}','../../..'))" \; \
-  | xargs rm -rf
+find ~/.npm/_npx -path "*/node_modules/@gwdn/nexus-mcp" -type d \
+  -exec rm -rf {} + 2>/dev/null
 ```
 
 Then start OpenCode normally. npx will fetch the latest version into a fresh

package/dist/nexus-api.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"nexus-api.d.ts","sourceRoot":"","sources":["../src/nexus-api.ts"],"names":[],"mappings":"AAAA;;;;;;GAMG;AAeH;;;GAGG;AACH,wBAAgB,cAAc,IAAI,IAAI,CAIrC;AA0BD,MAAM,WAAW,gBAAgB,CAAC,CAAC,GAAG,OAAO;IAC3C,EAAE,EAAE,OAAO,CAAA;IACX,MAAM,EAAE,MAAM,CAAA;IACd,IAAI,EAAE,CAAC,GAAG,IAAI,CAAA;IACd,KAAK,EAAE,MAAM,GAAG,IAAI,CAAA;CACrB;AAKD;;GAEG;AACH,wBAAsB,QAAQ,CAAC,CAAC,GAAG,OAAO,EACxC,IAAI,EAAE,MAAM,EACZ,OAAO,GAAE;IACP,MAAM,CAAC,EAAE,KAAK,GAAG,MAAM,GAAG,OAAO,GAAG,QAAQ,CAAA;IAC5C,IAAI,CAAC,EAAE,OAAO,CAAA;IACd,OAAO,CAAC,EAAE,MAAM,CAAC,MAAM,EAAE,MAAM,CAAC,CAAA;IAChC,SAAS,CAAC,EAAE,MAAM,CAAA;CACd,GACL,OAAO,CAAC,gBAAgB,CAAC,CAAC,CAAC,CAAC,CA6C9B;AAED;;GAEG;AACH,wBAAsB,SAAS,CAAC,CAAC,GAAG,OAAO,EACzC,IAAI,EAAE,MAAM,EACZ,IAAI,EAAE,OAAO,GACZ,OAAO,CAAC,gBAAgB,CAAC,CAAC,CAAC,CAAC,CAE9B;AAED;;GAEG;AACH,wBAAsB,QAAQ,CAAC,CAAC,GAAG,OAAO,EACxC,IAAI,EAAE,MAAM,GACX,OAAO,CAAC,gBAAgB,CAAC,CAAC,CAAC,CAAC,CAE9B"}
+{"version":3,"file":"nexus-api.d.ts","sourceRoot":"","sources":["../src/nexus-api.ts"],"names":[],"mappings":"AAAA;;;;;;GAMG;AAsBH;;;GAGG;AACH,wBAAgB,cAAc,IAAI,IAAI,CAIrC;AA0BD,MAAM,WAAW,gBAAgB,CAAC,CAAC,GAAG,OAAO;IAC3C,EAAE,EAAE,OAAO,CAAA;IACX,MAAM,EAAE,MAAM,CAAA;IACd,IAAI,EAAE,CAAC,GAAG,IAAI,CAAA;IACd,KAAK,EAAE,MAAM,GAAG,IAAI,CAAA;CACrB;AAKD;;GAEG;AACH,wBAAsB,QAAQ,CAAC,CAAC,GAAG,OAAO,EACxC,IAAI,EAAE,MAAM,EACZ,OAAO,GAAE;IACP,MAAM,CAAC,EAAE,KAAK,GAAG,MAAM,GAAG,OAAO,GAAG,QAAQ,CAAA;IAC5C,IAAI,CAAC,EAAE,OAAO,CAAA;IACd,OAAO,CAAC,EAAE,MAAM,CAAC,MAAM,EAAE,MAAM,CAAC,CAAA;IAChC,SAAS,CAAC,EAAE,MAAM,CAAA;CACd,GACL,OAAO,CAAC,gBAAgB,CAAC,CAAC,CAAC,CAAC,CA8C9B;AAED;;GAEG;AACH,wBAAsB,SAAS,CAAC,CAAC,GAAG,OAAO,EACzC,IAAI,EAAE,MAAM,EACZ,IAAI,EAAE,OAAO,GACZ,OAAO,CAAC,gBAAgB,CAAC,CAAC,CAAC,CAAC,CAE9B;AAED;;GAEG;AACH,wBAAsB,QAAQ,CAAC,CAAC,GAAG,OAAO,EACxC,IAAI,EAAE,MAAM,GACX,OAAO,CAAC,gBAAgB,CAAC,CAAC,CAAC,CAAC,CAE9B"}

package/dist/nexus-api.js

@@ -5,17 +5,25 @@
  * All operations go through /api/mcp/* endpoints, authenticated
  * via the NEXUS_PRIVATE_TOKEN (nxs_pat_*) Bearer token.
  */
+import { hostname } from 'node:os';
 import { getMachineId } from './machine-id.js';
 let _baseUrl = null;
 let _token = null;
 let _machineId = null;
-// Eagerly resolve machine ID (sync, cached after first call)
+let _machineName = null;
+// Eagerly resolve machine ID and hostname (sync, cached after first call)
 try {
     _machineId = getMachineId();
 }
 catch {
     // machine_id is optional enrichment — never block startup
 }
+try {
+    _machineName = hostname();
+}
+catch {
+    // hostname is optional enrichment
+}
 /**
  * Reset the cached API configuration. Used by E2E tests to switch
  * from mocked env vars to real credentials.
@@ -54,6 +62,7 @@ export async function nexusApi(path, options = {}) {
         Authorization: `Bearer ${token}`,
         'Content-Type': 'application/json',
         ...(_machineId ? { 'X-Nexus-Machine-Id': _machineId } : {}),
+        ...(_machineName ? { 'X-Nexus-Machine-Name': _machineName } : {}),
         ...(options.headers ?? {}),
     };
     try {

package/dist/nexus-api.js.map

@@ -1 +1 @@
-{"version":3,"file":"nexus-api.js","sourceRoot":"","sources":["../src/nexus-api.ts"],"names":[],"mappings":"AAAA;;;;;;GAMG;AAEH,OAAO,EAAE,YAAY,EAAE,MAAM,iBAAiB,CAAA;AAE9C,IAAI,QAAQ,GAAkB,IAAI,CAAA;AAClC,IAAI,MAAM,GAAkB,IAAI,CAAA;AAChC,IAAI,UAAU,GAAkB,IAAI,CAAA;AAEpC,6DAA6D;AAC7D,IAAI,CAAC;IACH,UAAU,GAAG,YAAY,EAAE,CAAA;AAC7B,CAAC;AAAC,MAAM,CAAC;IACP,0DAA0D;AAC5D,CAAC;AAED;;;GAGG;AACH,MAAM,UAAU,cAAc;IAC5B,QAAQ,GAAG,IAAI,CAAA;IACf,MAAM,GAAG,IAAI,CAAA;IACb,UAAU,GAAG,YAAY,EAAE,CAAA;AAC7B,CAAC;AAED,SAAS,SAAS;IAChB,IAAI,QAAQ,IAAI,MAAM;QAAE,OAAO,EAAE,OAAO,EAAE,QAAQ,EAAE,KAAK,EAAE,MAAM,EAAE,CAAA;IAEnE,MAAM,GAAG,GAAG,OAAO,CAAC,GAAG,CAAC,aAAa,IAAI,OAAO,CAAC,GAAG,CAAC,SAAS,CAAA;IAC9D,MAAM,KAAK,GAAG,OAAO,CAAC,GAAG,CAAC,mBAAmB,CAAA;IAE7C,IAAI,CAAC,GAAG,EAAE,CAAC;QACT,MAAM,IAAI,KAAK,CACb,mGAAmG,CACpG,CAAA;IACH,CAAC;IACD,IAAI,CAAC,KAAK,EAAE,CAAC;QACX,MAAM,IAAI,KAAK,CACb,oFAAoF,CACrF,CAAA;IACH,CAAC;IAED,kCAAkC;IAClC,QAAQ,GAAG,GAAG,CAAC,OAAO,CAAC,KAAK,EAAE,EAAE,CAAC,CAAA;IACjC,MAAM,GAAG,KAAK,CAAC,IAAI,EAAE,CAAA;IAErB,OAAO,EAAE,OAAO,EAAE,QAAQ,EAAE,KAAK,EAAE,MAAM,EAAE,CAAA;AAC7C,CAAC;AASD,yDAAyD;AACzD,MAAM,kBAAkB,GAAG,MAAM,CAAA;AAEjC;;GAEG;AACH,MAAM,CAAC,KAAK,UAAU,QAAQ,CAC5B,IAAY,EACZ,UAKI,EAAE;IAEN,MAAM,EAAE,OAAO,EAAE,KAAK,EAAE,GAAG,SAAS,EAAE,CAAA;IACtC,MAAM,MAAM,GAAG,OAAO,CAAC,MAAM,IAAI,KAAK,CAAA;IACtC,MAAM,GAAG,GAAG,GAAG,OAAO,GAAG,IAAI,EAAE,CAAA;IAE/B,MAAM,OAAO,GAA2B;QACtC,aAAa,EAAE,UAAU,KAAK,EAAE;QAChC,cAAc,EAAE,kBAAkB;QAClC,GAAG,CAAC,UAAU,CAAC,CAAC,CAAC,EAAE,oBAAoB,EAAE,UAAU,EAAE,CAAC,CAAC,CAAC,EAAE,CAAC;QAC3D,GAAG,CAAC,OAAO,CAAC,OAAO,IAAI,EAAE,CAAC;KAC3B,CAAA;IAED,IAAI,CAAC;QACH,MAAM,SAAS,GAAgB;YAC7B,MAAM;YACN,OAAO;YACP,MAAM,EAAE,WAAW,CAAC,OAAO,CAAC,OAAO,CAAC,SAAS,IAAI,kBAAkB,CAAC;SACrE,CAAA;QACD,IAAI,OAAO,CAAC,IAAI,KAAK,SAAS,EAAE,CAAC;YAC/B,SAAS,CAAC,IAAI,GAAG,IAAI,CAAC,SAAS,CAAC,OAAO,CAAC,IAAI,CAAC,CAAA;QAC/C,CAAC;QAED,MAAM,QAAQ,GAAG,MAAM,KAAK,CAAC,GAAG,EAAE,SAAS,CAAC,CAAA;QAC5C,MAAM,IAAI,GAAG,CAAC,MAAM,QAAQ,CAAC,IAAI,EAAE,CAA2B,CAAA;QAE9D,IAAI,CAAC,QAAQ,CAAC,EAAE,EAAE,CAAC;YACjB,OAAO;gBACL,EAAE,EAAE,KAAK;gBACT,MAAM,EAAE,QAAQ,CAAC,MAAM;gBACvB,IAAI,EAAE,IAAI;gBACV,KAAK,EACF,IAAgC,EAAE,KAAe;oBAClD,QAAQ,QAAQ,CAAC,MAAM,EAAE;aAC5B,CAAA;QACH,CAAC;QAED,OAAO,EAAE,EAAE,EAAE,IAAI,EAAE,MAAM,EAAE,QAAQ,CAAC,MAAM,EAAE,IAAI,EAAE,KAAK,EAAE,IAAI,EAAE,CAAA;IACjE,CAAC;IAAC,OAAO,GAAG,EAAE,CAAC;QACb,OAAO;YACL,EAAE,EAAE,KAAK;YACT,MAAM,EAAE,CAAC;YACT,IAAI,EAAE,IAAI;YACV,KAAK,EAAE,GAAG,YAAY,KAAK,CAAC,CAAC,CAAC,GAAG,CAAC,OAAO,CAAC,CAAC,CAAC,qBAAqB;SAClE,CAAA;IACH,CAAC;AACH,CAAC;AAED;;GAEG;AACH,MAAM,CAAC,KAAK,UAAU,SAAS,CAC7B,IAAY,EACZ,IAAa;IAEb,OAAO,QAAQ,CAAI,IAAI,EAAE,EAAE,MAAM,EAAE,MAAM,EAAE,IAAI,EAAE,CAAC,CAAA;AACpD,CAAC;AAED;;GAEG;AACH,MAAM,CAAC,KAAK,UAAU,QAAQ,CAC5B,IAAY;IAEZ,OAAO,QAAQ,CAAI,IAAI,EAAE,EAAE,MAAM,EAAE,KAAK,EAAE,CAAC,CAAA;AAC7C,CAAC"}
+{"version":3,"file":"nexus-api.js","sourceRoot":"","sources":["../src/nexus-api.ts"],"names":[],"mappings":"AAAA;;;;;;GAMG;AAEH,OAAO,EAAE,QAAQ,EAAE,MAAM,SAAS,CAAA;AAClC,OAAO,EAAE,YAAY,EAAE,MAAM,iBAAiB,CAAA;AAE9C,IAAI,QAAQ,GAAkB,IAAI,CAAA;AAClC,IAAI,MAAM,GAAkB,IAAI,CAAA;AAChC,IAAI,UAAU,GAAkB,IAAI,CAAA;AACpC,IAAI,YAAY,GAAkB,IAAI,CAAA;AAEtC,0EAA0E;AAC1E,IAAI,CAAC;IACH,UAAU,GAAG,YAAY,EAAE,CAAA;AAC7B,CAAC;AAAC,MAAM,CAAC;IACP,0DAA0D;AAC5D,CAAC;AACD,IAAI,CAAC;IACH,YAAY,GAAG,QAAQ,EAAE,CAAA;AAC3B,CAAC;AAAC,MAAM,CAAC;IACP,kCAAkC;AACpC,CAAC;AAED;;;GAGG;AACH,MAAM,UAAU,cAAc;IAC5B,QAAQ,GAAG,IAAI,CAAA;IACf,MAAM,GAAG,IAAI,CAAA;IACb,UAAU,GAAG,YAAY,EAAE,CAAA;AAC7B,CAAC;AAED,SAAS,SAAS;IAChB,IAAI,QAAQ,IAAI,MAAM;QAAE,OAAO,EAAE,OAAO,EAAE,QAAQ,EAAE,KAAK,EAAE,MAAM,EAAE,CAAA;IAEnE,MAAM,GAAG,GAAG,OAAO,CAAC,GAAG,CAAC,aAAa,IAAI,OAAO,CAAC,GAAG,CAAC,SAAS,CAAA;IAC9D,MAAM,KAAK,GAAG,OAAO,CAAC,GAAG,CAAC,mBAAmB,CAAA;IAE7C,IAAI,CAAC,GAAG,EAAE,CAAC;QACT,MAAM,IAAI,KAAK,CACb,mGAAmG,CACpG,CAAA;IACH,CAAC;IACD,IAAI,CAAC,KAAK,EAAE,CAAC;QACX,MAAM,IAAI,KAAK,CACb,oFAAoF,CACrF,CAAA;IACH,CAAC;IAED,kCAAkC;IAClC,QAAQ,GAAG,GAAG,CAAC,OAAO,CAAC,KAAK,EAAE,EAAE,CAAC,CAAA;IACjC,MAAM,GAAG,KAAK,CAAC,IAAI,EAAE,CAAA;IAErB,OAAO,EAAE,OAAO,EAAE,QAAS,EAAE,KAAK,EAAE,MAAO,EAAE,CAAA;AAC/C,CAAC;AASD,yDAAyD;AACzD,MAAM,kBAAkB,GAAG,MAAM,CAAA;AAEjC;;GAEG;AACH,MAAM,CAAC,KAAK,UAAU,QAAQ,CAC5B,IAAY,EACZ,UAKI,EAAE;IAEN,MAAM,EAAE,OAAO,EAAE,KAAK,EAAE,GAAG,SAAS,EAAE,CAAA;IACtC,MAAM,MAAM,GAAG,OAAO,CAAC,MAAM,IAAI,KAAK,CAAA;IACtC,MAAM,GAAG,GAAG,GAAG,OAAO,GAAG,IAAI,EAAE,CAAA;IAE/B,MAAM,OAAO,GAA2B;QACtC,aAAa,EAAE,UAAU,KAAK,EAAE;QAChC,cAAc,EAAE,kBAAkB;QAClC,GAAG,CAAC,UAAU,CAAC,CAAC,CAAC,EAAE,oBAAoB,EAAE,UAAU,EAAE,CAAC,CAAC,CAAC,EAAE,CAAC;QAC3D,GAAG,CAAC,YAAY,CAAC,CAAC,CAAC,EAAE,sBAAsB,EAAE,YAAY,EAAE,CAAC,CAAC,CAAC,EAAE,CAAC;QACjE,GAAG,CAAC,OAAO,CAAC,OAAO,IAAI,EAAE,CAAC;KAC3B,CAAA;IAED,IAAI,CAAC;QACH,MAAM,SAAS,GAAgB;YAC7B,MAAM;YACN,OAAO;YACP,MAAM,EAAE,WAAW,CAAC,OAAO,CAAC,OAAO,CAAC,SAAS,IAAI,kBAAkB,CAAC;SACrE,CAAA;QACD,IAAI,OAAO,CAAC,IAAI,KAAK,SAAS,EAAE,CAAC;YAC/B,SAAS,CAAC,IAAI,GAAG,IAAI,CAAC,SAAS,CAAC,OAAO,CAAC,IAAI,CAAC,CAAA;QAC/C,CAAC;QAED,MAAM,QAAQ,GAAG,MAAM,KAAK,CAAC,GAAG,EAAE,SAAS,CAAC,CAAA;QAC5C,MAAM,IAAI,GAAG,CAAC,MAAM,QAAQ,CAAC,IAAI,EAAE,CAA2B,CAAA;QAE9D,IAAI,CAAC,QAAQ,CAAC,EAAE,EAAE,CAAC;YACjB,OAAO;gBACL,EAAE,EAAE,KAAK;gBACT,MAAM,EAAE,QAAQ,CAAC,MAAM;gBACvB,IAAI,EAAE,IAAI;gBACV,KAAK,EACF,IAAgC,EAAE,KAAe;oBAClD,QAAQ,QAAQ,CAAC,MAAM,EAAE;aAC5B,CAAA;QACH,CAAC;QAED,OAAO,EAAE,EAAE,EAAE,IAAI,EAAE,MAAM,EAAE,QAAQ,CAAC,MAAM,EAAE,IAAI,EAAE,KAAK,EAAE,IAAI,EAAE,CAAA;IACjE,CAAC;IAAC,OAAO,GAAG,EAAE,CAAC;QACb,OAAO;YACL,EAAE,EAAE,KAAK;YACT,MAAM,EAAE,CAAC;YACT,IAAI,EAAE,IAAI;YACV,KAAK,EAAE,GAAG,YAAY,KAAK,CAAC,CAAC,CAAC,GAAG,CAAC,OAAO,CAAC,CAAC,CAAC,qBAAqB;SAClE,CAAA;IACH,CAAC;AACH,CAAC;AAED;;GAEG;AACH,MAAM,CAAC,KAAK,UAAU,SAAS,CAC7B,IAAY,EACZ,IAAa;IAEb,OAAO,QAAQ,CAAI,IAAI,EAAE,EAAE,MAAM,EAAE,MAAM,EAAE,IAAI,EAAE,CAAC,CAAA;AACpD,CAAC;AAED;;GAEG;AACH,MAAM,CAAC,KAAK,UAAU,QAAQ,CAC5B,IAAY;IAEZ,OAAO,QAAQ,CAAI,IAAI,EAAE,EAAE,MAAM,EAAE,KAAK,EAAE,CAAC,CAAA;AAC7C,CAAC"}

package/dist/server.js

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

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@gwdn/nexus-mcp",
-  "version": "0.8.6",
+  "version": "0.8.8",
   "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",
@@ -20,10 +21,16 @@
     "zod": "^4.3.6"
   },
   "devDependencies": {
+    "@types/node": "^25.9.1",
     "tsx": "^4.21.0",
     "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)
-    })
-  })
-})