Skip to content

feat(sheets): add Connected Sheets reads - #989

Merged
steipete merged 1 commit into
mainfrom
feat/sheets-connected-sheets-938
Aug 13, 2026
Merged

feat(sheets): add Connected Sheets reads#989
steipete merged 1 commit into
mainfrom
feat/sheets-connected-sheets-938

Conversation

@steipete

Copy link
Copy Markdown
Collaborator

Summary

  • add read-only Connected Sheets data-source list and describe commands with joined execution status
  • add extract discovery, full table descriptions, and bounded value reads keyed by the API's anchor-cell identity
  • keep BigQuery access explicitly opt-in through a dedicated Sheets-read-only plus BigQuery-read-only client
  • extend metadata JSON, generated command docs and Sheets agent guidance, and add a focused Connected Sheets guide

Proof

  • make ci
  • go test ./internal/cmd -run 'TestSheetsDataSource(ListAndDescribe|TableListDescribeAndRead)$' -count=1 -v
  • built-binary schema and invalid-input checks for the new command tree
  • autoreview clean with no accepted/actionable findings

The fixture exercises BigQuery table and query specs, DATA_SOURCE sheet status, extract anchor discovery, and bounded reads. Read-only live Connected Sheets validation was not available because the requested validation account does not currently have the required Sheets and BigQuery grants; no persistent authorization was changed.

Refs #938

@clawsweeper

clawsweeper Bot commented Aug 13, 2026

Copy link
Copy Markdown
Contributor

🦞👀
ClawSweeper picked this up.

Pull request received. I will update this pull request when review starts.

@steipete
steipete merged commit a4c96de into main Aug 13, 2026
11 checks 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.

1 participant