[PR #704] [MERGED] refactor: extract vinext:fix-use-server-closure-collision into its own plugin file #785

New issue

Closed

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

BreizhHardware commented

2026-05-06 13:10:05 +02:00

Owner

📋 Pull Request Information

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

Base: main ← Head: refactor/extract-fix-use-server-closure-collision-plugin

📝 Commits (1)

a88daca refactor: extract vinext:fix-use-server-closure-collision into its own plugin file

📊 Changes

2 files changed (+455 additions, -443 deletions)

View changed files

📝 packages/vinext/src/index.ts (+3 -443)
➕ packages/vinext/src/plugins/fix-use-server-closure-collision.ts (+452 -0)

📄 Description

Summary

Moves the vinext:fix-use-server-closure-collision plugin from an inline object in index.ts into its own file at packages/vinext/src/plugins/fix-use-server-closure-collision.ts, exporting fixUseServerClosureCollisionPlugin
index.ts now imports and references the constant, consistent with how asyncHooksStubPlugin, clientReferenceDedupPlugin, and createOptimizeImportsPlugin are handled
No behaviour change — all 27 existing tests/use-server-closure-collision.test.ts tests continue to pass

_{🔄 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/704 **Author:** [@james-elicx](https://github.com/james-elicx) **Created:** 3/28/2026 **Status:** ✅ Merged **Merged:** 3/28/2026 **Merged by:** [@james-elicx](https://github.com/james-elicx) **Base:** `main` ← **Head:** `refactor/extract-fix-use-server-closure-collision-plugin` --- ### 📝 Commits (1) - [`a88daca`](https://github.com/cloudflare/vinext/commit/a88daca944fa68ff352d572ef9dd4cc52044c6d1) refactor: extract vinext:fix-use-server-closure-collision into its own plugin file ### 📊 Changes **2 files changed** (+455 additions, -443 deletions) <details> <summary>View changed files</summary> 📝 `packages/vinext/src/index.ts` (+3 -443) ➕ `packages/vinext/src/plugins/fix-use-server-closure-collision.ts` (+452 -0) </details> ### 📄 Description ## Summary - Moves the `vinext:fix-use-server-closure-collision` plugin from an inline object in `index.ts` into its own file at `packages/vinext/src/plugins/fix-use-server-closure-collision.ts`, exporting `fixUseServerClosureCollisionPlugin` - `index.ts` now imports and references the constant, consistent with how `asyncHooksStubPlugin`, `clientReferenceDedupPlugin`, and `createOptimizeImportsPlugin` are handled - No behaviour change — all 27 existing `tests/use-server-closure-collision.test.ts` tests continue to pass --- <sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>