Skip to content

chore(cli): Bump node version#6253

Merged
rbwest merged 2 commits into
masterfrom
bump_cli_node_version
May 27, 2026
Merged

chore(cli): Bump node version#6253
rbwest merged 2 commits into
masterfrom
bump_cli_node_version

Conversation

@rbwest

@rbwest rbwest commented May 26, 2026

Copy link
Copy Markdown
Contributor

No description provided.

@superagent-security superagent-security Bot added contributor:verified Contributor passed trust analysis. pr:verified PR passed security analysis. labels May 26, 2026
@rbwest rbwest changed the title Bump CLI node version chore(cli): Bump node version May 26, 2026

@cubic-dev-ai cubic-dev-ai Bot left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

1 issue found across 1 file

Confidence score: 4/5

  • This PR looks safe to merge overall; the reported issue is low-to-moderate severity and doesn’t indicate a runtime regression in application logic.
  • In packages/cli/package.json, engines.node is pinned to >=22.22.2, which may unnecessarily block compatible Node 22 environments and create avoidable install/use friction for users.
  • Pay close attention to packages/cli/package.json - relax the Node engine constraint (for example to >=22 or >=22.0.0) to prevent overly strict version gating.

Reply with feedback, questions, or to request a fix.

Fix all with cubic | Re-trigger cubic

Comment thread packages/cli/package.json

@chatgpt-codex-connector chatgpt-codex-connector Bot 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.

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: 1bf302aa40

ℹ️ About Codex in GitHub

Codex has been enabled to automatically review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

When you sign up for Codex through ChatGPT, Codex can also answer questions or update the PR, like "@codex address that feedback".

Comment thread packages/cli/package.json
Comment thread Dockerfile
# New tmp image
# ------------------
FROM node:22.21.1-bookworm-slim AS build
FROM node:22.22.2-bookworm-slim AS build

@TBonnin TBonnin May 27, 2026

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

We don't have to update the services, even though it might be nice to keep the node version in sync.
It is only a minor version but let's make sure this doesn't bring any regression by running in staging for a bit.

@rbwest rbwest added this pull request to the merge queue May 27, 2026
Merged via the queue into master with commit c3cb83d May 27, 2026
28 checks passed
@rbwest rbwest deleted the bump_cli_node_version branch May 27, 2026 13:20
@rbwest rbwest self-assigned this Jun 15, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

contributor:verified Contributor passed trust analysis. pr:verified PR passed security analysis.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants