Uh oh!
There was an error while loading. Please reload this page.
feat(bigquery): validate integration + add dataset/table lifecycle tools - #5480
Conversation
…et/table lifecycle + query-result tools - Cross-checked existing query/list_datasets/list_tables/get_table/insert_rows tools against BigQuery REST v2 docs; no critical issues found - Added 6 new tools covered by the existing bigquery OAuth scope: create/delete dataset, create/delete table, list table data, get query results - Wired new operations into the block (subblocks, conditions, tools.config), registry, and barrel exports
The latest updates on your projects. Learn more about Vercel for GitHub. |
PR SummaryMedium Risk Overview The block gains new operation dropdown entries, operation-grouped subblocks (including Reviewed by Cursor Bugbot for commit 6ca7101. Configure here. |
Uh oh!
There was an error while loading. Please reload this page.
Uh oh!
There was an error while loading. Please reload this page.
Uh oh!
There was an error while loading. Please reload this page.
Uh oh!
There was an error while loading. Please reload this page.
Uh oh!
There was an error while loading. Please reload this page.
- create_table: guard JSON.parse on the schema field with a clear error, trim tableReference IDs in the request body - create_dataset: trim datasetId in the request body - block: stop leaking a stale query "location" value into create_dataset when datasetLocation is empty; clarify pageToken output description covers list_table_data and get_query_results too
waleedlatif1
commented
Jul 7, 2026
waleedlatif1
commented
Jul 7, 2026
bugbot run |
Uh oh!
There was an error while loading. Please reload this page.
Uh oh!
There was an error while loading. Please reload this page.
- create_table: validate parsed schema is a non-empty array of field objects with a name, not just syntactically valid JSON - get_query_results: guard timeoutMs with Number.isFinite before appending to the query string, matching maxResults
waleedlatif1
commented
Jul 7, 2026
@greptile please re-review; bugbot run |
…dation - Add missing projectId block output (returned by get_table/create_table/create_dataset but was silently dropped from the block's output schema) - Switch query/rows/schema subBlocks from long-input to code, matching the JSON/SQL field convention used by every other DB integration (clickhouse, mongodb, postgresql, supabase)
waleedlatif1
commented
Jul 7, 2026
@greptile please re-review; bugbot run — pushed a final alignment pass (added missing projectId block output, switched query/rows/schema subblocks from long-input to code to match repo convention for JSON/SQL fields). |
waleedlatif1
commented
Jul 7, 2026
bugbot run |
Uh oh!
There was an error while loading. Please reload this page.
…et/create_table Same class of bug as the earlier location leak: params() spread hidden datasetId/tableId from ...rest unconditionally, so a stale value from a previously-selected operation could survive into create_dataset/create_table if the dedicated new-ID field was somehow empty. Gate datasetId/tableId (and location) to the operations that actually use them, via shared op-list constants also reused by the corresponding subblock conditions.
waleedlatif1
commented
Jul 7, 2026
@greptile please re-review; bugbot run — fixed a stale datasetId/tableId leak into create_dataset/create_table (same class as the earlier location leak), and deduped the operation-list conditions into shared constants. |
waleedlatif1
commented
Jul 7, 2026
bugbot run |
There was a problem hiding this comment.
✅ Bugbot reviewed your changes and found no new issues!
Comment @cursor review or bugbot run to trigger another review on this PR
Reviewed by Cursor Bugbot for commit 6ca7101. Configure here.
Summary
bigqueryOAuth scope (no new scopes requested): create dataset, delete dataset, create table, delete table, list table data, get query resultstools.config), tool registry, and barrel exportsType of Change
Testing
bun run lint:checkcleanbun run type-checkclean (bigquery-related)bun run check:api-validationpassedChecklist