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

Project	Deployment	Preview	Comments	Updated (UTC)
rybbit	Building	Preview	Comment	Sep 23, 2025 5:59am

Walkthrough

Adds IP tracking capability: database schema updates (Postgres sites.trackIp, ClickHouse events.ip), server-side config/types and tracker pipeline changes (site config propagation, payload shape, conditional IP storage), API responses extended with ip fields, and client UI updates to toggle trackIp and display IP in session/user views. Minor docs formatting.

Changes

Sequence Diagram(s)

sequenceDiagram
  autonumber
  participant Client
  participant Tracker as Tracker Endpoint
  participant Utils as createBasePayload
  participant Queue as pageviewQueue
  participant DB as ClickHouse/Postgres

  Client->>Tracker: Tracking request (pageview/event)
  Tracker->>Utils: createBasePayload(request, type, payload, siteConfiguration)
  Utils-->>Tracker: Base payload (includes storeIp = siteConfiguration.trackIp)
  Tracker->>Queue: Enqueue TotalTrackingPayload
  Queue->>Queue: Build pageview record<br/>ip = storeIp ? ipAddress : null
  Queue->>DB: Insert event (ip nullable)
  Note over DB: ip persisted only when storeIp is true

sequenceDiagram
  autonumber
  participant UI
  participant API as Server APIs
  participant CH as ClickHouse
  participant PG as Postgres

  UI->>API: GET sessions / session / userInfo
  API->>CH: Query events (select argMax(ip,...), any(ip))
  CH-->>API: Rows with ip
  API-->>UI: JSON including ip fields

  UI->>API: GET/PUT site config (trackIp)
  API->>PG: Read/Update sites.trackIp
  PG-->>API: site config with trackIp
  API-->>UI: Response including trackIp

Estimated code review effort

🎯 4 (Complex) | ⏱️ ~60 minutes

Possibly related PRs

• fix tracking events #589 — Also changes tracker utilities (utils.ts) and createBasePayload handling, overlapping with this PR’s signature and payload updates.
• Exclude ips #527 — Introduces IP-related config (excludedIPs/isIPExcluded) in similar site config and tracking surfaces; closely related to adding trackIp.
• Track outbound events #536 — Modifies session API types and aggregations in getSessions, intersecting with this PR’s addition of ip to session responses.

Poem

A rabbit taps the keys—tip-tap, tip-tap, hooray!
New tracks for tiny hops: an IP on the way.
Toggle on, toggle off, secrets kept neat,
ClickHouse burrows deeper, Postgres keeps the beat.
Sessions show their footprints, softly in the snow—
With care, we store them only when you say so.
(_/) 🌙

reviews:
  pre_merge_checks:
    custom_checks:
      - name: "Undocumented Breaking Changes"
        mode: "warning"
        instructions: |
          Pass/fail criteria: All breaking changes to public APIs, CLI flags, environment variables, configuration keys, database schemas, or HTTP/GraphQL endpoints must be documented in the "Breaking Change" section of the PR description and in CHANGELOG.md. Exclude purely internal or private changes (e.g., code not exported from package entry points or explicitly marked as internal).

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.}