Skip to content

fix: require Node.js 22.23.4, 24.22.1 or 26.12.1 for the eMudhra intermediate CA compromise - #173

Merged
Winify merged 2 commits into
webdriverio:mainfrom
plum117:fix/node-floor-emudhra
Oct 25, 2026
Merged

Winify merged 2 commits into
webdriverio:mainfrom
plum117:fix/node-floor-emudhra

Conversation

@plum117

@plum117 plum117 commented Oct 25, 2026 •

Copy link
Copy Markdown
Contributor

Proposed changes

Fixes #172

Require the Node.js releases with the updated CA store for the eMudhra intermediate CA compromise ("ca-cert (eMudhra Intermediate CA Compromise) on 26.x, 24.x, 22.x" in the October 2026 security releases). Same range as webdriverio/webdriverio#16211.

Place Change
engines.node >=22.19.0 → >=22.23.4 <23 || >=24.22.1 <25 || >=26.12.1
lint.yml, test.yml, release.yml, publish-mcp-registry.yml check-latest: true on the .nvmrc setup: a bare 24 took the runner cache (24.21.0) in webdriverio/webdriverio#16211
README, Electron section Names the new floor (it said 22.19; @wdio/electron-service itself needs >=22.12.0)
tests/utils/docs-index.test.ts (separate commit) appium capabilities → /docs/api/appium.md within 2 (was 1). Since 2026-10-21 the live Capabilities page ranks first (18.30 against 17.45), so the test job fails on main and on every PR. It is a topic row, not an API-name row, and both pages answer the query

How you tested

Command Node 22.23.4 Node 24.22.1
pnpm install --frozen-lockfile no engine warning no engine warning
eslint src/ tests/, tsc --noEmit ok ok
pnpm test 793 passed, 30 skipped 793 passed, 30 skipped
pnpm run bundle ok ok

With the live corpus (~/.wdio-mcp/llms-full.txt): docs-index.test.ts failed 1 of 75 on main and before the test commit, then 75 passed; full suite 823 passed. Without it: 46 passed, 29 skipped.

The packed tarball has the new engines. npm install --engine-strict of it: Node 24.20.0 → EBADENGINE, exit 1; Node 24.22.1 → added, exit 0.

Types of changes

  • Polish (an improvement for an existing feature)
  • Bugfix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to not work as expected)
  • Documentation update (improvements to the project's docs)
  • Specification changes (updates to WebDriver command specifications)
  • Internal updates (everything related to internal scripts, governance documentation and CI files)

Checklist

  • I have squashed commits that belong together
  • I have tested with Claude Desktop (or another MCP-compatible client)
  • I have read the CONTRIBUTING doc
  • I have added the necessary documentation for new/changed tools (if appropriate)
  • I have added proper type definitions for new commands (if appropriate)

Further comments

engines only warns, except with Yarn 1 or npm --engine-strict. WebdriverIO shipped the same range as a minor version.

Reviewers: @webdriverio/project-committers

@plum117
plum117 marked this pull request as ready for review October 25, 2026 17:44
@greptile-apps

greptile-apps Bot commented Oct 25, 2026 •

Copy link
Copy Markdown

RetriggerView in GreptileConfidence Score: 5/5

[Medium impact] The PR appears safe to merge; no actionable new issues were found.

Summary

Raises the supported Node.js versions to releases with the updated CA store.

  • The package now requires patched releases of Node.js 22, 24, or 26.
  • GitHub jobs check the latest release of their Node.js line.
  • The docs-index test allows the Appium API page to rank second.

Reviews (2) · Last reviewed commit: "test: accept the live Capabilities page ..." · Reviewed by Greptile

… "appium capabilities"

The webdriver.io docs changed on 2026-10-21 and the test job fails on main
and on every PR since. It is a topic row, not an API-name row, and both
pages answer the query.
@plum117

plum117 commented Oct 25, 2026

Copy link
Copy Markdown
Contributor Author

@Winify, any concerns about merging this?

@Winify

Winify commented Oct 25, 2026

Copy link
Copy Markdown
Collaborator

@Winify, any concerns about merging this?

@plum117 No, lets go!

@Winify
Winify merged commit 0cc99e7 into webdriverio:main Oct 25, 2026
4 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.

Require the Node.js releases with the eMudhra CA fix (22.23.4, 24.22.1, 26.12.1)

2 participants