refactor(logging): convert to LoggerMessage source-generator

CA1873 modernization across protocol registry, sidecar plugin discovery,
and CLI command registry. Replaces the suppressed runtime null-check
`_logger?.LogXxx(...)` shape with source-generated partial methods that
emit IsEnabled-gated dispatch (Microsoft.Extensions.Logging.LoggerMessage
attribute). One internal static partial class per file holds the wrappers,
matching the convention already used by AuthProviderLog and
BackgroundUpdateCheckLog. Drops 5 CA1873 pragmas.

Author: beachwalker

src/Kuestenlogik.Bowire.Cli/BowireCliCommandRegistry.cs

@@ -51,11 +51,8 @@ public static BowireCliCommandRegistry Discover(
                     {
                         if (disabled.Contains(command.Id))
                         {
-#pragma warning disable CA1873
-                            logger?.LogInformation(
-                                "Skipping disabled CLI command '{CommandId}' (--disable-cli-command).",
-                                command.Id);
-#pragma warning restore CA1873
+                            if (logger is not null)
+                                CliCommandRegistryLog.SkippingDisabledCommand(logger, command.Id);
                             continue;
                         }
                         registry.Register(command);
@@ -72,3 +69,18 @@ public static BowireCliCommandRegistry Discover(
         return registry;
     }
 }
+
+/// <summary>
+/// Source-generated logger wrappers for
+/// <see cref="BowireCliCommandRegistry"/>. Keeps CA1873 happy — the
+/// generator emits the <c>IsEnabled</c>-gated dispatch the analyzer
+/// wants in place of the suppressed runtime null-check.
+/// </summary>
+internal static partial class CliCommandRegistryLog
+{
+    [LoggerMessage(
+        EventId = 1,
+        Level = LogLevel.Information,
+        Message = "Skipping disabled CLI command '{CommandId}' (--disable-cli-command).")]
+    public static partial void SkippingDisabledCommand(ILogger logger, string commandId);
+}

src/Kuestenlogik.Bowire/BowireProtocolRegistry.cs

@@ -120,11 +120,8 @@ public static BowireProtocolRegistry Discover(
                     {
                         if (disabled.Contains(protocol.Id))
                         {
-#pragma warning disable CA1873 // logger is already null-checked
-                            logger?.LogInformation(
-                                "Skipping disabled protocol plugin '{PluginId}' (Bowire:DisabledPlugins).",
-                                protocol.Id);
-#pragma warning restore CA1873
+                            if (logger is not null)
+                                ProtocolRegistryLog.SkippingDisabledPlugin(logger, protocol.Id);
                             // #29 -- still record the load attempt so the
                             // 'skipped because disabled' branch shows up
                             // in the Grafana panel separately from
@@ -218,3 +215,19 @@ private static void ForceLoadReferencedBowireAssemblies(ILogger? logger)
         }
     }
 }
+
+/// <summary>
+/// Source-generated logger wrappers for
+/// <see cref="BowireProtocolRegistry"/>. Spinning these out of the
+/// inline calls keeps CA1873 happy (the generator emits the
+/// <c>IsEnabled</c>-gated dispatch the analyzer wants and avoids
+/// boxing of the <c>{PluginId}</c> tag when the level isn't enabled).
+/// </summary>
+internal static partial class ProtocolRegistryLog
+{
+    [LoggerMessage(
+        EventId = 1,
+        Level = LogLevel.Information,
+        Message = "Skipping disabled protocol plugin '{PluginId}' (Bowire:DisabledPlugins).")]
+    public static partial void SkippingDisabledPlugin(ILogger logger, string pluginId);
+}

src/Kuestenlogik.Bowire/Plugins/Sidecar/SidecarPluginDiscovery.cs

@@ -55,32 +55,23 @@ public static IReadOnlyList<SidecarBowireProtocol> Discover(
             var manifest = SidecarPluginManifest.TryLoadFromFile(manifestPath);
             if (manifest is null)
             {
-#pragma warning disable CA1873
-                logger?.LogWarning(
-                    "Sidecar manifest at {Path} is missing or malformed; skipping.",
-                    manifestPath);
-#pragma warning restore CA1873
+                if (logger is not null)
+                    SidecarDiscoveryLog.ManifestMissingOrMalformed(logger, manifestPath);
                 continue;
             }
 
             if (string.IsNullOrEmpty(manifest.Protocol?.Id)
                 || string.IsNullOrEmpty(manifest.Executable))
             {
-#pragma warning disable CA1873
-                logger?.LogWarning(
-                    "Sidecar manifest at {Path} is missing protocol.id or executable; skipping.",
-                    manifestPath);
-#pragma warning restore CA1873
+                if (logger is not null)
+                    SidecarDiscoveryLog.ManifestMissingRequiredFields(logger, manifestPath);
                 continue;
             }
 
             if (disabled.Contains(manifest.Protocol.Id))
             {
-#pragma warning disable CA1873
-                logger?.LogInformation(
-                    "Skipping disabled sidecar plugin '{PluginId}' (Bowire:DisabledPlugins).",
-                    manifest.Protocol.Id);
-#pragma warning restore CA1873
+                if (logger is not null)
+                    SidecarDiscoveryLog.SkippingDisabledSidecar(logger, manifest.Protocol.Id);
                 continue;
             }
 
@@ -90,3 +81,30 @@ public static IReadOnlyList<SidecarBowireProtocol> Discover(
         return results;
     }
 }
+
+/// <summary>
+/// Source-generated logger wrappers for
+/// <see cref="SidecarPluginDiscovery"/>. Folds three CA1873 sites into
+/// one place; the generator emits <c>IsEnabled</c>-gated dispatch so
+/// the runtime null-check the analyzer flagged is gone.
+/// </summary>
+internal static partial class SidecarDiscoveryLog
+{
+    [LoggerMessage(
+        EventId = 1,
+        Level = LogLevel.Warning,
+        Message = "Sidecar manifest at {Path} is missing or malformed; skipping.")]
+    public static partial void ManifestMissingOrMalformed(ILogger logger, string path);
+
+    [LoggerMessage(
+        EventId = 2,
+        Level = LogLevel.Warning,
+        Message = "Sidecar manifest at {Path} is missing protocol.id or executable; skipping.")]
+    public static partial void ManifestMissingRequiredFields(ILogger logger, string path);
+
+    [LoggerMessage(
+        EventId = 3,
+        Level = LogLevel.Information,
+        Message = "Skipping disabled sidecar plugin '{PluginId}' (Bowire:DisabledPlugins).")]
+    public static partial void SkippingDisabledSidecar(ILogger logger, string pluginId);
+}