Fix Zhipu ASR endpoint handling

[H-Chris233]

User description

Summary

• Refs zhipu ASR: 404 due to doubled API path + errors invisible in app UI #507.
• Normalize Whisper/Zhipu ASR transcription URLs so /v4, /v4/audio, and full /v4/audio/transcriptions endpoints all post to the correct endpoint without doubling the path.
• Preserve query parameters on custom transcription endpoints.
• Fix a macOS CI-only test-server race by returning accepted test sockets to blocking mode before reading the multipart request.

Not Included

• This PR intentionally does not change Linux ASR error visibility/UI behavior.

Test Plan

• cargo test --manifest-path \"/home/chris233/openless/openless-all/app/src-tauri/Cargo.toml\" transcription_url_accepts_base_audio_or_full_endpoint -- --test-threads=1
• cargo test --manifest-path \"/home/chris233/openless/openless-all/app/src-tauri/Cargo.toml\" asr::whisper::tests::transcribe_posts_single_request_without_chunk_limit -- --test-threads=1
• git diff --check upstream/beta

Review

• Subagent review: APPROVE

---

PR Type

Bug fix, Tests

---

Description

• Normalize Zhipu ASR transcription URLs

• Avoid doubled /audio/transcriptions paths

• Preserve query strings on custom endpoints

• Fix macOS ASR test socket blocking

---

Diagram Walkthrough

flowchart LR
  A["ASR base URL"] -- "normalize" --> B["transcription_url()"]
  B -- "used by" --> C["Whisper transcription request"]
  B -- "verified by" --> D["URL normalization tests"]
  E["Accepted test socket"] -- "set blocking" --> F["Multipart request reader"]

Loading

File Walkthrough

	Relevant files
Bug fix	whisper.rs Normalize ASR URLs and strengthen tests                                    openless-all/app/src-tauri/src/asr/whisper.rs Replaces manual URL concatenation with transcription_url() Prevents duplicated /audio/transcriptions for Zhipu endpoints Preserves query parameters when rebuilding transcription URLs Adds regression coverage and fixes test socket blocking +42/-2

---

[github-actions]

PR Reviewer Guide 🔍

(Review updated until commit 7b39491)

Here are some key observations to aid the review process:

🎫 Ticket compliance analysis 🔶 507 - Partially compliant Compliant requirements: Prevent doubled /audio/transcriptions paths. Accept the transcription endpoint forms and preserve query parameters. Non-compliant requirements: Surface transcription failures in the app UI. Requires further human verification: None

⏱️ Estimated effort to review: 2 🔵🔵⚪⚪⚪

🧪 PR contains tests

🔒 No security concerns identified

⚡ No major issues detected

[github-actions]

Persistent review updated to latest commit 7b39491