Skip to content

Conversation

@IDisposable
Copy link
Contributor

@IDisposable IDisposable commented Nov 11, 2025

Summary

Upgraded all the ESLint and enforced the current rule set to get consistent code style across the UI codebase. It primarily consists of changes automatically applied by ESLint and Prettier.

  • Enforces consistent code style: Applies a comprehensive lint fix pass to adhere to updated ESLint rules and the current Vite project build structure.
  • Improves readability: Reorders Tailwind CSS classes for consistency and makes minor adjustments to whitespace and code structure.
  • No functional changes: These modifications are purely stylistic and do not alter the behavior or functionality of the application.

Details

  • Created a new Vite project with all the required packages and synched up the structure
  • Move the eslint.config.cjs to eslint.config.js as this was the only .cjs file in the whole system
  • Follow the current vite practice of split tsconfig.json into tsconfig.app.json and tsconfig.node.json
  • Added lint:only and lint:fixonly when no localization needs compiling
  • Eliminated unused lodash.throttle package
  • Moved the eslint-import-resolver-alias to the devDependences
  • Add missing typescript-eslint to devDependencies
  • Don't force blank lines between import groups (but allow them)
  • Allow slightly longer lines
  • Upgraded to Node 22.21.1 (current for LTS 22)
  • Upgraded packages
Package From To Notes
lodash.throttle 4.1.1 deleted Was not referenced
react-hook-form 7.66.0 7.66.1
react-router 7.9.5 7.9.6
react-simple-keyboard 3.8.132 3.8.133
recharts 3.3.0 3.4.1
tailwind-merge 3.3.1 3.4.0
validator 13.15.20 13.15.23
@eslint/compat 1.4.1 2.0.0
@types/react 19.2.3 19.2.6
@types/react-dom 19.2.2 19.2.3
@types/validator 13.15.4 13.15.10
@typescript-eslint/eslint-plugin 8.46.3 8.47.0
@typescript-eslint/parser 8.46.3 8.47.0
@vitejs/plugin-react-swc 4.2.1 4.2.2
autoprefixer 10.4.21 10.4.22
eslint-import-resolver-alias 1.1.2 1.1.2 Moved to devDependencies
typescript-eslint added 8.47.0 was missing
vite 7.1.12 7.2.2

@IDisposable
Copy link
Contributor Author

This changes everything ;) It was just a run of npm run lint:fix after all the package cleanup (pulling in vite 7.2.2 made a lot of things complain in eslint).

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 applies ESLint and Prettier formatting rules across the UI codebase, following current Vite project standards. The changes include splitting the TypeScript configuration into separate app and node configs, upgrading Node to 22.21.1, updating several packages, removing unused dependencies, and applying consistent code formatting (quote styles, spacing, line breaks, trailing commas). The changes are purely stylistic with no intended functional modifications.

Reviewed Changes

Copilot reviewed 125 out of 130 changed files in this pull request and generated 1 comment.

Show a summary per file
File Description
ui/tsconfig.json Restructured to reference split configs
ui/tsconfig.app.json New file with app-specific TypeScript config
ui/tsconfig.node.json Enhanced with additional compiler options
ui/src/**/*.ts(x) Applied consistent formatting: spacing, quotes, line breaks, trailing commas
ui/src/keyboardLayouts/*.ts Standardized semicolons and trailing commas

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

@IDisposable IDisposable requested review from adamshiervani and ym and removed request for adamshiervani November 11, 2025 23:36
@IDisposable
Copy link
Contributor Author

This one-time pass will enable us to put the eslint check that PR #491 was attempting to enable :)

Move from .cjs to .js file for eslint
Allow slightly longer lines
Don't force blank lines between import groups (but allow them)
Added lint:only and lint:fixonly when no localization needs compiling
Eliminated  unused lodash.throttle package.
Moved the eslint-import-resolver-alias to the devDependences
Add missing typescript-eslint to devDependencies
Model the split tsconfig*.json for modern vite projects.
Upgraded packages.
@IDisposable IDisposable force-pushed the chore/es-lint-redux branch 2 times, most recently from 51252d6 to 2cc6626 Compare November 20, 2025 03:51
Bumped devcontainer
Bump more packages
@IDisposable
Copy link
Contributor Author

Rebased on current dev and conflicts resolved.

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

Copilot reviewed 126 out of 131 changed files in this pull request and generated no new comments.


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

@IDisposable
Copy link
Contributor Author

Rebased

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.

2 participants