Remove silent exception swallowing in agent adapter gather_config()

The except Exception: pass blocks in smolagents, langgraph, and
llamaindex adapters silently swallowed errors during config collection,
causing loss of configuration data (including model_id) with no
visibility into why. For research code, truthfulness matters more than
uptime — let errors propagate so they can be diagnosed and fixed.

Also simplified SmolAgentAdapter.gather_config() by removing the
manual fallback that partially reconstructed config when to_dict()
failed, since the error now surfaces directly.

Author: cemde

maseval/interface/agents/langgraph.py

@@ -218,17 +218,15 @@ def gather_config(self) -> dict[str, Any]:
                     safe_config["configurable"] = {"has_thread_id": "thread_id" in value if isinstance(value, dict) else False}
             langgraph_config["config"] = safe_config
 
-        # Try to get graph structure info
+        # Get graph structure info — let errors propagate so they're
+        # visible in the registry's error output.
         if hasattr(self.agent, "get_graph"):
-            try:
-                graph = self.agent.get_graph()
-                if graph:
-                    langgraph_config["graph_info"] = {
-                        "num_nodes": len(graph.nodes) if hasattr(graph, "nodes") else None,
-                        "num_edges": len(graph.edges) if hasattr(graph, "edges") else None,
-                    }
-            except Exception:
-                pass
+            graph = self.agent.get_graph()
+            if graph:
+                langgraph_config["graph_info"] = {
+                    "num_nodes": len(graph.nodes) if hasattr(graph, "nodes") else None,
+                    "num_edges": len(graph.edges) if hasattr(graph, "edges") else None,
+                }
 
         if langgraph_config:
             base_config["langgraph_config"] = langgraph_config

maseval/interface/agents/llamaindex.py

@@ -211,14 +211,12 @@ def gather_config(self) -> Dict[str, Any]:
                     for tool in tools
                 ]
 
-        # Check if it's a workflow
+        # Check if it's a workflow — let errors propagate so they're
+        # visible in the registry's error output.
         if hasattr(self.agent, "get_config"):
-            try:
-                workflow_config = self.agent.get_config()
-                if workflow_config:
-                    llamaindex_config["workflow_config"] = workflow_config
-            except Exception:
-                pass
+            workflow_config = self.agent.get_config()
+            if workflow_config:
+                llamaindex_config["workflow_config"] = workflow_config
 
         if llamaindex_config:
             base_config["llamaindex_config"] = llamaindex_config

maseval/interface/agents/smolagents.py

@@ -425,32 +425,11 @@ def gather_config(self) -> dict[str, Any]:
         base_config = super().gather_config()
         _check_smolagents_installed()
 
-        # Get comprehensive config from smolagents' native to_dict() method
-        smolagents_config = {}
+        # Get comprehensive config from smolagents' native to_dict() method.
+        # No try/except: if to_dict() exists but fails, the error should
+        # propagate so it's visible in the registry's error output.
         if hasattr(self.agent, "to_dict"):
-            try:
-                smolagents_config = self.agent.to_dict()
-            except Exception:
-                # If to_dict fails, fall back to basic attributes
-                pass
-
-        # Add smolagents-specific config if available
-        if smolagents_config:
-            base_config["smolagents_config"] = smolagents_config
-        else:
-            # Fallback: manually collect common attributes
-            config_attrs = {}
-            for attr in ["max_steps", "planning_interval", "name", "description"]:
-                if hasattr(self.agent, attr):
-                    config_attrs[attr] = getattr(self.agent, attr)
-
-            # CodeAgent-specific attributes
-            for attr in ["additional_authorized_imports", "authorized_imports", "executor_type"]:
-                if hasattr(self.agent, attr):
-                    config_attrs[attr] = getattr(self.agent, attr)
-
-            if config_attrs:
-                base_config["smolagents_config"] = config_attrs
+            base_config["smolagents_config"] = self.agent.to_dict()
 
         return base_config