Skip to content

feat(python): add OciClient to additional_init_exports#712

Merged
daniel-cohere merged 1 commit intocohere-ai:mainfrom
fede-kamel:feat/add-oci-init-exports
Apr 9, 2026
Merged

feat(python): add OciClient to additional_init_exports#712
daniel-cohere merged 1 commit intocohere-ai:mainfrom
fede-kamel:feat/add-oci-init-exports

Conversation

@fede-kamel
Copy link
Copy Markdown
Contributor

@fede-kamel fede-kamel commented Mar 24, 2026

Summary

Add OciClient and OciClientV2 to the Python SDK's additional_init_exports in fern/apis/sdks/generators.yml, so they survive Fern regeneration of __init__.py.

This is a one-line config change that follows the exact same pattern as BedrockClient, SagemakerClient, and AwsClient.

Why

oci_client.py is a manually maintained file (listed in .fernignore in cohere-python). But __init__.py is auto-generated by Fern — without this config entry, the OciClient/OciClientV2 exports get wiped on every regeneration, breaking import cohere; cohere.OciClientV2(...).

Related

Change

- from: oci_client
  imports:
    - OciClient
    - OciClientV2

Add OCI client exports so they survive Fern regeneration of __init__.py.
Follows the same pattern as BedrockClient/SagemakerClient.

Related: cohere-ai/cohere-python#718
@fede-kamel fede-kamel requested review from a team as code owners March 24, 2026 12:29
@github-actions
Copy link
Copy Markdown

@fede-kamel
Copy link
Copy Markdown
Contributor Author

@mkozakov @fern-support Ready for review.

This is the companion config change for cohere-ai/cohere-python#718 (OCI client). Without this, OciClient/OciClientV2 get wiped from __init__.py on every Fern regeneration.

One-line change — same pattern as BedrockClient/SagemakerClient.

Tracking issue: cohere-ai/cohere-python#735

@fede-kamel
Copy link
Copy Markdown
Contributor Author

@fern-support is this something you can merge?

@daniel-cohere daniel-cohere merged commit 68b30c7 into cohere-ai:main Apr 9, 2026
1 check 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.

3 participants