chore: simplify template package names#127
Conversation
There was a problem hiding this comment.
Pull request overview
This PR simplifies the name fields inside the Biome, ESLint (per-variant), and Prettier template package.json files by removing the older rsbuild-* prefixes, aiming to keep template manifests aligned with tool/variant naming.
Changes:
- Renamed
template-prettier/package.json#namefromrsbuild-prettier→prettier. - Renamed
template-biome/package.json#namefromrsbuild-biome→biome. - Renamed ESLint variant template package names from
rsbuild-eslint-*→eslint-*across React/Vue/Svelte/Vanilla variants.
Reviewed changes
Copilot reviewed 10 out of 10 changed files in this pull request and generated 2 comments.
Show a summary per file
| File | Description |
|---|---|
| template-prettier/package.json | Simplifies the Prettier template package name. |
| template-biome/package.json | Simplifies the Biome template package name. |
| template-eslint/react-ts/package.json | Simplifies ESLint React TS template package name. |
| template-eslint/react-js/package.json | Simplifies ESLint React JS template package name. |
| template-eslint/vue-ts/package.json | Simplifies ESLint Vue TS template package name. |
| template-eslint/vue-js/package.json | Simplifies ESLint Vue JS template package name. |
| template-eslint/svelte-ts/package.json | Simplifies ESLint Svelte TS template package name. |
| template-eslint/svelte-js/package.json | Simplifies ESLint Svelte JS template package name. |
| template-eslint/vanilla-ts/package.json | Simplifies ESLint Vanilla TS template package name (but currently uses common naming). |
| template-eslint/vanilla-js/package.json | Simplifies ESLint Vanilla JS template package name (but currently uses common naming). |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
| @@ -1,5 +1,5 @@ | |||
| { | |||
| "name": "rsbuild-eslint-common-ts", | |||
| "name": "eslint-common-ts", | |||
There was a problem hiding this comment.
The ESLint template variant folder is vanilla-ts (and the code refers to the variant name vanilla-ts), but this package.json name uses eslint-common-ts. For consistency with the other variants (eslint-react-ts, eslint-vue-ts, etc.) and with the variant identifier, consider renaming this to eslint-vanilla-ts (or otherwise aligning the naming scheme across variants).
| "name": "eslint-common-ts", | |
| "name": "eslint-vanilla-ts", |
| @@ -1,5 +1,5 @@ | |||
| { | |||
| "name": "rsbuild-eslint-common-js", | |||
| "name": "eslint-common-js", | |||
There was a problem hiding this comment.
The ESLint template variant folder is vanilla-js (and the code refers to the variant name vanilla-js), but this package.json name uses eslint-common-js. For consistency with the other variants (eslint-react-js, eslint-svelte-js, etc.) and with the variant identifier, consider renaming this to eslint-vanilla-js (or otherwise aligning the naming scheme across variants).
| "name": "eslint-common-js", | |
| "name": "eslint-vanilla-js", |
Summary
rsbuild-*package names in the Biome, ESLint, and Prettier templates with shorter template package namesTesting
Related Links