Merge pull request 'fix(vinpin): warm on Fiat+Dialogys anchor (no starvation)' (#238) from dev into main

apps/api/src/integrations/vinpin/vinpin-driver.service.ts

@@ -521,18 +521,20 @@ export class VinpinDriverService implements OnModuleDestroy {
       // and re-binds by probing, so this is only a fast-path hint).
       await this.bindTaskbarCoords(page);
 
-      if (!(fiatUp && okR && okD)) {
-        // Require the FULL warm session (all three catalog windows). A PARTIAL
-        // session — e.g. Rpartstore DOWN so only Fiat + Dialogys open — is proven
-        // UNSTABLE: the missing window's launch-error modal + the re-warm cycles
-        // keep knocking the Fiat window off its VIN panel, so warm Fiat decodes
-        // thrash to the budget. When we can't fully warm, DON'T claim warm — the
-        // reliable cold per-decode path serves every brand (Fiat ePER + Renault
-        // Rpartstore/Dialogys, all proven). The daemon backs off and retries, so
-        // warm resumes automatically once Rpartstore recovers. Clean the seat (log
-        // out) first so the next attempt isn't sabotaged by this one's dirty desktop.
+      // Claim warm when the two catalogs needed to serve BOTH brands are up: Fiat
+      // ePER (all Fiat decodes) AND Dialogys (the reliable Renault catalog — the
+      // Rpartstore-DOWN cooldown already routes Renault straight to Dialogys, so
+      // Rpartstore is an optional bonus window, not a warm requirement). Requiring
+      // ALL THREE would never warm during a (prolonged) Rpartstore outage, and the
+      // daemon would then re-attempt warm-up every backoff cycle — each ~2min
+      // attempt holds the single-seat mutex and STARVES real decodes. With Fiat +
+      // Dialogys as the anchor, warm is claimed once and HELD (no re-warm loop, no
+      // starvation), and the Fiat-foreground fix (dropped the Dialogys-shared Поиск
+      // token) means warm Fiat raises the correct window. Rpartstore joins the warm
+      // session automatically once it recovers.
+      if (!(fiatUp && okD)) {
         this.logger.warn(
-          `warm-up incomplete (fiat=${fiatUp}, rpartstore=${okR}, dialogys=${okD}) — not claiming warm; cold path stays in effect until all three open`,
+          `warm-up incomplete (fiat=${fiatUp}, rpartstore=${okR}, dialogys=${okD}) — need Fiat+Dialogys; not claiming warm, cold path stays in effect`,
         );
         await this.cleanTeardown();
         await this.close();