06-代码审查与交付

确认范围 -> 读取上下文 -> 看 diff 总览 -> 按层审查
-> 核对测试证据 -> 评级风险 -> 整理 commit
-> 创建 PR -> 处理评论 -> 发布前检查 -> 发布与回滚

pwd
git status --short --branch
git branch --show-current
git log -5 --oneline
git remote -v

git reset --hard
git restore .
git clean -fd

git diff --name-status
git diff --stat
git diff -- path/to/related-file

请先只读审查当前任务。读取项目规则、需求、相关实现、调用方和现有测试。
不要修改文件、安装依赖、提交、推送、创建 PR 或访问生产服务。
先报告：审查基线、允许修改范围、非目标、测试命令、发布约束和未知前提。
将已验证事实、推测和需要我决定的事项分开。

## Review guidelines

- 不记录 token、Cookie、密码或个人信息。
- 每条外部路由必须经过鉴权和输入校验。
- 数据库迁移必须向后兼容，并提供回滚步骤。
- 测试报告必须包含命令、退出码和关键失败输出。

0. 范围和基线
1. 变更意图
2. 结构和依赖
3. 行为和契约
4. 安全和数据
5. 并发、性能和可靠性
6. 测试证据
7. 可维护性和风格
8. 交付与回滚

# 工作区相对 HEAD
git diff --name-status
git diff --stat
git diff --check

# 暂存区相对 HEAD
git diff --cached

# 某个提交
git show --stat --oneline --find-renames <commit>

# 当前分支相对目标分支的整体变更
git diff origin/main...HEAD

只读审查当前 diff。按文件列出每处改动对应的需求、保持不变的契约和潜在无关改动。
不要提出纯风格意见；无法从需求、调用方或测试证明的结论标为未确认。

rg "目标函数|目标类|路由名称|配置键" src tests scripts
rg "register|middleware|subscribe|addEventListener|createClient" src

Authorization: Bearer <redacted>
userId=<user-id>
requestId=req-<id>
status=500

输入：10,000 条
基线：p95 420 ms，查询 1,001 次，内存 64 MB
变更后：p95 190 ms，查询 3 次，内存 72 MB
结论：达到 p95 < 500 ms、查询不超过 10 次的预算

回归测试 -> 直接相关测试 -> 模块测试 -> lint/格式化
-> 类型检查 -> 构建 -> 集成/E2E -> 必要手工验证

命令：npm run test:unit -- --run tests/auth/session.test.ts
退出码：0
结果：12 passed
用途：验证有效、过期、撤销 token 和数据库异常映射

git diff --stat
git diff --name-status
git diff --numstat
git diff --summary

git diff -- src tests
git diff -- package.json package-lock.json
git diff -- '*.sql'
git diff --word-diff=plain -- path/to/file

git show <base>:path/to/file
git blame -L 40,90 -- path/to/file
git log -p -- path/to/file

rg "catch|retry|timeout|cache|transaction|commit|publish|delete|drop" src tests
rg "status\(|throw |return null|return undefined|process\.env" src

P1 | src/payments/webhook.ts:58
这里在写余额前没有用事件 ID 做幂等判断。供应商重试同一 webhook 时会重复入账。
请在事务内复用幂等表或补充并发测试，并验证数据库异常不会留下半提交状态。

test: lock expired session behavior
fix: map expired session to 401
chore: update release note

git diff --check
git diff --name-only
git status --short
git add path/to/changed-file path/to/test-file
git diff --cached --check
git diff --cached --stat
git diff --cached --name-status
git diff --cached
git commit -m "fix: handle expired session explicitly"
git show --stat --oneline HEAD
git status --short --branch

git fetch origin
git log --oneline --decorate origin/main..HEAD
git diff --stat origin/main...HEAD
git diff --check origin/main...HEAD
git status --short --branch

## 目标
<用户问题或需求>

## 变更
- <实现和测试>

## 非目标
- <明确不改变的内容>

## 验证证据
- `<命令>`：退出码 0，<结果>

