Skip to content

Update recommended Node.js version from 24 to 22#128

Open
Copilot wants to merge 2 commits intomainfrom
copilot/update-node-version-22
Open

Update recommended Node.js version from 24 to 22#128
Copilot wants to merge 2 commits intomainfrom
copilot/update-node-version-22

Conversation

Copy link
Contributor

Copilot AI commented Oct 8, 2025

概要

推奨Node.jsバージョンを24から22に更新しました。

変更内容

開発環境とドキュメントで参照されているNode.jsバージョンを以下のように更新:

  • mise.toml: node = "22"に変更
  • flake.nix: 全てのnodejs_24参照をnodejs_22に変更(5箇所)
    • dev、aituberkit、docs、bundleアプリ
    • devShellsのデフォルト設定
  • README.md & README-ja.md: 必要なNode.jsバージョンを>= 22に更新

互換性について

  • ビルドターゲット(packages/*/tsup.config.tstarget: "node22")は既に正しく設定されているため変更不要
  • CHANGELOGファイルは履歴として保持
  • 最小限の変更(4ファイル、8行)で要件を満たしています

これにより、開発者はNode.js 22以降を使用してプロジェクトを開発できるようになります。

Fixes #(issue_number)

Original prompt

This section details on the original issue you should resolve

<issue_title>推奨nodeバージョンを22にする</issue_title>
<issue_description></issue_description>

<agent_instructions>互換性の維持を心がけてください。最小限の変更でお願いします</agent_instructions>

Comments on the Issue (you are @copilot in this section)

Fixes #127

✨ Let Copilot coding agent set things up for you — coding agent works faster and does higher quality work when set up for your repo.

Summary by CodeRabbit

  • 新機能
    • Nix 環境で dev/docs/bundle などの実行バイナリが直接呼び出し可能に
  • ドキュメント
    • 必要な Node.js バージョンを >= 22 に更新(日本語版README含む)
  • チョア
    • 開発ツールチェーンの Node.js バージョンを 22 に統一(mise など)

@changeset-bot
Copy link

changeset-bot bot commented Oct 8, 2025

⚠️ No Changeset found

Latest commit: b8ff583

Merging this PR will not cause a version bump for any packages. If these changes should not result in a new version, you're good to go. If these changes should result in a version bump, you need to add a changeset.

This PR includes no changesets

When changesets are added to this PR, you'll see the packages that this PR includes changesets for and the associated semver types

Click here to learn what changesets are, and how to add one.

Click here if you're a maintainer who wants to add a changeset to this PR

@vercel
Copy link

vercel bot commented Oct 8, 2025

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Preview Comments Updated (UTC)
aikyo Ready Ready Preview Comment Oct 8, 2025 6:39am

@coderabbitai
Copy link

coderabbitai bot commented Oct 8, 2025

Important

Review skipped

Bot user detected.

To trigger a single review, invoke the @coderabbitai review command.

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

Walkthrough

Node.jsバージョン要件を24系から22系へ変更。README(英・日)を更新し、mise.tomlの[tools]でnodeを"22"に設定。flake.nixでnodejs_24をnodejs_22へ差し替え、各appのbin公開のために.outPath + "/bin/..." を明示。

Changes

Cohort / File(s) Summary of changes
Documentation: Node.js 要件更新
README.md, README-ja.md
最小要件をNode.js >= 24.2から>= 22へ更新。文面のみ変更。
Tooling: バージョン/ランタイム整備
flake.nix, mise.toml
flake.nixでruntimeをnodejs_24→nodejs_22へ変更し、appsのコマンドを.outPath + "/bin/dev"等で明示。mise.tomlの[tools]でnode = "24"node = "22"に変更。

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~10 minutes

Possibly related PRs

Suggested reviewers

  • marukun712

Poem

ぴょん、と跳ねて版上げる
24から22へ道ならし
nixもmiseも足並みそろえ
READMEにもしっかり刻印
今夜のニンジンは互換の味 🥕

Pre-merge checks and finishing touches

❌ Failed checks (1 warning)
Check name Status Explanation Resolution
Out of Scope Changes Check ⚠️ Warning flake.nixにおいてNode.jsバージョン更新以外に.outPath経由でバイナリを露出するパス設定が追加されていますが、Issue #127の目的には含まれていない変更であるため、本PRの範囲外と判断されます。 Node.jsバージョン変更以外のバイナリ露出設定は別のプルリクエストに分離するか、本PRから削除してください。
✅ Passed checks (4 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title Check ✅ Passed このプルリクエストのタイトルは変更の主目的であるNode.jsの推奨バージョンを24から22へ更新する点を明確かつ簡潔に示しており、内容と一致しています。変更対象のファイルや他の要素に言及せず主題に集中しているため、チームが履歴を見た際にも即座に意図が伝わります。
Linked Issues Check ✅ Passed リンクされたIssue #127の目的である推奨Node.jsバージョンを22に変更する要件について、READMEとREADME-ja.md、mise.toml、flake.nixのすべてでバージョンが22に統一されており、ビルドターゲットも既存設定と整合しているため要件を完全に満たしています。
Docstring Coverage ✅ Passed No functions found in the changes. Docstring coverage check skipped.

Comment @coderabbitai help to get the list of available commands and usage tips.

Co-authored-by: akazdayo <82073147+akazdayo@users.noreply.github.com>
Copilot AI changed the title [WIP] Update recommended Node version to 22 Update recommended Node.js version from 24 to 22 Oct 8, 2025
Copilot AI requested a review from akazdayo October 8, 2025 06:40
@akazdayo
Copy link
Collaborator

akazdayo commented Oct 8, 2025

@coderabbitai review

@coderabbitai
Copy link

coderabbitai bot commented Oct 8, 2025

✅ Actions performed

Review triggered.

Note: CodeRabbit is an incremental review system and does not re-review already reviewed commits. This command is applicable only when automatic reviews are paused.

@akazdayo akazdayo marked this pull request as ready for review October 8, 2025 07:33
@akazdayo akazdayo requested a review from marukun712 October 8, 2025 07:34
@marukun712
Copy link
Owner

なんで22にするんだっけ?

@akazdayo
Copy link
Collaborator

akazdayo commented Oct 9, 2025

@marukun712 24にしてた理由が、import.meta.mainを使いたかったのと、TypeScriptをコンパイルしてなかったから24以上を必須にしないと動かないからだったんだよね。
今は両方が解決してるからLTSがある22の方が安定する

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.

推奨nodeバージョンを22にする

3 participants