fix(flows-create): auto-generate node IDs and remap wires

AI agents supply logical node IDs (like 'inject1') for wiring references,
but those IDs can collide with existing Node-RED node IDs, causing
'duplicate id' errors. Now the tool always generates fresh UUID-style IDs,
builds an old→new remap, and rewires all connections.

Fixes the dogfooding gap where agents had to create a flow empty then
update it — now a single create call works.

Author: SilentDemonSD

src/tools/flows/create.ts

@@ -23,10 +23,25 @@ export function registerCreateFlowTool(server: McpServer, client: NodeRedClient)
     },
     async (args) => {
       const flowId = generateId();
+      const remap = new Map<string, string>();
+      for (const node of args.nodes) {
+        const oldId = (node.id as string) || generateId();
+        remap.set(oldId, generateId());
+      }
+      const nodes = args.nodes.map((n: Record<string, unknown>) => {
+        const oldId = (n.id as string) || "";
+        const next: Record<string, unknown> = { ...n, id: remap.get(oldId) ?? generateId(), z: flowId };
+        if (Array.isArray(next.wires)) {
+          next.wires = next.wires.map((ports: unknown) =>
+            Array.isArray(ports) ? ports.map((tid: string) => remap.get(tid) ?? tid) : ports
+          );
+        }
+        return next;
+      });
       const doc = {
         id: flowId,
         label: args.label,
-        nodes: args.nodes.map((n) => ({ ...n, id: (n.id as string) || generateId(), z: flowId })),
+        nodes,
         configs: args.configs,
         subflows: args.subflows,
       } as unknown as FlowDocument;