feat: add Datadog RUM session tracking integration by jordane · Pull Request #20 · linuxfoundation/lfx-v2-ui

jordane · 2025-07-30T22:39:43Z

Install @datadog/browser-rum package for real user monitoring
Add configurable Datadog RUM service with full feature support
Configure environment variables for applicationId, clientToken, and env
Enable RUM for dev/staging/prod, disabled for local development
Integrate user context tracking for authenticated users
Support session tracking, replay, user interactions, and error monitoring
Use datadoghq.com site with lfx-projects-self-service service name

🤖 Generated with Claude Code

- Install @datadog/browser-rum package for real user monitoring - Add configurable Datadog RUM service with full feature support - Configure environment variables for applicationId, clientToken, and env - Enable RUM for dev/staging/prod, disabled for local development - Integrate user context tracking for authenticated users - Support session tracking, replay, user interactions, and error monitoring - Use datadoghq.com site with lfx-projects-self-service service name 🤖 Generated with [Claude Code](https://claude.ai/code) Co-Authored-By: Claude <noreply@anthropic.com> Signed-off-by: Jordan Evans <jevans@linuxfoundation.org>

Copilot

Pull Request Overview

This PR adds Datadog Real User Monitoring (RUM) integration to track user sessions, performance, and errors across development, staging, and production environments. The integration is configurable via environment variables and includes user context tracking for authenticated users.

Adds @datadog/browser-rum package dependency
Creates a comprehensive DatadogRumService with full RUM feature support
Configures environment-specific settings with RUM disabled for local development

Reviewed Changes

Copilot reviewed 6 out of 7 changed files in this pull request and generated 4 comments.

File	Description
package.json	Adds @datadog/browser-rum dependency
environment*.ts	Configures Datadog settings for each environment (local disabled, dev/prod enabled)
datadog-rum.service.ts	Implements RUM service with initialization, user tracking, and monitoring features
app.component.ts	Integrates RUM service initialization and user context setting

Copilot · 2025-07-30T22:40:29Z

apps/lfx-pcc/src/environments/environment.prod.ts

+    applicationId: process.env['DD_APPLICATION_ID'] || '',
+    clientToken: process.env['DD_CLIENT_TOKEN'] || '',
+    site: 'datadoghq.com',
+    service: 'lfx-projects-self-service',
+    env: process.env['DD_ENV'] || 'prod',


Using process.env in browser environments exposes environment variables to client-side code. Consider using Angular's build-time environment replacement or a secure configuration service instead of runtime environment variables for sensitive tokens.

Suggested change

applicationId: process.env['DD_APPLICATION_ID'] || '',

clientToken: process.env['DD_CLIENT_TOKEN'] || '',

site: 'datadoghq.com',

service: 'lfx-projects-self-service',

env: process.env['DD_ENV'] || 'prod',

applicationId: 'DD_APPLICATION_ID_PLACEHOLDER',

clientToken: 'DD_CLIENT_TOKEN_PLACEHOLDER',

site: 'datadoghq.com',

service: 'lfx-projects-self-service',

env: 'DD_ENV_PLACEHOLDER',

Copilot · 2025-07-30T22:40:29Z

apps/lfx-pcc/src/environments/environment.dev.ts

+    applicationId: process.env['DD_APPLICATION_ID'] || '',
+    clientToken: process.env['DD_CLIENT_TOKEN'] || '',
+    site: 'datadoghq.com',
+    service: 'lfx-projects-self-service',
+    env: process.env['DD_ENV'] || 'dev',


Using process.env in browser environments exposes environment variables to client-side code. Consider using Angular's build-time environment replacement or a secure configuration service instead of runtime environment variables for sensitive tokens.

Suggested change

applicationId: process.env['DD_APPLICATION_ID'] || '',

clientToken: process.env['DD_CLIENT_TOKEN'] || '',

site: 'datadoghq.com',

service: 'lfx-projects-self-service',

env: process.env['DD_ENV'] || 'dev',

applicationId: 'dev-application-id', // Replace with actual dev value or placeholder

clientToken: 'dev-client-token', // Replace with actual dev value or placeholder

site: 'datadoghq.com',

service: 'lfx-projects-self-service',

env: 'dev', // Replace with actual dev value or placeholder

Copilot · 2025-07-30T22:40:29Z

apps/lfx-pcc/src/app/shared/services/datadog-rum.service.ts

+      datadogRum.init({
+        applicationId: environment.datadog.applicationId,
+        clientToken: environment.datadog.clientToken,
+        site: environment.datadog.site as 'datadoghq.com',


The type assertion 'as "datadoghq.com"' is brittle and could break if the site value changes. Consider using proper typing or validation to ensure the site value is a valid Datadog site.

Copilot · 2025-07-30T22:40:30Z

apps/lfx-pcc/src/app/shared/services/datadog-rum.service.ts

+        site: environment.datadog.site as 'datadoghq.com',
+        service: environment.datadog.service,
+        env: environment.datadog.env,
+        version: '1.0.0',


The version is hardcoded as '1.0.0'. Consider reading this from package.json or environment configuration to keep it synchronized with actual application versions.

Signed-off-by: Asitha de Silva <asithade@gmail.com>

- Add weekly-e2e-tests.yml workflow for comprehensive browser testing - Runs every Sunday at 2:00 AM UTC on main branch - Tests all browsers (Chromium, Firefox, Mobile Chrome) in parallel - Reuses existing e2e-tests.yml workflow for consistency - Creates GitHub issues automatically on test failures - Supports manual dispatch with browser/branch selection - Re-enable E2E tests dependency in quality-check workflow Generated with [Claude Code](https://claude.ai/code) Signed-off-by: Asitha de Silva <asithade@gmail.com>

- Replace complex participant management modal with streamlined add/edit participant form - Add checkbox option to add multiple participants in sequence - Implement participant deletion with confirmation dialog - Move participant management actions to individual meeting cards - Add real-time participant list refresh during modal operations - Remove unused participant list and management modal components - Update meeting service with participant update/delete endpoints - Enhance checkbox component to support reactive forms 🤖 Generated with [Claude Code](https://claude.ai/code) Signed-off-by: Asitha de Silva <asithade@gmail.com>

- Add effect-based meeting input synchronization for better reactivity - Implement refreshMeeting method to update meeting data after edits/deletes - Simplify participant list refresh logic - Update meeting modal delete method naming for consistency - Remove unnecessary meetingUpdated event emissions 🤖 Generated with [Claude Code](https://claude.ai/code) Signed-off-by: Asitha de Silva <asithade@gmail.com>

- Rename refreshParticipantsList to initParticipantsList for consistency - Fix duplicate participantsLoading.set(true) calls - Implement real-time participant refresh using onChildComponentLoaded - Update all participant list refresh calls to use consistent method naming 🤖 Generated with [Claude Code](https://claude.ai/code) Signed-off-by: Asitha de Silva <asithade@gmail.com>

- Add GitHub Actions workflows for Docker image builds: - docker-build-main.yml: builds on pushes to main branch - docker-build-tag.yml: builds on tag creation for releases - Add complete Helm chart for Kubernetes deployment: - Chart.yaml with metadata and version info - Comprehensive values.yaml with environment configuration - Template files for all Kubernetes resources: - Deployment with configurable replicas and environment variables - Service with ClusterIP type and port configuration - Ingress with TLS support and path-based routing - ServiceAccount with RBAC configuration - Helper templates for consistent naming and labeling - Configure environment variables for Auth0, Supabase, and PCC services - Set up proper resource limits and health checks - Include comprehensive README with deployment instructions This enables containerized deployment to Kubernetes environments with proper CI/CD integration for automated image builds. Signed-off-by: Alan Sherman <asherman@linuxfoundation.org>

Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com> Signed-off-by: Alan Sherman <alan@alansherman.org>

Signed-off-by: Alan Sherman <asherman@linuxfoundation.org>

test: add comprehensive E2E testing with Playwright and CI workflows

feat: refactor participant management to add/edit flow

feat: Add docker build and helm chart

github-actions · 2025-07-31T22:09:56Z

✅ E2E Tests Passed

Browser: chromium
Status: passed

All E2E tests passed successfully.

Test Configuration

Node.js: 22
Command: e2e
Browser: chromium
Base URL: http://localhost:4200

Copilot AI review requested due to automatic review settings July 30, 2025 22:39

github-actions bot requested review from andrest50, asithade, audigregorie and mauriciozanettisalomao July 30, 2025 22:39

github-actions bot assigned jordane Jul 30, 2025

Copilot AI reviewed Jul 30, 2025

View reviewed changes

asithade and others added 22 commits July 30, 2025 19:16

ci: fix e2e workflow

83666e0

Signed-off-by: Asitha de Silva <asithade@gmail.com>

ci: fix permissions for e2e flow

2bc9e94

Signed-off-by: Asitha de Silva <asithade@gmail.com>

ci: fix access to secret values

bc2f0ed

Signed-off-by: Asitha de Silva <asithade@gmail.com>

ci: fix e2e folder

dab38e0

Signed-off-by: Asitha de Silva <asithade@gmail.com>

ci: fix e2e folder

7d54b63

Signed-off-by: Asitha de Silva <asithade@gmail.com>

ci: hide secret values

5049a29

Signed-off-by: Asitha de Silva <asithade@gmail.com>

ci: hide secret values

793a9b1

Signed-off-by: Asitha de Silva <asithade@gmail.com>

ci: add secrets

3997569

Signed-off-by: Asitha de Silva <asithade@gmail.com>

ci: add secrets

ea29994

Signed-off-by: Asitha de Silva <asithade@gmail.com>

ci: fix permissions for comments

a168d32

Signed-off-by: Asitha de Silva <asithade@gmail.com>

ci: remove trace

1300c9e

Signed-off-by: Asitha de Silva <asithade@gmail.com>

ci: update existing comment for test status

c67fc00

Signed-off-by: Asitha de Silva <asithade@gmail.com>

Update charts/lfx-v2-pcc-ui/values.yaml

eda02a9

Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com> Signed-off-by: Alan Sherman <alan@alansherman.org>

Clean up docs

e9e6e73

Signed-off-by: Alan Sherman <asherman@linuxfoundation.org>

Merge branch 'main' into asherman/helm-chart

51e3578

Clean up docs

33af87c

Signed-off-by: Alan Sherman <asherman@linuxfoundation.org>

Merge pull request #19 from linuxfoundation/feat/playwright-tests

498f3f0

test: add comprehensive E2E testing with Playwright and CI workflows

dealako and others added 5 commits July 31, 2025 14:54

Merge branch 'main' into feat/edit-participant

d85ce85

Merge pull request #21 from linuxfoundation/feat/edit-participant

140c026

feat: refactor participant management to add/edit flow

Merge branch 'main' into asherman/helm-chart

5617ea6

Merge pull request #18 from linuxfoundation/asherman/helm-chart

4d549ab

feat: Add docker build and helm chart

Merge branch 'main' into jme/LFXV2-149

5b3c076

jordane force-pushed the main branch from 4d549ab to 28cf064 Compare August 1, 2025 00:13

asithade closed this Aug 12, 2025

asithade deleted the jme/LFXV2-149 branch August 12, 2025 22:16

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Comments

feat: add Datadog RUM session tracking integration#20

feat: add Datadog RUM session tracking integration#20
jordane wants to merge 28 commits intomainfrom
jme/LFXV2-149

jordane commented Jul 30, 2025

Uh oh!

Copilot AI left a comment

Uh oh!

Copilot AI Jul 30, 2025

Uh oh!

Copilot AI Jul 30, 2025

Uh oh!

Copilot AI Jul 30, 2025

Uh oh!

Copilot AI Jul 30, 2025

Uh oh!

github-actions bot commented Jul 31, 2025

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

4 participants

Comments

Conversation

jordane commented Jul 30, 2025

Uh oh!

Copilot AI left a comment

Choose a reason for hiding this comment

Pull Request Overview

Reviewed Changes

Uh oh!

Copilot AI Jul 30, 2025

Choose a reason for hiding this comment

Uh oh!

Copilot AI Jul 30, 2025

Choose a reason for hiding this comment

Uh oh!

Copilot AI Jul 30, 2025

Choose a reason for hiding this comment

Uh oh!

Copilot AI Jul 30, 2025

Choose a reason for hiding this comment

Uh oh!

github-actions bot commented Jul 31, 2025

✅ E2E Tests Passed

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

4 participants