Clarify where show_ui appears in generated docs

The code comments next to the show_ui schema entries (and the
uiOnlySchemaProperties allowlist) said the property is documented in
"toolsnaps / README". README is generated from the stripped (non-UI)
schema, so show_ui is not actually in it — it only appears in toolsnaps
and the feature-flag / insiders docs. Reword the comments to match
reality.

Comment-only change; no behavior or test impact.

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

Author: mattdholloway

pkg/github/issues.go

@@ -1922,8 +1922,10 @@ Options are:
 					// show_ui is hidden from clients that do not advertise MCP App
 					// UI support. The strip happens per-request in
 					// inventory.ToolsForRegistration; it is present in the static
-					// schema (and therefore in toolsnaps / README) so the UI-capable
-					// surface is fully documented.
+					// schema (and therefore in toolsnaps and the feature-flag /
+					// insiders docs) so the UI-capable surface is fully
+					// documented. It is intentionally not in the main README,
+					// which renders the stripped (non-UI) schema.
 					"show_ui": {
 						Type:        "boolean",
 						Description: "Whether to render the MCP App form instead of executing the request immediately. Defaults to true. Set to false to skip the form and execute directly — useful when you have all required values (especially ones the form does not collect, like labels, assignees, milestone, type, issue_fields, or state changes) and the user has already confirmed the action.",
@@ -2169,8 +2171,10 @@ Options are:
 					// show_ui is hidden from clients that do not advertise MCP App
 					// UI support. The strip happens per-request in
 					// inventory.ToolsForRegistration; it is present in the static
-					// schema (and therefore in toolsnaps / README) so the UI-capable
-					// surface is fully documented.
+					// schema (and therefore in toolsnaps and the feature-flag /
+					// insiders docs) so the UI-capable surface is fully
+					// documented. It is intentionally not in the main README,
+					// which renders the stripped (non-UI) schema.
 					"show_ui": {
 						Type:        "boolean",
 						Description: "Whether to render the MCP App form instead of executing the request immediately. Defaults to true. Set to false to skip the form and execute directly — useful when you have all required values (especially ones the form does not collect, like labels, assignees, milestone, type, or state changes) and the user has already confirmed the action.",

pkg/github/pullrequests.go

@@ -631,8 +631,10 @@ func CreatePullRequest(t translations.TranslationHelperFunc) inventory.ServerToo
 					// show_ui is hidden from clients that do not advertise MCP App
 					// UI support. The strip happens per-request in
 					// inventory.ToolsForRegistration; it is present in the static
-					// schema (and therefore in toolsnaps / README) so the UI-capable
-					// surface is fully documented.
+					// schema (and therefore in toolsnaps and the feature-flag /
+					// insiders docs) so the UI-capable surface is fully
+					// documented. It is intentionally not in the main README,
+					// which renders the stripped (non-UI) schema.
 					"show_ui": {
 						Type:        "boolean",
 						Description: "Whether to render the MCP App form instead of executing the request immediately. Defaults to true. Set to false to skip the form and execute directly — useful when you have all required values (especially ones the form does not collect, like reviewers) and the user has already confirmed the action.",

pkg/inventory/builder.go

@@ -410,8 +410,9 @@ func stripMCPAppsMetadata(tools []ServerTool) []ServerTool {
 
 // uiOnlySchemaProperties lists input-schema property names that should only
 // be visible to clients that advertise MCP Apps UI support. They live on the
-// static schema (so toolsnaps and README document the full UI-capable
-// surface) and are stripped per-request when the same gate that hides
+// static schema (so toolsnaps and the feature-flag / insiders docs document
+// the full UI-capable surface; the main README renders the stripped
+// non-UI schema) and are stripped per-request when the same gate that hides
 // _meta.ui is true.
 var uiOnlySchemaProperties = []string{
 	"show_ui", // explicit "render the MCP App form" toggle on form-backed write tools