From 051ae362704c20d8e801a219a3e3a75b42c172f3 Mon Sep 17 00:00:00 2001
From: Peter Hedenskog <peter@soulgalore.com>
Date: Tue, 25 Jun 2024 07:53:48 +0200
Subject: [PATCH] Add CLI params to turn off LCP/LS screenshots. (#4201)

They already exists in Browsertime.

https://github.com/sitespeedio/sitespeed.io/issues/4189
---
 lib/cli/cli.js | 18 ++++++++++++++++++
 1 file changed, 18 insertions(+)

diff --git a/lib/cli/cli.js b/lib/cli/cli.js
index 922574846..faed4b99b 100644
--- a/lib/cli/cli.js
+++ b/lib/cli/cli.js
@@ -1466,6 +1466,22 @@ export async function parseCommandLine() {
     })
 
     /** Screenshot */
+    .option('browsertime.screenshotLCP', {
+      alias: 'screenshotLCP',
+      type: 'boolean',
+      default: false,
+      describe:
+        'Save one screenshot per iteration that shows the largest contentful paint element (if the browser supports LCP).',
+      group: 'Screenshot'
+    })
+    .option('browsertime.screenshotLS', {
+      alias: 'screenshotLS',
+      type: 'boolean',
+      default: false,
+      describe:
+        'Save one screenshot per iteration that shows the layout shift elements (if the browser supports layout shift).',
+      group: 'Screenshot'
+    })
     .option('browsertime.screenshot', {
       type: 'boolean',
       describe: 'Set to false to disable screenshots',
@@ -2042,6 +2058,8 @@ export async function parseCommandLine() {
     .alias('help', 'h')
     .config(config)
     .hide('disableAPI')
+    .hide('browsertime.screenshotLCP')
+    .hide('browsertime.screenshotLS')
     .alias('version', 'V')
     .version(version)
     .coerce('budget', function (argument) {