Skip to content

fix: hide write UI resources in read-only mode#2528

Open
he-yufeng wants to merge 1 commit into
github:mainfrom
he-yufeng:fix/read-only-ui-resources
Open

fix: hide write UI resources in read-only mode#2528
he-yufeng wants to merge 1 commit into
github:mainfrom
he-yufeng:fix/read-only-ui-resources

Conversation

@he-yufeng
Copy link
Copy Markdown

Fixes #2519

Summary

  • pass read-only mode into MCP App UI resource registration
  • keep get_me_ui available in read-only mode
  • skip issue_write_ui and pr_write_ui when write tools are filtered out

To verify

  • go test ./pkg/github -run TestRegisterUIResources -count=1
  • go test ./cmd/github-mcp-server ./pkg/github
  • go test ./internal/ghmcp ./pkg/github
  • git diff --check

@he-yufeng he-yufeng requested a review from a team as a code owner May 22, 2026 16:41
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.

MCP App UI resources for write tools are registered in --read-only mode

2 participants