perf(cli): move SKIPPABLE_AUDITS definition to prompts.js and lazy-load inquirer

Author: Hugoer

lib/lab.js

@@ -4,24 +4,9 @@ const lighthouse = require('lighthouse').default;
 
 const logger = require('./logger');
 const { resolveProfileSettings } = require('./profiles');
+const { SKIPPABLE_AUDITS } = require('./prompts');
 const { ensureCommandDir, buildFilename } = require('./utils');
 
-// All audits available for skipping in interactive prompt
-const SKIPPABLE_AUDITS = [
-    { id: 'full-page-screenshot', label: 'Full-page screenshot capture', defaultSkip: true },
-    { id: 'screenshot-thumbnails', label: 'Filmstrip thumbnail screenshots', defaultSkip: true },
-    { id: 'final-screenshot', label: 'Final rendered state screenshot', defaultSkip: true },
-    { id: 'valid-source-maps', label: 'Source map validation check', defaultSkip: true },
-    { id: 'script-treemap-data', label: 'JS treemap/bundle data (often the largest audit)', defaultSkip: false },
-    { id: 'network-requests', label: 'Full network request log', defaultSkip: false },
-    { id: 'main-thread-tasks', label: 'Detailed main-thread task breakdown', defaultSkip: false },
-    { id: 'third-party-summary', label: 'Third-party resource analysis', defaultSkip: false },
-    { id: 'layout-shifts', label: 'Individual CLS shift elements', defaultSkip: false },
-    { id: 'long-tasks', label: 'Long main-thread tasks list', defaultSkip: false },
-    { id: 'bf-cache', label: 'Back/forward cache eligibility', defaultSkip: false },
-    { id: 'resource-summary', label: 'Resource count and size summary', defaultSkip: false },
-];
-
 // Audits skipped by default — derived from SKIPPABLE_AUDITS to avoid duplication
 const DEFAULT_SKIP_AUDITS = SKIPPABLE_AUDITS.filter((a) => a.defaultSkip).map((a) => a.id);
 
@@ -91,4 +76,4 @@ async function runLab(url, labOptions = {}) {
     }
 }
 
-module.exports = { runLab, CHROME_FLAGS, DEFAULT_SKIP_AUDITS, SKIPPABLE_AUDITS };
+module.exports = { runLab, CHROME_FLAGS, DEFAULT_SKIP_AUDITS };

lib/lab.test.js

@@ -1,6 +1,7 @@
 import { describe, it, expect } from 'vitest';
 
-const { DEFAULT_SKIP_AUDITS, SKIPPABLE_AUDITS } = require('./lab');
+const { DEFAULT_SKIP_AUDITS } = require('./lab');
+const { SKIPPABLE_AUDITS } = require('./prompts');
 
 describe('DEFAULT_SKIP_AUDITS', () => {
     it('should contain the expected default audit IDs', () => {

lib/prompts.js

@@ -1,10 +1,34 @@
 const fs = require('fs');
-const inquirer = require('inquirer');
 
-const { SKIPPABLE_AUDITS } = require('./lab');
 const logger = require('./logger');
 const { PROFILES, NETWORK_PRESETS, DEVICE_PRESETS } = require('./profiles');
 
+// Lazy-load inquirer (~230ms cold require) — only needed when prompts actually run
+let _inquirer;
+function getInquirer() {
+    if (!_inquirer) {
+        _inquirer = require('inquirer');
+    }
+    return _inquirer;
+}
+const inquirer = new Proxy({}, { get: (_, prop) => getInquirer()[prop] });
+
+// All audits available for skipping in interactive prompt
+const SKIPPABLE_AUDITS = [
+    { id: 'full-page-screenshot', label: 'Full-page screenshot capture', defaultSkip: true },
+    { id: 'screenshot-thumbnails', label: 'Filmstrip thumbnail screenshots', defaultSkip: true },
+    { id: 'final-screenshot', label: 'Final rendered state screenshot', defaultSkip: true },
+    { id: 'valid-source-maps', label: 'Source map validation check', defaultSkip: true },
+    { id: 'script-treemap-data', label: 'JS treemap/bundle data (often the largest audit)', defaultSkip: false },
+    { id: 'network-requests', label: 'Full network request log', defaultSkip: false },
+    { id: 'main-thread-tasks', label: 'Detailed main-thread task breakdown', defaultSkip: false },
+    { id: 'third-party-summary', label: 'Third-party resource analysis', defaultSkip: false },
+    { id: 'layout-shifts', label: 'Individual CLS shift elements', defaultSkip: false },
+    { id: 'long-tasks', label: 'Long main-thread tasks list', defaultSkip: false },
+    { id: 'bf-cache', label: 'Back/forward cache eligibility', defaultSkip: false },
+    { id: 'resource-summary', label: 'Resource count and size summary', defaultSkip: false },
+];
+
 const DEFAULT_CONCURRENCY = 5;
 
 function assertTTY() {
@@ -627,6 +651,7 @@ async function promptLinks(url) {
 
 module.exports = {
     DEFAULT_CONCURRENCY,
+    SKIPPABLE_AUDITS,
     normalizeUrls,
     promptForSubcommand,
     promptLab,