Skip to content

Document MCP client editing and deletion - #7940

Open
leemthompo wants to merge 5 commits into
mainfrom
docs/1617-mcp-client-editing
Open

Document MCP client editing and deletion#7940
leemthompo wants to merge 5 commits into
mainfrom
docs/1617-mcp-client-editing

Conversation

@leemthompo

@leemthompo leemthompo commented Aug 13, 2026

Copy link
Copy Markdown
Member

⚠️ Don't merge until elastic/kibana#284257 — MCP client and connection deletion UI is in and deployed on serverless

Summary

Closes elastic/docs-content-internal#1617

Source

Generative AI disclosure

  1. Did you use a generative AI (GenAI) tool to assist in creating this contribution?
  • Yes
  • No

Tool and model used: OpenAI Codex (GPT-5) for drafting, source verification, and consistency checks.

@github-actions

Copy link
Copy Markdown
Contributor

Elastic Docs AI PR menu

Check the box to run an AI review for this pull request.

  • Review docs changes (docs-review). Status: not started.

Powered by GitHub Agentic Workflows and docs-actions. For more information, reach out to the docs team.

@github-actions

Copy link
Copy Markdown
Contributor

✅ Elastic Docs Style Checker (Vale)

No issues found on modified lines!


The Vale linter checks documentation changes against the Elastic Docs style guide. To use Vale locally or report issues, refer to Elastic style guide for Vale.

@github-actions

github-actions Bot commented Aug 13, 2026

Copy link
Copy Markdown
Contributor

AI-Assisted: Codex drafted and validated the OAuth client management updates.
@leemthompo
leemthompo marked this pull request as ready for review August 13, 2026 12:54
@leemthompo
leemthompo requested a review from a team as a code owner August 13, 2026 12:54

@shainaraskas shainaraskas left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

🏁 with nits

Comment on lines +115 to +118
6. Edit any of the following fields:
- **Name**: Enter a name with no more than 128 characters.
- **Client logo (optional)**: Select a provided logo, upload a PNG, JPEG, or GIF of up to 192 KB, or remove the existing logo.
- **Redirect URI type**: Select **Local** or **Remote**, and then enter at least one redirect URI. Local clients support up to 20 redirect URIs. Remote clients support one HTTPS redirect URI. Each URI can contain up to 2,048 characters.

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

duplicating the requirements of these fields here feels like a recipe for breakage. consider trimming?

also the flow for uri editing is more than "enter"

Suggested change
6. Edit any of the following fields:
- **Name**: Enter a name with no more than 128 characters.
- **Client logo (optional)**: Select a provided logo, upload a PNG, JPEG, or GIF of up to 192 KB, or remove the existing logo.
- **Redirect URI type**: Select **Local** or **Remote**, and then enter at least one redirect URI. Local clients support up to 20 redirect URIs. Remote clients support one HTTPS redirect URI. Each URI can contain up to 2,048 characters.
6. Edit any of the following fields:
- **Name**: Enter a name with no more than 128 characters.
- **Client logo (optional)**: Select a provided logo, upload a PNG, JPEG, or GIF of up to 192 KB, or remove the existing logo.
- **Redirect URI type**: Select **Local** or **Remote**, and then add, remove, or edit an existing redirect URI. Local clients support up to 20 redirect URIs. Remote clients support one HTTPS redirect URI. Each URI can contain up to 2,048 characters.

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

Moved the shared constraints to the create procedure as the canonical source of truth and linked to it from the edit task. The URI guidance now covers adding, removing, and editing. 0fbde8833

Comment thread deploy-manage/app-connections/manage-app-connections.md
Comment thread deploy-manage/app-connections/manage-app-connections.md
Comment thread deploy-manage/app-connections/revoke-oauth-client.md Outdated
Comment thread deploy-manage/app-connections/revoke-oauth-client.md
@leemthompo

Copy link
Copy Markdown
Member Author

@shainaraskas assigning this to you because we can't merge it until next week 🙏
image


::::{step} Name the client
Enter a **Client name**. The name is visible to users during the authorization flow, so use something that clearly identifies the application (for example, `Claude Desktop — Engineering`).
Enter a **Client name** with no more than 128 characters. The name is visible to users during the authorization flow, so use something that clearly identifies the application (for example, `Claude Desktop — Engineering`).

@shainaraskas shainaraskas Aug 13, 2026

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

@leemthompo how strongly do you feel about these limits being included in the docs? imo this is unnecessary detail because the ui enforces the limits

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

overkill 💯

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.

2 participants