Skip to content

Comments

cleanup unused deps#1489

Merged
naps62 merged 2 commits intomainfrom
cleanup-deps
Oct 30, 2025
Merged

cleanup unused deps#1489
naps62 merged 2 commits intomainfrom
cleanup-deps

Conversation

@naps62
Copy link
Member

@naps62 naps62 commented Oct 30, 2025

No description provided.

Copilot AI review requested due to automatic review settings October 30, 2025 11:38
@naps62 naps62 added the C-chore tedious but necessary label Oct 30, 2025
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull Request Overview

This PR removes unused dependencies and updates the TanStack Router plugin import to use the newer API. The changes involve cleaning up development dependencies that are no longer being used in the codebase and adopting updated naming conventions.

  • Removed 4 unused development dependencies: assert, html-to-image, tsx, vite-plugin-babel-import, and vite-plugin-imp
  • Updated TanStack Router plugin import from deprecated TanStackRouterVite to the newer tanstackRouter function
  • Cleaned up associated transitive dependencies from the lock file

Reviewed Changes

Copilot reviewed 3 out of 4 changed files in this pull request and generated no comments.

File Description
pnpm-lock.yaml Removed lockfile entries for unused dependencies and their transitive dependencies
package.json Removed unused devDependencies: assert, html-to-image, and tsx
gui/package.json Removed unused vite plugin devDependencies: vite-plugin-babel-import and vite-plugin-imp
gui/vite.config.ts Updated TanStack Router plugin import to use newer tanstackRouter naming convention
Files not reviewed (1)
  • pnpm-lock.yaml: Language not supported

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@naps62 naps62 merged commit eeac099 into main Oct 30, 2025
10 checks passed
@naps62 naps62 deleted the cleanup-deps branch October 30, 2025 11:46
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

C-chore tedious but necessary

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant