fix broken issue update logic

Author: mattdholloway

pkg/github/issues.go

@@ -1083,8 +1083,15 @@ Options are:
 				return utils.NewToolResultError(err.Error()), nil, nil
 			}
 
-			// When insiders mode is enabled, show UI - the host will detect the UI metadata and display the form
-			if deps.GetFlags().InsidersMode {
+			// When insiders mode is enabled and this is an initial request (no title provided),
+			// show UI - the host will detect the UI metadata and display the form.
+			// If title is provided, this is a submission from the UI form, so proceed with the operation.
+			title, err := OptionalParam[string](args, "title")
+			if err != nil {
+				return utils.NewToolResultError(err.Error()), nil, nil
+			}
+
+			if deps.GetFlags().InsidersMode && title == "" {
 				if method == "update" {
 					issueNumber, numErr := RequiredInt(args, "issue_number")
 					if numErr != nil {
@@ -1095,11 +1102,6 @@ Options are:
 				return utils.NewToolResultText(fmt.Sprintf("Ready to create an issue in %s/%s. The interactive form will be displayed.", owner, repo)), nil, nil
 			}
 
-			title, err := OptionalParam[string](args, "title")
-			if err != nil {
-				return utils.NewToolResultError(err.Error()), nil, nil
-			}
-
 			// Optional parameters
 			body, err := OptionalParam[string](args, "body")
 			if err != nil {

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

@@ -478,14 +478,18 @@ function CreateIssueApp() {
             const assigneeLogins = (issueData.assignees || [])
               .map((a: { login?: string } | string) => typeof a === 'string' ? a : a.login)
               .filter(Boolean) as string[];
+            console.log("Assignees from issue data:", issueData.assignees, "-> logins:", assigneeLogins, "prefillApplied:", prefillApplied.current.assignees);
             if (assigneeLogins.length > 0 && !prefillApplied.current.assignees) {
+              console.log("Setting selectedAssignees to:", assigneeLogins.map(login => ({ id: login, text: login })));
               setSelectedAssignees(assigneeLogins.map(login => ({ id: login, text: login })));
               prefillApplied.current.assignees = true;
             }
 
             // Pre-fill issue type immediately from issue data
             const issueTypeName = issueData.type?.name || (typeof issueData.type === 'string' ? issueData.type : null);
+            console.log("Issue type from issue data:", issueData.type, "-> name:", issueTypeName, "prefillApplied:", prefillApplied.current.type);
             if (issueTypeName && !prefillApplied.current.type) {
+              console.log("Setting selectedIssueType to:", { id: issueTypeName, text: issueTypeName });
               setSelectedIssueType({ id: issueTypeName, text: issueTypeName });
               prefillApplied.current.type = true;
             }
@@ -652,7 +656,15 @@ function CreateIssueApp() {
         params.type = selectedIssueType.text;
       }
 
+      console.log("Submitting issue_write with params:", JSON.stringify(params, null, 2));
+      console.log("selectedLabels:", selectedLabels);
+      console.log("selectedAssignees:", selectedAssignees);
       const result = await callTool("issue_write", params);
+      console.log("issue_write result:", result);
+      console.log("issue_write result.content:", result.content);
+      if (result.content?.[0]?.text) {
+        console.log("issue_write result text:", result.content[0].text);
+      }
 
       if (result.isError) {
         const textContent = result.content?.find(
@@ -863,7 +875,7 @@ function CreateIssueApp() {
       </Box>
 
       {/* Metadata section */}
-      <Box display="flex" gap={3} mb={3} sx={{ flexWrap: "wrap" }}>
+      <Box display="flex" gap={4} mb={3} sx={{ flexWrap: "wrap" }}>
         {/* Labels dropdown */}
         <ActionMenu>
           <ActionMenu.Button size="small" leadingVisual={TagIcon}>