[Docs] add `CLAUDE.md`, updates to `AGENTS.md` by lucaspimentel · Pull Request #7602 · DataDog/dd-trace-dotnet

lucaspimentel · 2025-10-01T21:57:47Z

Summary

Adds CLAUDE.md to provide AI agent context for the repository and enhances AGENTS.md with coding style and PR guidelines.

Reason for change

Improves AI agent interaction with the repository by providing a clear entry point (CLAUDE.md) that references the comprehensive AGENTS.md guidelines. Also clarifies coding conventions and PR expectations.

Implementation details

Created .claude/CLAUDE.md that references AGENTS.md for detailed repository guidelines
Updated AGENTS.md with three clarifications:
- Coding style: Prefer adding using directives instead of fully-qualified type names
- Coding style: Use modern C# syntax while maintaining compatibility with older runtimes
- PR guidelines:follow the existing PR description template

Test coverage

Documentation-only change, no tests required.

Other details

This PR description was mostly AI-generated.

dd-trace-dotnet-ci-bot · 2025-10-01T22:54:36Z

Execution-Time Benchmarks Report ⏱️

Execution-time results for samples comparing the following branches/commits:

Execution-time benchmarks measure the whole time it takes to execute a program. And are intended to measure the one-off costs. Cases where the execution time results for the PR are worse than latest master results are shown in red. The following thresholds were used for comparing the execution times:

Welch test with statistical test for significance of 5%
Only results indicating a difference greater than 5% and 5 ms are considered.

Note that these results are based on a single point-in-time result for each branch. For full results, see the dashboard.

Graphs show the p99 interval based on the mean and StdDev of the test run, as well as the mean value of the run (shown as a diamond below the graph).

gantt
    title Execution time (ms) FakeDbCommand (.NET Framework 4.8) 
    dateFormat  X
    axisFormat %s
    todayMarker off
    section Bailout
    This PR (7602) - mean (72ms)  : 71, 73
     .   : milestone, 72,
    master - mean (73ms)  : 71, 75
     .   : milestone, 73,

    section Baseline
    This PR (7602) - mean (68ms)  : 66, 70
     .   : milestone, 68,
    master - mean (70ms)  : 67, 73
     .   : milestone, 70,

    section CallTarget+Inlining+NGEN
    This PR (7602) - mean (1,050ms)  : 1014, 1086
     .   : milestone, 1050,
    master - mean (1,050ms)  : 1011, 1090
     .   : milestone, 1050,

gantt
    title Execution time (ms) FakeDbCommand (.NET Core 3.1) 
    dateFormat  X
    axisFormat %s
    todayMarker off
    section Bailout
    This PR (7602) - mean (107ms)  : 105, 108
     .   : milestone, 107,
    master - mean (108ms)  : 106, 109
     .   : milestone, 108,

    section Baseline
    This PR (7602) - mean (106ms)  : 104, 108
     .   : milestone, 106,
    master - mean (107ms)  : 104, 109
     .   : milestone, 107,

    section CallTarget+Inlining+NGEN
    This PR (7602) - mean (744ms)  : 722, 767
     .   : milestone, 744,
    master - mean (747ms)  : 727, 767
     .   : milestone, 747,

gantt
    title Execution time (ms) FakeDbCommand (.NET 6) 
    dateFormat  X
    axisFormat %s
    todayMarker off
    section Bailout
    This PR (7602) - mean (100ms)  : 99, 102
     .   : milestone, 100,
    master - mean (102ms)  : 100, 104
     .   : milestone, 102,

    section Baseline
    This PR (7602) - mean (100ms)  : 98, 102
     .   : milestone, 100,
    master - mean (101ms)  : 98, 104
     .   : milestone, 101,

    section CallTarget+Inlining+NGEN
    This PR (7602) - mean (781ms)  : 739, 823
     .   : milestone, 781,
    master - mean (785ms)  : 750, 819
     .   : milestone, 785,

gantt
    title Execution time (ms) FakeDbCommand (.NET 8) 
    dateFormat  X
    axisFormat %s
    todayMarker off
    section Bailout
    This PR (7602) - mean (92ms)  : 91, 94
     .   : milestone, 92,
    master - mean (93ms)  : 92, 95
     .   : milestone, 93,

    section Baseline
    This PR (7602) - mean (92ms)  : 90, 94
     .   : milestone, 92,
    master - mean (93ms)  : 91, 96
     .   : milestone, 93,

    section CallTarget+Inlining+NGEN
    This PR (7602) - mean (660ms)  : 647, 673
     .   : milestone, 660,
    master - mean (666ms)  : 651, 682
     .   : milestone, 666,

gantt
    title Execution time (ms) HttpMessageHandler (.NET Framework 4.8) 
    dateFormat  X
    axisFormat %s
    todayMarker off
    section Bailout
    This PR (7602) - mean (201ms)  : 197, 206
     .   : milestone, 201,
    master - mean (213ms)  : 199, 226
     .   : milestone, 213,

    section Baseline
    This PR (7602) - mean (198ms)  : 193, 204
     .   : milestone, 198,
    master - mean (206ms)  : 193, 218
     .   : milestone, 206,

    section CallTarget+Inlining+NGEN
    This PR (7602) - mean (1,194ms)  : 1106, 1282
     .   : milestone, 1194,
    master - mean (1,225ms)  : 1148, 1301
     .   : milestone, 1225,

