Skip to content

fix: correct Node.js version prerequisite to >= 20.0.0#252

Open
dorukardahan wants to merge 1 commit into0gfoundation:mainfrom
dorukardahan:fix/nodejs-version-prereqs
Open

fix: correct Node.js version prerequisite to >= 20.0.0#252
dorukardahan wants to merge 1 commit into0gfoundation:mainfrom
dorukardahan:fix/nodejs-version-prereqs

Conversation

@dorukardahan
Copy link
Contributor

Summary

  • Fixed Node.js version prerequisite from >= 22.0.0 to >= 20.0.0 in two compute network docs pages

Why

The SDK's package.json engines field specifies >=20.0.0, and this was already corrected in the SDK README via PR #172. The docs pages still had the old 22.0.0 requirement, which could cause confusion for developers using Node.js 20.x or 21.x.

Files changed

File Change
docs/.../compute-network/inference.md Line 16: 22.0.020.0.0
docs/.../compute-network/account-management.md Line 44: 22.0.020.0.0

Verification

The SDK package.json and README specify Node.js >= 20.0.0 (fixed in
0g-serving-user-broker PR 0gfoundation#172). The docs still had the old 22.0.0
requirement in two compute network pages.
@vercel
Copy link

vercel bot commented Feb 23, 2026

@dorukardahan is attempting to deploy a commit to the 0g-Frontend Team on Vercel.

A member of the Team first needs to authorize it.

@dorukardahan
Copy link
Contributor Author

Self-review: improvements from a second pass

Verification note: The official llms-full.txt at docs.0g.ai also says Node.js >= 22.0.0 — this PR fixes the source markdown files that generate that output. The SDK's package.json has "node": ">=20.0.0" which is the authoritative source for the actual Node.js requirement.

Scope check: Searched all docs for remaining 22.0.0 references after this PR — none found in the compute network section. The llms-full.txt will update automatically when the docs rebuild.

Cross-reference: This is consistent with PR #172 on 0g-serving-user-broker (already merged) which fixed the same 22→20 version in the SDK README.

@dorukardahan
Copy link
Contributor Author

@Ravenyjh Quick question: The SDK package.json specifies "node": ">=20.0.0" and the README (fixed in PR #172) now says >= 20.0.0. But I noticed the docs have been at >= 22.0.0 for a while — is this intentional (e.g., CLI tools or other dependencies need Node 22)? Or should docs match the SDK at 20.0.0?

Happy to revert if 22.0.0 is the intended minimum.

@Ravenyjh
Copy link
Collaborator

@Ravenyjh Quick question: The SDK package.json specifies "node": ">=20.0.0" and the README (fixed in PR #172) now says >= 20.0.0. But I noticed the docs have been at >= 22.0.0 for a while — is this intentional (e.g., CLI tools or other dependencies need Node 22)? Or should docs match the SDK at 20.0.0?

Happy to revert if 22.0.0 is the intended minimum.

Version >20.0.0 is ok. The additional dependencies that previously required version 22 or higher are no longer present

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