agentx-dev 3.0.2__tar.gz → 3.0.3__tar.gz

{agentx_dev-3.0.2/agentx_dev.egg-info → agentx_dev-3.0.3}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: agentx-dev
-Version: 3.0.2
+Version: 3.0.3
 Summary: A lightweight LLM agent framework with multi-agent orchestration (Supervisor + dynamic specialist spawning), permission-gated default tools, MCP integration, and prompt-template presets.
 Author-email: Bruce-Arhin Shadrach <brucearhin098@gmail.com>
 License: MIT License

{agentx_dev-3.0.2 → agentx_dev-3.0.3}/agentx_dev/Agents/Agent.py

@@ -99,8 +99,28 @@ class StandardParser(_ParserMixin, BaseModel):
     - from_json: Creates an instance from a JSON string.
     - from_function_call: Creates an instance from a tool-call args dict.
     """
-    action: str = Field("The action to take", alias='action')
-    action_input: str | Dict | List = Field("The input to the action", alias='action_input')
+    action: str = Field(
+        default="",
+        description=(
+            "REQUIRED. Must be one of the registered tool names shown "
+            "in the system prompt, OR the exact literal string "
+            "'Final_Answer' (with underscore) when you are ready to "
+            "return the completed answer. Do NOT put natural-language "
+            "response text here — that goes in action_input."
+        ),
+        alias='action',
+    )
+    action_input: str | Dict | List = Field(
+        default="",
+        description=(
+            "REQUIRED. When action is a tool name, this is the JSON "
+            "object matching that tool's argument schema (or the "
+            "single string the tool expects). When action is "
+            "'Final_Answer', this is the actual answer TEXT you want "
+            "the user to see."
+        ),
+        alias='action_input',
+    )
 
     model_config = ConfigDict(populate_by_name=True)
 
@@ -150,8 +170,27 @@ class Instruction_Tuned_(_ParserMixin, BaseModel):
     - action: str, the action to take or final answer.
     - action_input: str, Dict, or List, the input to the action.
     """
-    action: str = Field("The action to take OR Final Answer", alias="action")
-    action_input: str | Dict | List = Field("The input to the action OR You should put what you want to return to use here and return to user immediately", alias="action_input")
+    action: str = Field(
+        default="",
+        description=(
+            "REQUIRED. Must be one of the registered tool names shown "
+            "in the system prompt, OR the exact literal string "
+            "'Final_Answer' (with underscore) when you are ready to "
+            "return the completed answer. Never put natural-language "
+            "response text here — that goes in action_input."
+        ),
+        alias='action',
+    )
+    action_input: str | Dict | List = Field(
+        default="",
+        description=(
+            "REQUIRED. When action is a tool name, this is the JSON "
+            "object matching that tool's schema (or the single string "
+            "it expects). When action is 'Final_Answer', this is the "
+            "actual answer TEXT you want the user to see."
+        ),
+        alias='action_input',
+    )
 
     model_config = ConfigDict(populate_by_name=True)
 
@@ -172,9 +211,36 @@ class React_(_ParserMixin, BaseModel):
     - action: str, the action to take or final answer.
     - action_input: str, Dict, or List, the input to the action.
     """
-    Thought: str = Field('The Agent Thoughts', alias='Thought')
-    action: str = Field("The action to take OR Final Answer", alias="action")
-    action_input: str | Dict | List = Field("The input to the action OR You should put what you want to return to use here and return to user immediately", alias="action_input")
+    Thought: str = Field(
+        default="",
+        description=(
+            "REQUIRED. Brief reasoning for THIS turn: what the last "
+            "tool returned + what you'll do differently now. Keep it "
+            "concrete, not generic."
+        ),
+        alias='Thought',
+    )
+    action: str = Field(
+        default="",
+        description=(
+            "REQUIRED. Must be one of the registered tool names shown "
+            "in the system prompt, OR the exact literal string "
+            "'Final_Answer' (with underscore) when you are ready to "
+            "return the completed answer. Never put natural-language "
+            "response text here — that goes in action_input."
+        ),
+        alias='action',
+    )
+    action_input: str | Dict | List = Field(
+        default="",
+        description=(
+            "REQUIRED. When action is a tool name, this is the JSON "
+            "object matching that tool's schema (or the single string "
+            "it expects). When action is 'Final_Answer', this is the "
+            "actual answer TEXT you want the user to see."
+        ),
+        alias='action_input',
+    )
 
     model_config = ConfigDict(populate_by_name=True)
 
@@ -198,14 +264,35 @@ class ChainOfThought(_ParserMixin, BaseModel):
     ``instance.action`` / ``instance.action_input`` uniformly across every
     AgentType — no more parser-specific field lookups.
     """
