ci: fix eslint 10 peer conflict + click PYSEC-2026-2132 (run #30854734500) - #155
Draft
Sbussiso wants to merge 1 commit into
Draft
ci: fix eslint 10 peer conflict + click PYSEC-2026-2132 (run #30854734500)#155Sbussiso wants to merge 1 commit into
Sbussiso wants to merge 1 commit into
Conversation
…4500) Run #30854734500 on dependabot/npm_and_yarn/frontend/eslint/js-10.0.1 failed two jobs independently: 1. Frontend "Install dependencies" (npm ci) — ERESOLVE peer conflict. Dependabot bumped @eslint/js 9.39.4 -> 10.0.1, whose peerOptional is eslint ^10.0.0, but eslint was still pinned at ^9.39.4. npm ci (strict) refuses the conflicting peer. Bump eslint ^9.39.4 -> ^10.0.0 to match; eslint-plugin-react-hooks (^7.1.1) and eslint-plugin-react-refresh (^0.5.3) both already accept eslint ^10 in their peer ranges. Regenerated package-lock.json (resolves eslint 10.8.0, @eslint/js 10.0.1). 2. Backend "Dependency scan (pip-audit --strict)" — PYSEC-2026-2132 in click 8.3.1 (transitive via uvicorn), fixed in 8.3.3. Follows the existing constraint-dependencies pattern in backend/pyproject.toml. Added "click>=8.3.3" constraint; uv lock resolves click 8.4.2. Verified locally against the exact CI steps: - frontend: npm ci / vitest (85 passed) / vite build — all green - backend: uv run pip-audit --strict — "No known vulnerabilities found" Note: the pre-existing react-router GHSA-qwww-vcr4-c8h2 advisory (high) also shows in npm audit on master today and is NOT addressed here — it predates this PR and is a breaking downgrade (7.18.2 -> 7.11.0) that belongs in its own change.
Sbussiso
referenced
this pull request
Aug 3, 2026
Bumps [@eslint/js](https://github.com/eslint/eslint/tree/HEAD/packages/js) from 9.39.4 to 10.0.1. - [Release notes](https://github.com/eslint/eslint/releases) - [Commits](https://github.com/eslint/eslint/commits/v10.0.1/packages/js) --- updated-dependencies: - dependency-name: "@eslint/js" dependency-version: 10.0.1 dependency-type: direct:development update-type: version-update:semver-major ... Signed-off-by: dependabot[bot] <support@github.com>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
CI fix for run #30854734500 (PR #75)
Run #30854734500 on
dependabot/npm_and_yarn/frontend/eslint/js-10.0.1failed two jobs independently. This branch fixes both.1. Frontend
npm ci— ERESOLVE peer conflictDependabot bumped
@eslint/js9.39.4→10.0.1, whosepeerOptionaliseslint ^10.0.0. Buteslintwas still pinned at^9.39.4, sonpm ci(strict) refused the conflicting peer dep:Fix: bump
eslint^9.39.4→^10.0.0infrontend/package.jsonand regenerate the lockfile.eslint-plugin-react-hooks(^7.1.1) andeslint-plugin-react-refresh(^0.5.3) both already accepteslint ^10in their peer ranges. Resolveseslint@10.8.0+@eslint/js@10.0.1.2. Backend
pip-audit --strict— PYSEC-2026-2132 in clickclick 8.3.1(transitive viauvicorn's CLI runner) has advisory PYSEC-2026-2132, fixed in8.3.3.pip-audit --strictflagged it and failed the deploy:Fix: follow the existing
constraint-dependenciespattern inbackend/pyproject.toml— add"click>=8.3.3"and regenerateuv.lock(resolvesclick 8.4.2). Remove the constraint onceuvicorn's own pin clearsclick 8.3.3.Local verification (matches CI steps)
frontend: npm cifrontend: npm test(vitest)frontend: npm run build(vite)backend: uv run pip-audit --strictnpm audit --audit-level=high --omit=devalso reports GHSA-qwww-vcr4-c8h2 (react-router CSRF, high). This advisory was published after the last greenTest & Deployon master (run 28823209746, 2026-07-06) and affects master too — it is not caused by PR #75. Its only fix is a breaking downgrade (react-router-dom 7.18.2 → 7.11.0) that belongs in its own change. Note: this run's frontend job failed atnpm cibefore reaching thenpm auditstep, so this advisory did not contribute to run #30854734500's failure.Draft until CI confirms green on this branch.