voidzero-dev / voidzero-dev/vite-plus
Remove allowCompoundComponents migration workaround after upstream support
- Vorherrschende Sprache
- Rust
- Sterne
- 5.8k
- Forks
- 262
- Ø Merge
- 1 T. 34 Min.
- Gemergte PRs (30 T.)
- 135
Beschreibung
PR #2702 adds a temporary migration workaround for `eslint-plugin-react-refresh@0.5.7`. Its Vite preset enables `allowCompoundComponents`. `@oxlint/migrate@1.82.0` copies this option to `react/only-export-components`, but the bundled Oxlint rejects it. Migration therefore removes the option from root and override rules and warns the user.
Remove this workaround when the bundled `@oxlint/migrate` and Oxlint support the option end to end. Migrator support alone is insufficient if Oxlint still rejects the generated configuration. The final behavior must preserve `allowCompoundComponents` and accept the migrated configuration in `vp check`.
- [ ] Update the bundled dependencies to versions that support `allowCompoundComponents`.
- [ ] Remove `stripUnsupportedReactRefreshOption`, its root and override calls, and the related warning from `packages/cli/src/migration/migrator/eslint.ts`.
- [ ] Replace the temporary schema guard and option-removal assertions in `packages/cli/src/migration/__tests__/react-refresh-options.spec.ts` with coverage that preserves the option in root and override rules without a warning.
- [ ] Update the `new_create_vite_migrates_eslint_prettier` snapshot to retain the supported option. Keep its `vp check` step and confirm that the generated React TypeScript project accepts the configuration.
The schema guard in #2702 will fail when the installed Oxlint schema gains the option, so the workaround is reviewed during the dependency update.
Beitragsleitfaden
Rechercherichtung
Start by checking the bundled dependency versions and the schema support for allowCompoundComponents. Read packages/cli/src/migration/migrator/eslint.ts and packages/cli/src/migration/__tests__/react-refresh-options.spec.ts, then update the related new_create_vite_migrates_eslint_prettier snapshot. Done means root and override rules preserve the option without a warning, and the generated React TypeScript project passes its vp check.
Vom Indexierungsmodell aus dem Issue-Text verfasst.
Bewertung
- Tech-Stack
- typescript, vite
- Bereich
- cli, testing, tooling
- Issue-Typ
- Refactoring
- Schwierigkeit
- 4/5
- Geschätzter Aufwand
- 3-5 Tage
- Aktivitätsstatus
- Aktiv
- Klarheit
- Klar beschrieben
- Anfängerfreundlichkeit
- 58/100