fix(vitest): propagate no-worker request error tags#9189
Conversation
Overall package sizeSelf size: 6.52 MB Dependency sizes| name | version | self size | total size | |------|---------|-----------|------------| | import-in-the-middle | 3.2.0 | 104.26 kB | 843.44 kB | | opentracing | 0.14.7 | 194.81 kB | 194.81 kB | | dc-polyfill | 0.1.11 | 25.74 kB | 25.74 kB |🤖 This report was automatically generated by heaviest-objects-in-the-universe |
|
@codex review |
BenchmarksBenchmark execution time: 2026-07-02 15:06:38 Comparing candidate commit baec21e in PR branch Found 0 performance improvements and 0 performance regressions! Performance is the same for 2247 metrics, 39 unstable metrics.
|
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: 556ed47b6f
ℹ️ About Codex in GitHub
Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you
- Open a pull request for review
- Mark a draft as ready
- Comment "@codex review".
If Codex has suggestions, it will comment; otherwise it will react with 👍.
Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".
Codecov Report❌ Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## master #9189 +/- ##
==========================================
+ Coverage 93.63% 93.65% +0.01%
==========================================
Files 896 898 +2
Lines 52220 52371 +151
Branches 12265 12322 +57
==========================================
+ Hits 48895 49046 +151
Misses 3325 3325
Flags with carried forward coverage won't be shown. Click here to find out more. ☔ View full report in Codecov by Harness. 🚀 New features to boost your workflow:
|
🎉 All green!🧪 All tests passed 🎯 Code Coverage (details) 🔗 Commit SHA: baec21e | Docs | Datadog PR Page | Give us feedback! |
|
@codex review |
|
/datadog autotest review |
|
Codex Review: Didn't find any major issues. Bravo. Reviewed commit: ℹ️ About Codex in GitHubYour team has set up Codex to review pull requests in this repo. Reviews are triggered when you
If Codex has suggestions, it will comment; otherwise it will react with 👍. Codex can also answer questions or update the PR. Try commenting "@codex address that feedback". |
There was a problem hiding this comment.
More details
PR correctly propagates Test Optimization request error tags to no-worker-init reporting. The implementation extracts tags from all request types (settings, known tests, test management), accumulates them in a module-level variable, and applies them to session, suite, and test events. No behavioral regressions detected.
🤖 Datadog Autotest · Commit baec21e · What is Autotest? · Any feedback? Reach out in #autotest
What does this PR do?
Propagates Test Optimization request error tags through Vitest no-worker-init reporting.
Specifically, this returns the current request error tags from settings, known-tests, and test-management requests, stores them in Vitest main no-worker state, and forwards them to no-worker session, suite, and test events, including skipped tests.
Motivation
When a Test Optimization request fails, normal worker instrumentation tags emitted events with the request error. No-worker-init reporting should keep that diagnostic signal on the same event types even when worker instrumentation is skipped.