Document MCP API token authentication#537
Conversation
Documents API token auth as an alternative to OAuth for the Trunk MCP server. Useful for CI/headless environments and clients that don't support OAuth. Source: trunk-io/trunk2#3381 Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
Documents API token auth as an alternative to OAuth for the Trunk MCP server. Covers the README overview and all four client config pages (Claude Code, Cursor, GitHub Copilot, Gemini). Shipped in trunk2 v148 (#3381). Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com> https://claude.ai/code/session_01U4k4g3jSvW8Nu7LiCMtPNk
… examples Resolves merge conflict with remote branch. Improves README authorization section with OAuth spec link and org-level attribution note. Adds dedicated API token sections to all four client config pages (Claude Code, Cursor, GitHub Copilot, Gemini) with env var interpolation examples. Shipped in trunk2 v148 (#3381). Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com> https://claude.ai/code/session_01U4k4g3jSvW8Nu7LiCMtPNk
|
Merging to
After your PR is submitted to the merge queue, this comment will be automatically updated with its status. If the PR fails, failure details will also be posted here |
|
Claude finished @samgutentag's task in 2m 3s —— View job PR Review
SummaryThis PR is well-structured and accurately documents the API token authentication option across the overview page and all four client-specific pages. No blocking issues found. What's good
Minor suggestions (non-blocking)
Pre-existing typos (not introduced by this PR, worth a follow-up):
Org-level caveat visibility — The README mentions "API token auth is org-level — all requests are attributed to the organization rather than to a specific user", but this isn't repeated on the per-client pages. Users who navigate directly to a client page won't see this important note. Consider adding a short callout or linking back to the overview. Overall: no blocking issues. The documentation is accurate, consistent, and follows the existing patterns in the repo. |
| { | ||
| "mcpServers": { | ||
| "trunk": { | ||
| "url": "https://mcp.trunk.io/mcp", |
There was a problem hiding this comment.
Minor: this generic example uses "url" which is correct for Claude Code, Cursor, and GitHub Copilot — but Gemini CLI requires "httpUrl" instead. Consider adding a note that the exact config key varies by client, with a pointer to the client-specific pages.
|
Claude finished @samgutentag's task in 1m 2s —— View job PR Review
SummaryThis PR is well-structured. The new API token sections are accurate, consistently formatted across all four client pages, and the client-specific differences (Gemini's What's good
Minor suggestions (non-blocking)
Consistency — The Overall: no blocking issues. Accurate, consistent, and well-structured. |
|
Triage status (2026-05-05): up to date with main. Review approved. Ready to merge. |
|
Verification status (2026-05-06): Verified: customers can use this. Ready to publish.
Branch is up to date with main. Suggested next action: ready to publish after content review. |
|
Claude finished @samgutentag's task in 1m 46s —— View job PR Review
SummaryLatest commit is a merge from What's good
Minor (non-blocking, carried over from prior reviews)These were flagged in the 2026-04-10 and 2026-05-05 reviews and are still open. None of them block merge:
Overall: no blocking issues. Approving the same as the prior two reviews — the documentation is accurate, well-structured, and matches the existing patterns in the repo. |
Documents API token authentication as an alternative to OAuth for the Trunk MCP server (shipped in v148, trunk-io/trunk2#3381).
Changes
What's documented
Co-Authored-By: Claude Opus 4.6 noreply@anthropic.com