53 lines
3.2 KiB
Markdown
53 lines
3.2 KiB
Markdown
# GOV-011 代码 Review
|
||
|
||
- reviewer_type：self
|
||
- reviewer：当前实现代理结构化自审
|
||
- 基线：`v0.0.8` / `9e85809`
|
||
- 审查实现：`cce77aa`
|
||
- 日期：2026-08-17
|
||
|
||
## 审查范围
|
||
|
||
审查 Node 特征测试、Schema 元数据校验、MySQL 健康客户端、治理测试、现行架构/规范/部署文档、GitNexus 影响和一次性 Schema 例外。重点检查版本格式、错误边界、历史清单兼容、健康信息泄漏和门禁真实性。
|
||
|
||
## 发现
|
||
|
||
### 已关闭问题
|
||
|
||
| 严重程度 | 位置 | 问题 | 处置 |
|
||
|---|---|---|---|
|
||
| Important | `tools/database/schema-baseline.test.mjs` 历史清单篡改测试 | 完整回归仍断言旧的“服务端必须为 5.7.42”文案，导致生产实现转为系列口径后测试失败。 | 更新为 `5.7.x` 断言；排除已授权发布基线用例后 Schema 28/28。 |
|
||
| Minor | `implementation-plan.md` 全量门禁预期 | 原计划把 `verify:release` 写为 PASS，但该命令包含必然触发一次性 Schema 例外的测试。 | 修正文档为真实失败并要求手工执行其余等价门禁。 |
|
||
|
||
### 未发现问题
|
||
|
||
未发现未关闭的 Critical、Important 或阻塞性 Minor。服务端正则只接受 `5.7.<数字>` 与 `-`/`+` 后缀；5.6、8.0、无 patch 和文字 patch 均拒绝。字符集、排序规则、库名、时区和生产数据库边界没有放宽。
|
||
|
||
## 测试缺口
|
||
|
||
没有针对每个 MySQL 5.7 patch 建立真实数据库兼容矩阵；这是用户明确接受的系列版本策略。本次使用纯函数、固定元数据和现有集成测试验证判断边界，不证明第三方数据库每个 patch 都没有厂商缺陷。首次真实迁移和查询仍必须在目标 5.7.x 环境验证。
|
||
|
||
## 安全与残余风险
|
||
|
||
- 版本字符串不执行、不进入 SQL；健康详情不回显原始后缀、地址、账号或密码。
|
||
- Schema 入口仍固定 development、开发库和主机白名单；本次未连接生产数据库。
|
||
- 残余风险一是不同 5.7 patch 的已知缺陷，二是 v0.0.8 缺少可验证 Schema 基线。前者由真实部署验证控制，后者由用户授权的一次性例外承担。
|
||
|
||
## 性能与资源影响
|
||
|
||
- 只增加常数时间正则捕获，没有新增查询、网络、重试、缓存、任务或依赖。
|
||
- 健康查询次数和连接池上限不变；Web 构建体积与 v0.0.8 一致。
|
||
- 不进入行情采集、分析、Redis、RocketMQ 或图表热路径，无需负载测试。
|
||
|
||
## GitNexus 与同步修改
|
||
|
||
开发前 `assertMysql57Metadata` 为 CRITICAL，5 个直接调用方、10 个受影响符号、5 组流程；`MysqlHealthClient` 为 LOW。开发后相对 v0.0.8 为 HIGH、23 文件、70 符号、11 流程，运行影响均属于计划中的 Schema 与 API 健康路径。HTTP/OpenAPI、Web API、消息、数据库 Schema、Redis、Scheduler、Worker 和部署拓扑没有计划外修改。
|
||
|
||
## v0.0.9 发布封存复核
|
||
|
||
- 发布只增加版本、CHANGELOG、索引和功能状态，不修改运行时判断、健康查询或数据库边界。
|
||
- 一次性 Schema 例外、HIGH GitNexus 风险和手工等价门禁证据均保留，没有改写为全量门禁通过。
|
||
- 发布标签必须指向通过上述 Review 的封存提交，并与 Gitea 远程引用核对。
|
||
|
||
最终结论：passed
|