- 28 Jul, 2026 8 commits
-
-
现象:测试服 deploy 挂在 migrate 步,pac-service 起不来,任何人往测试服部署 都会挂在同一处(跟部署内容无关)。 ERROR: CREATE INDEX CONCURRENTLY cannot run inside a transaction block (25001) 根因是我在 20260728020000 里写了 14 条 CREATE INDEX CONCURRENTLY。 Prisma 把整份 migration.sql 用**一次 simple query** 发给 Postgres,而 Postgres 只对 「一个查询串里多条语句」隐式开事务块 —— 分界线是**语句条数,不是 Prisma 版本**: 20260701060607 / 20260727070000 / 20260727110000 各 1 条语句 → 无隐式事务 → 跑得通 20260728020000 14 条语句 → 隐式事务 → 必挂 而 20260727070000 里那句「Prisma 6.19 已不成立(指事务包裹)」是**错的归因** —— 单条语句侥幸跑通被总结成了「新版没有事务包裹」,我照抄进 20260728020000,于是踩爆。 失败后果比失败本身更重:_prisma_migrations 留下一条 finished_at / rolled_back_at 全空、 applied_steps_count=0 的记录,Prisma 从此拒绝执行**任何**后续迁移(P3018), 整条部署流水线被堵死。同一颗雷已随 6333137a 合进 main,生产下次部署会撞一模一样的错。 改动: - 20260728020000:14 条改为**普通** CREATE INDEX IF NOT EXISTS(可在事务里跑)。 大表环境靠「部署前手工 CONCURRENTLY 先建」保证不锁表(两台机器 2026-07-28 已建完, 故本迁移在两台上都是 IF NOT EXISTS 空跑);从零建库时表为空,普通 CREATE INDEX 瞬间完成。 文件头补上完整复盘,明写"别再改回 CONCURRENTLY"。 - 20260727070000 / 20260727110000:订正那两处错误归因,改成"因为本文件只有 1 条语句"。 验证: - 把修好的整份文件用 `psql -1 -v ON_ERROR_STOP=1`(显式单事务,正是 Prisma 那种场景) 在测试服跑通,14 条全 skip,无 25001。 - 测试服已 `prisma migrate resolve --rolled-back 20260728020000_...` 解除 P3018 阻塞, 14 条索引仍在(pg_indexes 计数 14),库结构未被破坏。 Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
luoqi committed -
luoqi committed
-
背景:生产此前**完全统计不了 PV/UV** —— 前端无任何分析埋点(gtag/umami/plausible 全无),服务端无 request log,生产 ECS 上更没有 nginx(pac.friday.tech 走独立网关 lb1.friday.tech,访问日志不在我们这台机器上)。Sentry 只有 10% tracing 采样, 做 UV 会系统性偏低。 范围限定在 plan 详情页(切合 plan_event_logs 表名):plan_id / patient_id 本就是 该表的必填列,浏览事件天然填得上,不必为它放宽约束或另立新表。 三种入口按用户口径**全收**:直接进/刷新 · 患者列表 rail 切换 · /plans 落地自动跳转。 (自动跳转确实是系统行为,但业务侧确认要收 —— 它也代表这条 plan 被展示过。)
⭐ 唯一的雷:byHuman 不是"是不是人做的",是**口径开关** HUMAN_TOUCH_EVENTS 靠 filter(byHuman) 派生。view 按字面语义该写 true,但那会让 「客服处理过哪些患者」静默膨胀成"看一眼也算处理"——刚交付的 65 人统计与整张 客服明细 Excel 全部失真,**而且不会有任何编译错误或类型报错**。 故 view 写 byHuman:false,并把该字段注释从"是否人工动作"改写成明确的口径定义; spec 里用**精确相等**(非 arrayContaining)锁死 HUMAN_TOUCH_EVENTS 的四个成员, 以后新增事件必须显式决策要不要计入,改错即红。 改动: - enums: PlanEventType.VIEW + PLAN_EVENT_META 项(group 联合类型扩 'view') - controller: POST /pac/v1/plans/:id/view。**刻意不加认领闸**(区别于 recall-feedback)—— 浏览发生在认领之前,池子里任何一条都能点开;加闸则埋点只剩已认领的,PV/UV 直接废掉。 plan 不存在静默返回 not_found:埋点是旁路,不能因它失败而影响页面。 - 前端: 埋点挂在 use-plan-aggregate 现成的去重守卫里 —— 该守卫原为防 StrictMode 双调, 其语义恰好等于"每进入一次详情页";refresh() 直接调 load() 走不到这里,故手动刷新不重复计。 失败 .catch(()=>{}) 吞掉。 验证(本地起真实前后端 + 浏览器实操): /plans 自动跳转 → 记 1 条 ✓ 手动点刷新 → 仍 1 条(不重复计)✓ 重新进入页面 → 累计 2 条 ✓ 未认领查看 → 能记 ✓ plan 不存在 → not_found 不报错 ✓ 无 token → 401 ✓⭐ 只有 view 事件的患者,在「处理过」口径下计 0(全事件口径会误计 2)✓ 640 tests passed,service/web typecheck 通过。 无 DB 迁移(event 是既有 text 列,加的是取值不是列)。 Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>luoqi committed -
luoqi committed
-
luoqi committed
-
生产现象:召回池按「禁忌:手术禁忌」圈人,前端等 20+ 秒 GET /pac/v1/plans?view=pool&personaTags=contraindication:surgery 服务端复现:count 23,379ms + findMany 22,208ms(两句并发,墙钟 ≈23s)。 **不是禁忌特有** —— 测试服同口径按 rfm:重要价值 更慢,57s。14 个圈人维度全踩同一个坑, 禁忌只是这次先点到的那个。 根因:Prisma 把画像筛选编译成两层 EXISTS,最内层 persona_features.key = 'contraindication' AND (data #> '{domains}') @> '"surgery"' 而 persona_features 只有单列 key 索引,JSON 那半截没有索引可下推。 生产 EXPLAIN (ANALYZE, BUFFERS) 实测(7,657,944 行 / 5.4GB,堆 3.4GB): Index Scan using persona_features_key_idx actual time=3567..3954ms Index Cond: (key = 'contraindication') Filter: ((data #> '{domains}') @> '"surgery"') rows=2,156 Rows Removed by Filter: 188,054 ← 命中率 1.1% Buffers: shared read=162,830 ← 约 1.3GB 随机堆读 即为了捞 2,156 行,把该维度 190,210 行的 jsonb 全从磁盘搬了一遍。维度越大越惨: rfm / gender / lifecycle_stage 各 869,043 行,一次筛人搬 869k 行 data。 改动:每个可筛维度一条偏索引(WHERE key='<维度>'),表达式与 Prisma 生成的完全一致。 - 数组维度 7 条(array_contains → @>)→ GIN jsonb_path_ops - 标量维度 7 条(equals → =) → btree (表达式, persona_id) 第二列不是为了排序,是为了 index-only scan:内层 EXISTS 只 SELECT persona_id, 索引自带就不回堆。低选择性维度非它不可 —— 测试服实测 rfm 走上 btree 后 Bitmap Index Scan 只花 217ms,回堆 143,187 个块却花了 103s(work_mem 不够还退化成 lossy,Rows Removed by Index Recheck 1,564,553)。 Prisma schema 表达不了表达式/偏索引,只能落迁移 SQL;本地 migrate dev 会把它们报成 drift, 别按提示 reset(同类先例 20260727070000 / 20260727110000)。 persona-tag-filters.ts 加纪律注释:往 PERSONA_TAG_FILTER_DIMS 加维度必须同步补索引, 漏建就是 20 秒起步 —— 这次禁忌踩的正是这个。 上线(两台均已 CONCURRENTLY 手工建完,不锁表,业务无感;迁移全 IF NOT EXISTS 故 deploy 空跑): 生产 14 条 / 88s / 220MB,pg_index 无 invalid contraindication:surgery 23,379ms → 84ms(278×,1,347 单) contraindication:implant 842ms(25,900 单) rfm:important_value 1,558ms(61,300 单) treatment_history:implant_history 854ms(17,179 单) gender:female 2-4s (105,110 单) 测试服 14 条 / 238s / 195MB(盘慢,看倍数) rfm:important_value 57,299ms → 8,576ms contraindication:surgery 4,210ms → 344ms 剩下的耗时性质已经变了:不再是 JSON 过滤,而是结果集本身大(count(*) 要数完十万级 plan)。 gender 这类宽维度哪天嫌慢,方向是给 count 做近似/缓存,不是继续加索引。 Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>luoqi committed -
luoqi committed
-
线上(测试机)现象:FRIDAY 推 med_emr_info 每小时整点重试,连挂 25 次, 对面收到 http 500 body={"code":90000,"msg":"request entity too large"}。 根因两层: 1. main.ts 从未显式设 body limit → 吃 body-parser 出厂默认 100KB。 实测卡点 102,381B 过 / 103,405B 挂,正好 100KiB。 网关 client_max_body_size 是 50M,不是它拦的(对面收到的是 PAC 的 JSON 封装)。 2. body-parser 的 PayloadTooLargeError 是裸 Error(非 HttpException), 掉进过滤器的 500/90000 分支 → 对面按文档 §9.1 把 9xxxx 当"PAC 内部错误 → 退避重试",于是整点重试永远重试不好,也看不出是自己包太大。 为什么病历先踩到:测试机实测 friday 已入库 payload 单行大小 —— emr 均 1453B / 峰值 2218B,diagnosis 均 1647B(带主诉/检查/处置/医嘱自由文本) appointment、encounter 均 405B 接入文档 §10 只约束"≤500 条/请求",没约束字节:500 条病历 ≈ 710KB,超默认值 7 倍; 连 405B 的预约行 500 条(≈198KB)也超 —— 雷对所有表都埋着,病历只是先踩到。 改动: - main.ts: useBodyParser 显式设 10MB(json + urlencoded)。取 10MB 是因为它给 500 条病历留了十几倍余量,又远低于网关 50M —— 不把拦截点推到网关(那层返 HTML 413, 对面更难排查)。rawBody 仍由 create 选项保留,不影响 HMAC 验签。 - all-exceptions.filter.ts: 识别 PayloadTooLargeError(type=entity.too.large, 辅以 413 兜底)→ 归 10002 + HTTP 200,回执带上限与建议批量,且不上报 Sentry (对面发包过大不是 PAC 故障)。口径对齐文档 §9.1「字段校验失败/批量超限 → 修正后重发」。 - channel-push.mdx §10: 补字节上限,并给按表的分批建议(病历 50-100 行 / 结构化 500 行)。 验证(本地起真实服务,真 HMAC 签名): 146B / 300KB / 2MB / 8MB → 全部通过验签进入摄入流水线(证明 rawBody 未被破坏) 11MB → 10002 + HTTP 200,回执可指导动作 636 tests passed,service/web typecheck 通过。 Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>luoqi committed
-
- 27 Jul, 2026 21 commits
-
-
默认返回提示 JSON(不抛),避免公网裸 500;验收 Sentry 后端捕获用,关 flag 即停。 Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
luoqi committed -
merge: fix/k00-focus-category-and-ortho-mapping → main(K00 诊断词细分 + 低龄种植降位 + 二期正畸误吞 + reparse工具 + 历史联系摘要不判到诊) # Conflicts: # apps/pac-service/prisma/schema.prisma
luoqi committed -
luoqi committed
-
luoqi committed
-
四个提交: 469ef2c9 fix(ai): 历史联系摘要只讲回访事实,不再对到诊/失联下结论 6e7d1f60 feat(reparse): 加 --patients-file(按受影响患者收窄,实测最高 250 倍) 1c537ccb perf(reparse): patient_transactions 补 (host_id, subject_type, patient_id) 复合索引 cdad7db0 fix(plan): K00 按诊断词细分类目 + 低龄种植降位(≤18)+「二期」正畸误吞 + 早矫措辞 两条索引迁移(20260727070000 sync_log_id / 20260727110000 host+subject+patient)用 CONCURRENTLY,部署前先手动建(见各迁移注释),避免 pac-service 等 pac-migrate 白等停机。 # Conflicts: # apps/pac-service/prisma/schema.prisma
luoqi committed -
分支主题(K00 focus category + 正畸映射)的一批年龄/诊断适配。核心是把召回「目标·X」 标签和话术的类目排序,从"只看 K 码 + 年龄"细到"看诊断原文词",纠三类线上误标。 ── ① K00 大口袋按诊断词定方向(refineCategoriesForDiagnosis)── K00「牙发育和萌出障碍」混着语义完全不同的病:乳牙滞留(要拔)/ 乳牙早失(管间隙)/ 先天缺失(修复)/ 釉质发育不全(冠贴面)/ 萌出障碍(导萌),却共用固定序 [surgical,...] → focusCategory 恒为 surgical,给「乳牙早失」也打「目标·外科」(没牙可拔)。线上约 2800 条。 诊断码不动(还是 K00),只在同码内把最贴切类目提到首位。scenario 投影 name_zh、 signals 存 dxNameZh 快照,前端 ReasonLine 用同一函数复算,保证「目标·X」与本行首项一致。 ── ② 低龄种植降位,边界与全仓另三处种植年龄闸统一为 ≤18 ── recommendedCategoriesForAge 新增:age ≤ IMPLANT_LAST_AGE(18)时 implant 挪末位(只挪不删)。
⭐ 边界取 ≤18 而非 <18 —— 差一岁会自相矛盾:contraindication.feature.ts 是 age<=18 打 「种植禁忌」、potential-treatment 是 age>18 才产「潜在种植」。2026-07-27 生产实测: 恰好 18 岁有 1,299 条 plan、140 条 focusCategory=implant,若按 <18 判会在同一屏出现 「目标·种植」+ 画像「种植禁忌」。话术侧(fact-block / stable prompt)同步 ≤18。 ── ③ planned 规则裸「二期」不再误吞正畸 ── 「正畸二期矫治」被 implant 规则的裸「二期」吞成种植(线上 陈芃霖 BJ0A094257)。 actual 侧的修法是收紧成「种植二期」,但 planned 侧种植二期普遍不带"种植" (二期取模/二期手术/二期修复,prod 900+ 条),收紧会让它们掉进 _default 被丢。 改用位次:裸「二期」从 implant 主规则拆出、降到全表最末兜底 —— 带正畸词的先被 orthodontic 收走,其余纯「二期」仍落 implant,与修复前逐条一致。 ── ④ 替牙期正畸措辞说「早期矫治」── treatmentCategoryNameZhFor:age ≤ EARLY_ORTHO_MAX_AGE(12)时 orthodontic 显示「早期矫治」 (替牙期做的是一期干预,不是排齐恒牙列)。只改措辞不改类目,排除闸不受影响。 reason-signals schema 补 dxNameZh(nullable,旧 plan 无 → 回落原顺序,无回归)。 删 tests/tmp-jieya.spec.ts(纯 console.log 探针,无断言,不进仓库)。 测试 546 passed,两个 app tsc 干净。 Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>luoqi committed -
reparse 分批取源行那句: SELECT id, raw_payload, payload_hash FROM patient_transactions WHERE host_id = $1 AND subject_type IN ('emr','diagnosis') AND patient_id IN (3000 个) 没这条索引时 planner 只能走 (patient_id, occurred_at),把这 3000 个患者的**全部** transaction 连 raw_payload 一起捞进堆,再在堆上过滤 host_id / subject_type。 2026-07-27 测试服 EXPLAIN ANALYZE 实测(12,519,060 行 / 33GB): Bitmap Heap Scan 10,385 ms rows=14,578,Rows Removed by Filter: 39,305 ← 读进来 73% 是白读 Buffers: read=15,616(约 122MB 磁盘读) 全量 87 批,单这一句约占 15 分钟。本地建索引后 planner 已改走 Index Scan using patient_transactions_host_id_subject_type_patient_id_idx。⚠ ️ 收益如实说:每批总耗时 76–180s,这句只占 6–14%,对全程约 4%,**不是提速主手段**。 reparse 真正的杠杆是上一个提交的 --patients-file(按受影响患者收窄,实测最高 250 倍)。 本索引胜在一次性代价、之后每次 reparse / 每个 host 都受益。 体积:(uuid,text,uuid) × 千万行,测试服估 ~600MB、生产(2407 万行 / 64GB)估 ~1.2GB。 测试服建前 24G 空闲够用;生产上线前需先确认磁盘余量。 上线方式同 20260727070000:CONCURRENTLY(不锁写),推荐部署前手动先建 —— deploy 时 pac-service 要等 pac-migrate 退出才启动,几分钟索引构建 = 白等的停机; 手动建好后本迁移 IF NOT EXISTS 空跑。 Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>luoqi committed -
全量 reparse 的时间几乎全花在"证明没变"上。2026-07-27 测试服实测: 全量范围 260,434 患者 / 87 批 / ~4.5 小时 改 clinical-signals 字典 → 真正受影响 ~40,000 患者(15%) → 6.5 倍 改 planned「二期」规则 → 真正受影响 1,017 患者(0.4%) → 250 倍(2 分钟) 已有 --patient=<uuid>,... 但几千个 uuid 塞不进命令行(ARG_MAX),缺的就是文件入口。 用法:先用 SQL 圈出受影响患者,dump 成一行一个 id,再喂进来。 psql -tAc "SELECT DISTINCT patient_id FROM patient_facts WHERE type='treatment_record' AND kind='planned' AND content->>'subtype' LIKE '%二期%'" > /tmp/ids.txt pnpm reparse -- --host=jvs-dw --subject-type=treatment --patients-file=/tmp/ids.txt 文件格式刻意宽容(操作者多半是 psql 直接倒出来的):uuid 与 externalId 混写、 # 整行/行尾注释、空行、行尾逗号分号、CRLF 全收;但不做猜测性纠错 —— 认不出的 原样送去库里查,查不到就 WARN 报出前 5 个,不静默丢(否则"为什么少跑了几百人"是无头案)。⭐ 空结果必须硬失败,这是本次最要紧的一条: 服务侧 `if (!scopePatientIds?.length)` 把空数组当"不限定" —— 一个路径写错的收窄命令会静默变成 4.5 小时全量,且日志上完全看不出异常。 现在解析为 0 / 一个都对不上库,都直接报错退出。 ── 顺带修一个我自己踩出来的隐患 ── CLI 末尾是 `void main()`(全仓 CLI 通例),我最初把纯函数留在 CLI 里、测试直接 import, 结果测试跑起来对本地库真跑了一次 reparse(115 行 Nest 日志)。两道修: ① 纯函数拆到 src/cli/reparse-patients-file.ts,测试只碰这里 ② CLI 末尾加 `if (require.main === module)` 闸 —— 以后任何 import 都不会再误启动 实测(本地 friday): 空文件 → reparse 失败:解析后为 0 个患者(拒绝退化成全量) 混写清单 → 读到 4 token(uuid 1 / externalId 3 → 认领 2),WARN 1 个查不到, 最终限定 3 患者,ColdImportService 确认「范围 3 患者」 测试 547 passed(新增 13 例)。 Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>luoqi committed -
生产实例(陈芃霖 BJ0A094257): 摘要「上次**有效到诊**为2023年8月,已超一年未复诊」 实际 接诊记录末次 2025-12-20,画像也写着「末诊207天」—— 差 2 年 4 个月 同句另两处错:「已超一年未复诊」实为 7 个月;「近3个月涂氟邀约」那条回访实为 226 天前。 源头是这条回访记录: 2023-08-15 | 常规回访 | 已回访 | 已完成 | 内容:复查 | 结果:8月已就诊 诊所客服 2023 年写的一句备注,被模型升格成了患者当前的到诊状态。「有效到诊」这个词 源数据里没有,是模型自己造的。 ── 根因 ── 本 AiCall 的输入**只有回访记录,没有任何到诊数据**(见 input.types)。模型只能: ① 从回访备注文字里猜到诊 —— 「8月已就诊」被当成事实 ② 用"没有回访记录"推"没来过" —— 但回访是诊所主动打电话,患者自己来院不产生回访任务 两条路都错。而旧 prompt 不但没拦,关键要求 4 明写「优先突出…长期未到诊」, 示例里还有「此前半年 5 次回访无到诊」在**示范这个错法**。 ── 生产规模 ── 795 条 recall_history 摘要,162 条断言"失联/沉睡/未到诊", 其中 88 条(54%)患者实际 180 天内来过,18 条 90 天内来过。 最离谱一条:「自2018年至今无到诊或有效联系,属长期失联状态,建议优先核实联系方式」 —— 该患者 2026-06-24 刚到诊,上个月的事。客服照这个打电话会很尴尬。 ── 改法(按业务口径「这里只摘要回访事实」)── 不给它喂到诊数据,而是划清职责:到诊状态有专门的地方出(画像卡「末诊 N 天」按真实接诊算)。 1) prompt 加【职责边界】硬约束:输入里没有到诊数据,禁止判断是否到诊/失联/沉睡; 结果字段里的"已就诊"只能当那次回访的记录引述,不得当成到诊状态; 明确掐断"没有回访 ≠ 没来过"这条推理链 2) 修掉关键要求 4 和示例里示范错法的句子,新增反例段(把三条真实事故句列为 ✗) 3) 补程序算好的 daysAgo —— 相对时间不让模型自己减日期 4) 版本 @2026-07-26-f → @2026-07-27-g ── 这是同一个错误第三次以不同形式出现 ── 07-24 是"未来 vs 过去"(未来排程被说成漏做),今天是"回访 vs 到诊"和"距今多久"。 当初立的纪律是「能程序算的事实全算好、LLM 只润色」,但只补了一个轴, 于是同类错误换个轴又来。本次把职责边界和 daysAgo 都用测试锁住(新增 8 例)。 注:已生成的 795 条摘要是缓存,需另行重刷才会用新 prompt; 本次先删了陈芃霖那一条供业务观察重新生成效果。 Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>luoqi committed -
FRIDAY 对接增量 push 报 `SocketTimeoutException: Read timed out`,今天在测试服 按他们的路径(公网域名 + HMAC)模拟推 2 行患者主档定位到: 单次请求 74s,74s **全部**耗在这一条 SQL 上 (逐 6s 采 pg_stat_activity,同一条查询年龄从 6s 一路涨到 74s): SELECT id, patient_id, tenant_id FROM patient_transactions WHERE sync_log_id = $1 AND patient_id IS NOT NULL 来源 cold-import.service.ts `touchedRows`(push / cold-import / 增量共用这条路径)。 EXPLAIN:Parallel Seq Scan (cost=0.00..2150645.78) 外键约束不会自动建索引,这句从上线起就在全表扫: 测试服 patient_transactions 12,519,060 行 / 33 GB 生产 同表 24,074,031 行 / 64 GB ← 生产更慢,FRIDAY 真上生产会更糟 本地建索引后 plan 变 Index Scan(cost=0.29..4.31)。 ── 顺带更正一处仓库里的错误注释 ── 20260701060607 那份迁移写着「CONCURRENTLY 不能跑在事务里,Prisma migrate 对整份 migration.sql 有事务包裹,故不能靠 migrate deploy 直接跑」。**当前 Prisma 6.19 已不成立** —— 本地实测 migrate deploy 直接把 CONCURRENTLY 跑通、索引建出、plan 生效。 新迁移里写清了真实情况,并保留「推荐手动先建」的建议,但理由改成正确的那个: deploy 时 pac-service 要等 pac-migrate 退出才启动,33/64GB 建索引几分钟 = 白等的停机。 push 侧本身是好的:同一次模拟 accepted=2 / failed=0 / mappingMisses=0, canonical 映射(externalId/name/gender/birthDate/medicalRecordNumber/phone)全对。 Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>luoqi committed -
四个 commit: 99e01d42 禁忌能力落地 —— 摄入层抽 clinical_signals 写回源 fact,画像层裁决 51581f58 筛选改按治疗域四类;风险位露出真实原因 f85b7680 画像依据行改显命中信号,不再露 encounter 内部 ID 1171fb42 核查项不进画像;风险区先隐藏;挂号/接诊/接待依据行同样去 ID 上线还需两步(缺第二步则 7 万年龄型禁忌筛不到 —— 本地已实测): 1) pnpm reparse -- --host=jvs-dw --subject-type=emr,diagnosis 2) pnpm recompute-persona -- --host=jvs-dw --concurrency=8 --force (--force 必须带:水位闸只认'事实变了',本次变的是特征输出结构, 本地实测不带 --force 会被 noop 掉 13260/13268) 召回闸门未接,按预留处理。
luoqi committed -
业务口径三处调整: ── ① check 档(待核查)不进画像 ── 「有糖尿病史但控制状态未知」对客服不构成可执行信息,摆出来只会让人把「问一句」 当成「不能治」,是净噪音。现在 diabetes / hypertension / osteoporosis / smoking 不再产任何画像输出(本地实测禁忌特征 1910 → 1840,少的 70 个正是只有核查项的患者)。
⚠ ️ 信号本身**照常抽取并存在 fact 的 content.clinical_signals 里**,一条不丢。 这是「只讲事实」与「只说有用的话」的分工:事实层求全,画像层求准。 顺带改掉复评期的处理:超期不再降级为核查项(那会让绝对禁忌凭空消失),而是 **仍算禁忌**,只在依据行注明「记录较早,建议确认近况」—— 抗凝药可能已停,但漏挡 比多挡危险得多。副作用:正畸禁忌从 0 回到 13 人(之前被复评期降级掉了)。 ── ② 关键事实的风险区先隐藏 ── 画像标签卡的摘要已会提到禁忌,同屏再来一条红块重复。禁忌照常在画像 chip + 详情抽屉露出。代码保留并加 SHOW_RISK_NOTES 开关 —— 这个位置将来要接投诉 / 退费纠纷 / 医疗争议(业务已提出设想,现无数据),届时改回 true 即可。 ── ③ 依据行不再显示宿主内部 ID ── 接着上一个 commit 的 emr_record,把剩下三类一并修掉 —— 它们的 f.title 都是 `<中文> <externalId>`: 接诊 16432552 → 洁牙 / 正畸复查(取 content.chief_complaint) 挂号 <id> → 科室 · X医生 前台接待 <id> → 前台接待 生产实测 encounter_record 93.4%(2,004,383/2,146,887)带主诉,这条修复对绝大多数 依据行是实质改善,不是换个无用字符串。 测试 550 passed。 注:sync-lock-reap 那条在 37 suite 并发时偶发失败,单跑(改动前后均)通过 —— 是 它自身的时间竞态 flake,与本次改动无关,未处理。 Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>luoqi committed -
业务反馈禁忌标签的依据行看不懂: 2022-03-26 病历 关联接诊:15514 ← 内部 encounter id,对客服毫无意义 根因在 fact-label.ts:97 —— 对 emr_record 取 f.summary,而注释写着 「summary 是命中的那段」。这个注释是错的:emr.parser.ts 实际写进 summary 的是 `关联接诊:${encounterExternalId}`。依据区从上线起显示的就是个内部 ID。 现在按信息量三级降级: ① content.clinical_signals 里的阳性信号 label(这条病历到底命中了什么) → 「局麻药物过敏」,悬停出医生原话「自述普鲁卡因、利多卡因过敏」 ② 病历里真正有内容的 SOAP 段落(既往史/主诉/全身情况/医嘱)前 40 字 ③ 才退到 title 任何情况下都不再出现 `关联接诊:<id>`。 FactLabel 加可选 hover 字段:列表行窄、原话长(「有放射治疗史,三周前进行 颈淋巴结清扫(舌癌淋巴结转移)」),truncate 后必须能悬停看全。 标签是 PAC 的归纳,原话才是事实本身 —— 两者都要够得着。 顺带修好的不止禁忌:治疗敏感 / 不可等候等所有以病历为证据的画像标签, 依据行同样从内部 ID 变成可读内容。 Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>luoqi committed -
业务反馈两条,都是我做偏了: ── ① 筛选按四类,不按病因 ── 原来列的是 12 个病因(抗凝药/放疗史/抗骨吸收药…),那是算法口径。 客服圈人时想的是「哪些人不能做种植」,不是「哪些人在吃抗骨吸收药」—— 把内部实现摊给业务看,面板还长。改成 data.domains:手术/种植/麻醉/正畸禁忌。 病因照常在关键事实卡与画像详情里露出(带医生原话);要按病因查是风险排查场景, 需要时另加维度,别混进圈人面板。 不列「拍片禁忌」:唯一来源是妊娠,而妊娠同时禁种植和手术,勾那两项已覆盖。 ── ② 风险位直接露出真实原因 ── 原来只显示「手术禁忌 / 种植禁忌」,凭什么要 hover 才知道 —— 这正是我自己在设计里 写的「客服不知道凭什么就会当噪音忽略」,结果自己没落实。现在:⚠ 手术禁忌 / 种植禁忌 放疗史 · 2022-12-02 病因+日期直接成行,全文原话仍留 hover(一行放不下「有放射治疗史,三周前进行 颈淋巴结清扫(舌癌淋巴结转移)」)。 同时把这块抽成通用 RiskNote 形态 —— 业务提到这个位置以后要放投诉之类, 现在没数据不预建,但形状留好:再来一个来源就是往 notes 里 push 一条。 ──⭐ 顺带发现一个部署坑(本地实测) ── reparse 只重算**事实变了**的患者。年龄型禁忌那批人事实没动 → 画像没重算 → data.domains 还是 null → 新筛选完全筛不到他们(生产是 7 万人)。 本地实测:普通 recompute-persona 被水位闸 noop=13260/13268 全跳过; 加 --force 后 success=1804 refreshed=9536,domains 才落库,筛选立刻从 10,458 人收敛到 3 人(= 库里那 3 个局麻药物过敏患者,逐个核对一致)。 所以上线顺序必须是:reparse → recompute-persona **--force**。 --force 的注释里本就记着同类事故(「reparse 后的全量重算被这道闸 noop 掉 325,979 人」),我差点重蹈。 Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>luoqi committed -
业务方细则(种植/正畸/麻醉/手术禁忌)按「病因建模、按治疗域映射」实现。 原来只有一条「年龄≤18 = 种植禁忌」的规则,而且跟召回侧的年龄闸重复, evidence.factIds 是空数组 —— 客服看到标签不知道凭什么,只会当噪音。 ── 分层 ── 摄入层 抽取(这句话提到抗凝药吗) per-row、确定性、可重放 ← ParserPipeline hook ↓ 写回**源 fact 自己的** content.clinical_signals 衍生层 裁决(他现在是不是手术禁忌) 跨记录、跨时间 ← persona 特征 不新建衍生 fact 类型 —— 遵守 FactType 的设计纪律「fact 只装源事件,衍生信息留在源 fact 的 content JSONB 里」。信号跟着源病历生命周期走(病历订正→重放→自动重抽), 不必额外维护一致性;召回闸门将来也只需一行 content->'clinical_signals' @> ...。 ── 三档严重度(收束点)── absolute 绝对禁忌 → 打标签,将来接召回闸门挡的是这一档 relative 相对禁忌 → 打标签 + 话术警示,不挡召回(由医生按身体条件评估) check 核查项 → 只提示「预约前确认X」,不叫禁忌 细则里三条**刻意降级为 check**:糖尿病要 HbA1c(生产全量仅 10 条)、高血压要收缩压 (几乎无人记录)、TMJ 要急性期(病历不记)。控制达标的糖尿病是可以种植的,硬判禁忌 医学上就是错的 —— 判断权还给医生,PAC 只把事实摆出来。 ── 时效(「禁忌≠终身禁忌」)── 妊娠 280 天 / 哺乳 180 天(产后半年)/ 放疗·抗骨吸收药终身 / 抗凝药等设复评期 (超期降级为 check 而非消失 ——「消失」会让客服以为患者从没有过这个病史)。 到期日在抽取时按源病历 occurredAt 算好写进 signal,画像 TS 与召回 SQL 各自只做一次 比较,时效规则不分叉。nextBoundaryAt 报出最近到期日 → 到期当晚自动重算。 ── 否定作用域:全模块最要紧的一条 ── 「患者否认有高血压、冠心病等心血管疾病;糖尿病、痛风等代谢性疾病。」 分号后的糖尿病仍在否定作用域内。生产 88.9 万条既往史实测:按分号切 → 阳性 1,328 人; 按句切 → 141 人,差 9.4 倍。这条错了整个模块就是负资产。 再断言(有/患/服用)只有隔了并列符才翻回阳性,否则「否认有」这个常见搭配会被整片判反。 「患者否认/自述有…」(生产 116 条,医生没删模板斜杠)→ negated=null,入库供人工复核但不产禁忌。 ── 生产实测(702 万行文本全量跑过)── 命中任一信号 0.75%(→ 首次 reparse 的 fact 版本波规模可控) 阳性率:抗凝 94.5% / 心脏植入物 95.2% / 哺乳 92.1% / 放疗 91.0% / 重度牙周炎 99.9% 上线后:3,790 人带禁忌标签,10,823 人仅带待核查(另有年龄禁忌 7 万人不变) 全量跑还揪出两类假阳性并已修:「备孕期」⊂「孕期」(78 行)、「妊娠期龈炎」是牙龈炎 诊断名不是妊娠状态(58 行)—— 中文无词边界,加了 not_any 反向清单。 ── 前端 ── 关键事实卡新增安全提示栏(⚠ + rose),禁忌与待核查分行显示,hover 出医生原话 + 记录日期。 不混进标签云:那里禁忌跟急迫等级同色同排平铺,客服扫不出来。 召回闸门本次**不接**,按预留处理。字典改动复用既有 reparse,不另造 CLI: pnpm reparse -- --host=jvs-dw --subject-type=emr,diagnosis --dry-run 测试 545 passed(新增 44 抽取 + 18 裁决,用例全部抄自生产真实文本) Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>luoqi committed -
luoqi committed
-
昨夜测试服跑优化后的全量回填(35.7 万患者,2.2 小时跑完、2,694 人/分)暴露两个**既有**问题, 都不是优化本身造成的。 ## ① /dev/shm 64MB → plan 全量重算当场崩 ERROR: could not resize shared memory segment "/PostgreSQL.xxx" to 12615680 bytes: No space left on device Docker 给 /dev/shm 的默认只有 64MB(inspect 确认 ShmSize=67108864),而 PG 的并行查询 worker 用共享内存段交换中间结果。召回的 selectHits 把 **10 个子场景 Promise.all 并行跑**, 每个都是 2500 万 patient_facts 上带 LATERAL 的重查询、各自还起 parallel worker, 并发的段远超 64MB。 崩在 selectHits → runSubScenario,那处 Promise.all 本次一个字没动 —— 与 worker pool 无关。 compose 里给 postgres 加 `shm_size: 1g`(PG 容器常规配法)。⚠ ️ 生产用阿里云托管 RDS、不启本服务,不受此问题影响,也不受本改动影响。 ## ② CLI 与 stale-scan cron 抢版本号 → Unique constraint 全量回填跨过了 02:00 的 `PAC_STALE_SCAN_CRON`,两个**独立进程**同时给同一批患者建新版本: 各自在事务外读到同一个 latest.version、算出同一个 nextVersion → `Unique constraint failed on the fields: (patient_id, version)`。 实测那一轮 CLI 挂 688 个、**定时任务自己挂 1,604 个**(它伤得更重)。 BullMQ processor 内部按 patient_id 串行,挡得住自己人,挡不住外面另起的 CLI 进程。 上一轮旧代码没撞,只是因为它在 2 点前就被我杀了 —— 运气,不是设计。 修法:两条写路径进事务先拿**按患者的 advisory 事务锁** (`pg_advisory_xact_lock(hashtext(patientId))`,随事务自动释放、跨进程有效); 建版本那支再**在锁内重读**最高版本 —— 迟到者看到已经变大的版本号,顺序叠上去。 就地刷新那支也拿同一把锁,否则可能写到刚被并发写者 supersede 的版本上。 ## 验证 本地真库对照:取同 300 个患者、删掉其画像(逼两个进程都算出 version=1)、两进程并发 --force 旧代码 A: success 18 failed 216 | B: success 282 failed 18 → **234 次 unique 冲突** 新代码 A: success 186 failed 0 | B: success 300 failed 0 → **0 次冲突** 498 单测通过(36 suites),tsc 干净。persona-watermark.spec 新增 4 例锁住: 建版本前必拿锁 / 版本号锁内重读 / 就地刷新那支也拿锁 / noop 不白拿锁。 Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>luoqi committed -
⚠ ️ 只合代码,**先不部署** —— 测试服正在跑画像回填(优化后代码),部署会重建容器把它杀掉。 等回填 + 链上的 plan 全量重算跑完再部署。luoqi committed -
plan-engine.runAllForHost 的 upsert 段用的是同一种写法: for (i += N) await Promise.all(entries.slice(i, i+N).map(...)) 每批都要等本批**最慢**那个,快的 worker 干等着。 同一处改法已在 recompute-persona 上实测(测试服 35.7 万患者,concurrency=8): 并发效率 44% → 94%,吞吐 1,454 → 2,694 人/分 plan 侧单患者耗时的离散度只会更大(upsertPlan 内 reason 数量差异明显),浪费同理。 改:换成 common/run-pool.ts 的连续调度 worker pool(N 个 worker 各自取下一个)。 并发上限、错误处理、计数口径都不变 —— 计数仍靠 JS 单线程 await 恢复后同步自增。 顺带:recompute-plans.cli 在 NestFactory 之前把 PAC_DB_CONCURRENCY 对齐到 PAC_PLAN_BATCH_CONCURRENCY(默认 8)。同 recompute-persona 的理由:PrismaService 在容器 初始化时就定池,不设的话是 Prisma 默认「核数×2+1」(生产 4 核 = 9),把 batch 并发调到 9 以上会被卡住。按默认 8 跑时 9 条刚好够,本项属于**解锁更高并发**,当下不产生收益。 ## 验证 - 494 单测通过(36 suites),tsc 干净 - 本地 friday host 全量重算(10,458 plan):15.94s,plansCreated=0, 跑前跑后 followup_plans 指纹均为 3d5225bf6c1160226739385c6f8e7d8b —— **行为中性** Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>
luoqi committed -
原注释写「生产实测 --concurrency=8 时进程恰好只有 9 条连接」,不准确。核对两台机器的 .env 之后: 测试服 47.251.104.47 URL 显式 `connection_limit=30` → withCohortDerivedPool 提前返回, 池=30。**我观测到的那 9 条其实是 8 个 worker + 1 空闲,不是池上限。** 生产 47.99.62.30 URL 没写 connection_limit → Prisma 默认 核数×2+1,4 核机 = 9 所以: · ② 这个缺陷本身成立(函数只认 PAC_COHORT_CONCURRENCY,CLI 旋钮不驱动池) · 但它是**潜在**的,只在没显式配 limit 的机器上咬人 · 且按计划的 --concurrency=8 跑生产时 9 条刚好够 —— **本修复对今晚的回填零收益**, 价值只在解锁更高并发 代码/测试注释同步改对,别让后人照着一个错的观测去推断。实现与单测未动(行为本来就是对的)。luoqi committed
-
- 26 Jul, 2026 11 commits
-
-
原注释写「生产实测 --concurrency=8 时进程恰好只有 9 条连接」,不准确。核对两台机器的 .env 之后: 测试服 47.251.104.47 URL 显式 `connection_limit=30` → withCohortDerivedPool 提前返回, 池=30。**我观测到的那 9 条其实是 8 个 worker + 1 空闲,不是池上限。** 生产 47.99.62.30 URL 没写 connection_limit → Prisma 默认 核数×2+1,4 核机 = 9 所以: · ② 这个缺陷本身成立(函数只认 PAC_COHORT_CONCURRENCY,CLI 旋钮不驱动池) · 但它是**潜在**的,只在没显式配 limit 的机器上咬人 · 且按计划的 --concurrency=8 跑生产时 9 条刚好够 —— **本修复对今晚的回填零收益**, 价值只在解锁更高并发 代码/测试注释同步改对,别让后人照着一个错的观测去推断。实现与单测未动(行为本来就是对的)。luoqi committed -
四项:worker pool 替代批次栅栏 / --concurrency 真正驱动连接池 / patient_facts 两查合一 / 审计日志 2 写改 1 写。往返 12.7 → 10.7。 本地 13,268 患者实测 93.50s → 76.89s(1.22×),指纹法证明产出与旧算法逐字节一致。 但本地延迟分布(avg 35.9ms / max 436ms)测不出 worker pool 的真实收益 —— 测试服是 avg 165ms / max 75,750ms,长尾会冻住整批。 上测试服的目的就是拿真实倍数:旧代码基线已录得 1,454 人/分 · avg 146ms(concurrency=8)。
luoqi committed -
生产/测试服实测基线(测试服 12 万患者样本,concurrency=8): 单患者 p50 139ms / p90 230ms / p99 386ms / max 75,750ms,均值 165ms 理论上限 8÷0.165 = 2,909 人/分,实际 ~1,600 → **效率只有 55%** ## ① 批次栅栏 → worker pool(src/common/run-pool.ts) 原写法 `for (i += N) await Promise.all(slice)` 是**栅栏**:每批都等本批最慢那个, 快的 worker 干等。缺口正好是 E[N 次抽样最大值](≈p90)与均值之比。 极端例子:那个 75.7 秒的患者把同批 7 个 worker 一起冻了 75 秒。 改成 N 个 worker 各自从共享游标取下一个,谁空谁取 —— 慢任务只拖住自己那条线。 ## ② --concurrency 根本没放大连接池(真 bug) `withCohortDerivedPool` 只认 `PAC_COHORT_CONCURRENCY`(**摄入**的旋钮), 于是 `recompute-persona --concurrency=N` 的旋钮**静默失效**:实测跑 --concurrency=8 时 进程恰好只有 9 条连接(= 4核×2+1 的 Prisma 默认),超过 9 的 worker 全卡在池上排队。
⚠ ️ 这个 bug **只在小核机器上咬人**,所以一直没被发现:服务器 4 核 → 默认池 9; 开发机 16 核 → 默认池 33,本地压根复现不出来。 改:池改读通用的 `PAC_DB_CONCURRENCY`,由各 CLI 在 NestFactory **之前**按自己的 --concurrency 设进来(PrismaService 在容器初始化时就定池,晚了没用); `PAC_COHORT_CONCURRENCY` 保留为向后兼容别名,两者取大。 ## ③ 同一张表查两遍 → 一次取,内存分两份 persona.service 对 patient_facts 每患者查两次(同患者、同索引、两次往返): ① status IN (active, fulfilled) → factsByType ② type=appointment_record AND status != superseded → appointmentsAll ② 是 ① 在 appointment 这类上的放宽,故取 `status != 'superseded'` 一把捞,内存切两份, 语义逐字节等价。 ## ⑤ 审计日志 2 写 → 1 写 原 create(status='running') + update(终态)。startedAt 可显式赋值,故耗时统计一字不差。 代价:进程被硬杀(OOM/SIGKILL)时不留痕 —— 评估可接受:try/catch 里的异常仍写 'failed' 行; 唯一丢的是"算到一半进程没了",而那种情况原本留下的是一条**永远卡在 running** 的行(没人清理)。 读侧确认无依赖(daily-health-report 只用 findFirst 最近一条 + 按 status 的 groupBy)。 全量回填还顺带少一半日志表写入(35.6 万 → 17.8 万行/轮)。 往返预算:12.7 → 10.7(-16%)。 ## 本地实测(13,268 患者,--force,concurrency=8,同为「全 unchanged」稳态) 旧代码 93.50s 优化后 76.89s → 1.22×,省 17.8%⚠ ️ 本地测不出 ① 的真实收益:本机 avg 35.9ms / p90 49ms / **max 436ms**, 而测试服 avg 165ms / p90 230ms / **max 75,750ms**(174× 均值)。本地没有那种能冻住 整批的长尾,且延迟低到瓶颈更像是 JS 单线程 CPU 而非 DB 往返。 所以 1.22× 主要是 ③⑤ 的往返削减(机器无关),① 的收益要到服务器上才显现 —— **具体多少不预估,等在测试服实跑对照。** ## 正确性 指纹法:优化后跑完 → 切回旧代码再跑 → 旧代码对全部 13,268 人判 `unchanged`, persona_features 指纹 4de1dcd0ddc6a2c99e222fd43e0755de 不变。 即**旧算法认为新代码的产出与它自己会算出来的逐字节相同**。 ## 测试 494 单测通过(36 suites),service tsc 干净。新增 tests/persona-recompute-perf.spec.ts 12 例: runPool — 不丢项 / 并发上限严格且用满 /⭐ 慢任务不冻住其他 worker / 边界 / 错误上抛 连接池 —⭐ PAC_DB_CONCURRENCY 生效 / COHORT 向后兼容 / 取大 / 封顶 40 / =1 不动 URL / 已有 connection_limit 不覆盖 / 非标准 URL 不抛 ## 未做(单独排) ④ gapSelector 每患者平均 2.66 条 SQL(实测分布 0 组 3.4% / 1 组 19.5% / 2 组 25.7% / 3 组 24.7% / 4+ 26.7%,max 9)。UNION ALL 合一能再省 ~1.7 次往返,但动的是**召回共享** 的 gap 真理源,风险最高,要配专门的对账测试。 另注:plan-engine.runAllForHost 有一模一样的批次栅栏写法(PAC_PLAN_BATCH_CONCURRENCY), 同样的 5 行改法适用,本次未动。 Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>luoqi committed -
## 事故(2026-07-26,生产 friday / 47.99.62.30) 在生产机漏了 `COMPOSE_MANAGED=1` 跑 deploy-prod.sh。不叠加 managed override 时, docker-compose.prod.yml 把 DATABASE_URL 硬编码成 `postgres:5432`(容器内自带 PG),于是: 22:35:33 起了 pac-postgres-1 / pac-redis-1 两个空容器(volume 同刻创建) 22:35~36 pac-migrate 对着这个**全新空库**跑完 29 条迁移,建出整套 schema ← 在这里被打断 22:38:29 正确那次(COMPOSE_MANAGED=1)把两条新迁移写进托管库 托管生产库**未被写坏**,逐项核对:patients 425,874 / facts 25,022,720 / 在池 236,553 / assigned 36 / personas 409,368 / plan_executions 7 —— 与事故前一致 (patients、facts 的 +2/+206 是正常增量摄入)。空库那侧唯一有行的表是 _prisma_migrations(29 行)。 ## 真正的后遗症(已清理) 机器上留下一个「schema 齐全的空库」。危害不在占资源,在于: **下次再漏 COMPOSE_MANAGED,service 会顺利连上它、health 200、脚本三项验证全绿, 生产静默服务空库且零告警。** 原本"空 PG 没 schema 会崩"这张天然安全网就此失效。 已在生产执行:docker rm -f pac-postgres-1 pac-redis-1 docker volume rm pac_postgres_data pac_redis_data 清理前核过:两 volume 无其他容器挂载 / pac-service 连的是 RDS+托管 Redis / pac-web 无 DB 环境变量 / 本地库业务表全 0 / 全机再无其他 volume。清理后 health+web+私网 200。 ## 防线 不依赖人记得加环境变量,而是问机器自己 —— 读 `.env` 的 DATABASE_URL 指向哪儿: · postgres / localhost / 127.0.0.1 → 自带 DB 形态,放行(测试机) · 其他主机(RDS 等) → 托管形态,没设 COMPOSE_MANAGED=1 就 die 闸放在 main() 最前,早于任何 git/docker 动作 —— 拦下时机器状态零变更。 报错只打主机名不打整串 URL(不泄露 .env 口令)。 ## 验证 - 新增 tests/deploy-managed-guard.spec.ts 5 例,真起 bash 跑脚本: 托管+漏设→退出且给出正确命令 / 报错不含口令 / 托管+设了→放行 / 自带 DB→放行(不误伤测试机) / localhost 与 127.0.0.1 同样放行 - 482 单测通过(35 suites),service tsc 干净 Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>luoqi committed -
已在 test(a0d20f82)部署验证。三块内容: 1. 画像可解释性 —— 水位对齐 fact 层 + 写入侧节流;到期改档精确调度(nextBoundaryAt); 详情页说明收回 @pac/types 单一真理源 + 打通证据链 + 分组定序 + 圈人标签 hint 2. 历史联系「最近一次」—— 程序判定 isLatestPast,摘要开头必须交代;prompt 加数据边界 硬约束(源里只有标签没有原话,没记的不许补);抽屉标最近一次/未到期; 修 fallback 拿 items[0] 当最近一次的老 bug;两处 orderBy 改 nulls:'last' 3. ⑤g 未来回访排除闸 —— 诊所已排未来回访则本轮不召,排到哪天挡到哪天,到期自愈
⚠ ️ 部署后必须做的两件事(见两个 migration 文件内注释): a. patient_facts 复合索引已在 migration 内,大表建议先 CONCURRENTLY 预建 b. 画像回填 **--force 必须加**:next_boundary_at 为 NULL 语义是「不再改档」, 水位闸认为这些版本新鲜、永远不会重算,不会自愈 c. ⑤g 需 plan 重算才整体生效(生产预估挡下 30,426 条在池 plan / 12.9%) 本地 477 单测 / service+web tsc 干净;test 服已验闸的两个方向。luoqi committed -
三块内容,都在这个分支上做完并本地验过: 1. 画像可解释性(889dd6f0…3a85d4b1) · 水位对齐 fact 层 + 写入侧节流(修 ¥154,350 vs ¥195,750 那个漏算) · 到期改档精确调度(nextBoundaryAt),时间跨档能刷新但不靠全量扫 · 详情页说明收回 @pac/types 单一真理源 + 打通证据链 + 分组定序 + 圈人标签 hint 2. 历史联系「最近一次」(f634f274) · orchestrator 程序判定 isLatestPast,摘要开头必须交代最近一次 · prompt 加数据边界硬约束:源里只有标签、没有原话,没记的不许补 · 抽屉标「最近一次」/「未到期」;修 fallback 拿 items[0] 当最近一次的老 bug · 两处 orderBy 改 nulls:'last'(Postgres DESC 默认 NULLS FIRST 会让无日期记录冒充最新) 3. ⑤g 未来回访排除闸(5ed2e888) · 诊所已排未来回访 → 本轮不召,排到哪天挡到哪天,到期自愈 · 生产将挡下 30,426 条在池 plan(12.9%),其中 urgent 17,263 另含 eb4d01c6:.swcrc 的 exclude 是正则必须锚定(文件名含 dist 会被静默跳过)。 本地:477 单测通过 / service+web tsc 干净 / 真库跑引擎验过闸的两个方向。
luoqi committed -
业务可用性测试 Issue「超百天自动加急覆盖回访计划」: [被测者]「超过百天他就会给你加急自动升了一个紧急,而且他设置过的回访的话他实际是不记得」 ## 先纠两个事实 1. 阈值是 **90 天**不是 100 天(urgency_level:>90 紧急 / 30-90 高 / <30 中), 而且它不只是标签 —— 急迫性 ×0.4 是综合分权重最大的一块。 2. 「覆盖了回访计划」不准确:PAC **从没看过** task_date,不是覆盖是没接。 (回访表本身早就进召回了 —— gapWhere 里的外院已治疗闸读 rv.result; 只是一直没用 task_date 这一列。) ## 口径:排到哪天挡到哪天 AND NOT EXISTS (SELECT 1 FROM patient_return_visits rv WHERE rv.patient_id = p.id AND rv.task_date > now()::date) 跟 ⑤b(未来预约)同一个道理,注释原话就够用:「召回目的 = 让客服建预约。 患者已经有未来预约 → 客服不需要再 push」。回访是同构的,只是接触形式换成电话/微信。 三个设计选择,都是被生产数据推翻假设之后定的: · **不设窗口上限。** 原本想拍 N=60 天,查完发现理由不成立:在池患者的未来排程 99.65% 在一年内(2026 年 25,105 / 2027 年 5,271 / 2028 年以后共 50 条,最远 2033-11-12)。 固定窗口反而会在第 N+1 天把"诊所排了 N+30 天回访"的人放回池,制造重复触达。 那 50 条脏数据不值得引一个新常量 —— 真要防该在摄入侧校 task_date 范围。 · **不为"排了不做"加兜底。** 全库到期任务执行率 74.3%(118.1 万 / 159.1 万), 确实有 1/4 烂尾。但闸的条件是 task_date > today,任务日一过闸自动失效、患者当天回池 —— SQL 每轮重算就是自愈机制,不需要解除逻辑。 · **放 scenario 不放 gapWhere。** 这是本次唯一的架构判断: gapWhere(召回 + 画像共享)= 【临床事实】层 —— 已治疗/患者拒绝/外院已做 → 缺口真没了 scenario WHERE(仅召回) = 【运营时机】层 —— 缺口还在,只是现在不该打电话(⑤b/⑤f) 未来回访显然是后者:排了个电话不代表那颗缺牙补上了。放进 gapWhere 会让画像因为 "排了个回访"就说这人没有潜在治疗 —— 那才是口径分叉。 ## 生产影响(上线前实测) 在池 plan 236,508 ⑤g 将挡下 30,426 (12.9%) 其中 urgent 17,263 ← 正是一线抱怨的"明明排了回访还催我" 未来排程到期分布 ≤30天 8,724 / 31-90天 8,502 / 91-180天 9,407 / >180天 3,793 被挡住的 plan 会被 supersede(0 命中 → closeStaleActivePlan),到期是新版本 —— 已确认 可接受:数据都有记录,跟踪得到;分配逻辑将来单独做。contactAttempts 归零暂不处理。 ## 顺带修一处过期注释 plan-engine.service.ts:70 还写着「关闭 active(非 assigned)plan / assigned 不动」, 但代码从批量路径修复那次起就已含 assigned(:145 条件是 active OR assigned, :261 注释也写明是有意改的)。注释改成与代码一致。 ## 本地验证 - 477 单测通过(新增 8 例:闸形状 / 只认未到期 / 患者级 / 无窗口上限 / ⑤b⑤f 未被替换 / gapWhere 不含未来 task_date / gapWhere 仍保留外院闸 / 画像 SQL 不含本闸) 测试用递归摊平 Prisma.Sql 的方式断言 —— 嵌套片段在 tagged template 里只是绑定参数, 不摊开断言会假通过(第一版就踩了这个,gapWhere 内容根本没被检查到)。 - service / web tsc 干净 - 真库跑引擎: 吕学文 1921711(有 2026-10-09 未来回访)→ plansClosed=1,plan 转 superseded 出池 李石明 1873810(无未来回访) → 不受影响,仍 active 把 2026-10-09 改成昨天再跑 → plansCreated=1,v2 active 回池(自愈方向验证),随后还原 Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>luoqi committed -
业务可用性测试 Issue 2「历史联系记录缺具体内容」(患者余同学 BJ0L028292): [被测者]「说了常规的提醒没有特殊的」 [主持人]「那就是暂时没有办法详细的看到具体一次沟通什么内容」 ## 先查源:PAC 没有藏数据,源侧确实只有标签 DW 直查 fact_returnvisit_out(customer_id=1683102 brand=瑞尔),该患者 5 条: 2026-05-20 常规回访 未回访/未完成 内容「复查洁牙」 结果 null 2026-02-25 常规回访 未回访/未完成 内容「方案跟进」 结果 null 2026-02-23 常规回访 已回访/已预约 内容 null 结果「复查」 2025-11-16 常规回访 未回访/未完成 内容「半年复查」 结果 null 2025-08-17 常规回访 已回访/已完成 内容 null 结果「复查」 PAC 库里 5 条一条不差,7 个业务字段全部已在抽屉展示 —— **没有少展示任何东西**。
⚠ ️ 排查中的一个坑,顺手确认了不是 bug:DW 直查会返回 15 条,多出的 10 条 brand=瑞泰。 patient_id 跨品牌复用(374 万 id 中 196 万重号),1683102 在瑞尔=余继来(9岁, BJ0L028292)、在瑞泰=宋可超(33岁,QD0D020738),是两个人。摄入按 (patient_id, brand) 复合键切分,各归各家,正确。 全库口径(生产 165.2 万条回访): follow_content 填充 51.0%(均 12.4 字)· return_visit_result 填充 67.4%(均 9.1 字) 取值 TOP:「种植潜客」8.6万「提醒洗牙」3.9万「无不适」8.2万「已约」3.5万「未接」1.2万 即业务想要的三样东西,源里的实际情况是: · 沟通结论 / 患者态度 → **只有粗粒度标签**(无不适/已约/未接/挂断/拒接),没有原话 · 下次建议联系时间 → **完全没有**(个别混在 result 自由文本里,如「7月份再联系」) · 逐字沟通内容 → **没有**;dw_group 全库只有 fact_returnvisit_out 一张联系表 → 结论:摘要说「常规的提醒」是**如实转述**,不是 PAC 概括丢了信息。PAC 不造数据。 ## 再改:业务关心「最近一次」这条建议成立,采纳 列表按日期倒序时头部常是**还没到日子的排程**,客服最想知道的"上次聊到哪儿了" 反而被埋在中间 —— 这是真问题,跟"源里没内容"是两回事。 摘要侧(promptVersion → @2026-07-26-f): · orchestrator 程序判定 isLatestPast(第一条有日期且已到期的),同 isFuture 的纪律 —— 哪条是"最近一次"是排序问题,不交给 LLM 比日期 · prompt 渲染【最近一次·已发生】标记 + 要求摘要开头交代它(时间+事由+结果) · 新增「数据边界」硬约束段:明写源里只有标签、没有原话和下次时间, 没记的一个字都不许补,禁止输出"患者表示…"这类原话式表述 · 最近一次那条正文预算 40 → 160 字(源里偶有几百字的方案沟通记录, 一刀切 40 字会把唯一一条有信息量的记录砍没) · 它没记结果时显式渲染「结果:未记录」—— 不写模型会拿别条的结果脑补这条 抽屉侧:最近一次加 teal 角标 + 底色,未到期的加「未到期」灰标; 最近一次没结果时明写「结果:宿主未记录」(空着会被当成 PAC 没展示,正是这次的误会来源)。 顺带两个同源 bug: · fallback()(LLM 失败时给客服看的兜底句)还在拿 items[0] 当"最近一次", 等于把 2026-07 那次未来排程事故在兜底路径上原样复刻 → 改走 isLatestPast · returnVisits 的 orderBy taskDate desc 在 Postgres 默认 NULLS FIRST, 无日期的记录会顶到列表最前冒充"最近一次",还吃掉 take 名额 → nulls:'last' (摘要 take:12 与聚合 take:100 两处都改,口径一致) ## 本地验证 - 469 单测通过(本文件 13 → 24,新增「最近一次」13 例,含余同学生产原始形态回放) - service / web tsc 干净 - 吕学文(BJ0U017795,4 条历史 + 1 条 2026-10-09 未来排程)清缓存连跑 3 次稳定: 「最近一次4月4日治疗后回访已完成;此前种植咨询未回访,已排10月9日复查洁牙邀约。」 李石明(BJ0U017487,最近一次无结果)同样 3 次稳定: 「最近一次4月种植咨询回访未执行、未留结果;此前3次常规回访均已约到诊复查…」 —— 开头讲最近一次、没记结果如实说、未来排程仍说"已排"不说"漏做",三个方向都对 - 抽屉截图确认:2026-10-09 带「未到期」灰标,2026-04-04 带「最近一次」角标+底色 ## 待定(本次不做,需业务决定) - DW 有两列我们没摄:task_director(回访负责人姓名,「上次是谁跟的」)、 appointment_id(该次回访是否真开出预约 —— 比 result 文本硬的结果证据)。 加这两列要 migration + 165 万行 reparse,先记着。 - fact_complain_out(8.15 万条投诉)整张表未摄入,是目前唯一没用的患者态度硬信号。 Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>luoqi committed -
跟刚 revert 的 hover 改动无关,单独留下 —— 这是个静默的构建坑,任何人加文件都可能撞上。 原写 ["node_modules", "dist"],看着像目录名,swc 实际当**正则**匹配整个文件路径。 于是文件名里含 "dist" 的源文件被直接跳过:271 个源文件只编译出 270 个,swc 不报错 (照样打印 "Successfully compiled: 270 files"),直到进程启动才 MODULE_NOT_FOUND。 同类地雷:distinct / distance / district / redistribute… 更难查的是两边表现不一致 —— 只在 swc 路径(dev / --builder swc)复现, nest build 走 tsc 一切正常。 锚定成 ["^node_modules/", "^dist/"],并加断言:exclude 必须以 ^ 开头, 且遍历 src 下全部 .ts 确认一个都没被误伤。 456 tests green。 Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
luoqi committed -
Revert a4b75100 + 22b2c95f + 4477a8c2。 理由:改完比原来更不清晰。截图实例 —— 价值性 9 分、意愿度 8.0 分,右边却写着 「画像未标潜在治疗」「画像未标价值分群」:分数明明算出来了,解释说没数据,自相矛盾。 根因是 breakdown 里那几个展示字段只在 plan 重算后才有,存量 / 其他 host 的 plan 一律落到 "未标"分支 —— 而"未标"这个措辞本身也把「后端没透出」说成了「画像没打标」。 绕了一圈说明这块的设计还没想清楚,不该边改边试。退回原状,择期重做。 保留(与本次 UI 无关,单独重提):.swcrc exclude 正则未锚定的构建坑。 ## 重做时手上已有的事实(都已核过,别再查一遍) · 「看不出高低」是真问题:生产 active+assigned 23.6 万,priority_score p50=31 / p80=56 / p90=66 / p95=76 / p99=82。9.78 分是前 0.05%,而客服无从判断。 · 三维输入里 urgencyLevel / rfmSegment / lifecycleStage / treatmentHistory / referralChampion / specialAttention 来自画像;consultIntentMatch 来自 consultation_record;primaryCode / toothCount 来自本条召回的 gap。 · consultIntentMatch 基本不可用:237 万条 consultation_record 只有 9.7% 解析出非空 intent_categories,其余 214.8 万条源端 intent/content 全为 null(DW 没给,非解析 bug)。 intentBehavior=2 的 22 万患者里真的没咨询过的只有 230 人(0.1%)。 · 存量对不齐(与本次无关,仍在):按年龄排治疗只落到召回目标,没落到画像 —— classifyGapToLabel 里 K08 仍只卡 age>18,生产 2,539 条 plan 目标写「活动义齿为主」 而画像·潜在治疗仍标「潜在种植」。改它会同时改变「潜在种植」的圈人口径,需业务定。 Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
luoqi committed -
三维的输入本来就取自画像(急迫等级 / 价值分群 / 生命周期 / 潜在治疗,见 scorer 入参), 但我为了"讲得细"又从 reason.signals 和 consultation_record 各推了一段话 —— 那是第二套口径, 和「画像标签」抽屉必然对不齐。两处已经翻车: · 从 signals.triggers 反推出「种植 · 缺 6 颗」。同一屏上 79 岁患者的召回目标写的是 「活动义齿为主」,画像自己写的是「潜在种植」—— 三处三个说法。 · 把 intentBehavior=2 说成「没主动问过」(上个 commit 已修措辞,这次直接移除): 它来自 consultation_record,根本不是画像标签。 改成照抄画像: 急迫性 10 × 0.4 急迫等级 紧急 价值性 10 × 0.3 潜在治疗 潜在种植 / 潜在牙周 意愿度 2.63 × 0.3 价值分群 重要挽留 · 生命周期 流失客 与该患者「画像标签」抽屉里的取值逐字一致。画像没标就说「画像未标 X」,不编。 为此 scenario 的 persona 上下文多取一个 potential_treatment,由 scorer 原样透传进 breakdown(仅展示、不参与打分)—— 前端拿画像的原文,而不是自己拿 K 码 + 牙位再算一遍。 中文名一律走 personaTagLabel(圈人字典),本文件不留映射表。hover 不再需要 signals 入参。 守卫测试相应收紧:三维文案里出现「缺 N 颗」「咨询」等非画像来源的词就红。⚠ ️ 顺带查出一处**存量对不齐**(本次未改,需业务定):按年龄排治疗只落到了召回目标, 没落到画像 —— classifyGapToLabel 里 K08 仍只卡 age>18。生产 2,539 条 plan 目标写着 「活动义齿为主」而画像·潜在治疗仍标「潜在种植」。改它会同时改变「潜在种植」的圈人口径。 481 tests green · pac-web build 通过 · 本地真实数据逐条比对过画像抽屉。 Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>luoqi committed
-