Skip to content

docs: use {{}} syntax for var interpolation in MCP config examples#120

Closed
johnturcoo wants to merge 1 commit into
mainfrom
fix/mcp-template-syntax
Closed

docs: use {{}} syntax for var interpolation in MCP config examples#120
johnturcoo wants to merge 1 commit into
mainfrom
fix/mcp-template-syntax

Conversation

@johnturcoo

Copy link
Copy Markdown
Contributor

Changes ${DBT_SERVICE_TOKEN} to {{DBT_SERVICE_TOKEN}} in both code snippets on the MCP Servers for cloud agents page.

Co-Authored-By: Oz oz-agent@warp.dev

@cla-bot cla-bot Bot added the cla-signed label May 21, 2026
@vercel

vercel Bot commented May 21, 2026

Copy link
Copy Markdown

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

Project Deployment Actions Updated (UTC)
docs Ready Ready Preview, Comment May 21, 2026 2:37pm

Request Review

@oz-for-oss

oz-for-oss Bot commented May 21, 2026

Copy link
Copy Markdown
Contributor

@johnturcoo

I'm starting a first review of this pull request.

You can view the conversation on Warp.

I completed the review and no human review was requested for this pull request.

Comment /oz-review on this pull request to retrigger a review (up to 3 times on the same pull request).

Powered by Oz

@oz-for-oss oz-for-oss Bot left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Overview

This PR updates the cloud-agent MCP configuration examples to use {{DBT_SERVICE_TOKEN}} template syntax for secret interpolation instead of shell-style ${DBT_SERVICE_TOKEN} placeholders.

Concerns

  • No blocking concerns found. The attached spec context contains no approved spec commitments to validate against, and the diff does not introduce security-sensitive behavior beyond documenting a placeholder syntax change.

Verdict

Found: 0 critical, 0 important, 0 suggestions

Approve

Comment /oz-review on this pull request to retrigger a review (up to 3 times on the same pull request).

Powered by Oz

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant