Conversation
There was a problem hiding this comment.
Pull request overview
This PR adds a small feedback footer to the MCP Apps UI so users have an easy way to file feedback from within the app experience.
Changes:
- Added a new
FeedbackFootercomponent that displays a feedback message and URL. - Rendered
FeedbackFooterfromAppProviderso it appears across MCP Apps.
Reviewed changes
Copilot reviewed 2 out of 2 changed files in this pull request and generated 2 comments.
| File | Description |
|---|---|
| ui/src/components/FeedbackFooter.tsx | Introduces the footer content and (intended) feedback destination. |
| ui/src/components/AppProvider.tsx | Wires the footer into the shared app layout so it appears for all apps. |
| <Text sx={{ color: "fg.subtle", fontSize: 0, textAlign: "center" }}> | ||
| Help us improve MCP Apps support in the GitHub MCP Server | ||
| <br /> | ||
| github.com/github/github-mcp-server/issues/new?template=insiders-feedback.md |
There was a problem hiding this comment.
The feedback URL uses template=insiders-feedback.md, but this repository only has .github/ISSUE_TEMPLATE/bug_report.md and feature_request.md. As-is, the template query param won’t select a valid template; either add the referenced issue template file or update the link to an existing template (or use ?template=feature_request.md / bug_report.md).
| github.com/github/github-mcp-server/issues/new?template=insiders-feedback.md | |
| https://github.com/github/github-mcp-server/issues/new?template=feature_request.md |
| <Text sx={{ color: "fg.subtle", fontSize: 0, textAlign: "center" }}> | ||
| Help us improve MCP Apps support in the GitHub MCP Server | ||
| <br /> | ||
| github.com/github/github-mcp-server/issues/new?template=insiders-feedback.md | ||
| </Text> |
There was a problem hiding this comment.
The URL is rendered as plain text (and lacks an https:// scheme), so it may not be clickable and isn’t keyboard-accessible. Consider using Primer’s Link component with a full URL (and if using target="_blank", include rel="noopener noreferrer") so users can reliably open the feedback page.
There was a problem hiding this comment.
It isn't clickable as this isn't supported in MCP Apps
Summary
This pull request adds a feedback footer to the application layout, encouraging users to provide feedback and help improve MCP Apps support. The new footer is included at the bottom of every MCP App by updating the main application provider component.
Why
Closes https://github.com/github/copilot-mcp-core/issues/1470
What changed
UI Enhancements:
FeedbackFootercomponent that displays a message and a link for users to provide feedback about MCP Apps support. (ui/src/components/FeedbackFooter.tsx)AppProvidercomponent to include the newFeedbackFooterbelow the main content, ensuring it appears consistently across the app. (ui/src/components/AppProvider.tsx) [1] [2]MCP impact
Prompts tested (tool changes only)
Security / limits
Tool renaming
deprecated_tool_aliases.goNote: if you're renaming tools, you must add the tool aliases. For more information on how to do so, please refer to the official docs.
Lint & tests
./script/lint./script/testDocs