Skip to content

Fix daily docs coverage dependency conflict - #1809

Merged
merryman merged 1 commit into
mainfrom
codex/fix-daily-docs-coverage
Jul 20, 2026
Merged

Fix daily docs coverage dependency conflict#1809
merryman merged 1 commit into
mainfrom
codex/fix-daily-docs-coverage

Conversation

@merryman

Copy link
Copy Markdown
Member

What changed

Pin @babel/eslint-parser to version 7.29.7 in the daily docs-coverage job.

Why

The workflow pins ESLint 8 but installed the Babel ESLint parser without a version constraint. Since Babel parser 8 became the default release, npm resolves it with an ESLint 9/10 peer requirement and aborts with ERESOLVE before documentation linting starts.

This dependency conflict has made every scheduled daily run fail since June 17 even when the test job succeeds. The missing docs artifact also causes the downstream summary job to fail.

Impact

The docs-coverage job can install its tooling and run again without migrating the repository's legacy ESLint configuration to ESLint 9.

Validation

  • Reproduced the workflow's three npm install commands in a clean archive of main
  • Confirmed @babel/eslint-parser@7.29.7, eslint@8.57.0, and eslint-plugin-jsdoc install without peer conflicts
  • Ran npx eslint -c .eslintrc.js lively.*/*.js | grep '✖' successfully
  • Ran git diff --check

@merryman
merryman marked this pull request as ready for review July 19, 2026 22:25
@merryman
merryman merged commit 05bc267 into main Jul 20, 2026
3 checks passed
@merryman
merryman deleted the codex/fix-daily-docs-coverage branch July 20, 2026 07:21
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.

1 participant