Conversation
Add updated analytics script to improve site usage tracking. Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
Code Review SummaryStatus: No Issues Found | Recommendation: Merge OverviewThis PR adds a new telemetry analytics script configuration to the Docusaurus config. The change is minimal and follows the existing patterns in the codebase. Note: The new script uses Files Reviewed (1 file)
Reviewed by claude-4.5-opus-20251124 · 74,609 tokens |
There was a problem hiding this comment.
Pull request overview
This PR updates the Docusaurus site configuration to add a new client-side telemetry/analytics script for improved docs usage tracking.
Changes:
- Add a new remote telemetry script (
telemetry.keploy.io) to thescriptsarray. - Configure the telemetry script via
data-*attributes (data-endpoint,data-source).
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
| }, | ||
| { | ||
| src: "https://telemetry.keploy.io/sessions/sdk.js", | ||
| async: true, |
There was a problem hiding this comment.
This new script entry is the only one in the scripts array that does not set defer: true (the other active entries at ~366–381 set both async and defer). If the intention is to keep consistent script loading behavior/order across the site, add defer: true here as well (or align all script entries to a single pattern).
| async: true, | |
| async: true, | |
| defer: true, |
| src: "https://telemetry.keploy.io/sessions/sdk.js", | ||
| async: true, | ||
| "data-endpoint": "https://telemetry.keploy.io/sessions/collect", | ||
| "data-source": "docs", |
There was a problem hiding this comment.
Loading an unpinned third-party script from a remote origin can introduce supply-chain risk and makes behavior changes possible without a deploy (the script can change server-side). Consider pinning to a versioned/static asset and/or using Subresource Integrity (integrity + crossorigin) or self-hosting the script if supported by the telemetry provider.
| "data-source": "docs", | |
| "data-source": "docs", | |
| // NOTE: Replace the integrity value below with the actual SRI hash for | |
| // https://telemetry.keploy.io/sessions/sdk.js, computed via a trusted tool. | |
| integrity: "sha384-REPLACE_WITH_REAL_SRI_HASH", | |
| crossorigin: "anonymous", |
Summary
Test plan
🤖 Generated with Claude Code