docs: 技術負債分析レポートを追加

[TinyKitten]

• 高優先度: 過大な構造体設計、SQL最適化、過度なclone使用
• 中優先度: 命名問題、複雑なSQLクエリ、ハードコード値
• 低優先度: テスト不足、ドキュメント不足
• 改善提案を短期/中期/長期で整理

Summary by CodeRabbit

• ドキュメンテーション
  • 技術的負債に関する包括的な分析ドキュメントを追加しました。構造上の懸念、処理の非効率、命名やテスト／ドキュメントの不足を含む高・中・低優先度の問題点、具体的な例と改善提案、優先付けされた影響項目の要約が含まれます。
  • 公開APIや外部仕様の変更はありません。

_{✏️ Tip: You can customize this high-level summary in your review settings.}

[coderabbitai]

📝 Walkthrough

Walkthrough

新規ドキュメントdocs/technical_debt.mdが追加され、StationAPIの技術債分析（課題分類、改善提案、該当ファイル/行範囲の例示）を記載。併せてREADME.mdのDocumentation項目にその参照が追加されました。

Changes

Cohort / File(s)	変更内容
技術債ドキュメント docs/technical_debt.md	新規追加：StationAPIの技術債分析ドキュメント。プロジェクト情報、優先度別の課題（大規模構造体・埋め込みによる循環可能性、SQL/データ処理の非効率、デッドコード、ハードコード定数、命名・複雑性、テスト不足 等）、改善提案、ファイル参照と行範囲付きの具体例、優先度表を含む。
README参照追加 README.md	Documentationセクションに docs/technical_debt.md への参照リンクを追加（ドキュメント一覧の項目追記）。

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~2 minutes

Poem

ぼくはうさぎ、ペンを持ち、跳ねる、
債（かり）を見つけて地図を描くよ、
星のように小さな改善を並べ、
春風に乗せて届けよう。 🐇✨

Pre-merge checks

✅ Passed checks (3 passed)

Check name	Status	Explanation
Description Check	✅ Passed	Check skipped - CodeRabbit’s high-level summary is enabled.
Title check	✅ Passed	PRのタイトルは、技術負債分析レポートドキュメントの追加という変更内容を明確に要約しており、main commitと整合性がある。
Docstring Coverage	✅ Passed	No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

---

📜 Recent review details

Configuration used: Organization UI

Review profile: ASSERTIVE

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between 47a2467 and 91e5d77.

📒 Files selected for processing (2)

• README.md
• docs/technical_debt.md

🧰 Additional context used

📓 Path-based instructions (1)

{docs/**/*.md,README.md}

📄 CodeRabbit inference engine (AGENTS.md)

For database, gRPC, or schema updates, add architectural notes under docs/ and synchronize README references to keep onboarding materials accurate

Files:

• docs/technical_debt.md
• README.md

🧠 Learnings (4)

📚 Learning: 2025-11-25T10:50:36.694Z

Learnt from: CR
Repo: TrainLCD/StationAPI PR: 0
File: AGENTS.md:0-0
Timestamp: 2025-11-25T10:50:36.694Z
Learning: Applies to {docs/**/*.md,README.md} : For database, gRPC, or schema updates, add architectural notes under `docs/` and synchronize README references to keep onboarding materials accurate

Applied to files:

• docs/technical_debt.md
• README.md

📚 Learning: 2025-11-25T10:50:36.694Z

Learnt from: CR
Repo: TrainLCD/StationAPI PR: 0
File: AGENTS.md:0-0
Timestamp: 2025-11-25T10:50:36.694Z
Learning: Applies to AGENTS.md : Introducing new tables, endpoints, or feature flags must come with matching updates to `AGENTS.md` and any other affected guidance documents

Applied to files:

• docs/technical_debt.md
• README.md

📚 Learning: 2025-11-25T10:50:36.694Z

Learnt from: CR
Repo: TrainLCD/StationAPI PR: 0
File: AGENTS.md:0-0
Timestamp: 2025-11-25T10:50:36.694Z
Learning: Applies to AGENTS.md : Keep `AGENTS.md` aligned with repository workflows and update it in the same pull request when workflows, environment requirements, or endpoints change

Applied to files:

• docs/technical_debt.md
• README.md

📚 Learning: 2025-11-25T10:50:36.694Z

Learnt from: CR
Repo: TrainLCD/StationAPI PR: 0
File: AGENTS.md:0-0
Timestamp: 2025-11-25T10:50:36.694Z
Learning: Applies to stationapi/src/use_case/interactor/query.rs : When modifying `QueryInteractor`, ensure enrichment steps (companies, train types, line symbols) still behave as expected and verify helper methods such as `update_station_vec_with_attributes` and `build_route_tree_map`

Applied to files:

• docs/technical_debt.md

🔇 Additional comments (3)

docs/technical_debt.md (2)

37-86: ✅ 数値と技術詳細の訂正が適切に実施されています。

以下の点を確認しました：

