chore(deps): remove unused dependencies#516
Conversation
Several declared dependencies are never imported in source, invoked by an npm script, or referenced in any config (verified by a full source-tree scan plus a clean tsc build): dependencies: file-type, remark, remark-gfm, remark-parse, unified devDependencies: commander, ts-node, nexe Removing them drops ~4,900 net lockfile lines -- the unified/remark markdown stack, file-type, and nexe's large, stalled-beta subtree (archiver@5, got, caw, download, @yarnpkg/*, app-builder, ...). nexe is a single-executable builder that nothing in the repo invokes; its `latest` dist-tag is still a 2025 beta. Kept (depcheck flags them, but they ARE used): @tiptap/pm -- a required peerDependency of @tiptap/core and @tiptap/extension-table; and @anthropic-ai/mcpb -- invoked by the build:mcpb script via `npx @anthropic-ai/mcpb validate|pack`. Verified: with all eight removed, npm install resolves, `npm run build` (tsc over the full source tree) succeeds -- proving nothing in src imports them -- and exceljs round-trips pass. Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
|
No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configurationConfiguration used: defaults Review profile: CHILL Plan: Pro Run ID: ⛔ Files ignored due to path filters (1)
📒 Files selected for processing (1)
💤 Files with no reviewable changes (1)
📝 WalkthroughWalkthrough
ChangesDependency Cleanup
Estimated code review effort🎯 1 (Trivial) | ⏱️ ~2 minutes Poem
🚥 Pre-merge checks | ✅ 5✅ Passed checks (5 passed)
✏️ Tip: You can configure your own custom pre-merge checks in the settings. ✨ Finishing Touches🧪 Generate unit tests (beta)
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
What
Removes 8 dependencies that are declared but never used:
file-type,remark,remark-gfm,remark-parse,unifiedcommander,ts-node,nexeWhy
None of these are imported in source, invoked by an npm script, or referenced in any config. Removing them drops ~4,900 net lockfile lines — the unified/remark markdown stack,
file-type, andnexe's large stalled-beta subtree (archiver@5,got,caw,download,@yarnpkg/*,app-builder, …).nexe'slatestdist-tag is still a 2025 beta and nothing in the repo invokes it.Kept (depcheck flags them, but they ARE used)
@tiptap/pm— a requiredpeerDependencyof@tiptap/coreand@tiptap/extension-table.@anthropic-ai/mcpb— invoked by thebuild:mcpbscript vianpx @anthropic-ai/mcpb validate|pack.Verification
With all eight removed:
npm installresolves,npm run build(tsc over the full source tree) succeeds — proving nothing insrcimports them — and exceljs round-trips pass.Summary by CodeRabbit