Skip to content

Fix Flyde imports after file moves#265

Open
mdp28 wants to merge 1 commit into
flydelabs:mainfrom
mdp28:flyde-112-update-moved-imports
Open

Fix Flyde imports after file moves#265
mdp28 wants to merge 1 commit into
flydelabs:mainfrom
mdp28:flyde-112-update-moved-imports

Conversation

@mdp28
Copy link
Copy Markdown

@mdp28 mdp28 commented May 15, 2026

/claim #112

Summary

  • Updates .flyde file-source imports after VS Code file/folder rename events.
  • Handles moved imported files, moved importer files, simultaneous moves, folder moves, code-node .flyde.ts imports, legacy imports migrated by deserializeFlow, inline visual nodes, and overlapping rename mappings.
  • Keeps the change scoped to VS Code rename/import maintenance logic.

Verification

  • pnpm --filter flyde-vscode compile-tests passed locally before commit.
  • node .\vscode\node_modules\mocha\bin\mocha.js --ui tdd .\vscode\out\test\flowFileReferences.test.js passed locally: 10 passing.
  • pnpm --filter flyde-vscode lint passed locally before commit with existing unrelated warnings.
  • Re-ran the targeted mocha test after commit: 10 passing.

Demo

The regression tests demonstrate the moved-import flows directly: moved imported flow, moved importing flow, simultaneous moves, folder moves, inline visual nodes, legacy imports, code-node imports, nested rename mappings, and sibling path safety.

@mdp28 mdp28 force-pushed the flyde-112-update-moved-imports branch from ddd4be8 to 1f450c3 Compare May 15, 2026 06:29
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant