Skip to content

Workflow history branch 1#11946

Merged
jordanrfrazier merged 14 commits into
workflow-historyfrom
workflow-history-branch-1
Mar 2, 2026
Merged

Workflow history branch 1#11946
jordanrfrazier merged 14 commits into
workflow-historyfrom
workflow-history-branch-1

Conversation

@jordanrfrazier
Copy link
Copy Markdown
Collaborator

I know this fixes a few critical issues - allowing save of an empty state, versions overwriting preview state of current draft, and restoring auto-save functionality. Not sure about all the frontend changes though.

Fix version restore overwriting with draft and harden sidebar lifecycle

  - Fix didRestore flag being immediately cleared by clearPreview(),
    which caused restore to silently revert to the current draft
  - Isolate each sidebar cleanup step in its own try-catch so a failure
    in one doesn't skip auto-save restoration
  - Clone flow before processFlows to prevent caller mutation
  - Split restore error handling into API phase (user-facing) and
    cleanup phase (logged only)
  - Guard against undefined nodes/edges in draft ref initialization
  - Remove empty-flow save guard that prevented saving after deleting
    the last node
  - Remove dead code: unused previewKey field, redundant clearPreview
    in SaveSnapshotButton

@coderabbitai
Copy link
Copy Markdown
Contributor

coderabbitai Bot commented Feb 28, 2026

Important

Review skipped

Auto reviews are disabled on base/target branches other than the default branch.

Please check the settings in the CodeRabbit UI or the .coderabbit.yaml file in this repository. To trigger a single review, invoke the @coderabbitai review command.

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

Use the checkbox below for a quick retry:

  • 🔍 Trigger review
✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment
  • Commit unit tests in branch workflow-history-branch-1

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

@jordanrfrazier jordanrfrazier merged commit dd2dc66 into workflow-history Mar 2, 2026
9 of 12 checks passed
@jordanrfrazier jordanrfrazier deleted the workflow-history-branch-1 branch March 2, 2026 14:57
github-merge-queue Bot pushed a commit that referenced this pull request Mar 4, 2026
* initial implementation of flow history

* Save and restore correctly

* Fit to screen when restoring

* Change confirmation to pop up

* correectly export even when version is not selected

* remove unused code

* [autofix.ci] apply automated fixes

* [autofix.ci] apply automated fixes (attempt 2/3)

* use auto snapshot from activate in fe

* [autofix.ci] apply automated fixes

* [autofix.ci] apply automated fixes (attempt 2/3)

* Remove flow state -- not needed until integration with deploymnet

* [autofix.ci] apply automated fixes

* Strip keys from exported history; add some deepcopies for safety

* [autofix.ci] apply automated fixes

* Unique version numbers and tests for stripping keys

* rename flow history entry

* [autofix.ci] apply automated fixes

* Add prune warning dialog, add domain-specific exceptions

* Move version control to left side bar

* fix remove_api_key function, remove dupe code

* Remove full-history download for now, cleanup

* merged updated FE

* Remove the keep building from version page, fix double save, remove not needed timeout

* fixes keep buildig

* merge migrations

* [autofix.ci] apply automated fixes

* [autofix.ci] apply automated fixes (attempt 2/3)

* hide log canvas controls and flow toolbar when preview is active

* [autofix.ci] apply automated fixes

* updated styles

* initial implementation of flow history

* Save and restore correctly

* Fit to screen when restoring

* Change confirmation to pop up

* correectly export even when version is not selected

* remove unused code

* [autofix.ci] apply automated fixes

* use auto snapshot from activate in fe

* [autofix.ci] apply automated fixes

* [autofix.ci] apply automated fixes (attempt 2/3)

* Remove flow state -- not needed until integration with deploymnet

* [autofix.ci] apply automated fixes

* Strip keys from exported history; add some deepcopies for safety

* [autofix.ci] apply automated fixes

* Unique version numbers and tests for stripping keys

* rename flow history entry

* [autofix.ci] apply automated fixes

* Add prune warning dialog, add domain-specific exceptions

* Move version control to left side bar

* fix remove_api_key function, remove dupe code

* merged updated FE

* Remove full-history download for now, cleanup

* Remove the keep building from version page, fix double save, remove not needed timeout

* fixes keep buildig

* merge migrations

* hide log canvas controls and flow toolbar when preview is active

* [autofix.ci] apply automated fixes

* [autofix.ci] apply automated fixes (attempt 2/3)

* updated styles

* [autofix.ci] apply automated fixes

* [autofix.ci] apply automated fixes (attempt 2/3)

* removed keep building button

