<github.com/OpenHands/software-agent-sdk/pull/4479...
# proj-automation
q
github.com/OpenHands/software-agent-sdk/pull/4479 github.com/OpenHands/automation/pull/334 github.com/OpenHands/OpenHands/pull/16697 Adding structured response for agent summaries/blockers for preset automations, and surfacing them in automation run UI cc @breezy-kilobyte-39430 for design direction wrt the surfacing the summaries
l
Thanks! I’m a slight bit worried that we’re reinventing pydantic datamodels when we already have very similar ones… Have you asked your agent to audit that?
For instance, I feel like we might already have a status enum that looks a bit like this:
Copy code
TaskOutcomeStatus = Literal[
    "success",
    "partial_success",
    "blocked",
    "failed",
    "unknown",
]
b
I'll review tomorrow 🙂
q
For instance, I feel like we might already have a status enum that looks a bit like this
Ah didn't realize this, ill take a look
github.com/OpenHands/software-agent-sdk/pull/4479#… ^ breakdown of the existing pydantic models we already have, seems like none of them address task success Although I agree I'd like to simplify as much as possible, just agent summary text + success/blocked/failed enum should be sufficient structured response is meant to be flexible (and in the future we'd define it on a per automation basis); so the pydantic model introduced here is not baked into the core sdk its a preset we provide as a fallback, so i'm not super worried about introducing it
l
so many statuses 😂
sadnod 1
but yeah, sgtm
m
Had a look through the three PRs. I believe that the sdk pr can be 1 line change: you just need to register the
FinishTool
in
openhands-sdk/openhands/sdk/tool/builtins/finish.py
with
register_tool(FinishTool.name, FinishTool)
c
Conversation’s FINISHED without ERROR or STUCK is successful, or isn't it?
But I agree, I think we need to try stuff🤔
b
@quick-window-91705 which of these has the design changes in it? should I load all of them to test?
q
the last openhands repo pr has the changes, ill dm you instructions to test it @modern-fireman-99755 ack running to the dentist will send a cleaned up pr after
m
no rush...
q
github.com/OpenHands/software-agent-sdk/pull/4479/changes greatly simplified this pr that just adds to the preset options in tools package, ready for review
❤️ 1