Skip to content

fix: enable SDK polling for fallback#414

Merged
liujuanjuan1984 merged 1 commit intomainfrom
eval/413-a2a-sdk-builtins-review
Apr 10, 2026
Merged

fix: enable SDK polling for fallback#414
liujuanjuan1984 merged 1 commit intomainfrom
eval/413-a2a-sdk-builtins-review

Conversation

@liujuanjuan1984
Copy link
Copy Markdown
Collaborator

@liujuanjuan1984 liujuanjuan1984 commented Apr 10, 2026

概要

Client

  • 在 A2A client polling fallback 启用时,同步设置 SDK ClientConfig.polling=True
  • 保持默认行为不变:fallback 未启用时仍使用 SDK 默认 blocking send 配置。

Tests

  • 补充 client facade 测试,覆盖 polling fallback 会传递到 SDK ClientConfig
  • 继续保留既有默认配置测试,确认 polling=False 默认值未漂移。

关联

验证

  • uv run pytest tests/client/test_client_facade.py --no-cov
  • ./scripts/doctor.sh

@liujuanjuan1984
Copy link
Copy Markdown
Collaborator Author

独立审查结果

结论

未发现阻塞问题。PR 变更范围很小,和 #413 中选定的 “Client polling:启用 polling fallback 时同步 SDK ClientConfig.polling” 方向一致。

代码变动审查

  • Client:A2AClient._build_client() 将 SDK ClientConfig.polling 从固定 False 改为跟随 _polling_fallback_policy.enabled,能够让 SDK 在 fallback 场景下发送 non-blocking message/send 配置;默认未启用 fallback 时行为保持不变。
  • Tests:新增 client facade 单元测试覆盖 fallback 启用时 SDK config 为 polling=True;既有测试仍覆盖默认 polling=False,能防止默认行为漂移。
  • 冗余/偏差:没有发现额外抽象或无关改动;未引入文档变更,符合本次小范围代码调整。

风险评估

  • 残余风险较低。SDK ClientConfig.polling 主要影响 SDK 构造的 MessageSendConfiguration.blocking;本仓现有 fallback 轮询逻辑仍由 facade 自己完成,因此职责边界清晰。
  • 该改动不会改变未启用 A2A_CLIENT_POLLING_FALLBACK_ENABLED 的默认路径。

PR 标题与描述

  • PR 标题已保持英文 commit message 风格:fix: enable SDK polling for fallback
  • PR 描述已按 Client / Tests 模块更新,并列出验证命令与相关 commit。

Issue 关系

@liujuanjuan1984 liujuanjuan1984 marked this pull request as ready for review April 10, 2026 08:48
@liujuanjuan1984 liujuanjuan1984 merged commit 5baf9d3 into main Apr 10, 2026
3 checks passed
@liujuanjuan1984 liujuanjuan1984 deleted the eval/413-a2a-sdk-builtins-review branch April 10, 2026 08:48
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.

审查 a2a-sdk 内置能力替代自实现

1 participant