@kubun/plugin-workflow 0.17.1 → 0.18.0

package/lib/engine.js

@@ -186,9 +186,12 @@ export function createWorkflowEngine(deps) {
                 handlers: def.handlers,
                 stateValidator: def.stateValidator
             });
+            const initialAction = row.next_action ?? def.initialAction;
+            let currentAction = initialAction;
+            let stepInputState = row.state;
             const flow = generate({
-                state: row.state,
-                action: row.next_action ?? def.initialAction,
+                state: stepInputState,
+                action: initialAction,
                 signal: controller.signal
             });
             events.fire('started', {
@@ -210,11 +213,14 @@ export function createWorkflowEngine(deps) {
                     const held = await s.renew(instanceID, bootID, LEASE_MS, now());
                     if (!held) return;
                     const nextAction = value.status === 'action' ? value.action : null;
+                    const checkpointState = flow.getState();
                     await s.checkpoint(instanceID, {
-                        state: flow.getState(),
+                        state: checkpointState,
                         nextAction,
                         status: 'running'
                     }, now());
+                    stepInputState = checkpointState;
+                    currentAction = nextAction;
                     events.fire('checkpoint', {
                         instanceID,
                         name: def.name
@@ -244,7 +250,7 @@ export function createWorkflowEngine(deps) {
                     fireTerminal(def, instanceID, 'cancelled');
                     return;
                 }
-                // status === 'error'. `getState()` here is the PRE-handler input state.
+                // status === 'error'. Persist the failing step with the state it received.
                 const error = String(value.error);
                 const attempts = row.attempts + 1;
                 const maxAttempts = def.retry?.maxAttempts ?? 1;
@@ -252,8 +258,8 @@ export function createWorkflowEngine(deps) {
                     // Retry rides the queue: re-queue `pending` with a future run gate; the
                     // scheduler re-claims it when due. Not terminal, so no terminal event.
                     await s.fail(instanceID, {
-                        state: flow.getState(),
-                        nextAction: row.next_action,
+                        state: stepInputState,
+                        nextAction: currentAction,
                         error,
                         attempts,
                         runAfter: now() + backoff(attempts, def.retry),
@@ -262,8 +268,8 @@ export function createWorkflowEngine(deps) {
                     return;
                 }
                 const wrote = await s.fail(instanceID, {
-                    state: flow.getState(),
-                    nextAction: row.next_action,
+                    state: stepInputState,
+                    nextAction: currentAction,
                     error,
                     attempts,
                     runAfter: now(),

package/lib/schema.js

@@ -143,6 +143,9 @@ export function createWorkflowContextFactory(api) {
 // connector exposes numeric time (`ConnectorSyncEvent.duration`). `Int` would
 // overflow GraphQL's 32-bit range for a millisecond epoch. `state`/`next_action`
 // have no field here — the projection cannot leak them.
+// Retry rejects missing, active, or non-resumable instances and instances whose
+// singleton key is already held. Cancellation only requests a transition, so
+// the returned status may still be non-terminal.
 const SDL = `
 type WorkflowInstanceStatus {
   instanceID: ID!
@@ -183,12 +186,7 @@ extend type Query {
 extend type Mutation {
   setPeriodicSyncEnabled(id: ID!, enabled: Boolean!): PeriodicSync!
   setPeriodicSyncPolicy(id: ID!, policy: JSON!): PeriodicSync!
-  # Operator re-arm of a failed instance. Errors (not a null result) when the
-  # instance is missing, not failed, has no resumable action, or a live
-  # singleton already holds its key.
   retryWorkflow(instanceID: ID!): WorkflowInstanceStatus!
-  # Operator-initiated cancel. Async: cancel() only requests cancellation, so the
-  # returned status is a re-fetch right after and may not yet be terminal.
   cancelWorkflow(instanceID: ID!): WorkflowInstanceStatus!
 }
 

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@kubun/plugin-workflow",
-  "version": "0.17.1",
+  "version": "0.18.0",
   "keywords": [],
   "license": "see LICENSE.md",
   "sideEffects": false,
@@ -15,23 +15,23 @@
     "LICENSE.md"
   ],
   "dependencies": {
-    "@kubun/db": "^0.17.0",
-    "@kubun/engine": "^0.17.2",
-    "@kubun/graphql": "^0.17.2",
-    "@kubun/id": "^0.17.0",
-    "@kubun/logger": "^0.17.0",
-    "@kubun/plugin-p2p-api": "^0.17.0",
-    "@kubun/plugin-workflow-api": "^0.17.1",
-    "@kubun/store-workflow": "^0.17.1",
-    "@sozai/event": "^0.1.3",
+    "@kubun/db": "^0.18.0",
+    "@kubun/engine": "^0.18.0",
+    "@kubun/graphql": "^0.18.0",
+    "@kubun/id": "^0.18.0",
+    "@kubun/logger": "^0.18.0",
+    "@kubun/plugin-p2p-api": "^0.18.0",
+    "@kubun/plugin-workflow-api": "^0.18.0",
+    "@kubun/store-workflow": "^0.18.0",
+    "@sozai/event": "^0.1.6",
     "@sozai/flow": "^0.3.1",
     "@sozai/runtime": "^0.1.0",
-    "@sozai/schema": "^0.1.2"
+    "@sozai/schema": "^0.1.4"
   },
   "devDependencies": {
-    "@kubun/db-better-sqlite": "^0.17.0",
-    "@kubun/db-postgres": "^0.17.1",
-    "@testcontainers/postgresql": "^12.1.0"
+    "@kubun/db-better-sqlite": "^0.18.0",
+    "@kubun/db-postgres": "^0.18.0",
+    "@testcontainers/postgresql": "^12.2.1"
   },
   "publishConfig": {
     "access": "public"