@hyperfixation/workflows 0.1.4 → 0.1.6

package/dist/define-flow.js

@@ -20,6 +20,8 @@ export class UnknownQueue extends Error {
 /** Logged when an attempt finds the run already on a later one; it runs nothing. */
 export const SUPERSEDED_MARKER = "hf-run: superseded attempt, the flow was not run";
 const flows = new Map();
+/** Per name, what `fingerprint` made of the definition that got in first. */
+const fingerprints = new Map();
 /** The one source of a queue name for an enqueue, and of a flow name for `runs.start`. */
 export function definedFlows() {
     return flows;
@@ -33,8 +35,18 @@ export function definedFlows() {
  * The DBOS registration itself happens only in a worker process; see the comment on it.
  */
 export function defineFlow(name, fn, options) {
-    if (flows.has(name))
-        throw new DuplicateFlow(name);
+    // A second call with the *same* definition is one definition reaching here twice, not two
+    // flows fighting over a name: Next instantiates the app's `src/flows/*.ts` once per module
+    // layer — the rsc page layer and the server-action layer of one authenticated request — while
+    // this package stays external and singular, so the registry sees both. That is the first
+    // instance's flow, already registered with DBOS if this is a worker; hand it back rather than
+    // registering it again. Two different definitions of one name are still a collision.
+    const existing = flows.get(name);
+    if (existing !== undefined) {
+        if (fingerprints.get(name) !== fingerprint(fn, options))
+            throw new DuplicateFlow(name);
+        return existing;
+    }
     if (!QUEUES.some((queue) => queue.name === options.queue)) {
         throw new UnknownQueue(name, options.queue);
     }
@@ -76,8 +88,29 @@ export function defineFlow(name, fn, options) {
     const workflow = process.env.HF_PROCESS === WORKER_PROCESS ? DBOS.registerWorkflow(body, { name }) : body;
     const flow = { name, queue: options.queue, workflow };
     flows.set(name, flow);
+    fingerprints.set(name, fingerprint(fn, options));
     return flow;
 }
+/**
+ * What makes two definitions of a name the same definition. Two module copies of one app file
+ * produce distinct function objects with identical source, so identity is useless here and the
+ * text is what there is; a closed-over value that differs between the copies is invisible to it,
+ * which is the known limit of the check.
+ */
+function fingerprint(fn, options) {
+    // Length-prefixed, because a function's source can contain whatever a separator would be.
+    const json = stableJson(options);
+    return `${json.length}:${json}${fn.toString()}`;
+}
+/** `JSON.stringify` with object keys sorted, so key order is not part of the comparison. */
+function stableJson(value) {
+    if (typeof value !== "object" || value === null)
+        return JSON.stringify(value) ?? "undefined";
+    if (Array.isArray(value))
+        return `[${value.map(stableJson).join(",")}]`;
+    const entries = Object.entries(value).sort(([a], [b]) => a < b ? -1 : a > b ? 1 : 0);
+    return `{${entries.map(([key, inner]) => `${JSON.stringify(key)}:${stableJson(inner)}`).join(",")}}`;
+}
 function messageOf(error) {
     return error instanceof Error ? `${error.name}: ${error.message}` : String(error);
 }

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@hyperfixation/workflows",
-  "version": "0.1.4",
+  "version": "0.1.6",
   "license": "MIT",
   "description": "startWorker, defineFlow and the run lifecycle, the step wrapper, approvals, actions, queues, and the reconciler",
   "repository": {
@@ -26,7 +26,7 @@
   ],
   "dependencies": {
     "@dbos-inc/dbos-sdk": "4.27.6",
-    "@hyperfixation/db": "0.1.4",
+    "@hyperfixation/db": "0.1.6",
     "@langfuse/otel": "5.11.1",
     "@opentelemetry/api": "1.9.1",
     "@opentelemetry/context-async-hooks": "2.11.0",
@@ -39,8 +39,8 @@
     "zod": "4.6.5"
   },
   "devDependencies": {
-    "@hyperfixation/eslint-config": "0.1.4",
-    "@hyperfixation/testing": "0.1.4",
+    "@hyperfixation/eslint-config": "0.1.6",
+    "@hyperfixation/testing": "0.1.6",
     "@microsoft/api-extractor": "^7.59.1",
     "@sentry/node": "^10.74.0",
     "@types/pg": "^8.23.1",