开发方法论 (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 文档、生成重构报告