fix(osis): migrate hero_rpc_osis to broker-first hero_lib AI API #38

Merged

zaelgohary merged 1 commit from development_fix_osis_ai_broker_migration into development

2026-05-03 12:19:09 +00:00

zaelgohary commented

2026-05-03 12:10:34 +00:00

Member

Summary

hero_rpc_osis used the old sync AiClient::from_env() + Model::Llama3_3_70B enum API that hero_lib bb69b40a (broker-first rewrite) removed. Without this fix, every consumer pulling fresh hero_lib breaks at compile time (12 errors in hero_rpc_osis), which is why the workaround [patch] blocks kept getting committed and reverted.

Changes

AiClient::from_env() → AiClient::default_socket().await?
Model::Llama3_3_70B / Model::from_id / Model::default_general → Model::new(...)
chat_with_options(...) → prompt().model().user().temperature().execute().await
Removed spawn_blocking wrapper (broker client is async)
MockGenerator::from_env and generate_toml are now async
seed_config_handler queries broker via list_providers() / list_models()
Bumped Cargo.lock to hero_lib b8b01eba

## Summary hero_rpc_osis used the old sync `AiClient::from_env()` + `Model::Llama3_3_70B` enum API that hero_lib `bb69b40a` (broker-first rewrite) removed. Without this fix, every consumer pulling fresh hero_lib breaks at compile time (12 errors in `hero_rpc_osis`), which is why the workaround `[patch]` blocks kept getting committed and reverted. ## Changes - `AiClient::from_env()` → `AiClient::default_socket().await?` - `Model::Llama3_3_70B` / `Model::from_id` / `Model::default_general` → `Model::new(...)` - `chat_with_options(...)` → `prompt().model().user().temperature().execute().await` - Removed `spawn_blocking` wrapper (broker client is async) - `MockGenerator::from_env` and `generate_toml` are now async - `seed_config_handler` queries broker via `list_providers()` / `list_models()` - Bumped Cargo.lock to hero_lib `b8b01eba`