feat(manager): align CLI, MCP, and docs - #92
Conversation
Expose token-bound recovery and Manager-only CLI reset so automation shares the native safety contract. Refresh MCP metadata, the plugin workflow, and acceptance evidence to match the validated V1 surface.
|
@codex review |
|
@cursor 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". |
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 32b2cd7. Configure here.
|
@codex review |
|
@cursor review |
|
Codex Review: Didn't find any major issues. Can't wait for the next one! 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 32b2cd7. Configure here.
Summary
Validation
cargo test -p gore-mod(505 passed)cargo test -p gore(full suite passed)cargo test -p gore-mcp(315 passed)python scripts/check_plugin.pypython scripts/check_docs_links.pyquick_validate.py plugins/gore/skills/gore-moddingNote
Medium Risk
Changes installation recovery, undeploy ownership boundaries, and MCP consent classification for Manager operations—areas that can brick or mislead automation if wrong—but the diff adds extensive integration tests and keeps recovery token-bound and fail-closed.
Overview
Manager CLI and engine gain read-only
mgr preflight, token-boundmgr recover(with--yes/--json), richerstatus --json, and Reset that only undeploys Manager-owned state while refusing Studio deployments (including owner races).analyzeandremovemessaging now distinguish “no recognized conflicts” from incomplete footprint knowledge and remind users to Apply after removal.MCP (87 commands) introduces finer Manager safety classes (
ManagerWrite, reconcile, loadout edits), reclassifiesmgr importas protected library state (not an install mutation), documents ungated but non-read-onlylist/analyze/status, and wiresrecoverto force non-interactive confirmation over stdio.AngelScript composition documents and implements correct handling of prepared-mini StaticNames operands during loadout-wide remap (fail-closed on missing rows).
Plugin 0.3.0 and guides/changelogs expand copy to the full import→recover lifecycle, update consent wording, and record the 2026-08-18 Nexus/load-order/script probe campaign with explicit qualification limits.
Reviewed by Cursor Bugbot for commit 32b2cd7. Bugbot is set up for automated code reviews on this repo. Configure here.