-
Notifications
You must be signed in to change notification settings - Fork 2
refactor: consolidate set_status_tx across LLM providers #1990
Copy link
Copy link
Closed
Labels
architectureArchitecture improvementsArchitecture improvementsllmzeph-llm crate (Ollama, Claude)zeph-llm crate (Ollama, Claude)
Description
Epic: #1975
6 providers implement set_status_tx(&mut self, tx: StatusTx) identically:
- compatible.rs:68, gemini/mod.rs:148, router/mod.rs:335
- orchestrator/router.rs:36, orchestrator/mod.rs:83, any.rs:211
Options:
- Add
set_status_txtoLlmProvidertrait with default impl - Store status_tx in shared
ProviderBasestruct
Acceptance Criteria:
- Single implementation point for set_status_tx
- All provider tests pass
Complexity: Small | Effort: 1h
Ref: DRY-02
Reactions are currently unavailable
Metadata
Metadata
Assignees
Labels
architectureArchitecture improvementsArchitecture improvementsllmzeph-llm crate (Ollama, Claude)zeph-llm crate (Ollama, Claude)