Skip to content

scan: a folder holding only Steam's own files is not a save - #115

Merged
Omar-VF merged 1 commit into
Liquid-co:dev-publicfrom
ChakraFusion:pr/steam-only-folders
Oct 23, 2026
Merged

Omar-VF merged 1 commit into
Liquid-co:dev-publicfrom
ChakraFusion:pr/steam-only-folders

Conversation

@ChakraFusion

Copy link
Copy Markdown

Steam keeps remotecache.vdf in userdata/<user>/<appid> even for games that keep no save there. When measuring a found save folder, the scan counted it, so on my machines twenty-odd such folders were pointed out as new games with saves, and tracked with nothing to sync.

The change: measuring leaves out the files no save is made of (delta.NeverSynced, which sync already skips). A folder holding only Steam's file measures as empty, so the background scan doesn't point it out, and opensave scan lists it as empty.

Test: TestMeasure_OnlyLauncherFilesIsEmpty, which fails without the change ("measured 1 file(s)").

Checks: the full unit and e2e suites pass locally on today's dev-public.

🤖 Generated with Claude Code

Steam keeps remotecache.vdf in userdata/<user>/<appid> for games that
have no save there at all. Measuring a found save counted it, so twenty-odd
such folders were pointed out as new games with saves, and tracked. The
file is already never synced (delta.NeverSynced); measuring leaves it out
too, so such a folder is empty: not pointed out, and listed as empty.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
@vercel

vercel Bot commented Oct 21, 2026

Copy link
Copy Markdown

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

A member of the Team first needs to authorize it.

@Omar-VF
Omar-VF merged commit b402db9 into Liquid-co:dev-public Oct 23, 2026
10 of 11 checks passed
Omar-VF added a commit that referenced this pull request Oct 23, 2026
Seagulls pushed a commit to Seagulls/OpenSave that referenced this pull request Oct 25, 2026
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.

2 participants