Skip to content

docs: complete OAuth scope list (Sheets/Docs/Slides/Drive/Forms)#3

Merged
rparundekar merged 2 commits into
mainfrom
docs/oauth-scopes-full
Jun 30, 2026
Merged

docs: complete OAuth scope list (Sheets/Docs/Slides/Drive/Forms)#3
rparundekar merged 2 commits into
mainfrom
docs/oauth-scopes-full

Conversation

@rparundekar

Copy link
Copy Markdown
Collaborator

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:

  • The bundled Python scripts use Sheets + Drive only.
  • Docs, Slides, and Forms scopes are needed for interactive ops (connector/MCP) and for working with the chapter/series source docs and decks and the intake form directly.

Changeset

  • README.md — enable list now includes Slides (and keeps Docs/Forms); scope list adds documents and presentations, annotates spreadsheets/drive as (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.md passes (codespell, gitleaks, hygiene)

🤖 Generated with Claude Code

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>
Copilot AI review requested due to automatic review settings June 30, 2026 11:36

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

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.

Comment thread README.md Outdated
@rparundekar

Copy link
Copy Markdown
Collaborator Author

Hero Self-Review

Two 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)

  • [comment-analyzer] README.md:81-93 — The lead-in says "Grant read-write scopes — read-only access … fails on the first write," but the list includes forms.responses.readonly, a deliberately read-only scope. The earlier version reconciled this ("…and read Form responses"); the rewrite dropped that carve-out, leaving an apparent contradiction.
  • [comment-analyzer] README.md:83-85 — Attributing Docs/Slides/Forms scopes to "interactive ops via the connector/MCP" is slightly broader than Options B/C actually claim (they describe Drive/Sheets only). Lean the rationale on the chapter/series source docs/decks + intake form instead.

Strengths

  • All six OAuth scope URLs are exact, valid Google scope strings (both agents verified independently).
  • *(scripts)* tags sit on exactly the two scopes the code uses (spreadsheets, drive) — a precise, low-rot way to encode the script-vs-interactive split.
  • The API-enable list and the scope list match one-to-one; no contradictions with the verify command or gws auth options.

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>
@rparundekar

Copy link
Copy Markdown
Collaborator Author

Hero Self-Review Improvements

Suggestions (2 / 2 fixed):

  • README.md:81 — "grant read-write scopes" contradicted the listed forms.responses.readonly scope — reworded to "read-write for anything you write to" and noted form responses are read-only by nature.
  • README.md:83-85 — Docs/Slides/Forms rationale over-claimed connector/MCP support (Options B/C only cover Drive/Sheets) — reframed onto the chapter/series source docs/decks and intake form, with connector/MCP as an example rather than the basis.

Commit: e92faf8


Applied by Claude Code

@rparundekar

Copy link
Copy Markdown
Collaborator Author

Respond-to-PR Improvements

Code 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):

  • README.md — Copilot: "read-write paragraph contradicts the read-only forms.responses.readonly scope" — fixed in e92faf8 (this review ran against the earlier 0fba3dc). The lead-in now scopes read-write to "anything you write to" and notes form responses are read-only by nature. Thread replied to and resolved.

Commits: e92faf8 (pre-existing on branch)

Remaining unresolved threads: none


Applied by Claude Code

@rparundekar rparundekar merged commit 3a8ef17 into main Jun 30, 2026
2 checks passed
@rparundekar rparundekar deleted the docs/oauth-scopes-full branch July 1, 2026 09:22
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