[hhan2024]

Summary

Uses the shared serial-ID tests merged in #7820.

• Use OpenFeature get*Details APIs in the Node.js parametric adapter and forward the actual SDK value, reason, and flagMetadata.
• Preserve a null metadata field when the SDK supplies no metadata; do not construct serial IDs in the adapter.
• Enable the serial-ID tests for Node.js from v6.18.0, the released version validated locally. This is a conservative verified floor, not a claim that earlier releases fail.

The released Node.js SDK already exposes __dd_split_serial_id, so no SDK implementation change is needed.

Validation

• After rebasing onto main: rebuilt the parametric adapter; all 29 serial-ID and dynamic-evaluation tests passed with Node.js 6.21.0. ./format.sh --check passed.
• Node.js dd-trace 6.18.0: 29 passed without force-execution (5 serial-ID tests + 24 dynamic-evaluation tests).
• Positive IDs and zero are preserved; missing IDs are absent, with span enrichment enabled.
• ./format.sh passed.
• node --check and git diff --check passed.

Command: TEST_LIBRARY=nodejs ./run.sh PARAMETRIC --skip-parametric-build tests/parametric/test_ffe/test_serial_id_metadata.py tests/parametric/test_ffe/test_dynamic_evaluation.py

[github-actions]

CODEOWNERS have been resolved as:

manifests/nodejs.yml                                                    @DataDog/system-tests-reviewers
utils/build/docker/nodejs/parametric/server.js                          @DataDog/system-tests-reviewers

[datadog-official]

✅ All CI checks and tests passed.

🎉 All green!

🧪 All tests passed
❄️ No new flaky tests detected

_{This comment will be updated automatically if new data arrives.
🔗 Commit SHA: 7d614be | Docs | View more details | Give us feedback!}

[nccatoni]

Looks like there is a merge conflict with utils/pytest.py otherwise LGTM. You should also get a review from someone familiar whit the feature

[typotter]

lgtm pending merge conflicts