Skip to content

feat(manager): show owned deployment files - #81

Merged
dh0er merged 3 commits into
mainfrom
codex/mod-manager-v1-owned-details
Aug 12, 2026
Merged

feat(manager): show owned deployment files#81
dh0er merged 3 commits into
mainfrom
codex/mod-manager-v1-owned-details

Conversation

@dh0er

@dh0er dh0er commented Aug 12, 2026

Copy link
Copy Markdown
Owner

Summary

  • expose bounded recorded ownership evidence from the validated Manager deploy record
  • show five localized, collapsed path groups in Status details
  • keep the status wire additive and all paths display-only

Validation

  • full gore-mod, gore-ffi, gore, and Flutter suites
  • focused post-rebase native/FFI/Flutter tests
  • flutter analyze
  • three independent read-only reviews

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_status as optional manager_owned, grouped into live replacements, pristine backups, additive paks/containers, UE4SS dirs, and recovery paths/holders. Only records with exact owner manager publish 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.

dh0er added 2 commits August 12, 2026 15:49
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.
@dh0er

dh0er commented Aug 12, 2026

Copy link
Copy Markdown
Owner Author

@codex review

@chatgpt-codex-connector

Copy link
Copy Markdown

Codex Review: Didn't find any major issues. Nice work!

Reviewed commit: 0df4163cff

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

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".

@dh0er

dh0er commented Aug 12, 2026

Copy link
Copy Markdown
Owner Author

@codex review

@dh0er

dh0er commented Aug 12, 2026

Copy link
Copy Markdown
Owner Author

@cursor review

@chatgpt-codex-connector

Copy link
Copy Markdown

Codex Review: Didn't find any major issues. Keep it up!

Reviewed commit: 0df4163cff

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

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".

Comment thread apps/mod-manager/lib/status/ui/status_details_dialog.dart
@dh0er

dh0er commented Aug 12, 2026

Copy link
Copy Markdown
Owner Author

@codex review

@chatgpt-codex-connector

Copy link
Copy Markdown

Codex Review: Didn't find any major issues. 👍

Reviewed commit: d30d5af0ef

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

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".

@dh0er

dh0er commented Aug 12, 2026

Copy link
Copy Markdown
Owner Author

@codex review

@dh0er

dh0er commented Aug 12, 2026

Copy link
Copy Markdown
Owner Author

@cursor review

@chatgpt-codex-connector

Copy link
Copy Markdown

Codex Review: Didn't find any major issues. Swish!

Reviewed commit: d30d5af0ef

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

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".

@cursor cursor Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

✅ 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.

@dh0er
dh0er merged commit 06aec16 into main Aug 12, 2026
2 checks passed
@dh0er
dh0er deleted the codex/mod-manager-v1-owned-details branch August 12, 2026 15:52
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