Skip to content

Commit 584d0c9

Browse files
clarify user confirmation requirement in issue and pull request creation messages (#2094)
1 parent 391990a commit 584d0c9

File tree

2 files changed

+3
-3
lines changed

2 files changed

+3
-3
lines changed

pkg/github/issues.go

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -1111,9 +1111,9 @@ Options are:
11111111
if numErr != nil {
11121112
return utils.NewToolResultError("issue_number is required for update method"), nil, nil
11131113
}
1114-
return utils.NewToolResultText(fmt.Sprintf("Ready to update issue #%d in %s/%s. The user will review and confirm via the interactive form.", issueNumber, owner, repo)), nil, nil
1114+
return utils.NewToolResultText(fmt.Sprintf("Ready to update issue #%d in %s/%s. The user will review and confirm via the interactive form. The issue has NOT been updated yet - the user MUST confirm this operation.", issueNumber, owner, repo)), nil, nil
11151115
}
1116-
return utils.NewToolResultText(fmt.Sprintf("Ready to create an issue in %s/%s. The user will review and confirm via the interactive form.", owner, repo)), nil, nil
1116+
return utils.NewToolResultText(fmt.Sprintf("Ready to create an issue in %s/%s. The user will review and confirm via the interactive form. The issue has NOT been created yet - the user MUST confirm this operation.", owner, repo)), nil, nil
11171117
}
11181118

11191119
title, err := OptionalParam[string](args, "title")

pkg/github/pullrequests.go

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -607,7 +607,7 @@ func CreatePullRequest(t translations.TranslationHelperFunc) inventory.ServerToo
607607
uiSubmitted, _ := OptionalParam[bool](args, "_ui_submitted")
608608

609609
if deps.GetFlags(ctx).InsidersMode && clientSupportsUI(ctx, req) && !uiSubmitted {
610-
return utils.NewToolResultText(fmt.Sprintf("Ready to create a pull request in %s/%s. The user will review and confirm via the interactive form.", owner, repo)), nil, nil
610+
return utils.NewToolResultText(fmt.Sprintf("Ready to create a pull request in %s/%s. The user will review and confirm via the interactive form. The PR has NOT been created yet - the user MUST confirm this operation.", owner, repo)), nil, nil
611611
}
612612

613613
// When creating PR, title/head/base are required

0 commit comments

Comments
 (0)