Add issue dependency read/write MCP tools#2751
Open
zwick wants to merge 2 commits into
Open
Conversation
Add two feature-flagged tools for issue blocked-by / blocking relationships, gated behind the issue_dependencies flag so they stay off the default tool surface (auto-enabled for insiders only): - issue_dependency_read: get_blocked_by / get_blocking via the Issue.blockedBy / Issue.blocking GraphQL connections, cursor-paginated. - issue_dependency_write: add / remove x blocked_by / blocking via the addBlockedBy / removeBlockedBy mutations. Accepts issue numbers and resolves them to node IDs in a single aliased query; "blocking" is the inverse of "blocked_by" with the subject/related roles swapped. Closes the MCP gap behind the gh CLI dependency verbs (cli/cli#13057). Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Reject a write where the subject and related issue are identical before resolving node IDs or issuing a mutation, avoiding two API round-trips. Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
6 tasks
Contributor
There was a problem hiding this comment.
Pull request overview
Adds a new, opt-in “issue dependencies” capability to the GitHub MCP Server by introducing two feature-flagged MCP tools for reading and mutating native GitHub issue dependency edges (blocked-by / blocking) via GraphQL, including snapshots, tests, and generated docs updates.
Changes:
- Added
issue_dependency_read(cursor-paginated) to listblockedBy/blockingrelationships for an issue via GraphQL connections. - Added
issue_dependency_writeto add/remove dependency edges viaaddBlockedBy/removeBlockedBy, resolving issue numbers to node IDs in a single aliased query and rejecting self-dependencies early. - Introduced the
issue_dependenciesfeature flag and documented the newly gated tools in generated docs.
Show a summary per file
| File | Description |
|---|---|
| pkg/github/tools.go | Registers the new dependency tools in the overall tool inventory (feature-gated). |
| pkg/github/issue_dependencies.go | Implements issue_dependency_read + issue_dependency_write using GraphQL queries/mutations and cursor pagination. |
| pkg/github/issue_dependencies_test.go | Adds tool schema snapshot tests and behavioral tests for read/write operations and validation paths. |
| pkg/github/feature_flags.go | Introduces FeatureFlagIssueDependencies and wires it into allowed + insiders feature flag sets. |
| pkg/github/toolsnaps/issue_dependency_read_ff_issue_dependencies.snap | Adds the toolsnap for the feature-flagged read tool schema. |
| pkg/github/toolsnaps/issue_dependency_write_ff_issue_dependencies.snap | Adds the toolsnap for the feature-flagged write tool schema. |
| docs/insiders-features.md | Generated docs: documents the new issue_dependencies gated tools under insiders. |
| docs/feature-flags.md | Generated docs: documents the new issue_dependencies flag and its gated tools. |
Copilot's findings
- Files reviewed: 8/8 changed files
- Comments generated: 0
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
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
Add two feature-flagged MCP tools for issue dependencies:
issue_dependency_read(list an issue's blocked-by / blocking relationships) andissue_dependency_write(add / remove a blocked-by or blocking relationship).Why
github-mcp-serverhad no way to read or edit issue dependencies, although the GitHub API supports them and theghCLI shipped the verbs in v2.94.0. This closes that MCP gap.Refs #2391, #950
What changed
issue_dependency_read—get_blocked_by/get_blockingvia the GraphQLIssue.blockedBy/Issue.blockingconnections, cursor-paginated.issue_dependency_write—add/remove×blocked_by/blockingvia theaddBlockedBy/removeBlockedBymutations. Accepts issue numbers (resolved to node IDs in one aliased query);blockingis the inverse ofblocked_bywith subject/related swapped. Rejects a self-dependency before any API round-trip.issue_dependenciesfeature flag — opt-in (user-enableable via--features/X-MCP-Features, auto-enabled in insiders mode) and kept out of the default toolset, mirroring thefile_blameprecedent.MCP impact
Prompts tested (tool changes only)
get_blocked_byget_blockingadd+blocked_byremove+blockingSecurity / limits
reposcope; reads are read-only, writes use the documented dependency mutations.Tool renaming
Lint & tests
./script/lint./script/testDocs
Split out from #2726, which now contains only the always-on
issue_read→get_parentmethod.