Skip to content

Add merge commit message option to repo_update_pull_request#1217

Merged
danhellem merged 4 commits intomicrosoft:mainfrom
simontherry:feature/add-pr-merge-commit-message
May 7, 2026
Merged

Add merge commit message option to repo_update_pull_request#1217
danhellem merged 4 commits intomicrosoft:mainfrom
simontherry:feature/add-pr-merge-commit-message

Conversation

@simontherry
Copy link
Copy Markdown

Summary

  • Add an optional mergeCommitMessage parameter to repo_update_pull_request.
  • Pass the value through to completionOptions.mergeCommitMessage when autocomplete completion options are built.
  • Add Jest coverage for the autocomplete path.

Why

Azure DevOps supports customizing the final merge or squash commit message through pull request completion options, but the MCP tool did not expose that field.

Fixes #1209

Validation

  • npm test -- --runTestsByPath test/src/tools/repositories.test.ts
  • npm run validate-tools

@simontherry
Copy link
Copy Markdown
Author

@microsoft-github-policy-service agree

@simontherry simontherry marked this pull request as ready for review May 2, 2026 11:11
@simontherry simontherry requested a review from a team as a code owner May 2, 2026 11:11
@danhellem danhellem self-assigned this May 4, 2026
@danhellem danhellem added the Needs Review 👓 needs review by the product team label May 4, 2026
@codecov-commenter
Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
⚠️ Please upload report for BASE (main@feac5c9). Learn more about missing BASE report.

Additional details and impacted files
@@           Coverage Diff           @@
##             main    #1217   +/-   ##
=======================================
  Coverage        ?   90.85%           
=======================================
  Files           ?       19           
  Lines           ?     2175           
  Branches        ?      514           
=======================================
  Hits            ?     1976           
  Misses          ?      102           
  Partials        ?       97           
Flag Coverage Δ
unittests 90.85% <100.00%> (?)

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@danhellem danhellem merged commit 1f89641 into microsoft:main May 7, 2026
4 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Needs Review 👓 needs review by the product team

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Add merge commit message option to repo_update_pull_request

3 participants