@flowdrop/flowdrop 1.5.0 → 1.6.0

package/dist/chat/responseParser.d.ts

@@ -2,18 +2,18 @@
  * Response Parser for LLM Chat Interface
  *
  * Extracts DSL commands from LLM markdown responses by parsing
- * fenced code blocks (```flowdrop or bare ```).
+ * fenced code blocks labeled ```flowdrop. All other fenced blocks
+ * (bare ```, ```python, etc.) are treated as explanation text.
  *
  * @module chat/responseParser
  */
-import type { ExtractedCommands } from "../types/chat.js";
+import type { ExtractedCommands } from '../types/chat.js';
 /**
  * Extract DSL commands from an LLM response string.
  *
- * Parses fenced code blocks labeled `flowdrop` (preferred) or bare
- * fenced code blocks (fallback). Text outside code blocks becomes
- * the explanation. Empty lines and comment lines inside code blocks
- * are skipped.
+ * Only fenced code blocks labeled `flowdrop` are parsed for commands.
+ * All other fenced blocks are passed through as explanation text.
+ * Empty lines and comment lines inside flowdrop blocks are skipped.
  *
  * @param llmResponse - The raw LLM response text (may contain markdown)
  * @returns Extracted commands and explanation text

package/dist/chat/responseParser.js

@@ -2,17 +2,17 @@
  * Response Parser for LLM Chat Interface
  *
  * Extracts DSL commands from LLM markdown responses by parsing
- * fenced code blocks (```flowdrop or bare ```).
+ * fenced code blocks labeled ```flowdrop. All other fenced blocks
+ * (bare ```, ```python, etc.) are treated as explanation text.
  *
  * @module chat/responseParser
  */
 /**
  * Extract DSL commands from an LLM response string.
  *
- * Parses fenced code blocks labeled `flowdrop` (preferred) or bare
- * fenced code blocks (fallback). Text outside code blocks becomes
- * the explanation. Empty lines and comment lines inside code blocks
- * are skipped.
+ * Only fenced code blocks labeled `flowdrop` are parsed for commands.
+ * All other fenced blocks are passed through as explanation text.
+ * Empty lines and comment lines inside flowdrop blocks are skipped.
  *
  * @param llmResponse - The raw LLM response text (may contain markdown)
  * @returns Extracted commands and explanation text
@@ -20,23 +20,39 @@
 export function extractCommands(llmResponse) {
     const commands = [];
     const explanationParts = [];
-    const lines = llmResponse.split("\n");
+    const lines = llmResponse.split('\n');
     let inCodeBlock = false;
     let isFlowdropBlock = false;
     let currentExplanation = [];
     let multilineBuffer = null;
     for (const line of lines) {
         const trimmed = line.trim();
+        // Handle multiline buffer FIRST — prevents code fences inside """...""" from
+        // closing the outer flowdrop block or being misinterpreted as control lines.
+        if (multilineBuffer !== null) {
+            // Closing """ must be a standalone line (exact match after trimming).
+            // This prevents content lines that end with """ from prematurely closing
+            // the block (e.g. `Use Python """docstrings"""` or JSON ending with """).
+            if (trimmed === '"""') {
+                multilineBuffer.push(line);
+                commands.push(multilineBuffer.join('\n'));
+                multilineBuffer = null;
+            }
+            else {
+                multilineBuffer.push(line); // preserve raw indentation inside value
+            }
+            continue;
+        }
         // Check for code block fence opening/closing
-        if (trimmed.startsWith("```")) {
+        if (trimmed.startsWith('```')) {
             if (!inCodeBlock) {
                 // Opening fence
                 inCodeBlock = true;
                 const lang = trimmed.slice(3).trim().toLowerCase();
-                isFlowdropBlock = lang === "flowdrop";
+                isFlowdropBlock = lang === 'flowdrop';
                 // Flush accumulated explanation text
                 if (currentExplanation.length > 0) {
-                    explanationParts.push(currentExplanation.join("\n"));
+                    explanationParts.push(currentExplanation.join('\n'));
                     currentExplanation = [];
                 }
             }
@@ -49,23 +65,11 @@ export function extractCommands(llmResponse) {
         }
         if (inCodeBlock && isFlowdropBlock) {
             // Skip empty lines and comment lines inside code blocks
-            if (trimmed === "" || trimmed.startsWith("#") || trimmed.startsWith("//")) {
+            if (trimmed === '' || trimmed.startsWith('#') || trimmed.startsWith('//')) {
                 continue;
             }
             // Detect opening triple-quote without a closing one on the same line —
             // start accumulating a multiline value block
-            if (multilineBuffer !== null) {
-                // We're inside a triple-quoted value; check for closing """
-                if (trimmed === '"""' || trimmed.endsWith('"""')) {
-                    multilineBuffer.push(line);
-                    commands.push(multilineBuffer.join("\n"));
-                    multilineBuffer = null;
-                }
-                else {
-                    multilineBuffer.push(line); // preserve raw indentation inside value
-                }
-                continue;
-            }
             const tripleOpen = trimmed.indexOf('"""');
             if (tripleOpen !== -1 && trimmed.indexOf('"""', tripleOpen + 3) === -1) {
                 // Opening triple-quote with no closing on this line — start buffer
@@ -80,8 +84,8 @@ export function extractCommands(llmResponse) {
     }
     // Flush remaining explanation text
     if (currentExplanation.length > 0) {
-        explanationParts.push(currentExplanation.join("\n"));
+        explanationParts.push(currentExplanation.join('\n'));
     }
-    const explanation = explanationParts.join("\n").trim();
+    const explanation = explanationParts.join('\n').trim();
     return { explanation, commands };
 }

package/dist/commands/batch.d.ts

@@ -6,7 +6,7 @@
  *
  * @module commands/batch
  */
-import type { Command, CommandContext, BatchResult } from "./types.js";
+import type { Command, CommandContext, BatchResult } from './types.js';
 /**
  * Execute an array of commands as a single atomic transaction.
  *

package/dist/commands/batch.js

@@ -6,7 +6,7 @@
  *
  * @module commands/batch
  */
-import { executeCommand } from "./executor.js";
+import { executeCommand } from './executor.js';
 /**
  * Execute an array of commands as a single atomic transaction.
  *
@@ -22,12 +22,10 @@ export function executeBatch(commands, context) {
             ok: true,
             results: [],
             completedCount: 0,
-            totalCount: 0,
+            totalCount: 0
         };
     }
-    const description = totalCount === 1
-        ? `batch: 1 command`
-        : `batch: ${totalCount} commands`;
+    const description = totalCount === 1 ? `batch: 1 command` : `batch: ${totalCount} commands`;
     context.dispatch.startTransaction(description);
     const results = [];
     for (let i = 0; i < commands.length; i++) {
@@ -42,7 +40,7 @@ export function executeBatch(commands, context) {
                 results,
                 completedCount: i,
                 totalCount,
-                error: result.error,
+                error: result.error
             };
         }
     }
@@ -51,6 +49,6 @@ export function executeBatch(commands, context) {
         ok: true,
         results,
         completedCount: totalCount,
-        totalCount,
+        totalCount
     };
 }

package/dist/commands/executor.d.ts

@@ -6,8 +6,8 @@
  *
  * @module commands/executor
  */
-import type { Command, CommandContext, CommandResult } from "./types.js";
-import type { WorkflowNode } from "../types/index.js";
+import type { Command, CommandContext, CommandResult } from './types.js';
+import type { WorkflowNode } from '../types/index.js';
 /**
  * Convert an internal namespaced node ID to a DSL short ID.
  * "agentspec.llm_node.1" → "llm_node.1"