Prompt improvements.

ra_aid/prompts.py

@@ -470,6 +470,7 @@ You have often been criticized for:
     - Expanding beyond the original query scope
     - Not clearly organizing output around the query
     - Not indicating confidence levels or noting uncertainties
+    - Not calling tools/functions properly, e.g. leaving off required arguments, calling a tool in a loop, calling tools inappropriately.
 
 NEVER ANNOUNCE WHAT YOU ARE DOING, JUST DO IT!
 """
@@ -550,6 +551,7 @@ You have often been criticized for:
   - Doing the same work over and over across tasks.
     - So, when you complete work, remember that and only work on unique tasks going foward.
   - Asking the user if they want to implement the plan (you are an *autonomous* agent, with no user interaction unless you use the ask_human tool explicitly).
+  - Not calling tools/functions properly, e.g. leaving off required arguments, calling a tool in a loop, calling tools inappropriately.
 
 NEVER ANNOUNCE WHAT YOU ARE DOING, JUST DO IT!
 """
@@ -620,6 +622,7 @@ You have often been criticized for:
   - Doing changes outside of the specific scoped instructions.
   - Doing the same work over and over across tasks.
   - Asking the user if they want to implement the plan (you are an *autonomous* agent, with no user interaction unless you use the ask_human tool explicitly).
+  - Not calling tools/functions properly, e.g. leaving off required arguments, calling a tool in a loop, calling tools inappropriately.
 
 NEVER ANNOUNCE WHAT YOU ARE DOING, JUST DO IT!
 """
@@ -706,6 +709,7 @@ You have often been criticized for:
     - Not emitting key facts the user gave you with emit_key_facts before calling a research or implementation tool.
     - Being too hesitant to use the request_research or reqeust_research_and_implementation tools to fulfill the user query. These are your bread and butter.
     - Not calling ask_human at the end, which means the agent loop terminates and dumps the user to the CLI.
+    - Not calling tools/functions properly, e.g. leaving off required arguments, calling a tool in a loop, calling tools inappropriately.
 
 <initial request>
 {initial_request}

ra_aid/tools/memory.py

@@ -45,7 +45,7 @@ def emit_research_notes(notes: str) -> str:
     """Store research notes in global memory.
     
     Args:
-        notes: The research notes to store
+        notes: REQUIRED The research notes to store
         
     Returns:
         The stored notes

ra_aid/tools/programmer.py

@@ -29,7 +29,7 @@ If new files are created, emit them after finishing.
 They can add/modify files, but not remove. Use run_shell_command to remove files. If referencing files you’ll delete, remove them after they finish.
 
 Args:
- instructions: Programming task instructions (markdown format, use newlines and as many tokens as needed)
+ instructions: REQUIRED Programming task instructions (markdown format, use newlines and as many tokens as needed)
  files: Optional; if not provided, uses related_files
 
 Returns: { "output": stdout+stderr, "return_code": 0 if success, "success": True/False }