oci-api-mcp-server crashes if the output from the ocli cli isn't json

[AmedeeBulle]

Some oci cli commands are returning plain text -- E.g.:

$ oci iam db-token get --scope "${iot_db_token_scope}"       
Private key written at /home/<redacted>/.oci/db-token/oci_db_key.pem
db-token written at: /home/<redacted>/.oci/db-token/token
db-token is valid until 2025-10-20 23:13:48

The oci-api-mcp-server runs json.loads(result.stdout) without safeguard, generating a stack trace when it is not json:

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

Line 161 in c44d4a3

json.loads(result.stdout)

Possible fix would be to to wrap the output in json when it is not serializable -- e.g. { "response" : result.stdout}

[gebhardtr]

Thank you @AmedeeBulle, nice catch. Does this work for you? https://github.com/oracle/mcp/blob/main/src/oci-api-mcp-server/oracle/oci_api_mcp_server/server.py#L160-L166

[AmedeeBulle]

@gebhardtr I think your link points to the original code...

What I have done locally as workaround is:

        if result.stdout:
            try:
                return json.loads(result.stdout)
            except Exception:
                return {
                    "response": result.stdout
                }
        else:
            return {
                "error": result.stderr,
            }