include browser IDs in build info and snapshot fetching

Author: tech-sushant

src/tools/percy-change.ts

@@ -25,7 +25,7 @@ export async function fetchPercyChanges(
   });
 
   // Get build info (noBuilds, isFirstBuild, lastBuildId)
-  const { noBuilds, isFirstBuild, lastBuildId, orgId } =
+  const { noBuilds, isFirstBuild, lastBuildId, orgId, browserIds } =
     await getPercyBuildCount(percyToken);
 
   if (noBuilds) {
@@ -55,6 +55,7 @@ export async function fetchPercyChanges(
     lastBuildId,
     config,
     orgId,
+    browserIds,
   );
   logger.info(
     `Fetched ${snapshotIds.length} snapshot IDs for build: ${lastBuildId} as ${snapshotIds.join(", ")}`,

src/tools/review-agent-utils/build-counts.ts

@@ -20,20 +20,24 @@ export async function getPercyBuildCount(percyToken: string) {
   let isFirstBuild = false;
   let lastBuildId: string | undefined;
   let orgId: string | undefined;
+  let browserIds: string[] = [];
 
   if (builds.length === 0) {
     return {
       noBuilds: true,
       isFirstBuild: false,
       lastBuildId: undefined,
       orgId,
+      browserIds: [],
     };
-  } else if (builds.length === 1) {
-    isFirstBuild = true;
-    lastBuildId = builds[0].id;
   } else {
-    isFirstBuild = false;
+    isFirstBuild = builds.length === 1;
     lastBuildId = builds[0].id;
+
+    browserIds =
+      builds[0]?.relationships?.browsers?.data
+        ?.map((b: any) => b.id)
+        ?.filter((id: any) => typeof id === "string") ?? [];
   }
 
   // Extract orgId from the `included` projects block
@@ -42,5 +46,5 @@ export async function getPercyBuildCount(percyToken: string) {
     orgId = project.relationships.organization.data.id;
   }
 
-  return { noBuilds: false, isFirstBuild, lastBuildId, orgId };
+  return { noBuilds: false, isFirstBuild, lastBuildId, orgId, browserIds };
 }

src/tools/review-agent-utils/percy-diffs.ts

@@ -11,7 +11,7 @@ export async function getPercySnapshotDiff(
   percyToken: string,
 ): Promise<PercySnapshotDiff[]> {
   const apiUrl = `https://percy.io/api/v1/snapshots/${snapshotId}`;
-  
+
   const response = await fetch(apiUrl, {
     headers: {
       Authorization: `Token token=${percyToken}`,

src/tools/review-agent-utils/percy-snapshots.ts

@@ -7,6 +7,7 @@ export async function getChangedPercySnapshotIds(
   buildId: string,
   config: BrowserStackConfig,
   orgId: string | undefined,
+  browserIds: string[],
 ): Promise<string[]> {
 
   if (!buildId || !orgId) {
@@ -19,10 +20,10 @@ export async function getChangedPercySnapshotIds(
     buildId,
     orgId,
     category: ["changed"],
-    subcategories: ["unreviewed","approved","changes_requested"],
+    subcategories: ["unreviewed", "approved", "changes_requested"],
     groupSnapshotsBy: "similar_diff",
-    browserIds: ["63", "64", "69", "70", "71"],
-    widths: ["375","1280","1920"],
+    browserIds,
+    widths: ["375", "1280", "1920"],
   });
 
   const authString = getBrowserStackAuth(config);
@@ -77,26 +78,29 @@ export function constructPercyBuildItemsUrl({
 
   if (category && category.length > 0) {
     category.forEach((cat) =>
-      url.searchParams.append("filter[category][]", sanitizeUrlParam(cat))
+      url.searchParams.append("filter[category][]", sanitizeUrlParam(cat)),
     );
   }
   if (subcategories && subcategories.length > 0) {
     subcategories.forEach((sub) =>
-      url.searchParams.append("filter[subcategories][]", sanitizeUrlParam(sub))
+      url.searchParams.append("filter[subcategories][]", sanitizeUrlParam(sub)),
     );
   }
   if (browserIds && browserIds.length > 0) {
     browserIds.forEach((id) =>
-      url.searchParams.append("filter[browser_ids][]", sanitizeUrlParam(id))
+      url.searchParams.append("filter[browser_ids][]", sanitizeUrlParam(id)),
     );
   }
   if (widths && widths.length > 0) {
     widths.forEach((w) =>
-      url.searchParams.append("filter[widths][]", sanitizeUrlParam(w))
+      url.searchParams.append("filter[widths][]", sanitizeUrlParam(w)),
     );
   }
   if (groupSnapshotsBy) {
-    url.searchParams.set("filter[group_snapshots_by]", sanitizeUrlParam(groupSnapshotsBy));
+    url.searchParams.set(
+      "filter[group_snapshots_by]",
+      sanitizeUrlParam(groupSnapshotsBy),
+    );
   }
   return url.toString();
 }