@ema.co/mcp-toolkit 1.5.1 → 1.6.0

package/dist/sdk/client.js

@@ -100,80 +100,22 @@ export class EmaClient {
                 const controller = new AbortController();
                 const timeoutId = setTimeout(() => controller.abort(), this.timeoutMs);
                 const fullUrl = `${this.env.baseUrl.replace(/\/$/, "")}${path}`;
-                // #region agent log
-                // H3/H4: verify wire-level request shape + size (no secrets).
-                try {
-                    if (process.env.EMA_MCP_DEBUG_LOGS !== "1") {
-                        // disabled
-                    }
-                    else {
-                        const isUpdatePersona = path.includes("/api/personas/update_persona");
-                        const jsonAny = opts?.json;
-                        const workflowAny = jsonAny?.workflow ?? jsonAny?.workflow_def;
-                        const actions = workflowAny?.actions ?? [];
-                        const actionNames = Array.isArray(actions)
-                            ? actions
-                                .slice(0, 50)
-                                .map((a) => a?.name)
-                                .filter((n) => typeof n === "string")
-                            : [];
-                        const containsHitl = actionNames.some((n) => n.toLowerCase().includes("hitl")) || JSON.stringify(actionNames).toLowerCase().includes("hitl");
-                        const jsonSize = opts?.json !== undefined ? JSON.stringify(opts.json).length : 0;
-                        fetch('http://127.0.0.1:7245/ingest/c9b6768a-494d-4365-bd46-bf6c43dc1e22', { method: 'POST', headers: { 'Content-Type': 'application/json' }, body: JSON.stringify({ location: 'src/sdk/client.ts:requestWithRetries:pre', message: 'HTTP request', data: { env: this.env.name, codeDir: process.env.EMA_MCP_CODE_DIR ?? null, cwd: process.cwd(), method, path, isUpdatePersona, attempt, retries, hasJson: opts?.json !== undefined, jsonSize, hasWorkflow: !!workflowAny, actionCount: Array.isArray(actions) ? actions.length : 0, containsHitl }, timestamp: Date.now(), sessionId: 'debug-session', runId: 'pre-fix', hypothesisId: 'H3' }) }).catch(() => { });
-                    }
+                // gRPC-Connect endpoints need Connect-Protocol-Version header
+                const isGrpcConnect = path.includes(".v1.") || path.includes(".v2.");
+                const headers = {
+                    Authorization: `Bearer ${this.env.bearerToken}`,
+                    "Content-Type": "application/json",
+                };
+                if (isGrpcConnect) {
+                    headers["Connect-Protocol-Version"] = "1";
                 }
-                catch { }
-                // #endregion agent log
                 const response = await fetch(fullUrl, {
                     method,
-                    headers: {
-                        Authorization: `Bearer ${this.env.bearerToken}`,
-                        "Content-Type": "application/json",
-                    },
+                    headers,
                     body: opts?.json !== undefined ? JSON.stringify(opts.json) : undefined,
                     signal: controller.signal,
                 });
                 clearTimeout(timeoutId);
-                // #region agent log
-                // H1/H2/H4: capture response status + request id for correlation (no secrets).
-                try {
-                    if (process.env.EMA_MCP_DEBUG_LOGS !== "1") {
-                        // disabled
-                    }
-                    else {
-                        const reqId = response.headers.get("x-request-id") ?? response.headers.get("x-correlation-id") ?? response.headers.get("x-amzn-trace-id");
-                        fetch('http://127.0.0.1:7245/ingest/c9b6768a-494d-4365-bd46-bf6c43dc1e22', { method: 'POST', headers: { 'Content-Type': 'application/json' }, body: JSON.stringify({ location: 'src/sdk/client.ts:requestWithRetries:post', message: 'HTTP response', data: { env: this.env.name, method, path, status: response.status, ok: response.ok, attempt, durationMs: Date.now() - startedAt, reqId }, timestamp: Date.now(), sessionId: 'debug-session', runId: 'pre-fix', hypothesisId: 'H1' }) }).catch(() => { });
-                    }
-                }
-                catch { }
-                // #endregion agent log
-                // #region agent log
-                // H1/H2: for update_persona failures, log first part of error body (redacted/truncated).
-                try {
-                    if (process.env.EMA_MCP_DEBUG_LOGS !== "1") {
-                        // disabled
-                    }
-                    else {
-                        const isUpdatePersona = path.includes("/api/personas/update_persona");
-                        if (isUpdatePersona && !response.ok) {
-                            const cloned = response.clone();
-                            const txt = await cloned.text();
-                            const snippet = txt.length > 1200 ? `${txt.slice(0, 1200)}…(truncated)` : txt;
-                            let parsedDetail;
-                            let hasGweIssues = false;
-                            try {
-                                const j = JSON.parse(txt);
-                                const detail = j.detail ?? j.message ?? j.error;
-                                parsedDetail = typeof detail === "string" ? detail : undefined;
-                                hasGweIssues = Array.isArray(j.gwe_issues);
-                            }
-                            catch { }
-                            fetch('http://127.0.0.1:7245/ingest/c9b6768a-494d-4365-bd46-bf6c43dc1e22', { method: 'POST', headers: { 'Content-Type': 'application/json' }, body: JSON.stringify({ location: 'src/sdk/client.ts:requestWithRetries:errorBody', message: 'HTTP error body (update_persona)', data: { env: this.env.name, method, path, status: response.status, parsedDetail, hasGweIssues, bodySnippet: snippet }, timestamp: Date.now(), sessionId: 'debug-session', runId: 'pre-fix', hypothesisId: 'H2' }) }).catch(() => { });
-                        }
-                    }
-                }
-                catch { }
-                // #endregion agent log
                 // Handle 401 - attempt token refresh once
                 if (response.status === 401 && !hasAttemptedRefresh && this.tokenRefreshConfig?.refreshCallback) {
                     hasAttemptedRefresh = true;
@@ -324,15 +266,6 @@ export class EmaClient {
         });
         if (!resp.ok) {
             const body = await resp.text();
-            // #region agent log
-            try {
-                if (this.debugLogsEnabled) {
-                    const snippet = body.length > 2000 ? `${body.slice(0, 2000)}…(truncated)` : body;
-                    fetch('http://127.0.0.1:7245/ingest/c9b6768a-494d-4365-bd46-bf6c43dc1e22', { method: 'POST', headers: { 'Content-Type': 'application/json' }, body: JSON.stringify({ location: 'src/sdk/client.ts:checkWorkflow:error', message: 'CheckWorkflow failed', data: { env: this.env.name, status: resp.status, bodySnippet: snippet }, timestamp: Date.now(), sessionId: 'debug-session', runId: 'pre-fix', hypothesisId: 'H5' }) }).catch(() => { });
-                }
-            }
-            catch { }
-            // #endregion agent log
             throw new EmaApiError({
                 statusCode: resp.status,
                 body,
@@ -340,35 +273,70 @@ export class EmaClient {
             });
         }
         const data = (await resp.json());
-        // #region agent log
-        try {
-            if (this.debugLogsEnabled) {
-                const keys = Object.keys(data);
-                fetch('http://127.0.0.1:7245/ingest/c9b6768a-494d-4365-bd46-bf6c43dc1e22', { method: 'POST', headers: { 'Content-Type': 'application/json' }, body: JSON.stringify({ location: 'src/sdk/client.ts:checkWorkflow:ok', message: 'CheckWorkflow ok', data: { env: this.env.name, keys }, timestamp: Date.now(), sessionId: 'debug-session', runId: 'pre-fix', hypothesisId: 'H5' }) }).catch(() => { });
-            }
-        }
-        catch { }
-        // #endregion agent log
         return data;
     }
     async updateAiEmployee(req, opts) {
         // Use /api/personas/update_persona - same endpoint as Ema frontend
+        // Debug logging for troubleshooting
+        if (opts?.verbose || process.env.EMA_DEBUG) {
+            console.error("[EmaClient] updateAiEmployee request:", JSON.stringify({
+                persona_id: req.persona_id,
+                has_workflow: !!req.workflow,
+                workflow_actions: req.workflow ? req.workflow.actions : undefined,
+                has_proto_config: !!req.proto_config,
+            }, null, 2));
+        }
         const resp = await this.requestWithRetries("POST", "/api/personas/update_persona", {
             json: req,
         });
         if (!resp.ok) {
             const body = await resp.text();
+            // Debug logging for errors
+            if (opts?.verbose || process.env.EMA_DEBUG) {
+                console.error("[EmaClient] updateAiEmployee FAILED:", resp.status, body);
+                console.error("[EmaClient] Request that failed:", JSON.stringify(req, null, 2).slice(0, 5000));
+            }
             // Try to parse error body for better error messages
             let errorDetail = "";
             try {
                 const errorJson = JSON.parse(body);
                 // Ema API returns InvalidPersonaResponse with 'detail' field
                 errorDetail = errorJson.detail ?? errorJson.message ?? errorJson.error ?? "";
-                // Also check for GWE (workflow) issues
-                if (errorJson.gwe_issues) {
-                    const issues = errorJson.gwe_issues;
-                    if (Array.isArray(issues)) {
-                        errorDetail += ` Workflow issues: ${issues.map((i) => i.message ?? i.detail ?? JSON.stringify(i)).join("; ")}`;
+                // Parse GWE (workflow) issues - it's an object, not an array
+                // See WorkflowIssues schema in OpenAPI spec
+                if (errorJson.gwe_issues && typeof errorJson.gwe_issues === "object") {
+                    const gwe = errorJson.gwe_issues;
+                    const issuesParts = [];
+                    if (gwe.detail) {
+                        issuesParts.push(`Detail: ${gwe.detail}`);
+                    }
+                    if (Array.isArray(gwe.missing_widgets) && gwe.missing_widgets.length > 0) {
+                        issuesParts.push(`Missing widgets: ${gwe.missing_widgets.join(", ")}`);
+                    }
+                    if (Array.isArray(gwe.unready_widgets) && gwe.unready_widgets.length > 0) {
+                        const unready = gwe.unready_widgets.map((w) => `${w.widget_name}: ${w.message}`);
+                        issuesParts.push(`Unready widgets: ${unready.join("; ")}`);
+                    }
+                    if (Array.isArray(gwe.missing_inputs) && gwe.missing_inputs.length > 0) {
+                        const missing = gwe.missing_inputs.map((i) => `${i.action_name}.${i.input_name}`);
+                        issuesParts.push(`Missing inputs: ${missing.join(", ")}`);
+                    }
+                    if (Array.isArray(gwe.mismatched_inputs) && gwe.mismatched_inputs.length > 0) {
+                        const mismatched = gwe.mismatched_inputs.map((i) => `${i.action_name}.${i.input_name}`);
+                        issuesParts.push(`Mismatched inputs: ${mismatched.join(", ")}`);
+                    }
+                    if (Array.isArray(gwe.named_result_errors) && gwe.named_result_errors.length > 0) {
+                        const resultErrs = gwe.named_result_errors.map((e) => `${e.named_result_key}: ${e.error_description}`);
+                        issuesParts.push(`Result errors: ${resultErrs.join("; ")}`);
+                    }
+                    if (gwe.has_cycles) {
+                        issuesParts.push("Workflow has cycles");
+                    }
+                    if (gwe.has_no_outputs) {
+                        issuesParts.push("Workflow has no outputs");
+                    }
+                    if (issuesParts.length > 0) {
+                        errorDetail += ` [GWE Issues: ${issuesParts.join(" | ")}]`;
                     }
                 }
             }