Skip to content

limit google actions to the granted scopes and revoke on disconnect - #82

Merged
hkonsti merged 3 commits into
hkonsti/google-2-oauth-drivefrom
hkonsti/google-3-scopes
Oct 15, 2026
Merged

hkonsti merged 3 commits into
hkonsti/google-2-oauth-drivefrom
hkonsti/google-3-scopes

Conversation

@hkonsti

@hkonsti hkonsti commented Oct 13, 2026 •

Copy link
Copy Markdown
Collaborator

Google's consent screen lets a person untick scopes, and older connections lack scopes added later, so an action whose scope the sign-in didn't grant is now unavailable: the connection response carries unavailable (read from the decrypted credential per request through connections/unavailable.ts, no migration), defaults and agent settings refuse turning it on, runAgentTool refuses it with "Sign in to Google again" and logs it as denied, and the web shows it disabled with a "Sign in again" link that opens Google on the account (login_hint). Integrations gain an optional disclosure, Google's Limited Use line, shown above a connection's permissions and on the welcome step. Disconnecting a Google connection revokes the grant at Google first (connections/disconnect.ts), deleting it anyway when Google can't be reached.

Stack: 3 of 3 for the Google Workspace connector, on #81.


Open workspace in Conductor

@hkonsti
hkonsti added this pull request to stack #88 October 13, 2026 21:06
@hkonsti
hkonsti force-pushed the hkonsti/google-3-scopes branch from b228462 to 52965b2 Compare October 13, 2026 21:36
@hkonsti
hkonsti force-pushed the hkonsti/google-3-scopes branch from 52965b2 to 0f239d0 Compare October 14, 2026 00:49
@hkonsti
hkonsti force-pushed the hkonsti/google-3-scopes branch 2 times, most recently from 68c138f to 9ed1201 Compare October 14, 2026 18:14
…fuse plainly what the agent's own setting turns off, keep the make-default button off unavailable actions, and keep a google grant another connection uses
@hkonsti
hkonsti force-pushed the hkonsti/google-3-scopes branch from 9ed1201 to cf574dd Compare October 15, 2026 00:20
@hkonsti
hkonsti merged commit d36dbd7 into main Oct 15, 2026
4 checks passed
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.

1 participant