Skip to content

Continue As New #245

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Open
wants to merge 11 commits into
base: master
Choose a base branch
from
Open

Continue As New #245

wants to merge 11 commits into from

Conversation

rmcdaniel
Copy link
Contributor

@rmcdaniel rmcdaniel commented Jul 11, 2025

This PR adds support for “continue as new” workflows, updating core classes, model relations, and tests to enable workflows to restart with new input through a ContinuedWorkflow marker.

  • Introduces WorkflowContinuedStatus and a Continues trait to emit a ContinuedWorkflow and transition to the continued state.
  • Updates Workflow, WorkflowStub, and StoredWorkflow to handle active/continued workflows, plus new root/active relations.
  • Adds fixtures and feature tests for continue-as-new, and adjusts CI/env settings to account for failed queue tables.

@rmcdaniel rmcdaniel requested a review from Copilot July 12, 2025 03:47
Copy link

@Copilot Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Reviewed Changes

Copilot reviewed 15 out of 15 changed files in this pull request and generated 3 comments.

Show a summary per file
File Description
src/States/WorkflowContinuedStatus.php Adds new continued status class
src/Traits/Continues.php Implements continueAsNew() logic and promise marker
src/Models/StoredWorkflow.php Adds continuedWorkflows relation and root/active methods
src/Workflow.php Handles ContinuedWorkflow in handle()
src/WorkflowStub.php Routes calls to the active workflow instance
tests/Fixtures/TestContinueAsNewWorkflow.php New fixture for continue-as-new workflow
tests/Feature/ContinueAsNewWorkflowTest.php Feature test validating continue-as-new behavior
tests/Unit/States/WorkflowStatusTest.php Updates state config test to include continued status
tests/TestCase.php Registers failed queue table migration
tests/.env.unit, tests/.env.feature Defines QUEUE_FAILED_DRIVER=null
.github/workflows/php.yml Exports QUEUE_FAILED_DRIVER in CI and adds log artifact
Comments suppressed due to low confidence (1)

.github/workflows/php.yml:78

  • Quoting "null" makes it a string in the CI environment. Remove the quotes so that the runner interprets it as a true null value.
        QUEUE_FAILED_DRIVER: "null"

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
pFad - Phonifier reborn

Pfad - The Proxy pFad of © 2024 Garber Painting. All rights reserved.

Note: This service is not intended for secure transactions such as banking, social media, email, or purchasing. Use at your own risk. We assume no liability whatsoever for broken pages.


Alternative Proxies:

Alternative Proxy

pFad Proxy

pFad v3 Proxy

pFad v4 Proxy