Skip to content

Conversation

cy-moi
Copy link
Collaborator

@cy-moi cy-moi commented Aug 26, 2025

Motivation

We want to monitor the different results from tree walker approach and innerText.

Changes

Run both strategies behind the FF and send telemetries when they are different.

Test instructions

Checklist

  • Tested locally
  • Tested on staging
  • Added unit tests for this change.
  • Added e2e/integration tests for this change.

@cy-moi
Copy link
Collaborator Author

cy-moi commented Aug 26, 2025

/to-staging

@dd-devflow-routing-codex
Copy link

dd-devflow-routing-codex bot commented Aug 26, 2025

View all feedbacks in Devflow UI.

2025-08-26 12:09:27 UTC ℹ️ Start processing command /to-staging


2025-08-26 12:09:37 UTC ℹ️ Branch Integration: starting soon, merge expected in approximately 0s (p90)

Commit 6890d08280 will soon be integrated into staging-35.


2025-08-26 12:21:32 UTC ℹ️ Branch Integration: this commit was successfully integrated

Commit 6890d08280 has been merged into staging-35 in merge commit 1ddfd7ba59.

Check out the triggered pipeline on Gitlab 🦊

If you need to revert this integration, you can use the following command: /code revert-integration -b staging-35

dd-mergequeue bot added a commit that referenced this pull request Aug 26, 2025
@cy-moi cy-moi force-pushed the congyao/add-telemetry-for-action-name-diff branch from 1c93cdc to 9be8e60 Compare August 26, 2025 12:50
@datadog-datadog-prod-us1
Copy link

datadog-datadog-prod-us1 bot commented Aug 26, 2025

✅ Tests

🎉 All green!

❄️ No new flaky tests detected
🧪 All tests passed

🎯 Code Coverage
Patch Coverage: 100.00%
Total Coverage: 92.48% (-0.04%)

View detailed report

This comment will be updated automatically if new data arrives.
🔗 Commit SHA: 55397d5 | Docs | Was this helpful? Give us feedback!

@cy-moi cy-moi force-pushed the congyao/add-telemetry-for-action-name-diff branch from 9be8e60 to 55397d5 Compare August 26, 2025 12:55
Copy link

Bundles Sizes Evolution

📦 Bundle Name Base Size Local Size 𝚫 𝚫% Status
Rum 154.99 KiB 155.14 KiB 154 B 0.10%
Rum Recorder 19.53 KiB 19.53 KiB 0 B 0.00%
Rum Profiler 5.18 KiB 5.18 KiB 0 B 0.00%
Logs 55.11 KiB 55.11 KiB 0 B 0.00%
Flagging N/A 931 B 931 B N/A%
Rum Slim 112.86 KiB 113.00 KiB 148 B 0.13%
Worker 23.60 KiB 23.60 KiB 0 B 0.00%
🚀 CPU Performance
Action Name Base Average Cpu Time (ms) Local Average Cpu Time (ms) 𝚫
addglobalcontext 0.013 0.011 -0.001
addaction 0.043 0.035 -0.007
addtiming 0.009 0.007 -0.001
adderror 0.038 0.039 0.001
startstopsessionreplayrecording 0.002 0.002 0.000
startview 0.010 0.009 -0.002
logmessage 0.037 0.043 0.006
🧠 Memory Performance
Action Name Base Consumption Memory (bytes) Local Consumption Memory (bytes) 𝚫 (bytes)
addglobalcontext 25.64 KiB 25.52 KiB -118 B
addaction 45.51 KiB 46.00 KiB 499 B
addtiming 25.09 KiB 24.94 KiB -156 B
adderror 49.92 KiB 50.97 KiB 1.05 KiB
startstopsessionreplayrecording 24.40 KiB 25.18 KiB 793 B
startview 421.01 KiB 425.48 KiB 4.46 KiB
logmessage 75.54 KiB 41.81 KiB -34540 B

🔗 RealWorld

@cy-moi
Copy link
Collaborator Author

cy-moi commented Aug 26, 2025

/to-staging

@dd-devflow-routing-codex
Copy link

dd-devflow-routing-codex bot commented Aug 26, 2025

View all feedbacks in Devflow UI.

2025-08-26 14:33:51 UTC ℹ️ Start processing command /to-staging


2025-08-26 14:34:14 UTC ℹ️ Branch Integration: starting soon, merge expected in approximately 11m49s (p90)

Commit 55397d5963 will soon be integrated into staging-35.


2025-08-26 14:49:28 UTC ℹ️ Branch Integration: this commit was successfully integrated

Commit 55397d5963 has been merged into staging-35 in merge commit 258af36764.

Check out the triggered pipeline on Gitlab 🦊

If you need to revert this integration, you can use the following command: /code revert-integration -b staging-35

dd-mergequeue bot added a commit that referenced this pull request Aug 26, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant