docs(tasks): staging fixed-OTP login skips set-password (BUG-1152)
Co-Authored-By: Claude Opus 5.5 (1M context) <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01N4f2nya58RoRu4yEmJgRGE
This commit is contained in:
Jesse_Chen
co-authored by
Claude Opus 5.5
Claude Opus 5.5
parent
b62308c4ad
commit
d58df1b48b
@@ -402,3 +402,4 @@
|
||||
| `TASK-consult-plain-answer-20261001.md` | `PROGRESS-consult-plain-answer-20261001.md` | 普通对话「还是废话」：09-17 四步开场形状（格局名 → 谁推谁修 → 扮演哪个象）逼出谜语，问父母时爸妈被揉成一段；产品授权推翻该形状，改成先答 + 按问题里的对象分段 + 人话自检 + 空宫不单独下结论 + 父母卡标 mother/father（BUG-1132～1134） | **已实现，待 Claude 验收**（fork 子代理直接执行）；未推 staging、未部署；模型对比为环境缺口，真机清单 `docs/testing/consult-plain-answer-20261001.md` | 分支 `codex/consult-plain-answer-20261001` |
|
||||
| `TASK-consult-answer-clock-20261001.md` | `PROGRESS-consult-answer-clock-20261001.md` | 普通对话答题时钟 70 s 容不下推理模型（v4-pro 77 s），无出生分钟路线只受 110 s 工具钟管；产品定 5 分钟防卡死（BUG-1142） | **已实现，待 Claude 验收**（直接执行） | 分支 `codex/consult-answer-clock-20261001` |
|
||||
| `TASK-consult-timeline-no-plan-20261001.md` | `PROGRESS-consult-timeline-no-plan-20261001.md` | 普通对话进度栏提前打勾：回答提纲在算盘前就作为四行步骤显示并全部打勾（BUG-1145）；产品选 B 去掉提纲行，只显示真实步骤 | **实现完成，待 Claude 验收**（直接执行） | 分支 `codex/consult-timeline-no-plan-20261001` |
|
||||
| `TASK-staging-otp-skip-password-20261001.md` | `PROGRESS-staging-otp-skip-password-20261001.md` | staging 新邮箱用固定验证码登录后被强制设置密码，测试者误以为要密码登录；产品选「测试通道验证码登录跳过设置密码」，生产不变（BUG-1152） | **待领取** | 分支 `codex/staging-otp-skip-password-20261001` |
|
||||
|
||||
@@ -0,0 +1,84 @@
|
||||
# TASK · staging 固定验证码登录跳过「设置密码」（2026-10-01）
|
||||
|
||||
## 基线
|
||||
|
||||
- `origin/staging` = `b62308c4`（开工时重新 `git fetch origin --prune` 核对）
|
||||
- 分支 / worktree：`codex/staging-otp-skip-password-20261001` / `.worktrees/staging-otp-skip-password-20261001`
|
||||
- 只改前端登录组件与测试；不动服务端、数据库、workflow、环境变量
|
||||
|
||||
## 事故实证
|
||||
|
||||
产品负责人反馈：「staging 现在要密码才能登录，不是用固定验证码吗？」
|
||||
|
||||
| 事实 | 证据 |
|
||||
| --- | --- |
|
||||
| staging 固定码通道仍开启 | 线上 `/login` 页 RSC 负载带 `testOtp`（6 位码，值不写进文档）；对虚构邮箱调用 `POST /api/auth/email-otp/send-verification-otp` 返回 `200 {"success":true}` |
|
||||
| 固定码提示从常驻变为 10 秒浮层 | `3e0c615f`（09-30，BUG-1121/1122）删除登录卡片上的 `auth-test-channel` 段落，改成点「发送验证码」后才出现的 `notifyInfo(..., TEST_OTP_NOTICE_MS)`。**本单不恢复常驻提示**（产品只选了第 2 条） |
|
||||
| 新邮箱用验证码登录后被强制设置密码 | `frontend/src/components/email-otp-login.tsx` 的 `verifyOtp`：`selfHostedAuthActions.hasPassword()` 为 false 时 `setStep("set-password")`，该表单没有跳过入口。自 `ab2944f7`（07-27）起如此 |
|
||||
|
||||
## 根因
|
||||
|
||||
`verifyOtp` 不区分测试通道：staging 测试者每用一个新邮箱，都会在验证码通过后被拦在「设置登录密码」。验证码校验成功时服务端会话已经建立（`set-password` 只是登录后的界面步骤，`enterApp()` 只是跳转），所以这一步在测试通道里纯属阻碍。
|
||||
|
||||
## 决策记录
|
||||
|
||||
- 产品负责人 2026-10-01 选定：**staging 用固定验证码登录新邮箱时，跳过「设置密码」直接进入应用。**
|
||||
- 范围只限 `testMode`（即服务端配置了 `IDENTITY_TEST_OTP`，生产不配置，`testOtp` 为 null）。生产行为一字不变。
|
||||
- 只对「验证码登录」模式（`mode === "otp"`）生效。「注册账号」模式的说明就是「验证邮箱后设置密码」，测试者主动选它时仍走设置密码步骤。
|
||||
- 不恢复登录卡片上的常驻固定码提示（产品未选第 1 条）；BUG-1121/1122 的浮层设计不变。
|
||||
|
||||
## 硬红线
|
||||
|
||||
1. 不改服务端鉴权、Better Auth 配置、`IDENTITY_TEST_OTP` 读取逻辑、`/api/account/password`。
|
||||
2. 非测试通道（`testMode` 为 false）下 `verifyOtp` 的分支顺序与行为完全不变：二步验证 → 无密码进设置密码 → 注册模式已有账户提示 → 进入应用。
|
||||
3. 测试通道下二步验证分支照常优先（有 MFA 的账号仍要第二因素）。
|
||||
4. 不新增第二套登录表单、不新增加载动画；`page.tsx` 不增长。
|
||||
5. 不在任何文件写入真实固定码以外的账号信息；文档里不写固定码的值。
|
||||
|
||||
## 任务分解
|
||||
|
||||
### T1 跳过设置密码（BUG-1152）
|
||||
|
||||
`verifyOtp` 自托管分支内，`hasPassword` 为 false 时：若 `testMode && mode === "otp"` → 直接 `await enterApp()`；否则维持 `setStep("set-password")`。可以把 `hasPassword()` 调用在这种情况下省掉，也可以保留调用后判断，执行方择一并在进度记录里说明。
|
||||
|
||||
验收：
|
||||
- 测试通道 + 验证码登录 + 无密码账户 → 不进入 `set-password`，调用 `enterApp`（会记录协议同意并跳转 `/`）。
|
||||
- 测试通道 + 注册模式 + 无密码 → 仍进入 `set-password`。
|
||||
- 非测试通道 + 验证码登录 + 无密码 → 仍进入 `set-password`。
|
||||
- 测试通道 + 需要二步验证 → 仍进入 `two-factor`。
|
||||
|
||||
### T2 回归测试
|
||||
|
||||
在 `frontend/tests/identity-login-provider.test.ts` 补断言覆盖 T1 的四种情形（该文件现有风格是对组件源码做正则断言；能做行为级渲染测试更好，二选一并说明）。既有断言不改；若必须改，写「原值 / 新值 / 原因」。
|
||||
|
||||
### T3 记录
|
||||
|
||||
- `docs/BUG_HISTORY.md` 新增 BUG-1152：现象「staging 新邮箱用固定验证码登录后被要求设置密码，测试者误以为必须用密码登录」，根因见上，关联 BUG-1121/1122（提示改为浮层使问题更显眼）。
|
||||
- `CHANGELOG.md` 一行：staging 测试通道验证码登录不再要求设置密码（生产不受影响），Skill 版本不 bump。
|
||||
- `docs/testing/staging-otp-skip-password-20261001.md` 真机清单：新邮箱 → 发送验证码 → 输入浮层里的固定码 → 直接进入首页；再用「注册账号」走一遍确认仍要求设置密码。
|
||||
- 本文件在 `docs/tasks/README.md` 状态板登记，合入 staging 的同一次推送里改状态。
|
||||
|
||||
## 让步顺序
|
||||
|
||||
T1 > T2 > T3。行为级渲染测试做不起来时退回源码正则断言，不得省略四种情形。
|
||||
|
||||
## 开工前置命令
|
||||
|
||||
```bash
|
||||
cd /workspace/Jyotisha && git status -sb | head -1
|
||||
git fetch origin --prune
|
||||
git worktree add -b codex/staging-otp-skip-password-20261001 .worktrees/staging-otp-skip-password-20261001 origin/staging
|
||||
cd .worktrees/staging-otp-skip-password-20261001/frontend && npm ci
|
||||
./node_modules/.bin/tsc --noEmit && npm run lint && node --test tests/identity-login-provider.test.ts # 记录基线
|
||||
grep -oE "^## BUG-[0-9]+" ../docs/BUG_HISTORY.md | sort -t- -k2 -n | tail -1 # 应为 BUG-1151；若已被占用顺延
|
||||
```
|
||||
|
||||
## 验收口径
|
||||
|
||||
- `tsc --noEmit` 0 错；`npm run lint` 0 error；`identity-*` 测试全过，前端测试总数不低于基线。
|
||||
- `next build` 后 `/` 仍 Static；首屏 gzip ±2%（登录页不在首屏，预期不变）。
|
||||
- 部署后 staging `/api/health` 的 `deployment.gitCommit` 等于实现提交；真机按 `docs/testing/` 清单由产品负责人走查（Claude 无登录态时记为环境缺口）。
|
||||
|
||||
## BUG 编号起点
|
||||
|
||||
BUG-1152（开工时核对 `docs/BUG_HISTORY.md` 最大号）。
|
||||
Reference in New Issue
Block a user