feat(manager): show owned deployment files - #81
Conversation
Convert every prepared Manager root to a rename-compatible handle before any LockFileEx wait, so a queued Store snapshot cannot block the active writer's atomic loadout replacement.
|
@codex review |
|
Codex Review: Didn't find any major issues. Nice work! Reviewed commit: ℹ️ About Codex in GitHubYour team has set up Codex to review pull requests in this repo. Reviews are triggered when you
If Codex has suggestions, it will comment; otherwise it will react with 👍. Codex can also answer questions or update the PR. Try commenting "@codex address that feedback". |
|
@codex review |
|
@cursor review |
|
Codex Review: Didn't find any major issues. Keep it up! Reviewed commit: ℹ️ About Codex in GitHubYour team has set up Codex to review pull requests in this repo. Reviews are triggered when you
If Codex has suggestions, it will comment; otherwise it will react with 👍. Codex can also answer questions or update the PR. Try commenting "@codex address that feedback". |
|
@codex review |
|
Codex Review: Didn't find any major issues. 👍 Reviewed commit: ℹ️ About Codex in GitHubYour team has set up Codex to review pull requests in this repo. Reviews are triggered when you
If Codex has suggestions, it will comment; otherwise it will react with 👍. Codex can also answer questions or update the PR. Try commenting "@codex address that feedback". |
|
@codex review |
|
@cursor review |
|
Codex Review: Didn't find any major issues. Swish! Reviewed commit: ℹ️ About Codex in GitHubYour team has set up Codex to review pull requests in this repo. Reviews are triggered when you
If Codex has suggestions, it will comment; otherwise it will react with 👍. Codex can also answer questions or update the PR. Try commenting "@codex address that feedback". |
There was a problem hiding this comment.
✅ Bugbot reviewed your changes and found no new issues!
Comment @cursor review or bugbot run to trigger another review on this PR
Reviewed by Cursor Bugbot for commit d30d5af. Configure here.
Summary
Validation
Note
Medium Risk
Touches deploy status wire shape and Windows concurrent store locking, but ownership data is additive read-only UI with strict validation and no new cleanup actions.
Overview
Adds bounded, display-only ownership evidence from the validated Manager deploy record to
mgr_statusas optionalmanager_owned, grouped into live replacements, pristine backups, additive paks/containers, UE4SS dirs, and recovery paths/holders. Only records with exact ownermanagerpublish it; nothing deployed, Studio, and unknown states ignore the field even if present.The Mod Manager Status details dialog gains a collapsed Recorded ownership evidence section (localized) with selectable, sanitized path lists, truncation messaging, and lazy rendering for large sets. Dart parsing mirrors native caps and drops malformed wire data without breaking base status.
Also adjusts Windows manager root locking so every root is prepared for rename-compatible acquisition before the first blocking lock, and documents the feature in the app README and mod-manager guide.
Reviewed by Cursor Bugbot for commit d30d5af. Bugbot is set up for automated code reviews on this repo. Configure here.