chore(deps): remove unused deps, reduce overall dependencies & size#1436
Merged
waleedlatif1 merged 1 commit intostagingfrom Sep 24, 2025
Merged
chore(deps): remove unused deps, reduce overall dependencies & size#1436waleedlatif1 merged 1 commit intostagingfrom
waleedlatif1 merged 1 commit intostagingfrom
Conversation
Contributor
There was a problem hiding this comment.
Greptile Overview
Summary
This PR removes unused dependencies and dead code to reduce bundle size and improve maintenance. The developer used knip tool to identify unused dependencies and manually verified others.
Key Changes:
- Removed unused component: Deleted
header-links.tsxcomponent that was no longer referenced - Cleaned dependencies: Removed 25+ unused packages including
@chatscope/chat-ui-kit-*, OpenTelemetry web instrumentation packages,@vercel/og,recharts,lenis, and various file parsing libraries - Added missing dependency: Added
@radix-ui/react-tooltipwhich was likely being used but not declared - Updated versions: Bumped
crittersfrom^0.0.23to0.0.25 - Workspace cleanup: Removed unused root-level dependencies like
bcryptjsandgeist
The changes appear systematic and thorough, with verification showing no remaining imports of the removed packages in the codebase.
Confidence Score: 4/5
- This PR is safe to merge with low risk of breaking changes
- Score reflects systematic dependency cleanup with verification that removed packages are truly unused. Minor deduction due to large scope requiring thorough testing
- apps/sim/package.json requires verification that build and runtime still work correctly
Important Files Changed
File Analysis
| Filename | Score | Overview |
|---|---|---|
| apps/sim/package.json | 4/5 | Removed 25+ unused dependencies from production and dev dependencies, added @radix-ui/react-tooltip, updated critters version |
| package.json | 5/5 | Removed bcryptjs, geist, react-colorful, and @types/bcryptjs from root workspace dependencies |
Sequence Diagram
sequenceDiagram
participant Dev as Developer
participant Knip as Knip Tool
participant PM as Package Manager
participant Build as Build System
Dev->>Knip: Run dependency analysis
Knip->>Dev: Report unused dependencies
Dev->>PM: Remove unused packages from package.json
Dev->>PM: Delete unused component files
PM->>PM: Update bun.lock file
Dev->>Build: Verify build still works
Build->>Dev: Confirm successful build
3 files reviewed, no comments
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
knipto find unused dependencies and parsed through the rest manuallyType of Change
Testing
N/A
Checklist