Skip to content

Add a View on Goodreads button on Amazon book pages - #18

Merged
ion05 merged 1 commit into
masterfrom
feat/amazon-goodreads-button
Sep 7, 2026
Merged

ion05 merged 1 commit into
masterfrom
feat/amazon-goodreads-button

Conversation

@ion05

@ion05 ion05 commented Sep 7, 2026

Copy link
Copy Markdown
Owner

What this changes

On Amazon.com book product pages, BunnyLol now draws a View on Goodreads button under the title. It reads the labelled ISBN (or JSON-LD isbn) locally and navigates to goodreads.com/book/isbn/… for that same book. Search pages and listings without an ISBN are left alone.

Verified in Chrome for Testing with dist/ loaded unpacked: the button appears on Project Hail Mary and opens the matching Goodreads page.

Amazon (button under the title):

View on Goodreads button on an Amazon book page

After click (Goodreads for the same book):

Project Hail Mary on Goodreads

Which AGENTS.md invariant does this touch?

11 (user text reaches the DOM only via textContent/createElement). The button label is ours; the ISBN is never written into the page as HTML. The content script uses a shadow root and createElement / createElementNS.

Also privacy: PRIVACY.md no longer claims there are no content scripts.

Checks

  • pnpm typecheck passes
  • pnpm test passes
  • pnpm build passes
  • Loaded dist/ unpacked and exercised the change in a real browser
  • No new dependencies (devDependencies included)
  • If routing/DNR changed: replayed a real Chrome-generated search URL
    through the built rules, not just buildRules
  • Colours/sizes come from design/tokens.css, no literal hex or
    font-size: Npx in the UI sheets, and never color: var(--accent)

The Amazon button lives in a shadow root on amazon.com, so it cannot @import tokens.css. Values are copied by hand the same way go.html already does.

Stacked PRs

Base: master. Not stacked.

Made with Cursor

An Amazon listing already has the ISBN; jumping to the matching Goodreads
page should not require leaving the product page to search.

Co-authored-by: Cursor <cursoragent@cursor.com>
@ion05
ion05 marked this pull request as ready for review September 7, 2026 05:45
@ion05
ion05 merged commit f3b6fe1 into master Sep 7, 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.

1 participant