docs: complete OAuth scope list (Sheets/Docs/Slides/Drive/Forms)#3
Conversation
Document the full scope set the ops workflows need, and clarify which the bundled scripts actually use (Sheets + Drive) vs. which are for operating the Docs/Slides/Forms assets directly (interactive ops, source decks and docs). Replaces the earlier Sheets/Drive/Forms-only note that mislabeled Forms as a script dependency. Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
There was a problem hiding this comment.
Pull request overview
Updates the repository documentation to clearly enumerate the full set of Google OAuth scopes needed for AAIF meetup ops workflows, distinguishing what the bundled scripts require vs. what’s needed for interactive/asset operations.
Changes:
- Expands the “enable APIs” list to include Slides alongside Sheets/Docs/Drive/Forms.
- Documents additional OAuth scopes for Docs (
documents) and Slides (presentations). - Annotates which scopes are used by bundled scripts (Sheets/Drive) vs. interactive/asset operations (Docs/Slides/Forms).
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
Hero Self-ReviewTwo agents reviewed this doc change (comment-analyzer for accuracy, code-reviewer for formatting/URLs). Code-only analyzers were skipped — this PR has no code. Suggestions (2)
Strengths
Self-review by Claude Code |
- Reconcile the "grant read-write scopes" lead-in with the read-only forms.responses.readonly scope (form responses are read-only by nature) - Reframe the Docs/Slides/Forms rationale onto the chapter/series source docs/decks and intake form rather than over-claiming connector/MCP support Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
Hero Self-Review ImprovementsSuggestions (2 / 2 fixed):
Commit: e92faf8 Applied by Claude Code |
Respond-to-PR ImprovementsCode changes (0 applied): none — Copilot's single comment was already resolved by a commit pushed before this cycle. Questions answered: none Declined: none Already addressed (1):
Commits: e92faf8 (pre-existing on branch) Remaining unresolved threads: none Applied by Claude Code |
Summary
Follow-up to the self-review of PR #2. Documents the full OAuth scope set the AAIF meetup ops workflows need — Sheets, Docs, Slides, Drive, and Forms — and clarifies which scopes the bundled scripts actually exercise vs. which are for operating the underlying assets directly.
Why
The README scopes note added in #2 listed only Sheets/Drive/Forms and implied Forms was a script dependency. In practice:
Changeset
README.md— enable list now includes Slides (and keeps Docs/Forms); scope list addsdocumentsandpresentations, annotatesspreadsheets/driveas (scripts), and reframes the intro so Forms/Docs/Slides are correctly described as asset/interactive scopes rather than script requirements.Test Plan
pre-commit run --files README.mdpasses (codespell, gitleaks, hygiene)🤖 Generated with Claude Code