chore(deps): remove unused tmp dependency and other unused packages#341
Merged
Conversation
Codecov Report✅ All modified and coverable lines are covered by tests. 📢 Thoughts on this report? Let us know! |
Contributor
Security NoticeThis PR updates tmp to v0.2.6 to address a security vulnerability. All CI checks are passing and the PR is mergeable. Recommendation: Ready for merge. Maintainers @taltas @hannesrudolph — please review when possible. |
470be01 to
0b44719
Compare
edelauna
previously approved these changes
May 29, 2026
Contributor
Author
Edited/Blocked NotificationRenovate will not automatically rebase this PR, because it does not recognize the last commit author and assumes somebody else may have edited the PR. You can manually request rebase by checking the rebase/retry box above. |
fb9a2e4 to
360b46a
Compare
360b46a to
0299646
Compare
0299646 to
2cd8601
Compare
2cd8601 to
94b2d81
Compare
edelauna
approved these changes
May 30, 2026
Contributor
edelauna
left a comment
There was a problem hiding this comment.
Cleaning up dependency radius
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
Investigation of the Renovate security alert for
tmp(CVE-2025-54798, CVE-2026-44705) revealed thattmpwas declared as a dependency but never imported anywhere in the codebase. Rather than bumping a package we don't use, we removed it entirely.A broader audit of unused dependencies was performed at the same time.
Changes
Removed dependencies (
src/package.json):tmp— never imported; was the subject of the security alertscheerio,default-shell,node-ipc— no source imports (hits were only in compileddist/or stale test mocks)diff-match-patch,jwt-decode,pkce-challenge— no imports foundpuppeteer-chromium-resolver,puppeteer-core— no imports foundsound-play,stream-json,string-similarity,strip-ansi,turndown— no imports foundRemoved devDependencies:
@types/diff-match-patch,@types/node-ipc,@types/stream-json,@types/string-similarity,@types/turndown,@types/tmpTest cleanup:
vi.mock("default-shell", ...)fromsystem-prompt.spec.tsandadd-custom-instructions.spec.tsTesting
suite.skip), 0 failures