@oomol-lab/open-flow 0.1.0-beta.77 → 0.1.0-beta.78

package/src/flow/common/changeSchema.ts

@@ -6,7 +6,7 @@ import { limitDecisionTask } from '../../decision/common/decision.ts'
 import { assertHttpAuthBindings } from '../../http/common/authBindings.ts'
 import { isHttpHost, maxHttpHosts } from '../../http/common/hosts.ts'
 import { httpRequestExecutorSchema } from '../../http/common/httpRequest.ts'
-import { dedupeExecutorSchema } from '../../state/common/state.ts'
+import { skipDuplicatesExecutorSchema } from '../../state/common/state.ts'
 import { checkJsonDepth } from './json.ts'
 import { retryConfigurationSchema } from './retry.ts'
 import { triggerScheduleSchema } from './triggerScheduleSchema.ts'
@@ -130,7 +130,7 @@ const managedFields = z.object({
     }),
     z.strictObject({ kind: z.literal('workflow'), flowId: text, revisionId: text }),
     httpRequestExecutorSchema,
-    dedupeExecutorSchema,
+    skipDuplicatesExecutorSchema,
   ]),
 })
 
@@ -729,10 +729,10 @@ function checkModelVersion(content: RevisionContent): void {
   if (
     content.modelVersion < 13 &&
     Object.values(content.document.graph.nodes).some(
-      (candidateNode) => candidateNode.kind === 'task' && 'executor' in candidateNode.task && candidateNode.task.executor.kind === 'dedupe',
+      (candidateNode) => candidateNode.kind === 'task' && 'executor' in candidateNode.task && candidateNode.task.executor.kind === 'skip-duplicates',
     )
   )
-    throw new Error('Deduplication requires Flow model v13.')
+    throw new Error('Skip Duplicates requires Flow model v13.')
   if (
     content.modelVersion < 12 &&
     Object.values(content.document.graph.nodes).some(

package/src/flow/common/errorHandling.ts

@@ -26,7 +26,7 @@ export const nodeErrorSchema = {
 } as const
 
 export function supportsNodeErrorHandling(node: GraphNode): boolean {
-  return supportsNodeRetry(node) || (node.kind === 'task' && 'executor' in node.task && node.task.executor.kind === 'dedupe')
+  return supportsNodeRetry(node) || (node.kind === 'task' && 'executor' in node.task && node.task.executor.kind === 'skip-duplicates')
 }
 
 export function nodeSystemOutputPorts(node: GraphNode): Readonly<Record<string, PortDefinition>> {

package/src/flow/common/semantics.ts

@@ -2,7 +2,7 @@ import { dequal } from 'dequal/lite'
 import { decisionTaskIssues } from '../../decision/common/decision.ts'
 import { httpRequestIssues, httpAuthHandles, httpRequestTask, httpRequestUrl } from '../../http/common/httpRequest.ts'
 import { openApiIssues, authHandles } from '../../openapi/common/openapi.ts'
-import { dedupeTask } from '../../state/common/state.ts'
+import { skipDuplicatesTask } from '../../state/common/state.ts'
 import { isTriggerNode } from './change.ts'
 import { nodeWorkflowReferences } from './workflowReferences.ts'
 export { renderPrompt } from './promptTemplate.ts'
@@ -274,14 +274,14 @@ export async function validateFlow(revision: RevisionContent, engine: EngineCont
       })
     for (const message of openApiIssues(task))
       checked.diagnostics.push({ code: 'openapi.config-invalid', column: 0, line: 1, message, path: `/document/graph/nodes/${nodeId}/task/executor` })
-    if (task.executor.kind === 'dedupe') {
-      const expected = dedupeTask(task.executor)
+    if (task.executor.kind === 'skip-duplicates') {
+      const expected = skipDuplicatesTask(task.executor)
       if (
         !dequal(httpPortContracts(task.inputs), httpPortContracts(expected.inputs)) ||
         !dequal(httpPortContracts(task.outputs), httpPortContracts(expected.outputs))
       )
         checked.diagnostics.push({
-          code: 'dedupe.ports-invalid',
+          code: 'skip-duplicates.ports-invalid',
           column: 0,
           line: 1,
           message: 'Deduplication ports must match the fixed contract.',
@@ -457,7 +457,9 @@ export async function prepareFlow(revision: RevisionContent, engineContract: str
   if (engine == null) return { kind: 'engine-unsupported' }
   if (
     !engine.flowState &&
-    Object.values(revision.document.graph.nodes).some((node) => node.kind === 'task' && 'executor' in node.task && node.task.executor.kind === 'dedupe')
+    Object.values(revision.document.graph.nodes).some(
+      (node) => node.kind === 'task' && 'executor' in node.task && node.task.executor.kind === 'skip-duplicates',
+    )
   )
     return { kind: 'engine-unsupported' }
   if (
@@ -527,11 +529,15 @@ export async function prepareFlow(revision: RevisionContent, engineContract: str
 
 export function createRuntimeProgram(prepared: PreparedFlow, entryModuleId: string, engineDigest: string): RuntimeProgram | undefined {
   if (!Object.hasOwn(prepared.modules, entryModuleId)) return
+  const modules = new Set([entryModuleId])
+  for (const id of modules) {
+    for (const imported of prepared.modules[id]!.imports) modules.add(imported)
+  }
   return {
     engineContract: prepared.engineContract,
     engineDigest,
     entryModuleId,
-    modules: prepared.modules,
+    modules: Object.fromEntries([...modules].map((id) => [id, prepared.modules[id]!])),
   }
 }