单 Agent 任务委派 — 把重复劳动交给 AI#

任务：给当前 Express 项目的所有路由添加请求日志中间件。

请先读 src/index.ts 和 src/routes/ 下的全部路由文件，了解现有路由注册方式，
然后在 src/middleware/logger.ts 中创建一个请求日志中间件，要求：

1. 记录每个请求的 method、url、status code、响应时间（ms）
2. 使用 console.log 即可，格式为 [timestamp] METHOD /path - STATUS - XXms
3. 不要修改任何路由的业务逻辑，只注册中间件
4. 在 src/index.ts 中 app.use() 全局注册，不要在每个路由文件里单独引入

完成后列出：修改了哪些文件、为什么这样设计、如果有多种方案说明你的选择理由。

## 交付摘要

### 新建文件：src/middleware/logger.ts

```typescript
import type { Request, Response, NextFunction } from 'express'

export function requestLogger(req: Request, res: Response, next: NextFunction) {
  const start = Date.now()
  const { method, url } = req

  // 监听 response finish 事件，拿到实际 status code
  res.on('finish', () => {
    const duration = Date.now() - start
    const timestamp = new Date().toISOString()
    console.log(`[${timestamp}] ${method} ${url} - ${res.statusCode} - ${duration}ms`)
  })

  next()
}

## 现状分析

扫描 src/controllers/ 后，发现三种分散的错误处理模式：

| 文件 | 当前写法 | 问题 |
| ---- | -------- | ---- |
| userController.ts | `try-catch` + `res.status(500).json({ error: e.message })` | 状态码写死 500 |
| productController.ts | `try-catch` + 混用 `res.send` / `res.json` | 格式不统一 |
| orderController.ts | 无错误处理，let it crash | 缺失 |

## 设计方案

### 新建 src/utils/AppError.ts

```typescript
export class AppError extends Error {
  constructor(
    message: string,
    public statusCode: number = 500,
    public code?: string
  ) {
    super(message)
    this.name = 'AppError'
  }
}

## 源码分析

User 模块有 2 个端点：
- `GET /api/users/:id` — 查询单个用户
- `POST /api/users` — 创建用户（body: { name, email }）

jest.config.ts 配置：preset: 'ts-jest', testEnvironment: 'node'，支持 supertest。

## __tests__/user.test.ts

```typescript
import request from 'supertest'
import express, { type Express } from 'express'
import userRouter from '../src/routes/user'

// Mock User model — 隔离数据库依赖
jest.mock('../src/models/User', () => ({
  User: {
    findById: jest.fn(),
    create: jest.fn()
  }
}))

import { User } from '../src/models/User'

describe('User API', () => {
  let app: Express

  beforeEach(() => {
    app = express()
    app.use(express.json())
    app.use('/api/users', userRouter)
    jest.clearAllMocks()
  })

  describe('GET /api/users/:id', () => {
    it('should return user when found', async () => {
      const mockUser = { id: '1', name: 'Alice', email: 'alice@example.com' }
      ;(User.findById as jest.Mock).mockResolvedValue(mockUser)

      const res = await request(app).get('/api/users/1')

      expect(res.status).toBe(200)
      expect(res.body).toEqual(mockUser)
    })

    it('should return 404 when user not found', async () => {
      ;(User.findById as jest.Mock).mockResolvedValue(null)

      const res = await request(app).get('/api/users/999')

      expect(res.status).toBe(404)
      expect(res.body.error).toBeDefined()
    })

    it('should return 400 when id format is invalid', async () => {
      const res = await request(app).get('/api/users/invalid-id')

      expect(res.status).toBe(400)
    })
  })

  describe('POST /api/users', () => {
    it('should create a user and return 201', async () => {
      const newUser = { name: 'Bob', email: 'bob@example.com' }
      ;(User.create as jest.Mock).mockResolvedValue({ id: '2', ...newUser })

      const res = await request(app)
        .post('/api/users')
        .send(newUser)

      expect(res.status).toBe(201)
      expect(res.body.name).toBe('Bob')
    })

    it('should return 400 when email is missing', async () => {
      const res = await request(app)
        .post('/api/users')
        .send({ name: 'Bob' })

      expect(res.status).toBe(400)
    })
  })
})

PASS  __tests__/user.test.ts
  User API
    GET /api/users/:id
      ✓ should return user when found (12 ms)
      ✓ should return 404 when user not found (5 ms)
      ✓ should return 400 when id format is invalid (3 ms)
    POST /api/users
      ✓ should create a user and return 201 (8 ms)
      ✓ should return 400 when email is missing (4 ms)

Test Suites: 1 passed, 1 total
Tests:       5 passed, 5 total
Time:        2.1 s

# Agent A — 类风格
"用 AppError 类 + 全局 error handler middleware，参考 express 官方文档"

# Agent B — 函数式风格
"用 Result<T, E> 模式，每个 controller 返回 Result 类型，
 由 wrapAsync 高阶函数统一处理，不要用 class"

# Agent C — express-async-errors 方案
"用 express-async-errors 库消除 try-catch 样板代码，
 结合 zod 做输入校验，错误统一由 error handler 处理"