lhumina_code/hero_aibroker
24
0
Fork 0
Code Issues 4 Pull requests Projects Releases 1 Packages Wiki Activity Actions

fix(server): add hero_proxy support via web_v1.sock and dual route registration #46

Merged
fatmaebrahim merged 4 commits from development_hero_proxy_rest_support into development 2026-04-26 07:29:03 +00:00
Conversation 0 Commits 4 Files changed 23 +398 -266
fatmaebrahim commented 2026-04-21 11:39:40 +00:00
Member
Copy link

Summary

  • Add web_v1.sock proxy socket alongside existing rest.sock for hero_proxy routing
  • Dual route registration: root (for hero_proxy which strips /hero_aibroker/v1) and /v1 prefix (for direct socket access)
  • Force wav format and truncate input for Groq Orpheus TTS backend

Related Issue

Closes #45

Changes

  • crates/hero_aibroker_server/src/main.rs -- added proxy_socket_path() for web_v1.sock, server binds both sockets
  • crates/hero_aibroker_server/src/api/mod.rs -- split API routes into v1_routes, registered at root via .merge() and under /v1 via .nest()
  • crates/hero_aibroker_lib/src/service/tts.rs -- Groq backends force wav format, truncate input to 1000 chars

Test Results

  • Build: OK
  • Unit/doc tests: all passed
  • Manual verification: both proxy and direct socket access return 200 with valid WAV audio
## Summary - Add `web_v1.sock` proxy socket alongside existing `rest.sock` for hero_proxy routing - Dual route registration: root (for hero_proxy which strips `/hero_aibroker/v1`) and `/v1` prefix (for direct socket access) - Force `wav` format and truncate input for Groq Orpheus TTS backend ## Related Issue Closes https://forge.ourworld.tf/lhumina_code/hero_aibroker/issues/45 ## Changes - `crates/hero_aibroker_server/src/main.rs` -- added `proxy_socket_path()` for `web_v1.sock`, server binds both sockets - `crates/hero_aibroker_server/src/api/mod.rs` -- split API routes into `v1_routes`, registered at root via `.merge()` and under `/v1` via `.nest()` - `crates/hero_aibroker_lib/src/service/tts.rs` -- Groq backends force `wav` format, truncate input to 1000 chars ## Test Results - Build: OK - Unit/doc tests: all passed - Manual verification: both proxy and direct socket access return 200 with valid WAV audio
fix(server): add hero_proxy support via web_v1.sock and dual route registration
Some checks failed
Build and Test / build-and-test (pull_request) Failing after 2m7s
Details
4a84a375c6 
#45
fix(test): correct RPC socket path in integration test
Some checks failed
Build and Test / build-and-test (pull_request) Failing after 1m36s
Details
83f5aae3e2 
#45
style: cargo fmt --all
Some checks failed
Build and Test / build-and-test (pull_request) Failing after 1m39s
Details
de6e9a3b45 
#45
fix: resolve clippy warnings and format all files
All checks were successful
Build and Test / build-and-test (pull_request) Successful in 3m40s
Details
0b3f086efd 
Remove unused BasePath/HeroContext/HeroClaims structs, fix collapsible_if
in app components, prefix unused variable in navbar.

#45
fatmaebrahim merged commit eccda4e629 into development 2026-04-26 07:29:03 +00:00
fatmaebrahim deleted branch development_hero_proxy_rest_support 2026-04-26 07:29:03 +00:00
Sign in to join this conversation.
Reviewers
No reviewers
Labels
Clear labels   
prio_critical

   
prio_low

   
type_bug

   
type_contact

   
type_issue

   
type_lead

   
type_question

   
type_story

   
type_task
No labels
prio_critical
prio_low
type_bug
type_contact
type_issue
type_lead
type_question
type_story
type_task
Milestone
Clear milestone
No items
No milestone
Projects
Clear projects
No items
No project
Assignees
Clear assignees
No assignees
1 participant
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference
lhumina_code/hero_aibroker!46
No description provided.