Skip to content

段階的リファクタリング計画を docs/refactoring に追加 - #68

Merged
champierre merged 1 commit into
mainfrom
feature/refactoring-plan
Jun 11, 2026
Merged

段階的リファクタリング計画を docs/refactoring に追加#68
champierre merged 1 commit into
mainfrom
feature/refactoring-plan

Conversation

@champierre

Copy link
Copy Markdown
Contributor

概要

コードベースに蓄積した技術的負債を調査し、6 フェーズで解消する詳細なリファクタリング計画を docs/refactoring/ に追加しました。ドキュメントのみの追加で、コードの変更はありません。

調査で確認した主な負債

  • index.html 3,005 行の約 9 割が埋め込み CSS(25 以上のクラスが重複定義)
  • フロントテストが実コードを import せず、テスト内の実装コピーを検証する運用(実装を壊しても緑のまま)
  • server/src のルート層に try/catch 定型が 37 箇所、5 モデルにほぼ同一の CRUD 約 48 メソッド
  • CalendarService.ts 863 行に 16 責務が同居
  • E2E が全件 skip・CI 未接続、フロントのカバレッジ計測対象が空(閾値 3%)
  • CLAUDE.md の「アーキテクチャ」節が旧構成(ポート 8081 等)のまま

フェーズ構成

フェーズ 内容
0 リポジトリ衛生とドキュメント整合(挙動変更なし・即効)
1 テスト基盤の再建(以降の安全網。実コード接続・統合テスト導入)
2 バックエンド構造(エラーミドルウェア・モデル共通化・CalendarService 分割)
3 フロントエンド構造(重複排除・API 一元化・ES modules 化)
4 CSS/HTML(外部化・重複解消)
5 E2E と CI 品質ゲート(成果の固定化)

各フェーズ文書に、背景(ファイル:行番号つきの事実)、PR 分割単位の作業チェックリスト、受け入れ条件、リスクと対策、完了条件(DoD)を記載しています。

🤖 Generated with Claude Code

2026-06-11 のコードベース調査に基づき、蓄積した技術的負債を
6 フェーズで解消する詳細計画を文書化。

- README: 負債サマリ・フェーズ依存関係・共通原則・効果測定
- Phase 0: リポジトリ衛生とドキュメント整合(挙動変更なし)
- Phase 1: テスト基盤の再建(実コード未検証テストの解消)
- Phase 2: バックエンド構造(定型重複・CalendarService 分割)
- Phase 3: フロントエンド構造(重複排除・ES modules 化)
- Phase 4: CSS/HTML(埋め込み CSS 約2,700行の外部化・重複解消)
- Phase 5: E2E と CI 品質ゲート

Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
@champierre
champierre merged commit c234bd5 into main Jun 11, 2026
4 checks passed
@champierre
champierre deleted the feature/refactoring-plan branch June 11, 2026 15:32
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant