chore: add add-new-instrumentation agent skill#7564
chore: add add-new-instrumentation agent skill#7564gh-worker-dd-mergequeue-cf854d[bot] merged 2 commits intomasterfrom
Conversation
Adds a shared skill guide for creating new dd-trace instrumentations and plugins. The skill lives under `.agents/skills/` and is symlinked into both `.claude/skills/` and `.cursor/skills/` so it's available to all AI agents. Updates `.gitignore` to track the skill files while continuing to exclude other Claude/Cursor settings.
Overall package sizeSelf size: 4.68 MB Dependency sizes| name | version | self size | total size | |------|---------|-----------|------------| | import-in-the-middle | 2.0.6 | 81.92 kB | 813.08 kB | | dc-polyfill | 0.1.10 | 26.73 kB | 26.73 kB |🤖 This report was automatically generated by heaviest-objects-in-the-universe |
|
✨ Fix all issues with BitsAI or with Cursor
|
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## master #7564 +/- ##
=======================================
Coverage 80.29% 80.29%
=======================================
Files 732 732
Lines 31468 31468
=======================================
+ Hits 25266 25267 +1
+ Misses 6202 6201 -1 Flags with carried forward coverage won't be shown. Click here to find out more. ☔ View full report in Codecov by Sentry. 🚀 New features to boost your workflow:
|
gh-worker-dd-mergequeue-cf854d
Bot
merged commit 909261e
into
master
|
Hey, I would've expected a ping for the IDM team for this skill. Appreciate the effort in getting something out there, but the skill has some out of date code that we don't want to reinforce. For example, it uses traditional Shimmer based wrapping, which we would like to move away from in favor of Orchestrion wrapping. |
|
If you let me know what needs to be changed I'll happily update it, or if you think it's easier, I can approve a PR you create |
|
Maybe we should add IDM as a CODEOWNER for the folder as well then |
I have a skill that I have been developing for exactly this, i will open a PR for it and we can collaborate as needed on it. |
|
FYI: #7568. A bit more context, but I've been working on this skill for the last few weeks, it has served me well so far! |
chore: add add-new-instrumentation agent skill Adds a shared skill guide for creating new dd-trace instrumentations and plugins. The skill lives under `.agents/skills/` and is symlinked into both `.claude/skills/` and `.cursor/skills/` so it's available to all AI agents. Updates `.gitignore` to track the skill files while continuing to exclude other Claude/Cursor settings. Address review comments Co-authored-by: thomas.watson <thomas.watson@datadoghq.com>
chore: add add-new-instrumentation agent skill Adds a shared skill guide for creating new dd-trace instrumentations and plugins. The skill lives under `.agents/skills/` and is symlinked into both `.claude/skills/` and `.cursor/skills/` so it's available to all AI agents. Updates `.gitignore` to track the skill files while continuing to exclude other Claude/Cursor settings. Address review comments Co-authored-by: thomas.watson <thomas.watson@datadoghq.com>
chore: add add-new-instrumentation agent skill Adds a shared skill guide for creating new dd-trace instrumentations and plugins. The skill lives under `.agents/skills/` and is symlinked into both `.claude/skills/` and `.cursor/skills/` so it's available to all AI agents. Updates `.gitignore` to track the skill files while continuing to exclude other Claude/Cursor settings. Address review comments Co-authored-by: thomas.watson <thomas.watson@datadoghq.com>
3 participants
What does this PR do?
Adds an
add-new-instrumentationagent skill that provides a step-by-step guide for creating new dd-trace instrumentation files and plugins. The skill is stored under.agents/skills/add-new-instrumentation/SKILL.mdand symlinked into both.claude/skills/and.cursor/skills/, making it available to Claude and Cursor AI agents alike.Motivation
Adding new instrumentation requires coordinating changes across several files (
datadog-instrumentations/,datadog-plugin-*/,plugins/index.js,index.d.ts,docs/API.md, CI config, etc.). Encoding this workflow as a shared skill gives AI agents consistent, up-to-date guidance so they can scaffold new integrations correctly without missing steps.Additional Notes
.gitignoreis updated to stop ignoring.claude/skills/and.cursor/skills/so the symlinks are tracked in version control, while other Claude/Cursor local settings (e.g.settings.local.json) remain ignored.