Default the Datadog, New Relic and Amplitude MCP hosts to the US region - #7
Conversation
The region variables introduced in #5 now carry ${VAR:-default} so the plugins connect without any setup; validate.py checks only the variable name of a defaulted placeholder. Co-Authored-By: Mokshit Jain <mokshit@cognition.ai>
🤖 Devin AI EngineerI'll be helping with this pull request! Here's what you should know: ✅ I will automatically:
Note: I can only respond to comments from users who have write access to this repository. ⚙️ Control Options:
Original prompt from Mokshit
|
Summary
Follow-up to #5, which made the regional MCP host a required credential (
${DATADOG_DD_MCP_DOMAIN}etc.). Users on the default region shouldn't have to know their MCP hostname before the plugin works, so the five URLs now carry a shell-style default:With a saved credential the host is overridden (e.g.
mcp.us3.datadoghq.com,mcp.eu.newrelic.com,mcp.eu.amplitude.com); without one the US host is used and the field shows as optional.scripts/validate.pynow validates only the variable name of a${NAME:-default}reference (placeholder_name), so defaults pass the credential-name check.Do not merge before usacognition/devin-webapp#72615 is deployed. Until then the runtime strips/ignores the default and would connect to the literal
${…}host. Note also that the URL change is a declaration change, so existing installs of these plugins created from #5's URL will be replaced by the reconciler (saved domain/keys need re-entering) — #5 merged only recently, so exposure is small.Release notes
The Datadog, New Relic and Amplitude plugins now connect to the US region by default; set the region variable only if your account is hosted elsewhere.
Link to Devin session: https://staging.itsdev.in/sessions/422e2c28c48b4430af245ea6f267f68a
Open in Devin Desktop: https://staging.itsdev.in/desktop/session/422e2c28c48b4430af245ea6f267f68a?variant=devin-insiders
Requested by: @Mokshit06
Devin Review
67b4f6e