Support cross-repository parent issues

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

Copilot-Session: f71d9868-eef8-4fb0-84c6-df7c9a6a0ade

Author: zwick

README.md

@@ -937,7 +937,9 @@ The following sets of tools are available:
      (string, required)
   - `milestone`: Milestone number (number, optional)
   - `owner`: Repository owner (string, required)
-  - `parent_issue_number`: Issue number of the parent issue. Only used when method is 'create'. The new issue is created and attached to this parent in the same operation. (number, optional)
+  - `parent_issue_number`: Issue number of the parent issue. Only used when method is 'create' and cannot be combined with issue_fields. The new issue is created and attached to this parent in the same operation. (number, optional)
+  - `parent_owner`: Repository owner of the parent issue. Defaults to the value of owner. Only used when method is 'create' and parent_issue_number is provided. (string, optional)
+  - `parent_repo`: Repository name of the parent issue. Defaults to the value of repo. Only used when method is 'create' and parent_issue_number is provided. (string, optional)
   - `repo`: Repository name (string, required)
   - `state`: New state (string, optional)
   - `state_reason`: Reason for the state change. Ignored unless state is changed. (string, optional)

docs/feature-flags.md

@@ -67,7 +67,9 @@ runtime behavior (such as output formatting) won't appear here.
      (string, required)
   - `milestone`: Milestone number (number, optional)
   - `owner`: Repository owner (string, required)
-  - `parent_issue_number`: Issue number of the parent issue. Only used when method is 'create'. The new issue is created and attached to this parent in the same operation. (number, optional)
+  - `parent_issue_number`: Issue number of the parent issue. Only used when method is 'create' and cannot be combined with issue_fields. The new issue is created and attached to this parent in the same operation. (number, optional)
+  - `parent_owner`: Repository owner of the parent issue. Defaults to the value of owner. Only used when method is 'create' and parent_issue_number is provided. (string, optional)
+  - `parent_repo`: Repository name of the parent issue. Defaults to the value of repo. Only used when method is 'create' and parent_issue_number is provided. (string, optional)
   - `repo`: Repository name (string, required)
   - `state`: New state (string, optional)
   - `state_reason`: Reason for the state change. Ignored unless state is changed. (string, optional)
@@ -124,6 +126,8 @@ runtime behavior (such as output formatting) won't appear here.
   - `body`: Issue body content (optional) (string, optional)
   - `owner`: Repository owner (username or organization) (string, required)
   - `parent_issue_number`: Issue number of the parent issue. The new issue is created and attached to this parent in the same operation. (number, optional)
+  - `parent_owner`: Repository owner of the parent issue. Defaults to the value of owner. Only used when parent_issue_number is provided. (string, optional)
+  - `parent_repo`: Repository name of the parent issue. Defaults to the value of repo. Only used when parent_issue_number is provided. (string, optional)
   - `repo`: Repository name (string, required)
   - `title`: Issue title (string, required)
 

docs/insiders-features.md

@@ -61,7 +61,9 @@ The list below is generated from the Go source. It covers tool **inventory and s
      (string, required)
   - `milestone`: Milestone number (number, optional)
   - `owner`: Repository owner (string, required)
-  - `parent_issue_number`: Issue number of the parent issue. Only used when method is 'create'. The new issue is created and attached to this parent in the same operation. (number, optional)
+  - `parent_issue_number`: Issue number of the parent issue. Only used when method is 'create' and cannot be combined with issue_fields. The new issue is created and attached to this parent in the same operation. (number, optional)
+  - `parent_owner`: Repository owner of the parent issue. Defaults to the value of owner. Only used when method is 'create' and parent_issue_number is provided. (string, optional)
+  - `parent_repo`: Repository name of the parent issue. Defaults to the value of repo. Only used when method is 'create' and parent_issue_number is provided. (string, optional)
   - `repo`: Repository name (string, required)
   - `state`: New state (string, optional)
   - `state_reason`: Reason for the state change. Ignored unless state is changed. (string, optional)

pkg/github/__toolsnaps__/create_issue.snap

@@ -22,6 +22,14 @@
         "minimum": 1,
         "type": "number"
       },
