Skip to content

Vite 8 Update and dependency fix#348

Closed
PinJinx wants to merge 1 commit into
AOSSIE-Org:mainfrom
PinJinx:chore/dependency-cleanup-and-update
Closed

Vite 8 Update and dependency fix#348
PinJinx wants to merge 1 commit into
AOSSIE-Org:mainfrom
PinJinx:chore/dependency-cleanup-and-update

Conversation

@PinJinx
Copy link
Copy Markdown
Contributor

@PinJinx PinJinx commented May 8, 2026

Addressed Issues:

Fixes #327

Description

This PR updates and stabilizes the frontend tooling ecosystem with a focus on Vite 8 compatibility and dependency cleanup.

Changes Made

  • Upgraded Vite to v8
  • Updated @vitejs/plugin-react to maintain Vite 8 compatibility
  • Updated TanStack Vite/devtools dependencies to compatible versions
  • Updated jsdom to the latest stable version
  • Refreshed and stabilized the dependency graph and lockfile
  • Removed stale/corrupted dependency state caused by previous installs and audit fixes
  • Verified TanStack route generation and lazy-loaded route behavior after the upgrade
  • Verified the application works correctly after the dependency updates

React Compiler

React Compiler migration has not been completed in this PR yet.
This PR focuses on stabilizing the dependency ecosystem and Vite 8 migration first.

Screenshots/Recordings:

N/A

Functional Verification

Screen Mirror

  • Screen Mirror works.

Authentication

  • Connection doesn't work without a valid token.

Basic Gestures

  • One-finger tap: Verified as Left Click.

  • Two-finger tap: Verified as Right Click.

  • Click and drag: Verified selection behavior.

  • Pinch to zoom: Verified zoom functionality (if applicable).

Modes & Settings

  • Cursor mode: Cursor moves smoothly and accurately.

  • Scroll mode: Page scrolls as expected.

  • Sensitivity: Verified changes in cursor speed/sensitivity settings.

  • Copy and Paste: Verified both Copy and Paste functionality.

  • Invert Scrolling: Verified scroll direction toggles correctly.

Advanced Input

  • Key combinations: Verified "hold" behavior for modifiers (e.g., Ctrl+C) and held keys are shown in buffer.

  • Keyboard input: Verified Space, Backspace, and Enter keys work correctly.

  • Glide typing: Verified path drawing and text output.

  • Voice input: Verified speech-to-text functionality for full sentences.

  • Backspace doesn't send the previous input.

Any other gesture or input behavior introduced:

  • New Gestures: Verified any other gesture or input behavior introduced in this PR.

Additional Notes:

This PR intentionally avoids larger ecosystem upgrades such as:

  • Electron major version upgrades
  • TypeScript 6 migration
  • Vitest 4 migration
  • Biome 2 migration

These should be handled in isolated follow-up PRs to reduce migration risk and simplify debugging/review.

Checklist

  • My PR addresses a single issue, fixes a single bug or makes a single improvement.

  • My code follows the project's code style and conventions

  • I have performed a self-review of my own code

  • I have commented my code, particularly in hard-to-understand areas

  • If applicable, I have made corresponding changes or additions to the documentation

  • If applicable, I have made corresponding changes or additions to tests

  • My changes generate no new warnings or errors

  • I have joined the and I will share a link to this PR with the project maintainers there

  • I have read the

  • Once I submit my PR, CodeRabbit AI will automatically review it and I will address CodeRabbit's comments.

  • Incase of UI change I've added a demo video.

Summary by CodeRabbit

  • Chores
    • Updated core development dependencies including Vite (v8), React Vite plugin (v6), and jsdom (v29)
    • Updated TanStack DevTools packages to latest versions
    • Updated Nitro dependency to a pinned beta version
    • Simplified path alias configuration in build settings

@coderabbitai
Copy link
Copy Markdown
Contributor

coderabbitai Bot commented May 8, 2026

Review Change Stack

Walkthrough

This PR upgrades npm dependencies to support Vite 8 and newer React tooling. The nitro runtime is pinned to a beta version, Vite and its React plugin are bumped to major versions, TanStack devtools are updated, and jsdom is upgraded. The Vite configuration is simplified by removing the vite-tsconfig-paths plugin and using explicit path alias mapping instead.

Changes

Dependency and Build Configuration Updates

Layer / File(s) Summary
Dependency Version Updates
package.json
nitro pinned to ^3.0.260429-beta; @vitejs/plugin-react, vite, jsdom, and TanStack devtools packages bumped to newer major/minor versions.
Vite Configuration Simplification
vite.config.ts
vite-tsconfig-paths plugin import and invocation removed; path aliasing now handled via explicit resolve.alias configuration for @./src.

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~10 minutes

Suggested labels

Typescript Lang

Poem

🐰 Dependencies dance in their newest attire,
Vite spins to version eight, climbing higher,
Path aliases simplified, no plugins to parse,
Build configs streamlined with elegant grace!
Hop-hop to faster builds—hooray!

