fix: Plan 缓存校验 + CI 配置 (Issues #5, #6)

[urika]

Summary

修复 Issue #5: Plan 缓存键增加冲突处理 + Issue #6: 添加 CI 配置

Issue #5 修复

• load_cached_plan 增加 task 描述校验
• 缓存键碰撞时验证 task 描述是否完全匹配

Issue #6 修复

• 添加 .github/workflows/test.yml CI 配置文件
• 包含 pytest 测试和 ruff 检查

测试

• pytest tests/ -q (163 passed)

Checklist

• 测试通过
• task 校验已添加
• CI 配置已创建

[urika]

Code Review — PR #23: Plan 缓存校验 + CI 配置

🟡 Major

1. 将两个不相关的 Issue 打包在一个 PR 中

• Issue L3: Plan 缓存键增加冲突处理 #5: 缓存键碰撞修复（api.py）
• Issue L4: 缺少 CI 配置文件 #6: CI 配置（test.yml）

这两个修改没有技术依赖关系。打包在一起导致：

• Review 更困难
• 回滚不精确（如果 CI 需要回滚，缓存修复也会跟着）
• 与 PR fix: Plan 缓存键增加 task 描述校验 (Issue #5) #22（同样修复 Issue L3: Plan 缓存键增加冲突处理 #5）和 PR feat: 添加 ruff + mypy CI 检查 (Issue #13) #24（同样添加 CI）产生重叠冲突

建议: 拆分为两个独立 PR。

2. 与 PR #22 的 api.py 修改完全重复

PR #22 和 PR #23 对 agent_go/api.py 的修改字节级一致（添加 task 参数 + cached_task != task[:200] 校验）。合并其中一个会导致另一个冲突。应关闭其中一个。

3. 与 PR #24 的 .github/workflows/test.yml 冲突

两个 PR 都创建了同名文件但内容不同：

• PR fix: Plan 缓存校验 + CI 配置 (Issues #5, #6) #23: name: Test，单独安装 ruff，无 mypy
• PR feat: 添加 ruff + mypy CI 检查 (Issue #13) #24: name: Test & Lint，统一安装依赖，包含 mypy

4. CI 中 ruff 使用 || true，lint 失败不会阻塞 CI

与 PR #24 相同的问题。如果目的是渐进式引入，应添加注释和跟踪 issue。

🟢 Minor

5. ruff 在 lint 步骤中单独 pip install，而非在依赖安装步骤中统一安装
6. cached_task != task[:200] 截断比较是安全的（因为 save_cached_plan 也存储 task[:200]），但两个前 200 字符相同的长任务理论上会绕过校验。作为深度防御措施可以接受。

---

总结: 建议拆分为两个独立 PR，关闭与 PR #22 重复的部分，CI 部分与 PR #24 协调后合并。

[urika]

🔒 关闭本 PR — 与 PR #22 和 #24 重复

本 PR (#23) 打包了两个不相关的 Issue，分别与已有 PR 重复：

1. Issue L3: Plan 缓存键增加冲突处理 #5 (缓存校验) → 与 PR fix: Plan 缓存键增加 task 描述校验 (Issue #5) #22 完全重复，保留 PR fix: Plan 缓存键增加 task 描述校验 (Issue #5) #22

   • PR fix: Plan 缓存键增加 task 描述校验 (Issue #5) #22 更聚焦于单一 Issue，符合一个 PR 解决一个问题的原则
   • 两者对 api.py 的修改字节级一致

2. Issue L4: 缺少 CI 配置文件 #6 (CI 配置) → 被 PR feat: 添加 ruff + mypy CI 检查 (Issue #13) #24 取代

   • PR feat: 添加 ruff + mypy CI 检查 (Issue #13) #24 的 CI 更完整（包含 ruff + mypy）
   • 两者都创建 .github/workflows/test.yml，必然冲突

保留的 PR：

• Issue L3: Plan 缓存键增加冲突处理 #5 → PR fix: Plan 缓存键增加 task 描述校验 (Issue #5) #22 (fix/issue-5-cache-key)
• Issue L4: 缺少 CI 配置文件 #6 → PR feat: 添加 ruff + mypy CI 检查 (Issue #13) #24 (fix/issue-13-ruff-mypy)

本 PR 关闭，各 Issue 的修复在对应 PR 上继续。

[urika]

关闭：与 PR #22 (Issue #5) 和 PR #24 (Issue #6) 重复，分别保留对应 PR 继续修复。