Skip to content

Conversation

@nemuvski
Copy link
Member

@nemuvski nemuvski commented Nov 6, 2025

変更概要

  • actions/checkoutactions/setup-node の最新を利用する
  • .github/workflows/ci.yml
    • strategy.matrix.node-version22.x24.x を追記
  • .github/workflows/release.yml
    • Node.jsのバージョンに .node-version ファイルを読む

先に #94 をみてください。

Summary by CodeRabbit

  • Chores

    • CI/CD ワークフローを更新し、複数の Node.js バージョンでのビルド・テストを追加、ワークフロー参照を最新化しました。
    • リリース手順に型チェックや整形チェックを組み込み、リリースの堅牢性を向上しました。
  • Chores(パッケージ)

    • 開発スクリプトを拡張し、フォーマット検査/修正や型チェックを追加しました。
  • Tests

    • テストのモック表現を整理・明確化し、挙動や期待結果は変更していません。

@nemuvski nemuvski self-assigned this Nov 6, 2025
@coderabbitai
Copy link

coderabbitai bot commented Nov 6, 2025

Caution

Review failed

The pull request is closed.

Walkthrough

GitHub Actions ワークフローを更新し、アクション参照を特定コミットSHAへ変更、トリガーのクォート形式を統一、CI を静的チェックと拡張された Node.js マトリックス(18.x,20.x,22.x,24.x)で分離。package.json にフォーマット・型チェックスクリプトを追加し、テストモックの MSW 呼び出しを整形/分割。

Changes

Cohort / File(s) 変更内容
ワークフロー共通
/.github/workflows/ci.yml, /.github/workflows/release.yml
トリガーのクォートをシングル→ダブルへ統一。checkoutsetup-node の参照をタグ/バージョンから特定コミットSHAへ置換。
CI(ci.yml)
/.github/workflows/ci.yml
既存の build ジョブを static-check ジョブへ差し替え(checkout, setup-node, npm install, format, lint, typecheck)。新規 build-and-test ジョブを追加し Node.js マトリックスを [18.x, 20.x, 22.x, 24.x] に拡張、ステップ名やキャッシュ設定を明示化。npm キャッシュ表記をダブルクォート化。
リリース(release.yml)
/.github/workflows/release.yml
タグパターンをダブルクォート化("v*")。node-versionnode-version-file: ".node-version" に変更。setup-node/checkout をコミットSHAに更新。lint の後に npm run typecheck を追加。
パッケージ設定
package.json
lint / lint:fix./src ./tests を対象に拡張。新規スクリプト追加:format, format:fix, typechecktsc --noEmit)。
テスト(MSW mocks)
tests/get.test.ts, tests/getAllContentIds.test.ts, tests/getAllContents.test.ts
MSW の HTTP モック呼び出しをフォーマット/構文変更(複数行化、URL を第一引数にする明示的ハンドラ形式、{ once: true } の明示など)。一部テストで複数の明示的ハンドラへ分割し、レスポンスの呼び出し数を増やす調整。機能的動作は維持。

Sequence Diagram(s)

sequenceDiagram
  autonumber
  participant GH as GitHub Actions
  participant Repo as Repository
  participant Node as Node matrix jobs

  rect rgb(230,245,255)
    Note over GH,Repo: Push / pull_request / release trigger
  end

  GH->>Repo: checkout (specific SHA)
  GH->>GH: static-check job
  GH->>Node: build-and-test job (matrix: 18x,20x,22x,24x)
  Node->>GH: run Build
  Node->>GH: run Unit Tests
  GH->>GH: publish (release workflow)
Loading

Estimated code review effort

🎯 3 (Moderate) | ⏱️ ~25 minutes

確認が必要な箇所:

  • checkout / setup-node に使われたコミットSHAが意図したアクションバージョンと合致しているか
  • build-and-test のマトリックス追加によるランタイムやキャッシュ設定の妥当性
  • static-check に追加された format / typecheck のステップがローカルスクリプトと一致しているか(package.json の新スクリプト)
  • tests の MSW モック分割により期待するリクエストシーケンスが変わっていないか(タイミングや { once: true } の影響)

Pre-merge checks and finishing touches

✅ Passed checks (2 passed)
Check name Status Explanation
Title check ✅ Passed タイトルはGitHub Actionsのワークフロー保守に関する変更を正確に要約しており、変更セットの主要な内容と一致している。
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.

📜 Recent review details

Configuration used: CodeRabbit UI

Review profile: CHILL

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between fbcd81e and aae0130.

📒 Files selected for processing (6)
  • .github/workflows/ci.yml (1 hunks)
  • .github/workflows/release.yml (1 hunks)
  • package.json (1 hunks)
  • tests/get.test.ts (1 hunks)
  • tests/getAllContentIds.test.ts (4 hunks)
  • tests/getAllContents.test.ts (2 hunks)

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

Comment on lines -12 to +13
- uses: actions/checkout@v4
- uses: actions/setup-node@v4
- uses: actions/checkout@08c6903cd8c0fde910a37f88322edcfb5dd907a8 # v5.0.0
- uses: actions/setup-node@2028fbc5c25fe9cf00d9f06a71cc4710d4507903 # v6.0.0
Copy link
Member Author

Choose a reason for hiding this comment

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

バージョンアップと同時に、pinactでコミットハッシュで固定しました。

Comment on lines -15 to +16
node-version: '18.x'
registry-url: 'https://registry.npmjs.org'
node-version-file: ".node-version"
registry-url: "https://registry.npmjs.org"
Copy link
Member Author

Choose a reason for hiding this comment

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

バージョン指定を .node-version ファイルを読むようにしました。

追伸: フォーマットかかっていて、クオーテーションが変わってますが気にせずで。

strategy:
matrix:
node-version: [18.x, 20.x]
node-version: [18.x, 20.x, 22.x, 24.x]
Copy link
Member Author

Choose a reason for hiding this comment

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

現行で、24まで出ているので、加えておきました。

@nemuvski nemuvski requested a review from dc7290 November 6, 2025 00:39
Copy link
Member

@dc7290 dc7290 left a comment

Choose a reason for hiding this comment

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

良さそうです!

コードチェック強化: format, typecheck追加, eslintにtestsディレクトリを対象に加える
@nemuvski nemuvski merged commit 31f3f05 into main Nov 6, 2025
5 of 6 checks passed
@nemuvski nemuvski deleted the update-ghactions branch November 7, 2025 06:33
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.

3 participants