Skip to content

fix(agent): clarify auxiliary OAuth provider credential errors - #703

Closed
hashbender wants to merge 1 commit into
mainfrom
mirror/pr-56863
Closed

fix(agent): clarify auxiliary OAuth provider credential errors#703
hashbender wants to merge 1 commit into
mainfrom
mirror/pr-56863

Conversation

@hashbender

Copy link
Copy Markdown
Owner

Summary

  • Add credential-type-aware errors when explicit auxiliary provider client creation returns no client.
  • Mention Vertex ADC/OAuth/service-account setup and google-auth instead of suggesting a fake VERTEX_API_KEY.
  • Preserve existing API-key provider guidance for API-key-backed providers.

Fixes NousResearch#56810

Tests

  • scripts/run_tests.sh tests/agent/test_auxiliary_client.py -k 'test_vertex_unavailable_mentions_adc_oauth_not_fake_api_key' -v --tb=short (RED before fix: failed on VERTEX_API_KEY)
  • scripts/run_tests.sh tests/agent/test_auxiliary_client.py -k 'TestExplicitProviderUnavailableMessages' -v --tb=short (2 passed)
  • scripts/run_tests.sh tests/agent/test_auxiliary_client.py -v --tb=short (283 passed)
  • git diff --check

Mirror-of: NousResearch#56863
NousResearch#56863

@hashbender hashbender closed this Jul 2, 2026
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.

Misleading "API key was found" RuntimeError for non-API-key OAuth/ADC providers in auxiliary tasks

1 participant