Skip to content

feat(drive): Disconnect Google Drive + connection state; fix store-ID OAuth docs - #11

Merged
harshabala merged 2 commits into
mainfrom
harsha/lucid-ride-vwzc7t
Sep 27, 2026
Merged

harshabala merged 2 commits into
mainfrom
harsha/lucid-ride-vwzc7t

Conversation

@harshabala

Copy link
Copy Markdown
Owner

Closes the last open code-side item from the v0.3.0 release checklist ("in-app disconnect Google Drive button"). It also fixes store docs that went stale once the package started stripping key.

Changes

  • Disconnect Google Drive: a new DISCONNECT_GOOGLE_DRIVE message revokes the live access token at oauth2.googleapis.com/revoke (best effort) and always deletes the stored token and account email. The backup file in Drive appData is left in place.
  • Connection state: a new GET_DRIVE_STATUS message lets Settings show "Connected as …" on load, with Reconnect and Disconnect buttons. Both new messages are only reachable from extension pages; they are not on the content-script allowlist.
  • Bug fix: the connect handler read result.email from the response wrapper instead of res.data.email, so "Connected as …" never appeared after connecting.
  • Removed the developer-only "OAuth redirect (add in Google Cloud…)" hint from the user-facing Settings panel. It is still shown in Diagnostics and in connect error messages.
  • Privacy policy (docs/PRIVACY.md and docs/privacy.html): documents Disconnect; last-updated date is now September 27, 2026. tests/releaseDocs.test.ts guards both.
  • Store docs: store/MANIFEST_NOTES.md no longer says key must stay. The owner checklist and docs/GOOGLE_DRIVE_SETUP.md now say to register https://<store-item-id>.chromiumapp.org/ after the first upload. Without that, Drive sign-in fails for store installs because the store assigns a different ID.

Verification

  • npm run release-check: typecheck clean, 674/674 tests pass (4 new Drive tests), build succeeds, package check passes.
  • Loaded the unzipped subsume.zip (no key) in headless Chromium:
    • popup renders;
    • Settings shows "Connected as …" with a stored token;
    • Disconnect clears storage, hides the button and shows the notice;
    • the content script mounts on a movie page;
    • no page errors.

Owner follow-up (unchanged, see store/WHAT_YOU_STILL_DO.md)

Developer account and fee, OAuth consent screen in production, the store-ID redirect URI after the first upload, screenshot review, upload and submit.

🤖 Generated with Claude Code

https://claude.ai/code/session_01FHFhkzNyPBLTv6hU2PeX3e


Generated by Claude Code

… Settings

- Background: DISCONNECT_GOOGLE_DRIVE revokes the live access token with Google
  (best effort) and always clears the stored token and account email;
  GET_DRIVE_STATUS reports whether this device is connected.
- Settings: shows "Connected as ..." on load, Reconnect/Disconnect buttons, and
  hides the developer-only OAuth redirect hint (still in Diagnostics and in
  connect error messages).
- Fix: connect handler read the email from the response wrapper instead of
  res.data, so "Connected as ..." never appeared.
- Privacy policy (md + html) now documents disconnect; last updated
  September 27, 2026; releaseDocs test guards it.
- Store docs: MANIFEST_NOTES no longer says the key must stay (the store
  package strips it); owner checklist and Drive setup now say to register the
  store item's redirect URI after first upload.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01FHFhkzNyPBLTv6hU2PeX3e
The store listing, permissions doc, privacy policy, README and code of conduct
listed harsha16balakrishnan@proton.me. releaseDocs test now guards the
privacy policy against the old address.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01FHFhkzNyPBLTv6hU2PeX3e
@harshabala
harshabala marked this pull request as ready for review September 27, 2026 07:15
@harshabala
harshabala merged commit fc11e13 into main Sep 27, 2026
1 check 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.

2 participants