Refactor: `in_active` → `inactivated_at` の命名を統一し、不要になった `is_active` カラムを削除する

[yasulab]

• 対応 PR:
  • Refactor: is_active カラムを削除し inactivated_at に統一 #1744

📋 背景

PR #1726 で inactivated_at カラムを導入し、PR #1732 で年次フィルタリング機能を実装しました。現在、システム内に「inactive」と「inactivated」の2つの命名が混在しており、また is_active カラムと inactivated_at カラムが重複しています。

この Issue では、システム全体の一貫性を向上させるため、以下のリファクタリングを実施します。

✅ タスクリスト

• Updated tasks to this
  • Refactor: is_active カラムを削除し inactivated_at に統一 #1744

Phase 1: データベース最適化

• is_active カラムの削除
  • 全ての参照箇所を inactivated_at ベースのロジックに置き換え
  • マイグレーションファイルの作成
  • テストの更新

Phase 2: 命名の統一

• CSSクラス名の変更: inactive-item → inactivated-item

  • app/assets/stylesheets/custom.scss の更新
  • 全てのビューファイルでクラス名を更新
  • テストの更新（特に spec/requests/dojos_spec.rb）

• 変数名・メソッド名の統一

  • コントローラー内の変数名
  • モデルのスコープ名（必要に応じて）
  • コメントの更新

📝 技術的詳細

1. is_active カラムの削除

現在の状態：

• is_active (boolean) と inactivated_at (datetime) が重複
• 124個の非アクティブ道場でデータ整合性は確認済み

移行方法：

# 現在のロジック
dojo.is_active

# 新しいロジック  
dojo.inactivated_at.nil?

2. CSS クラス名の変更

影響範囲：

• /dojos ページのテーブル表示
• /stats#prefectures セクションのテーブル
• その他の統計関連ページ

3. 命名規則

統一方針：

• inactive (形容詞): 状態を表す場合は使わない
• inactivated (過去分詞): 「非アクティブ化された」という動作の結果を表す
• 例: inactivated_at (非アクティブ化された時刻)

🎯 受け入れ基準

1. 機能の維持: 既存の全機能が正常に動作すること
2. テスト: 全てのテストが成功すること
3. パフォーマンス: クエリ性能が劣化しないこと
4. 一貫性: システム全体で命名が統一されていること

🔗 関連情報

• PR 📊 Add inactivated_at to Dojo model to replace is_active boolean column #1726: inactivated_at カラムの追加
• PR 📥 Enable to download Dojo stats as CSV for specific year #1732: 年次フィルタリング機能の実装
• Issue Dojo モデルに inactived_at を追加し、より正確な統計情報にする #1373: 元の問題（統計グラフから非アクティブ道場が消える）

💡 実装のヒント

1. is_active の削除は慎重に

   • まず全ての参照箇所を洗い出す
   • grep -r "is_active" app/ lib/ spec/ で検索

2. CSSクラス名の変更

   • grep -r "inactive-item" app/views/ app/assets/ で影響範囲を確認

3. テストを先に修正

   • TDDアプローチで安全に進める

📌 メモ

このリファクタリングは機能追加ではなく、技術的負債の解消です。ユーザーに見える変化はありませんが、コードの保守性と一貫性が大幅に向上します。

---

優先度: 中（機能には影響しないが、長期的な保守性のために重要）
想定作業時間: 2-3時間
推奨時期: 次回の大きな機能追加の前

[yasulab]

@claude Review this issue, ultrathink.

[claude]

Claude finished @yasulab's task —— View job

---

リファクタリング計画のレビュー

このIssueで提案されているリファクタリングについて、コードベースを調査した結果をもとにフィードバックいたします。

✅ 良い実装

• 段階的なアプローチ: Phase 1（データベース最適化）→ Phase 2（命名統一）の順番は適切です
• データ整合性の確認: 124個の非アクティブ道場で整合性確認済みという事前準備が良いです
• 明確な受け入れ基準: 機能維持、テスト成功、パフォーマンス維持の基準が明確です
• inactivated_atの活用: タイムスタンプ型で非アクティブ化の履歴を保持できる設計が優れています

⚠️ 改善推奨

1. 移行期間の同期処理への依存

現在のapp/models/dojo.rbにはsync_active_statusという同期処理があります：

# app/models/dojo.rb:113-126
before_save :sync_active_status

def sync_active_status
  if is_active_changed?
    if is_active == false && inactivated_at.nil?
      self.inactivated_at = Time.current
    elsif is_active == true && inactivated_at.present?
      self.inactivated_at = nil
    end
  end
end

推奨事項:

• is_activeカラムを削除する際、この同期処理も削除する必要があります
• 削除前に、全データのis_activeとinactivated_atの整合性を最終確認するrakeタスクを作成することを推奨します

2. スコープの重複

現在2つのアプローチが混在しています：

# app/models/dojo.rb:17-18
scope :active,   -> { where(is_active: true ) }
scope :inactive, -> { where(is_active: false) }

# app/models/dojo.rb:21-24
scope :active_at, ->(date) { 
  where('created_at <= ?', date)
    .where('inactivated_at IS NULL OR inactivated_at > ?', date) 
}

推奨事項:

• is_active削除後、activeとinactiveスコープをinactivated_atベースに書き換える
• 新しいスコープ例：

  scope :active,   -> { where(inactivated_at: nil) }
  scope :inactive, -> { where.not(inactivated_at: nil) }

3. reactivate!メソッドの更新

# app/models/dojo.rb:106-108
update!(
  is_active: true,
  inactivated_at: nil
)

推奨事項: is_active削除後、このメソッドからis_active: trueを削除する必要があります

💡 提案

1. 安全な移行のための追加タスク

タスクリストに以下を追加することを提案します：

Phase 1の前に:

• データ整合性確認用のrakeタスクを作成
• 本番環境のバックアップ取得
• ステージング環境でのテスト実施

Phase 1.5: 並行実行期間（オプション）

• is_activeとinactivated_atの両方を残した状態でデプロイ
• 1-2週間の監視期間を設ける
• 問題がなければPhase 2へ進む

2. 命名規則の追加検討事項

inactive → inactivated の変更に加えて：

• Controllerレベル: app/controllers/dojos_controller.rb:31のソート条件

  order(is_active: :desc, order: :asc)

  を以下に変更：

  order(Arel.sql('CASE WHEN inactivated_at IS NULL THEN 0 ELSE 1 END'), order: :asc)

3. パフォーマンス最適化

• inactivated_atにインデックスは既に追加済み（db/migrate/20250805105147_add_inactivated_at_to_dojos.rbで確認）
• 複合インデックスの検討：頻繁にcreated_atとinactivated_atを同時に使用する場合

4. テストカバレッジの強化

現在のテスト（spec/models/dojo_spec.rb:89-101）は良い実装ですが、以下も追加推奨：

• active?メソッドのテスト
• active_at?メソッドのテスト
• CSS クラス名変更後の表示確認テスト

🚨 重大な問題

特に重大な問題は見つかりませんでした。提案されているリファクタリングは安全に実施可能です。

ただし、以下の点にご注意ください：

• Rakeタスクの更新: lib/tasks/dojos.rake:47でis_activeを設定している箇所の修正が必要
• YAMLデータの更新: db/dojos.yamlからis_activeフィールドを削除する必要があります

---

総評: このリファクタリング計画は良く練られており、実装可能です。上記の改善点を考慮いただければ、より安全で確実な移行が可能になります。

タスクリスト

• CLAUDE.md ファイルの確認
• 現在のコードベースの調査
• リファクタリング計画のレビューとフィードバック提供