From ed355a474fea43a74b9dab867cf23fff085fdb34 Mon Sep 17 00:00:00 2001 From: Kynan Ware <47394200+BagToad@users.noreply.github.com> Date: Tue, 12 May 2026 14:16:06 -0600 Subject: [PATCH] Reject --add-sub-issue when editing multiple issues A sub-issue can have only one parent, so applying --add-sub-issue across multiple parent args is ambiguous. Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com> --- pkg/cmd/issue/edit/edit.go | 4 ++++ pkg/cmd/issue/edit/edit_test.go | 5 +++++ 2 files changed, 9 insertions(+) diff --git a/pkg/cmd/issue/edit/edit.go b/pkg/cmd/issue/edit/edit.go index 78e8f1e83..81a3d79e9 100644 --- a/pkg/cmd/issue/edit/edit.go +++ b/pkg/cmd/issue/edit/edit.go @@ -200,6 +200,10 @@ func NewCmdEdit(f *cmdutil.Factory, runF func(*EditOptions) error) *cobra.Comman return cmdutil.FlagErrorf("multiple issues cannot be edited interactively") } + if len(opts.IssueNumbers) > 1 && len(opts.AddSubIssues) > 0 { + return cmdutil.FlagErrorf("`--add-sub-issue` cannot be used when editing multiple issues") + } + if runF != nil { return runF(opts) } diff --git a/pkg/cmd/issue/edit/edit_test.go b/pkg/cmd/issue/edit/edit_test.go index 6fc4d6ea7..f5811927e 100644 --- a/pkg/cmd/issue/edit/edit_test.go +++ b/pkg/cmd/issue/edit/edit_test.go @@ -336,6 +336,11 @@ func TestNewCmdEdit(t *testing.T) { AddSubIssues: []string{"123", "124"}, }, }, + { + name: "add-sub-issue rejected with multiple issues", + input: "23 24 --add-sub-issue 123", + wantsErr: true, + }, { name: "remove-sub-issue flag", input: "23 --remove-sub-issue 50",