add logging

Author: aitchiss

src/index.js

@@ -11,6 +11,7 @@ dotenv.config();
 const puppeteerCacheDir = join(homedir(), '.cache', 'puppeteer');
 
 const onPreBuild = async ({ utils } = {}) => {
+  console.log('onPreBuild handler called');
   console.log('Restoring Lighthouse cache...');
   // Puppeteer relies on a global cache since v19.x, which otherwise would not be persisted in Netlify builds
   await utils?.cache.restore(puppeteerCacheDir);
@@ -24,6 +25,7 @@ const persistCache = async ({ utils } = {}) => {
 };
 
 export default function lighthousePlugin(inputs) {
+  console.log('lighthouse plugin function called with inputs:', JSON.stringify(inputs, null, 2));
   // Run onSuccess by default, unless inputs specify we should fail_deploy_on_score_thresholds
   const defaultEvent =
     inputs?.fail_deploy_on_score_thresholds === 'true'
@@ -33,8 +35,12 @@ export default function lighthousePlugin(inputs) {
   if (defaultEvent === 'onSuccess') {
     return {
       onPreBuild,
-      onPostBuild: ({ utils }) => persistCache({ utils }),
+      onPostBuild: ({ utils }) => {
+        console.log('onPostBuild handler called (onSuccess mode)');
+        return persistCache({ utils });
+      },
       onSuccess: async ({ constants, utils, inputs } = {}) => {
+        console.log('onSuccess handler called');
         // Mock the required `utils` functions if running locally
         const { failPlugin, show } = getUtils({ utils });
 
@@ -51,6 +57,7 @@ export default function lighthousePlugin(inputs) {
     return {
       onPreBuild,
       onPostBuild: async ({ constants, utils, inputs } = {}) => {
+        console.log('onPostBuild handler called (onPostBuild mode)');
         // Mock the required `utils` functions if running locally
         const { failBuild, show } = getUtils({ utils });
 

src/lib/run-audit-with-server/index.js

@@ -16,7 +16,7 @@ const runAuditWithServer = async ({
   try {
     const { server } = getServer({ serveDir: serveDir, auditUrl: url });
 
-    const browserPath = await getBrowserPath();
+    const browserPath = getBrowserPath();
 
     const { error, results } = await new Promise((resolve) => {
       const instance = server.listen(async () => {

src/lib/run-audit-with-url/index.js

@@ -3,7 +3,7 @@ import { runLighthouse, getBrowserPath } from '../../run-lighthouse.js';
 
 const runAuditWithUrl = async ({ path = '', url, thresholds, settings }) => {
   try {
-    const browserPath = await getBrowserPath();
+    const browserPath = getBrowserPath();
 
     const getResults = async () => {
       const fullPath = path ? `${url}/${path}` : url;

src/run-lighthouse.js

@@ -6,7 +6,7 @@ import chromeLauncher from 'chrome-launcher';
 // Cache the browser path to avoid multiple launches
 let cachedBrowserPath;
 
-export const getBrowserPath = async () => {
+export const getBrowserPath = () => {
   if (cachedBrowserPath) {
     return cachedBrowserPath;
   }