@agentforge/patterns 0.18.6 → 0.18.8

package/dist/index.cjs

@@ -298,6 +298,37 @@ function formatScratchpad(scratchpad) {
 // src/react/nodes/reasoning.ts
 var import_core3 = require("@agentforge/core");
 
+// src/shared/model-response-content.ts
+function getModelResponseText(content) {
+  return typeof content === "string" ? content : void 0;
+}
+function getModelResponseTextParts(content) {
+  if (!Array.isArray(content)) {
+    return [];
+  }
+  return content.flatMap((part) => {
+    const text = getModelResponseText(part);
+    if (text !== void 0) {
+      return [text];
+    }
+    if (typeof part === "object" && part !== null && "text" in part && typeof part.text === "string") {
+      return [part.text];
+    }
+    return [];
+  });
+}
+function stringifyModelResponseContent(content) {
+  return getModelResponseText(content) ?? JSON.stringify(content);
+}
+function stringifyModelResponseContentSafely(content) {
+  try {
+    return stringifyModelResponseContent(content) ?? "undefined";
+  } catch (error) {
+    const reason = error instanceof Error ? error.message : String(error);
+    return `[Unserializable model content: ${reason}]`;
+  }
+}
+
 // src/react/nodes/shared.ts
 var import_messages = require("@langchain/core/messages");
 
@@ -454,7 +485,7 @@ function createReasoningNode(llm, tools, systemPrompt, maxIterations, verbose =
     });
     const messages = buildReasoningMessages(systemPrompt, state.messages, state.scratchpad);
     const response = await llmWithTools.invoke(messages);
