Skip to content

fix: resolve npm ci ERESOLVE peer dependency conflict in publish workflow#11

Merged
MrDoe merged 2 commits into
mainfrom
copilot/fix-github-actions-build-job
Jun 18, 2026
Merged

fix: resolve npm ci ERESOLVE peer dependency conflict in publish workflow#11
MrDoe merged 2 commits into
mainfrom
copilot/fix-github-actions-build-job

Conversation

Copilot AI commented Jun 18, 2026

Copy link
Copy Markdown
Contributor

npm ci fails in CI because tree-sitter-swift@0.7.1 requires tree-sitter@^0.22.1 as a peer, while other grammar packages (@derekstride/tree-sitter-sql, @tree-sitter-grammars/tree-sitter-markdown, etc.) pin to tree-sitter@^0.21.x.

Changes

  • .github/workflows/publish.yml: Add --legacy-peer-deps to both npm ci calls (build and publish-npm jobs), consistent with the project's documented approach for tree-sitter/LanceDB peer conflicts.

tree-sitter-swift@0.7.1 requires tree-sitter@^0.22.1 as a peer dependency,
but other grammar packages require tree-sitter@^0.21.x, causing ERESOLVE
failures in npm ci. Adding --legacy-peer-deps resolves this conflict.
Copilot AI changed the title [WIP] Fix failing GitHub Actions job build fix: resolve npm ci ERESOLVE peer dependency conflict in publish workflow Jun 18, 2026
Copilot finished work on behalf of MrDoe June 18, 2026 14:05
Copilot AI requested a review from MrDoe June 18, 2026 14:05
@MrDoe MrDoe marked this pull request as ready for review June 18, 2026 14:06
Copilot AI review requested due to automatic review settings June 18, 2026 14:06
@MrDoe MrDoe merged commit dad3456 into main Jun 18, 2026
1 check passed

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Updates the release publish GitHub Actions workflow to make CI installs resilient to known npm peer-dependency conflicts between tree-sitter-swift@0.7.1 (peer tree-sitter@^0.22.1) and other tree-sitter grammar packages pinned to ^0.21.x, so npm ci no longer fails with ERESOLVE during release builds/publishing.

Changes:

  • Add --legacy-peer-deps to the npm ci step in the build job.
  • Add --legacy-peer-deps to the npm ci step in the publish-npm job.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

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.

3 participants