fix: Handle non-dict flaws and HTTP error responses

1. Flaws can be strings or dicts - check type before .get()
2. Check response.ok for all HTTP errors, not just 402

Author: drkavner

agents/connectors/cogdx.py

@@ -70,6 +70,14 @@ def analyze_reasoning(self, reasoning_trace: str) -> Dict[str, Any]:
                     "logical_validity": None
                 }
 
+            # Handle other HTTP errors (500, 403, 429, etc.)
+            if not response.ok:
+                return {
+                    "error": f"http_{response.status_code}",
+                    "message": f"API returned status {response.status_code}",
+                    "logical_validity": None
+                }
+            
             return response.json()
 
         except Exception as e:
@@ -190,10 +198,18 @@ def verify_before_trade(
         else:
             recommendation = "reject"
 
+        # Handle flaws as either dicts or strings
+        issues = []
+        for f in flaws:
+            if isinstance(f, dict):
+                issues.append(f.get("name", str(f)))
+            else:
+                issues.append(str(f))
+        
         return {
             "approved": approved,
             "validity_score": validity,
-            "issues": [f.get("name", str(f)) for f in flaws],
+            "issues": issues,
             "recommendation": recommendation
         }