9.8 KiB
9.8 KiB
| phase | plan | type | wave | depends_on | gap_closure | files_modified | autonomous | requirements | must_haves | ||||||||||||||||||||||||||||||||||||||||||||||||||||||||
|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|
| 06- | 05 | execute | 4 |
|
true |
|
true |
|
|
Purpose: 收口 ATREF-02 与 ATREF-04，避免 Phase 06 继续停留在“代码可用但合同与回归不完整”的状态。
Output: 一次新的 gap-closure 执行会产出对齐后的输入框展示合同，以及不再永久 skip/不再 strict-locator flaky 的 E2E 回归。
<execution_context> @/home/mt/.codex/get-shit-done/workflows/execute-plan.md @/home/mt/.codex/get-shit-done/templates/summary.md </execution_context>
@.planning/STATE.md @.planning/ROADMAP.md @.planning/REQUIREMENTS.md @.planning/phases/06-/06-CONTEXT.md @.planning/phases/06-/06-RESEARCH.md @.planning/phases/06-/06-VERIFICATION.md @.planning/phases/06-/06-UAT.md @.planning/phases/06-/06-UI-SPEC.md @.planning/phases/06-/06-04-SUMMARY.md @frontend/src/components/workspace/input-box.tsx @frontend/tests/e2e/input-and-compose.spec.ts @frontend/tests/e2e/support/chat-helpers.ts From frontend/src/components/workspace/input-box.tsx: ```typescript const MAX_REFERENCES_PER_MESSAGE = 10;type MentionCandidate = { key: string; filename: string; path?: string; pathTail: string; ref_source: "artifact" | "upload"; ref_kind: "mention"; };
From frontend/tests/e2e/input-and-compose.spec.ts:
```typescript
test("DF-INPUT-008 失效引用不会阻断文本发送（可解释 skip）", async (...) => {});
test("DF-INPUT-009 引用上限为 10，第 11 个被阻止并提示", async (...) => {});
` 候选层，不允许继续保留自定义浮层作为最终实现。把 `MAX_REFERENCES_PER_MESSAGE`、所有用户可见文案和任何与上限绑定的辅助文案统一恢复为 10；扩展候选/已选引用的展示模型，明确渲染“文件名 + 类型 + 路径尾段”，其中“类型”必须是用户可见的独立维度，而不是仅靠 `ref_source` 或文件扩展隐含表达。若需要为 E2E 提供稳定定位，优先补充明确的 `data-testid`、`aria-label` 或可预测文案，避免依赖模糊文本匹配；不要重新引入 `Tag` 组件、不要把去歧义信息回退成纯路径尾段，也不要用新的自定义 `
` 候选层绕过 D-09。
- `input-box.tsx` 中引用上限常量和提示文案全部为 10，没有残留“6 个”。
- `@` 候选面板恢复为 `DropdownMenu` / `DropdownMenuContent` / `DropdownMenuItem` 渲染链路，不再使用自定义 `
` 候选层，满足 D-09。
- dropdown 候选与 inline preview 都能在同名场景表达“文件名 + 类型 + 路径尾段”，满足 ATREF-02。
- 提供给 E2E 使用的可定位语义是唯一且稳定的，不依赖 strict text locator 猜测。
cd frontend && rg -n "DropdownMenu(Content|Item)?|from ['\\\"]@/components/ui/dropdown-menu['\\\"]" src/components/workspace/input-box.tsx
cd frontend && rg -n "MAX_REFERENCES_PER_MESSAGE\\s*=\\s*10|单条消息最多引用 10 个文件|最多引用 10 个" src/components/workspace/input-box.tsx tests/e2e/input-and-compose.spec.ts
cd frontend && pnpm -s test:e2e --grep "DF-INPUT-007|DF-INPUT-009"
`input-box.tsx` 明确恢复为基于 `DropdownMenu*` 的候选面板实现，ATREF-02 的代码合同、可见文案和回归断言前提全部回到 requirement=10，且“类型”展示缺口被消除。
Task 2: 移除永久 skip 并稳定化 DF-INPUT-008/009 回归
frontend/tests/e2e/input-and-compose.spec.ts, frontend/tests/e2e/support/chat-helpers.ts
- .planning/phases/06-/06-VERIFICATION.md
- .planning/phases/06-/06-UAT.md
- .planning/phases/06-/06-04-SUMMARY.md
- frontend/tests/e2e/support/chat-helpers.ts
- frontend/tests/e2e/input-and-compose.spec.ts
直接关闭 verification gap 2。删除 DF-INPUT-008 中无条件 `testInfo.skip(true)`，改成可执行的稳定场景：优先通过 Playwright route/fixture 注入或专用 thread helper 制造“已选 artifact 引用在发送前 materialize 失败”的条件，验证错误 toast 出现且文本消息仍发送成功；只有在必需的 thread/env 完全缺失时才允许条件化跳过，并把 gate 收敛到 helper，不允许在测试体内永久 skip。同步重写 DF-INPUT-009：基于 helper 提供的可重复候选集或稳定 thread，覆盖 10 个成功 + 第 11 个被阻止的路径，并把当前容易多命中的 `getByText(...)` 断言替换为带作用域的 toast locator、唯一 data-testid 或明确 aria 语义。保持用例命名、编号和 Phase 06 回归范围不变，不新增与本 gap 无关的 UI 行为。
- DF-INPUT-008 不再包含永久 skip，且能验证 stale toast + 消息继续发送。
- DF-INPUT-009 明确断言“最多 10 个”，第 11 次添加失败，并且不再触发 strict locator 多命中。
- 对 thread/env 的依赖被集中到 helper 或 route stub，测试结果不再依赖偶然的线程候选数量。
cd frontend && pnpm -s test:e2e --grep "DF-INPUT-008|DF-INPUT-009"
ATREF-04 的 E2E 回归护栏稳定可运行，verification 中关于永久 skip 和 strict locator 的两条缺口都能被关闭。
- `cd frontend && node --test src/core/threads/hooks.test.ts`
- `cd frontend && pnpm -s typecheck`
- `cd frontend && pnpm -s test:e2e --grep "DF-INPUT-007|DF-INPUT-008|DF-INPUT-009"`
<threat_model>
Trust Boundaries
| Boundary | Description |
|---|---|
| 输入框 UI → 引用候选展示 | 非可信的文件名/路径元数据进入用户可见去歧义文案，容易因展示降级导致误选。 |
| Playwright fixture/route → 回归结论 | 测试数据与真实 UI 交互之间若没有稳定约束，会产生假阳性或 flaky 回归结果。 |
STRIDE Threat Register
| Threat ID | Category | Component | Disposition | Mitigation Plan |
|---|---|---|---|---|
| T-06-05-01 | T | frontend/src/components/workspace/input-box.tsx |
mitigate | 明确把类型、路径尾段和上限 10 写成单一展示合同；不要让 ref_source 或纯路径尾段承担全部去歧义语义。 |
| T-06-05-02 | D | frontend/tests/e2e/input-and-compose.spec.ts |
mitigate | 用 helper 或 route stub 固定 stale/上限场景，并使用唯一 locator 或 toast 作用域断言，消除 strict 模式多命中导致的回归失真。 |
| </threat_model> |
<success_criteria>
- ATREF-02：实现、用户文案、E2E 断言全部统一到“上限 10 + 文件名/类型/路径尾段”。
- ATREF-04：DF-INPUT-008 不再永久 skip，DF-INPUT-009 不再因 strict locator 多命中失败。
- 新计划只追加 gap-closure 修复，不回退
06-04-SUMMARY.md已记录的 artifact materialization 与软失败主链路。 </success_criteria>