@codemation/core-nodes 0.4.3 → 0.7.0

package/src/nodes/httpRequest.ts

@@ -1,5 +1,18 @@
-import { RetryPolicy, type RetryPolicySpec, type RunnableNodeConfig, type TypeToken } from "@codemation/core";
+import {
+  RetryPolicy,
+  type CredentialRequirement,
+  type RetryPolicySpec,
+  type RunnableNodeConfig,
+  type TypeToken,
+} from "@codemation/core";
 
+import type { HttpBodySpec } from "../http/httpRequest.types";
+import {
+  apiKeyCredentialType,
+  basicAuthCredentialType,
+  bearerTokenCredentialType,
+  oauth2ClientCredentialsType,
+} from "../credentials/index";
 import { HttpRequestNode } from "./HttpRequestNodeFactory";
 
 export type HttpRequestDownloadMode = "auto" | "always" | "never";
@@ -13,9 +26,33 @@ export type HttpRequestOutputJson = Readonly<{
   statusText: string;
   mimeType: string;
   headers: Readonly<Record<string, string>>;
+  json?: unknown;
+  text?: string;
   bodyBinaryName?: string;
+  /** Set when `responseFormat === "binary"`. Name of the binary slot the response was stored in. */
+  binarySlot?: string;
+  /** Set when `responseFormat === "binary"`. MIME type of the stored response. */
+  contentType?: string;
+  /** Set when `responseFormat === "binary"`. Size in bytes of the stored response. */
+  size?: number;
+  /** Set when `responseFormat === "binary"`. Filename inferred from URL or Content-Disposition. */
+  filename?: string;
 }>;
 
+/**
+ * The built-in HTTP request credential type IDs accepted by the `HttpRequest` node.
+ * These match the four generic credential types shipped with `@codemation/core-nodes`.
+ */
+export const HTTP_REQUEST_ACCEPTED_CREDENTIAL_TYPES: ReadonlyArray<string> = [
+  bearerTokenCredentialType.definition.typeId,
+  apiKeyCredentialType.definition.typeId,
+  basicAuthCredentialType.definition.typeId,
+  oauth2ClientCredentialsType.definition.typeId,
+] as const;
+
+/** Default maximum response size for binary mode: 100 MiB. */
+const DEFAULT_RESPONSE_SIZE_CAP_BYTES = 100 * 1024 * 1024;
+
 export class HttpRequest<
   TInputJson = Readonly<{ url?: string }>,
   TOutputJson = HttpRequestOutputJson,
@@ -23,14 +60,55 @@ export class HttpRequest<
   readonly kind = "node" as const;
   readonly type: TypeToken<unknown> = HttpRequestNode;
   readonly execution = { hint: "local" } as const;