* Workflow history branch 1 (#11946)

* Update language from history to version

* more language updates history -> version

* more language updates history -> version

* capture original draft at mount / restore

* Renames files from history to version

* clean up dead code, some more renames

* more naming fixes

* more naming fixes

* Fix API endpoint language

* [autofix.ci] apply automated fixes

* version -> versions

* ignore warning

---------

Co-authored-by: Deon Sanchez <69873175+deon-sanchez@users.noreply.github.com>
Co-authored-by: autofix-ci[bot] <114827586+autofix-ci[bot]@users.noreply.github.com>

* Update migration table naming

* fix import and update log to error on prune failure

* more import fixes

* fix migration paths

* more his->ver naming updates

* [autofix.ci] apply automated fixes

* [autofix.ci] apply automated fixes (attempt 2/3)

* Add checkbox to save draft or not

* add scroll, leave saved

* [autofix.ci] apply automated fixes

* fix migration path and some merge conflicts

* ruff

* remove max flow version size

* [autofix.ci] apply automated fixes

* Add index to created_at

---------

Co-authored-by: autofix-ci[bot] <114827586+autofix-ci[bot]@users.noreply.github.com>
Co-authored-by: Deon Sanchez <69873175+deon-sanchez@users.noreply.github.com>
HimavarshaVS pushed a commit that referenced this pull request Mar 10, 2026
* initial implementation of flow history

* Save and restore correctly

* Fit to screen when restoring

* Change confirmation to pop up

* correectly export even when version is not selected

* remove unused code

* [autofix.ci] apply automated fixes

* [autofix.ci] apply automated fixes (attempt 2/3)

* use auto snapshot from activate in fe

* [autofix.ci] apply automated fixes

* [autofix.ci] apply automated fixes (attempt 2/3)

* Remove flow state -- not needed until integration with deploymnet

* [autofix.ci] apply automated fixes

* Strip keys from exported history; add some deepcopies for safety

* [autofix.ci] apply automated fixes

* Unique version numbers and tests for stripping keys

* rename flow history entry

* [autofix.ci] apply automated fixes

* Add prune warning dialog, add domain-specific exceptions

* Move version control to left side bar

* fix remove_api_key function, remove dupe code

* Remove full-history download for now, cleanup

* merged updated FE

* Remove the keep building from version page, fix double save, remove not needed timeout

* fixes keep buildig

* merge migrations

* [autofix.ci] apply automated fixes

* [autofix.ci] apply automated fixes (attempt 2/3)

* hide log canvas controls and flow toolbar when preview is active

* [autofix.ci] apply automated fixes

* updated styles

* initial implementation of flow history

* Save and restore correctly

* Fit to screen when restoring

* Change confirmation to pop up

* correectly export even when version is not selected

* remove unused code

* [autofix.ci] apply automated fixes

* use auto snapshot from activate in fe

* [autofix.ci] apply automated fixes

* [autofix.ci] apply automated fixes (attempt 2/3)

* Remove flow state -- not needed until integration with deploymnet

* [autofix.ci] apply automated fixes

* Strip keys from exported history; add some deepcopies for safety

* [autofix.ci] apply automated fixes

* Unique version numbers and tests for stripping keys

* rename flow history entry

* [autofix.ci] apply automated fixes

* Add prune warning dialog, add domain-specific exceptions

* Move version control to left side bar

* fix remove_api_key function, remove dupe code

* merged updated FE

* Remove full-history download for now, cleanup

* Remove the keep building from version page, fix double save, remove not needed timeout

* fixes keep buildig

* merge migrations

* hide log canvas controls and flow toolbar when preview is active

* [autofix.ci] apply automated fixes

* [autofix.ci] apply automated fixes (attempt 2/3)

* updated styles

* [autofix.ci] apply automated fixes

* [autofix.ci] apply automated fixes (attempt 2/3)

* removed keep building button

* Workflow history branch 1 (#11946)

* Update language from history to version

* more language updates history -> version

* more language updates history -> version

* capture original draft at mount / restore

* Renames files from history to version

* clean up dead code, some more renames

* more naming fixes

* more naming fixes

* Fix API endpoint language

* [autofix.ci] apply automated fixes

* version -> versions

* ignore warning

---------

Co-authored-by: Deon Sanchez <69873175+deon-sanchez@users.noreply.github.com>
Co-authored-by: autofix-ci[bot] <114827586+autofix-ci[bot]@users.noreply.github.com>

* Update migration table naming

* fix import and update log to error on prune failure

* more import fixes

* fix migration paths

* more his->ver naming updates

* [autofix.ci] apply automated fixes

* [autofix.ci] apply automated fixes (attempt 2/3)

* Add checkbox to save draft or not

* add scroll, leave saved

* [autofix.ci] apply automated fixes

* fix migration path and some merge conflicts

* ruff

* remove max flow version size

* [autofix.ci] apply automated fixes

* Add index to created_at

---------

Co-authored-by: autofix-ci[bot] <114827586+autofix-ci[bot]@users.noreply.github.com>
Co-authored-by: Deon Sanchez <69873175+deon-sanchez@users.noreply.github.com>
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