Skip to content

docs(datadog): disclaim that the plugin cannot work until DCR is unlocked#214

Merged
dcramer merged 1 commit intomainfrom
devin/1776466146-datadog-dcr-disclaimer
Apr 17, 2026
Merged

docs(datadog): disclaim that the plugin cannot work until DCR is unlocked#214
dcramer merged 1 commit intomainfrom
devin/1776466146-datadog-dcr-disclaimer

Conversation

@devin-ai-integration
Copy link
Copy Markdown
Contributor

Summary

@sentry/junior-datadog can't actually connect today: Datadog's hosted MCP server (mcp.datadoghq.com) requires OAuth 2.1 + PKCE, but does not expose Dynamic Client Registration (DCR, RFC 7591) for third-party clients like Junior. Without DCR (or an equivalent registration path), Junior has no way to register an OAuth client, so the auth handshake fails at connect time and every Datadog tool call fails.

Rather than yank the plugin, per David's call we're keeping it in-tree and slapping a big disclaimer on it so nobody deploys it expecting it to work. When Datadog opens up DCR, we flip the disclaimer off.

Changes:

  • packages/junior-datadog/README.md: adds a > [!WARNING] callout at the top explaining the DCR lockdown and that the package is not production-ready.
  • packages/docs/src/content/docs/extend/datadog-plugin.md: adds the same disclaimer as a Starlight :::danger aside at the top of the tutorial page so it's visible on junior.sentry.dev.
  • Monorepo README.md: marks @sentry/junior-datadog as non-functional in the package table, linking to the package README.

No plugin, manifest, or runtime code is touched — this is docs/README only.

Review & Testing Checklist for Human

  • Read the two disclaimers (package README and docs page) and confirm the wording is how you want to communicate the DCR issue externally. I called it "Datadog has DCR locked down" — swap if you want softer/stronger language.
  • Confirm you're OK pointing external readers at RFC 7591 as the DCR reference. If there's a Datadog-side issue/tracking link you'd rather cite, drop that in.
  • Decide whether the monorepo README.md package-table should also drop the Datadog row entirely instead of marking it non-functional. I left it in since the package is still shipped.

Notes

  • Starlight supports :::danger asides out of the box, but nothing else in packages/docs currently uses asides, so the datadog-plugin.md page is the first to render one. Confirmed via pnpm docs:check that the build succeeds and produces /extend/datadog-plugin/.
  • pnpm lint passes. No typecheck/test changes since this is docs-only.
  • Intentionally did not remove @sentry/junior-datadog from apps/example/plugin-packages.ts, .craft.yml, or the release workflow. Happy to do that as a follow-up if you'd rather stop publishing releases until it works.

Link to Devin session: https://app.devin.ai/sessions/b41bfa31002d4335adca8cdb0d7cdf4d
Requested by: @dcramer

Datadog's hosted MCP server requires OAuth Dynamic Client Registration
(DCR, RFC 7591) for third-party clients, and DCR is not exposed by
Datadog. The @sentry/junior-datadog package therefore cannot complete
the OAuth handshake today. Add a prominent disclaimer at the top of the
package README and the Datadog plugin tutorial page so operators do not
deploy it expecting it to work, and mark the package as non-functional
in the monorepo README package table.

Co-Authored-By: Devin AI <devin-ai-integration[bot]@users.noreply.github.com>
@devin-ai-integration
Copy link
Copy Markdown
Contributor Author

🤖 Devin AI Engineer

I'll be helping with this pull request! Here's what you should know:

✅ I will automatically:

  • Address comments on this PR that start with 'DevinAI' or '@devin'.
  • Look at CI failures and help fix them

Note: I can only respond to comments from users who have write access to this repository.

⚙️ Control Options:

  • Disable automatic comment and CI monitoring

@devin-ai-integration devin-ai-integration Bot requested a review from dcramer April 17, 2026 22:51
@vercel
Copy link
Copy Markdown

vercel Bot commented Apr 17, 2026

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
junior-docs Ready Ready Preview, Comment Apr 17, 2026 10:51pm

Request Review

Copy link
Copy Markdown
Contributor Author

@devin-ai-integration devin-ai-integration Bot left a comment

Choose a reason for hiding this comment

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

✅ Devin Review: No Issues Found

Devin Review analyzed this PR and found no bugs or issues to report.

Open in Devin Review

@dcramer dcramer merged commit 5f9c80c into main Apr 17, 2026
15 checks passed
@dcramer dcramer deleted the devin/1776466146-datadog-dcr-disclaimer branch April 17, 2026 23:02
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant