Should really 1Password integration ask for credentials?

[wyan]

I have the 1Password CLI already installed in my system, and it is already signed in.
Also, I feel that being asked for my master password anywhere outside 1Password feels really not OK.
Would it be possible for Min to use the already installed CLI tool and/or let the user sign in manually?

[PalmerAL]

That's a fair point. I think I remember there being some complexity with external sessions not authenticating correctly, but it's been such a long time since I last worked on this that I don't remember too well. We use a separate named account in the CLI (--account=min-autofill), so you'd need to sign in to that externally, but if you did that, it seems like this could theoretically work. We would probably need to at least change isUnlocked to directly check the login status rather than assuming we're not logged in if we don't have a saved session key.