Skip to content

Conversation

RonenMars
Copy link
Contributor

@RonenMars RonenMars commented Sep 25, 2025

Description

Key Components

  • RUM Initialization: Integrated Datadog RUM in application bootstrap (main.tsx)
  • Error Tracking: Enhanced error handling with Datadog monitoring capabilities
  • Performance Monitoring: Enabled resource tracking, long tasks monitoring, and user interaction tracking
  • Privacy Controls: Implemented privacy-first configuration with user input masking

🛡️ Privacy & Security

  • User Input Masking: Default privacy level set to mask sensitive user inputs
  • Environment-Aware: Different configurations for production vs development
  • Domain Restrictions: Tracing limited to localhost and autokitteh.com domains

Linear Ticket

https://linear.app/autokitteh/issue/UI-1898/datadog-browser-session-recordings

What type of PR is this? (check all applicable)

  • 💡 (feat) - A new feature (non-breaking change which adds functionality)
  • 🔄 (refactor) - Code Refactoring - A code change that neither fixes a bug nor adds a feature
  • 🐞 (fix) - Bug Fix (non-breaking change which fixes an issue)
  • 🏎 (perf) - Optimization
  • 📄 (docs) - Documentation - Documentation only changes
  • 📄 (test) - Tests - Adding missing tests or correcting existing tests
  • 🎨 (style) - Styles - Changes that do not affect the meaning of the code (white-space, formatting, missing semi-colons, etc)
  • ⚙️ (ci) - Continuous Integrations - Changes to our CI configuration files and scripts (example scopes: Travis, Circle, BrowserStack, SauceLabs)
  • ☑️ (chore) - Chores - Other changes that don't modify src or test files
  • ↩️ (revert) - Reverts - Reverts a previous commit(s).

@@ -0,0 +1,76 @@
import React from "react";
Copy link
Member

Choose a reason for hiding this comment

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

why is this needed for dd?

@itayd
Copy link
Member

itayd commented Sep 28, 2025

why is this pr so big? isn't it just embedding dd tracking script magic somewhere?

@RonenMars
Copy link
Contributor Author

@itayd reduced it to this: #1333

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

2 participants