Skip to content

'添加七牛云API支持'#691

Open
wszqj2018 wants to merge 1 commit into
hsliuping:mainfrom
wszqj2018:feature/add_qiniu_api
Open

'添加七牛云API支持'#691
wszqj2018 wants to merge 1 commit into
hsliuping:mainfrom
wszqj2018:feature/add_qiniu_api

Conversation

@wszqj2018

Copy link
Copy Markdown

Pull Request 模板

📋 PR 类型

请标记此 PR 的类型:

  • 🌟 新功能 (feature)
  • 🐛 Bug 修复 (bugfix)
  • 🧹 代码重构 (refactor)
  • 📝 文档更新 (documentation)
  • 🎨 样式优化 (style)
  • ⚡ 性能优化 (performance)
  • 🔧 配置/构建 (config/build)
  • 🧪 测试相关 (test)
  • 🤖 LLM 适配器集成 (llm-adapter)

📖 PR 描述

变更摘要

变更详情

相关 Issue

🤖 LLM 适配器集成检查清单

注意: 如果此 PR 涉及 LLM 适配器集成,请完成以下检查清单。如果不涉及,可以跳过此部分。

✅ 代码实现检查

  • 适配器类实现

    • 创建了继承自 OpenAICompatibleBase 的适配器类
    • 正确设置了 provider_nameapi_key_env_varbase_url
    • 实现了必要的模型配置
  • 注册和集成

    • OPENAI_COMPATIBLE_PROVIDERS 字典中注册了提供商
    • __init__.py 中添加了适配器导出
    • 在前端 sidebar.py 中添加了提供商选项
  • 环境变量配置

    • .env.example 中添加了 API Key 示例
    • 环境变量命名遵循 {PROVIDER}_API_KEY 格式
    • 提供了正确的 base_url 配置

✅ 测试和验证

  • 基础功能测试

    • API 连接测试通过
    • 简单文本生成功能正常
    • 错误处理机制有效
  • 工具调用测试

    • Function calling 功能正常工作
    • 工具参数解析正确
    • 复杂工具调用场景稳定
  • 集成测试

    • 前端界面显示正常
    • 模型选择器工作正确
    • TradingGraph 集成成功
    • 端到端分析流程正常
  • 性能和稳定性测试

    • 响应时间合理(< 30秒)
    • 连续运行测试通过(> 30分钟)
    • 内存使用稳定
    • 并发请求处理正确

✅ 文档和配置

  • 代码文档

    • 适配器类包含完整的 docstring
    • 关键方法有适当的注释
    • 参数说明清晰
  • 用户文档

    • 更新了相关的用户指南(如果需要)
    • 提供了配置示例
    • 包含故障排除信息(如果适用)

📝 测试报告

如果这是 LLM 适配器 PR,请提供以下信息:

提供商信息:

  • 提供商名称:
  • 官方网站:
  • API 文档:
  • 支持的模型:

测试结果:

  • 基础连接: ✅/❌
  • 工具调用: ✅/❌
  • Web 集成: ✅/❌
  • 端到端: ✅/❌

性能指标:

  • 平均响应时间: ___ 秒
  • 工具调用成功率: ___%
  • 内存使用: ___ MB

已知问题:

🧪 测试说明

如何测试此 PR

测试环境

  • 本地开发环境
  • Docker 环境
  • 生产环境

破坏性变更

  • 此 PR 包含破坏性变更
  • 此 PR 不包含破坏性变更

如果包含破坏性变更,请说明:

📊 影响范围

请标记此 PR 影响的组件:

  • 核心交易逻辑
  • LLM 适配器
  • Web 界面
  • 数据获取
  • 配置系统
  • 测试框架
  • 文档
  • 部署配置

🔗 相关链接

  • 相关文档:
  • 参考资料:
  • 相关 PR:

📷 截图/演示

✅ 检查清单

请确认以下项目:

代码质量

  • 代码遵循项目的编码规范
  • 没有不必要的调试代码或注释
  • 变量和函数命名清晰明确
  • 代码复用性良好,避免重复代码

测试覆盖

  • 新功能有相应的测试用例
  • 所有测试通过
  • 手动测试已完成
  • 边界情况已考虑

文档更新

  • README 已更新(如果需要)
  • API 文档已更新(如果需要)
  • 变更日志已更新(如果需要)
  • 配置文档已更新(如果需要)

安全考虑

  • 没有硬编码的密钥或敏感信息
  • 输入验证充分
  • 错误处理不泄露敏感信息
  • 第三方依赖安全可靠

性能考虑

  • 新功能不会显著影响性能
  • 内存使用合理
  • 网络请求优化
  • 数据库查询优化(如果适用)

🏷️ 标签建议

请为此 PR 建议适当的标签:

  • enhancement - 新功能或改进
  • bug - Bug 修复
  • documentation - 文档相关
  • refactor - 代码重构
  • performance - 性能优化
  • security - 安全相关
  • llm-adapter - LLM 适配器
  • ui/ux - 用户界面/体验
  • config - 配置相关
  • testing - 测试相关

👥 审查者

建议的审查者:

📝 额外说明


感谢您的贡献! 🎉

请确保您已经阅读并遵循了我们的 贡献指南。如果您有任何问题,请随时在 PR 中提问或联系维护者。

echopoison0712 pushed a commit to echopoison0712/TradingAgents-CN that referenced this pull request Jun 8, 2026
…g#694)

* fix: 修复港股代码全链路支持,避免误走 A 股路径 (Fixes hsliuping#629, Fixes hsliuping#691)

- validation.ts: 新增 /^HK\d{5}$/ 港股前缀格式校验,修复 Web 端 HK01810 被拒问题
- base.py: normalize_stock_code 支持 1810.HK / hk1810 归一为 HK01810;
         _is_hk_market 精确匹配 HK+纯数字,防止误匹配;
         DataFetcherManager.get_realtime_quote 新增港股专用快速路径,
         直接调用 akshare_hk,不再按 A 股 source_priority 多次尝试
- tushare_fetcher.py: 四处方法(_convert_stock_code / _fetch_raw_data /
  get_realtime_quote / get_stock_name)增加 HK 早退,避免港股误走深市
- baostock_fetcher.py: 两处方法增加 HK 早退,防止生成 sz.HK01810 等无效请求
- pytdx_fetcher.py: 两处方法增加 HK 早退,规避 36 秒连接超时
- akshare_fetcher.py: get_chip_distribution 增加 HK 早退(stock_cyq_em 仅支持 A 股)
- data_tools.py: DataFetcherManager 改为 singleton,避免每次工具调用重置熔断器;
                 get_realtime_quote 失败时返回 retriable=false,阻止 LLM 重试
- runner.py: 新增工具调用非兼容缓存(non_retriable_tool_results),
             HK 代码不同写法(HK01810 / 1810.HK)共享同一缓存 key
- tests: 新增 test_hk_realtime_routing.py;补充 test_stock_code_bse / test_agent_executor

* fix: reset agent fetcher singleton on config reload

* fix: align web hk validation with backend formats
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