pass through USER/HOME to op cli

[theoephraim]

passes through USER and HOME env vars to op cli calls within 1password plugin. This is to fix op plugin behaviour on machines where op is installed via brew in a multi-user setup.

[changeset-bot]

🦋 Changeset detected

Latest commit: f66ba09

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 1 package

Name	Type
@varlock/1password-plugin	Patch

Not sure what this means? Click here to learn what changesets are.

Click here if you're a maintainer who wants to add another changeset to this PR

[pkg-pr-new]

Open in StackBlitz

npm i https://pkg.pr.new/@varlock/ci-env-info@564

npm i https://pkg.pr.new/@varlock/1password-plugin@564

commit: f66ba09

[theoephraim]

user reports they still need OP_BIOMETRIC_UNLOCK_ENABLED although I'm unclear why it's different on their system. We can probably just always pass through OP_BIOMETRIC_UNLOCK_ENABLED=true since this code path is only called when using the cli specifically for biometric auth.

[philmillman]

user reports they still need OP_BIOMETRIC_UNLOCK_ENABLED although I'm unclear why it's different on their system. We can probably just always pass through OP_BIOMETRIC_UNLOCK_ENABLED=true since this code path is only called when using the cli specifically for biometric auth.

is it always going to be biometric though? won't it just default to whatever the system level auth is?

[theoephraim]

the docs claim the setting

Toggles the 1Password app integration on or off.

so... 🤷

[philmillman]

the docs claim the setting

Toggles the 1Password app integration on or off.

so... 🤷

gotcha, naming things is hard