Skip to content

feat: replace plausible analytics with opentelemetry browser instrumentation#20

Merged
visoedhwa merged 8 commits into
mainfrom
19-replace-plausible-analytics-with-opentelemetry-browser-instrumentation
May 29, 2026
Merged

feat: replace plausible analytics with opentelemetry browser instrumentation#20
visoedhwa merged 8 commits into
mainfrom
19-replace-plausible-analytics-with-opentelemetry-browser-instrumentation

Conversation

@visoedhwa

Copy link
Copy Markdown
Contributor

Close #19

@visoedhwa visoedhwa linked an issue May 26, 2026 that may be closed by this pull request
17 tasks
@visoedhwa visoedhwa requested a review from arolos May 26, 2026 22:56
@visoedhwa visoedhwa changed the title 19 replace plausible analytics with opentelemetry browser instrumentation feat: replace plausible analytics with opentelemetry browser instrumentation May 26, 2026

@arolos arolos left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

  • Events lost on tab closeBatchLogRecordProcessor never flushed on pagehide/visibilitychange.
  • Duplicate click eventsnav_click + generic a[href] handler both fire content_section for the same hash link.
  • Scroll re-emits content_section after a click already emitted it (lastTrackedSection only updated in scroll path).
  • Inconsistent page_typeview_toggle computes its own variant that doesn't strip .html or normalize index.
  • Duplicated getPageType in js/analytics/init.js and js/page-setup.js.
  • Heartbeat interval never cleared, no session_end ever emitted.
  • Hostname-based isProduction treats every non-localhost host (staging, PR previews) as prod.
  • Wrong DNS resolver for K8sresolver 127.0.0.11 is Docker's embedded DNS, not CoreDNS.
  • Hard-coded collector address opentelemetry-collector.observability.svc:4318 not templated via Helm.
  • AGENTS.md says Plausible, code uses OpenTelemetry.

@visoedhwa visoedhwa requested a review from arolos May 28, 2026 20:13
@visoedhwa visoedhwa merged commit f2d7ab1 into main May 29, 2026
4 checks passed
@github-actions github-actions Bot mentioned this pull request May 29, 2026
@visoedhwa visoedhwa deleted the 19-replace-plausible-analytics-with-opentelemetry-browser-instrumentation branch May 29, 2026 23:09
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.

Replace Plausible Analytics with OpenTelemetry Browser Instrumentation

2 participants