[PR #342] [MERGED] fix: preserve next.config plugin aliases for next-intl #495

New issue

Closed

opened 2026-05-06 13:08:23 +02:00 by BreizhHardware · 0 comments

BreizhHardware commented

2026-05-06 13:08:23 +02:00

Owner

📋 Pull Request Information

Original PR: https://github.com/cloudflare/vinext/pull/342
Author: @NathanDrake2406
Created: 3/8/2026
Status: ✅ Merged
Merged: 3/8/2026
Merged by: @james-elicx

Base: main ← Head: fix/preserve-next-config-plugin-aliases

📝 Commits (3)

27745b6 fix: preserve next.config plugin aliases
48db49c refactor: probe wrapped webpack config once
c8bc0a1 fix: tighten next.config alias warnings

📊 Changes

11 files changed (+571 additions, -73 deletions)

View changed files

📝 packages/vinext/src/check.ts (+1 -1)
📝 packages/vinext/src/config/next-config.ts (+139 -43)
📝 packages/vinext/src/index.ts (+2 -1)
📝 pnpm-lock.yaml (+188 -11)
📝 tests/ecosystem.test.ts (+34 -2)
📝 tests/fixtures/ecosystem/next-intl/app/[locale]/layout.tsx (+10 -2)
📝 tests/fixtures/ecosystem/next-intl/i18n/request.ts (+6 -2)
➕ tests/fixtures/ecosystem/next-intl/next.config.mjs (+5 -0)
📝 tests/fixtures/ecosystem/next-intl/package.json (+1 -0)
📝 tests/next-config.test.ts (+176 -2)
📝 tests/shims.test.ts (+9 -9)

📄 Description

Summary

preserve explicit aliases from wrapped next.config plugins and merge them into Vite resolution
wire the next-intl ecosystem fixture to the upstream plugin-based App Router setup
add config and ecosystem regressions covering /en and /de SSR

Testing

pnpm build
pnpm test tests/ecosystem.test.ts
pnpm test tests/shims.test.ts
pnpm test tests/next-config.test.ts

_{🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.}

## 📋 Pull Request Information **Original PR:** https://github.com/cloudflare/vinext/pull/342 **Author:** [@NathanDrake2406](https://github.com/NathanDrake2406) **Created:** 3/8/2026 **Status:** ✅ Merged **Merged:** 3/8/2026 **Merged by:** [@james-elicx](https://github.com/james-elicx) **Base:** `main` ← **Head:** `fix/preserve-next-config-plugin-aliases` --- ### 📝 Commits (3) - [`27745b6`](https://github.com/cloudflare/vinext/commit/27745b610549f6ee76ace75e572b05ec41758197) fix: preserve next.config plugin aliases - [`48db49c`](https://github.com/cloudflare/vinext/commit/48db49c9fb2688a7acf86ad29f27640829a65dab) refactor: probe wrapped webpack config once - [`c8bc0a1`](https://github.com/cloudflare/vinext/commit/c8bc0a1c951130ebcb0f4245c5a8a43421255842) fix: tighten next.config alias warnings ### 📊 Changes **11 files changed** (+571 additions, -73 deletions) <details> <summary>View changed files</summary> 📝 `packages/vinext/src/check.ts` (+1 -1) 📝 `packages/vinext/src/config/next-config.ts` (+139 -43) 📝 `packages/vinext/src/index.ts` (+2 -1) 📝 `pnpm-lock.yaml` (+188 -11) 📝 `tests/ecosystem.test.ts` (+34 -2) 📝 `tests/fixtures/ecosystem/next-intl/app/[locale]/layout.tsx` (+10 -2) 📝 `tests/fixtures/ecosystem/next-intl/i18n/request.ts` (+6 -2) ➕ `tests/fixtures/ecosystem/next-intl/next.config.mjs` (+5 -0) 📝 `tests/fixtures/ecosystem/next-intl/package.json` (+1 -0) 📝 `tests/next-config.test.ts` (+176 -2) 📝 `tests/shims.test.ts` (+9 -9) </details> ### 📄 Description ## Summary - preserve explicit aliases from wrapped `next.config` plugins and merge them into Vite resolution - wire the `next-intl` ecosystem fixture to the upstream plugin-based App Router setup - add config and ecosystem regressions covering `/en` and `/de` SSR ## Testing - `pnpm build` - `pnpm test tests/ecosystem.test.ts` - `pnpm test tests/shims.test.ts` - `pnpm test tests/next-config.test.ts` --- <sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>