praisonai-code 0.0.6__tar.gz → 0.0.8__tar.gz

{praisonai_code-0.0.6 → praisonai_code-0.0.8}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: praisonai-code
-Version: 0.0.6
+Version: 0.0.8
 Summary: Agentic terminal CLI for PraisonAI — the terminal-native agent (run, chat, code, runtime, cli backends) extracted from the praisonai wrapper.
 Author: Mervin Praison
 License: MIT

{praisonai_code-0.0.6 → praisonai_code-0.0.8}/praisonai_code/cli/interactive/async_tui.py

@@ -111,6 +111,33 @@ def _detect_auth_error(text: Optional[str]) -> Optional[str]:
     return None
 
 
+def _extract_response_output(response) -> Optional[str]:
+    """Extract user-visible text from an agent response.
+
+    Handles plain strings as well as wrapper objects such as
+    ``AutonomyResult`` returned by ``agent.astart()`` which expose the
+    generated text via an ``output`` attribute.
+    """
+    if response is None:
+        return None
+    output = getattr(response, "output", None)
+    if output is not None:
+        text = str(output).strip()
+        return text or None
+    text = str(response).strip()
+    return text or None
+
+
+def _is_empty_agent_response(response) -> bool:
+    """True when the agent produced no user-visible output.
+
+    ``agent.astart()`` may return a truthy wrapper object (e.g.
+    ``AutonomyResult(success=True, output='')``) even on auth failure, so a
+    plain ``not response`` check is insufficient (issue #2568).
+    """
+    return _extract_response_output(response) is None
+
+
 class _LogCapture(logging.Handler):
     """Lightweight logging handler that records messages for failure detection."""
 
@@ -852,12 +879,31 @@ Example: /handoff code "refactor the auth module" """
             # Detect silent failures: some providers log an auth/permission
             # error and return None instead of raising. Surface these so the
             # CLI can exit with a non-zero status (issue #2562).
+            #
+            # NOTE: agent.astart() may return a truthy wrapper object such as
+            # AutonomyResult(success=True, output='') even on auth failure, so
+            # we must inspect the actual output rather than the object's
+            # truthiness (issue #2568).
+            output_text = _extract_response_output(response)
             auth_error = log_capture.find_auth_error()
-            if auth_error and not response:
+            if auth_error and output_text is None:
                 self._last_error = RuntimeError(auth_error)
                 return f"Error: {auth_error}"
 
-            return str(response) if response else None
+            # Surface explicit failures reported by wrapper objects even when
+            # no auth signature matched (e.g. AutonomyResult.success is False).
+            result_error = getattr(response, "error", None)
+            if result_error:
+                self._last_error = RuntimeError(str(result_error))
+                return f"Error: {result_error}"
+            if getattr(response, "success", None) is False:
+                self._last_error = RuntimeError("Agent execution failed")
+                # Preserve any partial/diagnostic output for the user instead
+                # of hiding it behind a generic message (still exits non-zero
+                # via ``execution_failed``).
+                return output_text if output_text is not None else "Error: Agent execution failed"
+
+            return output_text
         except Exception as e:
             logger.error(f"Error executing prompt: {e}", exc_info=True)
             self._last_error = e

{praisonai_code-0.0.6 → praisonai_code-0.0.8}/praisonai_code/cli_backends/__init__.py

@@ -6,6 +6,16 @@ Wrapper implementations of CLI backend protocols following AGENTS.md:
 - Lazy loading to avoid heavy imports at startup
 """
 
+def _is_cli_backend_instance(obj) -> bool:
+    """Return True if ``obj`` is a pre-resolved CliBackendProtocol instance.
+
+    A CliBackendProtocol instance exposes both ``execute()`` and ``stream()``.
+    This single predicate is the one place the protocol shape is checked, so any
+    future protocol change touches exactly one function.
+    """
+    return hasattr(obj, "execute") and hasattr(obj, "stream")
+
+
 def resolve_cli_backend_config(value):
     """Resolve a YAML/Python cli_backend value (str | dict | instance) to a CliBackendProtocol.
     
@@ -22,11 +32,16 @@ def resolve_cli_backend_config(value):
     """
     if value is None:
         return None
-        
-    # If already an instance, return as-is (duck typing)
-    if callable(value) or hasattr(value, 'process_turn'):
+
+    # Pre-resolved protocol instance: a CliBackendProtocol exposes execute() + stream().
+    if _is_cli_backend_instance(value):
         return value
-        
+
+    # Factory callable (not itself a protocol instance) - return as-is; the core
+    # Agent invokes it to obtain the protocol instance.
+    if callable(value):
+        return value
+
     # Import registry functions
     from .registry import resolve_cli_backend
     
@@ -68,6 +83,7 @@ __all__ = [
     "register_cli_backend",
     "resolve_cli_backend", 
     "resolve_cli_backend_config",
+    "_is_cli_backend_instance",
     "ClaudeCodeBackend",
     "list_cli_backends"
 ]

{praisonai_code-0.0.6 → praisonai_code-0.0.8}/praisonai_code.egg-info/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: praisonai-code
-Version: 0.0.6
+Version: 0.0.8
 Summary: Agentic terminal CLI for PraisonAI — the terminal-native agent (run, chat, code, runtime, cli backends) extracted from the praisonai wrapper.
 Author: Mervin Praison
 License: MIT

{praisonai_code-0.0.6 → praisonai_code-0.0.8}/pyproject.toml

@@ -1,6 +1,6 @@
 [project]
 name = "praisonai-code"
-version = "0.0.6"
+version = "0.0.8"
 description = "Agentic terminal CLI for PraisonAI — the terminal-native agent (run, chat, code, runtime, cli backends) extracted from the praisonai wrapper."
 readme = "README.md"
 license = {text = "MIT"}