Skip to content

Conversation

@IDisposable
Copy link
Contributor

@IDisposable IDisposable commented Nov 7, 2025

Package From To
dayjs 1.11.18 1.11.19
react-hook-form 7.65.0 7.66.0
react-simple-keyboard 3.8.131 3.8.132
validator 13.15.15 13.15.20
@eslint/js 9.39.0 9.39.1
@tailwindcss/postcss 4.1.16 4.1.17
@tailwindcss/vite 4.1.16 4.1.17
@types/validator 13.15.3 13.15.4
@typescript-eslint/eslint-plugin 8.46.2 8.46.3
@typescript-eslint/parser 8.46.2 8.46.3
@vitejs/plugin-react-swc 4.2.0 4.2.1
eslint 9.38.0 9.39.1
globals 16.4.0 16.5.0
tailwindcss 4.1.16 4.1.17
vite 7.1.12 7.2.2 unchanged

@IDisposable
Copy link
Contributor Author

Vite 7.2.2 gives an error when running npm i, something is not right in that version, so did not upgrade that one.

@IDisposable IDisposable requested a review from Copilot November 7, 2025 22:30
@IDisposable IDisposable added component: ui dependencies Pull requests that update a dependency file labels Nov 7, 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 updates the UI package dependencies to their latest versions, bringing bug fixes, security patches, and improvements from upstream packages.

  • Version bump from 2025.10.24.2140 to 2025.11.07.2130
  • Updates to 20+ npm packages including dev tools (eslint, typescript tooling, tailwindcss) and runtime dependencies (dayjs, react-hook-form, validator)
  • Updates to build tooling including esbuild (0.25.11 → 0.25.12), rollup (4.52.5 → 4.53.1), and vite (7.1.12 → 7.2.2)

Reviewed Changes

Copilot reviewed 1 out of 2 changed files in this pull request and generated 2 comments.

File Description
ui/package.json Updates dependency versions for production and development packages, including minor and patch version bumps
ui/package-lock.json Regenerated lockfile reflecting updated dependency versions and their transitive dependencies
Files not reviewed (1)
  • ui/package-lock.json: Language not supported

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

"tailwindcss": "^4.1.16",
"tailwindcss": "^4.1.17",
"typescript": "^5.9.3",
"vite": "^7.1.12",
Copy link

Copilot AI Nov 7, 2025

Choose a reason for hiding this comment

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

The vite version in package.json is specified as ^7.1.12, but the package-lock.json has been updated to version 7.2.2. This version (7.2.2) falls outside the semantic versioning range of ^7.1.12, which only allows patch updates (7.1.x). The package.json should be updated to ^7.2.2 to match the lock file, or the lock file should be regenerated to match the intended version constraint.

Suggested change
"vite": "^7.1.12",
"vite": "^7.2.2",

Copilot uses AI. Check for mistakes.
"version": "2025.11.07.2130",
"type": "module",
"engines": {
"node": "^22.20.0"
Copy link

Copilot AI Nov 7, 2025

Choose a reason for hiding this comment

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

[nitpick] Vite 7.2.2 requires Node.js version ^20.19.0 || >=22.12.0, but the project's engines field in package.json specifies ^22.20.0. This means the minimum Node.js version required by the project (22.20.0) is compatible with vite's requirements. However, note that vite actually allows Node 20.19.0+, so the project's engine constraint is more restrictive than necessary. If this is intentional for other reasons, no change is needed. Otherwise, consider whether the project could support Node 20.19.0+ to align with vite's requirements.

Suggested change
"node": "^22.20.0"
"node": "^20.19.0 || >=22.12.0"

Copilot uses AI. Check for mistakes.
@ym ym merged commit 31ea366 into jetkvm:dev Nov 10, 2025
11 checks passed
@IDisposable IDisposable deleted the chore/ui-package-upgrades branch November 10, 2025 18:43
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

component: ui dependencies Pull requests that update a dependency file

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants