Skip to content

feat(classifier): recognise Bitwarden YubiKey touches - #11

Draft
Talgarr wants to merge 3 commits into
mainfrom
add-rule-bitwarden
Draft

feat(classifier): recognise Bitwarden YubiKey touches#11
Talgarr wants to merge 3 commits into
mainfrom
add-rule-bitwarden

Conversation

@Talgarr

@Talgarr Talgarr commented Jun 23, 2026

Copy link
Copy Markdown
Owner

Bitwarden supports a YubiKey OTP two-factor method. Both the desktop app (the bitwarden Electron binary) and the bw CLI prompt for the YubiKey OTP, and the key emits the one-time code on touch (HID OTP). Match either name and report an authenticate against the Bitwarden vault. WebAuthn/passkey unlock goes through the browser and is handled elsewhere.

@Talgarr
Talgarr force-pushed the add-rule-bitwarden branch from ddc5e4c to e85cdd8 Compare June 25, 2026 16:29
Talgarr and others added 3 commits June 25, 2026 12:43
Bitwarden supports a YubiKey OTP two-factor method. Both the desktop
app (the `bitwarden` Electron binary) and the `bw` CLI prompt for the
YubiKey OTP, and the key emits the one-time code on touch (HID OTP).
Match either name and report an authenticate against the Bitwarden
vault. WebAuthn/passkey unlock goes through the browser and is handled
elsewhere.

Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
Add a manual e2e check (like the browser/1password tests): prompt for a
Bitwarden unlock/login that uses the YubiKey OTP 2FA method and assert the
classifier named `bitwarden`. Register it in the driver and document it.

Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
The bitwarden e2e test references `bw`; add `bitwarden-cli` so it lands
on PATH in `nix develop`.

Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
@Talgarr
Talgarr force-pushed the add-rule-bitwarden branch from e85cdd8 to 29eacf7 Compare June 25, 2026 16:43
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.

1 participant