Don't use Issue.Repository

JoannaaKL · JoannaaKL · commit 34e7f143b09c · 2025-11-07T13:41:18.000+01:00
diff --git a/pkg/github/issues.go b/pkg/github/issues.go
@@ -326,8 +326,8 @@ func GetIssue(ctx context.Context, client *github.Client, gqlClient *githubv4.Cl
 	}
 
 	if flags.LockdownMode {
-		if issue.User != nil && issue.Repository != nil && issue.Repository.Owner != nil && issue.Repository.Name != nil {
-			shouldRemoveContent, err := lockdown.ShouldRemoveContent(ctx, gqlClient, *issue.User.Login, *issue.Repository.Owner.Login, *issue.Repository.Name)
+		if issue.User != nil {
+			shouldRemoveContent, err := lockdown.ShouldRemoveContent(ctx, gqlClient, *issue.User.Login, owner, repo)
 			if err != nil {
 				return mcp.NewToolResultError(fmt.Sprintf("failed to check lockdown mode: %v", err)), nil
 			}

Original file line number	Diff line number	Diff line change
`@@ -326,8 +326,8 @@ func GetIssue(ctx context.Context, client github.Client, gqlClient githubv4.Cl`
`326`	`326`	`}`
`327`	`327`
`328`	`328`	`if flags.LockdownMode {`
`329`		`- if issue.User != nil && issue.Repository != nil && issue.Repository.Owner != nil && issue.Repository.Name != nil {`
`330`		`- shouldRemoveContent, err := lockdown.ShouldRemoveContent(ctx, gqlClient, issue.User.Login, issue.Repository.Owner.Login, *issue.Repository.Name)`
	`329`	`+ if issue.User != nil {`
	`330`	`+ shouldRemoveContent, err := lockdown.ShouldRemoveContent(ctx, gqlClient, *issue.User.Login, owner, repo)`
`331`	`331`	`if err != nil {`
`332`	`332`	`return mcp.NewToolResultError(fmt.Sprintf("failed to check lockdown mode: %v", err)), nil`
`333`	`333`	`}`