Repository navigation
Broken imports after moving files - #293
Open
LuckyTurtleLabs wants to merge 1 commit into
Open
LuckyTurtleLabs wants to merge 1 commit into
LuckyTurtleLabs wants to merge 1 commit into
Conversation
Signed-off-by: LuckyTurtle Labs <258566976+LuckyTurtleLabs@users.noreply.github.com>
This branch has not been deployed
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Issue: #112
Baseline: flydelabs/flyde
913ff75c6781259da35c32fd001a90607756cb24.Implemented a VS Code
onWillRenameFileslistener that returns aWorkspaceEditupdating local Flyde file references. It handles imported code/visual files, moved importers, folders, simultaneous moves, nested inline nodes, and legacy import keys. Only affected YAML scalars change; comments, configuration, package/custom/self sources, and dirty buffers are preserved. Cancellation or changed document versions discard pending edits. This prevents persisted references from retaining the old location.Verification (pnpm 10.12.4, Linux):
pnpm build: full monorepo build passed.pnpm --filter @flyde/loader test: 31 passing, 2 existing pending; includes 13 new regressions and actual code-node resolution before/after moving its file.pnpm --filter @flyde/core test: 135 passing, 8 pending.pnpm --filter flyde-vscode compile-testsandlint: passed; lint has 4 existing warnings.evidence/.pnpm --filter flyde-vscode exec playwright test: 4 screenshot mismatches (114 or 967 pixels). No visual baselines were updated. Seeevidence/visual-tests.log.PATCH.diffapplies cleanly to pristine baseline files and passes reverse-application checking against the modified checkout;git diff --checkpasses.Scope: local files moved through VS Code. External filesystem moves, remote providers, anchored/aliased YAML paths, and open visual-panel lifecycle behavior are not covered. No commits, pushes, PRs, or GitHub comments were made. No project AGENTS.md or CODEX_LAB_CONTEXT.md was present; the lab graph search returned no Flyde matches.
Closes #112
/claim #112
— LuckyTurtle Labs (ShellNex)