From c969f93a232c865ca70a15e415cfb387ad0aaefd Mon Sep 17 00:00:00 2001
From: Dom <dom@rpa-vision-v3.local>
Date: Wed, 6 May 2026 19:27:16 +0200
Subject: [PATCH] fix(replay): self-healing Win+D auto au retry 1
 (verification_failed)
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

Audit project-quality-guardian (2026-05-06) Cas #2 : le mécanisme
qui invoquait gesture_catalog.win_minimize_all (Win+D) en cas
d'échec de grounding a été archivé le 24/04 dans
_archive/dead_code_20260424/core/visual/rpa_integration_manager.py
(_attempt_self_healing_resolution). Le catalogue
agent_chat/gesture_catalog.py:84 reste intact mais orphelin —
aucun caller actif.

Conséquence : quand une fenêtre/popup obstrue la cible, Léa
retente N fois la même action ratée puis pose une pause supervisée,
alors qu'un Win+D ("Afficher le bureau") règle souvent le problème
en 200 ms.

L'audit proposait observe_reason_act.py mais ce module est utilisé
uniquement par /execute/instruction (lui aussi sans client actif,
Cas #10). Le bon point d'insertion dans le pipeline replay actif
est _schedule_retry (replay_engine.py) — la fonction qui construit
la liste d'actions à réinjecter en tête de queue avant chaque retry.

Modification :

Au next_retry == 1 ET reason in ("verification_failed",
"no_screen_change"), insertion en tête de queue de :

  1. Action key_combo {keys: ["super", "d"]} (format reconnu par
     agent_v1/core/executor.py:1151), tagué
     _recovery_gesture: "win_minimize_all" pour audit.
  2. Wait 500 ms pour laisser l'OS terminer l'animation Win+D.
  3. Le retry de l'action originale.

Au retry 2 et au-delà, comportement inchangé (wait 2s + retry).

Tests : 27/27 baseline sprint QW verts.
---
 agent_v0/server_v1/replay_engine.py | 34 ++++++++++++++++++++++++-----
 1 file changed, 29 insertions(+), 5 deletions(-)

diff --git a/agent_v0/server_v1/replay_engine.py b/agent_v0/server_v1/replay_engine.py
index 583ae3078..47a5be132 100644
--- a/agent_v0/server_v1/replay_engine.py
+++ b/agent_v0/server_v1/replay_engine.py
@@ -1432,14 +1432,38 @@ def _schedule_retry(
     # Stratégie de retry selon le numéro
     actions_to_insert = []
 
-    if next_retry == 2:
-        # Retry 2 : injecter un wait de 2s avant l'action
-        wait_action = {
+    if next_retry == 1 and reason in ("verification_failed", "no_screen_change"):
+        # Auto-recovery par gesture : tenter Win+D ("Afficher le bureau")
+        # avant le 1er retry quand l'action n'a rien changé à l'écran.
+        # Hypothèse : une fenêtre/popup non bloquante obstrue la cible.
+        # Cf. gesture_catalog.win_minimize_all (agent_chat/gesture_catalog.py:84)
+        # et audit project-quality-guardian Cas #2 — self-healing par gesture
+        # archivé le 24/04 dans _archive/dead_code_20260424/, reconnecté ici
+        # côté serveur dans le pipeline de retry plutôt que dans la cascade
+        # ORALoop (qui n'est plus sur le chemin actif du replay).
+        actions_to_insert.append({
+            "action_id": f"recovery_win_d_{uuid.uuid4().hex[:6]}",
+            "type": "key_combo",
+            "keys": ["super", "d"],
+            "_recovery_gesture": "win_minimize_all",
+        })
+        # Petit wait après Win+D pour laisser l'OS terminer l'animation
+        actions_to_insert.append({
+            "action_id": f"wait_recovery_{uuid.uuid4().hex[:6]}",
+            "type": "wait",
+            "duration_ms": 500,
+        })
+        logger.info(
+            "Auto-recovery : injection Win+D avant retry %d/%d (raison: %s)",
+            next_retry, max_retries, reason,
+        )
+    elif next_retry == 2:
+        # Retry 2 : injecter un wait de 2s avant l'action (loading possible)
+        actions_to_insert.append({
             "action_id": f"wait_retry_{uuid.uuid4().hex[:6]}",
             "type": "wait",
             "duration_ms": 2000,
-        }
-        actions_to_insert.append(wait_action)
+        })
 
     actions_to_insert.append(retry_action)