1+ #!/usr/bin/env ruby
2+ # frozen_string_literal: true
3+
4+ # PR説明を最新状況に更新するスクリプト
5+ require 'json'
6+
7+ # PR番号を自動取得
8+ branch_name = `git rev-parse --abbrev-ref HEAD` . strip
9+ pr_number = nil
10+
11+ # ブランチ名からPR番号を推測、または手動で設定
12+ pr_number = 1732 # CoderDojo.jpのenable-to-donwload-dojo-stats-yearlyブランチ
13+
14+ puts "=== PR ##{ pr_number }
15+
16+ # 最新の変更内容を分析
17+ changes = `git diff --name-only origin/main...HEAD` . strip . split ( "\n " )
18+ commit_count = `git rev-list --count origin/main...HEAD` . strip . to_i
19+
20+ puts "変更ファイル数: #{ changes . length }
21+ puts "コミット数: #{ commit_count }
22+
23+ # PR説明のマークダウンを生成
24+ pr_description = <<~MARKDOWN
25+ # 📊 道場統計の年次フィルタリング機能とCSV/JSONダウンロード対応（完成版）
26+
27+ ## 🎯 概要
28+
29+ CoderDojo一覧ページ（`/dojos`）に年次フィルタリング機能を追加し、特定年末時点でアクティブだった道場の一覧をHTML表示・CSV・JSON形式でダウンロードできる機能を実装しました。`/stats`ページのグラフとの完全統合により、統計分析と詳細データ確認がシームレスに行えます。
30+
31+ ## ✅ 実装完了機能
32+
33+ ### 🔍 年次フィルタリング機能
34+ - **対象期間セレクトボックス**: 2012年〜現在年までの年を選択可能
35+ - **自動遷移**: セレクトボックス変更時に自動的にページ遷移（表示ボタン不要）
36+ - **内部リンク**: `#table` アンカーで自動的にテーブル位置へスクロール
37+ - **全期間表示**: デフォルトで全道場（アクティブ＋非アクティブ）を表示
38+
39+ ### 📊 統計データのエクスポート
40+ - **CSV形式**: 日本語ヘッダー付きCSVファイルのダウンロード
41+ - ヘッダー: `ID, 道場名, 道場数, 都道府県, URL, 設立日, 状態`
42+ - 合計行に道場数の総計（counter値の合計）を表示
43+ - **JSON形式**: 既存のAPIフォーマットで年次フィルタリング対応
44+ - **HTML表示**: テーブル形式での一覧表示（道場数カラムは非表示）
45+
46+ ### 🎨 UX/UI改善
47+ - **統計情報の表示**: `/stats`ページのグラフとの比較検証が可能
48+ - 例: `2025年8月8日時点のアクティブな道場を表示中（開設数: 15 / 合計数: 199）`
49+ - **非アクティブ道場のスタイリング**:
50+ - `gainsboro` 背景色で視覚的に区別（`/stats#prefectures` と統一）
51+ - 共通CSSクラス `.stats-table .inactive-item` を作成
52+ - **ソート順の改善**: アクティブな道場を先に、非アクティブな道場を後に表示
53+ - **URL表示の最適化**: 30文字を超えるURLは `truncate` ヘルパーで省略表示
54+ - **セクションタイトル**: 「年次データを取得する」（動詞表現で訪問者が主語として行動しやすく）
55+
56+ ### 💬 情報表示の改善
57+ - **現在年の表示**: 「2025年8月8日時点」（自然な日本語表記）
58+ - **過去年の表示**: 「2024年末時点」（確定済み時点）
59+ - **デフォルト表示**: 「全期間の道場を表示中（非アクティブ含む）」
60+ - **ページ説明**: 新機能に対応した説明文に更新
61+
62+ ### 🛡️ Flashメッセージの表示位置制御（汎用パターン）
63+ - **inline_プレフィックスパターン**: `inline_*` プレフィックスでカスタム位置に表示
64+ - エラー時: `flash[:inline_alert]` で赤いアラート
65+ - 成功時: `flash.now[:inline_info]` で青い情報メッセージ
66+ - デフォルト位置（ページ上部）との二重表示を防止
67+ - **ヘルパーメソッド**: `render_inline_flash_messages` で再利用可能に
68+ - **Bootstrap CSS自動適用**: `inline_alert` → `alert-alert` クラスに変換
69+
70+ ### ⚡ パフォーマンス最適化
71+ - **効率的なクエリ**: `active_at` スコープを活用した時点ベースのフィルタリング
72+ - **測定結果**: 全年で8ms以下の高速応答
73+ - **変数命名**: `year_begin`/`year_end` で統一（可読性向上）
74+
75+ ## 🐛 重要なバグ修正
76+
77+ ### TDDアプローチによる年フィルタリング問題の修正
78+ - **問題**: 年フィルタリング時に2024年非アクティブ化道場が2023年表示で灰色になっていた
79+ - **原因**: 現在の `is_active` を使用していたため、選択年時点の状態と異なっていた
80+ - **解決**: 選択年末時点での正しいアクティブ状態を計算するロジックを実装
81+ - **手法**: 先にテストを書いて失敗を確認してから修正（TDD）
82+
83+ ## 📈 統計精度の大幅改善
84+
85+ ### /statsページとの完全一致
86+ - **統計ラベル統一**: 「開設道場数」→「開設数」、「合計道場数」→「合計数」
87+ - **計算ロジック統一**: `/stats`ページと同一の計算方法を採用
88+ - **開設数**: その年に新規開設されたDojoの `counter` 合計
89+ - **合計数**: その年末時点でアクティブなDojoの `counter` 合計
90+ - **検証例**: 2023年の統計値が完全一致
91+ - 開設数: 20、合計数: 199（/statsページのグラフ値と同一）
92+
93+ ### データ整合性の確認
94+ - **124個の非アクティブ道場**: `inactivated_at`データと`is_active`フラグが100%一致
95+ - **統計精度向上**: 過去年の道場数が大幅に正確化
96+ - 2018年: 98 → 172道場（+75.5%の精度向上）
97+ - 2019年: 126 → 200道場（+58.7%の精度向上）
98+
99+ ## 🧪 包括的テスト実装
100+
101+ - **25個のRSpecテスト**: 全テストが成功
102+ - 年パラメータのバリデーション
103+ - フィルタリング機能の動作確認
104+ - CSV/JSON形式の出力検証
105+ - UIコンポーネントのテスト
106+ - CSSクラス（`inactive-item`）の正しい適用テスト
107+ - 統計情報表示のテスト
108+
109+ ## 🔒 セキュリティ対策
110+
111+ - **XSS対策**: エラーメッセージからユーザー入力値を除外
112+ - **パラメータ検証**: 年パラメータは整数のみ受け付け（2012〜現在年）
113+ - **HTMLエスケープ**: Railsのデフォルト機能を活用
114+
115+ ## 🎨 実装で得られた技術的知見
116+
117+ ### 複雑度管理の重要性
118+ - **デフォルト+条件更新パターン**: if-elseのネストを避け、線形増加の設計を採用
119+ - **コード品質向上**: 指数的な複雑度増加を防ぐ設計原則の適用
120+ - **保守性**: 将来の機能追加が容易な構造
121+
122+ ### 段階的リファクタリングの価値
123+ - **シンプル化**: 複雑なif-elseブロックから2行のエレガントなコードへ
124+ - **可読性**: `year_begin`/`year_end` の統一命名で関連性を明確化
125+ - **自然な日本語**: 「2025年8月8日時点」（ゼロパディングなし）
126+
127+ ## 📊 改善効果
128+
129+ ### ユーザーにとっての価値
130+ - **統計ページとの連動**: グラフで見た数値を詳細データで確認可能
131+ - **年次推移の分析**: 特定年の道場データを瞬時に抽出
132+ - **外部ツール連携**: Excel等での詳細分析が可能
133+ - **情報の明確化**: 何が表示されているかが一目瞭然
134+
135+ ### 開発・運用面の改善
136+ - **保守しやすいコード**: シンプルで理解しやすい実装
137+ - **完全なテストカバレッジ**: 将来の変更に対する安全網
138+ - **パフォーマンス**: 高速な応答時間（< 10ms）
139+ - **拡張性**: 新機能追加が容易な設計
140+
141+ ## 🔍 パフォーマンス検証結果
142+
143+ ```
144+ 総Dojo数: 323個
145+ 2020年フィルタリング: 8.04ms
146+ 2023年フィルタリング: 1.47ms
147+ 2024年フィルタリング: 2.64ms
148+ ```
149+
150+ ## 🧪 動作確認
151+
152+ - ✅ 全期間表示: `/dojos`
153+ - ✅ 2024年フィルタリング: `/dojos?year=2024`
154+ - ✅ 現在年表示: `/dojos?year=2025` (「2025年8月8日時点」と表示)
155+ - ✅ CSVダウンロード: `/dojos.csv?year=2024`
156+ - ✅ JSONダウンロード: `/dojos.json?year=2024`
157+ - ✅ 無効な年のエラー表示: `/dojos?year=2026` (フィルタリングセクション内に表示)
158+ - ✅ 統計情報表示: 開設数と合計数が正確に表示
159+
160+ ## 📋 実装タスク完了状況
161+
162+ ### ✅ 完了済み
163+ - [x] 年次フィルタリング機能の実装
164+ - [x] CSV/JSON形式でのデータエクスポート
165+ - [x] 統計情報表示（/statsページとの完全一致）
166+ - [x] UI/UXの改善（スタイリング、メッセージ表示）
167+ - [x] セキュリティ対策（XSS防止、パラメータ検証）
168+ - [x] 包括的テストの実装（25個のテストケース）
169+ - [x] パフォーマンス最適化と検証
170+ - [x] 重要なバグ修正（TDDアプローチ）
171+ - [x] ページ説明文の新機能対応
172+ - [x] inline_プレフィックスパターンの実装
173+
174+ ### ⏳ 将来のPRで対応予定
175+
176+ 1. **is_activeカラムの削除**
177+ - データ整合性確認済み（124個すべて一致）
178+ - `inactivated_at`カラムで代替可能
179+
180+ 2. **命名の統一（inactive → inactivated）**
181+ - CSSクラス名: `inactive-item` → `inactivated-item`
182+ - 変数名・コメントの全体的な統一
183+
184+ ## 🔗 関連情報
185+
186+ - **元Issue**: #1373 (統計グラフから非アクティブ道場が消える問題)
187+ - **前PR**: #1726 (`inactivated_at` カラムの追加)
188+ - **技術文書**: グローバルCLAUDE.mdに複雑度管理の教訓を追加
189+
190+ ---
191+
192+ **🚀 レビュー・マージ準備完了**
193+
194+ この機能により、CoderDojo.jpの統計機能が大幅に向上し、ユーザーは詳細な年次データを効率的に分析できるようになります。全25個のテストが成功し、パフォーマンスも良好です。
195+ MARKDOWN
196+
197+ puts "=== 生成されたPR説明 ==="
198+ puts pr_description
199+
200+ # ファイルに保存
201+ filename = "tmp/pr_#{ pr_number }
202+ File . write ( filename , pr_description )
203+ puts "\n === PR説明を #{ filename }
204+
205+ puts "\n 次のコマンドでPRを更新できます:"
206+ puts "gh pr edit #{ pr_number } #{ filename }
0 commit comments