Repository navigation
[metadata-protocol] seed-loader:无 pass-2 时「引用解析不了 ⇒ 整条记录丢弃」计为 error 却一行日志都不打(#4729 判据的更深一格) #4997
Description
发现于 #4729(把 seed-loader 里「计数为错、日志为 warn」的行对齐到 error)的同文件盘点。本条不在 #4729 的判据内(它盘的是 this.logger.warn 调用),所以只记录、不在那个 PR 里改。
现象
packages/metadata-protocol/src/seed-loader.ts,pass-1 引用解析、multiPass 关闭的分支:
} else {
// Cannot resolve and no pass 2 will run — skip the whole record
// (LOUD: counted + reported). Writing it anyway would either
// carry the raw natural-key string into the FK column or, on
// update, corrupt the existing row.
pushError(
`Cannot resolve reference: ${objectName}.${ref.field} = '${String(unresolvedItem)}' → ` +
`${ref.targetObject}.${ref.targetField} not found`,
unresolvedItem,
);
unresolvedRefError = true;
}pushError 把错误推进 errors / allErrors(⇒ success: false),随后 unresolvedRefError 让 errored++ 并 continue —— 整条记录被丢弃,一行都没有种下去。但这里没有任何 logger 调用:注释写的 "LOUD" 只兑现了「计入结果对象」,控制台上一个字都没有。
为什么值得单开
#4729 的论点是「计数与日志级别应当一致」:结果对象里数得到、日志却停在没人读的 warn。这一处是同一条链的更深一格 —— 计数有、日志级别是「无」。对只看 boot 日志的运维/agent 来说,一次丢了 N 条记录的 seed 与一次干净的 seed 在控制台上完全一样(区别只在调用方是否检查了 result.success,而 packages/runtime 的若干 seed 调用点只是 await 之后继续)。
check:durability-log-level 把这种形态叫 silent-swallow,但那个门禁只扫 try/catch,而这里不是 catch,所以扫不到。
期望(待维护者确认)
- 在该分支补一条
this.logger.error(...),按 [convention] best-effort 降级导致"看起来正常、实则不持久"时不应记 warn——把 #4460 的点状修复定成规则 #4632 的要求说清后果(<object>第 #i 条记录整条未种下,因为<object>.<field>指向的<target>不存在)与修复动作(先种目标对象、或打开multiPass让 pass 2 回填、或修掉 seed 里的自然键); - 顺带确认 dry-run 分支(
[dry-run] Reference may not resolve)是否也该有一行 —— 那一处同样只进结果对象、不进日志,但 dry-run 的调用方本来就是来读结果的,可能维持现状更合适。
未认领。
Activity
claude commentedon Aug 4, 2026
🔒 认领:PM 循环第 8 轮(恢复派发,bug-only 口径)
会话:
session_01NrmBxj8rK2uGCnh9aipjwX(账号xuyushun441-sys)分支:
claude/issue-4997-seed-drop-silent-logWorktree:
objectstack-issue-4997域:
domain:engine文件面:
packages/metadata-protocol/src/seed-loader.ts及同目录测试(⛔ 与 #5088 的 protocol.ts 不交)⛔ 硬约束:
spec/**及生成物零改动;metadata-protocol/src/protocol.ts零改动(借道 #5088 在飞占用);releases/ 零触碰。全局在飞不相交已核:#5088(protocol.ts,借道)、#5087(plugin-email)、#5089/#5090(#5040 执行器)、#5096/#5026(lint)、#5011,与本文件面均不交。
Generated by Claude Code
claude commentedon Aug 4, 2026
✅ 验收通过 —— PR #5128(draft,CI 跑完入队)
对照实际 diff 复核(3 文件,+454/-11;⛔ spec / protocol.ts / releases 零触碰 —— 与借道在飞的 #5088 后续零交集,逐一核实):
CI 绿后转 ready 入合并队列。
Generated by Claude Code
fix(metadata-protocol): seed-loader 中「计为错误」的失败一律记 error,不再自相矛盾地记 warn (…
fix(metadata-protocol): log the seed record dropped for an unresolvab…
fix(metadata-protocol): report the pass-2 back-fill dropped when the …