Skip to content

Document optional payment provider clients - #564

Merged
AnnaXWang merged 4 commits into
mainfrom
hypeship/docs-provider-clients
Sep 13, 2026
Merged

Document optional payment provider clients#564
AnnaXWang merged 4 commits into
mainfrom
hypeship/docs-provider-clients

Conversation

@rgarcia

@rgarcia rgarcia commented Sep 11, 2026

Copy link
Copy Markdown
Contributor

summary

  • surface the managed-vs-customer-managed oauth choice before each provider lifecycle, then rejoin the shared payment flow at explicit anchors.
  • add typescript, python, and protected-file cli examples for provider registration and wallet creation, including exact credential and grant shapes.
  • distinguish organization-scoped provider configuration from project-scoped wallet operations and link to api-key setup.
  • keep link oauth itself application-owned: import the existing grant and hand refresh-token rotation to KERNEL after a successful wallet import.
  • clarify agentcard mode behavior and keep recovery guidance with payment item states.

validation

  • mintlify validate passed.
  • mintlify broken-links passed.
  • local previews returned 200 for the payments overview, both provider guides, and vaults.
  • shell syntax checked for bash examples in the changed payment guides.
  • typescript and python examples checked against the generated sdk resource types.

release dependency

publish after the provider-configuration api, cli commands, and generated sdk resources are available. the examples use vault-provider-configs, --provider-config-name, --tokens-file, and the generated vaultProviderConfigs / vault_provider_configs resources.

@mintlify

mintlify Bot commented Sep 11, 2026

Copy link
Copy Markdown
Contributor

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

Project Status Preview Updated
Kernel 🟢 Ready View Preview Sep 13, 2026, 5:20 PM

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

cursor[bot]
cursor Bot approved these changes Sep 11, 2026

@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 76efa4d. Configure here.

Comment thread vaults.mdx Outdated
Comment thread integrations/payments/stripe-link.mdx Outdated

@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 comment

Risk assessment: Very Low

Decision: Keep existing approval. Risk did not increase after the latest push. Not re-approving.

Re-evaluation (synchronize after prior approval)

This automation previously approved 76efa4d. The follow-up commit f488773 only reformats existing caveats in integrations/payments/stripe-link.mdx and vaults.mdx from dense prose into labeled bullets. Same facts, no new pages, no new commands, no config or code.

Scope (from the current diff vs main)

  • 5 files, all existing Mintlify .mdx pages (+153 / -23)
  • No docs.json, redirects, navigation, CI, workflows, or other config changes
  • No application code, infrastructure, schema, or auth implementation changes
  • No CODEOWNERS file in this repo, so codeowner review is not required

Why Very Low

Documentation-only: optional payment-provider-client guidance on existing pages (accordions, CLI examples, lifecycle/status notes). Blast radius is published docs copy. No production logic, shared service, or infra surface.

Residual notes (not risk-elevating)

  • Examples document CLI/API flags that may not be universally available yet (vault-provider-configs, --provider-config-name, --tokens-file). That is a docs-release timing concern, not runtime risk in this repo.
  • Credential-handling guidance stays documentation; it does not change how the platform stores or enforces secrets.

Approval stands. No revoke.

Open in Web View Automation 

Sent by Cursor Automation: Assign PR reviewers

@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 comment

Risk assessment: Very Low

Decision: Keep existing approval. Risk did not increase after the latest push. Not re-approving.

Re-evaluation (synchronize after prior approval)

This automation previously approved 76efa4d. The latest commit 6182ec0 is still documentation-only. Compared with the previously assessed tip (f488773), it:

  • Moves optional bring-your-own OAuth client guidance from Before you start accordions into appended sections on the existing Link and Agentcard pages
  • Reverts integrations/payments/overview.mdx so that file is unchanged vs main
  • Removes cutover/recovery guidance that had been added to the default onboarding path
  • Updates heading anchors and a few mode/recovery notes

No new pages, no config/code, and the changed surface vs main is smaller than the last assessment.

Scope (from the current diff vs main)

  • 4 files, all existing Mintlify .mdx pages (+98 / -6)
    • browsers/enable-payments-in-browser-agent.mdx
    • integrations/payments/agentcard.mdx
    • integrations/payments/stripe-link.mdx
    • vaults.mdx
  • No docs.json, redirects, navigation, CI, workflows, or other config changes
  • No application code, infrastructure, schema, or auth implementation changes
  • No CODEOWNERS file in this repo, so codeowner review is not required

Why Very Low

Documentation-only edits to existing product docs: appended CLI examples, provider-config semantics, and cross-links. Blast radius is published docs copy. No production logic, shared service, or infra surface.

Residual notes (not risk-elevating)

  • Examples document CLI/API flags that may not be universally available yet (vault-provider-configs, --provider-config-name, --tokens-file). That is a docs-release timing concern, not runtime risk in this repo.
  • Credential-handling guidance stays documentation; it does not change how the platform stores or enforces secrets.

Approval stands. No revoke.

Open in Web View Automation 

Sent by Cursor Automation: Assign PR reviewers

@rgarcia
rgarcia requested a review from AnnaXWang September 11, 2026 15:13

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

Risk assessment: Very Low

Decision: Keep existing approval. Risk did not increase after the latest push. Not re-approving.

Re-evaluation (synchronize after prior approval)

This automation previously approved 76efa4d. The latest commit 296db9d is still documentation-only. Compared with the previously assessed tip (6182ec0), it:

  • Restores a short managed-vs-customer-managed pointer on integrations/payments/overview.mdx
  • Moves optional bring-your-own OAuth setup into early Choose an oauth client sections on the existing Link and Agentcard pages
  • Adds TypeScript, Python, and CLI examples for provider-config registration and wallet creation
  • Expands vault provider-configuration semantics (recovery_required, org vs project scope, token/spec shapes)

No new pages, no config/code, and the risk category is unchanged.

Scope (from the current diff vs main)

  • 5 files, all existing Mintlify .mdx pages (+343 / -16)
    • browsers/enable-payments-in-browser-agent.mdx
    • integrations/payments/agentcard.mdx
    • integrations/payments/overview.mdx
    • integrations/payments/stripe-link.mdx
    • vaults.mdx
  • No docs.json, redirects, navigation, CI, workflows, or other config changes
  • No application code, infrastructure, schema, or auth implementation changes
  • No CODEOWNERS file in this repo, so codeowner review is not required

Why Very Low

Documentation-only edits to existing product docs: optional client-setup copy, SDK/CLI examples, and cross-links. Blast radius is published docs copy. No production logic, shared service, or infra surface.

Residual notes (not risk-elevating)

  • Examples document CLI/API surfaces that may not be universally available yet (vault-provider-configs, --provider-config-name, --tokens-file). That is a docs-release timing concern, not runtime risk in this repo.
  • Credential-handling guidance stays documentation; it does not change how the platform stores or enforces secrets.

Approval stands. No revoke.

Open in Web View Automation 

Sent by Cursor Automation: Assign PR reviewers

@AnnaXWang
AnnaXWang merged commit 8476ce3 into main Sep 13, 2026
4 checks passed
@AnnaXWang
AnnaXWang deleted the hypeship/docs-provider-clients branch September 13, 2026 17:27
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