Skip to content

Conversation

@LGLabGreg
Copy link
Owner

@LGLabGreg LGLabGreg commented Dec 15, 2025

Note

Adds a Google AdSense script to the docs layout and bumps eslint, @types/node, and lucide-react versions with corresponding lockfile updates.

  • Docs:
    • Add Google AdSense <Script> in apps/docs/src/app/layout.tsx using NEXT_PUBLIC_GOOGLE_ADS_CLIENT_ID.
  • Dependencies:
    • Bump eslint to ^9.39.2 and related packages (@eslint/js, eslint-plugin-react-refresh, config/peer versions).
    • Bump @types/node to ^25.0.2 across root, apps/*, and packages/react-qr-code.
    • Bump lucide-react to ^0.561.0 in apps/docs.
    • Update pnpm-lock.yaml to reflect these changes.

Written by Cursor Bugbot for commit 8cc42dd. This will update automatically on new commits. Configure here.

Summary by CodeRabbit

  • New Features

    • Google AdSense advertising has been integrated into the documentation site and is now active.
  • Chores

    • Development dependencies updated to latest stable versions across all projects.

✏️ Tip: You can customize this high-level summary in your review settings.

dependabot bot and others added 3 commits December 12, 2025 22:14
Bumps [lucide-react](https://github.com/lucide-icons/lucide/tree/HEAD/packages/lucide-react) from 0.560.0 to 0.561.0.
- [Release notes](https://github.com/lucide-icons/lucide/releases)
- [Commits](https://github.com/lucide-icons/lucide/commits/0.561.0/packages/lucide-react)

---
updated-dependencies:
- dependency-name: lucide-react
  dependency-version: 0.561.0
  dependency-type: direct:production
  update-type: version-update:semver-minor
...

Signed-off-by: dependabot[bot] <support@github.com>
…ucide-react-0.561.0

chore(deps): bump lucide-react from 0.560.0 to 0.561.0
@vercel
Copy link

vercel bot commented Dec 15, 2025

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Review Updated (UTC)
reactqrcode-com Ready Ready Preview, Comment Dec 15, 2025 6:46am

@coderabbitai
Copy link

coderabbitai bot commented Dec 15, 2025

Caution

Review failed

The pull request is closed.

Walkthrough

This PR updates multiple dependency versions across the project's package.json files and adds a Google AdSense script tag to the docs application layout with asynchronous loading and the client ID from environment variables.

Changes

Cohort / File(s) Change Summary
Dependency version updates
apps/docs/package.json, apps/playground/package.json, package.json, packages/react-qr-code/package.json
Bumped @types/node from ^25.0.1 to ^25.0.2 across all package.json files; lucide-react from ^0.560.0 to ^0.561.0 in apps/docs; @eslint/js, eslint, and eslint-plugin-react-refresh minor version bumps in root package.json
Monetization integration
apps/docs/src/app/layout.tsx
Added Google AdSense script tag (adsbygoogle.js) loaded asynchronously with crossOrigin set to anonymous, utilizing NEXT_PUBLIC_GOOGLE_ADS_CLIENT_ID environment variable

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~5 minutes

  • Dependency bumps are routine patch/minor version updates with no functional logic changes
  • AdSense script addition is a straightforward integration with no complex interactions
  • All changes are low-risk and non-invasive

Possibly related PRs

  • Chore/deps 12 12 25 #389: Both PRs modify apps/docs/package.json to bump lucide-react and @types/node dependencies
  • Chore/deps 06 12 25 #373: Both PRs bump @types/node and lucide-react versions in apps/docs/package.json
  • Chore deps #87: Both PRs perform dependency version bumps across apps/playground, root package.json, and packages/react-qr-code

Poem

A rabbit hops through version trees,
Updating deps with careful ease,
AdSense now greets the docs with cheer—
Fresh patches bring the new year near! 🐰✨

✨ Finishing touches
  • 📝 Generate docstrings
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment
  • Commit unit tests in branch chore/deps-15-12-25

📜 Recent review details

Configuration used: CodeRabbit UI

Review profile: CHILL

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between 37e5bda and 8cc42dd.

⛔ Files ignored due to path filters (1)
  • pnpm-lock.yaml is excluded by !**/pnpm-lock.yaml
📒 Files selected for processing (5)
  • apps/docs/package.json (2 hunks)
  • apps/docs/src/app/layout.tsx (1 hunks)
  • apps/playground/package.json (1 hunks)
  • package.json (1 hunks)
  • packages/react-qr-code/package.json (1 hunks)

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

Comment @coderabbitai help to get the list of available commands and usage tips.

@LGLabGreg LGLabGreg merged commit 52a1cb8 into main Dec 15, 2025
5 of 6 checks passed
@LGLabGreg LGLabGreg deleted the chore/deps-15-12-25 branch December 15, 2025 06:46
Copy link

@cursor cursor bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This is the final PR Bugbot will review for you during this billing cycle

Your free Bugbot reviews will reset on January 3

Details

You are on the Bugbot Free tier. On this plan, Bugbot will review limited PRs each billing cycle.

To receive Bugbot reviews on all of your PRs, visit the Cursor dashboard to activate Pro and start your 14-day free trial.

async
src={`https://pagead2.googlesyndication.com/pagead/js/adsbygoogle.js?client=${process.env.NEXT_PUBLIC_GOOGLE_ADS_CLIENT_ID}`}
crossOrigin='anonymous'
/>
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Bug: Google Ads script loads in all environments, not just production

The newly added Google Ads script is rendered unconditionally in all environments, while the Umami analytics script above it is wrapped in an IS_PRODUCTION check. This inconsistency means ads will load in development and preview environments, which is likely unintended and could affect ad metrics or cause development issues. The Google Ads script appears to be missing the same IS_PRODUCTION guard used for other third-party scripts.

Fix in Cursor Fix in Web

@coderabbitai coderabbitai bot mentioned this pull request Dec 21, 2025
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