Skip to content

feat: RUM Visits metric (replaces Avg Claps/Article card) - #137

Merged
dosteinacher merged 1 commit into
mainfrom
feat/rum-visits
Jun 23, 2026
Merged

feat: RUM Visits metric (replaces Avg Claps/Article card)#137
dosteinacher merged 1 commit into
mainfrom
feat/rum-visits

Conversation

@dosteinacher

Copy link
Copy Markdown
Collaborator

Adds a Visits metric from RUM and swaps it into the "Avg Claps / Article" card slot, per request.

Why Visits, not unique visitors

RUM is privacy-preserving and keeps no persistent per-user identity, so true unique visitors isn't computable (the RUM Explorer doesn't offer it either). The closest honest metric is Visits = sessions: bundles with an enter checkpoint (an arrival from outside the site), weighted — versus internal page-to-page navigation. A returning person counts as multiple visits, so it's labelled as sessions, not unique people (card tooltip says so).

Changes

  • RUM aggregation now also sums visits per path and site-wide.
  • The blog stat card "Avg Claps / Article" → "Visits", fed from RUM (shows — until page views load).
  • Removed the now-unused avg/median claps computation and the median() helper.

Verification

  • No leftover avgClaps/median references; inline script passes node --check.
  • Visually verified locally: the card renders as "Visits" with a value.

🤖 Generated with Claude Code

RUM can't report unique visitors (no persistent per-user identity), but
it can report Visits = sessions that started from outside the site
(bundles with an 'enter' checkpoint), weighted. Aggregate that per path
and site-wide alongside page views.

Replace the "Avg Claps / Article" stat card with a "Visits" card fed from
RUM. Drop the now-unused avg/median claps computation and median helper.
Card tooltip clarifies it counts sessions, not unique people.

Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
@aem-code-sync

aem-code-sync Bot commented Jun 23, 2026

Copy link
Copy Markdown

Hello, I'm the AEM Code Sync Bot and I will run some actions to deploy your branch and validate page speed.
In case there are problems, just click a checkbox below to rerun the respective action.

  • Re-run all PSI checks
  • Re-run failed PSI checks
  • Re-sync branch
Commits

@dosteinacher
dosteinacher merged commit 8e102f2 into main Jun 23, 2026
2 of 4 checks passed
@dosteinacher
dosteinacher deleted the feat/rum-visits branch June 23, 2026 13:28
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