🚥 Pre-merge checks | ✅ 3 | ❌ 1

❌ Failed checks (1 warning)

Check name Status Explanation Resolution
Linked Issues check ⚠️ Warning The PR addresses dependencies update [#327] and Vite 8 migration [#327], but does not complete React compiler migration [#327] as originally requested. Complete React compiler enablement and removal of redundant useCallback calls as specified in #327, or document this as deferred work in the PR description.
✅ Passed checks (3 passed)
Check name Status Explanation
Title check ✅ Passed The title accurately reflects the primary changes: Vite 8 upgrade and dependency updates including package.json and vite.config.ts.
Description check ✅ Passed The PR description provides detailed context on changes, linked issue, and verification steps, though it does not follow the repository's template structure precisely.
Out of Scope Changes check ✅ Passed All code changes (package.json versions and vite.config.ts path aliasing) align with Vite 8 migration and dependency cleanup objectives in #327; no out-of-scope changes detected.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests

Tip

💬 Introducing Slack Agent: The best way for teams to turn conversations into code.

Slack Agent is built on CodeRabbit's deep understanding of your code, so your team can collaborate across the entire SDLC without losing context.

  • Generate code and open pull requests
  • Plan features and break down work
  • Investigate incidents and troubleshoot customer tickets together
  • Automate recurring tasks and respond to alerts with triggers
  • Summarize progress and report instantly

Built for teams:

  • Shared memory across your entire org—no repeating context
  • Per-thread sandboxes to safely plan and execute work
  • Governance built-in—scoped access, auditability, and budget controls

One agent for your entire SDLC. Right inside Slack.

👉 Get started


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.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

Copy link
Copy Markdown
Contributor

@coderabbitai coderabbitai Bot left a comment

Choose a reason for hiding this comment

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

Actionable comments posted: 1

🤖 Prompt for all review comments with AI agents
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.

Inline comments:
In `@package.json`:
- Line 61: Update the vitest dependency in package.json from "vitest": "^3.0.5"
to a 4.1.0+ compatible range (e.g. "vitest": "^4.1.0" or "^4.1.0") to ensure
Vite 8 support; after changing the "vitest" entry, run your package manager
(npm/yarn/pnpm) to update the lockfile and reinstall devDependencies so the
project uses the new Vitest version.
🪄 Autofix (Beta)

Fix all unresolved CodeRabbit comments on this PR:

  • Push a commit to this branch (recommended)
  • Create a new PR with the fixes

ℹ️ Review info
⚙️ Run configuration

Configuration used: Path: .coderabbit.yaml

Review profile: ASSERTIVE

Plan: Pro

Run ID: 98895866-904e-4888-915b-16bc160018da

📥 Commits

Reviewing files that changed from the base of the PR and between b996759 and d0b94c3.

⛔ Files ignored due to path filters (1)
  • package-lock.json is excluded by !**/package-lock.json
📒 Files selected for processing (2)
  • package.json
  • vite.config.ts
💤 Files with no reviewable changes (1)
  • vite.config.ts

Comment thread package.json
"vite": "^7.1.7",
"vite-tsconfig-paths": "^6.0.2",
"vite": "^8.0.11",
"vitest": "^3.0.5",
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

⚠️ Potential issue | 🟠 Major | ⚡ Quick win

🧩 Analysis chain

🏁 Script executed:

#!/bin/bash
# Check the peer dependency range declared by vitest 3 against vite 8
npm info vitest@3 peerDependencies
npm info vitest@4 peerDependencies

Repository: AOSSIE-Org/Rein

Length of output: 12412


Vitest 3.x has no explicit vite peer dependency — but upgrade to vitest 4.1.x+ for guaranteed Vite 8 support

Vitest 3.x does not declare vite as a peer dependency in npm, so no peer dependency warning will occur. However, vitest 4.1.0+ explicitly declares support for vite ^8.0.0. To ensure compatibility with Vite 8.0.11, upgrade to vitest 4.1.0 or later.

🛠️ Proposed fix
-		"vitest": "^3.0.5",
+		"vitest": "^4.1.0",
📝 Committable suggestion

‼️ IMPORTANT
Carefully review the code before committing. Ensure that it accurately replaces the highlighted code, contains no missing lines, and has no issues with indentation. Thoroughly test & benchmark the code to ensure it meets the requirements.

Suggested change
"vitest": "^3.0.5",
"vitest": "^4.1.0",
🤖 Prompt for AI Agents
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.

In `@package.json` at line 61, Update the vitest dependency in package.json from
"vitest": "^3.0.5" to a 4.1.0+ compatible range (e.g. "vitest": "^4.1.0" or
"^4.1.0") to ensure Vite 8 support; after changing the "vitest" entry, run your
package manager (npm/yarn/pnpm) to update the lockfile and reinstall
devDependencies so the project uses the new Vitest version.

@PinJinx PinJinx closed this May 8, 2026
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.

Vite 8 and other dependency updates

1 participant