Skip to content

Fix ESLint config#191

Merged
mcmire merged 1 commit intomainfrom
fix-eslint-config-2
Jan 7, 2026
Merged

Fix ESLint config#191
mcmire merged 1 commit intomainfrom
fix-eslint-config-2

Conversation

@mcmire
Copy link
Copy Markdown
Contributor

@mcmire mcmire commented Jan 7, 2026

The *.tsx files in src/ui were not being properly linted. Now they are, and lint violations have been corrected. Additionally, vite.config.mjs was renamed to vite.config.mts so that it could be properly linted as well.


Note

Strengthens linting and applies UI code cleanups.

  • Expands ESLint to cover *.cjs, *.ts, *.tsx, *.mts and adds React rules for src/ui/**/*.tsx; updates ignore patterns
  • Updates lint:eslint to lint more extensions and adds eslint-plugin-react
  • UI TSX files: apply lint-driven refactors (JSDoc, safer conditionals, remove unnecessary void, improved names), import ./style.css for Tailwind, and minor logic tidy-ups
  • Minor formatting cleanup in babel.config.cjs
  • Regenerates yarn.lock for new/updated deps

Written by Cursor Bugbot for commit a7f83c4. This will update automatically on new commits. Configure here.

The *.tsx files in src/ui were not being properly linted. Now they are,
and lint violations have been corrected. Additionally, `vite.config.mjs`
was renamed to `vite.config.mts` so that it could be properly linted as
well.
@mcmire mcmire requested a review from a team as a code owner January 7, 2026 18:46
@socket-security
Copy link
Copy Markdown

socket-security bot commented Jan 7, 2026

Review the following changes in direct dependencies. Learn more about Socket for GitHub.

Diff Package Supply Chain
Security
Vulnerability Quality Maintenance License
Addedeslint-plugin-react@​7.37.59910010083100

View full report

@socket-security
Copy link
Copy Markdown

socket-security bot commented Jan 7, 2026

All alerts resolved. Learn more about Socket for GitHub.

This PR previously contained dependency changes with security issues that have been resolved, removed, or ignored.

Ignoring alerts on:

  • string.prototype.repeat@1.0.0
  • async-function@1.0.0

View full report

@mcmire
Copy link
Copy Markdown
Contributor Author

mcmire commented Jan 7, 2026

@SocketSecurity ignore npm/async-function@1.0.0

ljharb is a known NPM author.

@mcmire
Copy link
Copy Markdown
Contributor Author

mcmire commented Jan 7, 2026

@SocketSecurity ignore npm/string.prototype.repeat@1.0.0

nicolo-ribaudo is a known NPM author.

@mcmire mcmire merged commit 9c6c63c into main Jan 7, 2026
16 checks passed
@mcmire mcmire deleted the fix-eslint-config-2 branch January 7, 2026 20:26
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