Skip to content

Conversation

@wilsonccccc
Copy link
Contributor

@wilsonccccc wilsonccccc commented Aug 10, 2025

What Changed

  • Simplified provider name extraction logic by leveraging database-level intersection
  • Removed redundant adapter factory checks (user's provider keys are the authoritative source)
  • Enhanced support for custom providers not in the adapter factory

Key Improvements

  • Performance: Eliminated redundant set intersection operations by using pre-filtered allowed_provider_names
  • Security: allowed_provider_names already represents the intersection of user's provider keys and API key scope
  • Maintainability: Single function handles both optimized and comprehensive provider lookup
  • Custom Providers: Now works with any custom provider configured by users

@wilsonccccc wilsonccccc merged commit 844c313 into TensorBlock:main Aug 10, 2025
3 checks passed
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