FazBrowse GitHub Viewer | Trending |
URL:
| Home
Tools: [Download Repo ZIP]   [Original HTTPS Page]

Update #2601: merge main and address review feedback by advancedresearcharray · Pull Request #2619 · github/github-mcp-server · GitHub

Update #2601: merge main and address review feedback - #2619

Closed
advancedresearcharray wants to merge 4 commits into
github:feature/show-ui-parameterfrom
advancedresearcharray:pr2601-updates
Closed

Update #2601: merge main and address review feedback#2619
advancedresearcharray wants to merge 4 commits into
github:feature/show-ui-parameterfrom
advancedresearcharray:pr2601-updates

Conversation

advancedresearcharray commented Jun 6, 2026
edited
Loading

Copy link
Copy Markdown

Summary

Follow-up for #2601 to help move it toward merge:

  • Merged main into the PR branch — resolves BEHIND merge state (includes Add confidence parameter to issue mutation MCP tools #2605 and fix: Empty assignees array should clear assignees #2600)
  • Clarified form-params comments in issues.go and pullrequests.go: show_ui is a routing flag included so it does not trigger the non-form bypass, not a field the MCP App form collects
  • Fixed misleading show_ui description on create_pull_request: removed the "reviewers" example (that parameter exists on update_pull_request, not create_pull_request)
  • Regenerated docs and toolsnap for the updated description

All pkg/github and pkg/inventory tests pass locally.

Test plan

  • go test ./pkg/github/... ./pkg/inventory/...
  • Toolsnap updated for create_pull_request

RossTarrant and others added 2 commits June 4, 2026 10:17
* fix: Empty assignees array should clear assignees
Add an optional confidence integer parameter (0–100) to update_issue_type,
update_issue_labels, and set_issue_fields MCP tools. The confidence score
is passed through to the REST/GraphQL API on mutation calls.

- Rename structs to WithIntent (labelWithIntent, issueTypeWithIntent)
- Add confidence schema property (integer, min 0, max 100) with prompt
  guidance describing what different confidence levels represent
- Update tool descriptions to encourage including confidence scores
- Pass confidence in the API request body alongside rationale/suggest

Closes github/plan-track-agentic-toolkit#219

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
root added 2 commits June 6, 2026 04:42
- Clarify form-params comments: show_ui is a routing flag, not form-collected
- Fix create_pull_request show_ui description (remove misleading reviewers example)
- Regenerate docs and toolsnap

Copy link
Copy Markdown
Collaborator

Closing — this re-submits #2601, which is owned by its original author and being handled there. Please contribute via the original PR rather than opening a duplicate.

This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters. Learn more about bidirectional Unicode characters
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.

4 participants


Back | FazBrowse Home | New Git URL