Skip to content

Backport #4408: fix(core): start() with an explicit deploymentId outside a deployment - #4425

Merged
pranaygp merged 1 commit into
stablefrom
backport/pr-4408-to-stable
Sep 25, 2026
Merged

pranaygp merged 1 commit into
stablefrom
backport/pr-4408-to-stable

Conversation

@github-actions

Copy link
Copy Markdown
Contributor

Automated backport of #4408 to stable (backport job run).

AI recommendation: This is a patch-level bug fix to start() in @workflow/core, which exists on stable with the same defect: packages/core/src/runtime/start.ts on stable also calls world.getDeploymentId() unconditionally (line 178) before honoring opts.deploymentId, so Replay Run / recreateRunFromExisting fails from any process that is not itself a deployment. The cross-deployment start path it falls back to already exists on stable, and the change adds no new API surface — behavior is unchanged when the current deployment is available, and the accompanying tests cover existing behavior.

Merge conflicts were resolved by AI (opencode with anthropic/claude-opus-5). Please review the conflict resolution carefully before merging.

…#4408)

recreateRunFromExisting (Replay Run) from a process that is not itself a
deployment failed because start() called world.getDeploymentId()
unconditionally. With a concrete target, an unavailable current deployment
now means a cross-deployment start instead of an error.

Co-authored-by: Claude <noreply@anthropic.com>
Signed-off-by: Pranay Prakash <pranay.gp@gmail.com>
@changeset-bot

changeset-bot Bot commented Sep 25, 2026

Copy link
Copy Markdown

🦋 Changeset detected

Latest commit: 4b8d404

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 16 packages
Name Type
@workflow/core Patch
@workflow/builders Patch
@workflow/cli Patch
@workflow/next Patch
@workflow/nitro Patch
@workflow/vitest Patch
@workflow/web-shared Patch
@workflow/web Patch
workflow Patch
@workflow/world-testing Patch
@workflow/astro Patch
@workflow/nest Patch
@workflow/rollup Patch
@workflow/sveltekit Patch
@workflow/vite Patch
@workflow/nuxt Patch

Not sure what this means? Click here to learn what changesets are.

Click here if you're a maintainer who wants to add another changeset to this PR

@pranaygp
pranaygp merged commit b23170d into stable Sep 25, 2026
22 of 23 checks passed
@pranaygp
pranaygp deleted the backport/pr-4408-to-stable branch September 25, 2026 20:55
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