+  readonly icon = "lucide:globe" as const;
 
   constructor(
     public readonly name: string,
     public readonly args: Readonly<{
+      /** HTTP method (default: GET). */
       method?: string;
+      /**
+       * Legacy: field name on item.json to read the URL from.
+       * Use `url` for a literal/templated URL instead.
+       */
       urlField?: string;
+      /** Literal or templated URL. When present, takes precedence over `urlField`. */
+      url?: string;
+      /** Extra headers to add to every request. */
+      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`. */
+      body?: HttpBodySpec;
+      /**
+       * Credential slot key. When set, the node resolves a credential via
+       * `ctx.getCredential(credentialSlot)` and applies it to the request.
+       * The slot must be declared in `getCredentialRequirements()`.
+       */
+      credentialSlot?: string;
       binaryName?: string;
       downloadMode?: HttpRequestDownloadMode;
+      /**
+       * Controls how the response body is handled.
+       * - `"json"` / `"text"`: existing behaviour (parse + emit on `item.json`).
+       * - `"binary"`: read the response as raw bytes and store via `ctx.binary.attach`.
+       *   The output JSON contains `{ status, headers, binarySlot, contentType, size, filename }`
+       *   but NOT the raw bytes. Use `responseBinarySlot` to name the slot (default `"response"`).
+       *
+       * When omitted, the existing `downloadMode` logic applies (backward-compatible).
+       */
+      responseFormat?: "json" | "text" | "binary";
+      /**
+       * Name of the binary slot to write the response body into when `responseFormat === "binary"`.
+       * Defaults to `"response"`.
+       */
+      responseBinarySlot?: string;
+      /**
+       * Maximum response size in bytes for binary mode. Checked against the `Content-Length`
+       * response header before allocating memory. Defaults to 100 MiB (104857600).
+       * Requests whose `Content-Length` exceeds this cap are rejected before the body is read.
+       */
+      responseSizeCapBytes?: number;
       id?: string;
     }> = {},
     public readonly retryPolicy: RetryPolicySpec = RetryPolicy.defaultForHttp,
@@ -55,6 +133,33 @@ export class HttpRequest<
   get downloadMode(): HttpRequestDownloadMode {
     return this.args.downloadMode ?? "auto";
   }
+
+  get responseFormat(): "json" | "text" | "binary" | undefined {
+    return this.args.responseFormat;
+  }
+
+  get responseBinarySlot(): string {
+    return this.args.responseBinarySlot ?? "response";
+  }
+
+  get responseSizeCapBytes(): number {
+    return this.args.responseSizeCapBytes ?? DEFAULT_RESPONSE_SIZE_CAP_BYTES;
+  }
+
+  getCredentialRequirements(): ReadonlyArray<CredentialRequirement> {
+    if (!this.args.credentialSlot) {
+      return [];
+    }
+    return [
+      {
+        slotKey: this.args.credentialSlot,
+        label: "Authentication",
+        acceptedTypes: HTTP_REQUEST_ACCEPTED_CREDENTIAL_TYPES,
+        optional: true,
+        helpText: "Optional credential for authenticating the HTTP request.",
+      },
+    ];
+  }
 }
 
 export { HttpRequestNode } from "./HttpRequestNodeFactory";

package/src/nodes/if.ts

@@ -6,7 +6,7 @@ export class If<TInputJson = unknown> implements RunnableNodeConfig<TInputJson,
   readonly kind = "node" as const;
   readonly type: TypeToken<unknown> = IfNode;
   readonly execution = { hint: "local" } as const;
-  readonly icon = "lucide:split" as const;
+  readonly icon = "lucide:split@rot=90" as const;
   readonly declaredOutputPorts = ["true", "false"] as const;
   constructor(
     public readonly name: string,

package/src/nodes/isTestRun.ts

@@ -0,0 +1,24 @@
+import type { RunnableNodeConfig, TypeToken } from "@codemation/core";
+
+import { IsTestRunNode } from "./IsTestRunNode";
+
+/**
+ * Branches per-item on whether the current run is a test run. Output ports: `true`, `false`.
+ * The wire payload is unchanged — this is a router, not a transform.
+ */
+export class IsTestRun<TInputJson = unknown> implements RunnableNodeConfig<TInputJson, TInputJson> {
+  readonly kind = "node" as const;
+  readonly type: TypeToken<unknown> = IsTestRunNode;
+  readonly execution = { hint: "local" } as const;
+  readonly icon = "lucide:flask-conical" as const;
+  readonly declaredOutputPorts = ["true", "false"] as const;
+  readonly name: string;
+  readonly id?: string;
+
+  constructor(name: string = "Is test run?", id?: string) {
+    this.name = name;
+    this.id = id;
+  }
+}
+
+export { IsTestRunNode } from "./IsTestRunNode";

package/src/nodes/mapData.ts

@@ -16,6 +16,7 @@ export class MapData<TInputJson = unknown, TOutputJson = unknown> implements Run
   readonly execution = { hint: "local" } as const;
   /** Zero mapped items should still allow downstream nodes to run. */
   readonly continueWhenEmptyOutput = true as const;
+  readonly icon = "lucide:square-pen" as const;
   readonly keepBinaries: boolean;
 
   constructor(

package/src/nodes/merge.ts

@@ -10,7 +10,7 @@ export class Merge<TInputJson = unknown, TOutputJson = TInputJson> implements Ru
 > {
   readonly kind = "node" as const;
   readonly type: TypeToken<unknown> = MergeNode;
-  readonly icon = "lucide:git-merge" as const;
+  readonly icon = "lucide:merge@rot=90" as const;
 
   constructor(
     public readonly name: string,

package/src/nodes/noOp.ts

@@ -6,6 +6,7 @@ export class NoOp<TItemJson = unknown> implements RunnableNodeConfig<TItemJson,
   readonly kind = "node" as const;
   readonly type: TypeToken<unknown> = NoOpNode;
   readonly execution = { hint: "local" } as const;
+  readonly icon = "lucide:circle-dashed" as const;
 
   constructor(
     public readonly name: string = "NoOp",

package/src/nodes/split.ts

@@ -12,7 +12,7 @@ export class Split<TIn = unknown, TElem = unknown> implements RunnableNodeConfig
    * Mirrors {@link MapData}'s empty-output behavior.
    */
   readonly continueWhenEmptyOutput = true as const;
-  readonly icon = "lucide:ungroup" as const;
+  readonly icon = "builtin:split-rows" as const;
 
   constructor(
     public readonly name: string,

package/src/nodes/testTrigger.ts

@@ -0,0 +1,72 @@
+import type {
+  CredentialRequirement,
+  Item,
+  TestTriggerNodeConfig,
+  TestTriggerSetupContext,
+  TypeToken,
+} from "@codemation/core";
+
+import { TestTriggerNode } from "./TestTriggerNode";
+
+export interface TestTriggerOptions<TOutputJson> {
+  readonly name?: string;
+  readonly id?: string;
+  readonly icon?: string;
+  /** Cap on simultaneous in-flight test cases for one suite run. Default: 4 (orchestrator). */
+  readonly concurrency?: number;
+  readonly credentialRequirements?: ReadonlyArray<CredentialRequirement>;
+  /**
+   * Free-form description of where the test cases come from. Shown in the node properties
+   * panel and the Tests-tab suite-detail header so authors revisiting the workflow six months
+   * later remember which mailbox / folder / fixture file the cases originate from.
+   */
+  readonly description?: string;
+  /**
+   * Author callback that yields one item per test case. Items are dispatched as separate
+   * workflow runs by the TestSuiteOrchestrator, with `executionOptions.testContext` set.
+   * The provided context exposes credential resolution and an AbortSignal for cancellation.
+   */
+  generateItems(ctx: TestTriggerSetupContext<TestTrigger<TOutputJson>>): AsyncIterable<Item<TOutputJson>>;
+  /**
+   * Optional resolver: extract a human-readable label from a yielded item. The orchestrator
+   * persists this on the run, so the Tests-tab tree-table shows e.g. "RFQ for batch 14"
+   * instead of an opaque runId. Typical use: `(item) => item.json.subject` for mailbox tests.
+   */
+  caseLabel?(item: Item<TOutputJson>): string | undefined;
+}
+
+/**
+ * Trigger config for a test fixture source. Drop one (or more) of these on the canvas alongside
+ * a workflow's live triggers; clicking "Run tests" on the Tests tab invokes
+ * {@link TestTriggerOptions.generateItems} via the TestSuiteOrchestrator.
+ */
+export class TestTrigger<TOutputJson = unknown> implements TestTriggerNodeConfig<TOutputJson> {
+  readonly kind = "trigger" as const;
+  readonly triggerKind = "test" as const;
+  readonly type: TypeToken<unknown> = TestTriggerNode;
+  readonly icon: string;
+  readonly name: string;
+  readonly id?: string;
+  readonly concurrency?: number;
+  readonly description?: string;
+  readonly generateItems: TestTriggerOptions<TOutputJson>["generateItems"];
+  readonly caseLabel?: TestTriggerOptions<TOutputJson>["caseLabel"];
+  private readonly credentialRequirements: ReadonlyArray<CredentialRequirement>;
+
+  constructor(options: TestTriggerOptions<TOutputJson>) {
+    this.name = options.name ?? "Test trigger";
+    this.id = options.id;
+    this.icon = options.icon ?? "lucide:flask-conical";
+    this.concurrency = options.concurrency;
+    this.description = options.description;
+    this.credentialRequirements = options.credentialRequirements ?? [];
+    this.generateItems = options.generateItems;
+    this.caseLabel = options.caseLabel;
+  }
+
+  getCredentialRequirements(): ReadonlyArray<CredentialRequirement> {
+    return this.credentialRequirements;
+  }
+}
+
+export { TestTriggerNode } from "./TestTriggerNode";

package/src/nodes/wait.ts

@@ -8,6 +8,7 @@ export class Wait<TItemJson = unknown> implements RunnableNodeConfig<TItemJson,
   readonly execution = { hint: "local" } as const;
   /** Pass-through empty batches should still advance to downstream nodes. */
   readonly continueWhenEmptyOutput = true as const;
+  readonly icon = "lucide:hourglass" as const;
 
   constructor(
     public readonly name: string,

package/src/chatModels/OpenAIStructuredOutputMethodFactory.ts

@@ -1,46 +0,0 @@
-import type { ChatModelConfig, ChatModelStructuredOutputOptions } from "@codemation/core";
-import { injectable } from "@codemation/core";
-
-import { OpenAIChatModelFactory } from "./OpenAIChatModelFactory";
-
-@injectable()
-export class OpenAIStructuredOutputMethodFactory {
-  private static readonly isoDatePattern = /^\d{4}-\d{2}-\d{2}$/;
-
-  create(chatModelConfig: ChatModelConfig): ChatModelStructuredOutputOptions | undefined {
-    if (chatModelConfig.type !== OpenAIChatModelFactory) {
-      return undefined;
-    }
-    const model = this.readModelName(chatModelConfig);
-    if (!model) {
-      return { method: "functionCalling", strict: true };
-    }
-    return {
-      method: this.supportsJsonSchema(model) ? "jsonSchema" : "functionCalling",
-      strict: true,
-    };
-  }
-
-  private readModelName(chatModelConfig: ChatModelConfig): string | undefined {
-    const candidate = chatModelConfig as Readonly<{ model?: unknown }>;
-    return typeof candidate.model === "string" ? candidate.model : undefined;
-  }
-
-  private supportsJsonSchema(model: string): boolean {
-    if (model === "gpt-4o" || model === "gpt-4o-mini") {
-      return true;
-    }
-    return (
-      this.supportsSnapshotAtOrAfter(model, "gpt-4o-", "2024-08-06") ||
-      this.supportsSnapshotAtOrAfter(model, "gpt-4o-mini-", "2024-07-18")
-    );
-  }
-
-  private supportsSnapshotAtOrAfter(model: string, prefix: string, minimumSnapshotDate: string): boolean {
-    if (!model.startsWith(prefix)) {
-      return false;
-    }
-    const snapshotDate = model.slice(prefix.length);
-    return OpenAIStructuredOutputMethodFactory.isoDatePattern.test(snapshotDate) && snapshotDate >= minimumSnapshotDate;
-  }
-}