ponch-mcp-server 1.0.74 → 1.0.75

package/dist/index.js

@@ -6401,6 +6401,19 @@ function wrapWithContract(contract, helper, options = {}) {
     }
     const parseResult = contract.paramsSchema.safeParse(input);
     if (!parseResult.success) {
+      const validationErrors = parseResult.error.issues.map((i) => {
+        const issue = i;
+        return {
+          path: i.path,
+          code: i.code,
+          message: i.message,
+          received: issue.received,
+          expected: issue.expected
+        };
+      });
+      const camposFallidos = validationErrors.map((v) => v.path.join(".")).filter(Boolean);
+      const baseMsg = getWrapperMessage("input_invalido", locale);
+      const text = camposFallidos.length ? locale === "en" ? `${baseMsg} Issues with: ${camposFallidos.join(", ")}.` : `${baseMsg} Hubo problemas con: ${camposFallidos.join(", ")}.` : baseMsg;
       await writeAuditLog({
         tenantId: ctx.tenantId,
         brandId: ctx.brandId ?? null,
@@ -6409,13 +6422,14 @@ function wrapWithContract(contract, helper, options = {}) {
         motivo: "Input inv\xE1lido \u2014 Zod parse failed",
         conversacionId: ctx.conversacionId ?? null,
         status: "error",
-        errorMessage: `Input shape inv\xE1lido: ${parseResult.error.issues.map((i) => `${i.path.join(".")}: ${i.message}`).join("; ")}`,
+        errorMessage: `Input shape inv\xE1lido: ${validationErrors.map((v) => `${v.path.join(".")}: ${v.code} (${v.message})`).join("; ")}`,
         durationMs: Date.now() - startMs
       });
       return {
-        text: getWrapperMessage("input_invalido", locale),
+        text,
         structuredOutput: null,
-        state: "error"
+        state: "error",
+        validationErrors
       };
     }
     const parsedInput = parseResult.data;
@@ -6462,7 +6476,6 @@ function wrapWithContract(contract, helper, options = {}) {
     let output;
     try {
       output = await helper(parsedInput);
-      contract.outputSchema.parse(output);
     } catch (err) {
       await writeAuditLog({
         tenantId: ctx.tenantId,
@@ -6481,6 +6494,36 @@ function wrapWithContract(contract, helper, options = {}) {
         state: "error"
       };
     }
+    const outputParse = contract.outputSchema.safeParse(output);
+    if (!outputParse.success) {
+      const validationErrors = outputParse.error.issues.map((i) => {
+        const issue = i;
+        return {
+          path: i.path,
+          code: i.code,
+          message: i.message,
+          received: issue.received,
+          expected: issue.expected
+        };
+      });
+      await writeAuditLog({
+        tenantId: ctx.tenantId,
+        brandId: ctx.brandId ?? null,
+        actor: { type: "martin", uid: ctx.user.uid, nombre: ctx.user.nombre },
+        action: contract.auditAction,
+        motivo: `Output del helper inv\xE1lido \u2014 bug de "${contract.name}"`,
+        conversacionId: ctx.conversacionId ?? null,
+        status: "error",
+        errorMessage: `Output shape inv\xE1lido: ${validationErrors.map((v) => `${v.path.join(".")}: ${v.code} (${v.message})`).join("; ")}`,
+        durationMs: Date.now() - startMs
+      });
+      return {
+        text: martinSafeError(new Error("output_invalid"), locale),
+        structuredOutput: null,
+        state: "error",
+        validationErrors
+      };
+    }
     const maybeDisabled = output;
     if (maybeDisabled.disabled === true) {
       const code = maybeDisabled.code;
@@ -7838,7 +7881,14 @@ function registerMarketingTools(server, session) {
         input: { tenantId, brandId, plan },
         ctx
       });
-      const payload = result.state === "success" ? result.structuredOutput : { ok: false, state: result.state, mensaje: result.text };
+      const payload = result.state === "success" ? result.structuredOutput : {
+        ok: false,
+        state: result.state,
+        mensaje: result.text,
+        // HITO 6 A6.7: incluir detalles Zod estructurados para que
+        // Claude (LLM) pueda auto-recuperarse en el siguiente intento.
+        ...result.validationErrors && result.validationErrors.length > 0 ? { validationErrors: result.validationErrors } : {}
+      };
       return { content: [{ type: "text", text: JSON.stringify(payload) }] };
     }
   );
@@ -8046,7 +8096,14 @@ Si pasas campos dentro de "datos", se hace merge con los datos existentes (no lo
         input: { tenantId, brandId, mes, semana, slotIndex, cambios, accionContenidoExistente },
         ctx
       });
-      const payload = result.state === "success" ? result.structuredOutput : { ok: false, state: result.state, mensaje: result.text };
+      const payload = result.state === "success" ? result.structuredOutput : {
+        ok: false,
+        state: result.state,
+        mensaje: result.text,
+        // HITO 6 A6.7: incluir detalles Zod estructurados para que
+        // Claude (LLM) pueda auto-recuperarse en el siguiente intento.
+        ...result.validationErrors && result.validationErrors.length > 0 ? { validationErrors: result.validationErrors } : {}
+      };
       return { content: [{ type: "text", text: JSON.stringify(payload) }] };
     }
   );