Skip to content

feat: Support custom scopes and maxQueryResultRows#77

Merged
averikitsch merged 4 commits intomainfrom
shobsi-patch-1
Jan 29, 2026
Merged

feat: Support custom scopes and maxQueryResultRows#77
averikitsch merged 4 commits intomainfrom
shobsi-patch-1

Conversation

@shobsi
Copy link
Copy Markdown
Contributor

@shobsi shobsi commented Jan 28, 2026

After this change the user can optionally control authorization scopes for BigQuery interaction and max rows for query results via environment variables, e.g.:

$ export BIGQUERY_SCOPES="https://www.googleapis.com/auth/bigquery,https://www.googleapis.com/auth/drive"

$ export BIGQUERY_MAX_QUERY_RESULT_ROWS=50

This change addresses #76

After this change the user can optionally control authorization scopes for BigQuery interaction and max rows for query results via environment variables, e.g.:

```shell
$ export BIGQUERY_SCOPES="https://www.googleapis.com/auth/bigquery,https://www.googleapis.com/auth/drive"

$ export BIGQUERY_MAX_QUERY_RESULT_ROWS=50
```
@shobsi shobsi requested review from a team as code owners January 28, 2026 22:06
@github-actions github-actions bot requested a review from jiaxunwu January 28, 2026 22:06
Copy link
Copy Markdown
Contributor

@averikitsch averikitsch left a comment

Choose a reason for hiding this comment

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

Can you add these settings to the README?
We can also add them to settings in https://github.com/gemini-cli-extensions/bigquery-conversational-analytics/blob/main/gemini-extension.json so the CLI prompts the users to configure these as well.

Add optional environment variables for BigQuery configuration.
@shobsi
Copy link
Copy Markdown
Contributor Author

shobsi commented Jan 28, 2026

Can you add these settings to the README? We can also add them to settings in https://github.com/gemini-cli-extensions/bigquery-conversational-analytics/blob/main/gemini-extension.json so the CLI prompts the users to configure these as well.

Done, thanks!

@shobsi
Copy link
Copy Markdown
Contributor Author

shobsi commented Jan 28, 2026

Can you add these settings to the README? We can also add them to settings in https://github.com/gemini-cli-extensions/bigquery-conversational-analytics/blob/main/gemini-extension.json so the CLI prompts the users to configure these as well.

Done, thanks!

I meant done in README. Since these are optional, we don't need to add them in gemini-extension.json? @averikitsch

@shobsi
Copy link
Copy Markdown
Contributor Author

shobsi commented Jan 28, 2026

Can you add these settings to the README? We can also add them to settings in https://github.com/gemini-cli-extensions/bigquery-conversational-analytics/blob/main/gemini-extension.json so the CLI prompts the users to configure these as well.

Done, thanks!

I meant done in README. Since these are optional, we don't need to add them in gemini-extension.json? @averikitsch

Never mind, I see optional can go there too, so added. PTAL, thanks!

@averikitsch averikitsch merged commit c38438e into main Jan 29, 2026
7 checks passed
@averikitsch averikitsch deleted the shobsi-patch-1 branch February 18, 2026 17:58
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