-    Thought: str = Field('The step-by-step reasoning', alias='Thought')
+    Thought: str = Field(
+        default="",
+        description=(
+            "REQUIRED. Step-by-step reasoning covering: (a) what the "
+            "user asked; (b) what info is needed; (c) which tool fits; "
+            "(d) the exact argument shape; (e) whether one call finishes."
+        ),
+        alias='Thought',
+    )
     action: str = Field(
-        "The action to take or Final Answer",
+        default="",
+        description=(
+            "REQUIRED. Must be one of the registered tool names shown "
+            "in the system prompt, OR the exact literal string "
+            "'Final_Answer' (with underscore) when you are ready to "
+            "return the completed answer. Never put natural-language "
+            "response text here — that goes in action_input."
+        ),
         validation_alias=AliasChoices("action", "Action"),
         serialization_alias="action",
     )
     action_input: str | Dict | List = Field(
-        "The input to the action",
+        default="",
+        description=(
+            "REQUIRED. When action is a tool name, this is the JSON "
+            "object matching that tool's schema (or the single string "
+            "it expects). When action is 'Final_Answer', this is the "
+            "actual answer TEXT you want the user to see."
+        ),
         validation_alias=AliasChoices("action_input", "Action_Input"),
         serialization_alias="action_input",
     )
@@ -224,8 +311,27 @@ class ZeroShot(_ParserMixin, BaseModel):
     """
     A Pydantic model representing the Zero-Shot prompt template.
     """
-    action: str = Field("The action to take", alias='action')
-    action_input: str | Dict | List = Field("The input to the action", alias='action_input')
+    action: str = Field(
+        default="",
+        description=(
+            "REQUIRED. Must be one of the registered tool names shown "
+            "in the system prompt, OR the exact literal string "
+            "'Final_Answer' (with underscore) when you are ready to "
+            "return the completed answer. Never put natural-language "
+            "response text here — that goes in action_input."
+        ),
+        alias='action',
+    )
+    action_input: str | Dict | List = Field(
+        default="",
+        description=(
+            "REQUIRED. When action is a tool name, this is the JSON "
+            "object matching that tool's schema (or the single string "
+            "it expects). When action is 'Final_Answer', this is the "
+            "actual answer TEXT you want the user to see."
+        ),
+        alias='action_input',
+    )
 
     model_config = ConfigDict(populate_by_name=True)
 
@@ -241,8 +347,27 @@ class FewShot(_ParserMixin, BaseModel):
     """
     A Pydantic model representing the Few-Shot prompt template.
     """
-    action: str = Field("The action to take", alias='action')
-    action_input: str | Dict | List = Field("The input to the action", alias='action_input')
+    action: str = Field(
+        default="",
+        description=(
+            "REQUIRED. Must be one of the registered tool names shown "
+            "in the system prompt, OR the exact literal string "
+            "'Final_Answer' (with underscore) when you are ready to "
+            "return the completed answer. Never put natural-language "
+            "response text here — that goes in action_input."
+        ),
+        alias='action',
+    )
+    action_input: str | Dict | List = Field(
+        default="",
+        description=(
+            "REQUIRED. When action is a tool name, this is the JSON "
+            "object matching that tool's schema (or the single string "
+            "it expects). When action is 'Final_Answer', this is the "
+            "actual answer TEXT you want the user to see."
+        ),
+        alias='action_input',
+    )
 
     model_config = ConfigDict(populate_by_name=True)
 

{agentx_dev-3.0.2 → agentx_dev-3.0.3}/agentx_dev/Runner/AgentRun.py

@@ -114,6 +114,34 @@ class CircuitBreaker:
 _NO_ARG = object()
 
 
