docs(06-05): finalize verification gap summary
- record task commits and final regression status - document stale-toast contract fix and test hardening
This commit is contained in:
parent
4739e81e83
commit
0b7b315b2e
|
|
@ -8,9 +8,10 @@ requires:
|
|||
provides: Phase 06 回归基线与验证缺口
|
||||
provides:
|
||||
- 引用上限与去歧义展示合同对齐 requirement 10
|
||||
- DF-INPUT-008 稳定验证 stale toast 与提交不中断
|
||||
- DF-INPUT-009 回归场景稳定化(10 个成功 + 第 11 个阻止)
|
||||
- toast/候选面板定位 helper 去 flaky 化
|
||||
affects: [06-UAT, input-box, e2e, mention-picker]
|
||||
affects: [06-UAT, input-box, e2e, mention-picker, thread-submit]
|
||||
tech-stack:
|
||||
added: []
|
||||
patterns: [stable-e2e-locators, deterministic-toast-assertion, retry-open-picker]
|
||||
|
|
@ -18,52 +19,58 @@ key-files:
|
|||
created:
|
||||
- .planning/phases/06-/06-05-SUMMARY.md
|
||||
modified:
|
||||
- frontend/src/components/workspace/input-box.tsx
|
||||
- frontend/src/core/threads/hooks.ts
|
||||
- frontend/tests/e2e/input-and-compose.spec.ts
|
||||
- frontend/tests/e2e/support/chat-helpers.ts
|
||||
key-decisions:
|
||||
- "DF-INPUT-009 采用固定 fixture key + 明确 data-testid 断言,避免 strict text locator 多命中。"
|
||||
- "toast 断言统一 first(),消除并发提示叠加导致的 strict mode 抖动。"
|
||||
- "DF-INPUT-008 改为验证 stale toast + runs/stream 提交请求 + 输入框清空,避免依赖聊天区回显时序。"
|
||||
patterns-established:
|
||||
- "openReferencePicker 增加重试与回退(Backspace)机制,兼容 Dropdown 动画/重排时序。"
|
||||
- "引用上限回归按 1..10 逐步计数断言,再验证第 11 次被阻止。"
|
||||
requirements-completed: [ATREF-01, ATREF-02, ATREF-03, ATREF-04]
|
||||
duration: 12min
|
||||
duration: 24min
|
||||
completed: 2026-04-15
|
||||
---
|
||||
|
||||
# Phase 06 Plan 05: Verification Gaps Closure Summary
|
||||
|
||||
**Phase 06 最后一个 gap-closure 计划已收口:引用上限 10 的回归路径稳定可重复,DF-INPUT-009 不再因候选层/toast 定位抖动而失败。**
|
||||
**Phase 06 最后一个 gap-closure 计划已收口:输入框引用合同重新对齐 requirement=10,DF-INPUT-008/009 都已变成可重复运行的稳定回归。**
|
||||
|
||||
## Performance
|
||||
|
||||
- **Duration:** 12 min
|
||||
- **Duration:** 24 min
|
||||
- **Started:** 2026-04-15T05:06:00Z
|
||||
- **Completed:** 2026-04-15T05:18:04Z
|
||||
- **Completed:** 2026-04-15T06:02:00Z
|
||||
- **Tasks:** 2
|
||||
- **Files modified:** 2
|
||||
- **Files modified:** 4
|
||||
|
||||
## Accomplishments
|
||||
|
||||
- 验证并确认 `input-box.tsx` 已满足上限 `10`、`DropdownMenu*` 链路与“文件名 + 类型 + 路径尾段”展示合同。
|
||||
- 稳定化 DF-INPUT-009:固定候选 key 选择路径、逐步数量断言、11th 阻止与 toast 断言。
|
||||
- 强化 E2E helper:候选面板开启重试、toast 定位去严格模式冲突,提升回归确定性。
|
||||
- `input-box.tsx` 对齐到 requirement `10`,候选层恢复 `DropdownMenu*`,chip 与候选都显式展示“文件名 + 类型 + 路径尾段”。
|
||||
- `hooks.ts` 的 stale toast 文案恢复到 phase 合同值“部分引用文件已失效,已自动移除并继续发送。”。
|
||||
- DF-INPUT-008 通过 helper + route stub 稳定制造 stale artifact 场景,验证 toast 出现且提交流程继续。
|
||||
- DF-INPUT-009 使用固定 fixture key、唯一 locator 与串行执行,稳定覆盖“10 个成功 + 第 11 个阻止”。
|
||||
|
||||
## Task Commits
|
||||
|
||||
1. **Task 1: 对齐引用展示合同与上限 10** - 复核通过(本轮未新增代码改动)
|
||||
2. **Task 2: 移除永久 skip 并稳定化 DF-INPUT-008/009 回归** - 本轮改动在工作区完成(待与当前分支既有改动统一提交)
|
||||
1. **Task 1: 对齐引用展示合同与上限 10** - `16dca210` (feat)
|
||||
2. **Task 2: 移除永久 skip 并稳定化 DF-INPUT-008/009 回归** - `88be05ad` (test)
|
||||
3. **Rule 1 补丁: 收紧 stale-send 回归断言并消除共享线程抖动** - `a91c3c9e` (test)
|
||||
|
||||
## Files Created/Modified
|
||||
|
||||
- `.planning/phases/06-/06-05-SUMMARY.md` - 记录 plan 05 执行与验证结果。
|
||||
- `.planning/phases/06-/06-05-SUMMARY.md` - 记录 plan 05 执行、补丁与最终验证结果。
|
||||
- `frontend/src/components/workspace/input-box.tsx` - 恢复 `DropdownMenu` 候选链路、10 条上限合同、类型去歧义与稳定测试语义。
|
||||
- `frontend/src/core/threads/hooks.ts` - stale toast 文案与软失败合同对齐。
|
||||
- `frontend/tests/e2e/input-and-compose.spec.ts` - DF-INPUT-009 改为稳定 key 驱动的候选选择与计数断言。
|
||||
- `frontend/tests/e2e/support/chat-helpers.ts` - `openReferencePicker` 增加重试;`toastByText` 统一 `.first()`。
|
||||
- `frontend/tests/e2e/support/chat-helpers.ts` - `openReferencePicker` 增加重试;新增 fixture/stale helper,`toastByText` 统一 `.first()`。
|
||||
|
||||
## Decisions Made
|
||||
|
||||
- 不回退 Phase 06 既有主链路实现,仅针对 verification gaps 做最小、可回归的测试稳定性修复。
|
||||
- 保留 DF-INPUT-008 的条件 skip 语义(仅在必需 thread/env 缺失时跳过),移除永久 skip 风险。
|
||||
- 不回退 Phase 06 既有 artifact materialization 主链路,只在合同缺口与回归稳定性上追加最小修复。
|
||||
- DF-INPUT-008 不再保留永久 skip;仅在线程环境完全缺失时才允许 helper 层 gate。
|
||||
|
||||
## Deviations from Plan
|
||||
|
||||
|
|
@ -72,19 +79,27 @@ completed: 2026-04-15
|
|||
**1. [Rule 1 - Bug] DF-INPUT-009 候选点击在 Dropdown 动画期不稳定**
|
||||
- **Found during:** Task 2 验证
|
||||
- **Issue:** 候选项在可见但重排中导致 click actionability 抖动,产生超时。
|
||||
- **Fix:** helper 增加开启重试;测试改用稳定 key + DOM click + 数量递增断言。
|
||||
- **Fix:** helper 增加开启重试;测试改用稳定 key + DOM click + 串行执行与数量递增断言。
|
||||
- **Files modified:** `frontend/tests/e2e/input-and-compose.spec.ts`, `frontend/tests/e2e/support/chat-helpers.ts`
|
||||
- **Verification:** `pnpm -s test:e2e --grep "DF-INPUT-007|DF-INPUT-008|DF-INPUT-009"`(007/009 pass,008 conditional skip)
|
||||
- **Committed in:** N/A(当前分支存在并行未提交改动,待统一提交)
|
||||
- **Verification:** `pnpm -s test:e2e --grep "DF-INPUT-007|DF-INPUT-008|DF-INPUT-009"` 通过
|
||||
- **Committed in:** `88be05ad`, `a91c3c9e`
|
||||
|
||||
**2. [Rule 2 - Contract] stale toast 文案与 UI-SPEC 不一致**
|
||||
- **Found during:** Task 2 验证
|
||||
- **Issue:** 软失败主链路仍提示“部分引用已失效,已自动移除”,未对齐 phase 约定文案。
|
||||
- **Fix:** `frontend/src/core/threads/hooks.ts` 两条 submit 链路统一改为“部分引用文件已失效,已自动移除并继续发送。”。
|
||||
- **Files modified:** `frontend/src/core/threads/hooks.ts`
|
||||
- **Verification:** `pnpm -s test:e2e --grep "DF-INPUT-008"` 通过
|
||||
- **Committed in:** `16dca210`
|
||||
|
||||
---
|
||||
|
||||
**Total deviations:** 1 auto-fixed (Rule 1: bug)
|
||||
**Impact on plan:** 修复仅针对回归稳定性,无范围膨胀,不影响已确认功能链路。
|
||||
**Total deviations:** 2 auto-fixed (Rule 1: bug, Rule 2: contract)
|
||||
**Impact on plan:** 修复仅针对回归稳定性与合同文案,无范围膨胀,不影响已确认功能链路。
|
||||
|
||||
## Issues Encountered
|
||||
|
||||
- 并发 toast 与候选层动画导致 strict locator/actionability 偶发失败;已通过 helper 与断言收敛消除。
|
||||
- 共享线程 fixture 在并发 worker 下会互相污染;已将本文件串行化并改为固定 fixture stub。
|
||||
|
||||
## User Setup Required
|
||||
|
||||
|
|
@ -93,11 +108,11 @@ None - no external service configuration required.
|
|||
## Next Phase Readiness
|
||||
|
||||
- `06-05` summary 已补齐,phase completeness 可推进到 phase-level verification。
|
||||
- ATREF-04 对应的自动化护栏已可回归运行(008 条件化、009 稳定通过)。
|
||||
- ATREF-04 对应的自动化护栏已可回归运行(007/008/009 全部通过)。
|
||||
|
||||
## Self-Check: PASSED
|
||||
|
||||
- FOUND: `.planning/phases/06-/06-05-SUMMARY.md`
|
||||
- VERIFIED: `node --test src/core/threads/hooks.test.ts` 通过
|
||||
- VERIFIED: `pnpm -s typecheck` 通过
|
||||
- VERIFIED: `pnpm -s test:e2e --grep "DF-INPUT-007|DF-INPUT-008|DF-INPUT-009"` → 007/009 pass, 008 conditional skip
|
||||
- VERIFIED: `pnpm -s test:e2e --grep "DF-INPUT-007|DF-INPUT-008|DF-INPUT-009"` → 007/008/009 全通过
|
||||
|
|
|
|||
Loading…
Reference in New Issue