Skip to content

Move Node to 26 LTS everywhere at once - #211

Merged
wormeyman merged 1 commit into
mainfrom
chore/node-26
Nov 2, 2026
Merged

wormeyman merged 1 commit into
mainfrom
chore/node-26

Conversation

@wormeyman

@wormeyman wormeyman commented Nov 2, 2026 •

Copy link
Copy Markdown
Collaborator

Moves FactorioTools from Node 24 to Node 26.13.0, the first Node 26 LTS (Lithium, 2026-10-28). Node 24 has been on security fixes only since 2026-10-20.

Renovate's #209 changed only the two workflows. The @types/node hold in .github/renovate.json5 says the types and the workflows must move in the same PR, or the types describe a runtime CI does not use. This PR moves every place that names Node 24, and supersedes #209.

File Change
.github/workflows/ci.yml, deploy-cloudflare.yml node-version: 24 -> 26
src/vue/package.json + lock @types/node ^24.13.2 -> ^26.6.6 (lock 24.19.2 -> 26.6.6, plus its undici-types)
.github/renovate.json5 @types/node hold raised from <25.0.0 to <27.0.0
.devcontainer/Dockerfile Node v24.21.0 -> v26.13.0, both checksums from https://nodejs.org/dist/v26.13.0/SHASUMS256.txt
CLAUDE.md "Node 24" -> "Node 26" in Prerequisites and Devcontainer

Checked on tealc, Node 26.13.0 and npm 11.20.0

  • npm run build-wasm, npm run build (includes the vue-tsc type check against the new types) and npm test: build OK, 55 tests passed.
  • npx wrangler deploy --dry-run: read the 10 asset files, no errors.
  • renovate-config-validator from Renovate 44.205.2: config validated.
  • Dev container image built with Docker Desktop on amd64: the tarball checksum passed and the container reports v26.13.0 and npm 11.20.0. The arm64 checksum comes from the same SHASUMS file but was not built here.

Merging deploys the site, since deploy-cloudflare.yml runs on every push to main.

🤖 Generated with Claude Code

https://claude.ai/code/session_01Kd6o1asZ1umbDC4dVgANYJ

Summary by CodeRabbit

  • Chores
    • Updated the development environment and automated build and deployment workflows to use Node.js 26.
    • Updated the project’s Node.js type definitions to match the new runtime version.
  • Documentation
    • Updated the documented Node.js prerequisite and development environment version to Node.js 26. These changes affect project setup and maintenance; no end-user-facing changes are included.

Node 26.13.0 (Lithium) is the first Node 26 LTS. Node 24 has been on
security fixes only since 2026-10-20. Renovate's #209 moved only the two
workflows. The @types/node hold in renovate.json5 says the types and the
workflows must move in the same PR, so this PR moves all of them:

- ci.yml and deploy-cloudflare.yml: node-version 24 -> 26
- src/vue: @types/node ^24.13.2 -> ^26.6.6 (lock: 24.19.2 -> 26.6.6)
- renovate.json5: @types/node hold raised from <25 to <27
- .devcontainer/Dockerfile: Node v24.21.0 -> v26.13.0, with both
  checksums from nodejs.org's SHASUMS256.txt
- CLAUDE.md: Node 24 -> Node 26 in Prerequisites and Devcontainer

Supersedes #209.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01Kd6o1asZ1umbDC4dVgANYJ
@coderabbitai

coderabbitai Bot commented Nov 2, 2026 •

Copy link
Copy Markdown

Review in Change Stack →

No actionable comments were generated in the recent review.

Recent review info
Run configuration
  • Source mode: Auto
  • Repository: Unavailable
  • Branch: Unavailable
  • File: No configuration file from this repository applied
  • Commit: Unavailable
  • Layers: Organization UI
  • Review profile: CHILL
  • Plan: Advanced
  • Run ID: 22f683e6-8091-408f-9d59-c4df9751d3fd
Commits

Reviewing files that changed from the base of the PR and between 0e5c19d and 24680fd.

Files ignored due to path filters: 1
  • src/vue/package-lock.json is excluded by !**/package-lock.json
Files selected for processing: 6
  • .devcontainer/Dockerfile
  • .github/renovate.json5
  • .github/workflows/ci.yml
  • .github/workflows/deploy-cloudflare.yml
  • CLAUDE.md
  • src/vue/package.json

Included review availability: This review used your included allowance. Your plan provides up to 1 included review per hour; 0 remain after this review.

Walkthrough

Node 24 references are updated to Node 26 across the development container, CI and Cloudflare workflows, Node.js type definitions, Renovate settings, and documentation.

Changes: Node 26 Upgrade

Layer / File(s) Summary
Align Node runtime and tooling
.devcontainer/Dockerfile, .github/workflows/ci.yml, .github/workflows/deploy-cloudflare.yml, src/vue/package.json, .github/renovate.json5, CLAUDE.md
The development container and workflows now use Node 26. The @types/node version, Renovate settings, and documented prerequisites are updated to match.

Estimated code review effort: 2 (Simple) | ~10 minutes

Change: Other

Possibly related issues

  • Issue 41 in FactoryGameFan/FactorioTools tracks the same Node 24-to-26 update, including workflow pins and the @types/node version.
Merge Risk: Minimal · up to 24680

The Node 26 runtime, types, container pin, and Renovate configuration are aligned and mergeable.

Pre-merge checks: Passed 5
Check name Status Explanation
Description Check Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check Passed The title clearly and concisely describes the coordinated migration from Node 24 to Node 26 across the project.
Docstring Coverage Passed No eligible functions found in the changed files. Skipping docstring coverage check. Docstring coverage is scoped to functions touched by this diff with cyclomatic complexity of at least 2. Documentat…
Linked Issues check Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check Passed Check skipped because no linked issues were found for this pull request.
Full details: Docstring Coverage

Explanation

No eligible functions found in the changed files. Skipping docstring coverage check. Docstring coverage is scoped to functions touched by this diff with cyclomatic complexity of at least 2. Documentation-only edits and deletions within surviving functions are included; unchanged diff context and fully deleted functions are excluded. Analyzed 0 functions across 0 files. (6 skipped: 6 unsupported.)

Finishing Touches
Generate unit tests (beta)
  • Commit to this branch
  • Create a new PR
  • Autofix · Keep fixing CodeRabbit findings and required CI, and resolving merge conflicts

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

Share

@wormeyman
wormeyman merged commit 13cccae into main Nov 2, 2026
10 checks passed
@wormeyman
wormeyman deleted the chore/node-26 branch November 2, 2026 21:04
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