Skip to content

fix: Update Chrome Web Store crawler selector for user count#9

Merged
aklinker1 merged 3 commits intowxt-dev:mainfrom
whiteboard-works:main
Apr 18, 2026
Merged

fix: Update Chrome Web Store crawler selector for user count#9
aklinker1 merged 3 commits intowxt-dev:mainfrom
whiteboard-works:main

Conversation

@wbwSoftware
Copy link
Copy Markdown
Contributor

Google changed the CWS detail page DOM layout — a new "Add to Chrome" div was appended as the last child of the header section, causing the div:last-child selector to miss the user count element. The crawler threw on weeklyActiveUsers extraction, which nullified the entire extension entry in the GraphQL response.

Replace the positional div:last-child selector with a text-content search for the div containing " users", making it resilient to future DOM reordering. Add a test fixture from the current (Apr 2026) CWS HTML.

Google changed the CWS detail page DOM layout — a new "Add to Chrome"
div was appended as the last child of the header section, causing the
`div:last-child` selector to miss the user count element. The crawler
threw on `weeklyActiveUsers` extraction, which nullified the entire
extension entry in the GraphQL response.

Replace the positional `div:last-child` selector with a text-content
search for the div containing " users", making it resilient to future
DOM reordering. Add a test fixture from the current (Apr 2026) CWS HTML.

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
@aklinker1
Copy link
Copy Markdown
Member

Run bun test and commit the new snapshot

CI requires snapshots to exist — Bun won't create them in CI mode.

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
@shadoath
Copy link
Copy Markdown
Contributor

shadoath commented Apr 17, 2026

@aklinker1 fixed specs Will check again later

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
@aklinker1 aklinker1 merged commit 5555ebe into wxt-dev:main Apr 18, 2026
2 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.

3 participants