LLM provider connections. Would love a :eyes: on ...
# proj-agentm
millions-crayon-32880
08/17/2026, 7:03 PMLLM provider connections. Would love a š on these two
ā¢ sdk github.com/OpenHands/software-agent-sdk/pull/4492
ā¢ canvas github.com/OpenHands/OpenHands/pull/16616
c
calm-artist-7206
08/17/2026, 8:34 PMLeft a tiny question on the SDK PR. This looks good! We could use the agent's help to review the details - they can be good at finding corner cases
š 1
h
hundreds-twilight-32833
08/18/2026, 12:13 PM@quick-window-91705 could you take a look at these today if you get the chance?
c
calm-artist-7206
08/18/2026, 4:11 PMJust for clarity, I read the entire SDK PR, and the agent flagged some implementation issues.
The canvas PR is in draft; though I took a quick look at that too, and Iāll need my agent to set it up for me š¹
I think there are a couple of CI failures, as well - though mock LLM tests might be flaky
calm-artist-7206
08/18/2026, 4:14 PMš¤ I think the agentās findings here are nits, just would be nice for REST API robustness
m
millions-crayon-32880
08/18/2026, 9:16 PM@calm-artist-7206 I addresses yours and the agents concerns please re review.
millions-crayon-32880
08/18/2026, 9:18 PMThis my second take at this feature, first take was much more complex and every agent pass just made it more complex.
Decided to keep it minimal as possible.
ā¤ļø 1
c
calm-artist-7206
08/19/2026, 3:10 PMI basically like it and my thoughts are more around the PR, than the PR itself: I think maybe we should aim to remove now the obsolete-ish api keys in LLM profiles, so that we have a single source of truth of how to connect to a provider. š¤
Anything, in the PR or follow-ups, that can help move that forward, would be great!
I'm guessing we kind of canāt right now, until the Cloud also has Provider Connections, right?
calm-artist-7206
08/19/2026, 3:12 PMAh, I see you made changes and right now CI fails
calm-artist-7206
08/19/2026, 3:12 PMš
calm-artist-7206
08/19/2026, 3:22 PMOne thing I found helpful in understanding a PR is a small skill that tells the agent to show the human <%5Bhttps%3A//github.com/smolpaws/smolpaws/blob/main/.agents/skills/show-me/SKILL.md%5D(https://github.com/smolpaws/smolpaws/blob/main/.agents/skills/show-me/SKILL.md)|ābeforeā and āafterā like this> skill
(or something like that)
I think we have also some instruction somewhere telling the LLM to make a quick design doc in the temporary
.pr/<http://htmlpreview.github.io|htmlpreview.github.io>2 Views