-
Notifications
You must be signed in to change notification settings - Fork 2
config: unify bootstrap — single build_provider_from_entry() #2137
Copy link
Copy link
Closed
Labels
configConfiguration file changesConfiguration file changescorezeph-core cratezeph-core crateenhancementNew feature or requestNew feature or request
Description
Part of #2134 — Phase 3.
Spec: .local/specs/022-config-simplification/spec.md § Phase 3
Goal
Replace 6 parallel provider construction paths in bootstrap/provider.rs with a single function.
Current State
bootstrap/provider.rs contains:
named_claude()— reads[llm.cloud]+ secretspcfg_claude()— readsOrchestratorProviderConfig+ fallback to[llm.cloud]build_sub_provider("claude")— same with different entry pointsummary_claude()— reads model_spec string + fallback to[llm.cloud]- Equivalent 4-way duplication for OpenAI, Gemini, Ollama
Tasks
- Implement
build_provider_from_entry(&ProviderEntry, &Config) -> Result<AnyProvider> - Rewrite
create_provider()to dispatch onRoutingStrategyinstead ofProviderKind - Remove all
named_*,pcfg_*,build_sub_provider,summary_*functions - Update
zeph-core/src/config.rsre-exports accordingly
Acceptance
create_provider()uses onlybuild_provider_from_entry()- LLM serialization gate: live session test with orchestrator config before merge
- All 6397 tests green
Reactions are currently unavailable
Metadata
Metadata
Assignees
Labels
configConfiguration file changesConfiguration file changescorezeph-core cratezeph-core crateenhancementNew feature or requestNew feature or request