Set CI performance budget to 70 and clean up configs.

Added a `ci.budget` property with a value of 70 to both mobile and desktop configs. Removed redundant `--budget 30` flags from the GitHub workflow and improved general readability by cleaning up comments and unused settings. This ensures consistency and simplifies configuration management.

Author: sandroguras

.github/workflows/unlighouse.yml

@@ -36,7 +36,6 @@ jobs:
           unlighthouse-ci \
             --site http://localhost:4173 \
             --config-file mobile-v1.config.ts
-            --budget 30 \
             --reporter lighthouseServer \
             --lhci-host https://lhci-server-crimson-bush-1151.fly.dev \
             --lhci-build-token ${{ secrets.LHCI_BUILD_TOKEN }}
@@ -46,7 +45,6 @@ jobs:
           unlighthouse-ci \
             --site http://localhost:4173 \
             --config-file desktop-v1.config.ts
-            --budget 30 \
             --reporter lighthouseServer \
             --lhci-host https://lhci-server-crimson-bush-1151.fly.dev \
             --lhci-build-token ${{ secrets.LHCI_BUILD_TOKEN }}

desktop-v1.config.ts

@@ -1,6 +1,11 @@
 // This is the default that always gets used unless a different config file is specified
 export default {
 
+	// Maximum acceptable performance budget score for CI checks
+	ci: {
+		budget: 70
+	},
+
 	// // Increase accuracy by only allowing 1 page to be scanned at a time
 	// puppeteerClusterOptions: {
 	//   // only run 1 worker at a time
@@ -19,15 +24,15 @@ export default {
 		throttle: false,
 	//   //dynamicSampling: false, // or any number, default is 5
 
-	//   // exclude specific routes
-	//   // exclude: [
-	//     //   '/.*?pdf',
-	//     //   '.*/amp',
-	//     //   'en-*',
-	//     // ],
+		//   // exclude specific routes
+		//   // exclude: [
+		//     //   '/.*?pdf',
+		//     //   '.*/amp',
+		//     //   'en-*',
+		//     // ],
 
-	//     // run lighthouse for each URL 3 times
-		samples: 1,
+		//     // run lighthouse for each URL 3 times
+		//     // samples: 3,
 	},
 
 	////////////////////////////////////////////////////////////////////////////

mobile-v1.config.ts

@@ -1,6 +1,11 @@
 // This is the default that always gets used unless a different config file is specified
 export default {
 
+	// Maximum acceptable performance budget score for CI checks
+	ci: {
+		budget: 70
+	},
+
 	// // Increase accuracy by only allowing 1 page to be scanned at a time
 	// puppeteerClusterOptions: {
 	//   // only run 1 worker at a time
@@ -36,18 +41,18 @@ export default {
 	////////////////////////////////////////////////////////////////////////////
 	scanner: {
 		device: 'mobile',
-		throttle: true,
+		throttle: true
 
-		// exclude specific routes
-		// exclude: [
-		//   '/.*?pdf',
-		//   '.*/amp',
-		//   'en-*',
-		// ],
+		//   // exclude specific routes
+		//   // exclude: [
+		//     //   '/.*?pdf',
+		//     //   '.*/amp',
+		//     //   'en-*',
+		//     // ],
 
-		// run lighthouse for each URL 3 times
-		samples: 1,
-	},
+		//     // run lighthouse for each URL 3 times
+		//     // samples: 3,
+	}
 
 	////////////////////////////////////////////////////////////////////////////
 	// Emulate mobile with devtools settings:
@@ -72,6 +77,5 @@ export default {
 	//   throttlingMethod: 'devtools',
 	// },
 
-
 	// debug: true,
-}
+};