[codex] Remove duplicate idle_poll name parameter#377
Conversation
|
Someone is attempting to deploy a commit to the crazyboym's projects Team on Vercel. A member of the Team first needs to authorize it. |
The code change dropped the redundant agent_name parameter, but the trilingual README snippets still showed the old four-arg signature, the double-passed call site, and agent_name inside the function body (read_inbox and claim_task). Update all three language code blocks to the three-arg form and bump the translation-sync markers so the docs match the code.
|
Thanks for the PR. I am closing this one as well, for two reasons. First, it does not fit the teaching design here: s12's code.py is deliberately minimal, and this adds a defensive blockedBy normalization layer (json and ast.literal_eval parsing, plus type validation) and a new tests/ harness, which are production concerns the chapter is not about. If blockedBy handling genuinely confuses readers, that is better raised as an issue so we can decide how, or whether, to fold it into the lesson. Second, the same policy point as #377: we are declining bulk PRs generated by an agent and opened across many repositories at once (see CONTRIBUTING.md). No objection to the underlying observation in #348. Feel free to keep that issue open for discussion. |
2 participants
Summary
agent_nameparameter fromidle_pollnameconsistently for inbox polling and task claimingFixes #374.
Validation
python3 -m py_compile s17_autonomous_agents/code.pygit diff --check