Skip to content

[#384] Reorganize exports by named exports - agents-hosting-storage#559

Merged
tracyboehrer merged 3 commits intomainfrom
southworks/update/public-api-storage
Oct 10, 2025
Merged

[#384] Reorganize exports by named exports - agents-hosting-storage#559
tracyboehrer merged 3 commits intomainfrom
southworks/update/public-api-storage

Conversation

@ceciliaavila
Copy link
Copy Markdown
Collaborator

@ceciliaavila ceciliaavila commented Sep 4, 2025

Addresses #384

Description

This PR removes from the public API classes and methods that should be private because they are intended for internal use.

NOTE:
This PR introduces breaking changes, as previously public members are now inaccessible.

Detailed Changes

  • agents-hosting-storage-blob
    • Removed export clause in maybeCast function.
    • Prevented the export of the sanitizeBlobKey function by implementing selective exports in blobsTranscriptStore.ts.
  • agents-hosting-storage-cosmos
    • Made initialize function private as it's intended for internal use only.
    • Avoided exporting the DoOnce class by selectively exporting members in cosmosDbPartitionedStorage.ts.
  • Updated API docs to reflect the changes.

Testing

These images show two samples working. One using BlobStorage as transcriptMiddleware and the other using CosmosDb as the application storage.
image

@tracyboehrer tracyboehrer enabled auto-merge (squash) October 10, 2025 12:13
@tracyboehrer tracyboehrer merged commit a21b5e6 into main Oct 10, 2025
7 checks passed
@tracyboehrer tracyboehrer deleted the southworks/update/public-api-storage branch October 10, 2025 12:15
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants