审查 + 修复:v2 模块协议重构高置信 BLOCKER(main@34e658b)

[Tanya7z]

这是什么

对另一个 agent 推到 main 的 v2 模块协议重构(34e658b)的审查,并直接修复了其中高置信度、可端到端验证的 BLOCKER。含一份完整审查报告 docs/reviews/v2-module-protocol-review.md + 6 个源码修复提交(仅改 db-server/src 与 catalog.json,未碰模块业务逻辑)。

已修复(附运行时验证)

用「模拟真实模块 · 按 module-auth 派生 token」端到端验证:

编号	问题	修复文件
P-M1	v2 路由请求体恒空(读 req._body 从未赋值)	index.ts
B2	GET /configs/all 被 v2 鉴权遮蔽 401(SAPI 启动必用)	index.ts
回归	模块启停后 enabled 不翻转(读启动缓存 lock)	index.ts
B3	/db/tx 从 body 取 moduleId → 越权 + 事务恒拒	db-routes.ts
B4	整体 db:write:* 断言恒失败 → 所有事务被拒	tx-runner.ts
schema	finalize() 从未调用 → 表永不落地;softDelete 建表 SQL 语法错误	schema-registry.ts
B7	land-gui 未登记 catalog → 永不装载	catalog.json

验证结果

GET /api/sfmc/configs/all                          -> 200 (返回配置)
define-table lands (feature-land, 带 token)         -> 200 created
tx insert + query (feature-land)                   -> 200 (写入并查回)
get by id                                          -> 200
tx 伪造 body.moduleId=feature-afk(持 land token)   -> 以 feature-land 身份执行(忽略 body)
feature-afk 写 lands(无权限)                        -> 403 permission_denied
无 token                                           -> 401
tools/smoke-modules.js                             -> 全部通过
node tools/check-ootb.js                           -> 5/6(剩 sim-new-user 为既有 shim 缺失,与 v2 无关)
npm run build --workspaces                         -> 全部通过

仍未修复(需提交方更大改动,已在报告详述)

• B1 SDK 客户端从不带 token(host-bootstrap 未注入身份)→ 真实模块调用仍会 401。
• B5 tx() 录制-回放拿不到服务端生成的 id/结果 → 步骤间依赖失效。
• B6 service provider 无注册通路,land 的 12 个 provides 全是空操作。

这三项属于 SAPI 侧/协议级改动,须结合 Bedrock 运行时验证,不在本安全外科修复范围内。另有多个 MAJOR/MINOR(update/delete 按 rowid 定位、软删未过滤读、借贷方向反、land-gui 直查 land 私有表、afk afterWorldLoad 时机、clean-break 残留等)与规则偏离,清单见 docs/reviews/v2-module-protocol-review.md。

  

[cursor]

Bugbot is not enabled for your account, so this pull request was not reviewed.

Enable Bugbot in the Cursor dashboard to get automatic reviews on future PRs.