Narrow issue type removal to MCP tools

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

Copilot-Session: ea8faa5c-7f26-4e2d-bf9c-6f0b5f173e8c

Author: zwick

README.md

@@ -941,7 +941,7 @@ The following sets of tools are available:
   - `state`: New state (string, optional)
   - `state_reason`: Reason for the state change. Ignored unless state is changed. (string, optional)
   - `title`: Issue title (string, optional)
-  - `type`: Type of this issue. Use list_issue_types to get valid values. For updates, pass null to remove the current type. If the repository doesn't support issue types, omit this parameter. (string | null, optional)
+  - `type`: Type of this issue. Use list_issue_types to get valid values. For updates, pass null to remove the current type. If the repository doesn't support issue types, omit this parameter. (, optional)
 
 - **list_issue_fields** - List issue fields
   - **Required OAuth Scopes (any of)**: `repo`, `read:org`

cmd/github-mcp-server/generate_docs.go

@@ -276,19 +276,15 @@ func writeToolDoc(buf *strings.Builder, tool inventory.ServerTool) {
 			var typeStr string
 
 			// Get the type and description
-			if len(prop.Types) > 0 {
-				typeStr = strings.Join(prop.Types, " | ")
-			} else {
-				switch prop.Type {
-				case "array":
-					if prop.Items != nil {
-						typeStr = prop.Items.Type + "[]"
-					} else {
-						typeStr = "array"
-					}
-				default:
-					typeStr = prop.Type
+			switch prop.Type {
+			case "array":
+				if prop.Items != nil {
+					typeStr = prop.Items.Type + "[]"
+				} else {
+					typeStr = "array"
 				}
+			default:
+				typeStr = prop.Type
 			}
 
 			// Indent any continuation lines in the description to maintain markdown formatting

docs/feature-flags.md

@@ -71,7 +71,7 @@ runtime behavior (such as output formatting) won't appear here.
   - `state`: New state (string, optional)
   - `state_reason`: Reason for the state change. Ignored unless state is changed. (string, optional)
   - `title`: Issue title (string, optional)
-  - `type`: Type of this issue. Use list_issue_types to get valid values. For updates, pass null to remove the current type. If the repository doesn't support issue types, omit this parameter. (string | null, optional)
+  - `type`: Type of this issue. Use list_issue_types to get valid values. For updates, pass null to remove the current type. If the repository doesn't support issue types, omit this parameter. (, optional)
 
 - **ui_get** - Get UI data
   - **Required OAuth Scopes (any of)**: `repo`, `read:org`
@@ -200,7 +200,7 @@ runtime behavior (such as output formatting) won't appear here.
   - `confidence`: How confident you are in this choice. Use 'HIGH' for clear signal or explicit user request, 'MEDIUM' for reasonable inference with some ambiguity, 'LOW' for best guess with limited signal. (string, optional)
   - `is_suggestion`: If true, this issue type change is sent to the API as a suggestion (suggest:true) rather than an applied value. Whether the type is applied or recorded as a proposal is determined by the API. (boolean, optional)
   - `issue_number`: The issue number to update (number, required)
-  - `issue_type`: The issue type to set, or null to remove the current type (string | null, required)
+  - `issue_type`: The issue type to set, or null to remove the current type (, required)
   - `owner`: Repository owner (username or organization) (string, required)
   - `rationale`: One concise sentence explaining what specifically about the issue led you to choose this type. State the concrete signal (e.g. 'Reports a crash when saving' → bug, 'Asks for dark mode support' → feature). (string, optional)
   - `repo`: Repository name (string, required)

docs/insiders-features.md

@@ -65,7 +65,7 @@ The list below is generated from the Go source. It covers tool **inventory and s
   - `state`: New state (string, optional)
   - `state_reason`: Reason for the state change. Ignored unless state is changed. (string, optional)
   - `title`: Issue title (string, optional)
-  - `type`: Type of this issue. Use list_issue_types to get valid values. For updates, pass null to remove the current type. If the repository doesn't support issue types, omit this parameter. (string | null, optional)
+  - `type`: Type of this issue. Use list_issue_types to get valid values. For updates, pass null to remove the current type. If the repository doesn't support issue types, omit this parameter. (, optional)
 
 - **ui_get** - Get UI data
   - **Required OAuth Scopes (any of)**: `repo`, `read:org`

pkg/github/issues.go

@@ -2048,7 +2048,8 @@ var issueWriteFormParams = map[string]struct{}{
 }
 
 // issueWriteAwaitingFormResult builds the "awaiting form submission" stub
-// returned when issue_write hands off to the MCP App form. The result is marked IsError=true so
+// returned when issue_write hands off to the MCP App form. The body is shared
+// by IssueWrite and LegacyIssueWrite. The result is marked IsError=true so
 // agents that bail on error don't claim success or chain dependent tool calls
 // while the user is still interacting with the form; the host renders the UI
 // regardless because rendering is keyed off the tool's _meta.ui resourceUri.
@@ -2076,8 +2077,12 @@ func issueWriteAwaitingFormResult(method, owner, repo string, issueNumber int) *
 	return utils.NewToolResultAwaitingFormSubmission(msg)
 }
 
-// IssueWrite creates the consolidated issue_write tool. It is disabled when
-// the granular issue tools are enabled.
+// IssueWrite is the FeatureFlagIssueFields-enabled variant of issue_write
+// (with the issue_fields parameter). LegacyIssueWrite is served when the flag
+// is off. Both register under the tool name "issue_write"; exactly one is
+// active at a time via mutually exclusive feature-flag annotations. When the
+// flag is removed, delete LegacyIssueWrite outright and drop the feature-flag
+// fields on IssueWrite.
 func IssueWrite(t translations.TranslationHelperFunc) inventory.ServerTool {
 	st := NewTool(
 		ToolsetMetadataIssues,

ui/src/apps/issue-write/App.tsx

@@ -418,7 +418,6 @@ function CreateIssueApp() {
   // Issue types state
   const [availableIssueTypes, setAvailableIssueTypes] = useState<IssueTypeItem[]>([]);
   const [selectedIssueType, setSelectedIssueType] = useState<IssueTypeItem | null>(null);
-  const [issueTypeChanged, setIssueTypeChanged] = useState(false);
   const [issueTypesLoading, setIssueTypesLoading] = useState(false);
 
   // State transition state
@@ -456,7 +455,6 @@ function CreateIssueApp() {
   const method = (toolInput?.method as string) || "create";
   const issueNumber = toolInput?.issue_number as number | undefined;
   const isUpdateMode = method === "update" && issueNumber !== undefined;
-  const hasExplicitIssueType = Object.prototype.hasOwnProperty.call(toolInput ?? {}, "type");
 
   // Initialize from toolInput or selected repo
   const owner = selectedRepo?.owner || (toolInput?.owner as string) || "";
@@ -714,7 +712,7 @@ function CreateIssueApp() {
       labels: false,
       assignees: false,
       milestone: false,
-      type: hasExplicitIssueType && toolInput?.type === null,
+      type: false,
       issueFields: false,
     };
     setExistingIssueData(null);
@@ -724,7 +722,6 @@ function CreateIssueApp() {
     setSelectedAssignees([]);
     setSelectedMilestone(null);
     setSelectedIssueType(null);
-    setIssueTypeChanged(hasExplicitIssueType && toolInput?.type === null);
     setCurrentState("open");
     setStateReason("completed");
     setDuplicateOf("");
@@ -753,7 +750,7 @@ function CreateIssueApp() {
     } else {
       setSelectedRepo(null);
     }
-  }, [toolInput, hasExplicitIssueType]);
+  }, [toolInput]);
 
   // Load existing issue data when in update mode
   useEffect(() => {
@@ -803,7 +800,7 @@ function CreateIssueApp() {
 
             // Pre-fill issue type immediately from issue data
             const issueTypeName = issueData.type?.name || (typeof issueData.type === 'string' ? issueData.type : null);
-            if (issueTypeName && !prefillApplied.current.type && !hasExplicitIssueType) {
+            if (issueTypeName && !prefillApplied.current.type) {
               setSelectedIssueType({ id: issueTypeName, text: issueTypeName });
               prefillApplied.current.type = true;
             }
@@ -832,7 +829,7 @@ function CreateIssueApp() {
     };
 
     loadExistingIssue();
-  }, [isUpdateMode, owner, repo, issueNumber, app, callTool, existingIssueData, hasExplicitIssueType]);
+  }, [isUpdateMode, owner, repo, issueNumber, app, callTool, existingIssueData]);
 
   // Apply existing labels when available labels load
   useEffect(() => {
@@ -1035,12 +1032,6 @@ function CreateIssueApp() {
       }
       if (selectedIssueType) {
         params.type = selectedIssueType.text;
-      } else if (issueTypeChanged) {
-        if (isUpdateMode) {
-          params.type = null;
-        } else {
-          delete params.type;
-        }
       }
 
       if (requestedState) {
@@ -1124,7 +1115,6 @@ function CreateIssueApp() {
     selectedAssignees,
     selectedMilestone,
     selectedIssueType,
-    issueTypeChanged,
     isUpdateMode,
     issueNumber,
     stateReason,
@@ -1543,10 +1533,7 @@ function CreateIssueApp() {
                 <>
                   {selectedIssueType && (
                     <ActionList.Item
-                      onSelect={() => {
-                        setSelectedIssueType(null);
-                        setIssueTypeChanged(true);
-                      }}
+                      onSelect={() => setSelectedIssueType(null)}
                     >
                       Clear selection
                     </ActionList.Item>
@@ -1555,10 +1542,7 @@ function CreateIssueApp() {
                     <ActionList.Item
                       key={type.id}
                       selected={selectedIssueType?.id === type.id}
-                      onSelect={() => {
-                        setSelectedIssueType(type);
-                        setIssueTypeChanged(true);
-                      }}
+                      onSelect={() => setSelectedIssueType(type)}
                     >
                       {type.text}
                     </ActionList.Item>