aglinxinyuan opened a new pull request, #6156: URL: https://github.com/apache/texera/pull/6156
### What changes were proposed in this PR? Remove the developer-only `analyze` npm script and its `webpack-bundle-analyzer` devDependency from the frontend. | File | Change | | --- | --- | | `frontend/package.json` | Drop the `"analyze"` script and the `"webpack-bundle-analyzer": "4.5.0"` devDependency | | `frontend/yarn.lock` | Regenerated — removes `webpack-bundle-analyzer` and its **exclusive** transitive deps (`commander`, `duplexer`, `gzip-size`, `mrmime`, `sirv`, `totalist`, `ws@^7.3.1`). Deps shared with other packages (`acorn`, `lodash`, `opener`, `@polka/url`, `acorn-walk`) stay — only their merged version-descriptors are trimmed | | `frontend/README.md` | Drop the now-stale `Bundle analyzer → yarn analyze` row from the "Common commands" table | The `analyze` script — `ng build --configuration=production --stats-json && webpack-bundle-analyzer dist/stats.json` — is a manual, on-demand bundle-inspection helper. It is not invoked by any build, CI, container, or test path, and `webpack-bundle-analyzer` is not imported programmatically anywhere in the repo, so both the script and its devDependency are safe to drop. ### Any related issues, documentation, discussions? Split out of #4997 per review feedback in https://github.com/apache/texera/pull/4997#discussion_r3524565737, where this removal was flagged as unrelated to the monaco-languageclient v10 upgrade. That PR restores the script; this PR removes it independently so the change is reviewed on its own. ### How was this PR tested? - **Lockfile consistency:** regenerated `frontend/yarn.lock` with `yarn install --mode=update-lockfile` (Yarn 4.14.1). A second pass produced no further changes, so CI's `yarn install --immutable` stays green. - **Scope check:** confirmed the `yarn.lock` diff is limited to the `webpack-bundle-analyzer` subtree — shared transitive deps remain; only merged version-descriptors were trimmed (e.g. `acorn`'s `^8.0.4` spec dropped, the rest kept). - **No dangling references:** whole-repo grep for `webpack-bundle-analyzer` / `BundleAnalyzerPlugin` (no source imports), no custom webpack config under `frontend/`, and no hits in `.github/workflows/**`, Dockerfiles, shell scripts, `nx.json`, `angular.json`, or any `LICENSE-binary` (dev-only tool, so no license-manifest entry to sync). - `frontend/package.json` re-validated as parseable JSON after the edit. ### Was this PR authored or co-authored using generative AI tooling? Generated-by: Claude Code (Opus 4.8 [1M context]) -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
