fix: properly handle malformed command output

Signed-off-by: Richard Gebhardt <[email protected]>

Author: gebhardtr

src/oci-api-mcp-server/oracle/oci_api_mcp_server/server.py

@@ -161,17 +161,27 @@ def run_oci_command(
             check=True,
             shell=False,
         )
-        return (
-            json.loads(result.stdout)
-            if result.stdout
-            else {
-                "error": result.stderr,
-            }
-        )
+
+        result.check_returncode()
+
+        response = {
+            "stdout": result.stdout,
+            "stderr": result.stderr,
+            "returncode": result.returncode,
+        }
+
+        if response["stdout"]:
+            try:
+                response["stdout"] = json.loads(response["stdout"])
+            except json.JSONDecodeError as e:
+                pass
+        return response
     except subprocess.CalledProcessError as e:
         return {
-            "error": e.stderr,
-            "output": e.stdout,
+            "command": e.cmd,
+            "stderr": e.stderr,
+            "stdout": e.stdout,
+            "returncode": e.returncode,
         }