Skip to content

feat(sheets): add links command to retrieve cell hyperlinks#374

Merged
steipete merged 2 commits intosteipete:mainfrom
omothm:feat/sheets-links
Mar 3, 2026
Merged

feat(sheets): add links command to retrieve cell hyperlinks#374
steipete merged 2 commits intosteipete:mainfrom
omothm:feat/sheets-links

Conversation

@omothm
Copy link
Contributor

@omothm omothm commented Feb 26, 2026

Summary

  • Add gog sheets links (alias: hyperlinks) command that retrieves hyperlinks embedded in cells via spreadsheets.get with IncludeGridData
  • Mirrors the existing notes command pattern, using the CellData.Hyperlink field instead of Note
  • Supports both table and JSON output formats

Test plan

  • TestSheetsLinksCmd_JSON — mock API response with hyperlinks, verify JSON structure
  • TestSheetsLinksCmd_Text — verify table output with A1 → VALUE → LINK columns
  • TestSheetsLinksCmd_OffsetRange_JSON — test offset ranges (e.g., B2:C3) produce correct A1 references
  • TestSheetsLinksCmd_NoLinks — verify "No links found" message on stderr
  • Manual test against live spreadsheet with hyperlinked cells

🤖 Generated with Claude Code

omothm and others added 2 commits March 3, 2026 05:23
Add `gog sheets links` (alias: hyperlinks) command that retrieves
hyperlinks embedded in cells via spreadsheets.get with IncludeGridData.
Mirrors the existing `notes` command pattern using the CellData.Hyperlink field.

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
@steipete steipete force-pushed the feat/sheets-links branch from b2cbe04 to 7408c85 Compare March 3, 2026 05:25
@steipete steipete merged commit 0ed8997 into steipete:main Mar 3, 2026
@steipete
Copy link
Owner

steipete commented Mar 3, 2026

Landed via temp rebase onto main.

Thanks @omothm!

vidarbrekke pushed a commit to vidarbrekke/gogcli-enhanced that referenced this pull request Mar 3, 2026
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