ci: Crystal を固定し ameba を Crystal 1.21 対応コミットに追従させる（#108）

[limit7412]

close #108

課題

crystal: latest が 1.21.0 に上がったことで、shards install の ameba postinstall がコンパイルできず CI が常に落ちる状態だった。差分の内容によらず、いま出す PR はすべて赤になる。

Error: undefined method 'next_string_array_token' for Crystal::Lexer

Crystal 1.21 で Crystal::Lexer#next_string_array_token が削除されたが、~> 1.6 で解決される最新の安定版 ameba 1.6.4 はこのメソッドに依存している。

変更内容

方針は「Crystal 固定 + ameba 追従」の併用。

• .github/workflows/ci.yml
  • crystal: latest → crystal: 1.21.0。Crystal 側の更新でいきなり CI が壊れるのを防ぐ（今回は master への push が 3 週間無かったため気付くのが遅れた）
  • ameba のビルドステップを追加。追従先の ameba（1.7 系）は shard.yml から postinstall が外れており、shards install だけでは bin/ameba が作られない
• serverless.yml: ビルドイメージを crystallang/crystal:latest-alpine → 1.21.0-alpine に固定（レビュー指摘対応 / 下記）
• shard.yml: ameba を version: ~> 1.6 → 修正を含む master のコミットに固定
  • 対応（lexer.responds_to?(:next_string_array_token) での分岐）は master に入っているが、タグは v1.6.4 の次が v1.7.0-dev のみで安定版のリリースがまだ無い
  • ブランチ追従ではなくコミット固定にして、ビルドの再現性は保つ
• shard.lock: 上記に合わせて更新
• 既存コードの lint 修正 8 箇所（下記）

デプロイ用ビルドイメージの固定

CI だけを固定しても、serverless.yml の before:package:createDeploymentArtifacts は crystallang/crystal:latest-alpine でアプリをコンパイルするため、本番成果物は検証済みのコンパイラとは別のもので作られていた。Crystal が更新されれば互換性が壊れた瞬間に deploy も落ちる。本 PR の趣旨がデプロイ側で達成できていなかったので、同じ 1.21.0 に揃えた。

次に上げるときに両方揃えられるよう、ci.yml と serverless.yml に相互参照のコメントを入れている。

ameba 1.7 系で新たに検出された指摘

ameba 側で Style/RedundantNilInControlExpression が追加され、Style/RedundantBegin / Style/HeredocIndent の検出範囲も広がったため、master 由来の既存コードが 8 箇所引っかかった。いずれも ameba が [Correctable] と判定する機械的な修正で、挙動は変わらない。ameba を上げた本 PR の責任範囲として同梱した。

• 戻り値が nilable なメソッドの return nil → return（src/discord/models.cr ×4、src/github/models.cr、src/github/usecase.cr）
• ブロック直下の begin/rescue → ブロックの rescue（src/main.cr）
• ヒアドキュメント本文を 2 スペースインデント（spec/github/models_spec.cr）
  • <<- なので生成される文字列の内容は変わらない

今後

ameba 1.7 系がリリースされたら shard.yml を version: 指定に戻す。その旨は shard.yml のコメントにも残した。Crystal を上げるときは ameba 側の対応状況と serverless.yml のビルドイメージのタグも合わせて確認する。

影響

このブロッカーで CI が赤のままだった PR:

• fix: レビュー依頼スレッドの後続通知の文言を切り替える（#104） #106（issue レビューに入っている場合のメッセージが微妙 #104 対応）
• feat: チェックが成功していない PR をメンション対象から外す（#105） #107（issue CI、自動チェックが成功していないPRをメンション対象から外す #105 対応）
• ci: issue/PR 作成時に自動アサインする（#110） #111（issue issue/PR作成時に自動アサインする仕組みを作成する #110 対応）

いずれも本ブランチを取り込んで CI green を確認済み。本 PR が master にマージされれば、各 PR の差分からはこのコミット群が消える。

検証

CI green。shard.lock の解決は Installing ameba (1.7.0-dev at cdd58b3) で確認、固定したビルドイメージは deploy ジョブが実際にそれでコンパイルして成功していることで確認している。

実行環境から crystal-lang.org への到達がネットワークポリシーで遮断されており Crystal / shards をローカルにインストールできないため、検証は CI に依存している。shard.lock の version: は shards の GitResolver がコミット解決時に書き出す "#{spec.version.value}+git.commit.#{commit}" の形式に合わせた。

[chatgpt-codex-connector]

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: c580bf82fc

ℹ️ About Codex in GitHub

Your team has set up Codex to 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 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".