@anton.andrusenko/shopify-mcp-admin 1.0.1 → 1.1.0

package/README.md

@@ -108,7 +108,7 @@ Tokens are automatically refreshed every 24 hours.
 | `PORT` | No | `3000` | HTTP server port (when `TRANSPORT=http`) |
 | `DEBUG` | No | — | Enable debug logging (`1` or `true`) |
 | `LOG_LEVEL` | No | `info` | Log level: `debug`, `info`, `warn`, `error` |
-| `SHOPIFY_MCP_LAZY_LOADING` | No | `true` | Enable modular lazy loading (set to `false` for all tools at startup) |
+| `SHOPIFY_MCP_LAZY_LOADING` | No | `false` | Enable modular lazy loading (set to `true` for on-demand module loading) |
 | `SHOPIFY_MCP_ROLE` | No | — | Role preset for automatic module loading (see [Role Presets](#-role-presets)) |
 
 **⚡ Authentication:** Provide EITHER `SHOPIFY_ACCESS_TOKEN` (legacy) OR both `SHOPIFY_CLIENT_ID` and `SHOPIFY_CLIENT_SECRET` (Dev Dashboard).
@@ -832,7 +832,7 @@ export SHOPIFY_MCP_LAZY_LOADING=false
 |-------|----------|
 | Tool not found after upgrade | Load the appropriate module: `load-module(module: "collections")` |
 | "Module not found" error | Use `list-modules` to see valid module names |
-| Want all tools at startup | Set `SHOPIFY_MCP_LAZY_LOADING=false` |
+| Want modular lazy loading | Set `SHOPIFY_MCP_LAZY_LOADING=true` (requires MCP client that supports tool list refresh) |
 | AI agent loading too many modules | Use a role preset instead of on-demand loading |
 
 ---

package/dist/index.js

@@ -27,9 +27,11 @@ var configSchema = z.object({
   // Default: 5 minutes (300000ms) - configurable for performance tuning
   STORE_INFO_CACHE_TTL_MS: z.string().optional().default("300000").transform(Number).describe("Cache TTL for store info in milliseconds (default: 5 minutes)"),
   // Lazy loading configuration (Epic 12)
-  // Default: true - enables module-based lazy loading for optimal AI performance
-  // Set to 'false' or '0' to load all tools at startup (legacy mode)
-  SHOPIFY_MCP_LAZY_LOADING: z.string().optional().default("true").transform((val) => val === "true" || val === "1").describe("Enable lazy loading of tools via modules (default: true)"),
+  // Default: false - loads all tools at startup for maximum compatibility
+  // Note: Claude Desktop doesn't support dynamic tool refresh via notifications,
+  // so lazy loading is disabled by default. Set to 'true' for clients that
+  // properly handle notifications/tools/list_changed.
+  SHOPIFY_MCP_LAZY_LOADING: z.string().optional().default("false").transform((val) => val === "true" || val === "1").describe("Enable lazy loading of tools via modules (default: false)"),
   // Role preset configuration (Story 12.3: Progressive Loading)
   // Automatically loads appropriate modules at startup based on role
   // Valid roles: inventory-manager, product-manager, content-manager,
@@ -3199,20 +3201,42 @@ function getRegisteredTools() {
     annotations: tool.annotations
   }));
 }
+function getAllRegisteredToolNames() {
+  return Array.from(registeredTools.keys());
+}
 function getToolByName(name) {
   return registeredTools.get(name);
 }
 function getToolNames() {
   return Array.from(registeredTools.keys());
 }
+var toolListChangedCallback = null;
+function setToolListChangedCallback(callback) {
+  toolListChangedCallback = callback;
+  log.debug("Tool list changed callback registered");
+}
+function notifyToolListChanged() {
+  if (toolListChangedCallback) {
+    log.info("\u{1F514} SENDING tools/list_changed notification to client");
+    toolListChangedCallback();
+  } else {
+    log.warn("\u26A0\uFE0F notifyToolListChanged called but no callback registered!");
+  }
+}
 function enableTool(name) {
   const tool = registeredTools.get(name);
   if (!tool) {
     log.debug(`Cannot enable tool: ${name} not found`);
     return false;
   }
+  const previousState = toolEnabledState.get(name);
+  const wasEnabled = previousState ?? true;
   toolEnabledState.set(name, true);
-  log.debug(`Tool enabled: ${name}`);
+  log.debug(`Tool enabled: ${name} (was: ${previousState}, wasEnabled: ${wasEnabled})`);
+  if (!wasEnabled) {
+    log.info(`\u{1F4E2} Tool ${name} state changed from disabled to enabled, triggering notification`);
+    notifyToolListChanged();
+  }
   return true;
 }
 function disableTool(name) {
@@ -3221,8 +3245,12 @@ function disableTool(name) {
     log.debug(`Cannot disable tool: ${name} not found`);
     return false;
   }
+  const wasEnabled = toolEnabledState.get(name) ?? true;
   toolEnabledState.set(name, false);
   log.debug(`Tool disabled: ${name}`);
+  if (wasEnabled) {
+    notifyToolListChanged();
+  }
   return true;
 }
 function isToolEnabled(name) {
@@ -14185,6 +14213,10 @@ function getSuggestionsForTool(toolName) {
 
 // src/tools/index.ts
 function setupToolHandlers(server) {
+  setToolListChangedCallback(() => {
+    server.sendToolListChanged();
+    log.debug("Sent tools/list_changed notification to clients");
+  });
   server.server.setRequestHandler(ListToolsRequestSchema, async () => {
     const tools = getRegisteredTools();
     log.debug(`tools/list returning ${tools.length} tools`);
@@ -14208,16 +14240,35 @@ function setupToolHandlers(server) {
     }
     const tool = getToolByName(name);
     if (!tool) {
-      const availableTools = getRegisteredTools();
-      const toolList = availableTools.length > 0 ? `Available tools: ${availableTools.map((t) => t.name).join(", ")}` : "No tools are currently registered.";
-      log.warn(`Tool not found: ${name}${name !== rawName ? ` (received as "${rawName}")` : ""}`);
+      const allTools = getAllRegisteredToolNames();
+      const enabledTools = getRegisteredTools();
+      const totalRegistered = allTools.length;
+      const totalEnabled = enabledTools.length;
+      const similarTools = allTools.filter(
+        (t) => t.toLowerCase().includes(name.toLowerCase()) || name.toLowerCase().includes(t.toLowerCase())
+      );
+      let diagnosticInfo = `
+
+Diagnostics:
+- Total tools registered: ${totalRegistered}
+- Tools currently enabled: ${totalEnabled}`;
+      if (similarTools.length > 0) {
+        diagnosticInfo += `
+- Similar tool names found: ${similarTools.join(", ")}`;
+      }
+      const toolList = enabledTools.length > 0 ? `
+
+Enabled tools: ${enabledTools.map((t) => t.name).join(", ")}` : "\n\nNo tools are currently enabled. Try restarting your MCP client.";
+      log.warn(
+        `Tool not found: ${name}${name !== rawName ? ` (received as "${rawName}")` : ""}. Registered: ${totalRegistered}, Enabled: ${totalEnabled}`
+      );
       return {
         content: [
           {
             type: "text",
-            text: `Error: Tool "${name}" not found${name !== rawName ? ` (received as "${rawName}")` : ""}.
+            text: `Error: Tool "${name}" not found${name !== rawName ? ` (received as "${rawName}")` : ""}.${diagnosticInfo}
 
-Suggestion: Use tools/list to see available tools. ${toolList}`
+Suggestion: Use tools/list to refresh available tools. If this tool should exist, try restarting your MCP client (Claude Desktop).${toolList}`
           }
         ],
         isError: true
@@ -14375,8 +14426,12 @@ function registerAllTools(server) {
         `Lazy loading enabled: ${enabledCount} core tools active, ${totalToolCount - enabledCount} tools available via modules`
       );
     }
+    log.debug(
+      `Enabled tools: ${getRegisteredTools().map((t) => t.name).join(", ")}`
+    );
   } else {
     log.info(`Lazy loading disabled (legacy mode): ${totalToolCount} tools registered`);
+    log.debug(`All tools: ${getToolNames().join(", ")}`);
   }
 }
 function applyLazyLoading() {

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@anton.andrusenko/shopify-mcp-admin",
-  "version": "1.0.1",
+  "version": "1.1.0",
   "description": "MCP server for Shopify Admin API - enables AI agents to manage Shopify stores with 71 tools for products, inventory, collections, content, SEO, metafields, markets & translations",
   "type": "module",
   "main": "./dist/index.js",
@@ -24,6 +24,10 @@
     "inspect": "npx @modelcontextprotocol/inspector node dist/index.js",
     "inspect:dev": "npx @modelcontextprotocol/inspector npx tsx src/index.ts",
     "inspect:config": "npx @modelcontextprotocol/inspector --config mcp-inspector.json",
+    "fast-agent": "fast-agent",
+    "fast-agent:check": "fast-agent check",
+    "fast-agent:chat": "fast-agent go --servers shopify-mcp-admin",
+    "fast-agent:dev": "fast-agent go --servers shopify-mcp-admin-dev",
     "ci": "npm run lint && npm run typecheck && npm run test && npm run build",
     "version:patch": "npm version patch --no-git-tag-version",
     "version:minor": "npm version minor --no-git-tag-version",