開発方法論 (DDD/TDD)

# RED: 失敗するテストを先に作成
def test_user_registration():
    """
    GIVEN: 有効なユーザー情報があり
    WHEN: 会員登録をすると
    THEN: ユーザーが作成され、ウェルカムメールが送信されるべき
    """
    user_service = UserService()
    result = user_service.register(
        email="newuser@example.com",
        password="SecurePass123!"
    )

    assert result.success is True
    assert result.user.id is not None
    assert email_service.welcome_email_sent("newuser@example.com") 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="test@example.com",
            password="password123"
        )

        result = login_service.login("test@example.com", "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="test@example.com", password="password123")

        result = login_service.login("test@example.com", "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            # 全体カバレッジの目標

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

# manager-spec が SPEC-AUTH-REFACTOR-001 を生成

# ステップ 2: DDD の実行 (Run)
> /moai run SPEC-AUTH-REFACTOR-001

# manager-develop が ANALYZE-PRESERVE-IMPROVE サイクルを実行
# ANALYZE: コード分析、問題点リストの生成
# PRESERVE: 特性テスト 156個を作成
# IMPROVE: 3回の反復による段階的改善

# ステップ 3: ドキュメント同期 (Sync)
> /moai sync SPEC-AUTH-REFACTOR-001

# manager-docs が API ドキュメントの更新、リファクタリングレポートの生成