@wix/mcp 1.0.0 → 1.0.1

package/build/tool-utils.spec.js

@@ -0,0 +1,119 @@
+import { handleWixAPIResponse } from './tool-utils.js';
+import { expect, test, vi, describe, beforeEach } from 'vitest';
+// Mock the global fetch function
+global.fetch = vi.fn();
+describe('handleWixAPIResponse', () => {
+    beforeEach(() => {
+        // Reset the mock before each test
+        global.fetch.mockClear();
+    });
+    describe('safeParseJSON', () => {
+        test('should parse valid JSON string', async () => {
+            const jsonString = '{"key": "value", "number": 123}';
+            global.fetch.mockResolvedValueOnce(new Response(jsonString, { status: 200 }));
+            const result = await handleWixAPIResponse(new Response(jsonString, { status: 200 }));
+            expect(result).toEqual({ key: 'value', number: 123 });
+        });
+        test('should return the original string if parsing fails', async () => {
+            const invalidJSONString = 'this is not valid json';
+            global.fetch.mockResolvedValueOnce(new Response(invalidJSONString, { status: 200 }));
+            const result = await handleWixAPIResponse(new Response(invalidJSONString, { status: 200 }));
+            expect(result).toBe(invalidJSONString);
+        });
+        test('should return an empty string if the response text is empty', async () => {
+            global.fetch.mockResolvedValueOnce(new Response('', { status: 200 }));
+            const result = await handleWixAPIResponse(new Response('', { status: 200 }));
+            expect(result).toBe('');
+        });
+    });
+    test('should return parsed JSON data for a successful response', async () => {
+        const mockData = { message: 'Success!' };
+        global.fetch.mockResolvedValueOnce(new Response(JSON.stringify(mockData), {
+            status: 200,
+            statusText: 'OK',
+            headers: new Headers({ 'Content-Type': 'application/json' })
+        }));
+        const response = new Response(JSON.stringify(mockData), {
+            status: 200,
+            statusText: 'OK',
+            headers: new Headers({ 'Content-Type': 'application/json' })
+        });
+        const result = await handleWixAPIResponse(response);
+        expect(result).toEqual(mockData);
+    });
+    test('should return the original string if the successful response is not JSON', async () => {
+        const mockData = 'This is plain text';
+        global.fetch.mockResolvedValueOnce(new Response(mockData, { status: 200, statusText: 'OK' }));
+        const response = new Response(mockData, { status: 200, statusText: 'OK' });
+        const result = await handleWixAPIResponse(response);
+        expect(result).toBe(mockData);
+    });
+    test('should throw an error for a failed response with JSON error details', async () => {
+        const mockError = { error: 'Something went wrong', code: 500 };
+        const mockRequestId = 'some-request-id';
+        global.fetch.mockResolvedValueOnce(new Response(JSON.stringify(mockError), {
+            status: 500,
+            statusText: 'Internal Server Error',
+            headers: new Headers({ 'x-wix-request-id': mockRequestId })
+        }));
+        const response = new Response(JSON.stringify(mockError), {
+            status: 500,
+            statusText: 'Internal Server Error',
+            headers: new Headers({ 'x-wix-request-id': mockRequestId })
+        });
+        await expect(handleWixAPIResponse(response)).rejects.toThrowError([
+            'Failed to call Wix API: 500 Internal Server Error.',
+            `request id: ${mockRequestId}`,
+            '{"error":"Something went wrong","code":500}'
+        ].join('\n'));
+    });
+    test('should throw an error for a failed response with non-JSON error details', async () => {
+        const mockError = 'Internal server error occurred.';
+        global.fetch.mockResolvedValueOnce(new Response(mockError, {
+            status: 503,
+            statusText: 'Service Unavailable'
+        }));
+        const response = new Response(mockError, {
+            status: 503,
+            statusText: 'Service Unavailable'
+        });
+        await expect(handleWixAPIResponse(response)).rejects.toThrowError(['Failed to call Wix API: 503 Service Unavailable.', mockError].join('\n'));
+    });
+    test('should include the request ID in the error message if present', async () => {
+        const mockError = { message: 'Unauthorized' };
+        const mockRequestId = 'another-id';
+        global.fetch.mockResolvedValueOnce(new Response(JSON.stringify(mockError), {
+            status: 401,
+            statusText: 'Unauthorized',
+            headers: new Headers({ 'x-wix-request-id': mockRequestId })
+        }));
+        const response = new Response(JSON.stringify(mockError), {
+            status: 401,
+            statusText: 'Unauthorized',
+            headers: new Headers({ 'x-wix-request-id': mockRequestId })
+        });
+        await expect(handleWixAPIResponse(response)).rejects.toThrowError([
+            'Failed to call Wix API: 401 Unauthorized.',
+            `request id: ${mockRequestId}`,
+            '{"message":"Unauthorized"}'
+        ].join('\n'));
+    });
+    test('should return "Not found" in the error message for a 404 with HTML content', async () => {
+        const mockHtmlError = '<html><body><h1>404 Not Found</h1></body></html>';
+        global.fetch.mockResolvedValueOnce(new Response(mockHtmlError, { status: 404, statusText: 'Not Found' }));
+        const response = new Response(mockHtmlError, {
+            status: 404,
+            statusText: 'Not Found'
+        });
+        await expect(handleWixAPIResponse(response)).rejects.toThrowError(['Failed to call Wix API: 404 Not Found.', 'Not found'].join('\n'));
+    });
+    test('should include the HTML content in the error message for a 404 without typical HTML structure', async () => {
+        const mockNonHtmlError = 'Resource not found on the server.';
+        global.fetch.mockResolvedValueOnce(new Response(mockNonHtmlError, { status: 404, statusText: 'Not Found' }));
+        const response = new Response(mockNonHtmlError, {
+            status: 404,
+            statusText: 'Not Found'
+        });
+        await expect(handleWixAPIResponse(response)).rejects.toThrowError(['Failed to call Wix API: 404 Not Found.', mockNonHtmlError].join('\n'));
+    });
+});

