Skip to content

feat: add start node validation#807

Merged
hanorik merged 1 commit into
google:wolo/workflowsfrom
hanorik:start_validator
May 12, 2026
Merged

feat: add start node validation#807
hanorik merged 1 commit into
google:wolo/workflowsfrom
hanorik:start_validator

Conversation

@hanorik
Copy link
Copy Markdown
Contributor

@hanorik hanorik commented May 8, 2026

This PR introduces validation rules to ensure the integrity of the workflow graph's entry point.

Key Changes:

  • Start Node Requirement: Validates that the workflow graph contains a Start node as an origin for at least one edge, ensuring the workflow can begin execution and that no edges in the graph point back to the Start node.
  • New Error Types: Introduced ErrNoStartNode and ErrNodePointsToStart in to represent these failure cases.

Tests: Added corresponding unit tests to verify these conditions.

@hanorik hanorik requested review from anFatum and wolo-lab May 8, 2026 10:17
@hanorik hanorik force-pushed the start_validator branch from 2224d29 to 036c514 Compare May 8, 2026 11:20
Comment thread workflow/validation.go
@hanorik hanorik force-pushed the start_validator branch 4 times, most recently from de81398 to 9525891 Compare May 11, 2026 17:21
@hanorik hanorik force-pushed the start_validator branch from 9525891 to b739490 Compare May 11, 2026 17:27
@hanorik hanorik requested a review from wolo-lab May 11, 2026 17:33
@hanorik hanorik merged commit 61068df into google:wolo/workflows May 12, 2026
1 check passed
wolo-lab pushed a commit that referenced this pull request May 18, 2026
wolo-lab pushed a commit that referenced this pull request May 25, 2026
wolo-lab pushed a commit that referenced this pull request Jun 2, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants