Skip to content

Add MCP server reference pages for vault tools - #688

Open
dprevoznik wants to merge 8 commits into
mainfrom
hypeship/mcp-vault-tools-docs
Open

dprevoznik wants to merge 8 commits into
mainfrom
hypeship/mcp-vault-tools-docs

Conversation

@dprevoznik

@dprevoznik dprevoznik commented Oct 8, 2026 •

Copy link
Copy Markdown
Contributor

Summary

The MCP server exposes six vault tools (listed in the kernel-mcp-server README), but the MCP server reference had no pages for any of them. This adds a Vaults group under MCP Server → Tools:

  • manage_vaults: create, list, get, delete vaults; also indexes the other vault tools
  • manage_vault_credentials: KERNEL-hosted and 1Password credential paths, create/update specs, connect_account
  • manage_vault_items: list/get/invoke/events/delete, fill, webmcp_invoke inputs, 1Password operations
  • manage_vault_wallets: Link by Stripe and AgentCard wallet specs, payment_methods
  • manage_vault_cards: Link by Stripe and AgentCard card request specs
  • manage_vault_provider_configs: organization-owned client configurations

Also:

  • adds a Vaults card to the MCP Server overview's Explore grid (same pattern as browser_repl and webmcp)
  • adds a "Fill a login from a vault" example to the MCP Server examples page
  • adds the create-only vaults parameter to the manage_browsers page

Parameters and behavior come from the tool schemas in kernel-mcp-server/src/lib/mcp/tools/vault-*.ts. Each page carries the preview badge and notes that the tools only show up when vaults are enabled for the org.

Consistency with recent vault changes:

Testing

  • mint validate: build validation passed
  • mint broken-links: no broken links

🤖 Generated with Claude Code


Note

Low Risk
Documentation-only changes with no runtime or API behavior modifications.

Overview
Adds MCP Server documentation for the six preview vault tools that were missing from the reference, grouped under Tools → Vaults in docs.json.

New pages cover manage_vaults, manage_vault_credentials, manage_vault_items, manage_vault_wallets, manage_vault_cards, and manage_vault_provider_configs (actions, parameters, examples, and safety notes such as fill retry behavior and payment reconciliation). The MCP Server overview gets a Vaults card, examples adds a “fill a login from a vault” walkthrough, and manage_browsers documents the create-only vaults attachment parameter.

Reviewed by Cursor Bugbot for commit f920557. Bugbot is set up for automated code reviews on this repo. Configure here.

@mintlify

mintlify Bot commented Oct 8, 2026 •

Copy link
Copy Markdown
Contributor

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

Project Status Preview Updated
Kernel 🟢 Ready View Preview Oct 9, 2026, 3:45 PM

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

@cursor cursor Bot left a comment •

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Stale Bugbot comment from a previous run.

Comment thread reference/mcp-server/tools/manage-vault-cards.mdx Outdated
dprevoznik and others added 5 commits October 8, 2026 17:08
Document the six vault tools exposed by the MCP server (manage_vaults,
manage_vault_credentials, manage_vault_items, manage_vault_wallets,
manage_vault_cards, manage_vault_provider_configs) under a Vaults group
in the MCP tools nav, and add the create-only vaults parameter to
manage_browsers.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>

@cursor cursor Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Cursor Bugbot has reviewed your changes using default effort and found 2 potential issues.

Fix All in Cursor

❌ Bugbot Autofix is OFF. To automatically fix reported issues with cloud agents, enable autofix in the Cursor dashboard.

Reviewed by Cursor Bugbot for commit 118439d. Configure here.

Comment thread reference/mcp-server/tools/manage-vault-cards.mdx Outdated
Comment thread reference/mcp-server/tools/manage-vault-cards.mdx Outdated
…P docs

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
@dprevoznik

Copy link
Copy Markdown
Contributor Author

wasn't able to finish this if someone else could take it over the finish line. thanks!

@masnwilliams masnwilliams 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.

These pages fill a real gap and are written clearly. Three things to fix before merge (inline):

  1. Stale 1Password guidance: manage-vault-items.mdx:129
  2. Link card amount limit: manage-vault-cards.mdx:40
  3. vaults row on manage-browsers: missing two contract facts

