From 91cd327008762f2cc066daf99b68b5210decdc82 Mon Sep 17 00:00:00 2001 From: Matt Holloway Date: Wed, 25 Feb 2026 16:47:41 +0000 Subject: [PATCH] clarify user confirmation requirement in issue and pull request creation messages --- pkg/github/issues.go | 4 ++-- pkg/github/pullrequests.go | 2 +- 2 files changed, 3 insertions(+), 3 deletions(-) diff --git a/pkg/github/issues.go b/pkg/github/issues.go index d07ce3fed..81ce29df7 100644 --- a/pkg/github/issues.go +++ b/pkg/github/issues.go @@ -1111,9 +1111,9 @@ Options are: if numErr != nil { return utils.NewToolResultError("issue_number is required for update method"), nil, nil } - 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 + 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 } - 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 + 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 } title, err := OptionalParam[string](args, "title") diff --git a/pkg/github/pullrequests.go b/pkg/github/pullrequests.go index 8e1ad10e5..587a5c0c4 100644 --- a/pkg/github/pullrequests.go +++ b/pkg/github/pullrequests.go @@ -538,7 +538,7 @@ func CreatePullRequest(t translations.TranslationHelperFunc) inventory.ServerToo uiSubmitted, _ := OptionalParam[bool](args, "_ui_submitted") if deps.GetFlags(ctx).InsidersMode && clientSupportsUI(ctx, req) && !uiSubmitted { - 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 + 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 } // When creating PR, title/head/base are required