Skip to content

bugs? - #48

Open
Brandher58 wants to merge 1 commit into
madoiscool:mainfrom
Brandher58:main
Open

bugs?#48
Brandher58 wants to merge 1 commit into
madoiscool:mainfrom
Brandher58:main

Conversation

@Brandher58

Copy link
Copy Markdown
Contributor
  1. Single owner for revert toasts (no more double error)
    RevertDenuvoFix is now the sole emitter of revert feedback (done,
    partial, game-not-found, no-manifest), matching the apply path.
    ConfirmRevert just flips IsApplied on success. Previously a failed
    revert with the game running stacked two error toasts on top of each
    other — the exact failure mode you hit most.

  2. Manifest as the apply's commit point (fixes orphaned backups)
    ApplyDenuvoFix used to write the manifest at the end inside an empty
    catch{} — a busy/locked manifest path silently succeeded, leaving .bak
    files with nothing indexing them and no Revert button to ever surface.
    Apply now runs in three phases: plan every entry (File.Exists only),
    persist the manifest FIRST (abort untouched if it can't be written),
    then extract. Any mid-extract failure keeps the manifest on disk, so
    the partial result is always recoverable via Revert.

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