ci(python): update a2a-sdk requirement from <0.4.0,>=0.3.0 to >=0.3.0,<1.2.0 in /strands-py - #3313
dependabot[bot] wants to merge 1 commit into
Conversation
Updates the requirements on [a2a-sdk](https://github.com/a2aproject/a2a-python) to permit the latest version. - [Release notes](https://github.com/a2aproject/a2a-python/releases) - [Changelog](https://github.com/a2aproject/a2a-python/blob/main/CHANGELOG.md) - [Commits](a2aproject/a2a-python@v0.3.0...v1.1.1) --- updated-dependencies: - dependency-name: a2a-sdk dependency-version: 1.1.1 dependency-type: direct:development ... Signed-off-by: dependabot[bot] <support@github.com>
|
@strandly-the-agent is it safe to do it? i mean it's a major version update right? dowe need to update teh codebase, i'd expect some breaking stuff? |
|
No — this is not safe as a metadata-only update. The new range permits I’d restore Verified breakageThe lint log reports removed Upstream’s v0.3 compatibility mode is wire-protocol compatibility; it does not preserve the old Python API. I inspected the exact head and CI logs; I did not independently rerun the full matrix, and most unit-test legs were cancelled after the first failure. |
|
@strandly-the-agent can you audit and propose what is required to make the SDK compatible with this change and the latest version? |
|
Audited Spike results on 1.1.2: ported Proposed path: close this PR + add a Dependabot Two user-visible breaks need a decision: 1. What breaks (verified against 1.1.2)
2. Traps the upstream migration guide doesn't mention (found by running it)These came out of the spike, not the guide — they're the ones likely to burn a day each:
3. Change setSource — Tests — 237 focused unit tests to adapt: Packaging — collapse the two CI — one Linux/3.12 job pinned to Docs — 4. Strategy, parity, and processWhy not just merge this PR: it raises the ceiling to Cross-SDK parity is the real constraint. Process obligations: 5. Open questions only you can answer
What I verified vs. inferredVerified in my sandbox at The items in section 2 come from the spike's real server/client runs rather than a second independent check by me. The spike deliberately ported only enough of |
Updates the requirements on a2a-sdk to permit the latest version.
Release notes
Sourced from a2a-sdk's releases.
Changelog
Sourced from a2a-sdk's changelog.
... (truncated)
Commits
86c6b0dchore(main): release 1.1.1 (#1103)0d82ab9refactor(http): eliminate third-party httpx-sse dependency (#1132)723880erefactor(grpc): remove grpcio-status dependency (#1130)343373btest: expand itk scenarios with ts java and rust agents (#1129)e7eafe3chore(deps): bump cryptography from 48.0.0 to 48.0.1 (#1109)46b292bfix(server): improve Swagger /docs example annotations for A2A routes (#1092)b264a6fci: fix agent card routes in itk agent (#1102)c74332dchore(deps): bump the all group across 1 directory with 11 updates (#1100)6143e8achore(deps): Update dev dependencies (#1098)75991c1ci: merge install smoke tests profile matrix (#1097)You can trigger a rebase of this PR by commenting
@dependabot rebase.Dependabot commands and options
You can trigger Dependabot actions by commenting on this PR:
@dependabot rebasewill rebase this PR@dependabot recreatewill recreate this PR, overwriting any edits that have been made to it@dependabot show <dependency name> ignore conditionswill show all of the ignore conditions of the specified dependency@dependabot ignore this major versionwill close this PR and stop Dependabot creating any more for this major version (unless you reopen the PR or upgrade to it yourself)@dependabot ignore this minor versionwill close this PR and stop Dependabot creating any more for this minor version (unless you reopen the PR or upgrade to it yourself)@dependabot ignore this dependencywill close this PR and stop Dependabot creating any more for this dependency (unless you reopen the PR or upgrade to it yourself)