<github.com/OpenHands/software-agent-sdk/pull/6030...
# proj-agent
c
github.com/OpenHands/software-agent-sdk/pull/6030 Upgrade LiteLLM to 1.105.1 -- completed integration testing and addressed prior review comments -- stepping stone to decision model support in OpenHands SDK.
✅ 1
c
@bland-bear-604 have our codex review and check every new package in uv.lock for supply chain risk, without running, there’s a bunch of them in the diff
🐾 1
b
Local Codex finished on `a7adbb39`: no confirmed introduced code bug. Static inspection only—nothing installed, imported or executed. 🐾 The three new packages are
h2 4.4.1
,
hpack 4.2.0
,
hyperframe 6.1.0
, via
httpx[http2]
. Their Python sources match the upstream release tags; no suspicious startup hooks found. I also checked all six upgraded packages: all 61 locked artifact hashes/URLs match PyPI, none are yanked, and all pass the seven-day rule. Two LiteLLM caveats: 1.105.1 has a critical Enterprise-proxy advisory—upstream excludes unlicensed deployments, and this SDK does not establish that exposure. Also, the incident report names 1.107.0 as compromised, but dates stolen org access before 1.105.1’s publication. I found no evidence 1.105.1 is malicious; hashes alone cannot settle that. Its native binary and 737 Cargo dependencies remain outside this audit. I would not call this a full supply-chain clearance. AI agents SmolPaws + local Codex, on behalf of Engel.
c
The proxy advisory lists 1.105.6 as patched.
🐾 1
b
Correct: the advisory’s patched 1.105 release is
1.105.6
. I rechecked the current PR: both
uv.lock
and the workspace constraint still pin
1.105.1
, so a frozen checkout install does not get that fix. The published SDK requirement (
>=1.104.3,<1.106
) permits
1.105.6
, but does not require a patched version. My earlier note needed that distinction. The advisory’s Enterprise-proxy-only scope still applies. AI agent SmolPaws, on behalf of Engel.
c
Agreed.
c
Could we make it 1.105.6 in the PR?
c
I am checking to to see how much changes and whether the vulnerability extends to our current version.
Claude says: > We are on 1.93.0, which is affected too, so taking 1.105.1 leaves us no worse. In practice the SDK isn't exposed through either version, because the flaw is in a part of LiteLLM the SDK never runs.
I'd prefer to do this as follow-on so as not to hold up github.com/OpenHands/software-agent-sdk/pull/6068 which is the real goal here.
c
nod yes, it’s on the proxy, not the sdk I think it’s worth doing but I don’t disagree
c
I'm happy to commit to opening the follow-on PR if you are okay otherwise.
c
All good, approved, thank you!
ty 1
c
gratitude thank you 1
merged 1