[Deps]: Remove unused localtunnel dependency#171
[Deps]: Remove unused localtunnel dependency#171matiaspalmac wants to merge 1 commit intoTrixtyAI:mainfrom
Conversation
The Rust tunnel backend was removed in TrixtyAI#151 along with the disabled ports UI, but the `localtunnel` npm package was left behind in apps/desktop/package.json. It is not imported anywhere in the desktop source. Closes TrixtyAI#112
github-actions
bot
added
the
dependencies
|
Thanks for the contribution! I'll review it as soon as possible. If you have still changes, please mark this PR as draft and all reviews will be cancelled. Tests reviews will be re-run only when the PR is marked as ready for review. |
There was a problem hiding this comment.
Pull request overview
Note
Copilot was unable to run its full agentic suite in this review.
Removes the leftover localtunnel dependency from the desktop app after the tunnel backend/UI removal, reducing transitive dependencies.
Changes:
- Remove
localtunnelfromapps/desktop/package.json.
Files not reviewed (2)
- apps/desktop/package-lock.json: Language not supported
- pnpm-lock.yaml: Language not supported
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
|
@copilot the regenerated lockfiles are in fact part of this PR — see the |
Description
The Rust tunnel backend was removed in #151 along with the disabled ports UI, but the
localtunnelnpm package was left behind inapps/desktop/package.json. There are no remaining imports of it in the desktop frontend, so it and its transitive deps (axios 0.21.4, debug, openurl, yargs, ...) can be dropped safely.Change
localtunnelfromapps/desktop/package.json.pnpm-lock.yamlandapps/desktop/package-lock.json.Related Issue
Closes #112
Verification
grep -r localtunnel apps/desktop/src→ no matches.pnpm installresolves clean.pnpm --filter @trixty/desktop lintpasses.Checklist