Non-darwin KeychainUnlocker stub returns errors instead of panicking (closes #7) #12

clawbot · 2026-02-08T21:06:05+01:00

clawbot commented

2026-02-08 21:06:05 +01:00

First-time contributor

The non-darwin stub now returns proper errors instead of panicking, enabling cross-platform vault portability when vaults synced from macOS contain keychain unlockers.

sneak was assigned by clawbot

2026-02-08 21:06:05 +01:00

clawbot added 1 commit 2026-02-08 21:06:05 +01:00

fix: non-darwin KeychainUnlocker stub returns errors instead of panicking 4419ef7730

The stub previously panicked on all methods including NewKeychainUnlocker,
which is called from vault code when processing keychain-type unlocker
metadata. This caused crashes on Linux/Windows when a vault synced from
macOS contained keychain unlockers.

Now returns proper error values, allowing graceful degradation and
cross-platform vault portability.

This pull request can be merged automatically.

You are not authorized to merge this pull request.

View command line instructions.

Checkout

From your project repository, check out a new branch and test the changes.

git fetch -u fix/issue-7:clawbot-fix/issue-7

git checkout clawbot-fix/issue-7

Sign in to join this conversation.

No reviewers

No Label

No Milestone

No project

No Assignees

1 Participants

Notifications

Due Date

The due date is invalid or out of range. Please use the format 'yyyy-mm-dd'.

No due date set.

Dependencies

No dependencies set.

Reference: sneak/secret#12