Skip to content

Add update action for pull request comment threads - #1568

Merged
Dan Hellem (danhellem) merged 2 commits into
mainfrom
users/danhellem/update-pr-comments-1
Aug 31, 2026
Merged

Add update action for pull request comment threads#1568
Dan Hellem (danhellem) merged 2 commits into
mainfrom
users/danhellem/update-pr-comments-1

Conversation

@danhellem

@danhellem Dan Hellem (danhellem) commented Aug 31, 2026

Copy link
Copy Markdown
Collaborator

This pull request adds support for updating individual comments in pull request threads via the repo_pull_request_thread_write tool. It introduces a new update action, updates the schema and handler logic to support this operation, and adds comprehensive tests to ensure correct behavior and error handling.

Tooling enhancements:

  • Added a new update action to the repo_pull_request_thread_write tool, allowing users to update existing comments in pull request threads. The schema now requires threadId, commentId, and content for this action, and the handler logic invokes gitApi.updateComment to perform the update. [1] [2] [3]
  • Updated the tool documentation in TOOLSET.md to include the new update action for pull request threads.

Testing improvements:

  • Added tests for the new update action, including successful updates, full response handling, error cases when required parameters are missing, and when the update operation fails. [1] [2]
  • Mocked the new updateComment method in the test setup to enable testing of the update functionality. [1] [2]

GitHub issue number

#1560

Associated Risks

None

PR Checklist

  • I have read the contribution guidelines
  • I have read the code of conduct guidelines
  • Title of the pull request is clear and informative.
  • 👌 Code hygiene
  • 🔭 Telemetry added, updated, or N/A
  • 📄 Documentation added, updated, or N/A
  • 🛡️ Automated tests added, or N/A

🧪 How did you test it?

Ran manual tests against pr comments across different threads. Updated automated tests

@github-actions

Copy link
Copy Markdown

Dependency Review

✅ No vulnerabilities or license issues or OpenSSF Scorecard issues found.

Scanned Files

None

@danhellem
Dan Hellem (danhellem) merged commit 63f132a into main Aug 31, 2026
7 checks passed
@danhellem
Dan Hellem (danhellem) deleted the users/danhellem/update-pr-comments-1 branch August 31, 2026 19:19
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Tool request -- extend the PR comment tool to allow updating existing comment

2 participants