Skip to content

docs(cli): document MCP OAuth resource override#1327

Open
factory-ain3sh wants to merge 2 commits into
mainfrom
ainesh/cli-1432-mcp-docs
Open

docs(cli): document MCP OAuth resource override#1327
factory-ain3sh wants to merge 2 commits into
mainfrom
ainesh/cli-1432-mcp-docs

Conversation

@factory-ain3sh

@factory-ain3sh factory-ain3sh commented Jul 24, 2026

Copy link
Copy Markdown
Contributor

Description

What

The public MCP configuration reference omitted the supported oauth.resource override, so users could not discover how to target an OAuth audience that differs from the MCP endpoint or omit the resource indicator.

Why

CLI-1432 tracks the documentation gap introduced by the MCP OAuth resource override.

How

This PR adds the string | false setting to the OAuth Overrides table and documents the default replacement and explicit omission behavior.

Related Issue

Closes CLI-1432

Reviewer Guide

Diff shape: 100% documentation; one row in docs/cli/configuration/mcp.mdx.
Review depth: Skim — the change documents an existing configuration field without changing runtime behavior.
Read order:

  1. docs/cli/configuration/mcp.mdx — confirm the field type and resource-selection semantics match the CLI contract.

Risk & Impact

Low risk. The change is documentation-only and does not alter configuration parsing, OAuth requests, or runtime behavior.

Verification

Behavior verified. The table now describes oauth.resource as a custom resource indicator or explicit omission, matching the source feature's three-state behavior. Regression coverage. Not applicable to a documentation-only change. Not tested. No runtime test was added because the implementation is unchanged. Standard validators. git diff --check passed; no formatter, linter, typecheck, or test configuration is present for this docs-only repository.

List the new oauth.resource setting so users can configure a custom OAuth audience or omit the RFC 8707 resource indicator for compatible MCP servers.
@factory-droid

factory-droid Bot commented Jul 24, 2026

Copy link
Copy Markdown
Contributor

Droid finished @factory-ain3sh's task —— View job


Security Review

Droid review complete.

Docs change looks correct and low risk. Please keep the Japanese MCP OAuth reference in sync by adding the new resource row. Also, the provided PR description appears to describe a different setting and should be updated to match this PR’s actual change.

@mintlify

mintlify Bot commented Jul 24, 2026

Copy link
Copy Markdown
Contributor

Preview deployment for your docs. Learn more about Mintlify Previews.

Project Status Preview Updated (UTC)
factory 🟢 Ready View Preview Jul 24, 2026, 2:01 AM

💡 Tip: Enable Workflows to automatically generate PRs for you.

Comment thread docs/cli/configuration/mcp.mdx
@factory-ain3sh factory-ain3sh self-assigned this Jul 24, 2026
@factory-ain3sh factory-ain3sh added the documentation Improvements or additions to documentation label Jul 24, 2026
Document oauth.resource in the Japanese OAuth Overrides table so localized MCP configuration references expose the same supported field and omission behavior.
@mintlify

mintlify Bot commented Jul 24, 2026

Copy link
Copy Markdown
Contributor

Preview deployment for your docs. Learn more about Mintlify Previews.

Project Status Preview Updated (UTC)
factory 🟡 Building Jul 24, 2026, 2:00 AM

💡 Tip: Enable Workflows to automatically generate PRs for you.

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

Labels

documentation Improvements or additions to documentation

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant