feat: 添加 ruff + mypy CI 检查 (Issue #13)

[urika]

Summary

修复 Issue #13: 添加 ruff + mypy 自动化代码质量检查

修复

• 增强 .github/workflows/test.yml CI 配置
• 添加 ruff lint 检查 (E, F 规则)
• 添加 mypy 类型检查

测试

• pytest tests/ -q (163 passed)

Checklist

• 测试通过
• ruff 检查已添加
• mypy 检查已添加

[urika]

Code Review — PR #24: 添加 ruff + mypy CI 检查

🔴 Critical

1. ruff 和 mypy 都使用 || true，lint/type 检查永远不会使 CI 失败

- name: Ruff lint
  run: |
    ruff check agent_go/ --select=E,F --ignore=E501 --fix || true

- name: MyPy type check
  run: |
    mypy agent_go/ --ignore-missing-imports || true

即使存在数百个 lint 违规或类型错误，CI 都会显示为"绿色"，完全违背了 CI 门禁的目的。如果是为了渐进式采用，应添加注释说明，并创建一个跟踪 issue 后续移除 || true。

2. ruff 步骤的 --fix 标志在 CI 中不恰当

--fix 会原地修改文件，但 CI 中修改会被丢弃，永远不提交。结合 || true，会静默"修复"问题但退出码为 0，造成代码干净的假象。正确命令应为 ruff check agent_go/ --select=E,F --ignore=E501（不带 --fix，不带 || true）。

🟡 Major

3. 与 PR #23 在 .github/workflows/test.yml 上存在合并冲突

两个 PR 都创建了同名文件但内容不同。需要协调合并顺序。

4. 缺少 pyproject.toml

Issue #13 明确提到 "添加 pyproject.toml (ruff + mypy 配置)"。没有它，本地开发者无法复现 CI 检查。

5. 不必要的 types-requests 依赖

项目完全使用 urllib.request（stdlib），不使用 requests 库。安装 types-requests 浪费且有误导性。

6. __all__ 列出了 15 个私有名称（_ 前缀），违背约定

_slugify、_format_commit、_worktree_create 等以 _ 开头的符号按约定是私有的，不应出现在 __all__ 中。要么重命名使其变为公共的，要么从 __all__ 中移除。

🟢 Minor

7. __version__ 未包含在 __all__ 中，但它是公共 API
8. ruff 规则选择器 --select=E,F 过于狭窄，建议至少加上 W (warnings)
9. ruff 和 mypy 在不同步骤中运行，任一失败会阻止后续步骤执行，建议用独立 job 或 continue-on-error

---

总结: 需修复 Critical #1-2（移除 || true 和 --fix）和 Major #4-6 后方可合并。

[urika]

📝 重复 PR 说明: PR #23 (fix/issue-6-ci-config) 已关闭，其 CI 配置部分由本 PR (#24) 继续负责。PR #23 中的 Issue #5 (缓存校验) 部分由 PR #22 独立修复。

[urika]

本 PR 的所有 Review 反馈已在 PR #29 中修正。建议关闭本 PR，改用 #29 合并。

[urika]

已被 PR #29 替代，#29 包含所有 review 反馈的修正。关闭本 PR。

[urika]

🔒 关闭本 PR — 已被 PR #29 取代

PR #29 (fix/issue-13-ci-quality) 是本 PR 的修复版，解决了以下 review 问题：

1. 移除 || true，让 lint/type 检查真正阻塞 CI
2. 移除 --fix 标志
3. 添加 pyproject.toml 配置文件
4. 修正 __all__ 私有符号问题

本 PR 关闭，后续以 PR #29 为准。