@silasfmartins/testhub 1.0.3 → 1.0.5

package/dist/src/playwright/WebActions.d.ts

@@ -172,9 +172,8 @@ export declare class WebActions {
     }>;
     /** Timestamp do último refreshDOM (debounce para evitar múltiplas chamadas consecutivas) */
     private static _lastDOMRefreshTime;
-    /** Guard contra recursão infinita em withRecovery (click_other/navigateTo/switchTo dentro de recovery) */
-    private static _recoveryDepth;
-    private static readonly _maxRecoveryDepth;
+    /** Lock global: quando true, withRecovery executa fn direto sem entrar em recovery */
+    private static _inRecovery;
     /**
      * 🔄 Força atualização/estabilização do DOM antes de localizar/interagir com elementos.
      * Essencial para SPAs (ex.: Salesforce Lightning/Vlocity) que recriam Shadow DOM a cada interação.

package/dist/src/playwright/WebActions.js

@@ -873,9 +873,8 @@ export class WebActions {
     }
     /** Timestamp do último refreshDOM (debounce para evitar múltiplas chamadas consecutivas) */
     static _lastDOMRefreshTime = 0;
-    /** Guard contra recursão infinita em withRecovery (click_other/navigateTo/switchTo dentro de recovery) */
-    static _recoveryDepth = 0;
-    static _maxRecoveryDepth = 1;
+    /** Lock global: quando true, withRecovery executa fn direto sem entrar em recovery */
+    static _inRecovery = false;
     /**
      * 🔄 Força atualização/estabilização do DOM antes de localizar/interagir com elementos.
      * Essencial para SPAs (ex.: Salesforce Lightning/Vlocity) que recriam Shadow DOM a cada interação.
@@ -978,21 +977,15 @@ export class WebActions {
         }
     }
     static async withRecovery(originalElement, actionName, fn, valueForContext) {
-        // 🛡️ Guard contra recursão infinita: recovery actions (click_other, navigateTo, switchTo)
-        // chamam WebActions.click/navigateTo/switchTo que passam por withRecovery novamente.
-        // Se já estiver dentro de recovery, executar direto sem novo ciclo de recovery.
-        if (WebActions._recoveryDepth >= WebActions._maxRecoveryDepth) {
+        // 🛡️ Guard: se já estiver em recovery, executar fn direto sem novo ciclo.
+        // Impede a cadeia click→recovery→click_other→click→recovery→... (stack overflow)
+        if (WebActions._inRecovery) {
             const initialCandidate = await WebActions.resolveCandidateForRecovery(originalElement);
             return await fn(initialCandidate);
         }
-        // 🔄 Refresh DOM antes da primeira tentativa — garantir DOM fresco para SPAs/Shadow DOM
-        // 🛡️ Pular refreshDOM se já estiver em contexto de recovery (evitar overhead desnecessário)
-        if (WebActions._recoveryDepth <= 0) {
-            await WebActions.refreshDOM(150).catch(() => { });
-        }
-        // 🎯 Catalog pre-resolve + lazy auto-refresh (Item 18/20 v2.7.44)
-        // Se o target for texto/label (não XPath explícito), tentar resolver via catálogo local.
-        // Na primeira ação de cada URL, popular o catálogo automaticamente.
+        // 🔄 Refresh DOM antes da primeira tentativa
+        await WebActions.refreshDOM(150).catch(() => { });
+        // 🎯 Catalog pre-resolve + lazy auto-refresh
         if (typeof originalElement === "string") {
             try {
                 WebActions.ensureCatalogExtractor();
@@ -1011,20 +1004,19 @@ export class WebActions {
             }
             catch { }
         }
-        // 🛡️ Flag para rastrear se entramos no bloco de recovery (catch).
-        // Sem essa flag, o finally decrementa _recoveryDepth mesmo em chamadas bem-sucedidas,
-        // fazendo depth ficar negativo e desabilitando o guard de recursão em falhas futuras.
-        let enteredRecovery = false;
+        // 🛡️ Capturar estado do lock ANTES de entrar no try/catch.
+        // Somente quem mudou _inRecovery de false→true pode resetar no finally.
+        const wasAlreadyInRecovery = WebActions._inRecovery;
         try {
             const initialCandidate = await WebActions.resolveCandidateForRecovery(originalElement);
             return await fn(initialCandidate);
         }
         catch (error) {
-            enteredRecovery = true;
-            // 🔄 Refresh DOM imediato após falha — capturar estado mais recente do Shadow DOM
+            // 🔄 Refresh DOM imediato após falha
             await WebActions.refreshDOM(200).catch(() => { });
-            // 🛡️ Incrementar profundidade de recovery para evitar recursão infinita
-            WebActions._recoveryDepth++;
+            // 🛡️ Ativar lock — click_other/navigateTo/switchTo dentro deste recovery
+            // vão cair no guard acima e executar fn direto, sem nova recovery.
+            WebActions._inRecovery = true;
             try {
                 const MIN_CONF = Number(process.env.AUTOCORE_RECOVERY_MIN_CONFIDENCE || "0.7");
                 const maxRetries = Number(process.env.AUTOCORE_RECOVERY_MAX_RETRIES || "8");
@@ -1353,11 +1345,10 @@ export class WebActions {
             throw error;
         }
         finally {
-            // 🛡️ Decrementar profundidade de recovery SOMENTE se entramos no bloco de recovery.
-            // Sem essa condição, chamadas bem-sucedidas decrementam sem ter incrementado,
-            // fazendo _recoveryDepth ficar negativo e desabilitando o guard de recursão.
-            if (enteredRecovery) {
-                WebActions._recoveryDepth--;
+            // 🛡️ Só resetar o lock se ESTE withRecovery foi quem ativou.
+            // Se já estava em recovery antes (wasAlreadyInRecovery), não tocar no lock.
+            if (!wasAlreadyInRecovery) {
+                WebActions._inRecovery = false;
             }
         }
     }

package/package.json

@@ -1,7 +1,7 @@
 {
 	"name": "@silasfmartins/testhub",
 	"description": "Biblioteca de utilitários para automação de testes",
-	"version": "1.0.3",
+	"version": "1.0.5",
 	"author": "Silas Martins Feliciano da Silva <silas.martins2041@gmail.com>",
 	"type": "module",
 	"main": "dist/index.js",