fix(storage): surface real error when error body lacks message

[MRoshaan]

Description

In storage3, when the Storage API returns a non-2xx response whose JSON body is missing any of message, error or statusCode, the error-recovery path accessed resp.text where resp is the already-decoded dict — raising AttributeError: 'dict' object has no attribute 'text' and completely masking the real error.

Motivation

Closes #1563. The real status and body were being discarded exactly when they matter most (the error path). The fix reports the raw response text in a proper StorageApiError instead.

Changes

• src/storage3/_sync/file_api.py and src/storage3/_async/file_api.py: use exc.response.text on the recovery path.
• Added regression tests (tests/_sync/test_file_api.py, tests/_async/test_file_api.py) covering both sync and async _request.

Testing

• New tests pass for _sync and _async.
• Adjacent storage unit tests (bucket, exceptions, utils) pass: 28 passed.
• ruff check and ruff format --check clean on changed files.

Cc: cc @olirice @silentworks for review.

[Copilot]

Pull request overview

Fixes a bug in storage3 request error handling where the fallback path could raise AttributeError (masking the underlying HTTP error) when the decoded JSON body lacks expected keys. The change ensures the fallback error message uses the original HTTP response body text.

Changes:

• Update sync _request error recovery to use exc.response.text instead of resp.text.
• Update async _request error recovery to use exc.response.text instead of resp.text.
• Add sync + async regression tests covering the missing-keys error body case.

Reviewed changes

Copilot reviewed 4 out of 4 changed files in this pull request and generated no comments.

File	Description
src/storage/src/storage3/_sync/file_api.py	Fixes fallback error message to read raw response text from the HTTPX response.
src/storage/src/storage3/_async/file_api.py	Mirrors the sync fix for async requests.
src/storage/tests/_sync/test_file_api.py	Adds regression coverage to ensure StorageApiError is raised (not AttributeError) and includes body content.
src/storage/tests/_async/test_file_api.py	Adds the async equivalent regression test for _request.

---

💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.