Skip to content

docs(claude): record the ESLint 10 blocker alongside TypeScript 7 - #413

Merged
ValgulNecron merged 1 commit into
masterfrom
docs/eslint10-blocker
Sep 21, 2026
Merged

ValgulNecron merged 1 commit into
masterfrom
docs/eslint10-blocker

Conversation

@ValgulNecron

Copy link
Copy Markdown
Collaborator

Summary

The start-of-session check in CLAUDE.md covered a single blocked upgrade (TypeScript 7) and was named after it. A second blocker has the same shape, so the section now carries one entry per blocker, each deleted independently when its own wall clears.

The ESLint 10 wall

Dependabot #386 (eslint 9.39.5 → 10.x) fails at npm ci:

eslint-plugin-react@7.37.5   peer eslint: ^3 || ^4 || ^5 || ^6 || ^7 || ^8 || ^9.7   ← no ^10
eslint-plugin-react-hooks@7.1.1          … || ^9.0.0 || ^10.0.0                       ✓
@typescript-eslint/eslint-plugin@8.69.0  ^8.57.0 || ^9.0.0 || ^10.0.0                 ✓
@typescript-eslint/parser@8.69.0         ^8.57.0 || ^9.0.0 || ^10.0.0                 ✓

npm ci is step 4 of the web job, so it dies in ~10 seconds before eslint or any test runs, and web e2e (mock) and design vs browser visual diff fail identically — each begins with the same install.

eslint-plugin-react is the sole holdout; the other three plugins are already ready.

#387 is explicitly not blocked

Worth recording because it is counter-intuitive: #387 (@eslint/js 9 → 10) installs cleanly and its web jobs pass. The plugin's peer range constrains eslint itself, not @eslint/js, so the two do not move together. I initially assumed they did; the entry now says otherwise so the next session does not re-derive it wrongly.

Check command

npm view eslint-plugin-react@latest version peerDependencies.eslint

Blocked while that range lacks ^10. No workarounds — --legacy-peer-deps, overrides and pinning tricks are all ruled out by rule 4, and #386 stays open.

Docs only; no code, config or dependency changes.

🤖 Generated with Claude Code

https://claude.ai/code/session_01ESdYuAvpfkcZLPQ5TfDW55

The start-of-session check covered one blocked upgrade and was named
after it. A second one has the same shape, so the section now holds an
entry per blocker, each deleted independently when its own wall clears.

eslint 9 -> 10 (#386) cannot install: eslint-plugin-react@7.37.5 caps
its eslint peer range at ^9.7, so npm ci fails at step 4 of the web job
in about ten seconds, taking web e2e (mock) and the visual diff with it
since each starts with the same install. The other three plugins
already accept ^10, so the plugin is the sole holdout.

#387 (@eslint/js 9 -> 10) is deliberately called out as NOT blocked —
its web jobs pass, because the plugin's peer range constrains eslint
itself rather than @eslint/js. I had assumed the two moved together and
they do not, which is worth recording so the next session does not
re-derive it wrongly.

Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01ESdYuAvpfkcZLPQ5TfDW55
Signed-off-by: valgulnecron <39313199+ValgulNecron@users.noreply.github.com>
@coderabbitai

coderabbitai Bot commented Sep 21, 2026

Copy link
Copy Markdown

Important

  • 🔍 Trigger review

This repository does not receive automatic reviews because it has fewer than 10 stars.

⚙️ Run configuration

Configuration used: Repository: ValgulNecron/Gameplane/.coderabbit.yaml

Review profile: CHILL

Plan: Advanced

Run ID: 7594238c-2482-45f3-a182-600cdd9361fb


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.

@ValgulNecron ValgulNecron added area: shared netguard, gameaction, gameproto, svcutil and other shared packages type: docs Documentation only labels Sep 21, 2026
@ValgulNecron
ValgulNecron merged commit f0d5fd1 into master Sep 21, 2026
5 of 6 checks passed
@ValgulNecron
ValgulNecron deleted the docs/eslint10-blocker branch September 25, 2026 23:40
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

area: shared netguard, gameaction, gameproto, svcutil and other shared packages type: docs Documentation only

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant