chore(deps): remove webpack and it's deps#2017
chore(deps): remove webpack and it's deps#2017UnschooledGamer merged 1 commit intoAcode-Foundation:mainfrom
Conversation
Greptile SummaryThis PR removes Confidence Score: 5/5Safe to merge — the removal of webpack/webpack-cli is correct since the build system already uses rspack exclusively. Both build and start scripts invoke rspack (not webpack), and rspack.config.js uses @rspack/core directly. No webpack API is imported in source code — the webpackChunkName magic comments in source files are fully supported by rspack's compatibility layer. No P0 or P1 issues identified. No files require special attention.
|
| Filename | Overview |
|---|---|
| package.json | Removes webpack and webpack-cli from devDependencies; build scripts already use rspack exclusively so this is a correct cleanup. |
| package-lock.json | Lock file updated to reflect webpack/webpack-cli removal and reclassifies remaining webpack sub-packages as peer dependencies of other tools. |
Flowchart
%%{init: {'theme': 'neutral'}}%%
flowchart TD
A[npm run build / start] --> B[utils/scripts/build.sh or start.sh]
B --> C[node ./utils/config.js]
B --> D["rspack --mode [development|production]"]
D --> E["@rspack/cli + @rspack/core"]
E --> F[rspack.config.js]
F --> G[Bundled output → www/build/]
X["webpack + webpack-cli (REMOVED)"] -. no longer needed .-> D
style X fill:#ffcccc,stroke:#cc0000,stroke-dasharray: 5 5
style E fill:#ccffcc,stroke:#009900
Reviews (1): Last reviewed commit: "chore(deps): remove webpack and it's dep..." | Re-trigger Greptile
-# this is community PR, it should go under community tag/status in projects and labels too