Skip to content

fix(cloud/local): propagate destination listing errors - #47

Open
Seagulls wants to merge 1 commit into
Liquid-co:dev-publicfrom
Seagulls:research/opensave-46-local-listing-20261011
Open

Seagulls wants to merge 1 commit into
Liquid-co:dev-publicfrom
Seagulls:research/opensave-46-local-listing-20261011

Conversation

@Seagulls

Copy link
Copy Markdown

Refs #46 — partial fix only.

An unavailable or unreadable local-folder destination previously returned
an empty successful listing. This patch propagates the underlying os.ReadDir
error, allowing callers to distinguish a storage failure from an empty backup.

Tests cover absent paths, non-directory destinations and ordinary folders.

The mount-loss fallback-write problem is NOT fixed. Uploads can still create
a missing local directory, and detecting a vanished NAS mount requires a
separate opt-in mount-identity contract.

Based on dev-public at 7b69042.
Focused cloud tests, race detector and go vet pass.
The backend E2E suite passed during combined-branch validation.
The desktop app was not built because frontend/dist is absent.

@vercel

vercel Bot commented Oct 11, 2026

Copy link
Copy Markdown

Someone is attempting to deploy a commit to the sivadaboi's projects Team on Vercel.

A member of the Team first needs to authorize it.

This branch has not been deployed

No deployments
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