space/whisper-remote
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

cli: show HTTP status/body on transcription backend failures #2

Merged
space merged 1 commits from fix/whisper-remote-http-error-message into main 2026-05-25 19:40:14 +02:00
Conversation 3 Commits 1 Files Changed 2 +21 -1
luna commented 2026-05-25 19:37:16 +02:00
Collaborator
Copy Link

Summary

  • improve CLI error reporting when backend returns non-2xx
  • include request URL, HTTP status code, and response body snippet
  • add tests covering the new error surface

Why

Previously users only saw a generic Internal Server Error, which hid actionable backend context. This change makes failures diagnosable from a single run.

Testing

  • cd cli && pytest -q
## Summary - improve CLI error reporting when backend returns non-2xx - include request URL, HTTP status code, and response body snippet - add tests covering the new error surface ## Why Previously users only saw a generic `Internal Server Error`, which hid actionable backend context. This change makes failures diagnosable from a single run. ## Testing - cd cli && pytest -q
luna added 1 commit 2026-05-25 19:37:17 +02:00
cli: surface HTTP status and body on backend errors
All checks were successful
CI / Backend (pull_request) Successful in 32s
Details
CI / CLI (pull_request) Successful in 10s
Details
2b1c26781e
space commented 2026-05-25 19:38:15 +02:00
Owner
Copy Link

@codex review

@codex review
gitea-codex commented 2026-05-25 19:38:17 +02:00
First-time contributor
Copy Link

👀 Codex review queued for commit 2b1c267.

👀 Codex review queued for commit `2b1c267`.
gitea-codex commented 2026-05-25 19:39:05 +02:00
First-time contributor
Copy Link

No PR-introduced issues found in the specified diff. The CLI error-formatting change is internally consistent and covered by new unit tests; the README/docker-compose changes are documentation/infrastructure removal only and do not introduce code defects in the reviewed scope.

Note: model gpt-5.3-codex, input 32913, output 1173 tokens used.

ℹ️.codex-review.yml is not configured

<!-- codex-review:head_sha=2b1c26781e14a7f09161f057b6d8fde59697abe8 --> No PR-introduced issues found in the specified diff. The CLI error-formatting change is internally consistent and covered by new unit tests; the README/docker-compose changes are documentation/infrastructure removal only and do not introduce code defects in the reviewed scope. _Note: model `gpt-5.3-codex`, input `32913`, output `1173` tokens used._ > ℹ️.codex-review.yml is not configured
space merged commit 44af756bd3 into main 2026-05-25 19:40:14 +02:00
space deleted branch fix/whisper-remote-http-error-message 2026-05-25 19:42:28 +02:00
Sign in to join this conversation.
Reviewers
No Reviewers
Labels
No items
No Label
Milestone
No items
No Milestone
Projects
Clear projects
No project
Assignees
Clear assignees
luna space
No Assignees
3 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: space/whisper-remote#2
Delete Branch "fix/whisper-remote-http-error-message"

Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?