Skip to content

fix: use GET for AgentRouter check-in#104

Open
ppch53 wants to merge 1 commit into
millylee:mainfrom
ppch53:fix-agentrouter-get-checkin
Open

fix: use GET for AgentRouter check-in#104
ppch53 wants to merge 1 commit into
millylee:mainfrom
ppch53:fix-agentrouter-get-checkin

Conversation

@ppch53
Copy link
Copy Markdown

@ppch53 ppch53 commented May 10, 2026

📋 PR 描述

🎯 改动类型

  • 🐛 Bug 修复(不影响现有功能的缺陷修复)
  • ✨ 新功能(添加新的功能特性)
  • 💥 破坏性变更(可能影响现有用户的改动)
  • 📝 文档更新(仅文档改动)
  • 🎨 代码格式/风格调整(不影响功能的格式化)
  • ♻️ 重构(既不修复 bug 也不添加功能的代码改动)
  • ⚡ 性能优化(提升性能的代码改动)
  • ✅ 测试相关(添加或修正测试)
  • 🔧 配置/构建相关(构建脚本、CI 配置等)
  • 🔒 安全修复(修复安全漏洞)
  • 🚀 其他

📦 改动内容

🔗 相关 Issue

✅ 代码质量自检清单

基础检查

  • 代码已通过本地所有测试(uv run pytest tests/
  • 代码符合项目规范(已运行 uv run pre-commit run --all-files
  • 已添加/更新相关测试用例,覆盖新增或修改的代码
  • 已更新相关文档(README.md, CONTRIBUTING.md 等)
  • 提交信息遵循语义化提交规范
  • 已阅读并遵循贡献指南

最佳实践检查

  • 最小改动原则:此 PR 只包含必要的改动,没有无关的修改
  • 单一职责:此 PR 只解决一个问题或添加一个功能
  • 功能必要性:确认此功能是项目必需的,不是个人偏好
  • 代码可读性:代码易于理解,复杂逻辑有适当注释
  • 性能考虑:改动不会显著降低性能(如有性能影响请说明)

兼容性检查

  • 向后兼容:改动不会破坏现有用户的配置和使用方式
  • 配置兼容:新增配置项有默认值,不影响未设置该配置的用户
  • API 兼容:如果修改了 API,旧的调用方式仍然可用或有平滑迁移方案
  • 数据兼容:改动不会导致现有数据格式失效或需要迁移
  • 依赖兼容:新增或更新的依赖不会与现有依赖冲突

项目特定检查

  • 多账号支持:改动正确处理多账号场景
  • 多平台兼容:改动兼容 anyrouter、agentrouter 等所有支持的平台
  • 错误处理:网络请求和外部调用有适当的错误处理
  • 日志记录:重要操作有清晰的日志输出
  • 环境变量:新增环境变量在 README 中有文档说明
  • 通知兼容:如涉及通知功能,确保所有通知方式都能正常工作

安全检查

  • 代码不包含敏感信息(密码、token、私钥等)
  • 外部输入有适当的验证和清理
  • 已通过 bandit 安全扫描(uv run bandit -r . -c pyproject.toml

🧪 测试说明

测试环境

  • Python 版本:
  • 操作系统:
  • 相关平台:

测试用例

  • 场景 1:
  • 场景 2:
  • 边界条件:

📸 截图/演示

💥 破坏性变更说明

📋 发布说明

面向用户的描述:

💬 额外说明


📝 审查者指南

审查者在审查此 PR 时,请重点关注:

  • 代码逻辑正确性
  • 向后兼容性
  • 错误处理是否完善
  • 测试覆盖率是否足够
  • 文档是否完整清晰

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant