Walkthrough

Made the client env var NEXT_PUBLIC_GTM_ID optional in the schema and updated the app layout to conditionally mount Google Tag Manager only when that env var is defined.

Changes

Sequence Diagram(s)

sequenceDiagram
    participant Browser
    participant Layout
    participant GTM as GoogleTagManager

    Browser->>Layout: render app layout
    alt NEXT_PUBLIC_GTM_ID defined
        Layout->>GTM: mount(gtmId)
        GTM-->>Layout: initialized
    else NEXT_PUBLIC_GTM_ID undefined
        Layout-->>Browser: skip mounting GTM
    end

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~10 minutes

• Two related but small edits (schema + conditional rendering).
• Pay attention to:
  • Any other places that assume NEXT_PUBLIC_GTM_ID is always present.
  • Type expectations where the env value is consumed (props, components).

Possibly related PRs

• feat: add google tag manager #20 — Modifies src/app/layout.tsx and Google Tag Manager integration; likely directly related to how GTM is mounted.

Suggested reviewers

• sstefdev

Pre-merge checks and finishing touches

Learnt from: bassgeta
PR: RequestNetwork/easy-invoice#149
File: src/lib/env/server.js:29-43
Timestamp: 2025-09-24T12:28:41.783Z
Learning: In the RequestNetwork/easy-invoice project, VERCEL_URL is a code remnant that is not actually used, so it should be removed from environment schemas rather than validated.

Learnt from: bassgeta
PR: RequestNetwork/easy-invoice#149
File: src/lib/env/server.js:28-42
Timestamp: 2025-09-25T09:39:21.320Z
Learning: In the RequestNetwork/easy-invoice project, environment variable validation should only cover user-defined variables, not system/runtime injected variables like NODE_ENV and PORT which are always provided by the runtime environment.

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.

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

Congratulations, your pull request has been merged! Thank you for your valuable contribution to Request Network. As a reminder, every merged PR is automatically entered into our Best PR Initiative, offering a quarterly prize of $500. Your work significantly supports our project's growth, and we encourage you to continue engaging with our community. Additionally, if you want to build or add crypto payments and invoicing features, explore how our API can reduce deployment time from months to hours while offering advanced features. Book a call with our expert to learn more and fast-track your development.