#0227 workflows: add v1 if + delay nodes#178
Open
jiggai-automation-bot wants to merge 7 commits into
Open
Conversation
rjdjohnston
force-pushed
the
ticket/0227-if-delay
branch
from
c2d590a to
5a55af4
Compare
rjdjohnston
force-pushed
the
ticket/0227-if-delay
branch
from
5a55af4 to
5605755
Compare
rjdjohnston
force-pushed
the
ticket/0227-if-delay
branch
from
026733d to
30ea58e
Compare
Add first-class if and delay workflow node kinds while maintaining the decomposed module structure from PR #183. if node: - Evaluates action.{lhs, op, rhs} using dot-path references to prior node outputs - Supported operators: truthy, ==, !=, >, >=, <, <=, contains - Edges can use on: 'true' | 'false' for conditional branching - Writes boolean output to node-outputs/ delay node: - Pauses run for action.delaySeconds or action.delayMs (max 7 days) - Sets run status to 'paused' with resumeAt timestamp - Runner/tick automatically resumes when resumeAt passes - No new daemon needed — uses existing runner loop cadence Changes by module: - workflow-if.ts: NEW — evalIfCondition + lastIfValueFromRun - workflow-types.ts: add 'if' | 'delay' to NodeKind, 'true' | 'false' to EdgeOn, resumeAt to RunLog - workflow-utils.ts: pause semantics + true/false edge handling in pickNextRunnableNodeIndex - workflow-node-executor.ts: if + delay node execution in inline runner path - workflow-worker.ts: if + delay node execution in pull-based worker path - workflow-tick.ts: pick up paused runs for resume - workflow-runner.ts: re-export workflow-if (thin orchestrator preserved) 264/264 tests pass, 0 lint errors.
rjdjohnston
force-pushed
the
ticket/0227-if-delay
branch
from
30ea58e to
7c33556
Compare
- Merged workflow-types.ts to include all three node types - Resolved conflicts after rebasing ticket/0227-if-delay onto main
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Implements v1 workflow runner support for if and delay nodes.
Changes
Ticket: development-team #227