mail gem 2.7系向けワークアラウンドを削除し、コードをシンプル化

Action Mailboxが mail gem >= 2.8 を要件とするようになったことで、バージョン 2.7 以前向けのワークアラウンドが不要となり削除されました。これにより、アドレス解析のコードパスが一本化され、保守性が向上します。

背景

このワークアラウンドは、mail gem における address_list メソッドの改名に起因する互換性問題を吸収するために追加されたものです。#46643 によると、mikel/mail#1120 で address_list メソッドが element へと改名されたことでCIが壊れ、その修正として mail 2.7系と 2.8系の両方に対応する分岐コードが導入されました。

その後、#50668 において Action Mailbox の依存バージョンが mail >= 2.8 に引き上げられました。これにより、2.7系向けの address_list 呼び出しパスが実行されることはなくなっていました。

今回の #56994 は、この「すでに到達不能になっていたコード」を整理する変更です。

技術的な変更

actionmailbox/lib/action_mailbox/mail_ext/addresses.rb 内のアドレス取得メソッド群から、バージョン分岐を行うプライベートメソッド address_list が削除されました。

変更前:

def from_address
  address_list(header[:from])&.addresses&.first
end

private
  def address_list(obj)
    if obj.respond_to?(:element)
      # Mail 2.8+
      obj.element
    else
      # Mail <= 2.7.x
      obj&.address_list
    end
  end

変更後:

def from_address
  header[:from]&.element&.addresses&.first
end

変更前は address_list(header[:from]) 経由で element を取り出していたのに対し、変更後は header[:from]&.element と直接呼び出す形になっています。同様の変更が reply_to_address、to_addresses、cc_addresses、bcc_addresses の各メソッドに適用されています。

address_list メソッドは11行のコードでしたが、その削除により差し引き16行が除去され、ファイル全体がよりフラットな構成になっています。

設計判断

respond_to? による動的ディスパッチをやめ、直接呼び出しに統一する方針が取られています。

ワークアラウンドの核心は obj.respond_to?(:element) でのバージョン判定でした。依存バージョンの下限が引き上げられた今、この判定は常に true を返すため、分岐自体が意味を持ちません。不要になった抽象化レイヤーを取り除くことで、メソッドチェーンが読者にとって自明になります。

また、private ブロックが丸ごと削除されたことで、クラスの公開インターフェースと内部実装の区別が不要になり、クラス定義がよりシンプルになっています。

まとめ

本PRは機能追加を伴わない純粋なコード整理ですが、「依存バージョンの引き上げによって生まれた負債をすみやかに返済する」という姿勢を示しています。バージョン互換のためのワークアラウンドは導入時から削除を前提とすべきもので、前提条件が整った段階で迅速に取り除くことが、コードベースの長期的な健全性につながります。