Skip to content

Broken imports after moving files - #293

Open
LuckyTurtleLabs wants to merge 1 commit into
flydelabs:mainfrom
LuckyTurtleLabs:bounty-112
Open

LuckyTurtleLabs wants to merge 1 commit into
flydelabs:mainfrom
LuckyTurtleLabs:bounty-112

Conversation

@LuckyTurtleLabs

Copy link
Copy Markdown

Issue: #112
Baseline: flydelabs/flyde 913ff75c6781259da35c32fd001a90607756cb24.

Implemented a VS Code onWillRenameFiles listener that returns a WorkspaceEdit updating 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-tests and lint: passed; lint has 4 existing warnings.
  • Dedicated VS Code 1.98.2 extension-host run: 4 passing, including dirty buffers, importer rebasing, a real rename through the registered listener, and cancellation. Used a temporary Xvfb display with GLX disabled to avoid this machine's graphics-driver crashes. Runner and output are in evidence/.
  • pnpm --filter flyde-vscode exec playwright test: 4 screenshot mismatches (114 or 967 pixels). No visual baselines were updated. See evidence/visual-tests.log.
  • PATCH.diff applies cleanly to pristine baseline files and passes reverse-application checking against the modified checkout; git diff --check passes.

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)

Signed-off-by: LuckyTurtle Labs <258566976+LuckyTurtleLabs@users.noreply.github.com>

This branch has not been deployed

No deployments
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