@dypai-ai/mcp 1.4.0 → 1.4.1

package/package.json

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

package/src/tools/sync/transforms.js

@@ -74,14 +74,17 @@ export const NODE_FIELD_TRANSFORMS = [
     name: "sql_extraction",
     appliesWhen: (nodeType) => nodeType === "dypai_database",
     pull(params, ctx) {
-      if (params.query && !shouldInlineSql(params.query)) {
-        // Flat layout: sql/<endpoint>.sql if only one SQL node, else sql/<endpoint>.<node>.sql
-        const suffix = ctx.sqlNodeCount > 1 ? `.${ctx.nodeId}` : ""
-        const path = `sql/${ctx.endpointName}${suffix}.sql`
-        ctx.emitFile(path, params.query.trim() + "\n")
-        return { query_file: path }
+      if (!params.query) return {}
+      if (shouldInlineSql(params.query)) {
+        // Short SQL stays inline. Must be re-emitted here because
+        // `pullConsumes: ["query"]` deletes it from the base object.
+        return { query: params.query }
       }
-      return {}
+      // Flat layout: sql/<endpoint>.sql if only one SQL node, else sql/<endpoint>.<node>.sql
+      const suffix = ctx.sqlNodeCount > 1 ? `.${ctx.nodeId}` : ""
+      const path = `sql/${ctx.endpointName}${suffix}.sql`
+      ctx.emitFile(path, params.query.trim() + "\n")
+      return { query_file: path }
     },
     push(params, ctx) {
       if (params.query_file) {
@@ -97,13 +100,16 @@ export const NODE_FIELD_TRANSFORMS = [
     name: "prompt_extraction",
     appliesWhen: (nodeType) => nodeType === "agent",
     pull(params, ctx) {
-      if (params.system_prompt && params.system_prompt.length > PROMPT_INLINE_MAX_CHARS) {
-        const suffix = ctx.promptNodeCount > 1 ? `.${ctx.nodeId}` : ""
-        const path = `prompts/${ctx.endpointName}${suffix}.md`
-        ctx.emitFile(path, params.system_prompt.trim() + "\n")
-        return { system_prompt_file: path }
+      if (!params.system_prompt) return {}
+      if (params.system_prompt.length <= PROMPT_INLINE_MAX_CHARS) {
+        // Short prompt stays inline. Must be re-emitted here because
+        // `pullConsumes: ["system_prompt"]` deletes it from the base object.
+        return { system_prompt: params.system_prompt }
       }
-      return {}
+      const suffix = ctx.promptNodeCount > 1 ? `.${ctx.nodeId}` : ""
+      const path = `prompts/${ctx.endpointName}${suffix}.md`
+      ctx.emitFile(path, params.system_prompt.trim() + "\n")
+      return { system_prompt_file: path }
     },
     push(params, ctx) {
       if (params.system_prompt_file) {
@@ -119,14 +125,20 @@ export const NODE_FIELD_TRANSFORMS = [
     name: "code_extraction",
     appliesWhen: (nodeType) => nodeType === "javascript_code" || nodeType === "python_code",
     pull(params, ctx) {
-      if (params.code && params.code.length > SQL_INLINE_MAX_CHARS) {
-        const ext = ctx.nodeType === "python_code" ? "py" : "js"
-        const suffix = ctx.codeNodeCount > 1 ? `.${ctx.nodeId}` : ""
-        const path = `code/${ctx.endpointName}${suffix}.${ext}`
-        ctx.emitFile(path, params.code.trim() + "\n")
-        return { code_file: path }
+      if (!params.code) return {}
+      if (params.code.length <= SQL_INLINE_MAX_CHARS) {
+        // Short code stays inline. Must be re-emitted here because
+        // `pullConsumes: ["code"]` deletes it from the base object.
+        // Before this fix, short code was silently dropped from the YAML
+        // (neither extracted to a file nor kept inline) — endpoints pulled
+        // from the engine ended up with just `timeout_ms` and no handler.
+        return { code: params.code }
       }
-      return {}
+      const ext = ctx.nodeType === "python_code" ? "py" : "js"
+      const suffix = ctx.codeNodeCount > 1 ? `.${ctx.nodeId}` : ""
+      const path = `code/${ctx.endpointName}${suffix}.${ext}`
+      ctx.emitFile(path, params.code.trim() + "\n")
+      return { code_file: path }
     },
     push(params, ctx) {
       if (params.code_file) {

package/src/tools/sync/validate.js

@@ -580,7 +580,12 @@ function validateEndpoint(entry, ctx) {
           // Enum / range checks for primitive values
           const prop = properties[key]
           const v = node[key]
-          if (prop.enum && typeof v === "string" && !prop.enum.includes(v)) {
+          // Skip enum validation if the value contains a ${...} placeholder —
+          // the actual value is resolved at runtime, so the engine enforces
+          // the enum then. Common case: operation: ${input.operation} in
+          // dypai_storage / dypai_database nodes.
+          const hasPlaceholder = typeof v === "string" && v.includes("${")
+          if (prop.enum && typeof v === "string" && !hasPlaceholder && !prop.enum.includes(v)) {
             diagnostics.push({
               severity: "error",
               rule: "param_enum_violation",