name: maintenance-quality
description: 保守品質チェックの判断基準。maintenance-reviewerが使用する専用知識ベース。DRY・命名・複雑度・ドキュメントを対象とする。
user-invocable: false

保守品質基準 (maintenance-quality)

このドキュメントは maintenance-reviewer Agent 専用の知識ベースです。
長期的な保守性に関わる問題（DRY違反・命名・複雑度・ドキュメント不足）の検出基準を定義します。

DRY原則（重複コードの検出）

判定基準

• 同一ロジックの重複は共通関数化する
• 完全一致・ほぼ同一のコードブロックが2箇所以上 → HIGH
• 同一パターンが3箇所以上 → HIGH

検出手順

1. 同一文字列パターンを Grep で検索し、ヒット数が2件以上のものを候補とする
2. 同じ変数名で同じ処理をしているブロックを読んで確認する

典型パターン

深刻度

• 完全重複が2〜3箇所 → HIGH
• 無意味な分岐（両分岐が同一） → MEDIUM
• 軽微なパターン重複 → LOW

命名規則

判定基準（Python）

曖昧な名前の検出

以下の変数名は文脈によっては許容されるが、関数の戻り値や重要な中間変数に使われている場合は報告する：
- result, res, ret → 何のresultかを含む名前に改名
- data, info → 何のdataかを含む名前に改名
- tmp, temp → 一時変数でも意味のある名前を推奨
- x, y, i, j → ループ変数・座標以外での使用は警告

深刻度

• 命名規則違反（caseスタイル） → MEDIUM
• 曖昧な名前（意図不明） → LOW

複雑度

判定基準

複雑度超過への対処

• Early Return パターンで深いネストを解消
• 条件ロジックをヘルパー関数に切り出す
• 引数は dataclass / TypedDict にまとめる

ドキュメント

判定基準

• def 直後に """ 形式の docstring が存在しない → LOW
• docstring に Args / Returns / Raises セクションがない → LOW

推奨フォーマット（Google スタイル）

def get_user(username: str) -> Optional[tuple]:
    """ユーザー情報を取得する。

    Args:
        username: 検索対象のユーザー名。

    Returns:
        ユーザーレコードのタプル。存在しない場合は None。

    Raises:
        sqlite3.Error: DB接続・クエリ実行に失敗した場合。
    """

報告フォーマット

## 保守レビュー結果

### 発見事項

| 種別 | 深刻度 | 場所 | 問題 | 推奨対応 |
|---|---|---|---|---|
| DRY違反 | HIGH | search_users():62〜77行目 | role分岐の全分岐が同一処理 | 共通処理を1行に統合 |
| 複雑度超過 | HIGH | search_users():53〜96行目 | 分岐数13、閾値7超過 | ヘルパー関数に分割 |

### カウント
- CRITICAL: N件
- HIGH: N件
- MEDIUM: N件
- LOW: N件

### 総評
<保守性リスクの全体傾向と優先リファクタリング対象>

深刻度の優先順位: CRITICAL > HIGH > MEDIUM > LOW