+      "parent_owner": {
+        "description": "Repository owner of the parent issue. Defaults to the value of owner. Only used when parent_issue_number is provided.",
+        "type": "string"
+      },
+      "parent_repo": {
+        "description": "Repository name of the parent issue. Defaults to the value of repo. Only used when parent_issue_number is provided.",
+        "type": "string"
+      },
       "repo": {
         "description": "Repository name",
         "type": "string"

pkg/github/__toolsnaps__/issue_write.snap

@@ -92,10 +92,18 @@
         "type": "string"
       },
       "parent_issue_number": {
-        "description": "Issue number of the parent issue. Only used when method is 'create'. The new issue is created and attached to this parent in the same operation.",
+        "description": "Issue number of the parent issue. Only used when method is 'create' and cannot be combined with issue_fields. The new issue is created and attached to this parent in the same operation.",
         "minimum": 1,
         "type": "number"
       },
+      "parent_owner": {
+        "description": "Repository owner of the parent issue. Defaults to the value of owner. Only used when method is 'create' and parent_issue_number is provided.",
+        "type": "string"
+      },
+      "parent_repo": {
+        "description": "Repository name of the parent issue. Defaults to the value of repo. Only used when method is 'create' and parent_issue_number is provided.",
+        "type": "string"
+      },
       "repo": {
         "description": "Repository name",
         "type": "string"

pkg/github/issues.go

@@ -116,36 +116,31 @@ func getCloseStateReason(stateReason string) IssueClosedStateReason {
 }
 
 // issueFieldWriteMetadataNode queries only the fields needed to resolve a write: the field's
-// node ID, fullDatabaseId (BigInt scalar, returned as string), name, and data type for validation.
+// fullDatabaseId (BigInt scalar, returned as string) plus its name and data type for validation.
 // shurcooL/githubv4 cannot use interface-level fragments at union top-level, so we repeat
 // fullDatabaseId on each concrete type; all four implement IssueFieldCommon.
 type issueFieldWriteMetadataNode struct {
 	TypeName       githubv4.String `graphql:"__typename"`
 	IssueFieldText struct {
-		ID             githubv4.ID
 		FullDatabaseID githubv4.String `graphql:"fullDatabaseId"`
 		Name           githubv4.String
 		DataType       githubv4.String
 	} `graphql:"... on IssueFieldText"`
 	IssueFieldNumber struct {
-		ID             githubv4.ID
 		FullDatabaseID githubv4.String `graphql:"fullDatabaseId"`
 		Name           githubv4.String
 		DataType       githubv4.String
 	} `graphql:"... on IssueFieldNumber"`
 	IssueFieldDate struct {
-		ID             githubv4.ID
 		FullDatabaseID githubv4.String `graphql:"fullDatabaseId"`
 		Name           githubv4.String
 		DataType       githubv4.String
 	} `graphql:"... on IssueFieldDate"`
 	IssueFieldSingleSelect struct {
-		ID             githubv4.ID
 		FullDatabaseID githubv4.String `graphql:"fullDatabaseId"`
 		Name           githubv4.String
 		DataType       githubv4.String
 		Options        []struct {
-			ID             githubv4.ID
 			FullDatabaseID githubv4.String `graphql:"fullDatabaseId"`
 			Name           githubv4.String
 		}
@@ -313,9 +308,33 @@ func resolveIssueRequestFieldValues(ctx context.Context, gqlClient *githubv4.Cli
 		return nil, nil, nil
 	}
 
-	fieldByName, err := fetchIssueFieldWriteMetadata(ctx, gqlClient, owner, repo)
-	if err != nil {
-		return nil, nil, err
+	ctxWithFeatures := ghcontext.WithGraphQLFeatures(ctx, "issue_fields", "repo_issue_fields")
+	var query issueFieldWriteMetadataQuery
+	vars := map[string]any{
+		"owner": githubv4.String(owner),
+		"repo":  githubv4.String(repo),
+	}
+	if err := gqlClient.Query(ctxWithFeatures, &query, vars); err != nil {
+		return nil, nil, fmt.Errorf("failed to query issue fields metadata: %w", err)
+	}
+
+	// Build name → node map, dispatching on concrete type to extract name.
+	fieldByName := make(map[string]issueFieldWriteMetadataNode, len(query.Repository.IssueFields.Nodes))
+	for _, node := range query.Repository.IssueFields.Nodes {
+		var name string
+		switch string(node.TypeName) {
+		case "IssueFieldText":
+			name = string(node.IssueFieldText.Name)
+		case "IssueFieldNumber":
+			name = string(node.IssueFieldNumber.Name)
+		case "IssueFieldDate":
+			name = string(node.IssueFieldDate.Name)
+		case "IssueFieldSingleSelect":
+			name = string(node.IssueFieldSingleSelect.Name)
+		default:
+			continue
+		}
+		fieldByName[strings.ToLower(strings.TrimSpace(name))] = node
 	}
 
 	resolved := make([]*github.IssueRequestFieldValue, 0, len(issueFields))
@@ -382,38 +401,6 @@ func resolveIssueRequestFieldValues(ctx context.Context, gqlClient *githubv4.Cli
 	return resolved, fieldIDsToDelete, nil
 }
 
-func fetchIssueFieldWriteMetadata(ctx context.Context, gqlClient *githubv4.Client, owner, repo string) (map[string]issueFieldWriteMetadataNode, error) {
-	ctxWithFeatures := ghcontext.WithGraphQLFeatures(ctx, "issue_fields", "repo_issue_fields")
-	var query issueFieldWriteMetadataQuery
-	vars := map[string]any{
-		"owner": githubv4.String(owner),
-		"repo":  githubv4.String(repo),
-	}
-	if err := gqlClient.Query(ctxWithFeatures, &query, vars); err != nil {
-		return nil, fmt.Errorf("failed to query issue fields metadata: %w", err)
-	}
-
-	fieldByName := make(map[string]issueFieldWriteMetadataNode, len(query.Repository.IssueFields.Nodes))
-	for _, node := range query.Repository.IssueFields.Nodes {
-		var name string
-		switch string(node.TypeName) {
-		case "IssueFieldText":
-			name = string(node.IssueFieldText.Name)
-		case "IssueFieldNumber":
-			name = string(node.IssueFieldNumber.Name)
-		case "IssueFieldDate":
-			name = string(node.IssueFieldDate.Name)
-		case "IssueFieldSingleSelect":
-			name = string(node.IssueFieldSingleSelect.Name)
-		default:
-			continue
-		}
-		fieldByName[strings.ToLower(strings.TrimSpace(name))] = node
-	}
-
-	return fieldByName, nil
-}
-
 // fetchExistingIssueFieldValues retrieves the current field values for an issue
 // as IssueRequestFieldValue entries, ready to be merged before an update.
 func fetchExistingIssueFieldValues(ctx context.Context, gqlClient *githubv4.Client, owner, repo string, issueNumber int) ([]*github.IssueRequestFieldValue, error) {
@@ -2362,8 +2349,8 @@ var issueWriteFormParams = map[string]struct{}{
 	"_ui_submitted": {},
 }
 
-// parent_issue_number is intentionally omitted because the current form cannot
-// represent it. Calls that supply a parent bypass the form instead of dropping it.
+// Parent issue parameters are intentionally omitted because the current form cannot
+// represent them. Calls that supply a parent bypass the form instead of dropping them.
 
 // issueWriteAwaitingFormResult builds the "awaiting form submission" stub
 // returned when issue_write hands off to the MCP App form. The body is shared
@@ -2443,9 +2430,17 @@ Options are:
 					},
 					"parent_issue_number": {
 						Type:        "number",
-						Description: "Issue number of the parent issue. Only used when method is 'create'. The new issue is created and attached to this parent in the same operation.",
+						Description: "Issue number of the parent issue. Only used when method is 'create' and cannot be combined with issue_fields. The new issue is created and attached to this parent in the same operation.",
 						Minimum:     jsonschema.Ptr(1.0),
 					},
+					"parent_owner": {
+						Type:        "string",
+						Description: "Repository owner of the parent issue. Defaults to the value of owner. Only used when method is 'create' and parent_issue_number is provided.",
+					},
+					"parent_repo": {
+						Type:        "string",
+						Description: "Repository name of the parent issue. Defaults to the value of repo. Only used when method is 'create' and parent_issue_number is provided.",
+					},
 					"title": {
 						Type:        "string",
 						Description: "Issue title",
@@ -2644,12 +2639,26 @@ Options are:
 			if parentProvided && method != "create" {
 				return utils.NewToolResultError("parent_issue_number can only be used with the create method"), nil, nil
 			}
+			parentOwner, err := OptionalParam[string](args, "parent_owner")
+			if err != nil {
+				return utils.NewToolResultError(err.Error()), nil, nil
+			}
+			parentRepo, err := OptionalParam[string](args, "parent_repo")
+			if err != nil {
+				return utils.NewToolResultError(err.Error()), nil, nil
+			}
+			if !parentProvided && (parentOwner != "" || parentRepo != "") {
+				return utils.NewToolResultError("parent_owner and parent_repo can only be used when parent_issue_number is provided"), nil, nil
+			}
 
 			var issueFields []issueWriteFieldInput
 			issueFields, err = optionalIssueWriteFields(args)
 			if err != nil {
 				return utils.NewToolResultError(err.Error()), nil, nil
 			}
+			if parentProvided && len(issueFields) > 0 {
+				return utils.NewToolResultError("issue_fields cannot be used with parent_issue_number"), nil, nil
+			}
 
 			client, err := deps.GetClient(ctx)
 			if err != nil {
@@ -2661,35 +2670,29 @@ Options are:
 				return utils.NewToolResultErrorFromErr("failed to get GraphQL client", err), nil, nil
 			}
 
+			var issueFieldValues []*github.IssueRequestFieldValue
+			var fieldIDsToDelete []int64
+			if len(issueFields) > 0 {
+				issueFieldValues, fieldIDsToDelete, err = resolveIssueRequestFieldValues(ctx, gqlClient, owner, repo, issueFields)
+				if err != nil {
+					return utils.NewToolResultError(fmt.Sprintf("failed to resolve issue_fields: %v", err)), nil, nil
+				}
+			}
+
 			switch method {
 			case "create":
 				if parentProvided {
-					result, err := createIssueWithParent(ctx, client, gqlClient, owner, repo, title, body, assignees, labels, milestoneNum, issueType, issueFields, parentIssueNumber)
+					result, err := createIssueWithParent(ctx, client, gqlClient, owner, repo, title, body, assignees, labels, milestoneNum, issueType, parentIssueNumber, parentOwner, parentRepo)
 					return result, nil, err
 				}
 
-				var issueFieldValues []*github.IssueRequestFieldValue
-				if len(issueFields) > 0 {
-					issueFieldValues, _, err = resolveIssueRequestFieldValues(ctx, gqlClient, owner, repo, issueFields)
-					if err != nil {
-						return utils.NewToolResultError(fmt.Sprintf("failed to resolve issue_fields: %v", err)), nil, nil
-					}
-				}
 				result, err := CreateIssue(ctx, client, owner, repo, title, body, assignees, labels, milestoneNum, issueType, issueFieldValues)
 				return result, nil, err
 			case "update":
 				issueNumber, err := RequiredInt(args, "issue_number")
 				if err != nil {
 					return utils.NewToolResultError(err.Error()), nil, nil
 				}
-				var issueFieldValues []*github.IssueRequestFieldValue
-				var fieldIDsToDelete []int64
-				if len(issueFields) > 0 {
-					issueFieldValues, fieldIDsToDelete, err = resolveIssueRequestFieldValues(ctx, gqlClient, owner, repo, issueFields)
-					if err != nil {
-						return utils.NewToolResultError(fmt.Sprintf("failed to resolve issue_fields: %v", err)), nil, nil
-					}
-				}
 				result, err := UpdateIssue(ctx, client, gqlClient, owner, repo, issueNumber, title, body, assignees, labels, milestoneNum, issueType, issueFieldValues, fieldIDsToDelete, state, stateReason, duplicateOf, UpdateIssueOptions{
 					AssigneesProvided: assigneesProvided,
 					LabelsProvided:    labelsProvided,