fix(FEAT-QA-001): 隔离市场概览 E2E 测试时钟

2026-08-24 17:53:09 +08:00
parent 7904571ddd
commit 368672f001
19 changed files with 408 additions and 3 deletions
@@ -0,0 +1,50 @@
+# 代码 Review
+
+- 功能编号：FEAT-QA-001
+- reviewer_type：independent
+- reviewer：Codex 独立审查代理
+- 基线提交：1f6ed00
+- 审查提交：working-tree
+- 日期：2026-08-24
+
+## 审查范围
+
+市场概览 HTTP E2E 的测试时钟生命周期和对应功能档案。
+
+## PRD 与设计符合性
+
+实现符合 PRD：测试时钟固定到夹具交易日，生产行情和新鲜度逻辑不变；完整门禁已通过，等待关闭本轮独立 Review 发现项。
+
+## 发现
+
+| 严重程度 | 位置 | 问题 | 影响 | 处理 |
+|---|---|---|---|---|
+| Important | E2E cleanup | `app.close()` 抛错可能跳过时钟恢复 | 污染同进程后续测试 | 使用 `try/finally` 保证恢复 real timers，定向与完整门禁通过 |
+| Important | README | 将尚不存在的强势个股入口描述为已预留 | 项目状态说明失真 | 明确标记强势个股模块尚未开发 |
+| Minor | Review 文档 | 部分证据仍写成等待执行 | 治理记录与实际状态矛盾 | 同步完整门禁和 GitNexus 实际结果 |
+
+## 测试缺口
+
+完整 `pnpm verify` 已通过；Lint 为 0 error，35 条既有格式 warning 未增加。
+
+## 安全与残余风险
+
+生产逻辑保持不变；`finally` 保证测试结束后恢复 real timers。安全扫描未发现环境信息，无关 CSV 保持未跟踪并由逐路径 staging 排除。
+
+## 性能与资源影响
+
+仅进程内 fake timers，无运行时资源影响；定向 E2E 在 1 秒内完成，完整门禁通过且未增加外部调用。
+
+## GitNexus 与同步修改
+
+生产时钟函数 upstream 风险 CRITICAL、关联五个接口，因此明确不修改；测试文件 upstream 风险 LOW、0 执行流。开发后 detect changes 为 LOW、0 受影响执行流。
+
+## 修复与复审
+
+定向 GREEN 和完整门禁已完成。独立 Review 发现 `app.close()` 抛错时可能跳过时钟恢复，已将 `vi.useRealTimers()` 移入 `finally`；README 的强势个股入口夸大属于 FEAT-MO-003 范围，已同步纠正。复审未发现新的代码阻断项，最后一轮仅要求同步上述证据文字。
+
+## 最终结论
+
+最终复审确认 `try/finally` 清理、README 纠偏和证据同步均已关闭；无 Critical、Important 或 Minor 问题，`Ready to commit: Yes`。
+
+最终结论：passed
@@ -0,0 +1,33 @@
+# 开发规范符合性 Review
+
+- 功能编号：FEAT-QA-001
+- reviewer_type：independent
+- reviewer：Codex 独立审查代理
+- 审查提交：working-tree
+- 日期：2026-08-24
+
+## 检查结果
+
+- [x] 缺陷有稳定复现和 RED/GREEN 证据。
+- [x] 生产逻辑没有为测试让步。
+- [x] GitNexus 已执行 query、context 和 upstream impact。
+- [x] 完整测试、构建、E2E 和开发后分析已完成。
+- [x] 独立 Review 最终无 Critical、Important 或 Minor 问题。
+
+## 不符合项与处置
+
+完整门禁、开发后 GitNexus 和独立 Review 均已完成；`finally` 清理和文档纠偏通过复审。
+
+## 安全、性能与架构检查
+
+测试时钟限定在 suite 生命周期，不进入生产依赖注入或外部系统；无新增依赖和 I/O。
+
+## GitNexus 与开发总结检查
+
+功能档案已记录根因、生产符号 CRITICAL 风险和测试文件 LOW 风险；开发后变更检测为 LOW、0 执行流，提交与远程证据将在复审后补充。
+
+## 最终结论
+
+最终独立复审结论为 `Ready to commit: Yes`，无未关闭问题。
+
+最终结论：passed