Skip to content

Conversation

@wconti27
Copy link
Contributor

@wconti27 wconti27 commented Dec 16, 2025

What does this PR do?

This PR was auto-generated by the APM AI Integration Toolkit and adds APM Tracing for the bullmq package.

Summary of Changes

  • Adds tracing for Bullmq message production and consumption
  • Supports both ESM / CJS code paths
  • Adds context propagation, DSM, and peer service tagging

Motivation

Plugin Checklist

Additional Notes

@wconti27 wconti27 self-assigned this Dec 16, 2025
@github-actions
Copy link

github-actions bot commented Dec 16, 2025

Overall package size

Self size: 4.38 MB
Deduped: 5.21 MB
No deduping: 5.21 MB

Dependency sizes | name | version | self size | total size | |------|---------|-----------|------------| | import-in-the-middle | 2.0.0 | 68.46 kB | 797.03 kB | | dc-polyfill | 0.1.10 | 26.73 kB | 26.73 kB |

🤖 This report was automatically generated by heaviest-objects-in-the-universe

@codecov
Copy link

codecov bot commented Dec 16, 2025

Codecov Report

❌ Patch coverage is 0% with 1 line in your changes missing coverage. Please review.
✅ Project coverage is 69.78%. Comparing base (d0cc613) to head (810ed85).

Files with missing lines Patch % Lines
...ages/datadog-instrumentations/src/helpers/hooks.js 0.00% 1 Missing ⚠️
Additional details and impacted files
@@             Coverage Diff             @@
##           master    #7123       +/-   ##
===========================================
- Coverage   84.54%   69.78%   -14.76%     
===========================================
  Files         529      285      -244     
  Lines       22542    10505    -12037     
===========================================
- Hits        19058     7331    -11727     
+ Misses       3484     3174      -310     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@pr-commenter
Copy link

pr-commenter bot commented Dec 16, 2025

Benchmarks

Benchmark execution time: 2026-01-08 17:33:10

Comparing candidate commit 810ed85 in PR branch conti/bullmq-auto-generated with baseline commit d0cc613 in branch master.

Found 0 performance improvements and 0 performance regressions! Performance is the same for 292 metrics, 28 unstable metrics.

@wconti27 wconti27 added the AI Generated Largely based on code generated by an AI or LLM. This label is the same across all dd-trace-* repos label Dec 16, 2025
@tlhunter
Copy link
Member

Shouldn't there be a new Platform / instrumentation-bullmq (pull_request) action?

@wconti27
Copy link
Contributor Author

Shouldn't there be a new Platform / instrumentation-bullmq (pull_request) action?

@tlhunter What is this? I'm not aware of what this is

@datadog-datadog-prod-us1

This comment has been minimized.

@wconti27 wconti27 requested a review from tlhunter December 18, 2025 16:01
@tlhunter
Copy link
Member

@wconti27 What is this? I'm not aware of what this is

I was saying we need a new test to appear in the list of github actions that have executed as a result of this pull request. Such a test was not present when I reviewed previously but is now working.

Screenshot 2025-12-18 at 11 53 25

@wconti27 wconti27 closed this Dec 31, 2025
@wconti27 wconti27 reopened this Jan 5, 2026
@wconti27 wconti27 marked this pull request as ready for review January 5, 2026 15:26
@wconti27 wconti27 requested review from a team as code owners January 5, 2026 15:26
@wconti27 wconti27 changed the title DRAFT: AI WRITTEN INTEGRATION bullmq feat(tracing): add bullmq integration and dsm support Jan 5, 2026
@wconti27 wconti27 requested a review from tlhunter January 5, 2026 18:03
return {
queueName: ctx.self?.name || 'bullmq',
payloadSize: data ? getMessageSize(data) : 0,
injectTarget: data
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Does BullMQ just stick JSON strings into Redis and we're adding a new field to it for context prop?

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

yeah basically (as well as allowing just about any type)

wconti27 and others added 6 commits January 7, 2026 12:32
Co-authored-by: Thomas Hunter II <tlhunter@datadog.com>
Co-authored-by: Thomas Hunter II <tlhunter@datadog.com>
Co-authored-by: Thomas Hunter II <tlhunter@datadog.com>
@wconti27 wconti27 force-pushed the conti/bullmq-auto-generated branch from c657641 to aef1d02 Compare January 7, 2026 17:33
@wconti27 wconti27 changed the base branch from conti/all-tooling-changes to conti/all-tooling-changes-2 January 7, 2026 17:33
Base automatically changed from conti/all-tooling-changes-2 to master January 7, 2026 21:52
@bengl
Copy link
Collaborator

bengl commented Jan 8, 2026

@tlhunter The orchestrion configs look fine.

tlhunter
tlhunter previously approved these changes Jan 8, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

AI Generated Largely based on code generated by an AI or LLM. This label is the same across all dd-trace-* repos semver-minor

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants