fix(core): temp-dir cleanup must not flip cancelled runs to failed

ReportRunner's finally block deleted the per-job temp dir but caught only
IOException. On Windows a briefly-locked file throws UnauthorizedAccessException,
which escaped finally and replaced an in-flight OperationCanceledException — so a
cancelled job was recorded as Failed. Catch broadly: cleanup is best-effort and
must never change the outcome. Fixes the two cancellation jobs tests.

Author: thiagoluga

src/NeoReports.Core/Pipeline/ReportRunner.cs

@@ -217,9 +217,12 @@ await writer.InitializeAsync(
                 if (Directory.Exists(tempDir))
                     Directory.Delete(tempDir, recursive: true);
             }
-            catch (IOException)
+            catch (Exception)
             {
-                // Best-effort cleanup; a leftover temp file must not fail a completed report.
+                // Best-effort cleanup: a leftover temp file must never change the job's outcome.
+                // Catch broadly (IOException, UnauthorizedAccessException when a file is briefly
+                // locked on Windows, etc.) so cleanup cannot replace an in-flight cancellation
+                // exception and turn a cancelled run into a failed one.
             }
         }
     }