Open
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Coder Agent Improvements
This PR introduces several key improvements to the
coder_agent.pyscript to make it a functional and robust CLI tool.1. Fix: Infinite Loop Termination
Problem: The agent would enter an infinite loop because
is_goal_achievedlogic failed whentool_callswas present but empty/None (a behavior common in newer OpenAI SDK versions).Fix: Updated the termination condition to robustly check for falsy
tool_calls:2. Enhancement: Visible Assistant Responses
Problem: The agent's text responses were being generated but not printed to the console, leaving the user to see only "Thinking..." steps.
Fix: Added a print statement to output
completion.choices[0].message.contentwhen present.3. Enhancement: Continuous Chat Loop (REPL)
Problem: The script would exit after a single turn, forcing the user to restart the process for every follow-up question.
Fix: Wrapped the interaction logic in a
while Trueloop to allow for continuous conversation. Added an exit condition (exit/quit) andKeyboardInterrupthandling.Verified By