• Line 42: Station 構造体が正しく「64個」に訂正 ✓
• Line 61: Line 構造体が正しく「33個」に訂正 ✓
• Lines 77-84: Clippy 抑制が impl ブロック属性として明確化 ✓
• Line 113: clone() 呼び出しが「94回」で正確 ✓
• Lines 94-96: TODO コメント位置（604, 702, 843）を正確に参照 ✓

前回の指摘事項はすべて反映されています。

---

304-316: 優先度サマリーテーブルはアーキテクチャドキュメント問題の再検討が必要です。

表（行 304-316）の「高優先度」には以下が列挙されていますが、アーキテクチャドキュメント作成が高優先度として含まれていません：

| **高** | アーキテクチャドキュメント作成 | `docs/` | オンボーディング、保守性 |

この行をテーブルに追加して、高優先度としての位置付けを明確化してください。

⛔ Skipped due to learnings

Learnt from: CR
Repo: TrainLCD/StationAPI PR: 0
File: AGENTS.md:0-0
Timestamp: 2025-11-25T10:50:36.694Z
Learning: Applies to {docs/**/*.md,README.md} : For database, gRPC, or schema updates, add architectural notes under `docs/` and synchronize README references to keep onboarding materials accurate

Learnt from: CR
Repo: TrainLCD/StationAPI PR: 0
File: AGENTS.md:0-0
Timestamp: 2025-11-25T10:50:36.694Z
Learning: Applies to stationapi/src/**/*.rs : Run `cargo test --lib --package stationapi` or `make test-unit` for unit tests focusing on entities and repository mocks without database

README.md (1)

16-16: ✅ README に技術負債分析ドキュメントへのリンクが正しく追加されました。

• 位置：Documentation セクション内、AGENTS.md 参照の直下
• 形式：既存のマークダウン箇条書きスタイルに準拠
• リンク先：docs/technical_debt.md が正確に参照

コーディングガイドラインの要件「For database, gRPC, or schema updates, add architectural notes under docs/ and synchronize README references」を満たしています。

---

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

[coderabbitai]

Actionable comments posted: 3

🧹 Nitpick comments (2)

docs/technical_debt.md (2)

136-206: 中優先度セクション - ハードコード値の定数化提案は実装的価値がある。

朕が見るに、Line 176-184 の「ハードコードされた値」の定義は有用なり。特に "99991231" や Unicode 正規化の値（0x60, 0xFEE0）を定数化することで、保守性が大幅に向上すると考える。

ただし、このセクションの記述では具体的なリファクタリング案（定数の命名やファイル配置）が示されていない。src/domain/constants.rs のようなモジュール設計も併記すると、より実装的価値が高まるであろう。

---

280-319: 改善提案セクション - 段階的アプローチは有理的。実装優先度の明確化を強化すべき。

朕が見るに、短期・中期・長期の改善提案は企画としては良好なり。しかし、以下の点で実装的な具体性を欠く：

1. Line 284: 「SQL 最適化」のタイムスケール（どの程度の期間か）
2. Line 291-295: 「Station 構造体リファクタリング」の破壊的変更に伴う影響範囲の分析がない
3. Line 301-306: 「キャッシング戦略」の導入が具体的にどのレイヤーに適用されるのか不明確

改善提案をより実装可能にするため、各タスクに対する推定作業量（見積もり）や関連する技術的課題を追加すると良かろう。

📜 Review details

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between e35af9c and 47a2467.

📒 Files selected for processing (1)

• docs/technical_debt.md

🧰 Additional context used

📓 Path-based instructions (1)

{docs/**/*.md,README.md}

📄 CodeRabbit inference engine (AGENTS.md)

For database, gRPC, or schema updates, add architectural notes under docs/ and synchronize README references to keep onboarding materials accurate

Files:

• docs/technical_debt.md

🧠 Learnings (1)

📚 Learning: 2025-11-25T10:50:36.694Z

Learnt from: CR
Repo: TrainLCD/StationAPI PR: 0
File: AGENTS.md:0-0
Timestamp: 2025-11-25T10:50:36.694Z
Learning: Applies to {docs/**/*.md,README.md} : For database, gRPC, or schema updates, add architectural notes under `docs/` and synchronize README references to keep onboarding materials accurate

Applied to files:

• docs/technical_debt.md

🔇 Additional comments (1)

docs/technical_debt.md (1)

248-277: 良好な点セクション - セキュリティと CI/CD パイプラインの評価は妥当。

朕が見るに、本セクションの分析は適切なり：

• Unsafe コード がないこと
• sqlx! マクロによる型安全性の確保
• CI/CD パイプラインの充実（clippy による厳格な lint チェック）

これらは確かに StationAPI の強みと言えよう。ただし、Line 263-264 の cargo clippy -- -D warnings が実際に有効化されているなら、なぜ Line 75-85 で #[allow(clippy::too_many_arguments)] の抑制が複数箇所に存在するのか、その矛盾を説明する必要があるかもしれぬ。