Skip to content

Prevent loop self-edges from racing body execution - #338

Merged
ckakgun merged 2 commits into
mainfrom
codex/fix-loop-race-condition-lock
Jul 16, 2026
Merged

Prevent loop self-edges from racing body execution#338
ckakgun merged 2 commits into
mainfrom
codex/fix-loop-race-condition-lock

Conversation

@heym-coder

Copy link
Copy Markdown
Collaborator

Summary

  • ignore canvas loop self-edges when building the executable graph
  • advance loop state only from distinct body-node back-connections
  • add normal and streaming regressions for the supplied five-item workflow
  • verify every set execution completes before the next iteration

Validation

  • uv run ruff format --check .
  • uv run ruff check .
  • full backend suite: 2,140 tests and 170 subtests passed
  • ./check.sh frontend stage unavailable because Bun is not installed

heym-coder and others added 2 commits July 16, 2026 14:33
Fixed the loop race by excluding self-referential loop-back edges from execution and accepting iteration feedback only from actual body nodes. Added normal and streaming regression tests using the supplied five-item DSL, verifying sequential set outputs 1–5.

Validation:
Backend Ruff formatting/lint passed. Full sequential backend suite passed: 2,140 tests and 170 subtests. ./check.sh could not run frontend checks because Bun is unavailable; file-limit checks passed. The parallel test wrapper exceeded local resources, but the complete sequential suite passed.
@ckakgun
ckakgun enabled auto-merge (squash) July 16, 2026 13:37
@ckakgun
ckakgun merged commit bbae939 into main Jul 16, 2026
2 checks passed
@ckakgun
ckakgun deleted the codex/fix-loop-race-condition-lock branch July 16, 2026 13:38
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

2 participants