viewgate-mcp 1.0.45 → 1.0.47

package/dist/index.js

@@ -27,9 +27,10 @@ const __filename = fileURLToPath(import.meta.url);
 const __dirname = path.dirname(__filename);
 dotenv.config({ path: path.join(__dirname, "..", ".env") });
 const port = process.env.PORT || 3000;
-const BACKEND_URL = process.env.BACKEND_URL || "https://view-gate.vercel.app";
+const BACKEND_URL = process.env.VIEWGATE_BACKEND_URL || "https://view-gate.vercel.app";
 console.error(`[MCP Config] BACKEND_URL: ${BACKEND_URL}`);
-console.error(`[MCP Config] API_KEY Prefix: ${process.env.API_KEY?.substring(0, 5)}...`);
+const LOG_API_KEY = (process.env.VIEWGATE_API_KEY || process.env.API_KEY || "");
+console.error(`[MCP Config] API_KEY Prefix: ${LOG_API_KEY.substring(0, 5)}...`);
 const agentId = `${os.hostname()}-${process.pid}`;
 console.error(`[MCP Identity] Agent ID: ${agentId}`);
 // Store active sessions for SSE: sessionId -> { server, transport }
@@ -75,8 +76,8 @@ function createMcpServer(apiKey, personalKey) {
             case "planning":
                 return "planning";
             case "sync_endpoints":
-            case "get_synced_endpoints":
                 return "sync";
+            case "get_synced_endpoints":
             case "get_ai_resolved_tickets":
                 return "idle";
             default:
@@ -89,15 +90,14 @@ function createMcpServer(apiKey, personalKey) {
             resetGuard();
         }
         const desiredFlow = getToolFlow(toolName);
-        if (toolName === "get_ai_resolved_tickets") {
-            if (guard.flow !== "idle") {
-                throw new Error("TOOL_CALL_BLOCKED: tool not allowed in active flow");
-            }
+        // Neutral tools (idle flow) are always allowed and don't interrupt active flows
+        if (desiredFlow === "idle") {
             guard.lastTool = toolName;
             guard.lastActivityAt = now;
             return;
         }
-        if (guard.flow === "idle") {
+        if (guard.flow === "idle" || desiredFlow !== guard.flow) {
+            // Implicit flow reset if we are starting a new flow with a "start tool"
             switch (toolName) {
                 case "get_ui_components":
                     guard.flow = "ui_components";
@@ -123,16 +123,18 @@ function createMcpServer(apiKey, personalKey) {
                     guard.step = 1;
                     break;
                 default:
-                    throw new Error("TOOL_CALL_BLOCKED: tool not allowed in idle");
+                    if (guard.flow === "idle") {
+                        throw new Error("TOOL_CALL_BLOCKED: tool not allowed in idle");
+                    }
+                    else {
+                        throw new Error("TOOL_CALL_BLOCKED: tool not allowed in active flow");
+                    }
             }
             guard.startedAt = now;
             guard.lastTool = toolName;
             guard.lastActivityAt = now;
             return;
         }
-        if (desiredFlow !== guard.flow) {
-            throw new Error("TOOL_CALL_BLOCKED: tool not allowed in active flow");
-        }
         if (guard.flow === "ui_components") {
             if (toolName === "get_ui_components") {
                 if (guard.step > 2)
@@ -155,18 +157,17 @@ function createMcpServer(apiKey, personalKey) {
         }
         else if (guard.flow === "annotations") {
             if (toolName === "get_annotations") {
-                if (guard.step > 2)
-                    throw new Error("TOOL_CALL_BLOCKED: cannot restart flow at this step");
+                // Allow re-fetching at any time during the flow
                 guard.step = 1;
             }
             else if (toolName === "mark_annotation_ready") {
                 if (guard.step !== 1)
                     throw new Error("TOOL_CALL_BLOCKED: unexpected step");
-                guard.step = 2;
+                // Stop here and reset flow as per USER_REQUEST (MCP only reaches 'applied')
+                resetGuard();
             }
             else if (toolName === "mark_annotations_as_live") {
-                if (guard.step !== 2)
-                    throw new Error("TOOL_CALL_BLOCKED: unexpected step");
+                // Optional step if the model decides to use it, but no longer part of the required chain
                 resetGuard();
             }
             else {
@@ -175,18 +176,14 @@ function createMcpServer(apiKey, personalKey) {
         }
         else if (guard.flow === "ui_improvements") {
             if (toolName === "get_ui_improvements") {
-                if (guard.step > 2)
-                    throw new Error("TOOL_CALL_BLOCKED: cannot restart flow at this step");
                 guard.step = 1;
             }
             else if (toolName === "mark_annotation_ready") {
                 if (guard.step !== 1)
                     throw new Error("TOOL_CALL_BLOCKED: unexpected step");
-                guard.step = 2;
+                resetGuard();
             }
             else if (toolName === "mark_annotations_as_live") {
-                if (guard.step !== 2)
-                    throw new Error("TOOL_CALL_BLOCKED: unexpected step");
                 resetGuard();
             }
             else {
@@ -216,8 +213,7 @@ function createMcpServer(apiKey, personalKey) {
                 guard.step = 2;
             }
             else if (toolName === "get_synced_endpoints") {
-                if (guard.step !== 2)
-                    throw new Error("TOOL_CALL_BLOCKED: unexpected step");
+                // Actually get_synced_endpoints is now neutral, but if called as part of flow, reset.
                 resetGuard();
             }
             else {
@@ -758,7 +754,7 @@ Lang: ${rawData.preferredLanguage === 'es' ? 'ES' : 'EN'}
 const useSSE = process.argv.includes("--sse") || process.env.MCP_TRANSPORT === "sse";
 if (!useSSE) {
     const apiKey = process.env.VIEWGATE_API_KEY || process.env.API_KEY || "";
-    const personalKey = process.env.VIEWGATE_PERSONAL_KEY || "";
+    const personalKey = process.env.VIEWGATE_PERSONAL_KEY || process.env.PERSONAL_KEY || "";
     if (!apiKey) {
         console.error("Error: VIEWGATE_API_KEY environment variable is required.");
         process.exit(1);

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "viewgate-mcp",
-  "version": "1.0.45",
+  "version": "1.0.47",
   "main": "dist/index.js",
   "bin": {
     "viewgate-mcp": "./dist/index.js"