Conversation
Prevent the main nanoAgent entrypoint from hanging forever on blocking shell commands by enforcing the same 30s timeout already used in the richer agent variants. Ultraworked with [Sisyphus](https://github.com/code-yeongyu/oh-my-opencode) Co-authored-by: Sisyphus <clio-agent@sisyphuslabs.ai>
Route tool-call argument parsing through a helper so malformed JSON is surfaced back into the loop instead of crashing the agent, while preserving the 30-second bash timeout behavior already covered by this branch. Ultraworked with [Sisyphus](https://github.com/code-yeongyu/oh-my-openagent) Co-authored-by: Sisyphus <clio-agent@sisyphuslabs.ai>
|
Follow-up: I pushed a small branch-local fix on top of this PR so malformed tool arguments no longer crash the agent loop. Local verification after the update:
So the bash-timeout behavior is still intact, and invalid tool-call arguments now surface back into the loop as tool errors instead of aborting execution. |
|
Follow-up: I also refreshed this branch onto the current Local verification after the refresh:
So this PR now includes both the invalid-argument guard fix and a clean base refresh, with the agent regression suite still green. |
|
Closing this stale radar lane to keep the authored-PR queue current. I refreshed/revalidated the branch recently, but there has still been no maintainer response on this thread, so I am closing it for now instead of letting it sit indefinitely. If maintainers want this change later, I can reopen or resend it on a fresh branch. |
Summary
agent.py'sexecute_bash()path so the main CLI no longer hangs forever on blocking shell commandsValidation
python3 -m unittest tests.test_agent.AgentRegressionTests.test_execute_bash_passes_timeout tests.test_agent.AgentRegressionTests.test_execute_bash_returns_timeout_errorNotes
tests/test_agent.pysuite still contains pre-existing failures aroundparse_tool_argumentsonagent.py, which belong to the already-open tool-call guardrail line in#4; this PR intentionally stays scoped to bash timeout hardening only