Skip to content

Fix Google OAuth service policies - #1669

Merged
RhysSullivan merged 3 commits into
mainfrom
fix/gmail-settings-scopes
Aug 18, 2026
Merged

Fix Google OAuth service policies#1669
RhysSullivan merged 3 commits into
mainfrom
fix/gmail-settings-scopes

Conversation

@RhysSullivan

@RhysSullivanRhysSullivan commented Aug 18, 2026

Copy link
Copy Markdown
Collaborator

What changed

  • add an audited per-service Google OAuth policy for all 22 Discovery integrations
  • request the scopes each ordinary-user service actually needs and filter methods that require admin, app, partner, or domain-wide delegation credentials
  • fix OAuth alternative-scope semantics, scope compaction, required action inputs, and health probes
  • hide unsupported Google Keep user OAuth and remove retired Google Photos sharing actions
  • update the public Google data-use disclosures for the expanded first-party bundle

Why

The earlier Gmail fix exposed one instance of a broader issue: Discovery documents list alternative scopes and mixed credential modes, but Executor treated them as cumulative or universally callable. This left some tools missing, some tools guaranteed to fail, and some generated OAuth metadata overprivileged.

Validation

  • live conversion against the current Discovery documents for all 22 services
  • OpenAPI package: 46 files, 283 tests passed
  • provider service split: 2 files, 36 tests passed
  • Cloud and e2e typechecks passed
  • marketing typecheck and production build passed
  • repository lint, changelog, formatting, and diff checks passed

@github-actions

github-actionsBot commented Aug 18, 2026

Copy link
Copy Markdown
Contributor

Cloudflare preview

Torn down — the PR is closed.

@RhysSullivan
RhysSullivan marked this pull request as ready for review August 18, 2026 21:05
@pkg-pr-new

Copy link
Copy Markdown

Open in StackBlitz

@executor-js/codemode-core

npm i https://pkg.pr.new/@executor-js/codemode-core@1669

@executor-js/runtime-quickjs

npm i https://pkg.pr.new/@executor-js/runtime-quickjs@1669

@executor-js/cli

npm i https://pkg.pr.new/@executor-js/cli@1669

@executor-js/config

npm i https://pkg.pr.new/@executor-js/config@1669

@executor-js/execution

npm i https://pkg.pr.new/@executor-js/execution@1669

@executor-js/sdk

npm i https://pkg.pr.new/@executor-js/sdk@1669

@executor-js/plugin-file-secrets

npm i https://pkg.pr.new/@executor-js/plugin-file-secrets@1669

@executor-js/plugin-graphql

npm i https://pkg.pr.new/@executor-js/plugin-graphql@1669

@executor-js/plugin-keychain

npm i https://pkg.pr.new/@executor-js/plugin-keychain@1669

@executor-js/plugin-mcp

npm i https://pkg.pr.new/@executor-js/plugin-mcp@1669

@executor-js/plugin-onepassword

npm i https://pkg.pr.new/@executor-js/plugin-onepassword@1669

@executor-js/plugin-openapi

npm i https://pkg.pr.new/@executor-js/plugin-openapi@1669

executor

npm i https://pkg.pr.new/executor@1669

commit: e77e09c

@cloudflare-workers-and-pages

cloudflare-workers-and-pagesBot commented Aug 18, 2026

Copy link
Copy Markdown

Deploying with Cloudflare Workers Cloudflare Workers

The latest updates on your project. Learn more about integrating Git with Workers.

StatusNameLatest CommitUpdated (UTC)
✅ Deployment successful!
View logs
executor-cloud42d4e49Aug 18 2026, 11:42 PM

@cloudflare-workers-and-pages

cloudflare-workers-and-pagesBot commented Aug 18, 2026

Copy link
Copy Markdown

Deploying with Cloudflare Workers Cloudflare Workers

The latest updates on your project. Learn more about integrating Git with Workers.

StatusNameLatest CommitPreview URLUpdated (UTC)
✅ Deployment successful!
View logs
executor-marketing42d4e49Commit Preview URL

Branch Preview URL
Aug 18 2026, 11:42 PM

@RhysSullivanRhysSullivan changed the title Add Gmail settings accessFix Google OAuth service policiesAug 18, 2026
@RhysSullivan
RhysSullivan merged commit 44d9ed3 into mainAug 18, 2026
38 of 41 checks passed
@RhysSullivanRhysSullivan mentioned this pull request Aug 18, 2026
Sign up for freeto 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

@RhysSullivan