Skip to content

revert(migrate): stop merging Oxc configs in existing projects - #2696

Merged
fengmk2 merged 1 commit into
mainfrom
revert/2654-oxc-config-finalization
Sep 14, 2026
Merged

fengmk2 merged 1 commit into
mainfrom
revert/2654-oxc-config-finalization

Conversation

@fengmk2

@fengmk2 fengmk2 commented Sep 14, 2026

Copy link
Copy Markdown
Member

Revert #2654. vp migrate again preserves standalone Oxc configs in projects that already use Vite+.

The added merge step deletes configs that imports or package scripts still use. It also copies JSON extends paths into incompatible inline config. Ecosystem tests reproduced these failures in freecut, delta-comic, pdfme, and sigle.

This revert restores an earlier limitation: interrupted migrations can leave Oxc configs unmerged. #2695 tracks the follow-up fix.

This reverts commit e3bd9db.

Preserve standalone Oxc configs when migrating existing Vite+ projects.
The added merge step can delete referenced configs and copy JSON extends
paths into incompatible inline config.
@cloudflare-workers-and-pages

Copy link
Copy Markdown

🚀 Deploying Preview to Cloudflare 🚀

Preview URL: https://revert-2654-oxc-config-finalization-viteplus-dev.voidzero-docs.workers.dev (commit c03404e)

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://e51d338f-viteplus-dev.voidzero-docs.workers.dev c03404e 2026-09-14T09:03:22.648Z Visit the dashboard ↗

@fengmk2
fengmk2 requested a review from cpojer September 14, 2026 09:04
@fengmk2 fengmk2 self-assigned this Sep 14, 2026
@fengmk2
fengmk2 merged commit 5b30ff0 into main Sep 14, 2026
62 checks passed
@fengmk2
fengmk2 deleted the revert/2654-oxc-config-finalization branch September 14, 2026 09:12
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.

2 participants