Restore silently continues when individual file restoration fails #21

New Issue

clawbot · 2026-02-08T17:16:56+01:00

clawbot commented

2026-02-08 17:16:56 +01:00

In restore.go Restore(), file restore errors are logged but skipped:

if err := v.restoreFile(...); err != nil {
    log.Error("Failed to restore file", "path", file.Path, "error", err)
    continue
}

The final result does not report how many files failed, and the function returns nil (success) even if files failed. Should track failed files and report them, similar to how verify tracks FilesFailed.

Ref: parent issue #1

In `restore.go` `Restore()`, file restore errors are logged but skipped: ```go if err := v.restoreFile(...); err != nil { log.Error("Failed to restore file", "path", file.Path, "error", err) continue } ``` The final result does not report how many files failed, and the function returns nil (success) even if files failed. Should track failed files and report them, similar to how verify tracks `FilesFailed`. Ref: parent issue #1

clawbot referenced this issue

2026-02-08 17:17:10 +01:00

find bugs or incomplete functionality for 1.0 #1

clawbot was assigned by sneak

2026-02-08 17:31:34 +01:00

clawbot referenced this issue from a commit

2026-02-08 17:34:20 +01:00

fix: track and report file restore failures

clawbot referenced a pull request that will close this issue

2026-02-08 17:34:24 +01:00

fix: track and report file restore failures #22

Sign in to join this conversation.