Skip to content

feat(managed): patcher provenance into deployment manifests (U6)#181

Merged
oldrepublicwizard merged 1 commit into
masterfrom
feat/parity-u6-patcher-provenance
Jul 17, 2026
Merged

feat(managed): patcher provenance into deployment manifests (U6)#181
oldrepublicwizard merged 1 commit into
masterfrom
feat/parity-u6-patcher-provenance

Conversation

@oldrepublicwizard

Copy link
Copy Markdown
Owner

Summary

  • Pre-install game-dir hash snapshot + post-deploy diff for components that run Patcher.
  • DeploymentService.RecordLiveGameFilesAsync merges live paths into the component manifest (with staged entries).
  • Uninstall can delete recorded patcher outputs; in-place overwrite restore via ImmutableCheckpoint CAS is still deferred.
  • Wizard summary distinguishes recorded provenance vs remaining warnings.

Test plan

  • RecordLiveGameFiles_ThenUninstall_RemovesPatcherWrittenFile
  • RecordLiveGameFiles_MergesWithExistingStagedManifest
  • CI green
  • Optional desktop: managed install of a patcher mod → Deployed badge → uninstall removes patcher outputs

Snapshot game-dir hashes before a managed component install, diff after
deploy, and merge HoloPatcher-touched paths into the deployment manifest
so uninstall can delete them. Full CAS pre-image restore remains deferred.
@chatgpt-codex-connector

Copy link
Copy Markdown

You have reached your Codex usage limits for code reviews. You can see your limits in the Codex usage dashboard.

@oldrepublicwizard
oldrepublicwizard merged commit 66f2bba into master Jul 17, 2026
11 checks passed
@oldrepublicwizard
oldrepublicwizard deleted the feat/parity-u6-patcher-provenance branch July 17, 2026 07:27
oldrepublicwizard added a commit that referenced this pull request Jul 17, 2026
Mark Phase 6 FOMOD gate Merged (#170), record #176/#177/#179/#180/#181/#171,
and set Next to residual gaps only.

Co-authored-by: Copilot <th3w1zard1@users.noreply.github.com>
@oldrepublicwizard
oldrepublicwizard restored the feat/parity-u6-patcher-provenance branch July 25, 2026 02:16
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