Skip to content

Update Flyde imports when referenced files move - #287

Closed
shiv-999 wants to merge 1 commit into
flydelabs:mainfrom
shiv-999:fix/update-moved-flyde-imports
Closed

shiv-999 wants to merge 1 commit into
flydelabs:mainfrom
shiv-999:fix/update-moved-flyde-imports

Conversation

@shiv-999

@shiv-999 shiv-999 commented Sep 25, 2026 •

Copy link
Copy Markdown

Fixes #112.

Fixes broken local .flyde imports after a referenced visual node file is moved in VS Code.

When VS Code reports a .flyde file rename/move, the extension now scans workspace .flyde files and rewrites any matching local relative import path to point at the file's new location. Package imports are left unchanged.

Validation:

  • Added a focused test for updating a moved local visual node import and preserving unrelated imports.
  • Ran git diff --check successfully.

Note: I could not run the full TypeScript test suite locally because this shell does not have Node.js/pnpm available.

@shiv-999

Copy link
Copy Markdown
Author

Closing after local validation found that the focused regression test fails: the implementation reads deprecated top-level imports after loader migrations have removed them, and it does not cover the current source: file representation or moves of the importing flow. Existing submissions already cover the broader rename cases, so this PR should not remain open as a duplicate.

@shiv-999 shiv-999 closed this Sep 26, 2026
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.

Broken imports after moving files

1 participant