@skillsmith/mcp-server 0.4.11 → 0.4.13

package/dist/src/validation.test.js

@@ -0,0 +1,102 @@
+/**
+ * @fileoverview Unit tests for `safeParseOrError` helper.
+ * @see SMI-4313: MCP tool-dispatch safeParse envelope refactor
+ */
+import { describe, expect, it } from 'vitest';
+import { z } from 'zod';
+import { safeParseOrError } from './validation.js';
+const demoSchema = z.object({
+    skillId: z.string().min(1),
+    limit: z.number().int().positive().optional(),
+    mode: z.enum(['fast', 'thorough']).optional(),
+});
+describe('safeParseOrError', () => {
+    it('returns { ok: true, data } for valid input', () => {
+        const result = safeParseOrError(demoSchema, { skillId: 'owner/repo', limit: 10 }, 'demo_tool');
+        expect(result.ok).toBe(true);
+        if (result.ok) {
+            expect(result.data).toEqual({ skillId: 'owner/repo', limit: 10 });
+        }
+    });
+    it('returns { ok: false, response } for missing required field', () => {
+        const result = safeParseOrError(demoSchema, {}, 'demo_tool');
+        expect(result.ok).toBe(false);
+        if (!result.ok) {
+            expect(result.response.isError).toBe(true);
+            expect(result.response.content).toHaveLength(1);
+            const textEntry = result.response.content[0];
+            expect(textEntry.type).toBe('text');
+            const body = JSON.parse(textEntry.text);
+            expect(body.error).toBe('ValidationError');
+            expect(body.tool).toBe('demo_tool');
+            expect(body.issues.length).toBeGreaterThan(0);
+            const skillIdIssue = body.issues.find((issue) => issue.path === 'skillId');
+            expect(skillIdIssue).toBeDefined();
+            expect(skillIdIssue?.code).toBeTruthy();
+            expect(skillIdIssue?.message).toBeTruthy();
+        }
+    });
+    it('returns { ok: false, response } for wrong type', () => {
+        const result = safeParseOrError(demoSchema, { skillId: 'x', limit: 'abc' }, 'demo_tool');
+        expect(result.ok).toBe(false);
+        if (!result.ok) {
+            const body = JSON.parse(result.response.content[0].text);
+            expect(body.issues.some((issue) => issue.path === 'limit')).toBe(true);
+        }
+    });
+    it('returns { ok: false, response } for invalid enum value', () => {
+        const result = safeParseOrError(demoSchema, { skillId: 'x', mode: 'bogus' }, 'demo_tool');
+        expect(result.ok).toBe(false);
+        if (!result.ok) {
+            const body = JSON.parse(result.response.content[0].text);
+            expect(body.issues.some((issue) => issue.path === 'mode')).toBe(true);
+        }
+    });
+    it('joins nested path segments with "." (zod path array → dotted string)', () => {
+        const nested = z.object({
+            outer: z.object({
+                inner: z.string(),
+            }),
+        });
+        const result = safeParseOrError(nested, { outer: {} }, 'nested_tool');
+        expect(result.ok).toBe(false);
+        if (!result.ok) {
+            const body = JSON.parse(result.response.content[0].text);
+            expect(body.issues[0]?.path).toBe('outer.inner');
+        }
+    });
+    it('threads the toolName through unchanged', () => {
+        const result = safeParseOrError(demoSchema, {}, 'uninstall_skill');
+        expect(result.ok).toBe(false);
+        if (!result.ok) {
+            const body = JSON.parse(result.response.content[0].text);
+            expect(body.tool).toBe('uninstall_skill');
+        }
+    });
+    it('matches the documented envelope shape (inline snapshot)', () => {
+        const result = safeParseOrError(demoSchema, {}, 'demo_tool');
+        expect(result.ok).toBe(false);
+        if (!result.ok) {
+            const body = JSON.parse(result.response.content[0].text);
+            // Guard against silent drift back to stringified ZodError.message.
+            expect(body).toMatchInlineSnapshot({
+                issues: expect.any(Array),
+            }, `
+        {
+          "error": "ValidationError",
+          "issues": Any<Array>,
+          "tool": "demo_tool",
+        }
+      `);
+            // Each issue must have the documented keys
+            for (const issue of body.issues) {
+                expect(issue).toEqual(expect.objectContaining({
+                    path: expect.any(String),
+                    message: expect.any(String),
+                    code: expect.any(String),
+                }));
+            }
+        }
+    });
+});
+//# sourceMappingURL=validation.test.js.map

package/dist/src/validation.test.js.map

