-
Notifications
You must be signed in to change notification settings - Fork 10
feat: workflow reruns #78
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
Conversation
- persist run inputs and expose rerun APIs so backend can replay prior configurations - add 'rerun last/this' affordances in run selector + builder and auto-fill forms from saved inputs - refresh execution inspector UI (event cards, toasts, payload parsing) to match the new rerun/debug flow
- remove temporary toast preview trigger and debug layout selector - lock event inspector to stacked-soft layout with flatter diagnostics card - fix timeline duration + step-forward behavior so seeking reaches real run end
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
💡 Codex Review
Here are some automated review suggestions for this pull request.
ℹ️ About Codex in GitHub
Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you
- Open a pull request for review
- Mark a draft as ready
- Comment "@codex review".
If Codex has suggestions, it will comment; otherwise it will react with 👍.
Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".
| setIsLoading(true) | ||
| try { | ||
| await api.workflows.commit(workflowId) | ||
|
|
||
| const runId = await useExecutionStore.getState().startExecution( |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Skip compiling current graph when replaying prior run
The new rerun flow delegates to executeWorkflow, which always calls api.workflows.commit(workflowId) before starting a run. When handleRerun invokes this helper with a stored workflowVersionId, the intent is to replay an archived execution without touching the current builder state. If the current graph contains unsaved or invalid changes, the commit step will fail and the rerun cannot start even though the stored version is valid. This makes “Run again” unreliable whenever the editor has incomplete work. Consider bypassing the commit when a specific versionId is provided (or when the builder isn’t dirty) so reruns operate on the captured version independently of the current graph.
Useful? React with 👍 / 👎.
Summary
Testing
Solves #56