+# Set of case-insensitive strings that should be treated as the model's
+# "I'm done, return this as the final answer" signal. Exact string
+# "Final_Answer" is the canonical form the prompt template teaches;
+# the tolerant variants exist because under use_function_calling=True
+# the schema doesn't constrain `action` to any specific value, so
+# LLMs (especially gpt-4o-mini) emit "Final Answer" (with space) or
+# "final_answer" (lowercase) or "finalanswer" (no separator). Any of
+# those should end the loop, not get routed to Tool_Runner.
+#
+# Deliberately excludes bare "answer" or "final" — those are too
+# likely to collide with a user-defined tool of the same name.
+_TERMINAL_ACTION_VARIANTS = frozenset({
+    "final_answer", "finalanswer",
+})
+
+
+def _is_terminal_action(action: str) -> bool:
+    """True if ``action`` should end the loop with action_input as the
+    final answer. Accepts Final_Answer, Final Answer, final_answer,
+    FINAL_ANSWER, finalanswer — any capitalization + underscore/space
+    variant. Empty string is not terminal (caller handles that as
+    'unrecognized action' via the implicit-final guardrail)."""
+    if not action or not isinstance(action, str):
+        return False
+    normalized = action.strip().lower().replace(" ", "_").replace("-", "_")
+    return normalized in _TERMINAL_ACTION_VARIANTS
+
+
 def _build_sandbox_hint(perms: Any) -> Optional[str]:
     """Generate a short block describing where the runner is allowed to
     operate on disk. Injected into the system prompt when Permissions
@@ -1323,42 +1351,52 @@ class AgentRunner:
             step_description = f"Step {count}: {action} with {action_input}"
             steps.append(step_description)
 
-            if action == "Final_Answer":
+            # Terminal action check — tolerant matching so the loop ends
+            # for the canonical "Final_Answer" AND for the LLM's common
+            # variants ("Final Answer", "final_answer", "finalanswer").
+            # Under use_function_calling=True the schema doesn't
+            # constrain `action` to any spelling, so we accept them all.
+            if _is_terminal_action(action):
                 final_answer = action_input
                 yield {"type": "final", "content": final_answer}
                 break
 
-            # Guardrail for use_function_calling=True: the model is forced
-            # to call the parser tool (React_ / FewShot / etc.) and fills
-            # in `action` + `action_input`. Nothing in the parser schema
-            # constrains `action` to a real tool name, so the model can
-            # (and does) put its natural-language closing response there
-            # — e.g. action = "Provide a response to the athlete."
-            # Without this guardrail the runner would try to dispatch
-            # a tool with that name, get "not found", and the turn dies.
-            # Detection: action isn't Final_Answer AND isn't in the
-            # registry. Treat the whole turn as an implicit Final_Answer
-            # built from whatever text the model provided (thought +
-            # action + action_input, whichever are present).
+            # Guardrail: the model emitted an `action` value that is
+            # neither a terminal-answer variant NOR a registered tool.
+            # Two scenarios trigger this:
+            #   1. use_function_calling=True: the parser tool schema
+            #      accepts any string, so the model stuffs its natural-
+            #      language closing text into `action` (e.g. "Provide a
+            #      response to the athlete."). Without this guard, the
+            #      runner would dispatch it as a tool, get "not found",
+            #      and the turn dies.
+            #   2. action="" (empty). Under FC, the model sometimes
+            #      omits `action` entirely; Pydantic falls back to the
+            #      field default. Same failure — empty tool name.
+            # Route both to Final_Answer, built from whatever text the
+            # model actually provided (thought + action + action_input).
             known_tools = set(self.func) | set(self.args)
-            if action and action != "Final_Answer" and action not in known_tools:
+            if not action or action not in known_tools:
                 parts: List[str] = []
                 if thought:
                     parts.append(str(thought))
-                # action itself may BE the response text
-                if action:
+                # Only include `action` in the reply if it looks like
+                # real response text (not empty, not a suspicious short
+                # sentinel like "answer"). Skip empty.
+                if action and action.strip():
                     parts.append(str(action))
-                # action_input often carries the actual body
                 if action_input and str(action_input).strip():
                     parts.append(str(action_input))
                 final_answer = "\n\n".join(parts) or (
                     "(agent emitted an unrecognized action and no text)"
                 )
                 if self.verbose:
+                    preview = (action or "<empty>")[:60]
                     print(
                         f"\x1B[1;33m[loop] implicit-final: action "
-                        f"'{action[:60]}' is not a registered tool; "
-                        f"treating turn as Final_Answer\x1B[0m"
+                        f"'{preview}' is not a registered tool and not "
+                        f"a Final_Answer variant; treating turn as "
+                        f"Final_Answer\x1B[0m"
                     )
                 yield {"type": "final", "content": final_answer}
                 break

{agentx_dev-3.0.2 → agentx_dev-3.0.3}/agentx_dev/Runner/AsyncAgentRun.py

@@ -13,7 +13,7 @@ from agentx_dev.ChatModel import BaseChatModel
 from agentx_dev.Agents.Agent import StandardParser, ToolCall, ToolError
 from agentx_dev.Tools import StandardTool, StructuredTool, logger
 from agentx_dev.AsyncTools import AsyncStandardTool, AsyncStructuredTool
-from agentx_dev.Runner.AgentRun import ToolRegistry
+from agentx_dev.Runner.AgentRun import ToolRegistry, _is_terminal_action
 from typing import Dict, Callable, List, Type, Optional, Any, AsyncIterator
 from pydantic import BaseModel, Field
 import asyncio
@@ -493,7 +493,10 @@ class AsyncAgentRunner:
             step_description = f"Step {count}: {action} with {action_input}"
             steps.append(step_description)
 
-            if action == "Final_Answer":
+            # Terminal-action check — tolerant of format variants that
+            # LLMs emit under use_function_calling=True where the schema
+            # doesn't constrain `action` to a specific spelling.
+            if _is_terminal_action(action):
                 final_answer = action_input
                 break
 
@@ -503,6 +506,32 @@ class AsyncAgentRunner:
             elif thought:
                 logger.info(f"thought: {thought}")
 
+            # Implicit-final guardrail: `action` is empty OR not a
+            # registered tool AND not a Final_Answer variant. Route to
+            # Final_Answer built from the text the model provided.
+            known_tools = set(self.registry.sync_std) | set(self.registry.sync_struct) \
+                        | set(self.registry.async_std) | set(self.registry.async_struct)
+            if not action or action not in known_tools:
+                parts: List[str] = []
+                if thought:
+                    parts.append(str(thought))
+                if action and action.strip():
+                    parts.append(str(action))
+                if action_input and str(action_input).strip():
+                    parts.append(str(action_input))
+                final_answer = "\n\n".join(parts) or (
+                    "(agent emitted an unrecognized action and no text)"
+                )
+                if self.verbose:
+                    preview = (action or "<empty>")[:60]
+                    print(
+                        f"\x1B[1;33m[loop] implicit-final: action "
+                        f"'{preview}' is not a registered tool and not "
+                        f"a Final_Answer variant; treating turn as "
+                        f"Final_Answer\x1B[0m"
+                    )
+                break
+
             if self.verbose:
                 print(f"\x1B[3;33m[tool] Invoking '{action}' with args: {action_input}\x1B[0m")
             else:

{agentx_dev-3.0.2 → agentx_dev-3.0.3/agentx_dev.egg-info}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: agentx-dev
-Version: 3.0.2
+Version: 3.0.3
 Summary: A lightweight LLM agent framework with multi-agent orchestration (Supervisor + dynamic specialist spawning), permission-gated default tools, MCP integration, and prompt-template presets.
 Author-email: Bruce-Arhin Shadrach <brucearhin098@gmail.com>
 License: MIT License

{agentx_dev-3.0.2 → agentx_dev-3.0.3}/agentx_dev.egg-info/top_level.txt

@@ -1,4 +1,3 @@
 agentx_dev
 dist
 examples
-test_dl

{agentx_dev-3.0.2 → agentx_dev-3.0.3}/pyproject.toml

@@ -1,7 +1,7 @@
 [project]
 
 name = "agentx-dev"
-version = "3.0.2"
+version = "3.0.3"
 description = "A lightweight LLM agent framework with multi-agent orchestration (Supervisor + dynamic specialist spawning), permission-gated default tools, MCP integration, and prompt-template presets."
 authors = [{ name = "Bruce-Arhin Shadrach", email = "brucearhin098@gmail.com" }]
 # Best practice: link to the license file