observation: runUpdateManyLoop 缺少 batchData update 分支那道 `!record.id` 守卫 —— 两个 by-id update 写面对畸形行的分类不一致(当前被路由 Zod 挡住,dormant)

[os-zhuang]

观察类记录(observation-class),当前用户打不到。读 #5088 时发现,如实登记,不建议单独排期。

现状

packages/metadata-protocol/src/protocol.ts 里两个 by-id 的 update 写面,对「行里没有 id」的处理不一致:

• runBatchDataLoop 的 update 分支:if (!record.id) throw rowRequiredIdError('update') → VALIDATION_FAILED / 400。这是 fix(metadata-protocol)!: batch 逐行结果迁移到 BatchOperationResultSchema 形状 —— 方案 B 已拍板,硬切 + 诚实迁移说明(Blocked-by #4620) #4793 特意加的,注释写着「a caller error, so it carries VALIDATION_FAILED / 400 rather than falling through toRowApiError's unclassified-throw default」。
• runUpdateManyLoop:没有这道守卫,直接把 record.id 递给引擎。

#5088 之后这条差异变得可见了一点:updateMany 里一个没有 id 的行,现在会先撞上存在性探针,拿到 RECORD_NOT_FOUND / 404,而同样一个行走 batch 的 update 分支拿到的是 VALIDATION_FAILED / 400。两者都不算"错",但对同一个 caller 错误给出两种分类,正是 #4620 对本文件立的规矩要避免的("must be ONE implementation, not three that agree today")。

为什么现在打不到

updateMany 的唯一入口(POST /api/v1/data/:object/updateMany)在 rest-server.ts 里跑 UpdateManyDataRequestSchema.safeParse,而 UpdateManyRecordSchema 的 id 是必填(#3939 特意不复用宽松的 BatchRecordSchema)。所以没有 id 的行到不了协议层 —— 只有直接调用协议方法的进程内 caller 才可能构造出来。故归为 dormant / 口径漂移,不是活缺陷。

建议(若将来收拢)

在 runUpdateManyLoop 的循环开头补同一句 if (!record.id) throw rowRequiredIdError('update'),让两个写面对畸形行给出同一个分类。改动很小,但会改变一个(目前不可达的)错误码,所以不适合搭在 #5088 的修复里顺手带 —— 那单的验收面明确是"每行自带 id"的路径。

发现于 #5088 的实现过程。

https://claude.ai/code/session_01BotUP49pqhvqGY393n2HfU

[os-zhuang]

维护者已裁（2026-08-04）：补齐守卫，升格为待办 —— 并认领

裁定：在 runUpdateManyLoop 循环头补 if (!record.id) throw rowRequiredIdError('update')，与 batchData update 分支（#4793）同一分类（VALIDATION_FAILED/400）。理由超出正文的"分类漂移"判读两点：

1. dormancy 依赖的是非本地不变量（"唯一入口跑了 UpdateManyRecordSchema"写在 spec/rest-server，protocol.ts 本地无痕迹），任何新入口都会无声地把 dormant 变 live；
2. 未设防路径把 { id: undefined } 递进探针与写两层，行为取决于 driver 对 undefined where 键的处理（未实测，如实标注）——守卫使畸形行的命运与 driver 语义解耦。

这是 protocol.ts 已知的最后一处写面口径漂移（#4435→#5088、#5099 同族）；关掉它即关掉整类。顺带对齐 record.data 裸递 vs record.data || {} 的同族微漂移。patch changeset 如实记录一个当前不可达错误分类的变化（404→400）。

认领

• 会话：session_01BotUP49pqhvqGY393n2HfU（data: updateMany runs hooks for a nonexistent id — the row fails INTERNAL_ERROR from a hook-condition abort instead of RECORD_NOT_FOUND #5088 / data: batchData 的 upsert 分支用「调用者可见性」而非「存在性」决定 update-or-insert —— RLS 作用域外的既有行会走插入分支 #5099 / PR fix(metadata-protocol): batchData upsert fork asks existence, not caller visibility (#5099) #5187 的同一会话）
• 分支：claude/updatemany-hooks-nonexistent-id-p6pxb8（本会话指定分支）
• 串行约束：与 PR fix(metadata-protocol): batchData upsert fork asks existence, not caller visibility (#5099) #5187 同文件（protocol.ts），在 fix(metadata-protocol): batchData upsert fork asks existence, not caller visibility (#5099) #5187 落地后基于新 main 重启分支实施；文件面 packages/metadata-protocol/src/protocol.ts + 同包测试 + changeset，越界即停。

---

Generated by Claude Code