開発方法論 (DDD/TDD)

# RED: 失敗するテストを先に作成
def test_user_registration():
    """
    GIVEN: 有効なユーザー情報があり
    WHEN: 会員登録を行うと
    THEN: ユーザーが作成され、ウェルカムメールが送信されること
    """
    user_service = UserService()
    result = user_service.register(
        email="[email protected]",
        password="SecurePass123!"
    )
 
    assert result.success is True
    assert result.user.id is not None
    assert email_service.welcome_email_sent("[email protected]") is True
 
# テスト実行 (失敗を想定 - 未実装)
# > pytest test_user_service.py - test_user_registration FAILED
 
# ====================================
 
# GREEN: 最小限のコードでテスト通過
class UserService:
    def register(self, email: str, password: str) -> RegistrationResult:
        user = User.create(email, password)
        user_repository.save(user)
        email_service.send_welcome(email)
        return RegistrationResult.success(user)
 
# テスト実行 (通過)
# > pytest test_user_service.py - test_user_registration PASSED
 
# ====================================
 
# REFACTOR: コード品質を改善 (テストは引き続き通過)
class UserService:
    def __init__(
        self,
        user_repo: UserRepository,
        email_service: EmailService,
        password_validator: PasswordValidator
    ):
        self.user_repo = user_repo
        self.email_service = email_service
        self.password_validator = password_validator
 
    def register(self, email: str, password: str) -> RegistrationResult:
        if not self.password_validator.validate(password):
            return RegistrationResult.failure("パスワードが無効です")
 
        user = User.create(email, password)
        self.user_repo.save(user)
        self.email_service.send_welcome(email)
        return RegistrationResult.success(user)
 
# テスト実行 (依然として通過)
# > pytest test_user_service.py - test_user_registration PASSED

間違った方法: 「コード全体を一度にすべて変更します!」
  --> 既存機能が壊れるリスクが高い
  --> 問題が発生した時、どこで間違ったか見つけにくい

正しい方法: 「テストで現在の動作を記録し、少しずつ変更します!」
  --> 既存機能が壊れたらテストがすぐに知らせてくれる
  --> 問題が発生したら最後の変更だけ元に戻せばよい

## コード分析レポート
 
- 対象: src/auth/ (認証モジュール)
- ファイル: 8 つの Python ファイル
- コード行数: 1,850 行
- テストカバレッジ: 5%
 
## 発見された問題
1. 重複した認証ロジック (3 箇所で同じコードが繰り返し)
2. ハードコードされた秘密鍵 (config.py に直接記述)
3. SQL Injection 脆弱性 (user_repository.py)
4. 不十分なテスト (5%、目標 85%)

class TestExistingLoginBehavior:
    """既存のログイン関数の現在の動作を記録するキャラクタリゼーションテスト"""
 
    def test_valid_login_returns_token(self):
        """
        GIVEN: 登録済みユーザーが存在し
        WHEN: 正しいパスワードでログインすると
        THEN: 現在の実装が返すレスポンスをそのまま記録
        """
        user = create_test_user(
            email="[email protected]",
            password="password123"
        )
 
        result = login_service.login("[email protected]", "password123")
 
        # 現在の動作をそのまま記録 (正否を判断しない)
        assert result["status"] == "success"
        assert result["token"] is not None
        assert result["expires_in"] == 3600  # 現在の有効期限
 
    def test_wrong_password_returns_error(self):
        """間違ったパスワードでのログイン時の現在の動作を記録"""
        create_test_user(email="[email protected]", password="password123")
 
        result = login_service.login("[email protected]", "wrongpassword")
 
        assert result["status"] == "error"
        assert result["code"] == 401

# BEFORE: 改善前のコード
def login(email, password):
    # SQL Injection 脆弱性
    user = db.query("SELECT * FROM users WHERE email = '" + email + "'")
    if user and check_password(user.password, password):
        token = generate_token(user.id)
        return {"status": "success", "token": token}
    return {"status": "error", "code": 401}
 
# ====================================
 
# AFTER: 改善後のコード (3 回のイテレーションを経て完成)
def login(email: str, password: str) -> LoginResult:
    """ユーザーログインを処理します。"""
    # イテレーション 1: パラメータ化クエリで SQL Injection を防止
    user = user_repository.find_by_email(email)
 
    if not user:
        return LoginResult.failure("資格情報が無効です")
 
    # イテレーション 2: 認証ロジックを集約
    if not auth_service.verify_password(user, password):
        return LoginResult.failure("資格情報が無効です")
 
    # イテレーション 3: トークンサービスを分離
    token = token_service.generate(user.id)
    return LoginResult.success(token)

# TDD 実行 (development_mode: tdd の場合)
> /moai run SPEC-AUTH-001

# DDD 実行 (development_mode: ddd の場合)
> /moai run SPEC-AUTH-001

constitution:
  development_mode: tdd  # TDD 方法論を使用
 
  tdd_settings:
    test_first_required: true         # 実装前のテスト作成必須
    red_green_refactor: true          # RED-GREEN-REFACTOR サイクル遵守
    min_coverage_per_commit: 80       # コミットあたりの最小カバレッジ
    mutation_testing_enabled: false   # ミューテーションテスト (任意)
 
  test_coverage_target: 85            # 全体カバレッジ目標

constitution:
  development_mode: ddd  # DDD 方法論を使用
 
  ddd_settings:
    require_existing_tests: true      # リファクタリング前の既存テスト必須
    characterization_tests: true      # キャラクタリゼーションテストの自動生成
    behavior_snapshots: true          # スナップショットテストの使用
    max_transformation_size: small    # 変更サイズの制限
    preserve_before_improve: true     # 保存してから改善必須
 
  test_coverage_target: 85            # 全体カバレッジ目標

問題点:
- SQL Injection 脆弱性 2 箇所
- ハードコードされた秘密鍵
- 重複した認証ロジック 3 箇所
- テストカバレッジ 5%
- コード複雑度が高い

# ステップ 1: SPEC 作成 (Plan)
> /moai plan "レガシー認証システムのリファクタリング。SQL Injection 修正、秘密鍵の環境変数化、認証ロジックの集約"
 
# manager-spec が SPEC-AUTH-REFACTOR-001 を作成

# ステップ 2: DDD 実行 (Run)
> /moai run SPEC-AUTH-REFACTOR-001
 
# manager-ddd が ANALYZE-PRESERVE-IMPROVE サイクルを実行
# ANALYZE: コード分析、問題点リスト作成
# PRESERVE: キャラクタリゼーションテスト 156 件作成
# IMPROVE: 3 回のイテレーションで段階的改善

# ステップ 3: ドキュメント同期 (Sync)
> /moai sync SPEC-AUTH-REFACTOR-001
 
# manager-docs が API ドキュメント更新、リファクタリングレポートを作成