fix(perf): bump build URL + preserve perf.csv on PerfSampler.End failure

  Bump cloud_build_url default to pr-22430-3a0b8d5, which carries the
  PerfSampler.PercentWorst guard against tiny sample windows
  (decentraland/unity-explorer#8826).

  Improve AttachPerf diagnostics: log ex.ToString() instead of just
  ex.Message so AltTester's wrapped TargetInvocationException reveals the
  real Player-side stack in the Allure report. Best-effort attach
  perf.csv on the End-failure path — PerfSampler.End disposes the CSV
  before WriteSummary runs, so a summary-side crash still leaves a
  complete CSV on disk that can be inspected post-mortem.

Author: popuz

.github/workflows/runner-validation-windows.yml

@@ -52,7 +52,7 @@ on:
         # at every severity), AND the TEMP-gut that bypasses MinimumSpecsScreen modal
         # so the T4 runner (4 vCPU - fails MinimumSpecsGuard) doesn't hang awaiting
         # a human click. Revert to '' to fall back to "latest non-prerelease release".
-        default: 'https://explorer-artifacts.decentraland.org/@dcl/unity-explorer/branch/feat/test-automation/autopilot-alttester/pr-22429-d800d89/Decentraland_windows64.zip'
+        default: 'https://explorer-artifacts.decentraland.org/@dcl/unity-explorer/branch/feat/test-automation/autopilot-alttester/pr-22430-3a0b8d5/Decentraland_windows64.zip'
       test_filter:
         description: 'NUnit --filter expression. Default runs the full InWorld category (~17 tests).'
         type: string

explorer/Tests/Tests/BaseTest.cs

@@ -71,7 +71,18 @@ public void AttachPerf()
         }
         catch (Exception ex)
         {
-            Reporter.Log($"PerfSampler.End call failed (Player crash?): {ex.Message}");
+            // AltTester wraps the Player-side exception in a TargetInvocationException;
+            // ex.Message is the unhelpful "Exception has been thrown by the target of an
+            // invocation." Log ToString() so the inner trace from AltTester's error
+            // payload makes it into the Allure report.
+            Reporter.Log($"PerfSampler.End call failed (Player crash?):\n{ex}");
+
+            // Best-effort: PerfSampler closes the CSV BEFORE writing the summary, so a
+            // summary-side crash still leaves a complete perf.csv on disk. Attach what
+            // we have so a chassis run with a broken summary writer is still
+            // post-mortemable.
+            if (File.Exists(_perfCsvPath))
+                AllureApi.AddAttachment("perf.csv", "text/csv", File.ReadAllBytes(_perfCsvPath));
             return;
         }