@codemation/core-nodes 0.10.0 → 0.10.2

package/CHANGELOG.md

@@ -1,5 +1,32 @@
 # @codemation/core-nodes
 
+## 0.10.2
+
+### Patch Changes
+
+- [#209](https://github.com/MadeRelevant/codemation/pull/209) [`681f5a4`](https://github.com/MadeRelevant/codemation/commit/681f5a4bfcfe4496206dda1de4fcf5eeec27e0ec) Thanks [@cblokland90](https://github.com/cblokland90)! - fix(core-nodes): HttpRequest body kind:"json" encodes data exactly once
+
+  `HttpBodyBuilder` always ran `JSON.stringify(data)`, so callers that passed an
+  already-stringified string (`JSON.stringify({...})`) shipped a double-encoded
+  `"\"...\""` payload. The `kind:"json"` `data` field is now typed `object` (a bare
+  string/primitive is a compile-time error), with a runtime guard that throws a clear
+  message if a string still reaches the builder (e.g. via `unknown`/`as` or a resolved
+  expression). Six examples that passed pre-stringified data are fixed to pass the object
+  directly: node-istestrun, node-webhooktrigger, activate-with-credentials,
+  file-upload-ocr-drive, hitl-cp-inbox-approval, scenario-invoice-scan-and-post.
+
+- [#210](https://github.com/MadeRelevant/codemation/pull/210) [`d4c7aca`](https://github.com/MadeRelevant/codemation/commit/d4c7aca5248352b82313efb16d5792cc24875005) Thanks [@cblokland90](https://github.com/cblokland90)! - AgentToolFactory.asTool() now accepts a per-binding `onRejected` ("halt" | "return"). Setting it marks that tool binding as human-in-the-loop and takes precedence over any `humanApprovalToolBehavior` default carried by the backing node, so two tools backed by the same node can reject differently without cloning the node config.
+
+- Updated dependencies [[`3317947`](https://github.com/MadeRelevant/codemation/commit/33179472db68b2f4fc1a15961ac389df25d8cb6f), [`d4c7aca`](https://github.com/MadeRelevant/codemation/commit/d4c7aca5248352b82313efb16d5792cc24875005)]:
+  - @codemation/core@0.13.2
+
+## 0.10.1
+
+### Patch Changes
+
+- Updated dependencies [[`2fcb715`](https://github.com/MadeRelevant/codemation/commit/2fcb7153d9c732b2f846b8a8d1cc5626b4363fa6)]:
+  - @codemation/core@0.13.1
+
 ## 0.10.0
 
 ### Minor Changes

package/dist/index.cjs

@@ -525,10 +525,13 @@ var HttpRequestExecutor = class {
 var HttpBodyBuilder = class {
 	async build(spec, item, ctx) {
 		if (!spec || spec.kind === "none") return;
-		if (spec.kind === "json") return {
-			body: JSON.stringify(spec.data),
-			contentType: "application/json"
-		};
+		if (spec.kind === "json") {
+			if (typeof spec.data === "string") throw new Error("HttpRequest body kind:\"json\" expects a serializable object for \"data\", but received a string. Pass the object directly (e.g. { a: 1 }), not a pre-stringified JSON string (JSON.stringify(...)).");
+			return {
+				body: JSON.stringify(spec.data),
+				contentType: "application/json"
+			};
+		}
 		if (spec.kind === "form") {
 			const params = new URLSearchParams();
 			for (const [key, value] of Object.entries(spec.data)) params.append(key, value);
@@ -6570,14 +6573,17 @@ let AIAgentNode = class AIAgentNode$1 {
 		});
 	}
 	/**
-	* Detects whether a tool config is backed by a `defineHumanApprovalNode` marker
-	* and returns the HITL behavior config, or `undefined` when not a HITL tool.
+	* Resolves the HITL behavior for a tool binding, or `undefined` when it is not a HITL tool.
+	* A binding is HITL if either the backing node carries a `defineHumanApprovalNode` marker or the
+	* binding sets a per-binding `onRejected` via `asTool(..., { onRejected })`. The per-binding value
+	* wins over the node marker, so two tools backed by the same node can reject differently.
 	*/
 	resolveHumanApprovalBehavior(config$1) {
 		if (!this.isNodeBackedToolConfig(config$1)) return void 0;
 		const marker = config$1.node.humanApprovalToolBehavior;
-		if (marker === void 0) return void 0;
-		return { onRejected: marker.onRejected ?? "return" };
+		const perBinding = config$1.onRejected;
+		if (marker === void 0 && perBinding === void 0) return void 0;
+		return { onRejected: perBinding ?? marker?.onRejected ?? "return" };
 	}
 	/**
 	* Invoke a text turn using the merged tool set from item-scoped tools (coordinator-managed)