fix: resolve registry regressions and remove duplicate Gemini adapter method

Agent-Logs-Url: https://github.com/MervinPraison/PraisonAI/sessions/5a46a1b0-a487-465c-bb7f-e56c29673ba7

Co-authored-by: MervinPraison <454862+MervinPraison@users.noreply.github.com>

Author: Copilot

src/praisonai-agents/praisonaiagents/agent/agent.py

@@ -211,6 +211,21 @@ def get_registered_agents(self, port: int) -> dict:
         with self._lock:
             return self._registered_agents.get(port, {}).copy()
 
+    def cleanup_agent_registrations(self, agent_id: str) -> None:
+        """Remove all registrations for an agent ID and clean empty port state."""
+        with self._lock:
+            ports_to_clean = []
+            for port, path_dict in self._registered_agents.items():
+                paths_to_remove = [path for path, registered_id in path_dict.items() if registered_id == agent_id]
+                for path in paths_to_remove:
+                    del path_dict[path]
+                if not path_dict:
+                    ports_to_clean.append(port)
+
+            for port in ports_to_clean:
+                self._registered_agents.pop(port, None)
+                self._server_started.pop(port, None)
+
 # Backward compatibility - use default instance
 def _get_default_server_registry() -> ServerRegistry:
     return ServerRegistry.get_default_instance()
@@ -4611,28 +4626,7 @@ def _cleanup_server_registrations(self) -> None:
             return  # No ID generated, nothing registered
 
         try:
-            agent_id = self._agent_id
-            with _server_lock:
-                # Remove from _registered_agents
-                ports_to_clean = []
-                for port, path_dict in _registered_agents.items():
-                    paths_to_remove = []
-                    for path, registered_id in path_dict.items():
-                        if registered_id == agent_id:
-                            paths_to_remove.append(path)
-                    
-                    for path in paths_to_remove:
-                        del path_dict[path]
-                    
-                    # If no paths left for this port, mark port for cleanup
-                    if not path_dict:
-                        ports_to_clean.append(port)
-                
-                # Clean up empty port entries
-                for port in ports_to_clean:
-                    _registered_agents.pop(port, None)
-                    _server_started.pop(port, None)
-                    # Note: We don't clean up _shared_apps here as other agents might be using them
+            _get_default_server_registry().cleanup_agent_registrations(self._agent_id)
 
         except Exception as e:
             import sys
@@ -4669,4 +4663,3 @@ def is_closed(self) -> bool:
 
     def __str__(self):
         return f"Agent(name='{self.name}', role='{self.role}', goal='{self.goal}')"
-

src/praisonai-agents/praisonaiagents/llm/adapters/__init__.py

@@ -209,9 +209,6 @@ def format_tools(self, tools: List[Dict[str, Any]]) -> List[Dict[str, Any]]:
     def supports_streaming_with_tools(self) -> bool:
         # Gemini has issues with streaming + tools
         return False
-    
-    def supports_structured_output(self) -> bool:
-        return True
 
 
 # Provider adapter registry - public for extension
@@ -285,4 +282,4 @@ def has_provider_adapter(name: str) -> bool:
     'add_provider_adapter',
     'list_provider_adapters',
     'has_provider_adapter',
-]
+]

src/praisonai-agents/praisonaiagents/memory/adapters/registry.py

@@ -74,7 +74,7 @@ def add_memory_factory(name: str, factory_func: Callable[..., MemoryProtocol]) -
 
 def has_memory_adapter(name: str) -> bool:
     """Canonical alias for is_available (preferred naming per AGENTS.md)."""
-    return _memory_registry.is_available(name)
+    return get_default_memory_registry().is_available(name)
 
 
 def get_first_available_memory_adapter(
@@ -93,7 +93,7 @@ def get_first_available_memory_adapter(
     if preferences is None:
         preferences = ["sqlite", "in_memory"]
 
-    return _memory_registry.get_first_available(preferences, **kwargs)
+    return get_default_memory_registry().get_first_available(preferences, **kwargs)
 
 
 __all__ = [