Skip to content

Enforce the Node.js 22.22.0 CLI minimum - #4613

Merged
steve8708 merged 3 commits into
mainfrom
steve8708/changes-7142
Sep 9, 2026
Merged

Enforce the Node.js 22.22.0 CLI minimum#4613
steve8708 merged 3 commits into
mainfrom
steve8708/changes-7142

Conversation

@steve8708

Copy link
Copy Markdown
Contributor

Summary

  • Make the CLI reject Node.js 22.21.x and older before scaffolding.
  • Match the package and template engine requirement of Node.js 22.22.0.
  • Add a patch changeset for @agent-native/core.

Validation

  • corepack pnpm guards
  • corepack pnpm --filter @agent-native/core typecheck
  • corepack pnpm exec oxfmt --check packages/core/src/cli/index.ts
  • git diff --check

Feedback review

  • Reviewed the current npx feedback sweep and released reviewed items with verified terminal reactions.
  • Confirmed data/pglite.agent-native-pglite.lock is already covered by the root /data/ ignore rule.
  • Left reproduction-dependent and cross-team items open for their owners.

steve8708 added a commit that referenced this pull request Sep 9, 2026
steve8708 added a commit that referenced this pull request Sep 9, 2026
builder-io-integration[bot]

This comment was marked as outdated.

steve8708 added a commit that referenced this pull request Sep 9, 2026
steve8708 added a commit that referenced this pull request Sep 9, 2026
steve8708 added a commit that referenced this pull request Sep 9, 2026
builder-io-integration[bot]

This comment was marked as outdated.

steve8708 added a commit that referenced this pull request Sep 9, 2026
steve8708 added a commit that referenced this pull request Sep 9, 2026
@github-actions

github-actions Bot commented Sep 9, 2026

Copy link
Copy Markdown
Contributor

Here's a visual recap of what changed:

Visual recap

Open the full interactive recap

steve8708 added a commit that referenced this pull request Sep 9, 2026
steve8708 added a commit that referenced this pull request Sep 9, 2026
steve8708 added a commit that referenced this pull request Sep 9, 2026
steve8708 added a commit that referenced this pull request Sep 9, 2026
steve8708 added a commit that referenced this pull request Sep 9, 2026
# Conflicts:
#	packages/core/src/cli/index.ts
steve8708 added a commit that referenced this pull request Sep 9, 2026

@builder-io-integration builder-io-integration 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.

Builder reviewed your changes — looks good ✅

Review Details

Incremental Code Review Summary

The latest PR head adds the Node.js minimum guard and patch changeset relative to the current base. The guard now requires a stable numeric major.minor.patch version and enforces Node.js >=22.22.0, rejecting older releases and prerelease/nightly versions before CLI scaffolding begins. The error message and exit status remain clear and appropriate, and the changeset correctly targets @agent-native/core.

Two parallel review agents inspected the current diff and surrounding CLI flow. No new actionable bugs, security issues, performance concerns, or release inconsistencies were found. The previously reported prerelease issue was already fixed and resolved in the prior review.

🧪 Browser testing: Skipped — PR only modifies CLI runtime validation and release metadata, with no user-facing browser impact.

steve8708 added a commit that referenced this pull request Sep 9, 2026
steve8708 added a commit that referenced this pull request Sep 9, 2026
steve8708 added a commit that referenced this pull request Sep 9, 2026
@steve8708
steve8708 merged commit 804113d into main Sep 9, 2026
48 checks passed
@steve8708
steve8708 deleted the steve8708/changes-7142 branch September 9, 2026 18:33
steve8708 added a commit that referenced this pull request Sep 9, 2026
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