Skip to content

ci: ignore unsatisfiable protobuf major updates in dependabot#489

Merged
liujuanjuan1984 merged 1 commit into
mainfrom
fix/actions-failure-diagnosis
May 15, 2026
Merged

ci: ignore unsatisfiable protobuf major updates in dependabot#489
liujuanjuan1984 merged 1 commit into
mainfrom
fix/actions-failure-diagnosis

Conversation

@liujuanjuan1984
Copy link
Copy Markdown
Collaborator

@liujuanjuan1984 liujuanjuan1984 commented May 15, 2026

背景

最近一次 Dependabot uv 更新任务失败。失败点不在主验证工作流,而在 Dependabot 尝试对 protobuf 发起 major 升级时触发了解算冲突。

根因分析

当前项目依赖仍要求 protobuf < 7,而 Dependabot 在 grouped update 中尝试将 protobuf 升级到 7.x。这会与 a2a-sdk 的版本约束共同导致依赖不可解,因此更新任务稳定失败。

本次改动

  • .github/dependabot.yml 中忽略 protobufversion-update:semver-major
  • 保留现有 weekly 调度、分组更新和其它依赖的自动更新能力

审查结论

  • 改动方向合理:直接约束当前不可满足的更新路径,命中失败根因
  • 改动边界收敛:只屏蔽 protobuf major 更新,不影响 patch / minor 更新,也不扩大其它自动化行为
  • 风险可控:后续当 a2a-sdk 或项目自身支持 protobuf 7 后,可以移除该忽略规则

验证

  • 已运行 bash ./scripts/doctor.sh
  • 本地验证通过:701 个测试通过,coverage gate、build、built wheel smoke test 全部通过

备注

  • 本次任务按要求未新建 issue

@liujuanjuan1984 liujuanjuan1984 changed the title fix: ignore unsatisfiable protobuf major bumps ci: ignore unsatisfiable protobuf major updates in dependabot May 15, 2026
@liujuanjuan1984
Copy link
Copy Markdown
Collaborator Author

独立审查结论:当前 PR 改动合理,未发现阻塞性问题。

审查要点:

  • 改动直接命中最近一次 Dependabot 失败的根因,即 protobuf 7.x 与当前依赖约束不兼容
  • 变更边界较小,只忽略 protobuf 的 major 更新,不影响 weekly 依赖更新中的 patch / minor 路径
  • 实现方式稳健,优先在自动化配置层收敛无效更新尝试,没有引入额外脚本或工作流复杂度

剩余注意点:

  • a2a-sdk 或项目自身后续支持 protobuf 7 时,需要移除这条 ignore 规则,否则 Dependabot 不会自动提出该升级

@liujuanjuan1984 liujuanjuan1984 marked this pull request as ready for review May 15, 2026 03:07
@liujuanjuan1984 liujuanjuan1984 merged commit b1074b7 into main May 15, 2026
3 checks passed
@liujuanjuan1984 liujuanjuan1984 deleted the fix/actions-failure-diagnosis branch May 15, 2026 03:07
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