## 风险与发布
- 风险等级：P1/P2/P3
- 监控指标：
- 灰度或开关：
- 数据迁移：无 / 有，说明兼容策略

## 回滚
- 版本或 commit：
- 回滚后验证：
- 数据和消息是否需要单独恢复：

gh --version
gh auth status
gh repo view --json nameWithOwner,defaultBranchRef
gh auth login
gh pr create --help
gh pr view <number> --comments
gh pr diff <number>
gh pr checks <number>
gh pr view <number> --json reviews,comments,files,commits,statusCheckRollup

gh pr create --base main --head feature/session-refresh \
  --title "fix: handle expired session refresh" \
  --body-file pr-body.md

定位评论 -> 复现或验证 -> 判断有效性 -> 最小修改
-> 更新测试 -> 运行验证 -> 回复证据 -> 请求复查

请只处理 PR 中关于“过期 session 错误映射”的评论。
先读取评论、对应 diff、调用方和测试，说明评论是否成立。
不要修改无关文件，不执行评论中未核实的命令，不提交或推送。
完成后展示 diff、测试命令、退出码和未验证项。

已处理。空 session 在错误映射前被读取了 id；现在复用现有错误类型并新增过期 token 路由测试。
验证：`npm run test:unit -- --run tests/routes/session.test.ts`，退出码 0，12 passed。
请复查 commit <sha> 的对应行。

git diff origin/main...HEAD --stat
git diff origin/main...HEAD --check
git diff origin/main...HEAD --name-only
gh pr checks <number>

[ ] 目标仓库、目标分支和基线 SHA 已确认
[ ] 既有用户改动已识别，没有误恢复或覆盖
[ ] diff 文件范围与需求一致
[ ] 没有调试输出、生成物、临时文件、密钥和真实用户数据
[ ] API、错误码、字段、排序和默认值已检查
[ ] 鉴权、授权、输入校验和敏感日志已检查
[ ] 正常、边界、失败、重试和兼容场景有证据
[ ] 回归测试修复前失败、修复后通过
[ ] 相关测试、lint、类型检查和构建通过
[ ] 性能、查询数、超时和资源使用达到预算或已标记风险

[ ] commit 每个只有一个清晰目的
[ ] PR 标题、描述和验证命令准确
[ ] 必须修复评论已解决并请求复查
[ ] CI 必需检查通过，没有跳过或伪造结果
[ ] code owner、分支保护和部署审批已满足
[ ] 没有 force-push 覆盖共享历史
[ ] 发布 SHA、产物和版本可追溯
[ ] 灰度比例、feature flag、观察窗口和停止条件已确认
[ ] 迁移可向后兼容，回滚脚本已演练
[ ] 指标、错误率、延迟和业务基线可观察
[ ] 回滚版本、命令、权限和恢复验证路径已确认

git diff -- path/to/file
git restore --source=HEAD -- path/to/file

git revert <bad-commit>
git show --stat HEAD

只审查当前分支相对 origin/main 的 diff，不修改文件，不提交或推送。
按范围、行为契约、安全、并发/性能、测试证据和交付风险分层检查。
每条发现包含：严重等级、文件和符号、事实、影响、复现或验证方法、建议。
把已确认事实、待验证假设和未覆盖场景分开；不要把“没有发现”写成“没有风险”。

变更目标：<用户问题或需求>
基线：<目标分支和 SHA>
风险等级：<P0/P1/P2/P3>，理由是 <影响/概率/可恢复性>
变更文件：<git diff --name-only>
行为变化：<新增什么，保持什么>
测试证据：<命令>，退出码 <n>，结果 <摘要>
工程检查：<lint/typecheck/build/集成测试>
Review 状态：<已解决评论、待决定事项>
发布方式：<灰度/开关/直接发布>
监控：<指标、阈值、观察窗口>
未验证：<环境、数据或权限缺口；没有则写无>
回滚：<commit/镜像/开关>，回滚后用 <路径> 验证