@dypai-ai/mcp 1.7.1 → 1.7.2

package/package.json

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

package/src/tools/sync/push.js

@@ -483,7 +483,8 @@ export const dypaiPushTool = {
     "After validation, regenerates `dypai/types/endpoints.gen.ts` from effective Flow/YAML contracts (check the `types` field in the response). " +
     "ALWAYS run dypai_diff first to preview what will be staged. " +
     "The public dypai_push wrapper also saves frontend source to Studio. " +
-    "By default, endpoints in remote but missing locally are kept (safe). Pass delete_orphans: true to stage their deletion as a draft as well.",
+    "By default, endpoints in remote but missing locally are kept (safe). Pass delete_orphans: true to stage their deletion as a draft as well. " +
+    "PARTIAL PUSH: an endpoint whose flow has errors is SKIPPED (it keeps its live version) and the rest are pushed — one broken endpoint no longer blocks the whole push. Skipped ones are listed in `skipped_endpoints` (with their errors) and `partial_success` is set. A project-level error still blocks. Use skip_validation:true to push everything, errors included.",
   inputSchema: {
     type: "object",
     properties: {
@@ -513,7 +514,7 @@ export const dypaiPushTool = {
       },
       skip_validation: {
         type: "boolean",
-        description: "Skip the dypai_validate pre-flight check. Use only when you know the validator is wrong. Default: false.",
+        description: "Skip the dypai_validate pre-flight check and push EVERY endpoint, including ones with errors (no per-endpoint skipping). By default the push already applies the valid endpoints and only skips the erroring ones — so use this only when you know the validator is wrong. Default: false.",
         default: false,
       },
     },
@@ -529,19 +530,31 @@ export const dypaiPushTool = {
     const configProjectId = config?.project_id || null
     const targetProjectId = project_id || configProjectId
 
-    // Pre-flight validation (lint). Blocks the push unless skip_validation is set.
+    // Pre-flight validation (lint). Partial push: endpoints whose flow has an
+    // error are SKIPPED (they keep their live version) and the rest is pushed —
+    // one broken endpoint no longer tanks the whole push. A project-level error
+    // (no endpoint to isolate it to) still blocks, since it can't be scoped out.
+    // skip_validation bypasses all of this and pushes everything.
+    let erroringEndpoints = new Set()
+    let skippedDiagnostics = []
     if (!skip_validation && !dry_run) {
       try {
         const validation = await runValidation(rootDir, project_id || configProjectId)
         if (!validation.success) {
-          return {
-            success: false,
-            applied: false,
-            reason: "validation_failed",
-            summary: validation.summary,
-            diagnostics: validation.diagnostics.filter(d => d.severity === "error").slice(0, 20),
-            hint: `${validation.summary.errors} error(s) would cause runtime failures. Fix them, or retry with skip_validation: true to override.`,
+          const errorDiags = (validation.diagnostics || []).filter(d => d.severity === "error")
+          const projectLevel = errorDiags.filter(d => !d.endpoint)
+          if (projectLevel.length) {
+            return {
+              success: false,
+              applied: false,
+              reason: "validation_failed",
+              summary: validation.summary,
+              diagnostics: projectLevel.slice(0, 20),
+              hint: `${projectLevel.length} project-level error(s) can't be isolated to a single endpoint. Fix them, or retry with skip_validation: true to override.`,
+            }
           }
+          skippedDiagnostics = errorDiags
+          erroringEndpoints = new Set(errorDiags.map(d => d.endpoint).filter(Boolean))
         }
       } catch (e) {
         // Don't block the push on a validator crash — just warn
@@ -640,6 +653,30 @@ export const dypaiPushTool = {
       stateSnapshot,
       liveRemote: remote,
     })
+
+    // Partial push: drop endpoints that failed validation from create/update so
+    // 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 skippedNames = []
+    if (erroringEndpoints.size) {
+      const drop = (arr) => (arr || []).filter((item) => {
+        if (erroringEndpoints.has(item.name)) { skippedNames.push(item.name); return false }
+        return true
+      })
+      plan.create = drop(plan.create)
+      plan.update = drop(plan.update)
+    }
+    const skipped_endpoints = skippedNames.length
+      ? [...new Set(skippedNames)].map((name) => ({
+          endpoint: name,
+          errors: skippedDiagnostics
+            .filter((d) => d.endpoint === name)
+            .slice(0, 5)
+            .map((d) => ({ rule: d.rule, message: d.message, file: d.file, loc: d.loc })),
+        }))
+      : undefined
+
     const groupChanges = (plan.groups?.create?.length || 0) + (plan.groups?.delete?.length || 0)
     const totalChanges = plan.create.length + plan.update.length + plan.delete.length + groupChanges
 
@@ -676,8 +713,10 @@ export const dypaiPushTool = {
           ? "source_checkpoint_failed"
           : automationSyncHasFailed
             ? "automation_sync_failed"
+            : skipped_endpoints && totalChanges === 0 ? "all_skipped"
             : totalChanges === 0 ? "no_changes" : "dry_run",
         summary: summaryFromPlan(plan),
+        skipped_endpoints,
         plan,
         non_blocking_conflicts: conflictSplit.non_blocking.length ? conflictSplit.non_blocking : undefined,
         types: typesGenerated,
@@ -847,7 +886,7 @@ export const dypaiPushTool = {
     return {
       success: errors.length === 0 && !sourceCheckpointFailed && !automationSyncHasFailed,
       applied: endpointTotal > 0 || (realtime && !realtime.skipped),
-      partial_success: partialSuccess || sourceCheckpointFailed || automationSyncHasFailed,
+      partial_success: partialSuccess || sourceCheckpointFailed || automationSyncHasFailed || !!skipped_endpoints,
       reason: sourceCheckpointFailed
         ? "source_checkpoint_failed"
         : automationSyncHasFailed
@@ -878,6 +917,7 @@ export const dypaiPushTool = {
       },
       details: applied,
       endpoint_results,
+      skipped_endpoints,
       non_blocking_conflicts: conflictSplit.non_blocking.length ? conflictSplit.non_blocking : undefined,
       errors: errors.length ? errors : undefined,
       types: typesGenerated,
@@ -887,7 +927,9 @@ export const dypaiPushTool = {
       // Only one next_step — and only when it's non-obvious. Drafts win
       // over the test suggestion because tests against live won't see
       // the change until the drafts are promoted.
-      next_step: errors.length
+      next_step: skipped_endpoints
+        ? `⚠️ ${skipped_endpoints.length} endpoint(s) SKIPPED (they keep their live version) because their flow has errors: ${skipped_endpoints.map(s => s.endpoint).slice(0, 5).join(", ")}${skipped_endpoints.length > 5 ? "…" : ""}. Everything else was pushed. Fix the skipped ones (see skipped_endpoints[].errors) and push again — or use skip_validation:true to push them anyway.`
+        : errors.length
         ? partialSuccess
           ? `${endpointTotal} endpoint(s) saved, ${failedTotal} failed. Fix failed endpoints and push again. Failed: ${errors.slice(0, 3).map((item) => item.endpoint || item.group || item.op).join(", ")}${errors.length > 3 ? "…" : ""}`
           : "Fix the offending YAMLs and push again."