Document 1Password SSO/identity flow limitation#439
Conversation
Add a callout clarifying that Managed Auth fills direct logins (username/password + TOTP) from 1Password, and that SSO/identity-provider flows are unsupported because the 1Password API only exposes stored credential values. Co-Authored-By: Claude Opus 4.7 <noreply@anthropic.com>
![cursor[bot]](https://avatars.githubusercontent.com/in/1210556?s=60&v=4){kind=small}
There was a problem hiding this comment.
Risk assessment: Very Low.
I inspected the actual diff and found a single documentation-only addition in integrations/1password.mdx clarifying supported 1Password login types. There are no codepath, infrastructure, auth, permission, or production behavior changes, and I didn't find a CODEOWNERS file requiring owner review. No prior approval was present, so I'm approving this PR.
Sent by Cursor Automation: Assign PR reviewers
Direct logins to identity providers (e.g. signing into Google/Gmail with a stored username/password/TOTP) work. The unsupported case is 1Password's "sign in with" linked-item reference, which is not exposed via the API. Co-Authored-By: Claude Opus 4.7 <noreply@anthropic.com>
There was a problem hiding this comment.
Risk assessment: Very Low.
I inspected the actual diff for the current head and found a single documentation-only addition in integrations/1password.mdx clarifying supported 1Password login types. The change doesn't modify codepaths, prompts used for model behavior, infrastructure, auth/permissions logic, or production runtime behavior. I also found no CODEOWNERS file in the repository and no pending review requests indicating code-owner review.
This PR was already approved, so I'm leaving this assessment as a comment-only review rather than re-approving.
Sent by Cursor Automation: Assign PR reviewers
|
Preview deployment for your docs. Learn more about Mintlify Previews.
💡 Tip: Enable Workflows to automatically generate PRs for you. |
1 participant
Summary
Why
The docs described TOTP and direct credential support but never stated that SSO/identity flows are unsupported, which came up as a gap when explaining the integration's scope.
Preview
https://tbd-6fc993ce-hypeship-1password-no-sso-note.mintlify.app/integrations/1passwordTest plan
Note
Low Risk
Documentation-only change with no runtime or security behavior impact.
Overview
Adds a Supported Login Types section after TOTP in
integrations/1password.mdxso Managed Auth scope is explicit: direct 1Password logins (username/password and optional TOTP), including signing into an IdP like Google with stored credentials.A Warning callout states that 1Password linked-item “sign in with” references (e.g. PostHog → sign in with Google) are not available via the 1Password API, so users should store a direct login for the target site instead.
Reviewed by Cursor Bugbot for commit 222a52c. Bugbot is set up for automated code reviews on this repo. Configure here.