Skip to content

feat(mcp): add feedback tool - #8544

Open
joshblack wants to merge 4 commits into
mainfrom
feat/add-feedback-tool
Open

joshblack wants to merge 4 commits into
mainfrom
feat/add-feedback-tool

Conversation

@joshblack

@joshblack joshblack commented Oct 14, 2026 •

Copy link
Copy Markdown
Member

Closes https://github.com/github/primer/issues/7254

Add prepare_feedback to the MCP server to prepare reviewable feedback drafts without submitting issues. Feedback defaults to github/primer, with primer/react available only when the user cannot create an issue in github/primer.

Drafts include a minimal reproduction attempt (or an explicit user-requested skip), optional related links, and the final task outcome. Timing and privacy guidance live in the tool description, not server-wide initialization instructions.

Drafts use a prefilled issue link within a conservative 6,000-byte URL budget. Larger drafts return structured gh issue create arguments so the agent can offer to submit on the user's behalf after explicit approval, passing the full body through stdin. The tool never executes the command or files an issue itself.

Changelog

New

  • Add prepare_feedback for the eight feedback categories, with reproduction details, optional related links, and blocked, worked-around, or resolved task outcomes.
  • Add a GitHub CLI handoff for drafts that exceed the prefilled issue URL budget, without truncating the report or submitting automatically.

Changed

Removed

Rollout strategy

  • No changes needed in github/github-ui
  • Needs changes in github/github-ui
  • Ships with feature flag(s)
  • Not sure
  • None

Testing & Reviewing

Copilot AI balanced review requested due to automatic review settings October 14, 2026 16:38
@joshblack
joshblack requested a review from a team as a code owner October 14, 2026 16:38
@joshblack
joshblack requested a review from siddharthkp October 14, 2026 16:38
@changeset-bot

changeset-bot Bot commented Oct 14, 2026 •

Copy link
Copy Markdown

🦋 Changeset detected

Latest commit: 8b8e5f7

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 1 package
Name Type
@primer/mcp Minor

Not sure what this means? Click here to learn what changesets are.

Click here if you're a maintainer who wants to add another changeset to this PR

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>

Copilot-Session: cafa3ab6-d90a-4f88-8953-d754886f57a4

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

🟢 Approval recommended

Static review found no blocking issues in the draft-only implementation and accompanying coverage; tests were not executed.

0 open findings

What changed in this PR

Adds prepare_feedback to the Primer MCP server to generate reviewable issue drafts without submitting them.

Changes:

  • Adds validated feedback drafts with reproduction details, outcomes, and optional links.
  • Defaults to github/primer, with a guarded public-repository fallback.
  • Adds integration tests and a minor-release changeset.
File Description
packages/​mcp/​src/​server.ts Registers the feedback tool.
packages/​mcp/​src/​server.test.ts Tests draft generation, validation, and repository selection.
packages/​mcp/​src/​feedback.ts Implements feedback guidance, schemas, and draft generation.
.changeset/​agent-feedback-drafts.md Records the new MCP feature for release.

🧠 Review effort: Balanced


💡 Add a code-review agent skill for context-aware, tailored reviews. Learn more in the docs.

joshblack and others added 2 commits October 14, 2026 14:04
Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>\nCopilot-Session: cafa3ab6-d90a-4f88-8953-d754886f57a4
Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Copilot-Session: cafa3ab6-d90a-4f88-8953-d754886f57a4

This branch has not been deployed

No deployments
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.

2 participants