@codemation/core-nodes 0.10.1 → 0.10.2

package/dist/index.d.cts

@@ -1428,6 +1428,7 @@ declare class NodeBackedToolConfig<TNodeConfig extends RunnableNodeConfig<any, a
   readonly toolKind: "nodeBacked";
   readonly description?: string;
   readonly presentation?: AgentCanvasPresentation;
+  readonly onRejected?: "halt" | "return";
   private readonly inputSchemaValue;
   private readonly outputSchemaValue;
   private readonly mapInputValue?;
@@ -1589,6 +1590,13 @@ type NodeBackedToolConfigOptions<TNodeConfig extends RunnableNodeConfig<any, any
   outputSchema: TOutputSchema;
   mapInput?: NodeBackedToolInputMapper<TNodeConfig, input<TInputSchema>>;
   mapOutput?: NodeBackedToolOutputMapper<TNodeConfig, input<TInputSchema>, output<TOutputSchema>>;
+  /**
+   * Marks THIS tool binding as human-in-the-loop and sets the behavior when a human rejects the
+   * approval: `"return"` feeds the rejection back to the agent as a tool result, `"halt"` stops the
+   * run. Set per binding, so two tools backed by the same node can reject differently. When set, it
+   * takes precedence over any `humanApprovalToolBehavior` marker carried by the backing node.
+   */
+  onRejected?: "halt" | "return";
 }>;
 interface AgentNodeConfig<TInputJson$1 = unknown, TOutputJson$1 = unknown> extends RunnableNodeConfig<TInputJson$1, TOutputJson$1> {
   readonly messages: AgentMessageConfig<TInputJson$1>;
@@ -1670,7 +1678,13 @@ type HttpBodySpec = Readonly<{
   kind: "none";
 }> | Readonly<{
   kind: "json";
-  data: unknown;
+  /**
+   * Serializable object/array to encode as the JSON body. Encoded exactly once via
+   * `JSON.stringify`, so pass the value directly (e.g. `{ a: 1 }`) — never a
+   * pre-stringified string (`JSON.stringify(...)`), which would double-encode it.
+   * Typed as `object` so a bare string/primitive is a compile-time error.
+   */
+  data: object;
 }> | Readonly<{
   kind: "form";
   data: Readonly<Record<string, string>>;
@@ -2680,8 +2694,10 @@ declare class AIAgentNode implements RunnableNode<AIAgent<any, any>> {
   private resolveTools;
   private createItemScopedTools;
   /**
-   * 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.
    */
   private resolveHumanApprovalBehavior;
   /**
@@ -2972,7 +2988,7 @@ declare class HttpRequest<TInputJson$1 = Readonly<{
     headers?: Readonly<Record<string, string>>;
     /** Query parameters to append to the URL. */
     query?: Readonly<Record<string, string>>;
-    /** Request body specification. For canvas use, pass a JSON string in `body.data`. */
+    /** Request body specification. For `kind:"json"`, pass the object directly in `body.data` — it is JSON-encoded exactly once, so never a pre-stringified string. */
     body?: HttpBodySpec;
     /**
      * Credential slot.
@@ -3054,7 +3070,7 @@ declare class HttpRequest<TInputJson$1 = Readonly<{
     headers?: Readonly<Record<string, string>>;
     /** Query parameters to append to the URL. */
     query?: Readonly<Record<string, string>>;
-    /** Request body specification. For canvas use, pass a JSON string in `body.data`. */
+    /** Request body specification. For `kind:"json"`, pass the object directly in `body.data` — it is JSON-encoded exactly once, so never a pre-stringified string. */
     body?: HttpBodySpec;
     /**
      * Credential slot.

package/dist/index.d.ts

@@ -1428,6 +1428,7 @@ declare class NodeBackedToolConfig<TNodeConfig extends RunnableNodeConfig<any, a
   readonly toolKind: "nodeBacked";
   readonly description?: string;
   readonly presentation?: AgentCanvasPresentation;
+  readonly onRejected?: "halt" | "return";
   private readonly inputSchemaValue;
   private readonly outputSchemaValue;
   private readonly mapInputValue?;
@@ -1589,6 +1590,13 @@ type NodeBackedToolConfigOptions<TNodeConfig extends RunnableNodeConfig<any, any
   outputSchema: TOutputSchema;
   mapInput?: NodeBackedToolInputMapper<TNodeConfig, input<TInputSchema>>;
   mapOutput?: NodeBackedToolOutputMapper<TNodeConfig, input<TInputSchema>, output<TOutputSchema>>;
+  /**
+   * Marks THIS tool binding as human-in-the-loop and sets the behavior when a human rejects the
+   * approval: `"return"` feeds the rejection back to the agent as a tool result, `"halt"` stops the
+   * run. Set per binding, so two tools backed by the same node can reject differently. When set, it
+   * takes precedence over any `humanApprovalToolBehavior` marker carried by the backing node.
+   */
+  onRejected?: "halt" | "return";
 }>;
 interface AgentNodeConfig<TInputJson$1 = unknown, TOutputJson$1 = unknown> extends RunnableNodeConfig<TInputJson$1, TOutputJson$1> {
   readonly messages: AgentMessageConfig<TInputJson$1>;
@@ -1670,7 +1678,13 @@ type HttpBodySpec = Readonly<{
   kind: "none";
 }> | Readonly<{
   kind: "json";
-  data: unknown;
+  /**
+   * Serializable object/array to encode as the JSON body. Encoded exactly once via
+   * `JSON.stringify`, so pass the value directly (e.g. `{ a: 1 }`) — never a
+   * pre-stringified string (`JSON.stringify(...)`), which would double-encode it.
+   * Typed as `object` so a bare string/primitive is a compile-time error.
+   */
+  data: object;
 }> | Readonly<{
   kind: "form";
   data: Readonly<Record<string, string>>;
@@ -2680,8 +2694,10 @@ declare class AIAgentNode implements RunnableNode<AIAgent<any, any>> {
   private resolveTools;
   private createItemScopedTools;
   /**
-   * 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.
    */
   private resolveHumanApprovalBehavior;
   /**
@@ -2972,7 +2988,7 @@ declare class HttpRequest<TInputJson$1 = Readonly<{
     headers?: Readonly<Record<string, string>>;
     /** Query parameters to append to the URL. */
     query?: Readonly<Record<string, string>>;
-    /** Request body specification. For canvas use, pass a JSON string in `body.data`. */
+    /** Request body specification. For `kind:"json"`, pass the object directly in `body.data` — it is JSON-encoded exactly once, so never a pre-stringified string. */
     body?: HttpBodySpec;
     /**
      * Credential slot.
@@ -3054,7 +3070,7 @@ declare class HttpRequest<TInputJson$1 = Readonly<{
     headers?: Readonly<Record<string, string>>;
     /** Query parameters to append to the URL. */
     query?: Readonly<Record<string, string>>;
-    /** Request body specification. For canvas use, pass a JSON string in `body.data`. */
+    /** Request body specification. For `kind:"json"`, pass the object directly in `body.data` — it is JSON-encoded exactly once, so never a pre-stringified string. */
     body?: HttpBodySpec;
     /**
      * Credential slot.

package/dist/index.js

@@ -495,10 +495,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);
@@ -6540,14 +6543,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)