Skip to content

chore(client): migrate to ESLint 10 - #187

Merged
koydas merged 3 commits into
mainfrom
claude/eslint-10-migration
Oct 1, 2026
Merged

koydas merged 3 commits into
mainfrom
claude/eslint-10-migration

Conversation

@koydas

@koydas koydas commented Sep 27, 2026

Copy link
Copy Markdown
Owner

Closes #185. Supersedes #172.

Changes

  • eslint 8.57 → 10.11.
  • Declare @eslint/js (10.0.1) and globals (17.x) as explicit devDependencies: client/eslint.config.js imports both, but they were only available transitively through ESLint 8, which is why chore(deps-dev): bump eslint from 8.57.1 to 10.11.0 in /client #172 failed with ERR_MODULE_NOT_FOUND: @eslint/js.
  • No config or source change needed: the flat config already in place is ESLint 10 compatible.

Validation (local, client/)

  • npm run lint: clean, 33 files linted; a probe file with debugger; is reported (exit 1), so the config is effective, not empty.
  • npx vitest run: 26/26. npm run build: OK.
  • Lockfile: all resolved from registry.npmjs.org, no new install scripts.

Checklist

  • All CI checks pass
  • README.md updated if behaviour changed — N/A, devDependencies only
  • ADR created or updated if an architectural decision was made — N/A, applies ADR-004
  • New service registered in discover-services.js and smoke tests added if applicable — N/A

🤖 Generated with Claude Code

https://claude.ai/code/session_01HcKxmDJ1KBXLKgbTxp7Wtx


Generated by Claude Code

Bump eslint 8.57 -> 10.11 and declare @eslint/js and globals, which
client/eslint.config.js imports but were only available transitively
through ESLint 8. Supersedes Dependabot #172.

Refs #185

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01HcKxmDJ1KBXLKgbTxp7Wtx
npm install re-sorted devDependencies; restore the original order so the
diff only carries the eslint bump and the two added packages.

Refs #185

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01HcKxmDJ1KBXLKgbTxp7Wtx
Resolve client/package-lock.json conflict with React 19 (#188) and
vite/vitest patch (#189) by regenerating from main's lockfile with
npm install.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01AjJSKb3q2o1MHd4gRs98tL
@koydas
koydas merged commit af5cd23 into main Oct 1, 2026
5 checks passed
koydas pushed a commit that referenced this pull request Oct 1, 2026
koydas pushed a commit that referenced this pull request Oct 1, 2026
…205)

- CI tests each service on its Dockerfile's runtime (enforced by
  runtime-versions.test.js since #184).
- Coupled upgrades land together: grouped in dependabot.yml, split PRs
  superseded by one migration PR (#187, #188, #202).
- Database images follow ADR-005.
- Group dotnet/* Docker images so sdk and aspnet bump in one PR
  (they were split into #195/#196).

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01AjJSKb3q2o1MHd4gRs98tL
koydas added a commit that referenced this pull request Oct 1, 2026
…206)

- CI tests each service on its Dockerfile's runtime (enforced by
  runtime-versions.test.js since #184).
- Coupled upgrades land together: grouped in dependabot.yml, split PRs
  superseded by one migration PR (#187, #188, #202).
- Database images follow ADR-005.
- Group dotnet/* Docker images so sdk and aspnet bump in one PR
  (they were split into #195/#196).


Claude-Session: https://claude.ai/code/session_01AjJSKb3q2o1MHd4gRs98tL

Co-authored-by: Claude <noreply@anthropic.com>
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.

[FEATURE] - Migrate client to ESLint 10

2 participants