Skip to content

fix: use Node.js 22.12.0 in release workflow to satisfy engines.node requirement - #3

Merged
insane66613 merged 2 commits into
mainfrom
copilot/update-node-version-release-workflow
Mar 7, 2026
Merged

insane66613 merged 2 commits into
mainfrom
copilot/update-node-version-release-workflow

Conversation

Copilot AI commented Mar 7, 2026 •

Copy link
Copy Markdown

Release workflow was pinned to Node 20, causing pnpm install to fail because package.json declares engines.node: >=22.12.0 and pnpm enforces this constraint.

Changes

  • .github/workflows/release.yml: bump node-version from 20 → '22.12.0'
- name: Setup Node
  uses: actions/setup-node@v4
  with:
    node-version: '22.12.0'  # was: 20
    cache: 'pnpm'

Priority*

  • High: This PR needs to be merged first, before other tasks.
  • Medium: This PR should be merged quickly to prevent conflicts due to common changes. (default)
  • Low: This PR does not affect other tasks, so it can be merged later.

Purpose of the PR*

Fix release workflow CI failure caused by Node version mismatch between the workflow (20) and the engines.node field in package.json (>=22.12.0).

Changes*

  • .github/workflows/release.yml: Set node-version: '22.12.0' in the Setup Node step (previously 20).

How to check the feature

Trigger the Release Extension workflow via workflow_dispatch and verify the Install dependencies step completes without an engines constraint error.

Reference

package.json engines field: "node": ">=22.12.0"

Original prompt

The Release workflow (.github/workflows/release.yml) currently sets up Node.js v20 in the Setup Node step, but the repository package.json specifies an engines.node requirement of ">=22.12.0". This causes the workflow job to fail with the error: "This is happening because the package's manifest has an engines.node field specified. To fix this issue, install the required Node version."

Change required:

  • Update the .github/workflows/release.yml file to install a Node.js version that satisfies the package.json engines requirement (>=22.12.0). Use actions/setup-node@v4 and set node-version to the minimum compatible version (22.12.0) or a compatible range (22.x).

Proposed updated file content (replace the current Setup Node step):

  • name: Setup Node
    uses: actions/setup-node@v4
    with:
    node-version: '22.12.0'
    cache: 'pnpm'

Full file (after change):

name: Release Extension

on:
workflow_dispatch:

jobs:
release:
runs-on: ubuntu-latest

steps:
  - name: Checkout repository
    uses: actions/checkout@v4

  - name: Install pnpm
    uses: pnpm/action-setup@v4
    with:
      version: 9.15.1

  - name: Setup Node
    uses: actions/setup-node@v4
    with:
      node-version: '22.12.0'
      cache: 'pnpm'

  - name: Install dependencies
    run: pnpm install

  - name: Build extension
    run: pnpm build

  - name: Create ZIP package
    run: pnpm zip

  - name: Upload Release Artifact
    uses: actions/upload-artifact@v4
    with:
      name: extension-zip
      path: ./chrome-extension/dist/*.zip

  - name: Create GitHub Release
    uses: softprops/action-gh-release@v2
    with:
      files: ./chrome-extension/dist/*.zip
    env:
      GITHUB_TOKEN: ${{ secrets.GITHUB_TOKEN }}

Notes:

  • The change ensures the workflow uses Node 22.12.0 which satisfies package.json's engines.node (>=22.12.0).
  • If you'd prefer to track the major line (auto-pick latest 22.x) you can use '22.x' instead of the exact patch version.

Please create a pull request with this change in the repository 'insane66613/MCP-SuperAssistant'.

This pull request was created from Copilot chat.


✨ Let Copilot coding agent set things up for you — coding agent works faster and does higher quality work when set up for your repo.

Co-authored-by: insane66613 <8126035+insane66613@users.noreply.github.com>
Copilot AI changed the title [WIP] Update Node.js version in release workflow fix: use Node.js 22.12.0 in release workflow to satisfy engines.node requirement Mar 7, 2026
@insane66613
insane66613 marked this pull request as ready for review March 7, 2026 02:43
@insane66613
insane66613 merged commit 765c8bc into main Mar 7, 2026
6 checks passed
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.

2 participants