Skip to content

fix(routing): union configured and known providers#3

Merged
coniferconcepts merged 1 commit intomainfrom
fix/routing-known-provider-union
Mar 14, 2026
Merged

fix(routing): union configured and known providers#3
coniferconcepts merged 1 commit intomainfrom
fix/routing-known-provider-union

Conversation

@coniferconcepts
Copy link
Copy Markdown
Owner

@coniferconcepts coniferconcepts commented Mar 14, 2026

Summary

  • treat a provider as valid for routing override validation when it appears in either the configured provider set or the loaded plugin set
  • fix a standard-launcher startup edge case where configured providers like ollama could be rejected as unknown when known_providers came from a narrower plugin list
  • add a focused regression test covering the exact available-vs-known mismatch that the local runtime exposed after the v1/v2 routing work merged

Testing

  • pytest tests/test_routing_policy.py tests/test_client_routing_policy.py
  • python3 -m py_compile "src/rotator_library/routing_policy.py" "tests/test_routing_policy.py"

Guardrails

  • no routing selection semantics change beyond provider validation
  • weighted and single routing behavior remain unchanged after validation succeeds
  • this only widens provider recognition to the safe union of configured providers and loaded providers

Runtime Context

  • this follow-up came from a real local ./scripts/start-all.sh failure
  • the local runtime now starts cleanly with the merged v1/v2 overrides after this fix

@coniferconcepts coniferconcepts merged commit 55e9275 into main Mar 14, 2026
1 of 3 checks passed
@coniferconcepts coniferconcepts deleted the fix/routing-known-provider-union branch March 14, 2026 16:02
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant