PraisonAI 4.7.7__tar.gz → 4.7.8__tar.gz

{praisonai-4.7.7 → praisonai-4.7.8}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: PraisonAI
-Version: 4.7.7
+Version: 4.7.8
 Summary: PraisonAI is an AI Agents Framework with Self Reflection. PraisonAI application combines PraisonAI Agents, AutoGen, and CrewAI into a low-code solution for building and managing multi-agent LLM systems, focusing on simplicity, customisation, and efficient human-agent collaboration.
 Author: Mervin Praison
 License: MIT
@@ -11,14 +11,14 @@ Description-Content-Type: text/markdown
 Requires-Dist: rich>=13.7
 Requires-Dist: markdown>=3.5
 Requires-Dist: pyparsing>=3.0.0
-Requires-Dist: praisonaiagents>=1.7.5
-Requires-Dist: praisonai-code>=0.1.4
-Requires-Dist: praisonai-bot>=0.1.4
-Requires-Dist: praisonai-train>=0.1.4
-Requires-Dist: praisonai-browser>=0.1.4
-Requires-Dist: praisonai-mcp>=0.1.6
-Requires-Dist: praisonai-sandbox>=0.1.4
-Requires-Dist: praisonai-deploy>=0.1.4
+Requires-Dist: praisonaiagents>=1.7.6
+Requires-Dist: praisonai-code>=0.1.5
+Requires-Dist: praisonai-bot>=0.1.5
+Requires-Dist: praisonai-train>=0.1.5
+Requires-Dist: praisonai-browser>=0.1.5
+Requires-Dist: praisonai-mcp>=0.1.7
+Requires-Dist: praisonai-sandbox>=0.1.5
+Requires-Dist: praisonai-deploy>=0.1.5
 Requires-Dist: python-dotenv>=0.19.0
 Requires-Dist: litellm<2,>=1.83.14
 Requires-Dist: PyYAML>=6.0

{praisonai-4.7.7 → praisonai-4.7.8}/PraisonAI.egg-info/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: PraisonAI
-Version: 4.7.7
+Version: 4.7.8
 Summary: PraisonAI is an AI Agents Framework with Self Reflection. PraisonAI application combines PraisonAI Agents, AutoGen, and CrewAI into a low-code solution for building and managing multi-agent LLM systems, focusing on simplicity, customisation, and efficient human-agent collaboration.
 Author: Mervin Praison
 License: MIT
@@ -11,14 +11,14 @@ Description-Content-Type: text/markdown
 Requires-Dist: rich>=13.7
 Requires-Dist: markdown>=3.5
 Requires-Dist: pyparsing>=3.0.0
-Requires-Dist: praisonaiagents>=1.7.5
-Requires-Dist: praisonai-code>=0.1.4
-Requires-Dist: praisonai-bot>=0.1.4
-Requires-Dist: praisonai-train>=0.1.4
-Requires-Dist: praisonai-browser>=0.1.4
-Requires-Dist: praisonai-mcp>=0.1.6
-Requires-Dist: praisonai-sandbox>=0.1.4
-Requires-Dist: praisonai-deploy>=0.1.4
+Requires-Dist: praisonaiagents>=1.7.6
+Requires-Dist: praisonai-code>=0.1.5
+Requires-Dist: praisonai-bot>=0.1.5
+Requires-Dist: praisonai-train>=0.1.5
+Requires-Dist: praisonai-browser>=0.1.5
+Requires-Dist: praisonai-mcp>=0.1.7
+Requires-Dist: praisonai-sandbox>=0.1.5
+Requires-Dist: praisonai-deploy>=0.1.5
 Requires-Dist: python-dotenv>=0.19.0
 Requires-Dist: litellm<2,>=1.83.14
 Requires-Dist: PyYAML>=6.0

{praisonai-4.7.7 → praisonai-4.7.8}/PraisonAI.egg-info/requires.txt

@@ -1,14 +1,14 @@
 rich>=13.7
 markdown>=3.5
 pyparsing>=3.0.0
-praisonaiagents>=1.7.5
-praisonai-code>=0.1.4
-praisonai-bot>=0.1.4
-praisonai-train>=0.1.4
-praisonai-browser>=0.1.4
-praisonai-mcp>=0.1.6
-praisonai-sandbox>=0.1.4
-praisonai-deploy>=0.1.4
+praisonaiagents>=1.7.6
+praisonai-code>=0.1.5
+praisonai-bot>=0.1.5
+praisonai-train>=0.1.5
+praisonai-browser>=0.1.5
+praisonai-mcp>=0.1.7
+praisonai-sandbox>=0.1.5
+praisonai-deploy>=0.1.5
 python-dotenv>=0.19.0
 litellm<2,>=1.83.14
 PyYAML>=6.0

