Remove unused npm dependencies: @netlify/blobs and fflate#3889
Remove unused npm dependencies: @netlify/blobs and fflate#3889
Conversation
✅ Deploy Preview for kubestellarconsole ready!
To edit notification comments on pull requests, go to your Netlify project configuration. |
|
👋 Hey @Copilot — thanks for opening this PR!
This is an automated message. |
❌ PR Title Verification FailedYour PR title does not follow the required format. Current title: Required FormatPR titles must start with one of these emoji prefixes:
How to FixEdit your PR title to start with the appropriate emoji. For example:
You can edit the title by clicking the Edit button next to your PR title. This comment was automatically posted by the PR Title Verifier workflow. |
Signed-off-by: copilot <copilot@github.com> Agent-Logs-Url: https://github.com/kubestellar/console/sessions/2159a82c-bbb7-4331-9f32-1596307a3a24 Co-authored-by: clubanderson <407614+clubanderson@users.noreply.github.com>
|
[APPROVALNOTIFIER] This PR is NOT APPROVED This pull-request has been approved by: The full list of commands accepted by this bot can be found here. DetailsNeeds approval from an approver in each of these files:Approvers can indicate their approval by writing |
|
Thanks for your pull request. Before we can look at it, you'll need to add a 'DCO signoff' to your commits. 📝 Please follow instructions in the contributing guide to update your commits with the DCO Full details of the Developer Certificate of Origin can be found at developercertificate.org. The list of commits missing DCO signoff:
DetailsInstructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes-sigs/prow repository. I understand the commands that are listed here. |
There was a problem hiding this comment.
Pull request overview
Removes two npm dependencies from the web/ frontend package that were flagged as unused, reducing dependency surface area and transitive installs.
Changes:
- Removed
@netlify/blobsfromweb/package.jsonand pruned its transitive dependencies fromweb/package-lock.json. - Removed
fflatefromweb/package.json(while leaving any transitivefflatedependencies intact in the lockfile).
Reviewed changes
Copilot reviewed 1 out of 2 changed files in this pull request and generated 1 comment.
| File | Description |
|---|---|
web/package.json |
Drops @netlify/blobs and fflate from direct dependencies. |
web/package-lock.json |
Updates the lockfile to reflect the removed direct deps and prunes now-unneeded transitive packages. |
Files not reviewed (1)
- web/package-lock.json: Language not supported
| "@dnd-kit/core": "^6.3.1", | ||
| "@dnd-kit/sortable": "^10.0.0", | ||
| "@dnd-kit/utilities": "^3.2.2", | ||
| "@netlify/blobs": "^10.7.4", | ||
| "@react-three/drei": "^9.122.0", | ||
| "@react-three/fiber": "^8.17.10", |
There was a problem hiding this comment.
The PR description/body should start with Fixes #<issue-number> so GitHub can automatically link/close the tracked issue when this PR is merged (project workflow requirement).
🔄 Auto-Applying Copilot Code ReviewCopilot code review found 0 code suggestion(s) and 1 general comment(s). Also address these general comments:
Push all fixes in a single commit. Run Auto-generated by copilot-review-apply workflow. |
Auto-QA flagged 5 potentially unused dependencies. Investigation found 3 are legitimately required; 2 have no usage anywhere in the codebase and are safe to drop.
Dependency audit results
@netlify/blobssrc/fflatesrc/; transitive dependents bundle it themselvesreact-isrechartsat build timesucraselib/dynamic-cards/compiler.ts@sqlite.org/sqlite-wasmlib/cache/worker.tsThe Auto-QA scan missed the last two because they use
await import(...)syntax rather than static top-level imports.Changes Made
@netlify/blobsfromweb/package.jsonfflatefromweb/package.jsonChecklist
Please ensure the following before submitting your PR:
git commit -s)Screenshots or Logs (if applicable)
Build passes cleanly after removal (
tsc -b && vite build✓).👀 Reviewer Notes
react-islooks unused by source code but is required byrecharts— removing it breaks the Vite/Rollup build with an unresolved import error. Keep it.💬 Send tasks to Copilot coding agent from Slack and Teams to turn conversations into code. Copilot posts an update in your thread when it's finished.