chore(agents): add verbatim dd-apm-sdk-review skill copy - #12459
chore(agents): add verbatim dd-apm-sdk-review skill copy#12459robertomonteromiguel wants to merge 7 commits into
Conversation
Mirror the shared skill plus Claude/Cursor links. Repo-specific overrides and the perf-review replacement land in the follow-up.
|
@codex review |
|
Codex Review: Didn't find any major issues. More of your lovely PRs please. Reviewed commit: ℹ️ About Codex in GitHubCodex has been enabled to automatically review pull requests in this repo. Reviews are triggered when you
If Codex has suggestions, it will comment; otherwise it will react with 👍. When you sign up for Codex through ChatGPT, Codex can also answer questions or update the PR, like "@codex address that feedback". |
This comment has been minimized.
This comment has been minimized.
🟢 Java Benchmark SLOs — All performance SLOs passed
PR vs. master results
Commit: Load and DaCapo benchmarks can be triggered manually in the GitLab pipeline. Results will appear in the Benchmarking Platform UI after completion. |
Bring in the portable gh --repo pin, optional repo-context, and .claude/.cursor instruction-only shortcut without tracer-specific edits.
|
@codex review |
|
Codex Review: Didn't find any major issues. What shall we delve into next? Reviewed commit: ℹ️ About Codex in GitHubCodex has been enabled to automatically review pull requests in this repo. Reviews are triggered when you
If Codex has suggestions, it will comment; otherwise it will react with 👍. When you sign up for Codex through ChatGPT, Codex can also answer questions or update the PR, like "@codex address that feedback". |
Load the data-not-instructions rule before repo-context.md.
|
@codex review |
|
Codex Review: Didn't find any major issues. Can't wait for the next one! Reviewed commit: ℹ️ About Codex in GitHubCodex has been enabled to automatically review pull requests in this repo. Reviews are triggered when you
If Codex has suggestions, it will comment; otherwise it will react with 👍. When you sign up for Codex through ChatGPT, Codex can also answer questions or update the PR, like "@codex address that feedback". |
Scan PR title, labels, and commit subjects before printing, using the same SECRET_GREP path as the diffs.
|
@codex review |
|
Codex Review: Didn't find any major issues. Nice work! Reviewed commit: ℹ️ About Codex in GitHubCodex has been enabled to automatically review pull requests in this repo. Reviews are triggered when you
If Codex has suggestions, it will comment; otherwise it will react with 👍. When you sign up for Codex through ChatGPT, Codex can also answer questions or update the PR, like "@codex address that feedback". |
|
Hi! 👋 Thanks for your pull request! 🎉 To help us review it, please make sure to:
If you need help, please check our contributing guidelines. |
![datadog-datadog-prod-us1[bot]](https://avatars.githubusercontent.com/u/365230?s=60&v=4){kind=small}
There was a problem hiding this comment.
More details
The ownership rule covers Cursor content. The ignore rule excludes local Cursor data and allows the shared skill link.
🤖 Datadog Autotest · Commit 745e7a3 · What is Autotest? · @DataDog review to ask questions · Any feedback? Reach out in #autotest
|
The "Check pull requests" status check is failing: "Please add at least one type, and one component or instrumentation label to the pull request." Per this repo's conventions, please add:
|
robertomonteromiguel
added
type: feature
Verbatim copy: binary secret-scan fail-closed, LC_ALL=C git diff, and private routing for no-harness security findings.
2 participants
What does this PR do?
Adds a verbatim copy of the shared
dd-apm-sdk-reviewskill so local agents can load it from disk..agents/skills/dd-apm-sdk-review/plus Claude/Cursor skill links@DataDog/apm-java(including/.cursor/so the Cursor link is owned)This is PR 1 of 2. Java-specific overrides and the
perf-reviewreplacement land in the follow-up stacked on this branch.Motivation
Split out of #12364 so Java reviewers are not asked to read the shared skill copy.
Additional Notes
How to review
.agents/skills/dd-apm-sdk-review/. That is a mirror of dd-apm-sdk-review-core#1..gitignore(Cursor skill-link exception) only.Do not merge this PR on its own — land it with the overrides follow-up.
Made with Cursor