Skip to content

Support GCS paths for server and tool instructions#183

Merged
keyurva merged 3 commits intodatacommonsorg:mainfrom
keyurva:gcs
Apr 3, 2026
Merged

Support GCS paths for server and tool instructions#183
keyurva merged 3 commits intodatacommonsorg:mainfrom
keyurva:gcs

Conversation

@keyurva
Copy link
Copy Markdown
Contributor

@keyurva keyurva commented Apr 3, 2026

Copy link
Copy Markdown
Contributor

@gemini-code-assist gemini-code-assist bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Code Review

This pull request introduces support for reading external content from Google Cloud Storage (GCS) in addition to local file paths. Key changes include the implementation of GCS-specific utility functions for path parsing and content retrieval, the addition of the google-cloud-storage dependency, and new unit tests to verify GCS integration. Feedback was provided regarding the GCS path parsing logic, specifically to handle potential double slashes from trailing slashes in the base path and to ensure the bucket name is not empty.

@keyurva keyurva requested a review from clincoln8 April 3, 2026 07:02
Copy link
Copy Markdown
Contributor

@clincoln8 clincoln8 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Nice!!

@keyurva keyurva merged commit 60694c8 into datacommonsorg:main Apr 3, 2026
8 of 9 checks passed
@keyurva keyurva deleted the gcs branch April 3, 2026 18:02
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.

2 participants