Skip to content

fix(onion/saves): 'Beetle Supafaust' to 'Supafaust' - #256

Open
pawndev wants to merge 1 commit into
rommapp:mainfrom
pawndev:patch-3
Open

fix(onion/saves): 'Beetle Supafaust' to 'Supafaust'#256
pawndev wants to merge 1 commit into
rommapp:mainfrom
pawndev:patch-3

Conversation

@pawndev

@pawndev pawndev commented Jul 22, 2026

Copy link
Copy Markdown
Collaborator

@greptile-apps

greptile-apps Bot commented Jul 22, 2026

Copy link
Copy Markdown

Greptile Summary

This PR corrects Onion's Supafaust save-directory name for SNES.

  • Renames Beetle Supafaust to Supafaust in the SNES mapping.

Confidence Score: 4/5

Existing saves under the former directory name can disappear from sync results.

  • The scanner checks only directory names listed in this mapping.
  • Missing directories are skipped silently.
  • Keeping the old name as a secondary alias preserves existing saves while using the corrected name for new paths.

cfw/onion/data/save_directories.json

Important Files Changed

Filename Overview
cfw/onion/data/save_directories.json Changes the primary SNES Supafaust directory but removes compatibility with saves stored under the former name.

Fix All in Claude Code

Prompt To Fix All With AI
Fix the following 1 code review issue. Work through them one at a time, proposing concise fixes.

---

### Issue 1 of 1
cfw/onion/data/save_directories.json:142
**Legacy Saves Become Undiscoverable**

When an existing Onion installation stores SNES saves under `Beetle Supafaust`, replacing that entry makes the scanner check only `Supafaust` and silently skip the old directory. Those saves then disappear from sync results, so the old name should remain as a compatibility alias.

Reviews (1): Last reviewed commit: "fix(onion/saves): 'Beetle Supafaust' to ..." | Re-trigger Greptile

],
"snes": [
"Beetle Supafaust",
"Supafaust",

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

P1 Legacy Saves Become Undiscoverable

When an existing Onion installation stores SNES saves under Beetle Supafaust, replacing that entry makes the scanner check only Supafaust and silently skip the old directory. Those saves then disappear from sync results, so the old name should remain as a compatibility alias.

Prompt To Fix With AI
This is a comment left during a code review.
Path: cfw/onion/data/save_directories.json
Line: 142

Comment:
**Legacy Saves Become Undiscoverable**

When an existing Onion installation stores SNES saves under `Beetle Supafaust`, replacing that entry makes the scanner check only `Supafaust` and silently skip the old directory. Those saves then disappear from sync results, so the old name should remain as a compatibility alias.

How can I resolve this? If you propose a fix, please make it concise.

Fix in Claude Code

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