Skip to content

fix(migrate): safely merge duplicate imports - #3439

Merged
fengmk2 merged 2 commits into
mainfrom
triagebot/fix-3430
Nov 12, 2026
Merged

fengmk2 merged 2 commits into
mainfrom
triagebot/fix-3430

Conversation

@voidzero-bot

@voidzero-bot voidzero-bot Bot commented Nov 12, 2026 •

Copy link
Copy Markdown
Contributor

vp migrate can create duplicate vite-plus imports when it rewrites a vite import. These duplicates cause import/no-duplicates errors.

Merge adjacent imports from the same module when migration changes at least one import source. Preserve quoted export names and aliases. Keep import type for groups that contain only type declarations, and use inline type specifiers for mixed groups.

Keep intervening code, side-effect imports, and comments intact. Leave unsupported declarations, conflicting bindings, and unchanged import groups separate.

Closes #3430

@cloudflare-workers-and-pages

cloudflare-workers-and-pages Bot commented Nov 12, 2026 •

Copy link
Copy Markdown

🚀 Deploying Preview to Cloudflare 🚀

Preview URL: https://triagebot-fix-3430-viteplus-dev.voidzero-docs.workers.dev (commit e7224c2)

This URL reflects your latest Preview deployment

Preview Deployments by commit

Status Deployment URL Commit Updated (UTC) See this deployment's details
  • Build: Success ✅
  • Deployment: Success ✅

View logs ↗
https://823f9396-viteplus-dev.voidzero-docs.workers.dev e7224c2 2026-11-12T10:58:34.976Z Visit the dashboard ↗
  • Build: Success ✅
  • Deployment: Success ✅

View logs ↗
https://d762a412-viteplus-dev.voidzero-docs.workers.dev 7a49ffd 2026-11-12T10:05:56.271Z Visit the dashboard ↗

@github-actions

github-actions Bot commented Nov 12, 2026 •

Copy link
Copy Markdown
Contributor

CLI artifact sizes (e7224c2)

Final release artifacts built by the canonical build-upstream and build-windows-cli actions.
The dist rows use the Linux build. The core total excludes .node files to match the release artifact.

Artifact Format Base PR Change
packages/cli/dist Directory total 2.34 MiB 2.34 MiB 0 B (0.00%)
packages/core/dist Directory total 4.11 MiB 4.11 MiB 0 B (0.00%)
Combined package dist Directory total 6.45 MiB 6.45 MiB 0 B (0.00%)
vp (Linux x64) Binary 8.35 MiB 8.35 MiB 0 B (0.00%)
vp (Linux x64) gzip -9 3.82 MiB 3.82 MiB 0 B (0.00%)
NAPI (Linux x64) Binary 36.78 MiB 36.79 MiB +12.00 KiB (+0.03%)
NAPI (Linux x64) gzip -9 14.70 MiB 14.70 MiB +8.54 KiB (+0.06%)
vp (macOS ARM64) Binary 5.46 MiB 5.46 MiB 0 B (0.00%)
vp (macOS ARM64) gzip -9 3.25 MiB 3.25 MiB 0 B (0.00%)
NAPI (macOS ARM64) Binary 43.43 MiB 43.43 MiB 0 B (0.00%)
NAPI (macOS ARM64) gzip -9 18.86 MiB 18.87 MiB +1.25 KiB (+0.01%)
vp (Windows x64) Binary 6.19 MiB 6.19 MiB 0 B (0.00%)
vp (Windows x64) gzip -9 2.86 MiB 2.86 MiB +1 B (+0.00%)
NAPI (Windows x64) Binary 32.81 MiB 32.82 MiB +10.50 KiB (+0.03%)
NAPI (Windows x64) gzip -9 13.45 MiB 13.46 MiB +6.33 KiB (+0.05%)
Trampoline (Windows x64) Binary 13.50 KiB 13.50 KiB 0 B (0.00%)
Trampoline (Windows x64) gzip -9 7.09 KiB 7.09 KiB +1 B (+0.01%)
Installer (Windows x64) Binary 4.68 MiB 4.68 MiB 0 B (0.00%)
Installer (Windows x64) gzip -9 2.25 MiB 2.25 MiB +3 B (+0.00%)

@fengmk2 fengmk2 self-assigned this Nov 12, 2026
@socket-security

Copy link
Copy Markdown

Review the following changes in direct dependencies. Learn more about Socket for GitHub.

Diff Package Supply Chain
Security
Vulnerability Quality Maintenance License
Addednpm/​@​voidzero-dev/​vite-plus-core@​1.5.0981007798100
Addednpm/​vite-plus@​1.5.09810010099100

View full report

@fengmk2 fengmk2 changed the title Merge duplicate imports created by vp migrate fix(migrate): safely merge duplicate imports Nov 12, 2026
@fengmk2 fengmk2 added test: e2e Auto run e2e tests test: install-e2e run vite install e2e test test: create-e2e Run `vp create` e2e tests labels Nov 12, 2026
@fengmk2
fengmk2 merged commit fc32837 into main Nov 12, 2026
163 of 184 checks passed
@fengmk2
fengmk2 deleted the triagebot/fix-3430 branch November 12, 2026 12:29
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

fix verified test: create-e2e Run `vp create` e2e tests test: e2e Auto run e2e tests test: install-e2e run vite install e2e test

Projects

None yet

Development

Successfully merging this pull request may close these issues.

vp migrate adds a second vite-plus import when it rewrites a vite import

1 participant