package/build/wix-mcp-server.d.ts

@@ -0,0 +1,16 @@
+import type { z, ZodRawShape, ZodTypeAny } from 'zod';
+import type { RequestHandlerExtra } from '@modelcontextprotocol/sdk/shared/protocol.js';
+import type { CallToolResult } from '@modelcontextprotocol/sdk/types.js';
+import type { PanoramaClientForComponent } from '@wix/panorama-client-node';
+import { McpServer } from '@modelcontextprotocol/sdk/server/mcp.js';
+type WixRequestHandlerExtra = RequestHandlerExtra & {
+    panorama: PanoramaClientForComponent;
+};
+type WixToolCallback<Args extends undefined | ZodRawShape = undefined> = Args extends ZodRawShape ? (args: z.objectOutputType<Args, ZodTypeAny>, extra: WixRequestHandlerExtra) => CallToolResult | Promise<CallToolResult> : (extra: WixRequestHandlerExtra) => CallToolResult | Promise<CallToolResult>;
+export declare class WixMcpServer extends McpServer {
+    tool(name: string, cb: WixToolCallback): void;
+    tool(name: string, description: string, cb: WixToolCallback): void;
+    tool<Args extends ZodRawShape>(name: string, paramsSchema: Args, cb: WixToolCallback<Args>): void;
+    tool<Args extends ZodRawShape>(name: string, description: string, paramsSchema: Args, cb: WixToolCallback<Args>): void;
+}
+export {};

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@wix/mcp",
-  "version": "1.0.0",
+  "version": "1.0.1",
   "description": "A Model Context Protocol server for Wix AI tools",
   "type": "module",
   "bin": "./bin.js",
@@ -11,10 +11,10 @@
     "build": "tsc",
     "test": "vitest run",
     "test:watch": "vitest",
-    "start": "npm run build && node ./build/index.js",
+    "start": "npm run build && node ./build/bin.js",
     "watch": "tsc --watch",
     "lint": "eslint .",
-    "inspector": "npx @modelcontextprotocol/inspector build/index.js"
+    "inspector": "npx @modelcontextprotocol/inspector build/bin.js"
   },
   "publishConfig": {
     "registry": "https://registry.npmjs.org/",
@@ -22,32 +22,33 @@
   },
   "dependencies": {
     "@modelcontextprotocol/sdk": "^1.9.0",
-    "@sentry/node": "^9.12.0",
+    "@sentry/node": "^9.13.0",
     "@wix/panorama-client-node": "^3.227.0",
     "execa": "^9.5.2",
     "minimist": "^1.2.8",
     "strip-ansi": "^7.1.0",
     "uuid": "^11.1.0",
     "wait-for-expect": "^3.0.2",
-    "zod": "^3.24.2"
+    "zod": "^3.24.3"
   },
   "devDependencies": {
-    "@eslint/js": "^9.24.0",
+    "@eslint/js": "^9.25.0",
     "@types/express": "^5.0.1",
     "@types/minimist": "^1.2.5",
     "@types/node": "^20.17.30",
-    "eslint": "^9.24.0",
+    "eslint": "^9.25.0",
     "eslint-config-prettier": "^10.1.2",
     "eslint-plugin-prettier": "^5.2.6",
     "globals": "^16.0.0",
     "prettier": "^3.5.3",
     "typescript": "^5.8.3",
     "typescript-eslint": "^8.30.1",
-    "vitest": "^3.1.1"
+    "vitest": "^3.1.2"
   },
   "exports": {
     ".": {
-      "import": "./build/index.js"
+      "import": "./build/index.js",
+      "types": "./build/index.d.ts"
     },
     "./package.json": "./package.json"
   },
@@ -62,5 +63,5 @@
       ]
     }
   },
-  "falconPackageHash": "bf847433e48397ca724ff50a4883a0a7a5aedc5c5cb6af9048001461"
+  "falconPackageHash": "45d0dbf4ab7e282591f50831a041971ab0ea17988030cb5d68ab0d86"
 }