M3: Write actions on issues/PRs (+ mutation infrastructure)#290
Draft
michaeljolley wants to merge 1 commit into
Draft
M3: Write actions on issues/PRs (+ mutation infrastructure)#290michaeljolley wants to merge 1 commit into
michaeljolley wants to merge 1 commit into
Conversation
Add shared mutation infrastructure and destructive-action confirmation: - IGitHubMutationManager/GitHubMutationManager for Octokit REST writes (close/reopen issue, close/reopen/merge pull request) - MutationMediator (weak event) to refresh surfaces after a write - GitHubMutationCommand (write + success/error toast + notify) - ConfirmedCommand wrapping destructive actions in CommandResult.Confirm - MutationCommandsFactory building state-aware commands Wire mutation commands into the issue/PR list pages and detail content pages. Destructive ops (merge, close) require confirmation and are marked critical; reopen runs immediately. Add resource strings and unit tests. Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Part of the CmdPal GitHub extension parity roadmap (stacked on M2, PR #289).
Summary
Introduces shared write/mutation infrastructure and the first state-changing actions on existing issues and pull requests.
Mutation infrastructure (reused by M4/M5)
Wiring
Safety
Destructive ops (merge, close) are marked \IsCritical\ and require an explicit confirmation step; reopen runs immediately.
Deferrals (documented in plan)
Tests
Localization
Added command names, success/error toasts, and confirmation title/description strings to \Resources.resw.