@@ -0,0 +1 @@
+{"version":3,"file":"validation.test.js","sourceRoot":"","sources":["../../src/validation.test.ts"],"names":[],"mappings":"AAAA;;;GAGG;AAEH,OAAO,EAAE,QAAQ,EAAE,MAAM,EAAE,EAAE,EAAE,MAAM,QAAQ,CAAA;AAC7C,OAAO,EAAE,CAAC,EAAE,MAAM,KAAK,CAAA;AACvB,OAAO,EAAE,gBAAgB,EAAE,MAAM,iBAAiB,CAAA;AAElD,MAAM,UAAU,GAAG,CAAC,CAAC,MAAM,CAAC;IAC1B,OAAO,EAAE,CAAC,CAAC,MAAM,EAAE,CAAC,GAAG,CAAC,CAAC,CAAC;IAC1B,KAAK,EAAE,CAAC,CAAC,MAAM,EAAE,CAAC,GAAG,EAAE,CAAC,QAAQ,EAAE,CAAC,QAAQ,EAAE;IAC7C,IAAI,EAAE,CAAC,CAAC,IAAI,CAAC,CAAC,MAAM,EAAE,UAAU,CAAC,CAAC,CAAC,QAAQ,EAAE;CAC9C,CAAC,CAAA;AAEF,QAAQ,CAAC,kBAAkB,EAAE,GAAG,EAAE;IAChC,EAAE,CAAC,4CAA4C,EAAE,GAAG,EAAE;QACpD,MAAM,MAAM,GAAG,gBAAgB,CAAC,UAAU,EAAE,EAAE,OAAO,EAAE,YAAY,EAAE,KAAK,EAAE,EAAE,EAAE,EAAE,WAAW,CAAC,CAAA;QAE9F,MAAM,CAAC,MAAM,CAAC,EAAE,CAAC,CAAC,IAAI,CAAC,IAAI,CAAC,CAAA;QAC5B,IAAI,MAAM,CAAC,EAAE,EAAE,CAAC;YACd,MAAM,CAAC,MAAM,CAAC,IAAI,CAAC,CAAC,OAAO,CAAC,EAAE,OAAO,EAAE,YAAY,EAAE,KAAK,EAAE,EAAE,EAAE,CAAC,CAAA;QACnE,CAAC;IACH,CAAC,CAAC,CAAA;IAEF,EAAE,CAAC,4DAA4D,EAAE,GAAG,EAAE;QACpE,MAAM,MAAM,GAAG,gBAAgB,CAAC,UAAU,EAAE,EAAE,EAAE,WAAW,CAAC,CAAA;QAE5D,MAAM,CAAC,MAAM,CAAC,EAAE,CAAC,CAAC,IAAI,CAAC,KAAK,CAAC,CAAA;QAC7B,IAAI,CAAC,MAAM,CAAC,EAAE,EAAE,CAAC;YACf,MAAM,CAAC,MAAM,CAAC,QAAQ,CAAC,OAAO,CAAC,CAAC,IAAI,CAAC,IAAI,CAAC,CAAA;YAC1C,MAAM,CAAC,MAAM,CAAC,QAAQ,CAAC,OAAO,CAAC,CAAC,YAAY,CAAC,CAAC,CAAC,CAAA;YAC/C,MAAM,SAAS,GAAG,MAAM,CAAC,QAAQ,CAAC,OAAO,CAAC,CAAC,CAAmC,CAAA;YAC9E,MAAM,CAAC,SAAS,CAAC,IAAI,CAAC,CAAC,IAAI,CAAC,MAAM,CAAC,CAAA;YACnC,MAAM,IAAI,GAAG,IAAI,CAAC,KAAK,CAAC,SAAS,CAAC,IAAI,CAIrC,CAAA;YACD,MAAM,CAAC,IAAI,CAAC,KAAK,CAAC,CAAC,IAAI,CAAC,iBAAiB,CAAC,CAAA;YAC1C,MAAM,CAAC,IAAI,CAAC,IAAI,CAAC,CAAC,IAAI,CAAC,WAAW,CAAC,CAAA;YACnC,MAAM,CAAC,IAAI,CAAC,MAAM,CAAC,MAAM,CAAC,CAAC,eAAe,CAAC,CAAC,CAAC,CAAA;YAC7C,MAAM,YAAY,GAAG,IAAI,CAAC,MAAM,CAAC,IAAI,CAAC,CAAC,KAAK,EAAE,EAAE,CAAC,KAAK,CAAC,IAAI,KAAK,SAAS,CAAC,CAAA;YAC1E,MAAM,CAAC,YAAY,CAAC,CAAC,WAAW,EAAE,CAAA;YAClC,MAAM,CAAC,YAAY,EAAE,IAAI,CAAC,CAAC,UAAU,EAAE,CAAA;YACvC,MAAM,CAAC,YAAY,EAAE,OAAO,CAAC,CAAC,UAAU,EAAE,CAAA;QAC5C,CAAC;IACH,CAAC,CAAC,CAAA;IAEF,EAAE,CAAC,gDAAgD,EAAE,GAAG,EAAE;QACxD,MAAM,MAAM,GAAG,gBAAgB,CAAC,UAAU,EAAE,EAAE,OAAO,EAAE,GAAG,EAAE,KAAK,EAAE,KAAK,EAAE,EAAE,WAAW,CAAC,CAAA;QAExF,MAAM,CAAC,MAAM,CAAC,EAAE,CAAC,CAAC,IAAI,CAAC,KAAK,CAAC,CAAA;QAC7B,IAAI,CAAC,MAAM,CAAC,EAAE,EAAE,CAAC;YACf,MAAM,IAAI,GAAG,IAAI,CAAC,KAAK,CAAE,MAAM,CAAC,QAAQ,CAAC,OAAO,CAAC,CAAC,CAAsB,CAAC,IAAI,CAE5E,CAAA;YACD,MAAM,CAAC,IAAI,CAAC,MAAM,CAAC,IAAI,CAAC,CAAC,KAAK,EAAE,EAAE,CAAC,KAAK,CAAC,IAAI,KAAK,OAAO,CAAC,CAAC,CAAC,IAAI,CAAC,IAAI,CAAC,CAAA;QACxE,CAAC;IACH,CAAC,CAAC,CAAA;IAEF,EAAE,CAAC,wDAAwD,EAAE,GAAG,EAAE;QAChE,MAAM,MAAM,GAAG,gBAAgB,CAAC,UAAU,EAAE,EAAE,OAAO,EAAE,GAAG,EAAE,IAAI,EAAE,OAAO,EAAE,EAAE,WAAW,CAAC,CAAA;QAEzF,MAAM,CAAC,MAAM,CAAC,EAAE,CAAC,CAAC,IAAI,CAAC,KAAK,CAAC,CAAA;QAC7B,IAAI,CAAC,MAAM,CAAC,EAAE,EAAE,CAAC;YACf,MAAM,IAAI,GAAG,IAAI,CAAC,KAAK,CAAE,MAAM,CAAC,QAAQ,CAAC,OAAO,CAAC,CAAC,CAAsB,CAAC,IAAI,CAE5E,CAAA;YACD,MAAM,CAAC,IAAI,CAAC,MAAM,CAAC,IAAI,CAAC,CAAC,KAAK,EAAE,EAAE,CAAC,KAAK,CAAC,IAAI,KAAK,MAAM,CAAC,CAAC,CAAC,IAAI,CAAC,IAAI,CAAC,CAAA;QACvE,CAAC;IACH,CAAC,CAAC,CAAA;IAEF,EAAE,CAAC,sEAAsE,EAAE,GAAG,EAAE;QAC9E,MAAM,MAAM,GAAG,CAAC,CAAC,MAAM,CAAC;YACtB,KAAK,EAAE,CAAC,CAAC,MAAM,CAAC;gBACd,KAAK,EAAE,CAAC,CAAC,MAAM,EAAE;aAClB,CAAC;SACH,CAAC,CAAA;QACF,MAAM,MAAM,GAAG,gBAAgB,CAAC,MAAM,EAAE,EAAE,KAAK,EAAE,EAAE,EAAE,EAAE,aAAa,CAAC,CAAA;QACrE,MAAM,CAAC,MAAM,CAAC,EAAE,CAAC,CAAC,IAAI,CAAC,KAAK,CAAC,CAAA;QAC7B,IAAI,CAAC,MAAM,CAAC,EAAE,EAAE,CAAC;YACf,MAAM,IAAI,GAAG,IAAI,CAAC,KAAK,CAAE,MAAM,CAAC,QAAQ,CAAC,OAAO,CAAC,CAAC,CAAsB,CAAC,IAAI,CAE5E,CAAA;YACD,MAAM,CAAC,IAAI,CAAC,MAAM,CAAC,CAAC,CAAC,EAAE,IAAI,CAAC,CAAC,IAAI,CAAC,aAAa,CAAC,CAAA;QAClD,CAAC;IACH,CAAC,CAAC,CAAA;IAEF,EAAE,CAAC,wCAAwC,EAAE,GAAG,EAAE;QAChD,MAAM,MAAM,GAAG,gBAAgB,CAAC,UAAU,EAAE,EAAE,EAAE,iBAAiB,CAAC,CAAA;QAClE,MAAM,CAAC,MAAM,CAAC,EAAE,CAAC,CAAC,IAAI,CAAC,KAAK,CAAC,CAAA;QAC7B,IAAI,CAAC,MAAM,CAAC,EAAE,EAAE,CAAC;YACf,MAAM,IAAI,GAAG,IAAI,CAAC,KAAK,CAAE,MAAM,CAAC,QAAQ,CAAC,OAAO,CAAC,CAAC,CAAsB,CAAC,IAAI,CAE5E,CAAA;YACD,MAAM,CAAC,IAAI,CAAC,IAAI,CAAC,CAAC,IAAI,CAAC,iBAAiB,CAAC,CAAA;QAC3C,CAAC;IACH,CAAC,CAAC,CAAA;IAEF,EAAE,CAAC,yDAAyD,EAAE,GAAG,EAAE;QACjE,MAAM,MAAM,GAAG,gBAAgB,CAAC,UAAU,EAAE,EAAE,EAAE,WAAW,CAAC,CAAA;QAC5D,MAAM,CAAC,MAAM,CAAC,EAAE,CAAC,CAAC,IAAI,CAAC,KAAK,CAAC,CAAA;QAC7B,IAAI,CAAC,MAAM,CAAC,EAAE,EAAE,CAAC;YACf,MAAM,IAAI,GAAG,IAAI,CAAC,KAAK,CAAE,MAAM,CAAC,QAAQ,CAAC,OAAO,CAAC,CAAC,CAAsB,CAAC,IAAI,CAI5E,CAAA;YACD,mEAAmE;YACnE,MAAM,CAAC,IAAI,CAAC,CAAC,qBAAqB,CAChC;gBACE,MAAM,EAAE,MAAM,CAAC,GAAG,CAAC,KAAK,CAAC;aAC1B,EACD;;;;;;OAMD,CACA,CAAA;YACD,2CAA2C;YAC3C,KAAK,MAAM,KAAK,IAAI,IAAI,CAAC,MAAM,EAAE,CAAC;gBAChC,MAAM,CAAC,KAAK,CAAC,CAAC,OAAO,CACnB,MAAM,CAAC,gBAAgB,CAAC;oBACtB,IAAI,EAAE,MAAM,CAAC,GAAG,CAAC,MAAM,CAAC;oBACxB,OAAO,EAAE,MAAM,CAAC,GAAG,CAAC,MAAM,CAAC;oBAC3B,IAAI,EAAE,MAAM,CAAC,GAAG,CAAC,MAAM,CAAC;iBACzB,CAAC,CACH,CAAA;YACH,CAAC;QACH,CAAC;IACH,CAAC,CAAC,CAAA;AACJ,CAAC,CAAC,CAAA"}

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@skillsmith/mcp-server",
-  "version": "0.4.11",
+  "version": "0.4.13",
   "mcpName": "io.github.smith-horn/skillsmith",
   "description": "MCP server for Skillsmith skill discovery",
   "type": "module",
@@ -22,7 +22,7 @@
   },
   "dependencies": {
     "@modelcontextprotocol/sdk": "^1.27.1",
-    "@skillsmith/core": "^0.5.3",
+    "@skillsmith/core": "^0.5.7",
     "esbuild": "0.27.2"
   },
   "devDependencies": {
@@ -71,6 +71,6 @@
     "tool-management"
   ],
   "engines": {
-    "node": ">=22.0.0"
+    "node": ">=22.22.0"
   }
 }

package/server.json

@@ -8,13 +8,9 @@
     "url": "https://github.com/smith-horn/skillsmith",
     "source": "github"
   },
-  "version": "0.4.11",
+  "version": "0.4.13",
   "_meta": {
-    "io.skillsmith/categories": [
-      "developer-tools",
-      "ai-agents",
-      "skill-management"
-    ],
+    "io.skillsmith/categories": ["developer-tools", "ai-agents", "skill-management"],
     "io.skillsmith/keywords": [
       "claude-code",
       "agent-skills",
@@ -27,7 +23,7 @@
     {
       "registryType": "npm",
       "identifier": "@skillsmith/mcp-server",
-      "version": "0.4.11",
+      "version": "0.4.13",
       "transport": {
         "type": "stdio"
       },