viewgate-mcp 1.0.47 → 1.0.48

package/dist/index.js

@@ -68,7 +68,6 @@ function createMcpServer(apiKey, personalKey) {
             case "mark_ui_component_generated":
                 return "ui_components";
             case "get_annotations":
-            case "mark_annotation_ready":
             case "mark_annotations_as_live":
                 return "annotations";
             case "get_ui_improvements":
@@ -124,10 +123,13 @@ function createMcpServer(apiKey, personalKey) {
                     break;
                 default:
                     if (guard.flow === "idle") {
-                        throw new Error("TOOL_CALL_BLOCKED: tool not allowed in idle");
+                        throw new Error(`TOOL_CALL_BLOCKED: tool '${toolName}' not allowed in idle. Please start a workflow with 'get_annotations', 'get_ui_components', or 'get_ui_improvements'.`);
                     }
                     else {
-                        throw new Error("TOOL_CALL_BLOCKED: tool not allowed in active flow");
+                        const nextTool = guard.flow === "annotations" ? "mark_annotations_as_live" :
+                            guard.flow === "ui_components" ? "mark_ui_component_generated" :
+                                guard.flow === "ui_improvements" ? "mark_annotations_as_live" : "the next workflow step";
+                        throw new Error(`TOOL_CALL_BLOCKED: tool '${toolName}' not allowed while in '${guard.flow}' workflow. You must call '${nextTool}' to complete the current task before starting another.`);
                     }
             }
             guard.startedAt = now;
@@ -160,16 +162,12 @@ function createMcpServer(apiKey, personalKey) {
                 // Allow re-fetching at any time during the flow
                 guard.step = 1;
             }
-            else if (toolName === "mark_annotation_ready") {
+            else if (toolName === "mark_annotations_as_live") {
                 if (guard.step !== 1)
                     throw new Error("TOOL_CALL_BLOCKED: unexpected step");
                 // Stop here and reset flow as per USER_REQUEST (MCP only reaches 'applied')
                 resetGuard();
             }
-            else if (toolName === "mark_annotations_as_live") {
-                // Optional step if the model decides to use it, but no longer part of the required chain
-                resetGuard();
-            }
             else {
                 throw new Error("TOOL_CALL_BLOCKED: tool not allowed in active flow");
             }
@@ -178,14 +176,11 @@ function createMcpServer(apiKey, personalKey) {
             if (toolName === "get_ui_improvements") {
                 guard.step = 1;
             }
-            else if (toolName === "mark_annotation_ready") {
+            else if (toolName === "mark_annotations_as_live") {
                 if (guard.step !== 1)
                     throw new Error("TOOL_CALL_BLOCKED: unexpected step");
                 resetGuard();
             }
-            else if (toolName === "mark_annotations_as_live") {
-                resetGuard();
-            }
             else {
                 throw new Error("TOOL_CALL_BLOCKED: tool not allowed in active flow");
             }
@@ -290,8 +285,8 @@ function createMcpServer(apiKey, personalKey) {
                     },
                 },
                 {
-                    name: "mark_annotation_ready",
-                    description: "Mark as ready/applied. Use internal IDs. IMPORTANT: appliedChanges must be in the project's preferredLanguage (e.g. SPANISH).",
+                    name: "mark_annotations_as_live",
+                    description: "Mark tickets (functional or UI/UX) as applied. Use this tool after fixing the code to submit your changes. IMPORTANT: appliedChanges must be in the project's preferredLanguage (e.g. SPANISH).",
                     inputSchema: {
                         type: "object",
                         properties: {
@@ -310,21 +305,6 @@ function createMcpServer(apiKey, personalKey) {
                         required: ["results"]
                     },
                 },
-                {
-                    name: "mark_annotations_as_live",
-                    description: "Mark as live (ready_for_review). IDs required.",
-                    inputSchema: {
-                        type: "object",
-                        properties: {
-                            ids: {
-                                type: "array",
-                                items: { type: "string" },
-                                description: "Internal IDs."
-                            }
-                        },
-                        required: ["ids"]
-                    },
-                },
                 {
                     name: "planning",
                     description: "Planning tool for backlog tickets. Fetch tickets or submit analysis. IMPORTANT: aiAnalysis must be in the project's preferredLanguage (e.g. SPANISH).",
@@ -608,6 +588,12 @@ Endpoints: ${ann.backendEndpoints?.length ? ann.backendEndpoints.join(', ') : 'N
 Feedback previo: ${ann.corrections && Array.isArray(ann.corrections) && ann.corrections.length > 0 ? ann.corrections[ann.corrections.length - 1].text : 'Ninguno'}
 Tarea: ${ann.message}
 Lang: ${rawData.preferredLanguage === 'es' ? 'ES' : 'EN'}
+
+### FLUJO DE TRABAJO OBLIGATORIO ###
+1. Analiza el código y el problema.
+2. Aplica la solución técnica necesaria.
+3. AL TERMINAR, DEBES LLAMAR A 'mark_annotations_as_live' con el id '${ann._id}' y un resumen de tus cambios en 'appliedChanges' (en ESPAÑOL).
+4. El estado final del ticket será 'applied'. NO intentes usar otros estados o herramientas hasta completar este paso.
 (RECUERDA: Toda tu respuesta, comentarios y análisis DEBEN estar en ESPAÑOL)`
                         };
                     });
@@ -618,10 +604,10 @@ Lang: ${rawData.preferredLanguage === 'es' ? 'ES' : 'EN'}
                         content: [{ type: "text", text: langHint + JSON.stringify({ preferredLanguage: rawData.preferredLanguage || 'en', annotations: annotationsWithTips }, null, 2) }],
                     };
                 }
-                case "mark_annotation_ready": {
+                case "mark_annotations_as_live": {
                     const args = argsAny;
                     const results = args.results;
-                    console.error(`[mark_annotation_ready] Submitting results for ${results?.length} items`);
+                    console.error(`[mark_annotations_as_live] Submitting results for ${results?.length} items as applied`);
                     const response = await fetch(`${BACKEND_URL}/api/mcp/annotations/batch-ready`, {
                         method: 'PATCH',
                         headers: {
@@ -641,27 +627,6 @@ Lang: ${rawData.preferredLanguage === 'es' ? 'ES' : 'EN'}
                         content: [{ type: "text", text: JSON.stringify(data, null, 2) }],
                     };
                 }
-                case "mark_annotations_as_live": {
-                    const args = argsAny;
-                    const ids = args.ids;
-                    console.error(`[mark_annotations_as_live] Marking ${ids?.length} items as live`);
-                    const response = await fetch(`${BACKEND_URL}/api/mcp/annotations/mark-as-live`, {
-                        method: 'POST',
-                        headers: {
-                            'Content-Type': 'application/json',
-                            'x-api-key': apiKey,
-                            'x-mcp-tool-name': toolName,
-                            'x-personal-key': personalKey || ''
-                        },
-                        body: JSON.stringify({ ids })
-                    });
-                    if (!response.ok)
-                        throw new Error(`Backend responded with ${response.status}`);
-                    const data = (await response.json());
-                    return {
-                        content: [{ type: "text", text: JSON.stringify(data, null, 2) }],
-                    };
-                }
                 case "planning": {
                     const args = argsAny;
                     const url = args.results ? `${BACKEND_URL}/api/mcp/annotations/batch-planning` : `${BACKEND_URL}/api/mcp/backlog`;
@@ -681,6 +646,7 @@ Lang: ${rawData.preferredLanguage === 'es' ? 'ES' : 'EN'}
                 }
                 case "get_ui_improvements": {
                     const args = argsAny;
+                    const agentId = personalKey || 'mcp-agent-ui-' + (apiKey?.slice(-6) || 'generic');
                     const fetchUrl = new URL(`${BACKEND_URL}/api/mcp/ui-improvements`);
                     if (args.limit)
                         fetchUrl.searchParams.append("limit", args.limit.toString());
@@ -690,6 +656,10 @@ Lang: ${rawData.preferredLanguage === 'es' ? 'ES' : 'EN'}
                         fetchUrl.searchParams.append("key", args.key);
                     if (args.ids)
                         fetchUrl.searchParams.append("ids", args.ids);
+                    // Add lock and agentId parameters to enable blocking flow
+                    fetchUrl.searchParams.append("lock", "true");
+                    fetchUrl.searchParams.append("agentId", agentId);
+                    console.error(`[get_ui_improvements] Fetching with lock: ${fetchUrl.toString()}`);
                     const response = await fetch(fetchUrl, {
                         headers: {
                             'x-api-key': apiKey,
@@ -701,11 +671,33 @@ Lang: ${rawData.preferredLanguage === 'es' ? 'ES' : 'EN'}
                         const errorBody = await response.text();
                         throw new Error(`Backend responded with ${response.status}: ${errorBody}`);
                     }
-                    const data = (await response.json());
-                    const langHint = data.preferredLanguage === 'es'
-                        ? "\n*** [INSTRUCTION: Project is in SPANISH. Provide all CSS/Visual changes in SPANISH comments if requested.] ***\n\n\n\n"
-                        : "";
-                    return { content: [{ type: "text", text: langHint + JSON.stringify(data, null, 2) }] };
+                    const rawData = (await response.json());
+                    let rawImprovements = Array.isArray(rawData) ? rawData : (rawData?.data || rawData?.improvements || []);
+                    if (!Array.isArray(rawImprovements)) {
+                        return { content: [{ type: "text", text: "Error: Invalid format from backend" }], isError: true };
+                    }
+                    const improvementsWithTips = rawImprovements.map((ann) => {
+                        return {
+                            ...ann,
+                            _ia_fix_instruction: `### INSTRUCCIONES PARA LA IA (MEJORA UI - IMPORTANTE) ###
+A continuación se presenta un ticket de MEJORA VISUAL/UX que requiere tu atención.
+DEBES RESPONDER EN ESPAÑOL.
+Proyecto: ${rawData.name || 'ViewGate'}
+Tarea: ${ann.message}
+Lang: ${rawData.preferredLanguage === 'es' ? 'ES' : 'EN'}
+
+### FLUJO DE TRABAJO OBLIGATORIO ###
+1. Analiza el diseño y los estilos CSS actuales.
+2. Aplica los cambios visuales solicitados (STRICTLY LIMITED TO VISUAL/CSS CHANGES).
+3. AL TERMINAR, DEBES LLAMAR A 'mark_annotations_as_live' con el id '${ann._id}' y un resumen de tus cambios en 'appliedChanges' (en ESPAÑOL).
+4. El estado final del ticket será 'applied'. NO intentes usar otros estados o herramientas hasta completar este paso.
+(RECUERDA: Toda tu respuesta, comentarios y análisis DEBEN estar en ESPAÑOL)`
+                        };
+                    });
+                    const langHint = rawData.preferredLanguage === 'es'
+                        ? "\n*** [INSTRUCTION: Project is in SPANISH. Provide all comments and changes in SPANISH ONLY.] ***\n\n"
+                        : (rawData.preferredLanguage === 'en' ? "\n*** [INSTRUCTION: Provide all comments and analysis in English.] ***\n\n" : "");
+                    return { content: [{ type: "text", text: langHint + JSON.stringify({ preferredLanguage: rawData.preferredLanguage || 'en', improvements: improvementsWithTips }, null, 2) }] };
                 }
                 case "sync_endpoints": {
                     const args = argsAny;

package/package.json

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