Skip to content

fix(make): detect Google sign-in taps via GIS click_listener (v0.149.1) - #311

Merged
beveradb merged 1 commit into
mainfrom
fix/google-click-listener
Nov 6, 2026
Merged

beveradb merged 1 commit into
mainfrom
fix/google-click-listener

Conversation

@beveradb

@beveradb beveradb commented Nov 6, 2026 •

Copy link
Copy Markdown
Contributor

Follow-up to #310. A test on production found the window-blur check misses some Google taps (the button is a cross-origin iframe). GIS renderButton now also gets its documented click_listener, with blur kept as a backup and taps within 1.5 s de-duplicated.

Verified in Chromium against the live page with this make.js swapped in: one tap event, popup opened, then stall and the email hint.

Local CodeRabbit skipped, so no ignore line.

🤖 Generated with Claude Code

https://claude.ai/code/session_012RYLVUT9R9DTcUZB1zhh88

Summary by CodeRabbit

  • Bug Fixes
    • Improved detection of Google sign-in taps, with duplicate reports counted only once.
  • Documentation
    • Updated the changelog with the sign-in detection change and verification results.

@coderabbitai

coderabbitai Bot commented Nov 6, 2026 •

Copy link
Copy Markdown

Explain Change →

Caution

Review failed

The pull request is closed.

Walkthrough

Google sign-in tap detection now uses GIS click_listener alongside the window-blur fallback. Same-provider taps detected within 1.5 seconds count once. The project version and changelog were updated to v0.149.1.

Changes

Google sign-in tap detection

Layer / File(s) Summary
Google tap reporting and release notes
kj-controller/static-sing/make.js, kj-controller/pyproject.toml, docs/CHANGELOG.md
The Google button invokes a tap callback stored on its holder. The blur fallback remains active, and same-provider taps within 1.5 seconds are ignored. The project version and changelog were updated to v0.149.1.

Estimated code review effort: 2 (Simple) | ~10 minutes

Change: Bug fix

Merge Risk: Low · up to 1266d

The Google detection change has bounded fallback gaps: a rapid Apple retry or backward device-clock adjustment can prevent the email hint from appearing if sign-in stalls. Sign-in itself remains available; address these gaps or explicitly accept them before merging.

- [ ] Autofix · Keep fixing CodeRabbit findings and required CI, and resolving merge conflicts

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

Share

A rabbit taps the Google sign,
The click and blur both send a sign.
One tap counts once within the time,
While changelog notes the version climb.
Then hops away, content and fine.

@beveradb
beveradb merged commit 877012f into main Nov 6, 2026
1 of 2 checks passed
beveradb added a commit that referenced this pull request Nov 6, 2026
…312)

CodeRabbit (local review of #306-#311): a glob '?' is a wildcard before
Playwright 1.52, so 'social?*' would also catch the social-event posts.
Use one explicit regex.


Claude-Session: https://claude.ai/code/session_012RYLVUT9R9DTcUZB1zhh88

Co-authored-by: Claude Opus 5.5 <noreply@anthropic.com>
@beveradb
beveradb deleted the fix/google-click-listener branch November 6, 2026 21:42
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