gantt
    title Execution time (ms) HttpMessageHandler (.NET Core 3.1) 
    dateFormat  X
    axisFormat %s
    todayMarker off
    section Bailout
    This PR (7602) - mean (287ms)  : 279, 294
     .   : milestone, 287,
    master - mean (295ms)  : 277, 312
     .   : milestone, 295,

    section Baseline
    This PR (7602) - mean (286ms)  : 277, 296
     .   : milestone, 286,
    master - mean (298ms)  : 278, 318
     .   : milestone, 298,

    section CallTarget+Inlining+NGEN
    This PR (7602) - mean (951ms)  : 915, 988
     .   : milestone, 951,
    master - mean (983ms)  : 938, 1028
     .   : milestone, 983,

gantt
    title Execution time (ms) HttpMessageHandler (.NET 6) 
    dateFormat  X
    axisFormat %s
    todayMarker off
    section Bailout
    This PR (7602) - mean (289ms)  : 283, 296
     .   : milestone, 289,
    master - mean (305ms)  : 287, 323
     .   : milestone, 305,

    section Baseline
    This PR (7602) - mean (290ms)  : 278, 301
     .   : milestone, 290,
    master - mean (302ms)  : 281, 322
     .   : milestone, 302,

    section CallTarget+Inlining+NGEN
    This PR (7602) - mean (1,012ms)  : 961, 1062
     .   : milestone, 1012,
    master - mean (1,042ms)  : 985, 1098
     .   : milestone, 1042,

gantt
    title Execution time (ms) HttpMessageHandler (.NET 8) 
    dateFormat  X
    axisFormat %s
    todayMarker off
    section Bailout
    This PR (7602) - mean (278ms)  : 271, 285
     .   : milestone, 278,
    master - mean (292ms)  : 271, 312
     .   : milestone, 292,

    section Baseline
    This PR (7602) - mean (277ms)  : 268, 286
     .   : milestone, 277,
    master - mean (291ms)  : 270, 311
     .   : milestone, 291,

    section CallTarget+Inlining+NGEN
    This PR (7602) - mean (867ms)  : 843, 892
     .   : milestone, 867,
    master - mean (932ms)  : 840, 1024
     .   : milestone, 932,

Quote the glob pattern '**/CLAUDE.md' to prevent it from being interpreted as a YAML anchor alias. 🤖 Generated with [Claude Code](https://claude.com/claude-code) Co-Authored-By: Claude <noreply@anthropic.com>

lucaspimentel added 2 commits October 1, 2025 17:56

add some notes to AGENTS.md

8a00ff9

add CLAUDE.md with a reference to existing AGENTS.md

c2eee0d

lucaspimentel added the area:docs label Oct 1, 2025

lucaspimentel added 2 commits October 1, 2025 18:03

move file

34a1315

add note to follow PR description template

9c89c44

lucaspimentel changed the title ~~add CLAUDE.md~~ add CLAUDE.md, updates to AGENTS.md Oct 1, 2025

lucaspimentel marked this pull request as ready for review October 1, 2025 22:11

lucaspimentel requested a review from a team as a code owner October 1, 2025 22:11

tonyredondo approved these changes Oct 1, 2025

View reviewed changes

lucaspimentel enabled auto-merge (squash) October 1, 2025 22:13

link04 approved these changes Oct 1, 2025

View reviewed changes

lucaspimentel and others added 3 commits October 1, 2025 19:05

skip building when these files are modified

d36534a

clarify

3e7d403

quote the literal glob pattern string

0a10dd8

Quote the glob pattern '**/CLAUDE.md' to prevent it from being interpreted as a YAML anchor alias. 🤖 Generated with [Claude Code](https://claude.com/claude-code) Co-Authored-By: Claude <noreply@anthropic.com>

lucaspimentel merged commit 45217f4 into master Oct 2, 2025
149 of 151 checks passed

lucaspimentel deleted the lpimentel/claude.md branch October 2, 2025 02:15

github-actions bot added this to the vNext-v3 milestone Oct 2, 2025

lucaspimentel mentioned this pull request Oct 2, 2025

Add performance, testing, logging, Azure Functions, and glossary to AGENTS.md #7609

Merged

lucaspimentel changed the title ~~add CLAUDE.md, updates to AGENTS.md~~ [Docs] add CLAUDE.md, updates to AGENTS.md Oct 27, 2025

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

[Docs] add `CLAUDE.md`, updates to `AGENTS.md`#7602

[Docs] add `CLAUDE.md`, updates to `AGENTS.md`#7602
lucaspimentel merged 7 commits intomasterfrom
lpimentel/claude.md

lucaspimentel commented Oct 1, 2025 •

edited

Loading

Uh oh!

dd-trace-dotnet-ci-bot bot commented Oct 1, 2025 •

edited

Loading

Uh oh!

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

3 participants

Conversation

lucaspimentel commented Oct 1, 2025 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Summary

Reason for change

Implementation details

Test coverage

Other details

Uh oh!

dd-trace-dotnet-ci-bot bot commented Oct 1, 2025 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Execution-Time Benchmarks Report ⏱️

Uh oh!

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

3 participants

lucaspimentel commented Oct 1, 2025 •

edited

Loading

dd-trace-dotnet-ci-bot bot commented Oct 1, 2025 •

edited

Loading