Actually I think this might get automatically handled by pre-commit after this PR #465?

No, this PR adds a cron job that makes this issue every week #460
"The codebase contains multiple instances of vague types and union return types that force callers to manually narrow types via if/else or assert. "
I dont think that this is detectable by static tools. Sometimes a variable should be allowed to be optional, right? But if it's supposed to be well defined during runtime it should not be optional.

My hope is that this cron-job issue does meaningful code simplifications.

ah got it, make sense!

And this is the "output" of the issue #461
It fixed the multiple output create which is good but didn't pick up the issue with self.executor: Executor | None
I will make the same issue exactly. tag openhands again and see if it suggests other improvements

Agent Server image for this PR

Pull (multi-arch manifest):

docker pull ghcr.io/all-hands-ai/agent-server:b222f83

Run:

docker run -it --rm \
  -p 8000:8000 \
  --name agent-server-b222f83 \
  ghcr.io/all-hands-ai/agent-server:b222f83

This tag is a multi-arch manifest (amd64/arm64). Your client pulls the right arch automatically.