Remove repositoryDiffs (#12205)

cmd/frontend/graphqlbackend/campaigns.go

@@ -170,7 +170,6 @@ type CampaignResolver interface {
 	Changesets(ctx context.Context, args *ListChangesetsArgs) (ChangesetsConnectionResolver, error)
 	OpenChangesets(ctx context.Context) (ChangesetsConnectionResolver, error)
 	ChangesetCountsOverTime(ctx context.Context, args *ChangesetCountsArgs) ([]ChangesetCountsResolver, error)
-	RepositoryDiffs(ctx context.Context, args *graphqlutil.ConnectionArgs) (RepositoryComparisonConnectionResolver, error)
 	Status(context.Context) (BackgroundProcessStatus, error)
 	ClosedAt() *DateTime
 	DiffStat(ctx context.Context) (*DiffStat, error)

cmd/frontend/graphqlbackend/repository_comparison.go

@@ -20,12 +20,6 @@ import (
 	"github.com/sourcegraph/sourcegraph/internal/vcs/git"
 )
 
-type RepositoryComparisonConnectionResolver interface {
-	Nodes(ctx context.Context) ([]*RepositoryComparisonResolver, error)
-	TotalCount(ctx context.Context) (int32, error)
-	PageInfo(ctx context.Context) (*graphqlutil.PageInfo, error)
-}
-
 type RepositoryComparisonInput struct {
 	Base         *string
 	Head         *string

cmd/frontend/graphqlbackend/schema.graphql

@@ -620,18 +620,6 @@ input UserPermission {
     permission: RepositoryPermission = READ
 }
 
-# A paginated list of repository diffs committed to git.
-type RepositoryComparisonConnection {
-    # A list of repository diffs committed to git.
-    nodes: [RepositoryComparison!]!
-
-    # The total number of repository diffs in the connection.
-    totalCount: Int!
-
-    # Pagination information.
-    pageInfo: PageInfo!
-}
-
 # The state of a background process.
 enum BackgroundProcessState {
     # The background process is currently processing items.
@@ -694,10 +682,6 @@ type Campaign implements Node {
     # The date and time when the campaign was updated.
     updatedAt: DateTime!
 
-    # The combined diff of all changesets (that already exist on the code host) across all affected
-    # repositories.
-    repositoryDiffs(first: Int): RepositoryComparisonConnection!
-
     # The changesets in this campaign that already exist on the code host.
     changesets(
         first: Int

enterprise/internal/campaigns/resolvers/campaigns.go

@@ -2,7 +2,6 @@ package resolvers
 
 import (
 	"context"
-	"fmt"
 	"path"
 	"sync"
 	"time"
@@ -207,21 +206,6 @@ func (r *campaignResolver) ChangesetCountsOverTime(
 	return resolvers, nil
 }
 
-func (r *campaignResolver) RepositoryDiffs(
-	ctx context.Context,
-	args *graphqlutil.ConnectionArgs,
-) (graphqlbackend.RepositoryComparisonConnectionResolver, error) {
-	changesetsConnection := &changesetsConnectionResolver{
-		store: r.store,
-		opts: ee.ListChangesetsOpts{
-			CampaignID: r.Campaign.ID,
-			Limit:      int(args.GetFirst()),
-		},
-		optsSafe: true,
-	}
-	return &changesetDiffsConnectionResolver{changesetsConnection}, nil
-}
-
 func (r *campaignResolver) DiffStat(ctx context.Context) (*graphqlbackend.DiffStat, error) {
 	changesetsConnection := &changesetsConnectionResolver{
 		store: r.store,
@@ -260,36 +244,3 @@ func (r *campaignResolver) Status(ctx context.Context) (graphqlbackend.Backgroun
 	// 🚨 SECURITY: GetCampaignStatus checks whether current user is authorized.
 	return svc.GetCampaignStatus(ctx, r.Campaign)
 }
-
-type changesetDiffsConnectionResolver struct {
-	*changesetsConnectionResolver
-}
-
-func (r *changesetDiffsConnectionResolver) Nodes(ctx context.Context) ([]*graphqlbackend.RepositoryComparisonResolver, error) {
-	changesets, err := r.changesetsConnectionResolver.Nodes(ctx)
-	if err != nil {
-		return nil, err
-	}
-
-	resolvers := make([]*graphqlbackend.RepositoryComparisonResolver, 0, len(changesets))
-	for _, c := range changesets {
-		switch c := c.(type) {
-		case *changesetResolver:
-			comp, err := c.Diff(ctx)
-			if err != nil {
-				return nil, err
-			}
-
-			if comp != nil {
-				resolvers = append(resolvers, comp)
-			}
-		case *hiddenChangesetResolver:
-			// Do not include hidden changesets in diff and diffstats
-			continue
-		default:
-			return nil, fmt.Errorf("changesetResolver has unknown type: %T", c)
-		}
-
-	}
-	return resolvers, nil
-}