voidzero-dev / voidzero-dev/vite-plus
fix(migrate): detect Yarn PnP config even when env overrides nodeLinker
- 主要语言
- Rust
- 星标
- 5.8k
- 派生
- 262
- 平均合并
- 1 天 34 分钟
- 30 天内合并 PR
- 135
描述
`vp migrate` currently treats `YARN_NODE_LINKER=node-modules` as enough to skip Yarn PnP handling. However, if the repository itself has `.yarnrc.yml` with `nodeLinker: pnp`, migration can complete successfully while leaving unsupported PnP config in the project. Future installs without that environment override will return to PnP mode.
Expected behavior:
- If `.yarnrc.yml` explicitly sets `nodeLinker: pnp`, migration should not silently pass just because `YARN_NODE_LINKER=node-modules` is set.
- Preferably, migration should stop and ask the user to change the project config manually, or apply the existing node-modules conversion flow if that is the intended behavior.
- `YARN_NODE_LINKER=pnp` should still hard-block migration as today.
Suggested test:
Create a Yarn Berry fixture with:
```yaml
# .yarnrc.yml
nodeLinker: pnp
```
Run migrate with:
```bash
YARN_NODE_LINKER=node-modules vp migrate --no-interactive
```
Assert that migration does not report the project as successfully migrated while leaving `nodeLinker: pnp` unchanged.
贡献指南
评估
这个 Issue 还没有评估数据。