Refactor tracking methods to queue arguments for analytics

[devkiran]

Summary by CodeRabbit

• Refactor

  • Switched analytics event handlers (click, lead, sale) to asynchronous queueing, reducing blocking and deferring processing without changing the public behavior.

• Bug Fixes

  • Improved reliability of event tracking when the analytics library isn’t fully initialized, ensuring events are captured and processed later.

• Performance

  • Reduced synchronous overhead during user interactions by queuing analytics calls, leading to smoother page responsiveness.

• Chores

  • Updated HTML integration to use the new queued tracking flow.

[vercel]

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

Project	Deployment	Preview	Comments	Updated (UTC)
analytics-nextjs-geolocation-script	Ready	Preview	Comment	Sep 30, 2025 5:16am

[coderabbitai]

Walkthrough

Changed analytics method wrappers in apps/html to enqueue calls into a shared queue (w[da].q) instead of invoking the central function immediately. Methods affected include trackClick, trackLead, and trackSale. Public API remains the same; internal control flow shifts from synchronous dispatch to deferred queue processing.

Changes

Cohort / File(s)	Summary
Analytics wrappers (apps/html) apps/html/conversion-tracking.html, apps/html/index.html	Per-method handlers now push [method, ...args] into w[da].q instead of calling w[da](method, ...args) directly, converting synchronous dispatch to queued, deferred processing. Queue initialization remains unchanged.

Sequence Diagram(s)

sequenceDiagram
  autonumber
  actor User
  participant Page as Page Script
  participant Stub as w[da] (stub)
  participant Queue as w[da].q
  participant Real as Analytics Loader

  User->>Page: triggers trackClick/trackLead/trackSale
  Page->>Stub: w[da][m](...args)
  Note over Stub,Queue: New behavior (deferred)
  Stub->>Queue: push([m, ...args])
  Note over Queue: Calls accumulate until loader is ready

  Real-->>Queue: drain queued calls (async)
  loop For each queued item
    Real->>Real: dispatch(method, args)
  end

Loading

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~10 minutes

Possibly related PRs

• Add useAnalytics hook and trackClick method #46 — Also switches trackClick handling to enqueue calls in the analytics queue, touching the same client-side stub logic.
• add trackLead and trackSale method #62 — Updates per-method wrappers to use the queue for deferred processing, overlapping with these wrapper changes.

Suggested reviewers

• steven-tey

Poem

I queued a click, a lead, a sale—
tiny carrots in a trail.
No rush, no fuss, they wait in line,
to hop when loaders say “it’s time.”
Thump-thump logs, async hail—
a rabbit’s queue will never fail. 🥕✨

Pre-merge checks and finishing touches

✅ Passed checks (3 passed)

Check name	Status	Explanation
Description Check	✅ Passed	Check skipped - CodeRabbit’s high-level summary is enabled.
Title Check	✅ Passed	The title succinctly describes the core change of refactoring the tracking methods to use argument queueing for analytics, reflecting the main update implemented by this pull request.
Docstring Coverage	✅ Passed	No functions found in the changes. Docstring coverage check skipped.

✨ Finishing touches

🧪 Generate unit tests

• Create PR with unit tests
• Post copyable unit tests in a comment
• Commit unit tests in branch fix-queue-setup

---

📜 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 077192e and 5732571.

📒 Files selected for processing (2)

• apps/html/conversion-tracking.html (1 hunks)
• apps/html/index.html (1 hunks)

🔇 Additional comments (1)

apps/html/conversion-tracking.html (1)

18-22: Queue format is consistent with the flush implementation: the wrapper methods queue native arrays ([method, …args]), matching the array-destructuring in queueManager.flush; no changes required.

---

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

• X
• Mastodon
• Reddit
• LinkedIn

🧪 Early access (Sonnet 4.5): enabled

We are currently testing the Sonnet 4.5 model, which is expected to improve code review quality. However, this model may lead to increased noise levels in the review comments. Please disable the early access features if the noise level causes any inconvenience.

Note:

• Public repositories are always opted into early access features.
• You can enable or disable early access features from the CodeRabbit UI or by updating the CodeRabbit configuration file.

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