Skip to content

Remove base_client.py from .fernignore#60

Merged
adeelehsan merged 1 commit intomainfrom
fix/remove-base-client-from-fernignore
Apr 20, 2026
Merged

Remove base_client.py from .fernignore#60
adeelehsan merged 1 commit intomainfrom
fix/remove-base-client-from-fernignore

Conversation

@adeelehsan
Copy link
Copy Markdown
Contributor

Summary

Removes base_client.py from .fernignore so Fern can fully regenerate it.

Why

Fern now supports api_key as a native auth scheme via the auth: any config added in vectara/docs#565. This means base_client.py no longer needs our custom api_key-only authentication path — Fern will generate it natively with:

auth-schemes:
  OAuthScheme:
    scheme: oauth
    ...
  ApiKey:
    header: x-api-key
    type: string
    env: VECTARA_API_KEY
api:
  auth:
    any:
      - OAuthScheme
      - ApiKey

Merge order

  1. Merge docs PR vectara/docs#565 (auth config fix)
  2. Merge this PR
  3. Trigger Fern SDK generation — base_client.py will be regenerated with native api_key + OAuth support

Test plan

  • After Fern regeneration, verify Vectara(api_key="...") works
  • Verify Vectara(client_id="...", client_secret="...") still works
  • Run SDK test suite from api_test_suite

🤖 Generated with Claude Code

Fern now supports api_key as a native auth scheme via the `auth: any`
config (docs PR #565). This means base_client.py no longer needs custom
code for api_key-only authentication — Fern will generate it natively.

After the docs PR is merged and Fern regenerates, base_client.py will
include both OAuth and ApiKey auth paths out of the box.

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
@adeelehsan adeelehsan merged commit 61ab665 into main Apr 20, 2026
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