[finding] isMissingSourceError 仍把 Postgres 的「缺列」措辞 column "c" of relation "t" does not exist 判为「缺源」——文档明说不该,今天靠读路径不产生该措辞而无害 #6035

Activity

1. hotlong

   added

   domain:services

   finding

   on Aug 6, 2026
2. claude commented on Aug 6, 2026

   

   claudebot

   on Aug 6, 2026 – with Claude

   Contributor

   More actions

   发现分诊轮(#4949 纪律):晋级 pm:queue,摘 finding。域维持 domain:services。

   • 过时前提检查(origin/main 9e3709a)—— 逐行实读,前提成立,并对正文的一处措辞做事实修正****:
     • packages/services/service-analytics/src/analytics-service.ts:89–91 的 docblock 逐字仍在:Deliberately scoped to MISSING SOURCE (table/object/relation) — not column/syntax errors, which stay hard failures so real query bugs still surface;
     • :96 的 isMissingSourceError 判据仍是宽串对 msg.includes('relation') && msg.includes('does not exist') —— 即 [finding] queryDataset 里还有第二个 message 嗅探器 isMissingSourceError,命中即静默返回空结果 —— dataset-compiler 的一条拒收措辞已经命中它,只因抛点在 try 之外才没出事 #5717 的收紧并没有落在这个函数上,它落在 :125 的 missingSourceRelation 正则表(/relation\s+["']?([A-Za-z0-9_$.]+)["']?\s+does not exist/i)。正文写作「[finding] queryDataset 里还有第二个 message 嗅探器 isMissingSourceError,命中即静默返回空结果 —— dataset-compiler 的一条拒收措辞已经命中它,只因抛点在 try 之外才没出事 #5717 收紧后同样命中」在结论上正确(两条判据对 column "label" of relation "acct" does not exist 都 HIT),但收紧发生的位置要按这里读:宽判据本身仍在服役,不是「收紧后残留」而是「收紧未覆盖」。这个区别会改变修法落点(要动的是 :96,不只是正则)。
   • 为什么晋级:① 判据与自己的文档不一致 —— 这是本仓一贯认真对待的 declared ≠ enforced 形状,而且方向是吞掉文档明说要响亮失败的那一类(列名写错);② 修法有仓内先例可照抄:packages/rest/src/rest-server.ts:825 已经用「先摘缺列、再判缺表」的两步法处理同一族措辞(analytics 的 filter 拒收到不了调用方:service 侧多数拒收没有 ADR-0112 信封,REST 面又用 message 正则嗅探,一律答 500 #5352 面),对应用例在 packages/rest/src/rest.test.ts:2417;③ 成本封顶,零裁决。
   • 今天无害的判断本轮核过并同意(故优先级低):analytics 是只读面,Postgres 在 SELECT 下的未知列措辞是 column "x" does not exist(不含 relation,不命中);column … of relation … does not exist 是 INSERT/UPDATE/ALTER 的措辞,dataset 查询路径不产生它。⇒ 入队但不是 p0,排在真实用户可达的单之后。
   • 给执行座位的一条前置判断(不是阻塞项):正文列的第二条方向(顺 [finding] queryDataset 里还有第二个 message 嗅探器 isMissingSourceError,命中即静默返回空结果 —— dataset-compiler 的一条拒收措辞已经命中它,只因抛点在 try 之外才没出事 #5717 方向 A,驱动侧结构化 error code)会让本单变得不必要。派发前请先确认那条方向尚未立单/未在飞,若已起则本单并入、⛔ 不要两处各修一半。附带说明:方向 A 落 driver-sql / Postgres 面,不在 driver-memory / driver-mongodb 的投入冻结面内([裁决] driver-memory / driver-mongodb 投入冻结 —— 维护者 2026-08-05 口径（跨单锚点） #5499),故不因冻结受阻。
   • 查重:三仓无影子单;[finding] queryDataset 里还有第二个 message 嗅探器 isMissingSourceError,命中即静默返回空结果 —— dataset-compiler 的一条拒收措辞已经命中它,只因抛点在 try 之外才没出事 #5717(来源)、analytics 的 filter 拒收到不了调用方:service 侧多数拒收没有 ADR-0112 信封,REST 面又用 message 正则嗅探,一律答 500 #5352(先例出处)为关联。本轮 31 条 open PR 无在飞 PR 触 analytics-service.ts;同车道本轮另有 finding(service-automation): engine.ts 的 resumeInternal 把驱动错误插进 message —— #5737 修完后,这条 resume 路径上仅剩的一处外来 cause 拼接 #5912 晋级、[finding] filter-normalizer 的 $not allowNull 守卫在 #5298 第二批之后变成冗余的第二层(谓词等价,SQL 多一层括号) #6005 持有,三者文件互不相交。

   本评论来自分诊座位 Routine(#5474 试点),不构成认领。

   ---

   Generated by Claude Code
3. claude

   added

   pm:queue

   and removed

   finding

   on Aug 6, 2026
4. hotlong

   self-assigned this

   on Aug 7, 2026
5. hotlong commented on Aug 7, 2026

   

   hotlong

   on Aug 7, 2026 – with Claude

   ContributorAuthor

   More actions

   认领:PM 循环第 8 轮(补位;同文件串行前提 #5918→PR #6292 已 MERGED,解除)
   会话:session_015a5qkLzpGXhLL2F5gvJ7dD
   分支:claude/issue-6035-column-phrase-missing-source
   Worktree:objectstack-6035
   域:domain:services
   文件面:packages/services/service-analytics/src/analytics-service.ts(isMissingSourceError :96 的宽判据本体——分诊事实修正:#5717 的收紧落在 :125 的 missingSourceRelation 正则,宽判据仍在服役,修法要动 :96)+ 测试

   修法照分诊钉死:抄 packages/rest/src/rest-server.ts:825 的两步法先例(先摘 column … of relation … does not exist 缺列措辞、再判缺表),让缺列保持硬失败(与函数自身文档一致);派前条件已核:#5717 方向 A(驱动侧结构化 code)无开放单/在飞,无并入对象。⚠️ 该文件今日已被 #6292(dotted measure)触及,基于当前 origin/main。

   ---

   Generated by Claude Code
6. hotlong commented on Aug 7, 2026

   

   hotlong

   on Aug 7, 2026 – with Claude

   ContributorAuthor

   More actions

   ACCEPT → PR #6346(draft 转 ready + auto-merge)。

   先更正本座位派发令里的一处过时前提(dev 用测量推翻,公开留档):我在认领贴里照抄了分诊评论的「事实修正——:96 宽判据仍在服役,修法要动 :96」。该判读基于 origin/main 9e3709a,而 #5717 的实现 PR #6045 在其后合并,已把 isMissingSourceError 的 postgres 支收紧为锚定正则 ⇒ 该前提今天不成立。真正的落点是判定顺序:column "c" of relation "t" does not exist 字面上就是合法缺表措辞的超串,任何形式的「收紧」都排除不掉它,只有先问更具体的问题(先摘缺列)才行。派发令按旧前提写,dev 未照单执行而是先测再改——正确做法。

   • 修法:新增 MISSING_COLUMN_OF_RELATION,在 isMissingSourceError 与 missingSourceRelation 两处判定之前摘除;正则逐字抄 rest-server.ts 的 mapDataError 先例(analytics 的 filter 拒收到不了调用方:service 侧多数拒收没有 ADR-0112 信封,REST 面又用 message 正则嗅探,一律答 500 #5352 起),不另造方言。
   • 两处都改的实测依据(issue 要求实测决定):改前 missingSourceRelation 对该措辞答 sys_team——只修其一会让两个判定点互相矛盾,正是 [finding] queryDataset 里还有第二个 message 嗅探器 isMissingSourceError,命中即静默返回空结果 —— dataset-compiler 的一条拒收措辞已经命中它,只因抛点在 try 之外才没出事 #5717 刚消除的那种分歧。
   • 验收主证:逆向验证方向写死在测试文件头再跑,预测「只有该措辞的用例变红」→ 实测 3 红恰为预测三例,其余 12 行语料 + [finding] queryDataset 里还有第二个 message 嗅探器 isMissingSourceError,命中即静默返回空结果 —— dataset-compiler 的一条拒收措辞已经命中它,只因抛点在 try 之外才没出事 #5717 文件 11/11 全绿;两条红的输出即缺陷本身(对拼写错误讲拓扑故事 / 什么都不抛让控件拿到空网格)。13 条语料仅 1 条改判。包内 1238 测全绿、rest 消费半径 105 测绿、CI 24 检 22 success + 2 skipped。
   • 诚实声明采信:该 guard 今天在只读路径不可达,dev 明确不为它写会空过的测试,并写进代码与 PR 正文——比造一条假装有覆盖的用例强。
   • check:type-check-debt 首跑红经复核为 fresh-worktree 未建依赖假阳性(补跑同一条构建后 PASS,CI 上该 job success 验证诊断)——记录在案,是仓内已知陷阱的又一次命中。
   • 衍生:[finding] isMissingTableError 同样把 Postgres 写路径的「缺列」措辞判为「缺表」——其 docblock 明说 42703 必须响亮失败,而消费点会据此「从 1 开始编号」 #6347(packages/metadata 的 isMissingTableError 同族子串洞,matchesDriverError 的 code/errno/message 并列或使 42703 也挡不住;同因只读面不产生该措辞而休眠)observation 类立案交分诊。

   ---

   Generated by Claude Code
7. hotlong

   closed this as completedin #6346on Aug 7, 2026