Refactor health checks and error handling

-  Updated
  error handling to catch a general `Exception`.
- Adjusted health check mappings in `Program.cs`,
  swapping predicates for "/healthz/ready" and
  "/healthz/live" endpoints, affecting readiness
  and liveliness reporting.

Author: marcominerva

src/PdfSmith/BackgroundServices/InstallPlaywrightBackgroundService.cs

@@ -1,21 +1,20 @@
-using Microsoft.Playwright;
-using PdfSmith.HealthChecks;
+using PdfSmith.HealthChecks;
 
 namespace PdfSmith.BackgroundServices;
 
 public class InstallPlaywrightBackgroundService(PlaywrightHealthCheck playwrightHealthCheck, ILogger<InstallPlaywrightBackgroundService> logger) : BackgroundService
 {
     protected override async Task ExecuteAsync(CancellationToken stoppingToken)
     {
-        // On Windows, it is installed in %USERPROFILE%\AppData\Local\ms-playwright by default
-        // We can use PLAYWRIGHT_BROWSERS_PATH environment variable to change the default location
+        // On Windows, it is installed in %USERPROFILE%\AppData\Local\ms-playwright by default.
+        // We can use PLAYWRIGHT_BROWSERS_PATH environment variable to change the default location.
         var returnCode = await Task.Run(() =>
         {
             try
             {
                 return Microsoft.Playwright.Program.Main(["install", "chromium"]);
             }
-            catch (PlaywrightException ex)
+            catch (Exception ex)
             {
                 logger.LogError(ex, "Error while installing Chromium");
                 return -1;

src/PdfSmith/Program.cs

@@ -171,15 +171,15 @@
 app.UseRateLimiter();
 app.UseRequestTimeouts();
 
-app.MapHealthChecks("/healthz/ready", new HealthCheckOptions
+app.MapHealthChecks("/healthz/live", new HealthCheckOptions
 {
-    Predicate = healthCheck => healthCheck.Tags.Contains("ready"),
+    Predicate = _ => false,
     ResponseWriter = HealthChecksResponseWriter()
 });
 
-app.MapHealthChecks("/healthz/live", new HealthCheckOptions
+app.MapHealthChecks("/healthz/ready", new HealthCheckOptions
 {
-    Predicate = _ => false,
+    Predicate = healthCheck => healthCheck.Tags.Contains("ready"),
     ResponseWriter = HealthChecksResponseWriter()
 });