Cheap, but optional:

  • Use one "required" convention in the parameter tables. cards.provider says "Required.", but wallets.provider/spec only say "(create)" even though the tool rejects a create without them, and manage_vault_provider_configs doesn't mark what create needs or that config is required for get/update/delete. Existing pages use "Required for get and delete."
  • The PR description says every page has the "only visible when vaults are enabled" note, but only manage-vaults does. Either add it to the other pages (a shared snippet would work) or update the description.

Follow-up, not blocking: several sections repeat rules that already live in vaults/fill.mdx, vaults/1password.mdx, and integrations/wallets/*, such as the fill retry paragraph, the payment reconciliation warning, and the AgentCard checkout flow. Item 1 shows how those copies go stale. Over time, trimming these pages to MCP-specific behavior (invoke rejecting unadvertised operations, wait only on get/events, error mapping, org-scoped writes) and linking out for the rest would keep them in sync.


Give the approval link, unmodified, only to the account owner outside the agent-controlled browser. Never open or approve it yourself.

`fill_submitted` means the extension submitted the form, not that the login succeeded. Never retry `fill_unknown` in the same browser. If a request is uncertain, the item has no advertised operations; don't delete or recreate it to retry.

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.

"Never retry fill_unknown in the same browser" is the old wording. The MCP server and vaults/1password.mdx (#692) now say fill_unknown means the form may have been filled or submitted, so inspect the page to see the result of the fill. Can we update this line, or replace it with a link to the 1Password guide? The PR description's note that the 1pw_fill no-retry guidance is unchanged needs updating too.

|-------|-------------|
| `wallet` | Key of the Link wallet in the same vault. |
| `payment_method_id` | Payment method the user selected from the wallet's `payment_methods`. |
| `amount` | Spending limit, from 1 to 50000. |

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.

50000 matches the API and stripe-link.mdx, but the MCP tool schema (linkCardSpecSchema) allows up to 500000, so this page will disagree with one of them either way. Suggest dropping the number and linking to Create and authorize a card item. The schema mismatch should be fixed separately in kernel-mcp-server.

| `save_profile_changes` | (create) Save session changes back to the profile on close. |
| `proxy_id` | (create) Proxy ID for traffic routing. |
| `extension_id` / `extension_name` | (create) Extension to load. |
| `vaults` | (create) Up to 20 [vaults](/reference/mcp-server/tools/manage-vaults) to attach, each as `{ "id": ... }` or `{ "name": ... }`. Attachments can't be added or changed after creation. |

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.

Two facts from the tool schema are missing here: attaching a vault gives the browser access to every item in it, including items added later, and vaults can't be attached to pooled browsers. The first one matters for security.

Suggested change
| `vaults` | (create) Up to 20 [vaults](/reference/mcp-server/tools/manage-vaults) to attach, each as `{ "id": ... }` or `{ "name": ... }`. Attachments can't be added or changed after creation. |
| `vaults` | (create) Up to 20 [vaults](/reference/mcp-server/tools/manage-vaults) to attach, each as `{ "id": ... }` or `{ "name": ... }`. The browser can use every item in an attached vault, including items added later. Attachments can't be added or changed after creation, and aren't available for pooled browsers. |

| Field | Description |
|-------|-------------|
| `account` | Key of a connected `credential_account` item in the same vault. Each end user's vault connects its own account. |
| `logins` | 1–5 logins the owner approves together, each with an `https://` `website` and optional `reason` (up to 100 characters) and `keywords` (up to 5). |

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.

nit: MCP turns logins into the API's requests: { version: 2, entries: [...] }, so this shape doesn't match what readers see in vaults/1password.mdx. One sentence saying how they map would help.

@masnwilliams masnwilliams 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.

Approving. Please address the three inline items from my previous review before merging: the stale fill_unknown guidance, the Link card amount limit, and the vaults row on manage-browsers. The rest is optional.

This branch was successfully deployed

1 active deployment
staging — f920557b Deployed Oct 9, 2026 by mintlify[bot]
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