fix: remove includeSummary flag to avoid direct dashboards comparison

Author: CountRedClaw

src/builder.ts

@@ -277,8 +277,7 @@ export class PackageVersionBuilder implements IPackageVersionBuilder {
     version?: string,
     packageId?: string,
     operationIds?: OperationId[],
-    includeData: boolean = true,
-    operationsCount?: number,
+    includeData = true,
   ): Promise<ResolvedOperations | null> {
     if (!version) {
       return null
@@ -305,7 +304,6 @@ export class PackageVersionBuilder implements IPackageVersionBuilder {
       packageId,
       operationIds,
       includeData,
-      operationsCount,
     )
 
     // validate for missing operationData
@@ -403,10 +401,8 @@ export class PackageVersionBuilder implements IPackageVersionBuilder {
   async versionResolver(
     version: string,
     packageId: string,
-    includeOperations?: boolean,
-    includeSummary?: boolean,
   ): Promise<VersionCache | null> {
-    const compositeKey = getCompositeKey(packageId, version, String(includeOperations), String(includeSummary))
+    const compositeKey = getCompositeKey(packageId, version)
 
     if (this.canBeResolvedLocally(version, packageId)) {
       return this.currentVersion
@@ -422,7 +418,7 @@ export class PackageVersionBuilder implements IPackageVersionBuilder {
       throw new Error('No versionResolver provided')
     }
 
-    const versionContent = await versionResolver(packageId, version, includeOperations, includeSummary)
+    const versionContent = await versionResolver(packageId, version)
 
     if (!versionContent) {
       this.notifications.push({

src/components/compare/compare.operations.ts

@@ -66,8 +66,8 @@ export async function compareVersionsOperations(
   const { versionResolver } = ctx
 
   // resolve all version with operation hashes
-  const prevVersionData = prev && (await versionResolver(...prev, false, true) ?? { version: prev?.[0], packageId: prev?.[1] })
-  const currVersionData = curr && (await versionResolver(...curr, false, true) ?? { version: curr?.[0], packageId: curr?.[1] })
+  const prevVersionData = prev && (await versionResolver(...prev) ?? { version: prev?.[0], packageId: prev?.[1] })
+  const currVersionData = curr && (await versionResolver(...curr) ?? { version: curr?.[0], packageId: curr?.[1] })
 
   // compare operations of each type
   for (const apiType of getUniqueApiTypesFromVersions(prevVersionData, currVersionData)) {

src/types/internal/apiBuilder.ts

@@ -99,12 +99,7 @@ export interface ApiBuilder<T = any, O = any, M = any> {
 }
 
 // internal
-export type _VersionResolver = (
-  packageId: PackageId,
-  version: VersionId,
-  includeOperations?: boolean,
-  includeSummary?: boolean,
-) => Promise<VersionCache | null>
+export type _VersionResolver = (packageId: PackageId, version: VersionId) => Promise<VersionCache | null>
 export type _VersionReferencesResolver = (packageId: PackageId, version: VersionId) => Promise<BuildConfigRef[]>
 
 export type _ParsedFileResolver = (fileId: string) => Promise<File | null>