From 6a2dcde095dfda0ffb0ac5f61b285da180027d70 Mon Sep 17 00:00:00 2001 From: Jesse_Chen Date: Thu, 3 Sep 2026 02:16:29 +0000 Subject: [PATCH] =?UTF-8?q?docs(rectification):=20adopt-flow=20fix=20brief?= =?UTF-8?q?=20adds=20=C2=A74b=20=E2=80=94=20consult=20handoff=20must=20ref?= =?UTF-8?q?resh=20profile,=20carry=20the=20pending=20question,=20fail=20lo?= =?UTF-8?q?udly?= MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit Co-Authored-By: Claude Fable 5 Claude-Session: https://claude.ai/code/session_0155nFCgCHtoA7jhSDGmZmMu --- TASK-rectification-adopt-flow-fix-20260903.md | 64 +++++++++++++++++-- 1 file changed, 58 insertions(+), 6 deletions(-) diff --git a/TASK-rectification-adopt-flow-fix-20260903.md b/TASK-rectification-adopt-flow-fix-20260903.md index 5a525bb5..bd6e02e2 100644 --- a/TASK-rectification-adopt-flow-fix-20260903.md +++ b/TASK-rectification-adopt-flow-fix-20260903.md @@ -1,11 +1,20 @@ -# 任务书 · 采用流程修复单：采集期出口兜底、「改选」可点、开场题不占 education、核对题年份持久化（2026-09-03） +# 任务书 · 采用流程修复单：采集期出口兜底、「改选」可点、开场题不占 education、核对题年份持久化、「用这个时间看盘」真正接到看盘（2026-09-03，09-03 增补 §4b） -基线：`origin/staging` `35e5781e`（`TASK-rectification-adopt-flow-20260902.md` 的实现，BUG-497～500）。本任务书是对该实现 review 后的修复单，**不改 `35e5781e` 已定的形态**（公开 `can_adopt` 按 session_outcome 收紧、accept 409、核对题沿用持久化 questionId、卡归出卡消息、状态条承接），只修 1 个 P1 和 3 个 P2。 +基线：`origin/staging` `35e5781e`（`TASK-rectification-adopt-flow-20260902.md` 的实现，BUG-497～500）。本任务书是对该实现 review 后的修复单，**不改 `35e5781e` 已定的形态**（公开 `can_adopt` 按 session_outcome 收紧、accept 409、核对题沿用持久化 questionId、卡归出卡消息、状态条承接），只修 1 个 P1 和 4 个 P2（§4b 为 09-03 用户实测后增补）。 ## 0. Review 结论摘要 `35e5781e` 门禁全绿（tsc / lint 0 错 / 1109 测试 fail=0），两个 P0 的修法与任务书一致。但收紧公开 `can_adopt` 之后，**非终止轮出口检查还在看内部 `canAdopt`**，会把一种此前靠"漏出的采用卡"兜住的状态变成真正的死角；另外三处是形态没做完或副作用。 +**09-03 增补来源**：用户在真实环境走完一轮（采用 04:53，不可分区间约 29 分钟）后反馈"下面那个 `rectification-consult-handoff`「用这个时间看盘」按钮没用吧"。核对结论：按钮不是死的，但只做了"开一个空会话 + 往输入框塞一句服务端根本不读的话"，而且没接住采用时间同步与"回到原来的问题"两条承诺——见 §4b。用户同一份记录里其余现象**已被 `35e5781e` 覆盖、本单不重复处理**，实现者不要再改： + +| 用户看到的 | 归属 | 状态 | +| --- | --- | --- | +| 题干出现两遍（正文一遍、选项卡上方一遍） | 旧构建底部问题槽 + 消息内题干并存 | `35e5781e` 已删底部槽，嵌入卡 legend 为 `sr-only`，题干只剩消息内一处；待部署 | +| 核对题卡片底部仍有「先这样，先看当前范围」 | reverse_verify 卡应为「这题跳过」 | `35e5781e` §3.4 已做；待部署 | +| 收口报告之后又问一道 2024 年 5 月感情核对题 | 采用后最多核两件前事，是 Skill 既定流程 | 不是缺陷，不改 | +| 「用这个时间看盘」是列表末尾裸按钮 | `rectification-consult-handoff` | `35e5781e` 已移进状态条并删裸按钮；**但 onClick 的行为没改**，本单 §4b 修 | + ## 1. P1 · 采集期"无题、无卡、无出口" **位置**：`frontend/src/lib/rectification-agentic/v9/answer-choice.ts` `inspectNonTerminalTurnExit`（行号线索 ~992–1013，按符号名定位）。 @@ -64,11 +73,53 @@ 2. `keepAcceptedFocus` 的 `keepProbeYear = matchingVerify?.year || schemaYear`，删除 `promptYear` 兜底。年份仍缺时不给 `probe_year`，`validateSpokenPrompt` 的 `year_missing` 只在 `probe_year` 非空时触发（现有语义），不会误伤。 3. 测试：`rectification-question-ownership.test.ts` 加一条：采用后 set-focus 建的 reverse_verify focus，`expectedAnswerSchema.probe_year` 等于探针年份；`keepAcceptedFocus` 在 schema 无 `probe_year`、无匹配探针时 followup 不带 `probe_year`，且 `method-followup.ts` 源码不再含 `/19\\d{2}|20\\d{2}/`。 +## 4b. P2-d · 「用这个时间看盘」只是开空会话 + 塞一句废话（09-03 增补） + +**位置**：`frontend/src/hooks/use-session-management.ts` `startConsultationAfterRectification`（~315）；`frontend/src/app/page.tsx` `onStartConsultation` / `onSaved`（~1951）；`rectification-agentic-chat.tsx` 状态条 `用这个时间看盘` 按钮（~1503）；`frontend/src/hooks/use-consultation-run.ts` `resolveBirthTimeConsultationRoute(profile, …)`（~517）。 + +**现状（静态核对，不是猜）**： + +```ts +async function startConsultationAfterRectification() { + await startNewChat(); + setDraft("请用刚才采用的代表性出生时间看盘。"); + setComposerNotice("已用刚才采用的时间作为当前排盘。这还不是唯一分钟确认。"); +} +``` + +1. **那句话是废话**：普通咨询的排盘时间来自服务端 `profiles.active_birth_time`（accept RPC 已写 `active_birth_time` + `birth_time_status='accepted'`，`/api/consult` 的 `prepareConsultationRoute` 按 `verified_chart` 取 `active_birth_time`），服务端不读这句话；用户还得自己按发送，发出去的是一个没有问题的"看盘"。 +2. **静默失败**：`startNewChat()` 在 `!account || !modelCatalog || creatingSession` 或 `persistSession` 失败时返回 `null`，但草稿和提示照样设置，用户留在校正面上什么也没发生。 +3. **模式由客户端缓存决定，采用后没有等同步**：`use-consultation-run.ts` 用客户端 `profile.birthTimeStatus / profile.time` 走 `resolveBirthTimeConsultationRoute`；采用只触发 `onSaved={() => void refreshAccount()}`（不等待、失败无感）。缓存未更新时新会话会带 `consultationMode: "unverified_birth_time"`（或 `declared_birth_window` / `general_no_birth_time`）发请求；服务端资料已是 `accepted`，`resolveServerChartConsultation` 抛 `mode_changed`，首问直接报错——或更糟，走无分钟模式排了别的盘。 +4. **食言**：校正面顶部写着"先陪你核对出生时间范围，之后会回到你原来的问题：“…”"（`pendingConsultationQuestion`），但按钮不带这个问题，塞的是那句废话。 + +**修法**（不自动发送——产品拍板：没有待回问题时自动发一句空泛的"看盘"是浪费额度；有待回问题也让用户过目一眼再发）： + +1. 抽纯函数 `frontend/src/lib/rectification-consult-handoff.ts`： + ```ts + export function planRectificationConsultHandoff(input: { + savedTime: string; // 状态条上的已采用时间 + profile: { birthTimeStatus: string; time: string | null }; // refreshAccount 之后的资料 + pendingQuestion: string | null; + }): { ok: false; reason: "profile_stale" } | { ok: true; draft: string; notice: string } + ``` + `birthTimeStatus ∉ {accepted, confirmed}` 或 `profile.time !== savedTime` → `profile_stale`；否则 `draft = pendingQuestion?.trim() ?? ""`，`notice` 为「当前排盘按已采用的 04:53（代表性候选，不是唯一分钟确认）。直接输入你想问的事。」（04:53 处用 `savedTime`）。 +2. `startConsultationAfterRectification(input: { savedTime: string; pendingQuestion: string | null }): Promise`： + - 先 `await refreshAccount()`，再从刷新后的资料（用 ref 或让 `refreshAccount` 返回资料，不得读闭包里的旧 `profile`）调 `planRectificationConsultHandoff`；`profile_stale` → `setRectificationError("采用的时间还没同步到资料，请稍后再点。")`，返回 false，不建会话。 + - `const session = await startNewChat(); if (!session) { setRectificationError("新对话未能创建，请重试。"); return false; }` + - `setDraft(plan.draft)`、`setComposerNotice(plan.notice)`；删除 `"请用刚才采用的代表性出生时间看盘。"` 字面量。返回 true。 +3. `RectificationAgenticChat` 的 `onStartConsultation` 签名改为 `(savedTime: string) => void | Promise`，状态条按钮传 `savedTime`，按钮在 `busy` 或 `creatingSession`（新增 prop 或复用 `pending`）时 `disabled`；`page.tsx` 传 `(savedTime) => void startConsultationAfterRectification({ savedTime, pendingQuestion: rectificationPendingQuestion })`。`onSaved` 里的 `void refreshAccount()` 保留作预热。 +4. 不改 `resolveBirthTimeConsultationRoute` 的客户端判定，也不改服务端 `mode_changed` 语义（它是防串盘的）。本单只保证"点按钮时客户端资料已是 accepted"。 + +**测试**（并入 `tests/rectification-adopt-flow-fix-20260903.test.ts`）： +- `planRectificationConsultHandoff`：`status=accepted, time=04:53, savedTime=04:53, pendingQuestion="我明年适合换工作吗？"` → `ok`、`draft` 等于该问题；`pendingQuestion=null` → `draft=""`、`notice` 含 `04:53`；`status=rectifying` 或 `time=04:50` → `profile_stale`。 +- 源码锁：`use-session-management.ts` 不再含 `请用刚才采用的代表性出生时间看盘`；`startConsultationAfterRectification` 函数体内 `refreshAccount` 出现在 `startNewChat` 之前；`rectification-agentic-chat.tsx` 里 `用这个时间看盘` 所在 ``，点击滚到出卡消息；`showSelectionCards` 为 false 时不渲染。 3. 开场题 `target_domain = other`、id `collect:other:collect_method_evidence`；跳过开场题后 education 不进 declined、教育采集题无 `collect_retry`。 4. reverse_verify focus schema 带 `probe_year`；`method-followup.ts` 无年份正则。 -5. tsc + lint + 四组测试 fail=0；BUG_HISTORY 追加条目。 -6. 真实环境人工清单（部署后，沿用上一单 §9.7）：采集阶段把题答到没题 → 出现"先这样"入口而不是空 composer；采用后点「改选」能跳回卡片。 +5. 「用这个时间看盘」：`use-session-management.ts` 无那句废话字面量；点击先刷新资料再建会话，资料未同步时报错不建会话，`startNewChat` 返回 null 时报错；有待回问题时草稿就是该问题，没有时草稿为空、提示含已采用时间；`planRectificationConsultHandoff` 三个用例通过。 +6. tsc + lint + 四组测试 fail=0；BUG_HISTORY 追加条目（含 BUG-502）。 +7. 真实环境人工清单（部署后，沿用上一单 §9.7）：采集阶段把题答到没题 → 出现"先这样"入口而不是空 composer；采用后点「改选」能跳回卡片；从首页带问题进校正 → 采用 → 点「用这个时间看盘」→ 新会话输入框里是原问题、提示写着已采用的时间 → 发送后回答按该时间排盘（回复不出现"使用未校正填报时间"提示）。