clawbot/secret
1
0
Fork 0
forked from sneak/secret
Code Pull Requests Activity

Change missing metadata log from Debug to Warn for visibility without --verbose

Browse Source 
Per review feedback: missing unlocker metadata should produce a warning
visible in normal output, not hidden behind debug flags.
This commit is contained in:
user 2026-02-19 23:57:39 -08:00
parent 1a96360f6a
commit c0f221b1ca
2 changed files with 11 additions and 1 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

10
internal/secret/debug.go
View File

@ -58,6 +58,16 @@ func IsDebugEnabled() bool {
return debugEnabled return debugEnabled
} }
// Warn logs a warning message to stderr unconditionally (visible without --verbose or debug flags)
func Warn(msg string, args ...any) {
output := fmt.Sprintf("WARNING: %s", msg)
for i := 0; i+1 < len(args); i += 2 {
output += fmt.Sprintf(" %s=%v", args[i], args[i+1])
}
output += "\n"
fmt.Fprint(os.Stderr, output)
}
// Debug logs a debug message with optional attributes // Debug logs a debug message with optional attributes
func Debug(msg string, args ...any) { func Debug(msg string, args ...any) {
if !debugEnabled { if !debugEnabled {

2
internal/vault/unlockers.go
View File

@ -213,7 +213,7 @@ func (v *Vault) ListUnlockers() ([]UnlockerMetadata, error) {
return nil, fmt.Errorf("failed to check if metadata exists for unlocker %s: %w", file.Name(), err) return nil, fmt.Errorf("failed to check if metadata exists for unlocker %s: %w", file.Name(), err)
} }
if !exists { if !exists {
secret.Debug("Skipping unlocker directory with missing metadata file", "directory", file.Name()) secret.Warn("Skipping unlocker directory with missing metadata file", "directory", file.Name())
continue continue
} }