{praisonai-4.7.7 → praisonai-4.7.8}/praisonai/app/agentos.py

@@ -13,6 +13,16 @@ import os
 from praisonaiagents import AgentOSConfig, AgentOSProtocol
 
 
+def _run_to_dict(record: Any) -> Dict[str, Any]:
+    """A RunRecord as JSON. Uses whatever the record exposes rather than
+    assuming a shape, so a ledger with extra fields is not silently truncated."""
+    if hasattr(record, "as_dict"):
+        return record.as_dict()
+    if hasattr(record, "__dict__"):
+        return {k: v for k, v in vars(record).items() if not k.startswith("_")}
+    return {"run": str(record)}
+
+
 class AgentOS:
     """
     Production platform for deploying AI agents as web services.
@@ -139,7 +149,7 @@ class AgentOS:
     
     def _register_routes(self, app: Any) -> None:
         """Register API routes."""
-        from fastapi import HTTPException
+        from fastapi import HTTPException, Query
         from pydantic import BaseModel
         
         class ChatRequest(BaseModel):
@@ -166,6 +176,123 @@ class AgentOS:
         async def health():
             return {"status": "healthy"}
         
+        # ── Read endpoints ────────────────────────────────────────────────
+        # AgentOS could list agents and take a chat turn, and nothing else. A
+        # session could not be replayed, a run inspected, or a pending approval
+        # seen over HTTP, even though the protocols to expose all three already
+        # existed. These are READ-only on purpose: mutating a run over HTTP is a
+        # much larger design question than surfacing one.
+        #
+        # Each returns 503 with what to configure when its store is absent.
+        # Returning an empty list would be indistinguishable from "no runs yet",
+        # which is the failure this codebase keeps finding.
+
+        def _unavailable(what: str, how: str):
+            raise HTTPException(
+                status_code=503,
+                detail=f"{what} is not available: {how}",
+            )
+
+        @app.get(f"{self.config.api_prefix}/runs")
+        async def list_runs(limit: int = Query(50, ge=1, le=500)):
+            ledger = getattr(self, "run_ledger", None)
+            if ledger is None:
+                _unavailable(
+                    "Run history",
+                    "no run ledger is configured on this AgentOS instance. "
+                    "Attach a praisonaiagents.runs.SQLiteRunLedger as `run_ledger`.",
+                )
+            records = ledger.list_all(limit=limit)
+            return {"runs": [_run_to_dict(r) for r in records], "count": len(records)}
+
+        @app.get(f"{self.config.api_prefix}/runs/{{run_id}}")
+        async def get_run(run_id: str):
+            ledger = getattr(self, "run_ledger", None)
+            if ledger is None:
+                _unavailable(
+                    "Run history",
+                    "no run ledger is configured on this AgentOS instance.",
+                )
+            record = ledger.get(run_id)
+            if record is None:
+                raise HTTPException(status_code=404, detail=f"No run {run_id!r}")
+            return _run_to_dict(record)
+
+        @app.get(f"{self.config.api_prefix}/sessions")
+        async def list_sessions(limit: int = Query(20, ge=1, le=200)):
+            store = getattr(self, "session_store", None)
+            if store is None or not hasattr(store, "recent"):
+                _unavailable(
+                    "Session history",
+                    "no session store is configured on this AgentOS instance. "
+                    "Attach a praisonaiagents.session store as `session_store`.",
+                )
+            summaries = store.recent(limit=limit)
+            return {
+                "sessions": [
+                    s.as_dict() if hasattr(s, "as_dict") else dict(s) for s in summaries
+                ],
+                "count": len(summaries),
+            }
+
+        @app.get(f"{self.config.api_prefix}/sessions/{{session_id}}")
+        async def get_session(session_id: str, limit: int = Query(100, ge=1, le=500)):
+            store = getattr(self, "session_store", None)
+            if store is None:
+                _unavailable(
+                    "Session history",
+                    "no session store is configured on this AgentOS instance.",
+                )
+            if hasattr(store, "session_exists"):
+                if not store.session_exists(session_id):
+                    raise HTTPException(status_code=404, detail=f"No session {session_id!r}")
+            elif not store.get_chat_history(session_id, limit):
+                # Stores without session_exists cannot distinguish an unknown
+                # session from an empty one; treat an empty transcript as absent
+                # rather than returning a healthy-looking 200 for a bad id.
+                raise HTTPException(status_code=404, detail=f"No session {session_id!r}")
+            return {
+                "session_id": session_id,
+                "messages": store.get_chat_history(session_id, limit),
+            }
+
+        @app.get(f"{self.config.api_prefix}/approvals")
+        async def list_approvals():
+            try:
+                from praisonaiagents.approval import get_approval_registry
+            except ImportError:
+                _unavailable("Approvals", "praisonaiagents.approval is not installed.")
+            registry = get_approval_registry()
+
+            # The registry stores approval requirements across four fields --
+            # global tools plus per-agent overrides -- rather than a single
+            # dict. Reading the public accessors (is_required/get_risk_level)
+            # keeps this decoupled from that private shape. Fall back to the
+            # attributes only to enumerate which (agent, tool) pairs exist.
+            global_tools = getattr(registry, "_required_tools", None)
+            agent_tools = getattr(registry, "_agent_required_tools", None)
+            if global_tools is None and agent_tools is None:
+                _unavailable(
+                    "Approvals",
+                    "this build's approval registry exposes no requirement listing.",
+                )
+
+            requirements = []
+            for tool in sorted(global_tools or ()):
+                requirements.append({
+                    "tool": tool,
+                    "agent": None,
+                    "risk_level": registry.get_risk_level(tool),
+                })
+            for agent, tools in (agent_tools or {}).items():
+                for tool in sorted(tools):
+                    requirements.append({
+                        "tool": tool,
+                        "agent": agent,
+                        "risk_level": registry.get_risk_level(tool, agent),
+                    })
+            return {"requirements": requirements}
+
         @app.get(f"{self.config.api_prefix}/agents")
         async def list_agents():
             return {

{praisonai-4.7.7 → praisonai-4.7.8}/praisonai/cli/_shim.py

@@ -112,7 +112,27 @@ def _register_submodules(old_name: str, new_name: str, module) -> None:
     new_prefix = new_name + "."
 
     # Alias whatever is already imported first (e.g. the package __init__).
-    for mod_name, mod in list(sys.modules.items()):
+    #
+    # Snapshot ``sys.modules`` defensively: a concurrent import in another
+    # thread (e.g. an async client loading a module during a parallel test) can
+    # mutate ``sys.modules`` while ``items()`` is being materialised, and
+    # CPython raises ``RuntimeError: dictionary changed size during iteration``
+    # even for ``list(sys.modules.items())``. Retry the snapshot a few times,
+    # then fall back to reading each value by key — indexing individual keys
+    # never trips the size-change guard.
+    snapshot = None
+    for _attempt in range(3):
+        try:
+            snapshot = list(sys.modules.items())
+            break
+        except RuntimeError:
+            continue
+    if snapshot is None:
+        snapshot = [
+            (name, sys.modules.get(name)) for name in list(sys.modules.keys())
+        ]
+
+    for mod_name, mod in snapshot:
         # ``sys.modules`` can legitimately hold ``None`` placeholders (failed
         # optional imports). Aliasing one would poison the old dotted name so
         # that ``import old_name.sub`` raises "None in sys.modules" even though

{praisonai-4.7.7 → praisonai-4.7.8}/praisonai/cli/features/sandbox_cli.py

@@ -201,29 +201,3 @@ class SandboxHandler:
                 print(f"  {name}: {flag}")
         except ImportError as exc:
             print(f"  Error: {exc}")
-
-
-def handle_sandbox_command(args) -> None:
-    """Handle sandbox CLI command."""
-    handler = SandboxHandler()
-    
-    subcommand = getattr(args, "sandbox_command", None) or "status"
-    
-    if subcommand == "run":
-        handler.run(
-            code=getattr(args, "code", None),
-            file=getattr(args, "file", None),
-            sandbox_type=getattr(args, "type", "subprocess"),
-            image=getattr(args, "image", "python:3.11-slim"),
-            timeout=getattr(args, "timeout", 60),
-        )
-    elif subcommand == "shell":
-        handler.shell(
-            sandbox_type=getattr(args, "type", "subprocess"),
-            image=getattr(args, "image", "python:3.11-slim"),
-        )
-    elif subcommand == "status":
-        handler.status()
-    else:
-        print(f"Unknown sandbox command: {subcommand}")
-        print("Available commands: run, shell, status")

{praisonai-4.7.7 → praisonai-4.7.8}/praisonai/db/adapter.py

@@ -376,7 +376,9 @@ class PraisonAIDB:
         if messages is None:
             return []
         
-        # Convert to DbMessage format
+        # Convert to DbMessage format. Carry the structured tool fields so a
+        # resumed tool-using session rebuilds the same transcript the model saw
+        # before (Issue #5075 — parity with the JSON store fix in #3089).
         from praisonaiagents.db.protocol import DbMessage
         return [
             DbMessage(
@@ -384,7 +386,9 @@ class PraisonAIDB:
                 content=msg.content,
                 metadata=msg.metadata or {},
                 timestamp=msg.created_at or time.time(),
-                id=msg.id
+                id=msg.id,
+                tool_calls=getattr(msg, "tool_calls", None),
+                tool_call_id=getattr(msg, "tool_call_id", None),
             )
             for msg in messages
         ]
@@ -449,6 +453,81 @@ class PraisonAIDB:
             msg,
         )
     
+    def on_assistant_message(
+        self,
+        session_id: str,
+        content: str,
+        tool_calls: Optional[List[Any]] = None,
+        metadata: Optional[Dict[str, Any]] = None,
+    ) -> None:
+        """Persist an assistant turn that requested tools (Issue #5075).
+
+        Stores the structured ``tool_calls`` verbatim on the
+        ``ConversationMessage`` so a resumed session replays the same
+        assistant turn the model produced. Falls back to a plain assistant
+        message when no tool calls are present.
+        """
+        self._init_stores()
+        if not self._conversation_store:
+            return
+
+        from ..persistence.conversation.base import ConversationMessage
+        import uuid
+
+        msg = ConversationMessage(
+            id=f"msg-{uuid.uuid4().hex[:12]}",
+            session_id=session_id,
+            role="assistant",
+            content=content,
+            tool_calls=tool_calls,
+            metadata=metadata or {},
+            created_at=time.time(),
+        )
+        self._call_store(
+            self._conversation_store,
+            "add_message",
+            "async_add_message",
+            session_id,
+            msg,
+        )
+
+    def on_tool_message(
+        self,
+        session_id: str,
+        content: str,
+        tool_call_id: str,
+        metadata: Optional[Dict[str, Any]] = None,
+    ) -> None:
+        """Persist a tool result turn linked to its call id (Issue #5075).
+
+        Keeps the ``role="tool"`` result and its ``tool_call_id`` so the
+        resumed message list interleaves results in order, matching the JSON
+        store behaviour from #3089.
+        """
+        self._init_stores()
+        if not self._conversation_store:
+            return
+
+        from ..persistence.conversation.base import ConversationMessage
+        import uuid
+
+        msg = ConversationMessage(
+            id=f"tool-{uuid.uuid4().hex[:12]}",
+            session_id=session_id,
+            role="tool",
+            content=content,
+            tool_call_id=tool_call_id,
+            metadata=metadata or {},
+            created_at=time.time(),
+        )
+        self._call_store(
+            self._conversation_store,
+            "add_message",
+            "async_add_message",
+            session_id,
+            msg,
+        )
+
     @staticmethod
     def _serialize_tool_call(tool_name: str, args: Any, result: Any) -> str:
         """Serialise a tool call for persistence without dropping data.

praisonai-4.7.8/praisonai/version.py

@@ -0,0 +1 @@
+__version__ = "4.7.8"

{praisonai-4.7.7 → praisonai-4.7.8}/pyproject.toml

@@ -12,14 +12,14 @@ dependencies = [
     "rich>=13.7",
     "markdown>=3.5",
     "pyparsing>=3.0.0",
-    "praisonaiagents>=1.7.5",
-    "praisonai-code>=0.1.4",
-    "praisonai-bot>=0.1.4",
-    "praisonai-train>=0.1.4",
-    "praisonai-browser>=0.1.4",
-    "praisonai-mcp>=0.1.6",
-    "praisonai-sandbox>=0.1.4",
-    "praisonai-deploy>=0.1.4",
+    "praisonaiagents>=1.7.6",
+    "praisonai-code>=0.1.5",
+    "praisonai-bot>=0.1.5",
+    "praisonai-train>=0.1.5",
+    "praisonai-browser>=0.1.5",
+    "praisonai-mcp>=0.1.7",
+    "praisonai-sandbox>=0.1.5",
+    "praisonai-deploy>=0.1.5",
     "python-dotenv>=0.19.0",
     "litellm>=1.83.14,<2",
     "PyYAML>=6.0",

praisonai-4.7.7/praisonai/version.py

@@ -1 +0,0 @@
-__version__ = "4.7.7"