@nodaro/shared 3.11.0 → 3.12.0

package/src/workflow-export.ts

@@ -1,5 +1,6 @@
 import type { GenericNode, GenericEdge } from "./types.js"
 import { EXECUTION_DATA_KEYS } from "./node-runtime-keys.js"
+import { SOCIAL_POST_NODE_TYPES } from "./social-post.js"
 
 /** A named media variant (expression, pose, angle, etc.) produced during entity generation. */
 interface AssetVariant {
@@ -192,6 +193,10 @@ const GENERATED_FIELDS: readonly string[] = [
 
 /** Per-node-type extra generated fields beyond GENERATED_FIELDS. Unknown types get no extras ([] default). */
 const NODE_EXTRA_FIELDS: Record<string, string[]> = {
+  // A template must never import ARMED: the switch is the importer's to flip
+  // (a schedule starts paused), and the rules themselves are config that
+  // travels.
+  "schedule-trigger": ["active"],
   character: ["expressions", "poses", "lightingVariations", "angles", "customVariations"],
   object: ["angles", "materials", "variations", "customVariations"],
   creature: ["angles", "poses", "variations", "customVariations"],
@@ -216,9 +221,40 @@ const NODE_EXTRA_FIELDS: Record<string, string[]> = {
   "sub-workflow": ["referencedWorkflowId"],
 }
 
+/**
+ * Node fields that point at a row the IMPORTER does not own — a stored HTTP
+ * credential behind a Webhook Output, a connected social account behind a
+ * publisher. The id alone is useless without owning the row (resolution is
+ * owner-scoped), so nothing leaks either way; the point is that the imported
+ * node lands UNBOUND and the importer picks their own, instead of carrying a
+ * dangling pointer at the exporter's account. Applied by every export shape,
+ * including the asset bundle that otherwise keeps node data verbatim.
+ */
+const UNOWNED_REF_FIELDS: Record<string, readonly string[]> = {
+  "webhook-output": ["credentialId"],
+  ...Object.fromEntries([...SOCIAL_POST_NODE_TYPES].map((type) => [type, ["connectionId"]])),
+}
+
+/** Clear owner-bound references (credential / connection ids). Returns new node objects; inputs are not mutated. */
+export function stripUnownedRefs(nodes: GenericNode[]): GenericNode[] {
+  return nodes.map((node) => {
+    const fields = UNOWNED_REF_FIELDS[node.type]
+    if (!fields || !node.data) return node
+    const data = { ...(node.data as Record<string, unknown>) }
+    let changed = false
+    for (const field of fields) {
+      if (field in data) {
+        delete data[field]
+        changed = true
+      }
+    }
+    return changed ? { ...node, data } : node
+  })
+}
+
 /** Strip generated/transient content from nodes for template export. Returns new node objects; inputs are not mutated. */
 export function stripExportContent(nodes: GenericNode[]): GenericNode[] {
-  return nodes.map((node) => {
+  return stripUnownedRefs(nodes).map((node) => {
     const data = { ...(node.data as Record<string, unknown>) }
     for (const field of GENERATED_FIELDS) delete data[field]
     const extras = NODE_EXTRA_FIELDS[node.type] ?? []