Skip to content

[WIP] Fix code based on review comments - #4894

Draft
heavy-d with Copilot wants to merge 3 commits into
mainfrom
copilot/fix-comments-in-review-thread
Draft

[WIP] Fix code based on review comments#4894
heavy-d with Copilot wants to merge 3 commits into
mainfrom
copilot/fix-comments-in-review-thread

Conversation

Copilot AI commented Aug 14, 2026

Copy link
Copy Markdown
Contributor
  • Inspect the PR review thread and identify each requested fix
  • Create a new branch for the review-thread fixes
  • Update the affected workflow store and Electron backend env code
  • Add or adjust focused tests for the review-thread fixes
  • Run targeted verification, secret scan, and final required checks

- Preserve tab metadata in openWorkflows when workflow was edited during
  save (hoist editedDuringSave out of if(nodeStore) block; only update
  updated_at in the tab entry when edits were in flight)
- Adopt server updated_at after saveExample so the next saveWorkflow /
  autosave sends the correct concurrency token instead of a stale one
- Explicitly set NODETOOL_ENV=development in Electron backendEnv to
  prevent inherited production mode from disabling local-only features
- Update electron spawn test to include NODETOOL_ENV requirement
- Add two new test cases: tab metadata preservation and saveExample token adoption

Co-authored-by: heavy-d <3121000+heavy-d@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.

3 participants