refactor: narrow sitemap catch clause to InvalidOperationException

BenjaminMichaelis · BenjaminMichaelis · commit 653055fd6693 · 2026-05-13T21:59:40.000-07:00
EnsureSitemapHealthy only throws InvalidOperationException; using a
generic catch was overly broad. Addresses GitHub code quality feedback.
diff --git a/EssentialCSharp.Web/Program.cs b/EssentialCSharp.Web/Program.cs
@@ -609,7 +609,7 @@ await McpJsonRpcResponseWriter.WriteErrorAsync(
             SitemapXmlHelpers.EnsureSitemapHealthy(siteMappingService.SiteMappings.ToList());
             LogSitemapValidationSucceeded(logger);
         }
-        catch (Exception ex)
+        catch (InvalidOperationException ex)
         {
             LogSitemapValidationFailed(logger, ex);
             // Continue startup even if sitemap validation fails

Original file line number	Diff line number	Diff line change
`@@ -609,7 +609,7 @@ await McpJsonRpcResponseWriter.WriteErrorAsync(`
`609`	`609`	`SitemapXmlHelpers.EnsureSitemapHealthy(siteMappingService.SiteMappings.ToList());`
`610`	`610`	`LogSitemapValidationSucceeded(logger);`
`611`	`611`	`}`
`612`		`- catch (Exception ex)`
	`612`	`+ catch (InvalidOperationException ex)`
`613`	`613`	`{`
`614`	`614`	`LogSitemapValidationFailed(logger, ex);`
`615`	`615`	`// Continue startup even if sitemap validation fails`