Skip to content

Conversation

@zyw400
Copy link
Member

@zyw400 zyw400 commented Oct 31, 2025

GitHub issue number: 634

Associated Risks

The current ADO API support much more search criteria, adding those back to the current search commits tool in MCP.

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?

Replace with use cases tested and models used

@zyw400 zyw400 requested a review from a team as a code owner October 31, 2025 16:11
@codecov-commenter
Copy link

codecov-commenter commented Oct 31, 2025

Codecov Report

❌ Patch coverage is 60.00000% with 14 lines in your changes missing coverage. Please review.
⚠️ Please upload report for BASE (main@52076fa). Learn more about missing BASE report.

Files with missing lines Patch % Lines
src/tools/repositories.ts 60.00% 8 Missing and 6 partials ⚠️
Additional details and impacted files
@@           Coverage Diff           @@
##             main     #645   +/-   ##
=======================================
  Coverage        ?   96.42%           
=======================================
  Files           ?       16           
  Lines           ?     1231           
  Branches        ?      241           
=======================================
  Hits            ?     1187           
  Misses          ?       17           
  Partials        ?       27           
Flag Coverage Δ
unittests 96.42% <60.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 linked an issue Nov 2, 2025 that may be closed by this pull request
@danhellem danhellem requested a review from nikolapeja6 November 2, 2025 18:06
@danhellem danhellem added the Needs Review 👓 needs review by the product team label Nov 2, 2025
@danhellem
Copy link
Contributor

I have assigned to @nikolapeja6 for review

Copy link
Member

@nikolapeja6 nikolapeja6 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks for your contribution @zyw400, lgtm

@nikolapeja6 nikolapeja6 merged commit f432438 into microsoft:main Nov 5, 2025
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.

Adding new commit/pull request search tools

4 participants