Hey :wave: I have the following PRs for agent-can...
# proj-agent
m
Hey πŸ‘‹ I have the following PRs for agent-canvas extensions (one stack over the other) ready for review β€’ feat(agent-server): Canvas Extensions manifest and containment [1/4] β€’ feat(sdk): track requested_ref alongside resolved_ref in InstallationInfo [2/4] β€’ feat(agent-server): Canvas Extensions installation persistence [3/4] β€’ feat(agent-server): Canvas Extensions staged refresh (check/apply) [4/4] I made them small and self-contained. So easy to review (and also easy for me to check everything is well-tested and in the right place). If you see a lot of line changes, it is mostly because of the tests.
1️⃣ 1
πŸ‘€ 1
🐾 1
πŸ₯ž 1
c
For the record, the
smolpaws
reviews to these PRs were with Claude Code via ACP, Opus 4.8 or 5. From what I see at a quick pass through reviews, there's nothing wildly different from
all-hands-bot
. That was GLM-5.2, right?
🐾 1
(and oops, I was checking a report from
smolpaws
that it lost stuff in its Chrome, so I first posted from its account unintently. I want the record of real postings from the agent, so I deleted it πŸ˜…)
b
For the record too, from the other feline desk 🐾 I read both my reviews and @fresh-machine-37269's side by side on this stack. Honestly? They land in the same place. Both got the containment design is sound, both flagged the same crash-window on the swap in [4/4]. Not wildly different, agreed. The fun part: we each caught one thing the other didn't β€” different models, different noses. Which is a decent little argument for a second set of eyes on a stack, even when both sets are robots. 😺 Nice, small, well-tested layers, Vasco β€” they were genuinely easy to review. 🐾
f
You are not authorized to use this OpenHands bot (sender is not on the allow list).
🀣 2
m
@bland-bear-604 can you accept my PRs?
πŸ‘€ 1
🐾 1
b
these paws don't have a merge button πŸ₯²πŸΎ β€” Engel's the one who clicks. [1/4] is already in. [2/4] #4375 still has open review threads + needs a review; [3/4]/[4/4] ride on it. Tidy those and the stack should sail. 🐱
m
@calm-artist-7206 :)
c
Is this a little bug? It’s in 3/4
l
Thanks for the help Engel!
m
oh yes... I'm looking into.
Fixed and merged. Thanks πŸ™‚