Skip to content

feat(sheets): add update-note command to write cell notes#430

Merged
steipete merged 3 commits intoopenclaw:mainfrom
andybergon:feat/sheets-update-note
Mar 8, 2026
Merged

feat(sheets): add update-note command to write cell notes#430
steipete merged 3 commits intoopenclaw:mainfrom
andybergon:feat/sheets-update-note

Conversation

@andybergon
Copy link
Copy Markdown
Contributor

Summary

The existing notes command can read cell notes but not write them. This adds update-note (alias: set-note) to set or clear notes via batchUpdate with updateCells + fields: "note".

  • --note "text" for inline, --note-file path for file content
  • Works on single cells (Sheet1!A1) or ranges (Sheet1!A1:B2)
  • --note "" clears notes
  • Named to match the slides update-notes pattern
gog sheets update-note <id> "Sheet1!A1" --note "Hello world"
gog sheets update-note <id> "Sheet1!A1" --note-file notes.txt
gog sheets update-note <id> "Sheet1!A1" --note ""

Closes #429

Test plan

  • Single cell JSON output
  • Range (A1:B2 → 4 cells)
  • Clear note (empty string)
  • Note from file (multiline)
  • Missing --note/--note-file → error
  • Missing sheet name in range → error
  • Full test suite passes (go test ./...)

andybergon and others added 3 commits March 8, 2026 01:00
The existing `notes` command reads cell notes but cannot write them.
Add `update-note` (alias: `set-note`) to set or clear notes via
batchUpdate with updateCells + fields: "note".

Supports --note for inline text, --note-file for file content,
single cells and ranges. Naming follows slides update-notes pattern.

Closes openclaw#429

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
@steipete steipete force-pushed the feat/sheets-update-note branch from fec89cb to c55371a Compare March 8, 2026 01:05
@steipete steipete merged commit f22763c into openclaw:main Mar 8, 2026
2 of 5 checks passed
@steipete
Copy link
Copy Markdown
Collaborator

steipete commented Mar 8, 2026

Landed via temp rebase onto main.

  • Gate: go test ./internal/cmd -run 'TestSheetsUpdateNoteCmd_|TestBuildReminders' -count=1, make ci
  • Land commit: c55371a
  • Merge commit: f22763c9991847b994abb751ac1e04bed6667e59

Fixups included:

  • switch note writes to a single repeatCell request over the full grid range instead of one updateCells request per cell
  • strengthen request-payload regression coverage
  • changelog entry

Thanks @andybergon!

klodr pushed a commit to klodr/gogcli that referenced this pull request Apr 22, 2026
klodr pushed a commit to klodr/gogcli that referenced this pull request Apr 22, 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.

feat(sheets): support writing cell notes

2 participants