patchpal 0.25.1__tar.gz → 0.25.3__tar.gz

{patchpal-0.25.1/patchpal.egg-info → patchpal-0.25.3}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: patchpal
-Version: 0.25.1
+Version: 0.25.3
 Summary: An agentic coding and automation assistant, supporting both local and cloud LLMs
 Author: PatchPal Contributors
 License-Expression: Apache-2.0

{patchpal-0.25.1 → patchpal-0.25.3}/patchpal/__init__.py

@@ -1,6 +1,6 @@
 """PatchPal - An open-source Claude Code clone implemented purely in Python."""
 
-__version__ = "0.25.1"
+__version__ = "0.25.3"
 
 from patchpal.agent import create_agent, create_react_agent
 from patchpal.cli.autopilot import autopilot_loop

{patchpal-0.25.1 → patchpal-0.25.3}/patchpal/agent/function_calling.py

@@ -1161,6 +1161,12 @@ Repository root takes priority if both exist."""
             The agent's final response
         """
         # Agent loop
+        # Track consecutive empty responses (no content + no tool calls).
+        # Common with some local models (e.g., GPT-OSS-120B on vLLM) due to
+        # intermittent harmony-format parsing hiccups. A bounded retry usually
+        # recovers since the failure is probabilistic.
+        empty_response_retries = 0
+        empty_response_max_retries = int(os.environ.get("PATCHPAL_EMPTY_RETRY_MAX", "2"))
         for iteration in range(max_iterations):
             # Clear any stale pending images from previous iteration (safety check)
             # Images should already be injected, but this prevents accumulation on error
@@ -1335,6 +1341,8 @@ Repository root takes priority if both exist."""
 
             # Check if there are tool calls
             if hasattr(assistant_message, "tool_calls") and assistant_message.tool_calls:
+                # A non-empty turn succeeded; reset the empty-response retry budget
+                empty_response_retries = 0
                 # Print explanation text before executing tools (render as markdown)
                 if assistant_message.content and assistant_message.content.strip():
                     console = Console()
@@ -1834,9 +1842,25 @@ Repository root takes priority if both exist."""
                 if assistant_message.content:
                     return assistant_message.content
                 else:
-                    # Model returned no content and no tool calls
-                    # This can happen with some local models (e.g., GPT-OSS-120B)
-                    # Using "NO_RESPONSE:" prefix for programmatic detection
+                    # Model returned no content and no tool calls.
+                    # This happens intermittently with some local models
+                    # (e.g., GPT-OSS-120B on vLLM) due to harmony-format parsing
+                    # hiccups. Because the failure is probabilistic, retry the turn
+                    # a bounded number of times before giving up.
+                    if empty_response_retries < empty_response_max_retries:
+                        empty_response_retries += 1
+                        # Drop the empty assistant message so it doesn't pollute history
+                        if self.messages and self.messages[-1] is assistant_msg:
+                            self.messages.pop()
+                        print(
+                            f"\033[2m↻ Empty response from model; retrying "
+                            f"({empty_response_retries}/{empty_response_max_retries})...\033[0m",
+                            flush=True,
+                        )
+                        continue
+
+                    # Retries exhausted: surface the honest failure.
+                    # Using "NO_RESPONSE:" prefix for programmatic detection.
                     return "NO_RESPONSE: Model returned empty response. Try rephrasing your request or type 'continue' to prompt again."
 
         # Max iterations reached

{patchpal-0.25.1 → patchpal-0.25.3}/patchpal/cli/sandbox.py

@@ -198,7 +198,14 @@ def detect_llm_endpoints_from_env():
         or os.environ.get("AWS_REGION_NAME")  # LiteLLM-specific
     )
     if aws_region and (
-        os.environ.get("AWS_ACCESS_KEY_ID") or os.environ.get("AWS_SECRET_ACCESS_KEY")
+        os.environ.get("AWS_ACCESS_KEY_ID")
+        or os.environ.get("AWS_SECRET_ACCESS_KEY")
+        # SSO/ADFS profiles (created via `aws configure sso`) authenticate via a
+        # named profile and a cached SSO token instead of static access keys.
+        # In that case AWS_PROFILE is set but AWS_ACCESS_KEY_ID is not, so detect
+        # it here too — otherwise Bedrock endpoints would be omitted from the
+        # firewall allowlist and traffic would be blocked.
+        or os.environ.get("AWS_PROFILE")
     ):
         # Add Bedrock endpoints for the region
         # Support both GovCloud and standard regions
@@ -659,6 +666,19 @@ def build_container_args(sandbox_args, patchpal_args, env_file_keys=None):
         container_args.extend(["-v", f"{patchpal_dir}:/root/.patchpal"])
         mounted_paths.add(patchpal_dir)
 
+    # Mount ~/.aws if it exists (read-only).
+    # Required for AWS SSO/ADFS authentication: credentials for a named profile
+    # (created via `aws configure sso`) are resolved from ~/.aws/config, and the
+    # short-lived SSO token is cached under ~/.aws/sso/cache/. Mounting this
+    # directory lets boto3/LiteLLM inside the container resolve AWS_PROFILE just
+    # as it would on the host. It also covers static shared-credential setups
+    # (~/.aws/credentials). Mounted read-only; refresh the token on the host with
+    # `aws sso login --profile <name>` when it expires.
+    aws_dir = os.path.expanduser("~/.aws")
+    if os.path.isdir(aws_dir) and aws_dir not in mounted_paths:
+        container_args.extend(["-v", f"{aws_dir}:/root/.aws:ro"])
+        mounted_paths.add(aws_dir)
+
     # Development mode: mount local patchpal source code
     dev_mode = False
     if sandbox_args.dev:

{patchpal-0.25.1 → patchpal-0.25.3/patchpal.egg-info}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: patchpal
-Version: 0.25.1
+Version: 0.25.3
 Summary: An agentic coding and automation assistant, supporting both local and cloud LLMs
 Author: PatchPal Contributors
 License-Expression: Apache-2.0