-    const thought = typeof response.content === "string" ? response.content : "";
+    const thought = getModelResponseText(response.content) ?? "";
     const toolCalls = extractToolCalls(response);
     const shouldContinue = toolCalls.length > 0 && currentIteration + 1 < maxIterations;
     reasoningLogger.info("Reasoning complete", {
@@ -1142,40 +1173,15 @@ var plannerLogger = createPatternLogger("agentforge:patterns:plan-execute:planne
 var executorLogger = createPatternLogger("agentforge:patterns:plan-execute:executor");
 var replannerLogger = createPatternLogger("agentforge:patterns:plan-execute:replanner");
 
-// src/plan-execute/model-response.ts
-function hasTextContentPart(value) {
-  return typeof value === "object" && value !== null && "text" in value && typeof value.text === "string";
-}
-function stringifyModelContent(value) {
+// src/plan-execute/response-parsing.ts
+function parseJsonModelResponse(content, context) {
+  const text = getModelResponseText(content);
+  const textParts = Array.isArray(content) ? getModelResponseTextParts(content.filter((part) => typeof part !== "string")).filter(
+    (part) => part.length > 0
+  ) : [];
+  const representedContent = text ?? (textParts.length > 0 ? textParts.join("\n") : stringifyModelResponseContentSafely(content));
   try {
-    const serialized = JSON.stringify(value);
-    return serialized === void 0 ? "undefined" : serialized;
-  } catch (error) {
-    const reason = error instanceof Error ? error.message : String(error);
-    return `[Unserializable model content: ${reason}]`;
-  }
-}
-function normalizeModelContent(content) {
-  if (typeof content === "string") {
-    return content;
-  }
-  if (Array.isArray(content)) {
-    const textParts = [];
-    for (const part of content) {
-      if (hasTextContentPart(part) && part.text.length > 0) {
-        textParts.push(part.text);
-      }
-    }
-    if (textParts.length > 0) {
-      return textParts.join("\n");
-    }
-  }
-  return stringifyModelContent(content);
-}
-function parseModelResponse(content, context) {
-  const normalized = normalizeModelContent(content);
-  try {
-    return JSON.parse(normalized);
+    return JSON.parse(representedContent);
   } catch (parseError) {
     throw new Error(`Failed to parse ${context} from LLM response: ${parseError}`);
   }
@@ -1207,7 +1213,7 @@ function createPlannerNode(config) {
         new import_messages2.HumanMessage(userPrompt)
       ];
       const response = await model.invoke(messages);
-      const parsed = parseModelResponse(response.content, "plan");
+      const parsed = parseJsonModelResponse(response.content, "plan");
       const plan = {
         steps: parsed.steps.slice(0, maxSteps),
         goal: parsed.goal || state.input || "",
@@ -1472,7 +1478,10 @@ function createReplannerNode(config) {
         new import_messages3.HumanMessage(userPrompt)
       ];
       const response = await model.invoke(messages);
-      const decision = parseModelResponse(response.content, "replan decision");
+      const decision = parseJsonModelResponse(
+        response.content,
+        "replan decision"
+      );
       if (decision.shouldReplan) {
         replannerLogger.info("Replanning triggered", {
           reason: decision.reason,
@@ -1862,9 +1871,6 @@ var import_messages4 = require("@langchain/core/messages");
 var generatorLogger = createPatternLogger("agentforge:patterns:reflection:generator");
 var reflectorLogger = createPatternLogger("agentforge:patterns:reflection:reflector");
 var reviserLogger = createPatternLogger("agentforge:patterns:reflection:reviser");
-function serializeModelContent(content) {
-  return typeof content === "string" ? content : JSON.stringify(content);
-}
 function buildRevisionHistorySection(revisions, heading) {
   if (revisions.length === 0) {
     return "";
@@ -1900,7 +1906,7 @@ ${lastReflection.critique}` : "";
         new import_messages4.SystemMessage(systemPrompt),
         new import_messages4.HumanMessage(userPrompt)
       ]);
-      const content = serializeModelContent(response.content);
+      const content = stringifyModelResponseContent(response.content);
       generatorLogger.info("Response generated", {
         attempt: state.iteration + 1,
         responseLength: content.length,
@@ -1984,7 +1990,7 @@ function createReflectorNode(config) {
         new import_messages5.SystemMessage(systemPrompt),
         new import_messages5.HumanMessage(userPrompt)
       ]);
-      const reflection = parseReflection(serializeModelContent(response.content));
+      const reflection = parseReflection(stringifyModelResponseContent(response.content));
       reflectorLogger.info("Reflection complete", {
         attempt: state.iteration,
         ...reflection.score !== void 0 ? { score: reflection.score } : {},
@@ -2041,7 +2047,7 @@ function createReviserNode(config) {
         new import_messages6.SystemMessage(systemPrompt),
         new import_messages6.HumanMessage(userPrompt)
       ]);
-      const content = serializeModelContent(response.content);
+      const content = stringifyModelResponseContent(response.content);
       reviserLogger.info("Revision complete", {
         attempt: state.iteration,
         revisionLength: content.length,
@@ -2433,12 +2439,9 @@ function convertWorkerToolsForLangChain(tools) {
   const safeTools = tools ?? [];
   return (0, import_core8.toLangChainTools)(safeTools);
 }
-function serializeModelContent2(content) {
-  if (typeof content === "string") {
-    return content;
-  }
+function requireSerializedModelContent(content) {
   try {
-    const serialized = JSON.stringify(content);
+    const serialized = stringifyModelResponseContent(content);
     if (typeof serialized !== "string") {
       const error = new Error(
         "Failed to serialize model content: JSON.stringify returned undefined"
@@ -2506,30 +2509,12 @@ function isRecord(value) {
 function isContentCarrier(value) {
   return isRecord(value) && "content" in value;
 }
-function serializeRoutingContent(content) {
-  if (typeof content === "string") {
-    return content;
-  }
-  if (Array.isArray(content)) {
-    const textParts = content.flatMap((part) => {
-      if (typeof part === "string") {
-        return [part];
-      }
-      if (isRecord(part) && typeof part.text === "string") {
-        return [part.text];
-      }
-      return [];
-    });
-    if (textParts.length > 0) {
-      return textParts.join("\n");
-    }
-    return JSON.stringify(content);
-  }
-  return JSON.stringify(content);
-}
 function normalizeRoutingDecisionInput(decision) {
   if (isContentCarrier(decision)) {
-    return JSON.parse(serializeRoutingContent(decision.content));
+    const text = getModelResponseText(decision.content);
+    const textParts = getModelResponseTextParts(decision.content);
+    const representedContent = text ?? (textParts.length > 0 ? textParts.join("\n") : stringifyModelResponseContent(decision.content));
+    return JSON.parse(representedContent);
   }
   return decision;
 }
@@ -2815,23 +2800,22 @@ function getReActResultShape(value) {
 }
 
 // src/multi-agent/utils-react-result.ts
-function safeSerializeContent(content) {
+function getWrappedResponseContent(content) {
   if (content === null || content === void 0) {
     return void 0;
   }
-  if (typeof content === "string") {
-    return content.length > 0 ? content : void 0;
+  const text = getModelResponseText(content);
+  if (text !== void 0) {
+    return text.length > 0 ? text : void 0;
   }
   if (Array.isArray(content)) {
     const parts = content.map((part) => {
-      if (typeof part === "string") {
-        return part;
-      }
-      if (isRecord2(part) && typeof part.text === "string" && part.text.length > 0) {
-        return part.text;
+      const partText = getModelResponseTextParts([part])[0];
+      if (partText !== void 0 && partText.length > 0) {
+        return partText;
       }
       try {
-        return JSON.stringify(part);
+        return stringifyModelResponseContent(part);
       } catch {
         return String(part);
       }
@@ -2839,7 +2823,7 @@ function safeSerializeContent(content) {
     return parts.length > 0 ? parts.join("\n") : void 0;
   }
   try {
-    const serialized = JSON.stringify(content);
+    const serialized = stringifyModelResponseContent(content);
     if (typeof serialized === "string" && serialized.length > 0 && serialized !== "null") {
       return serialized;
     }
@@ -2850,7 +2834,7 @@ function safeSerializeContent(content) {
 }
 function extractResponse(resultShape) {
   const lastMessage = resultShape.messages?.[resultShape.messages.length - 1];
-  const serialized = safeSerializeContent(lastMessage?.content);
+  const serialized = getWrappedResponseContent(lastMessage?.content);
   return serialized ?? "No response";
 }
 function extractToolsUsed(resultShape) {
@@ -3048,7 +3032,7 @@ function createAggregatorNode(config = {}) {
       const messages = createPromptMessages(systemPrompt, createAggregationPrompt(state));
       logger.debug("Invoking aggregation LLM");
       const response = await model.invoke(messages);
-      const aggregatedResponse = serializeModelContent2(response.content);
+      const aggregatedResponse = requireSerializedModelContent(response.content);
       logger.info("Aggregation complete", {
         responseLength: aggregatedResponse.length
       });
@@ -3243,7 +3227,7 @@ async function invokeWorkerModel(model, config, assignment) {
   }
   logger.debug("Invoking LLM", { workerId: config.id });
   const response = await modelToUse.invoke(messages);
-  const result = serializeModelContent2(response.content);
+  const result = requireSerializedModelContent(response.content);
   logger.info("Worker task completed", {
     workerId: config.id,
     assignmentId: assignment.id,

package/dist/index.js

@@ -193,6 +193,37 @@ function formatScratchpad(scratchpad) {
 // src/react/nodes/reasoning.ts
 import { toLangChainTools } from "@agentforge/core";
 
+// src/shared/model-response-content.ts
+function getModelResponseText(content) {
+  return typeof content === "string" ? content : void 0;
+}
+function getModelResponseTextParts(content) {
+  if (!Array.isArray(content)) {
+    return [];
+  }
+  return content.flatMap((part) => {
+    const text = getModelResponseText(part);
+    if (text !== void 0) {
+      return [text];
+    }
+    if (typeof part === "object" && part !== null && "text" in part && typeof part.text === "string") {
+      return [part.text];
+    }
+    return [];
+  });
+}
+function stringifyModelResponseContent(content) {
+  return getModelResponseText(content) ?? JSON.stringify(content);
+}
+function stringifyModelResponseContentSafely(content) {
+  try {
+    return stringifyModelResponseContent(content) ?? "undefined";
+  } catch (error) {
+    const reason = error instanceof Error ? error.message : String(error);
+    return `[Unserializable model content: ${reason}]`;
+  }
+}
+
 // src/react/nodes/shared.ts
 import {
   HumanMessage,
@@ -354,7 +385,7 @@ function createReasoningNode(llm, tools, systemPrompt, maxIterations, verbose =
     });
     const messages = buildReasoningMessages(systemPrompt, state.messages, state.scratchpad);
     const response = await llmWithTools.invoke(messages);
-    const thought = typeof response.content === "string" ? response.content : "";
+    const thought = getModelResponseText(response.content) ?? "";
     const toolCalls = extractToolCalls(response);
     const shouldContinue = toolCalls.length > 0 && currentIteration + 1 < maxIterations;
     reasoningLogger.info("Reasoning complete", {
@@ -1042,40 +1073,15 @@ var plannerLogger = createPatternLogger("agentforge:patterns:plan-execute:planne
 var executorLogger = createPatternLogger("agentforge:patterns:plan-execute:executor");
 var replannerLogger = createPatternLogger("agentforge:patterns:plan-execute:replanner");
 
-// src/plan-execute/model-response.ts
-function hasTextContentPart(value) {
-  return typeof value === "object" && value !== null && "text" in value && typeof value.text === "string";
-}
-function stringifyModelContent(value) {
+// src/plan-execute/response-parsing.ts
+function parseJsonModelResponse(content, context) {
+  const text = getModelResponseText(content);
+  const textParts = Array.isArray(content) ? getModelResponseTextParts(content.filter((part) => typeof part !== "string")).filter(
+    (part) => part.length > 0
+  ) : [];
+  const representedContent = text ?? (textParts.length > 0 ? textParts.join("\n") : stringifyModelResponseContentSafely(content));
   try {
-    const serialized = JSON.stringify(value);
-    return serialized === void 0 ? "undefined" : serialized;
-  } catch (error) {
-    const reason = error instanceof Error ? error.message : String(error);
-    return `[Unserializable model content: ${reason}]`;
-  }
-}
-function normalizeModelContent(content) {
-  if (typeof content === "string") {
-    return content;
-  }
-  if (Array.isArray(content)) {
-    const textParts = [];
-    for (const part of content) {
-      if (hasTextContentPart(part) && part.text.length > 0) {
-        textParts.push(part.text);
-      }
-    }
-    if (textParts.length > 0) {
-      return textParts.join("\n");
-    }
-  }
-  return stringifyModelContent(content);
-}
-function parseModelResponse(content, context) {
-  const normalized = normalizeModelContent(content);
-  try {
-    return JSON.parse(normalized);
+    return JSON.parse(representedContent);
   } catch (parseError) {
     throw new Error(`Failed to parse ${context} from LLM response: ${parseError}`);
   }
@@ -1107,7 +1113,7 @@ function createPlannerNode(config) {
         new HumanMessage2(userPrompt)
       ];
       const response = await model.invoke(messages);
-      const parsed = parseModelResponse(response.content, "plan");
+      const parsed = parseJsonModelResponse(response.content, "plan");
       const plan = {
         steps: parsed.steps.slice(0, maxSteps),
         goal: parsed.goal || state.input || "",
@@ -1372,7 +1378,10 @@ function createReplannerNode(config) {
         new HumanMessage3(userPrompt)
       ];
       const response = await model.invoke(messages);
-      const decision = parseModelResponse(response.content, "replan decision");
+      const decision = parseJsonModelResponse(
+        response.content,
+        "replan decision"
+      );
       if (decision.shouldReplan) {
         replannerLogger.info("Replanning triggered", {
           reason: decision.reason,
@@ -1762,9 +1771,6 @@ import { HumanMessage as HumanMessage4, SystemMessage as SystemMessage4 } from "
 var generatorLogger = createPatternLogger("agentforge:patterns:reflection:generator");
 var reflectorLogger = createPatternLogger("agentforge:patterns:reflection:reflector");
 var reviserLogger = createPatternLogger("agentforge:patterns:reflection:reviser");
-function serializeModelContent(content) {
-  return typeof content === "string" ? content : JSON.stringify(content);
-}
 function buildRevisionHistorySection(revisions, heading) {
   if (revisions.length === 0) {
     return "";
@@ -1800,7 +1806,7 @@ ${lastReflection.critique}` : "";
         new SystemMessage4(systemPrompt),
         new HumanMessage4(userPrompt)
       ]);
-      const content = serializeModelContent(response.content);
+      const content = stringifyModelResponseContent(response.content);
       generatorLogger.info("Response generated", {
         attempt: state.iteration + 1,
         responseLength: content.length,
@@ -1884,7 +1890,7 @@ function createReflectorNode(config) {
         new SystemMessage5(systemPrompt),
         new HumanMessage5(userPrompt)
       ]);
-      const reflection = parseReflection(serializeModelContent(response.content));
+      const reflection = parseReflection(stringifyModelResponseContent(response.content));
       reflectorLogger.info("Reflection complete", {
         attempt: state.iteration,
         ...reflection.score !== void 0 ? { score: reflection.score } : {},
@@ -1941,7 +1947,7 @@ function createReviserNode(config) {
         new SystemMessage6(systemPrompt),
         new HumanMessage6(userPrompt)
       ]);
-      const content = serializeModelContent(response.content);
+      const content = stringifyModelResponseContent(response.content);
       reviserLogger.info("Revision complete", {
         attempt: state.iteration,
         revisionLength: content.length,
@@ -2333,12 +2339,9 @@ function convertWorkerToolsForLangChain(tools) {
   const safeTools = tools ?? [];
   return toLangChainTools2(safeTools);
 }
-function serializeModelContent2(content) {
-  if (typeof content === "string") {
-    return content;
-  }
+function requireSerializedModelContent(content) {
   try {
-    const serialized = JSON.stringify(content);
+    const serialized = stringifyModelResponseContent(content);
     if (typeof serialized !== "string") {
       const error = new Error(
         "Failed to serialize model content: JSON.stringify returned undefined"
@@ -2406,30 +2409,12 @@ function isRecord(value) {
 function isContentCarrier(value) {
   return isRecord(value) && "content" in value;
 }
-function serializeRoutingContent(content) {
-  if (typeof content === "string") {
-    return content;
-  }
-  if (Array.isArray(content)) {
-    const textParts = content.flatMap((part) => {
-      if (typeof part === "string") {
-        return [part];
-      }
-      if (isRecord(part) && typeof part.text === "string") {
-        return [part.text];
-      }
-      return [];
-    });
-    if (textParts.length > 0) {
-      return textParts.join("\n");
-    }
-    return JSON.stringify(content);
-  }
-  return JSON.stringify(content);
-}
 function normalizeRoutingDecisionInput(decision) {
   if (isContentCarrier(decision)) {
-    return JSON.parse(serializeRoutingContent(decision.content));
+    const text = getModelResponseText(decision.content);
+    const textParts = getModelResponseTextParts(decision.content);
+    const representedContent = text ?? (textParts.length > 0 ? textParts.join("\n") : stringifyModelResponseContent(decision.content));
+    return JSON.parse(representedContent);
   }
   return decision;
 }
@@ -2715,23 +2700,22 @@ function getReActResultShape(value) {
 }
 
 // src/multi-agent/utils-react-result.ts
-function safeSerializeContent(content) {
+function getWrappedResponseContent(content) {
   if (content === null || content === void 0) {
     return void 0;
   }
-  if (typeof content === "string") {
-    return content.length > 0 ? content : void 0;
+  const text = getModelResponseText(content);
+  if (text !== void 0) {
+    return text.length > 0 ? text : void 0;
   }
   if (Array.isArray(content)) {
     const parts = content.map((part) => {
-      if (typeof part === "string") {
-        return part;
-      }
-      if (isRecord2(part) && typeof part.text === "string" && part.text.length > 0) {
-        return part.text;
+      const partText = getModelResponseTextParts([part])[0];
+      if (partText !== void 0 && partText.length > 0) {
+        return partText;
       }
       try {
-        return JSON.stringify(part);
+        return stringifyModelResponseContent(part);
       } catch {
         return String(part);
       }
@@ -2739,7 +2723,7 @@ function safeSerializeContent(content) {
     return parts.length > 0 ? parts.join("\n") : void 0;
   }
   try {
-    const serialized = JSON.stringify(content);
+    const serialized = stringifyModelResponseContent(content);
     if (typeof serialized === "string" && serialized.length > 0 && serialized !== "null") {
       return serialized;
     }
@@ -2750,7 +2734,7 @@ function safeSerializeContent(content) {
 }
 function extractResponse(resultShape) {
   const lastMessage = resultShape.messages?.[resultShape.messages.length - 1];
-  const serialized = safeSerializeContent(lastMessage?.content);
+  const serialized = getWrappedResponseContent(lastMessage?.content);
   return serialized ?? "No response";
 }
 function extractToolsUsed(resultShape) {
@@ -2948,7 +2932,7 @@ function createAggregatorNode(config = {}) {
       const messages = createPromptMessages(systemPrompt, createAggregationPrompt(state));
       logger.debug("Invoking aggregation LLM");
       const response = await model.invoke(messages);
-      const aggregatedResponse = serializeModelContent2(response.content);
+      const aggregatedResponse = requireSerializedModelContent(response.content);
       logger.info("Aggregation complete", {
         responseLength: aggregatedResponse.length
       });
@@ -3143,7 +3127,7 @@ async function invokeWorkerModel(model, config, assignment) {
   }
   logger.debug("Invoking LLM", { workerId: config.id });
   const response = await modelToUse.invoke(messages);
-  const result = serializeModelContent2(response.content);
+  const result = requireSerializedModelContent(response.content);
   logger.info("Worker task completed", {
     workerId: config.id,
     assignmentId: assignment.id,

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@agentforge/patterns",
-  "version": "0.18.6",
+  "version": "0.18.8",
   "description": "Production-ready agent workflow patterns for TypeScript including ReAct and Planner-Executor, built on LangGraph.",
   "type": "module",
   "main": "./dist/index.cjs",
@@ -41,13 +41,13 @@
     "url": "https://github.com/TVScoundrel/agentforge/issues"
   },
   "dependencies": {
-    "@agentforge/core": "0.18.6",
+    "@agentforge/core": "0.18.8",
     "@langchain/core": "^1.1.17",
     "@langchain/langgraph": "^1.1.2",
     "zod": "^3.23.8"
   },
   "devDependencies": {
-    "@agentforge/testing": "0.18.6",
+    "@agentforge/testing": "0.18.8",
     "@eslint/js": "^9.17.0",
     "@types/node": "^22.10.2",
     "eslint": "^9.17.0",