Skip to main content
← Back to list
01Issue
BugShippedSwamp CLIPublic
Assigneeszack

Relationships

#3561 VaultService capability errors label the vault instance name as its type

Opened by zack · 10/16/2026· Shipped 10/16/2026

The require* helpers in src/domain/vaults/vault_service.ts (requireDeleteProvider, requireAnnotationProvider, requireRefreshHookProvider, and requireExistenceProvider added for swamp-club#3557) throw messages of the form: Vault 'name' (type: X) does not support ... where X is provider.getName(). For local_encryption and mock, getName() returns the vault instance name, not the vault type, so the message reads Vault 'erp' (type: erp) and misleads the reader. The actual type is in the service's vaultTypes map. Found by the code review on swamp-club#3557; pre-existing in three siblings and copied by the fourth for consistency, so filed separately. Fix: read the type from the vaultTypes map (or drop the parenthetical) in all four helpers, with a test on the message.

02Bog Flow
✓OPEN✓TRIAGED✓IN PROGRESS✓SHIPPED+ 1 MOREASSIGNED+ 2 MOREREVIEW+ 7 MOREPR_MERGED+ 2 MORESESSION_SUMMARIZED

Shipped

10/16/2026, 8:07:28 PM

Click a lifecycle step above to view its details.

03Sludge Pulse
zack assigned zack10/16/2026, 7:39:16 PM

Sign in to post a ripple.