Skip to content

Fix nested parallel transition targets - #7

Merged
SandroMaglione merged 1 commit into
mainfrom
codex/fix-nested-parallel-targets
Aug 1, 2026
Merged

Fix nested parallel transition targets#7
SandroMaglione merged 1 commit into
mainfrom
codex/fix-nested-parallel-targets

Conversation

@SandroMaglione

Copy link
Copy Markdown
Member

What changed

  • Require complete region selection when target.local or target.branch enters an inactive nested parallel state.
  • Preserve partial branch updates for parallel states that are already active.
  • Carry complete nested target subtrees through normalization while preserving outer active parallel regions and ancestor values.
  • Document the semantics and add a patch changeset so the Changesets release PR (Version Packages #6) updates after merge.

Root cause

Local and branch target builders represented every destination as a single leaf path. TypeScript therefore accepted an incomplete nested parallel target, but runtime snapshot construction later failed because one or more active regions were missing.

Verification

  • pnpm check
  • 196 runtime tests passed.
  • 88 type tests / 337 assertions passed.
  • Strict consumer and package checks passed.

@SandroMaglione
SandroMaglione marked this pull request as ready for review August 1, 2026 13:12
@SandroMaglione
SandroMaglione merged commit 4ec33e1 into main Aug 1, 2026
2 checks passed
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.

1 participant