Skip to content

feat(jms): toolkit-generated JMS 1.1 instrumentation v3 [DO NOT MERGE]#11596

Draft
jordan-wong wants to merge 3 commits into
masterfrom
apm-ai-toolkit/new_integration/jms/20260608-033204
Draft

feat(jms): toolkit-generated JMS 1.1 instrumentation v3 [DO NOT MERGE]#11596
jordan-wong wants to merge 3 commits into
masterfrom
apm-ai-toolkit/new_integration/jms/20260608-033204

Conversation

@jordan-wong
Copy link
Copy Markdown
Contributor

@jordan-wong jordan-wong commented Jun 8, 2026 •
edited
Loading

🤖 Generated with APM Instrumentation Toolkit

Summary

  • AI-generated JMS 1.1 instrumentation from blind test on fresh master (`3db6cc5d36`)
  • Module placed in `javax-jms-1.1-generated/` — compare with existing `javax-jms-1.1/` side-by-side in the diff viewer
  • Instruments all 9 targets: `MessageProducer.send` (4 overloads), `MessageConsumer.receive` (2 overloads), `receiveNoWait`, `MessageListener.onMessage`, `Message.acknowledge`
  • Features: DSM (Data Streams Monitoring), distributed context propagation, peer service

To merge

  1. Rename `dd-java-agent/instrumentation/jms/javax-jms-1.1-generated/` → `javax-jms-1.1/`
  2. Update `settings.gradle.kts` entry accordingly
  3. Remove `[DO NOT MERGE]` from title

Generation details

Field Value
Workflow `new_integration` (v3)
Toolkit commit `569fcc87`
Base commit `3db6cc5d36` (fresh master)
Reviewer approved ✅ (10 iterations)
Reported cost ~$51

Test plan

  • `:check` (compile + spotless + codeNarc + unit tests)
  • `:muzzle` AssertPass
  • `:instrumentationLatestDepTest`
  • `:forkedTest` on dependents (spring-jms-3.1)

🤖 Generated with APM Instrumentation Toolkit

jordan-wong and others added 2 commits June 7, 2026 23:24
@jordan-wong
eval: delete jms for blind test v3 (master 3db6cc5)
f80c597
@jordan-wong @claude
feat(jms): toolkit-generated JMS 1.1 instrumentation v3 [DO NOT MERGE]
0545d35 
Generated by APM Instrumentation Toolkit new_integration workflow.
Reviewer approved (10 iterations, 568 turns).
Cost: ~$50.87. Java tests: JMS1Test.java.
Features: DSM + context propagation + peer service.
Blind test on master 3db6cc5.

Co-Authored-By: Claude Sonnet 4.6 (1M context) <noreply@anthropic.com>
@datadog-datadog-prod-us1-2
Copy link
Copy Markdown
Contributor

datadog-datadog-prod-us1-2 Bot commented Jun 8, 2026 •
edited by datadog-prod-us1-5 Bot
Loading

Pipelines

✨ Fix all issues with BitsAI

⚠️ Warnings

🚦 33 Pipeline jobs failed

DataDog/apm-reliability/dd-trace-java | Amazon_Linux_2023_amd64.SI94: [test-app-java-buildpack]   View in Datadog   GitLab

DataDog/apm-reliability/dd-trace-java | Ubuntu_20_amd64.SI94: [test-app-java-buildpack]   View in Datadog   GitLab

DataDog/apm-reliability/dd-trace-java | check-instrumentation-naming   View in Datadog   GitLab

View all 33 failed jobs.

Useful? React with 👍 / 👎

This comment will be updated automatically if new data arrives.
🔗 Commit SHA: 6b8935a | Docs | Datadog PR Page | Give us feedback!

@dd-octo-sts
Copy link
Copy Markdown
Contributor

dd-octo-sts Bot commented Jun 8, 2026 •
edited
Loading

🟢 Java Benchmark SLOs — All performance SLOs passed

Suite Status
Startup 🟢 pass

SLO thresholds are defined here based on automatically generated metrics. A warning is raised when results are within 5% of the threshold.

PR vs. master results
Scenario Candidate master Δ (95% CI of mean)
startup:insecure-bank:iast:Agent 13.89 s 13.98 s [-1.6%; +0.2%] (no difference)
startup:insecure-bank:tracing:Agent 12.90 s 12.95 s [-1.5%; +0.7%] (no difference)
startup:petclinic:appsec:Agent 16.90 s 16.60 s [+0.3%; +3.3%] (maybe worse)
startup:petclinic:iast:Agent 16.87 s 16.99 s [-1.8%; +0.3%] (no difference)
startup:petclinic:profiling:Agent 16.72 s 16.79 s [-1.7%; +0.9%] (no difference)
startup:petclinic:sca:Agent 16.87 s 16.53 s [+0.5%; +3.6%] (maybe worse)
startup:petclinic:tracing:Agent 16.00 s 15.04 s [-2.1%; +14.9%] (unstable)

Commit: 6b8935a0 · CI Pipeline · Benchmarking Platform UI

Load and DaCapo benchmarks can be triggered manually in the GitLab pipeline. Results will appear in the Benchmarking Platform UI after completion.

@jordan-wong @claude
refactor(jms): rename javax-jms-1.1 -> javax-jms-1.1-generated for ev…
6b8935a 
…al convention

Eval runs place generated modules in {module}-generated/ directories so
reviewers can compare generated vs existing side-by-side in the PR diff.
To merge: rename back to javax-jms-1.1 and update settings.gradle.kts.

Co-Authored-By: Claude Sonnet 4.6 (1M context) <noreply@anthropic.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

1 participant

@jordan-wong