diff --git a/packages/playwright/src/runner/testGroups.ts b/packages/playwright/src/runner/testGroups.ts
index a3a8af14f3..cda065dd6f 100644
--- a/packages/playwright/src/runner/testGroups.ts
+++ b/packages/playwright/src/runner/testGroups.ts
@@ -14,7 +14,7 @@
  * limitations under the License.
  */
 
-import { JSONReport, JSONReportSuite, JSONReportTestResult } from 'packages/playwright-test/reporter';
+import type { JSONReport, JSONReportSuite } from 'packages/playwright-test/reporter';
 import type { Suite, TestCase } from '../common/test';
 
 export type TestGroup = {
@@ -29,7 +29,7 @@ type testDetails = {
   id: string,
   name: string,
   duration: number
-}
+};
 
 export function createTestGroups(projectSuite: Suite, workers: number): TestGroup[] {
   // This function groups tests that can be run together.
@@ -180,23 +180,23 @@ export function filterForShardFromTimingFile(timingFile: JSONReport, shard: { to
   recordedTests.forEach(group => result.add(group));
 
   // If not all test groups have recorded timing information, filter the remaining groups based on shard distribution
-  if (testDetails.length != testGroups.length) {
+  if (testDetails.length !== testGroups.length) {
     const allTestIds = testDetails.map(shardDetails => shardDetails.id);
     const UnrecordedTests = testGroups.filter(group => !allTestIds.includes(group.tests[0].id));
 
     let shardableTotal = 0;
     for (const group of UnrecordedTests)
       shardableTotal += group.tests.length;
-  
+
     // Each shard gets some tests.
     const shardSize = Math.floor(shardableTotal / shard.total);
     // First few shards get one more test each.
     const extraOne = shardableTotal - shardSize * shard.total;
-  
+
     const currentShard = shard.current - 1; // Make it zero-based for calculations.
     const from = shardSize * currentShard + Math.min(extraOne, currentShard);
     const to = from + shardSize + (currentShard < extraOne ? 1 : 0);
-  
+
     let current = 0;
     for (const group of UnrecordedTests) {
       // Any test group goes to the shard that contains the first test of this group.
@@ -211,7 +211,7 @@ export function filterForShardFromTimingFile(timingFile: JSONReport, shard: { to
 
 // Extracts test details from a JSON report and returns an array of test details objects.
 function getAllTestDetails(report: JSONReport) {
-  const allTestDetails:testDetails[] = [];
+  const allTestDetails: testDetails[] = [];
   report.suites.forEach((suite: JSONReportSuite) => {
     getTestDetails(suite, allTestDetails);
   });
@@ -222,9 +222,9 @@ function getAllTestDetails(report: JSONReport) {
 // Recursively traverses through test suites in a JSON report and extracts test details.
 function getTestDetails(suite: JSONReportSuite, allTestDetails: testDetails[]) {
   if (suite.specs) {
-    suite.specs.forEach((spec) => {
-      spec.tests.forEach((test) => {
-        test.results.forEach((result) => {
+    suite.specs.forEach(spec => {
+      spec.tests.forEach(test => {
+        test.results.forEach(result => {
           allTestDetails.push({
             id: spec.id,
             name: spec.title,
@@ -236,7 +236,7 @@ function getTestDetails(suite: JSONReportSuite, allTestDetails: testDetails[]) {
   }
 
   if (suite.suites) {
-    suite.suites.forEach((subSuite) => {
+    suite.suites.forEach(subSuite => {
       getTestDetails(subSuite, allTestDetails);
     });
   }
@@ -245,13 +245,13 @@ function getTestDetails(suite: JSONReportSuite, allTestDetails: testDetails[]) {
 function mapTestDetailsToShards(testDetails: testDetails[], shard: {total: number, current: number}) {
   testDetails.sort((a, b) => b.duration - a.duration);
 
-  const result:testDetails[][] = Array.from({ length: shard.total }, () => []);
+  const result: testDetails[][] = Array.from({ length: shard.total }, () => []);
   const sums = Array(shard.total).fill(0);
 
   // Distribute tests to shards based on their durations
-  for (let testDetailsObj of testDetails) {
+  for (const testDetailsObj of testDetails) {
     // Find the shard with the smallest total duration and assign the test to that shard
-    let minIndex = sums.indexOf(Math.min(...sums));
+    const minIndex = sums.indexOf(Math.min(...sums));
     result[minIndex].push(testDetailsObj);
     sums[minIndex] += testDetailsObj.duration;
   }