@dypai-ai/mcp 1.2.2 → 1.2.3

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@dypai-ai/mcp",
-  "version": "1.2.2",
+  "version": "1.2.3",
   "description": "DYPAI MCP Server — AI agent toolkit for building and deploying full-stack apps",
   "type": "module",
   "main": "src/index.js",

package/src/tools/proxy.js

@@ -129,6 +129,15 @@ export async function proxyToolCall(toolName, args) {
       if (typeof parsed === "string" && /^(Error:|Input validation error:|You don't have)/i.test(parsed)) {
         throw new Error(parsed)
       }
+      // Other remote errors come as a JSON object with success:false but no isError
+      // flag. The push pipeline was treating these as successes — promote them to
+      // throws here so the caller's try/catch sees them.
+      if (parsed && typeof parsed === "object" && !Array.isArray(parsed)) {
+        if (parsed.success === false || parsed.ok === false) {
+          const msg = parsed.error || parsed.message || parsed.detail || JSON.stringify(parsed).slice(0, 300)
+          throw new Error(typeof msg === "string" ? msg : JSON.stringify(msg))
+        }
+      }
       return parsed
     }
     return response.result

package/src/tools/sync/push.js

@@ -61,26 +61,52 @@ function endpointPayload(row) {
   return p
 }
 
+/**
+ * Treat a remote tool response as a definite success only when it has at
+ * least one of the markers we expect from a real mutation. Anything else
+ * (empty object, error string masquerading as data, missing id) becomes
+ * an error so the push doesn't lie to the agent.
+ */
+function assertMutationOK(result, op, name) {
+  if (!result || typeof result !== "object") {
+    throw new Error(`${op} ${name}: empty response from remote (got ${JSON.stringify(result).slice(0, 120)})`)
+  }
+  if (result.error) {
+    throw new Error(`${op} ${name}: ${typeof result.error === "string" ? result.error : JSON.stringify(result.error)}`)
+  }
+  // Successful create returns { id, name } or { ok: true }; update/delete return
+  // { message } or { ok: true }. If none of these markers are present the call
+  // probably failed silently.
+  const hasMarker = result.id || result.ok === true || result.message || result.endpoint_id || result.success === true
+  if (!hasMarker) {
+    throw new Error(`${op} ${name}: response missing success markers — ${JSON.stringify(result).slice(0, 200)}`)
+  }
+  return result
+}
+
 async function applyCreate(canonicalRow, projectId) {
-  return proxyToolCall("create_endpoint", {
+  const res = await proxyToolCall("create_endpoint", {
     ...(projectId ? { project_id: projectId } : {}),
     ...endpointPayload(canonicalRow),
   })
+  return assertMutationOK(res, "create", canonicalRow.name)
 }
 
 async function applyUpdate(canonicalRow, endpointId, projectId) {
-  return proxyToolCall("update_endpoint", {
+  const res = await proxyToolCall("update_endpoint", {
     ...(projectId ? { project_id: projectId } : {}),
     endpoint_id: endpointId,
     updates: endpointPayload(canonicalRow),
   })
+  return assertMutationOK(res, "update", canonicalRow.name)
 }
 
 async function applyDelete(endpointId, projectId) {
-  return proxyToolCall("delete_endpoint", {
+  const res = await proxyToolCall("delete_endpoint", {
     ...(projectId ? { project_id: projectId } : {}),
     endpoint_id: endpointId,
   })
+  return assertMutationOK(res, "delete", endpointId)
 }
 
 // ─── Realtime policies sync ────────────────────────────────────────────────