fix(odc-cli): report the status for an error response with an empty body

[jay23606]

Fixes #1779

Summary

• `HttpResponse.to_json` merged the body into the output with `props.update(**self.data)` on a non-2xx status. An empty error body is stored as `data=None`, so the CLI crashed with `TypeError: dict.update() argument after ** must be a mapping, not NoneType`.
• When the error body is not an object, the output now carries the `statusCode`. Object bodies are merged as before.

Test

• `cli/` has no test harness, and AGENTS.md says nothing checks it on a pull request, so I verified it by loading the script directly:
  • `HttpResponse(data=None, status=502).to_json(include_data=False)` gives `{"statusCode": 502}`. The original script raises the `TypeError`.
  • An object error body still merges its fields as before.
• Not run: the rest of the repo's checks, which do not cover `cli/`.

🤖 Generated with Claude Code

[gdevenyi]

Thank you for this fix. The crash is gone: I ran the CLI against a server that returns an empty 502, and it now prints {"statusCode": 502} instead of the traceback. Two changes are necessary before we can merge:

1. The commit scope fails Commitlint. fix(odc-cli): ... uses the scope odc-cli, but commitlint.config.ts accepts only workspace names in scope-enum, and cli/ is not a pnpm workspace. The Commitlint CI job will fail on this commit. Please reword it to fix: report the status for an error response with an empty body (no scope) and force-push.

2. The status code is still lost for an error body that is an object. In cli/odc-cli:216-221, statusCode is added only when the body is not a dict. A proxy that returns 502 with an HTML body goes through _build_response, which stores {'error': 'Invalid JSON response'}. That is a dict, so the output is {"error": "Invalid JSON response"} with no status. Issue #1779 asks for statusCode on every failure. Please set props['statusCode'] = self.status or 'N/A' for every non-2xx response, then merge self.data on top when it is a dict, so an API body that has its own statusCode still wins. For a body that is not a dict, also add error from http.HTTPStatus(self.status).phrase (catch ValueError for a status code that is not standard), so the output matches the expected {"statusCode": 405, "error": "Method Not Allowed"} in the issue. http is in the standard library and works on Python 3.8.

If you hand this to Claude Code, Sonnet 5 is the right size, because the change is in one file and the items above specify all of it.

Reviewed at commit 4dbfdae.