Fix/analytics review comments

[dhananjay6561]

Overview

This PR addresses the review comments raised on #807 by resolving script loading consistency and supply-chain risk concerns flagged for the telemetry script added to docusaurus.config.js.

Problem

The telemetry script entry added in #807 was missing the defer: true attribute, making it inconsistent with all other script entries in the scripts array (feedback.js, clarity.js, apollo-init.js), which all specify both async: true and defer: true. This inconsistency could affect script execution order and page load behavior.

Solution

Added defer: true to the telemetry script configuration to align with the existing script loading pattern across the Docusaurus config.

Regarding the SRI (integrity + crossorigin) suggestion — since telemetry.keploy.io is an owned domain and not a true third-party source, the supply-chain risk is minimal. Adding SRI would require manually updating the hash on every SDK update, introducing maintenance overhead and risk of silently breaking analytics on the docs site. This has been consciously deferred.

Files Changed

• docusaurus.config.js — Added defer: true to the telemetry.keploy.io/sessions/sdk.js script entry.

[kilo-code-bot]

Code Review Summary

Status: No Issues Found | Recommendation: Merge

Overview

The PR adds a telemetry script configuration to docusaurus.config.js. The implementation:

• Follows the existing pattern for script entries (both async: true and defer: true)
• Uses proper syntax for data attributes ("data-endpoint", "data-source")
• Is consistent with the project's script loading conventions

The configuration is straightforward and correctly structured.

Files Reviewed (1 file)

• docusaurus.config.js - 0 issues

---

_{Reviewed by claude-4.5-opus-20251124 · 60,910 tokens}

[Copilot]

Pull request overview

Aligns the newly added telemetry script configuration with the existing Docusaurus scripts loading pattern to ensure consistent script execution behavior across the docs site.

Changes:

• Adds defer: true to the https://telemetry.keploy.io/sessions/sdk.js script entry to match other scripts in docusaurus.config.js.

---

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.