retry

Author: aitchiss

src/index.js

@@ -10,9 +10,9 @@ dotenv.config();
 
 const puppeteerCacheDir = join(homedir(), '.cache', 'puppeteer');
 
-// Guard to prevent multiple executions
-let hasRunOnSuccess = false;
-let hasRunOnPostBuild = false;
+// Global guard to prevent multiple executions across all plugin instances
+global.lighthouseHasRunOnSuccess = global.lighthouseHasRunOnSuccess || false;
+global.lighthouseHasRunOnPostBuild = global.lighthouseHasRunOnPostBuild || false;
 
 const onPreBuild = async ({ utils } = {}) => {
   console.log('onPreBuild handler called');
@@ -29,7 +29,7 @@ const persistCache = async ({ utils } = {}) => {
 };
 
 export default function lighthousePlugin(inputs) {
-  console.log('lighthouse plugin function called with inputs:', JSON.stringify(inputs, null, 2));
+  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'
@@ -46,11 +46,11 @@ export default function lighthousePlugin(inputs) {
       onSuccess: async ({ constants, utils, inputs } = {}) => {
         console.log('onSuccess handler called');
 
-        if (hasRunOnSuccess) {
-          console.log('onSuccess already executed, skipping duplicate call');
+        if (global.lighthouseHasRunOnSuccess) {
+          console.log('⚠️ onSuccess already executed, skipping duplicate call');
           return;
         }
-        hasRunOnSuccess = true;
+        global.lighthouseHasRunOnSuccess = true;
 
         // Mock the required `utils` functions if running locally
         const { failPlugin, show } = getUtils({ utils });
@@ -70,11 +70,11 @@ export default function lighthousePlugin(inputs) {
       onPostBuild: async ({ constants, utils, inputs } = {}) => {
         console.log('onPostBuild handler called (onPostBuild mode)');
 
-        if (hasRunOnPostBuild) {
-          console.log('onPostBuild already executed, skipping duplicate call');
+        if (global.lighthouseHasRunOnPostBuild) {
+          console.log('⚠️ onPostBuild already executed, skipping duplicate call');
           return;
         }
-        hasRunOnPostBuild = true;
+        global.lighthouseHasRunOnPostBuild = true;
 
         // Mock the required `utils` functions if running locally
         const { failBuild, show } = getUtils({ utils });