Skip to content

Conversation

@betterclever
Copy link
Contributor

@betterclever betterclever commented Nov 10, 2025

Summary

  • persist run inputs + expose rerun APIs and UI so last/any run can be relaunched without re-entering data
  • finalize the new Execution Inspector UI (soft stacked event list, diagnostics accordion, toast cleanup) and remove debug-only selectors
  • fix playback timeline duration/step-forward so seeking reaches the true run end, smoothing rerun verification flows

Testing

  • bun --filter=@shipsec/studio-frontend typecheck

Solves #56

- 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
Copy link

@chatgpt-codex-connector chatgpt-codex-connector bot left a 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".

Comment on lines 222 to 226
setIsLoading(true)
try {
await api.workflows.commit(workflowId)

const runId = await useExecutionStore.getState().startExecution(

Choose a reason for hiding this comment

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

P1 Badge 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 👍 / 👎.

@betterclever betterclever changed the title feat: polish workflow reruns experience (#56) feat: workflow reruns Nov 10, 2025
@betterclever betterclever merged commit 0fc2913 into main Nov 10, 2025
1 check passed
@betterclever betterclever mentioned this pull request Nov 10, 2025
26 tasks
@LuD1161 LuD1161 deleted the feat/rerun-ux branch December 19, 2025 13:20
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.

2 participants