@dypai-ai/mcp 1.7.3 → 1.7.4

package/package.json

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

package/src/lib/cloudBackendCompiler.js

@@ -23,7 +23,10 @@ async function compileSnapshot(projectId, command, files, endpoint = undefined)
     files,
   }
   if (endpoint) args.endpoint = endpoint
-  return proxyToolCall("backend_compile_snapshot", args)
+  // Compiler `ok:false` is a domain validation result, not a transport/tool
+  // failure. It intentionally carries valid payloads plus diagnostics so the
+  // push pipeline can skip only broken flows instead of dropping all source.
+  return proxyToolCall("backend_compile_snapshot", args, { allowFailureResult: true })
 }
 
 export async function buildSnapshotPayload(dypaiRootDir, projectRoot = null) {

package/src/tools/proxy.js

@@ -209,7 +209,15 @@ async function ensureInitialized() {
   return initPromise
 }
 
-export async function proxyToolCall(toolName, args) {
+function shouldPromoteRemoteFailure(parsed, allowFailureResult = false) {
+  return !allowFailureResult
+    && parsed
+    && typeof parsed === "object"
+    && !Array.isArray(parsed)
+    && (parsed.success === false || parsed.ok === false)
+}
+
+export async function proxyToolCall(toolName, args, { allowFailureResult = false } = {}) {
   async function callOnce() {
     await ensureInitialized()
 
@@ -242,11 +250,9 @@ export async function proxyToolCall(toolName, args) {
         // 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))
-          }
+        if (shouldPromoteRemoteFailure(parsed, allowFailureResult)) {
+          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
       }
@@ -270,3 +276,7 @@ export async function proxyToolCall(toolName, args) {
     throw error
   }
 }
+
+export const __testing = {
+  shouldPromoteRemoteFailure,
+}

package/src/tools/sync/planner.js

@@ -539,15 +539,6 @@ export async function readLocalEffectiveState(rootDir, projectId = null, deps =
     if (shadowedMeta) shadowed.push({ name: payload.name, shadowed: shadowedMeta })
   }
 
-  for (const diagnostic of diagnostics) {
-    if (diagnostic?.severity !== "error") continue
-    yamlState.errors.push({
-      file: diagnostic.file || diagnostic.endpoint,
-      error: diagnostic.message || "Failed to build flow push payload",
-      rule: diagnostic.rule || "flow_push_payload_failed",
-    })
-  }
-
   for (const item of rejected) {
     if (!isBackendSourcePath(item?.path)) continue
     yamlState.errors.push({
@@ -562,7 +553,14 @@ export async function readLocalEffectiveState(rootDir, projectId = null, deps =
     byName[name] = { ...entry, source: "legacy-yaml" }
   }
 
-  return { byName, errors: yamlState.errors, shadowed, runnerWarning: null, automations: bulkResult.data?.automations ?? null }
+  return {
+    byName,
+    errors: yamlState.errors,
+    shadowed,
+    runnerWarning: null,
+    compilerDiagnostics: diagnostics,
+    automations: bulkResult.data?.automations ?? null,
+  }
 }
 
 // ─── Normalization + diff ──────────────────────────────────────────────────

package/src/tools/sync/push.js

@@ -310,6 +310,41 @@ function dropErroringFromPlan(plan, erroringEndpoints, skippedDiagnostics) {
     : undefined
 }
 
+function skippedEndpointsFromDiagnostics(diagnostics) {
+  const names = [...new Set(
+    (diagnostics || [])
+      .filter((item) => item?.severity === "error" && item.endpoint)
+      .map((item) => item.endpoint),
+  )]
+  return names.length
+    ? names.map((name) => ({
+        endpoint: name,
+        errors: diagnostics
+          .filter((item) => item.endpoint === name && item.severity === "error")
+          .slice(0, 5)
+          .map((item) => ({
+            rule: item.rule,
+            message: item.message,
+            file: item.file,
+            loc: item.loc,
+          })),
+      }))
+    : undefined
+}
+
+function mergeSkippedEndpointSummaries(...groups) {
+  const merged = new Map()
+  for (const group of groups) {
+    for (const item of group || []) {
+      const current = merged.get(item.endpoint)
+      merged.set(item.endpoint, current
+        ? { ...current, errors: [...(current.errors || []), ...(item.errors || [])].slice(0, 5) }
+        : item)
+    }
+  }
+  return merged.size ? [...merged.values()] : undefined
+}
+
 function splitBlockingConflicts(plan, conflicts) {
   const touched = new Set([
     ...(plan.update || []).map((item) => item.name),
@@ -741,6 +776,20 @@ export const dypaiPushTool = {
         error: e.message,
       }
     }
+    const compilerValidation = partitionValidationErrors(local.compilerDiagnostics)
+    if (compilerValidation.projectLevel.length) {
+      return {
+        success: false,
+        applied: false,
+        reason: "compiler_validation_failed",
+        diagnostics: compilerValidation.projectLevel.slice(0, 20),
+        hint: "The compiler reported project-level errors that cannot be isolated to one endpoint.",
+      }
+    }
+    for (const endpoint of compilerValidation.erroringEndpoints) {
+      erroringEndpoints.add(endpoint)
+    }
+    skippedDiagnostics.push(...compilerValidation.errorDiags)
     try {
       remote = await fetchRemoteState(targetProjectId)
     } catch (e) {
@@ -817,7 +866,10 @@ export const dypaiPushTool = {
     // they aren't pushed (they keep their live version). Deletes stay — a broken
     // local flow shouldn't block removing endpoints. skip_validation leaves
     // erroringEndpoints empty, so nothing is dropped.
-    const skipped_endpoints = dropErroringFromPlan(plan, erroringEndpoints, skippedDiagnostics)
+    const skipped_endpoints = mergeSkippedEndpointSummaries(
+      dropErroringFromPlan(plan, erroringEndpoints, skippedDiagnostics),
+      skippedEndpointsFromDiagnostics(compilerValidation.errorDiags),
+    )
 
     const groupChanges = (plan.groups?.create?.length || 0) + (plan.groups?.delete?.length || 0)
     const totalChanges = plan.create.length + plan.update.length + plan.delete.length + groupChanges
@@ -830,8 +882,13 @@ export const dypaiPushTool = {
         success: false,
         applied: false,
         reason: "conflicts_detected",
+        summary: summaryFromPlan(plan),
         conflicts: conflictSplit.blocking,
         non_blocking_conflicts: conflictSplit.non_blocking.length ? conflictSplit.non_blocking : undefined,
+        skipped_endpoints,
+        compiler_diagnostics: local.compilerDiagnostics?.length
+          ? local.compilerDiagnostics.slice(0, 50)
+          : undefined,
         hint: "Remote changes overlap endpoints this push would update/delete. Run dypai_pull to refresh local state, resolve the overlap, then push again. To override, pass force=true.",
       }
     }
@@ -861,6 +918,9 @@ export const dypaiPushTool = {
             : totalChanges === 0 ? "no_changes" : "dry_run",
         summary: summaryFromPlan(plan),
         skipped_endpoints,
+        compiler_diagnostics: local.compilerDiagnostics?.length
+          ? local.compilerDiagnostics.slice(0, 50)
+          : undefined,
         plan,
         non_blocking_conflicts: conflictSplit.non_blocking.length ? conflictSplit.non_blocking : undefined,
         types: typesGenerated,
@@ -1073,6 +1133,9 @@ export const dypaiPushTool = {
       details: applied,
       endpoint_results,
       skipped_endpoints,
+      compiler_diagnostics: local.compilerDiagnostics?.length
+        ? local.compilerDiagnostics.slice(0, 50)
+        : undefined,
       non_blocking_conflicts: conflictSplit.non_blocking.length ? conflictSplit.non_blocking : undefined,
       errors: errors.length ? errors : undefined,
       types: typesGenerated,
@@ -1131,6 +1194,8 @@ export const __testing = {
   syncAutomationMetadata,
   partitionValidationErrors,
   dropErroringFromPlan,
+  skippedEndpointsFromDiagnostics,
+  mergeSkippedEndpointSummaries,
   normalizeEndpointSelection,
   scopePlanToEndpoints,
   backendSourcePathsForEndpointSelection,