@codemation/core-nodes 1.0.0 → 1.0.1

package/CHANGELOG.md

@@ -1,5 +1,28 @@
 # @codemation/core-nodes
 
+## 1.0.1
+
+### Patch Changes
+
+- [#95](https://github.com/MadeRelevant/codemation/pull/95) [`328c975`](https://github.com/MadeRelevant/codemation/commit/328c9759d45b711c177ea9a360ed4960ffdf5ffa) Thanks [@cblokland90](https://github.com/cblokland90)! - Workflow-canvas icon system redesign: LTR-oriented control-flow icons, pixel-perfect Split / Aggregate SVGs, and a single icon renderer shared by the canvas and the execution tree panel.
+
+  **Why**
+
+  The canvas reads left-to-right, but Lucide's `split`, `merge`, and `git-*` family are oriented vertically (top-to-bottom git-graph convention), so `If` and `Merge` nodes rendered 90° off from the flow direction. The execution-tree panel also ran a parallel icon-rendering path that only understood Lucide names — every time a plugin node set an icon as `builtin:<id>`, `si:<slug>`, or a URL, the tree panel silently fell back to a type-substring-guessed Lucide glyph (e.g. `"wait".includes("ai")` → the agent Bot icon). That duplication is gone.
+
+  **What changed**
+  - **Rotation suffix**: `NodeConfigBase.icon` now accepts an optional `@rot=<0|90|180|270>` tail modifier on any icon token (`lucide:`, `builtin:`, `si:`, or URL). Parsed by a strict tail regex so URLs with `@` (for example `http://user@host/icon.svg`) are unaffected, and non-orthogonal angles are rejected so glyphs stay pixel-crisp.
+  - **LTR control-flow icons**:
+    - `If`: `lucide:split@rot=90` — Y-fork with the single leg on the left and two arms fanning right.
+    - `Merge`: `lucide:merge@rot=90` — chevron pointing right, two arms merging from the left.
+  - **Pixel-perfect builtin SVGs** shipped under `packages/next-host/public/canvas-icons/builtin/`:
+    - `builtin:split-rows`: 1 source square on the left, tree trunk / spine, 3 output lines on the right.
+    - `builtin:aggregate-rows`: mirror — 3 input lines on the left converging through a spine into 1 summary square on the right.
+    - Stroke-based SVGs (matching Lucide stroke weight next to them on the canvas).
+  - **`@codemation/core-nodes` built-in node icon updates** that plug into the above: `If` (rotated split), `Merge` (rotated merge), `Split` (`builtin:split-rows`), `Aggregate` (`builtin:aggregate-rows`), `Wait` (`lucide:hourglass`), `MapData` (`lucide:square-pen`), `HttpRequest` (`lucide:globe`), `WebhookTrigger` (`lucide:webhook`), `NoOp` (`lucide:circle-dashed`).
+  - **One renderer, role-only fallback**: `WorkflowExecutionInspectorTreePanelContent` now renders `<WorkflowCanvasNodeIcon />` so `builtin:`, `si:`, URLs, and `@rot=…` resolve identically in the execution tree panel and on the canvas. `WorkflowNodeIconResolver.resolveFallback` shrank from 11 branches of type-substring guessing to 4 role mappings (`agent` / `nestedAgent` → `Bot`, `languageModel` → `Brain`, `tool` → `Wrench`, else → `Boxes`). Plugin nodes that forget to set an icon now get a generic `Boxes` — a clear visual signal rather than a silent substring mismatch.
+  - **New test file** `test/canvas/workflowCanvasNodeIcon.test.tsx` pins 12 behaviours across the rotation parser and the role-only fallback, including a regression case that the pre-fix `"wait".includes("ai")` mis-mapping is impossible now.
+
 ## 1.0.0
 
 ### Major Changes

package/dist/index.cjs

@@ -4049,6 +4049,7 @@ var HttpRequest = class {
 	kind = "node";
 	type = HttpRequestNode;
 	execution = { hint: "local" };
+	icon = "lucide:globe";
 	constructor(name, args = {}, retryPolicy = __codemation_core.RetryPolicy.defaultForHttp) {
 		this.name = name;
 		this.args = args;
@@ -4090,7 +4091,7 @@ var Aggregate = class {
 	type = AggregateNode;
 	execution = { hint: "local" };
 	keepBinaries = true;
-	icon = "lucide:layers";
+	icon = "builtin:aggregate-rows";
 	constructor(name, aggregate, id) {
 		this.name = name;
 		this.aggregate = aggregate;
@@ -4185,7 +4186,7 @@ var If = class {
 	kind = "node";
 	type = IfNode;
 	execution = { hint: "local" };
-	icon = "lucide:split";
+	icon = "lucide:split@rot=90";
 	declaredOutputPorts = ["true", "false"];
 	constructor(name, predicate, id) {
 		this.name = name;
@@ -4250,7 +4251,7 @@ var Split = class {
 	* Mirrors {@link MapData}'s empty-output behavior.
 	*/
 	continueWhenEmptyOutput = true;
-	icon = "lucide:ungroup";
+	icon = "builtin:split-rows";
 	constructor(name, getElements, id) {
 		this.name = name;
 		this.getElements = getElements;
@@ -4321,6 +4322,7 @@ var MapData = class {
 	execution = { hint: "local" };
 	/** Zero mapped items should still allow downstream nodes to run. */
 	continueWhenEmptyOutput = true;
+	icon = "lucide:square-pen";
 	keepBinaries;
 	constructor(name, map, options = {}) {
 		this.name = name;
@@ -4390,7 +4392,7 @@ MergeNode = __decorate([(0, __codemation_core.node)({ packageName: "@codemation/
 var Merge = class {
 	kind = "node";
 	type = MergeNode;
-	icon = "lucide:git-merge";
+	icon = "lucide:merge@rot=90";
 	constructor(name, cfg = { mode: "passThrough" }, id) {
 		this.name = name;
 		this.cfg = cfg;
@@ -4415,6 +4417,7 @@ var NoOp = class {
 	kind = "node";
 	type = NoOpNode;
 	execution = { hint: "local" };
+	icon = "lucide:circle-dashed";
 	constructor(name = "NoOp", id) {
 		this.name = name;
 		this.id = id;
@@ -4520,6 +4523,7 @@ var Wait = class {
 	execution = { hint: "local" };
 	/** Pass-through empty batches should still advance to downstream nodes. */
 	continueWhenEmptyOutput = true;
+	icon = "lucide:hourglass";
 	constructor(name, milliseconds, id) {
 		this.name = name;
 		this.milliseconds = milliseconds;
@@ -4570,7 +4574,7 @@ WebhookTriggerNode = __decorate([(0, __codemation_core.node)({ packageName: "@co
 var WebhookTrigger = class WebhookTrigger {
 	kind = "trigger";
 	type = WebhookTriggerNode;
-	icon = "lucide:globe";
+	icon = "lucide:webhook";
 	constructor(name, args, handler = WebhookTrigger.defaultHandler, id) {
 		this.name = name;
 		this.args = args;