Files
a-share-analysis/docs/features/GOV-012-node-schema-exporter/review/code-review.md

1.6 KiB

代码 Review

范围

Review 覆盖 mysql2-schema-exporter.mjs、schema-baseline.mjs、对应测试、format v3 清单和文档变更。Task 2/3 Review 发现的问题已通过修复提交关闭:触发器元数据严格校验、目录/SHOW CREATE/元数据错误脱敏、超时错误语义保留、例程字面量不误改、主机信息拒绝,以及 MySQL 5.7 视图目录回退。

安全与残余风险

  • 只读 information_schema 和 SHOW CREATE,禁用多语句,不读取业务数据,不执行 DDL/DML,不锁表。
  • 配置边界固定为 development、a_share_analysis_dev 和开发主机白名单;密码不进入日志、异常、SQL、清单或 checksum。
  • 连接、查询、总时长、对象数和定义大小均有上限,所有路径最终关闭连接。
  • 既有 Web 测试依赖链 glob 的 high 漏洞是用户批准的范围限定治理欠项,不在本功能新增路径;若 mysql2 或导出链出现 high/critical,必须阻断发布。

性能与资源影响

  • 单连接、按目录顺序读取,查询 5 秒超时、总时长 30 秒、对象 5,000、单定义 1 MiB;视图排序为确定性 Kahn 算法。
  • 治理命令不在在线请求路径,不新增 API、Redis、RocketMQ 或 Worker 负载;空库实际生成 captured_empty。

GitNexus 与同步修改

  • MCP Transport closed,已用 CLI reindex/status、静态引用和回归测试交叉核对;影响集中在 Schema 治理链路。
  • HTTP/OpenAPI、Web API、消息、Redis、API、Scheduler、Worker、部署均无需同步修改;Schema 文档、测试脚本和 v0.0.9 基线已同步。

最终结论:passed