Skip to content

Remove unused dependencies; bump version to 0.2.3 - #30

Merged
cohogan merged 1 commit into
mainfrom
f/dep-cleanup
Aug 7, 2026
Merged

Remove unused dependencies; bump version to 0.2.3#30
cohogan merged 1 commit into
mainfrom
f/dep-cleanup

Conversation

@cohogan

@cohogan cohogan commented Aug 7, 2026

Copy link
Copy Markdown
Contributor

Summary

  • Remove 8 unused @radix-ui/react-* packages — leftovers from before the shadcn CLI upgrade (9b5caa0) switched all UI components to importing from the radix-ui umbrella package. Zero imports anywhere in src/; the primitives still ship as transitive deps of radix-ui, so runtime is unchanged.
  • Remove @tauri-apps/plugin-log (npm JS bindings only) — nothing in the frontend imports it. The Rust tauri-plugin-log crate stays and logging behavior is unchanged.
  • Bump version to 0.2.3 in package.json, Cargo.toml, tauri.conf.json (also syncs the lagging Cargo.lock version entry).

Effectively a no-op at runtime — also serves to exercise the release build with the new Rust cache from #29 once merged.

Verification

  • pnpm build (tsc + vite) passes with the removals.
  • grep confirms zero imports of any removed package in src/.

Drop 8 individual @radix-ui/react-* packages (components import via the radix-ui umbrella package since the shadcn CLI upgrade) and the @tauri-apps/plugin-log JS bindings (logging is Rust-side only).
@cursor

cursor Bot commented Aug 7, 2026

Copy link
Copy Markdown

Bugbot is not enabled for your account, so this pull request was not reviewed.

Enable Bugbot in the Cursor dashboard to get automatic reviews on future PRs.

@github-actions

github-actions Bot commented Aug 7, 2026

Copy link
Copy Markdown

Claude finished @cohogan's task in 57s —— View job


Code Review: Remove unused dependencies; bump version to 0.2.3

  • Review changed files (package.json, Cargo.toml, Cargo.lock, tauri.conf.json)
  • Verify no remaining imports of removed packages
  • Check version consistency across all three files
  • Post review feedback

Looks good — clean, low-risk PR. Verified the following:

Dependency removals — Confirmed zero imports of any of the 8 removed @radix-ui/react-* packages or @tauri-apps/plugin-log anywhere in the repo (outside pnpm-lock.yaml). The radix-ui umbrella package (^1.4.3) remains as a direct dependency, and the Rust tauri-plugin-log crate stays in Cargo.toml:41 — only the unused JS bindings were dropped.

Version bump — All three version sources are now consistent at 0.2.3:

File Before After
package.json 0.2.2 0.2.3
src-tauri/Cargo.toml 0.2.2 0.2.3
src-tauri/tauri.conf.json 0.2.2 0.2.3
src-tauri/Cargo.lock 0.2.1 (was lagging) 0.2.3

The Cargo.lock version drift (0.2.1 → 0.2.3) is also fixed — good catch.

No issues found. LGTM.

@cohogan
cohogan merged commit 5cd0e03 into main Aug 7, 2026
1 check passed
@cohogan
cohogan deleted the f/dep-cleanup branch August 7, 2026 05:16
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant