Skip to content

[PANA-6266] Remove ExperimentalFlag composed_path_selector 🚩#4396

Open
LFBarreto wants to merge 1 commit intomainfrom
luiz.barreto/PANA-6266-sdk-remove-ff-for-composed-path-selector
Open

[PANA-6266] Remove ExperimentalFlag composed_path_selector 🚩#4396
LFBarreto wants to merge 1 commit intomainfrom
luiz.barreto/PANA-6266-sdk-remove-ff-for-composed-path-selector

Conversation

@LFBarreto
Copy link
Copy Markdown
Collaborator

@LFBarreto LFBarreto commented Mar 25, 2026 •
edited
Loading

Motivation

PANA-6266

After internal testing the updated introduced in #4211 was validated we can now remove the experimental feature flag and collect the updated selector data for customers using future sdk versions.

work to support and use this new field has been merged on web-ui:
[PANA-6268] Support composedPathSelector alongside legacy cssSelector

Changes

This removes uses of experimental flag composed_path_selector

Test instructions

  • CI should be green
  • In the sandbox check that click events now contain _dd.action.target.composed_path_selector without the FF required
Screenshot 2026-02-24 at 11 46 54

Checklist

  • Tested locally
  • Tested on staging
  • Added unit tests for this change.
  • Added e2e/integration tests for this change.
  • Updated documentation and/or relevant AGENTS.md file

@github-actions
Copy link
Copy Markdown

github-actions bot commented Mar 25, 2026 •
edited
Loading

All contributors have signed the CLA ✍️ ✅
Posted by the CLA Assistant Lite bot.

@datadog-datadog-prod-us1-2
Copy link
Copy Markdown

datadog-datadog-prod-us1-2 bot commented Mar 25, 2026 •
edited by datadog-official bot
Loading

⚠️ Tests

✨ Fix all issues with BitsAI or with Cursor

⚠️ Other Violations

🧪 2 Tests failed

serializeNodeAsChange for snapshotted documents for a simple document when the <html> element's privacy level is MASK matches the snapshot from Chrome 63.0.3239.84 (Windows 10)   View in Datadog   (Fix with Cursor) 
Error: Timeout - Async function did not complete within 5000ms (set by jasmine.DEFAULT_TIMEOUT_INTERVAL)
    at <Jasmine>
serializeNodeAsChange for snapshotted documents for a simple document when the <html> element's privacy level is MASK matches the snapshot from Firefox 67.0 (Windows 10)   View in Datadog   (Fix with Cursor) 
Error: Timeout - Async function did not complete within 5000ms (set by jasmine.DEFAULT_TIMEOUT_INTERVAL) in node_modules/jasmine-core/lib/jasmine-core/jasmine.js (line 8638)
<Jasmine>

ℹ️ Info

No other issues found (see more)

❄️ No new flaky tests detected

🎯 Code Coverage (details)
• Patch Coverage: 100.00%
• Overall Coverage: 77.61% (-0.00%)

This comment will be updated automatically if new data arrives.
🔗 Commit SHA: 838ccae | Docs | Datadog PR Page | Was this helpful? React with 👍/👎 or give us feedback!

@cit-pr-commenter-54b7da
Copy link
Copy Markdown

cit-pr-commenter-54b7da bot commented Mar 25, 2026 •
edited
Loading

Bundles Sizes Evolution

📦 Bundle Name Base Size Local Size 𝚫 𝚫% Status
Rum 177.94 KiB 177.81 KiB -130 B -0.07% ✅
Rum Profiler 6.16 KiB 6.16 KiB 0 B 0.00% ✅
Rum Recorder 27.03 KiB 27.03 KiB 0 B 0.00% ✅
Logs 56.40 KiB 56.35 KiB -50 B -0.09% ✅
Rum Slim 133.81 KiB 133.69 KiB -122 B -0.09% ✅
Worker 23.63 KiB 23.63 KiB 0 B 0.00% ✅
🚀 CPU Performance
Action Name Base CPU Time (ms) Local CPU Time (ms) 𝚫%
RUM - add global context 0.0085 0.0058 -31.76%
RUM - add action 0.0264 0.0144 -45.45%
RUM - add error 0.0247 0.0133 -46.15%
RUM - add timing 0.0053 0.0029 -45.28%
RUM - start view 0.0233 0.0141 -39.48%
RUM - start/stop session replay recording 0.0014 0.0007 -50.00%
Logs - log message 0.0306 0.0172 -43.79%
🧠 Memory Performance
Action Name Base Memory Consumption Local Memory Consumption 𝚫
RUM - add global context 25.97 KiB 31.95 KiB +5.98 KiB
RUM - add action 98.72 KiB 111.06 KiB +12.35 KiB
RUM - add timing 26.43 KiB 32.28 KiB +5.85 KiB
RUM - add error 85.28 KiB 106.66 KiB +21.39 KiB
RUM - start/stop session replay recording 26.30 KiB 32.26 KiB +5.96 KiB
RUM - start view 486.03 KiB 490.40 KiB +4.37 KiB
Logs - log message 92.85 KiB 99.83 KiB +6.98 KiB

🔗 RealWorld

@LFBarreto LFBarreto force-pushed the luiz.barreto/PANA-6266-sdk-remove-ff-for-composed-path-selector branch 2 times, most recently from 978e156 to 4680b7a Compare March 30, 2026 12:37
@LFBarreto LFBarreto marked this pull request as ready for review March 30, 2026 12:50
@LFBarreto LFBarreto requested a review from a team as a code owner March 30, 2026 12:50
@LFBarreto LFBarreto force-pushed the luiz.barreto/PANA-6266-sdk-remove-ff-for-composed-path-selector branch from 4680b7a to 838ccae Compare April 7, 2026 14:58
@allspain allspain mentioned this pull request Apr 7, 2026
Open
7 tasks
@LFBarreto
Copy link
Copy Markdown
Collaborator Author

LFBarreto commented Apr 7, 2026

I have read the CLA Document and I hereby sign the CLA

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

Reviewers

@BenoitZugmeyer BenoitZugmeyer BenoitZugmeyer approved these changes

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@LFBarreto @BenoitZugmeyer