chore(chat): 7 个 e2e spec 仍用 ?projectId= 测试非 chat 本身的功能,需要迁移 fixture
- Dominant language
- TypeScript
- Stars
- 0
- Forks
- 0
- Avg merge
- 1h 7m
- Merged PRs (30d)
- 969
Description
源于 #2457(DA-19h 旧轨道退役)。
以下 7 个 e2e spec 用 \`?projectId=\` 深链作为测试入口,但它们实际测的是*其他*功能(不是 chat 本身),只是顺手用了项目态线程当固定装置:
- \`e2e/chat-agent-skill-context.spec.ts\`(skill 上下文)
- \`e2e/chat-canvas-guidance-render.spec.ts\`(canvas 引导渲染)
- \`e2e/chat-diagram-save-reopen-roundtrip.spec.ts\`(diagram 保存/重开)
- \`e2e/core-loop.spec.ts\`(核心冒烟)
- \`e2e/feedback-loop-smoke.spec.ts\`
- \`e2e/chat-task-workbench-copy.spec.ts\`
- \`e2e/chat-task-workbench-p1-efficiency.spec.ts\`
项目内对话已经彻底停用(DA-19h,#2457),这些 spec 现在导航到的 URL 落到 v2 的 \`/chat/:threadId\`(因为都带 \`thread=\`),但 fixture(\`CHAT_READ_E2E\`/\`FULLSTACK_E2E\`)provision 的是**项目态**线程——需要确认:
1. v2 侧能不能正确处理一个"项目态线程"的 id(`getThread` 会不会因为 `projectId !== null` 拒绝/表现异常)?
2. 如果不能,`apps/api/scripts/seed-chat-read-e2e.ts` 需不需要新增一个**个人态**线程 fixture,这 7 个 spec 改用它?
本仓沙箱没有 docker,跑不了真栈 e2e,这条调查加修复需要在有 docker 的环境(或 CI)里做,不能盲改——改错了会把"看起来测了"变成"其实没测到"。
🤖 Generated with [Claude Code](https://claude.com/claude-code)
Contributor guide
No contributing guide indexed for this repository
Research direction
Start with the seven listed specs and apps/api/scripts/seed-chat-read-e2e.ts, then inspect how v2 handles the supplied thread IDs and whether project-state threads are accepted. Run the affected e2e coverage in Docker or CI, determine whether a personal-state fixture is needed, and finish by migrating the specs to the correct fixture with passing tests.
Written by the indexing model from the issue text.
Assessment
- Tech stack
- docker, typescript
- Domain
- testing
- Issue type
- Refactor
- Difficulty
- 4/5
- Estimated time
- 3-5 days
- Activity status
- Active
- Clarity
- Mostly clear
- Newbie friendliness
- 55/100