From a6668944619fded3d9562082db775467af156318 Mon Sep 17 00:00:00 2001
From: Arthur van de Vondervoort <arthur.vandevondervoort@vanroey.be>
Date: Wed, 10 Jun 2026 15:59:50 +0200
Subject: [PATCH] fix: upgrade ALCops.Analyzers to v0.8.6 and improve AD0001
 handling
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

Resolves #10 — LinterCop analyzer crashed with MissingMethodException
when the AL Language extension provides newer BC DevTools than what
ALCops.Analyzers v0.5.0 was compiled against. The reflection-based
wrapper for QuoteIdentifierIfNeeded was introduced in v0.6.0+.

- Bump ALCops.Analyzers from 0.5.0 to 0.8.6
- Log AD0001 (analyzer crash) diagnostics to stderr for debugging
  while still including them in results
- Log BC DevTools version at startup for easier triage
- Add Dependabot config scoped to ALCops.Analyzers

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
---
 .github/dependabot.yml                         | 8 ++++++++
 src/ALCops.Mcp/ALCops.Mcp.csproj               | 2 +-
 src/ALCops.Mcp/Services/BcDevToolsBootstrap.cs | 7 +++++++
 src/ALCops.Mcp/Services/DiagnosticsRunner.cs   | 3 +++
 4 files changed, 19 insertions(+), 1 deletion(-)
 create mode 100644 .github/dependabot.yml

diff --git a/.github/dependabot.yml b/.github/dependabot.yml
new file mode 100644
index 0000000..8d2baeb
--- /dev/null
+++ b/.github/dependabot.yml
@@ -0,0 +1,8 @@
+version: 2
+updates:
+  - package-ecosystem: nuget
+    directory: /src/ALCops.Mcp
+    schedule:
+      interval: weekly
+    allow:
+      - dependency-name: ALCops.Analyzers
diff --git a/src/ALCops.Mcp/ALCops.Mcp.csproj b/src/ALCops.Mcp/ALCops.Mcp.csproj
index a8a3447..6e549f9 100644
--- a/src/ALCops.Mcp/ALCops.Mcp.csproj
+++ b/src/ALCops.Mcp/ALCops.Mcp.csproj
@@ -26,7 +26,7 @@
         <PackageReference Include="ModelContextProtocol" Version="0.9.0-preview.2" />
         <PackageReference Include="Microsoft.Extensions.Hosting" Version="9.0.0" />
         <PackageReference Include="System.Composition.AttributedModel" Version="8.0.0" />
-        <PackageReference Include="ALCops.Analyzers" Version="0.5.0" PrivateAssets="none" />
+        <PackageReference Include="ALCops.Analyzers" Version="0.8.6" PrivateAssets="none" />
     </ItemGroup>
 
     <!-- BC Dev Tools — needed at compile time, excluded from package by ExcludeBcDevToolsFromPublish.
diff --git a/src/ALCops.Mcp/Services/BcDevToolsBootstrap.cs b/src/ALCops.Mcp/Services/BcDevToolsBootstrap.cs
index acbe337..92f50ff 100644
--- a/src/ALCops.Mcp/Services/BcDevToolsBootstrap.cs
+++ b/src/ALCops.Mcp/Services/BcDevToolsBootstrap.cs
@@ -21,6 +21,13 @@ internal static class BcDevToolsBootstrap
         {
             RegisterAssemblyResolver(path);
             Console.Error.WriteLine($"BC DevTools: {path}");
+            try
+            {
+                var version = System.Diagnostics.FileVersionInfo
+                    .GetVersionInfo(Path.Combine(path, MarkerDll));
+                Console.Error.WriteLine($"BC DevTools version: {version.FileVersion}");
+            }
+            catch { /* non-critical */ }
         }
         else
         {
diff --git a/src/ALCops.Mcp/Services/DiagnosticsRunner.cs b/src/ALCops.Mcp/Services/DiagnosticsRunner.cs
index 49ae871..b97e656 100644
--- a/src/ALCops.Mcp/Services/DiagnosticsRunner.cs
+++ b/src/ALCops.Mcp/Services/DiagnosticsRunner.cs
@@ -62,6 +62,9 @@ public async Task<IReadOnlyList<DiagnosticResult>> RunAsync(
             if (diagnostic.IsSuppressed)
                 continue;
 
+            if (diagnostic.Id == "AD0001")
+                Console.Error.WriteLine($"Analyzer exception: {diagnostic.GetMessage()}");
+
             // Ruleset filter: suppress diagnostics set to None, override severity for others
             var effectiveSeverity = diagnostic.Severity;
             if (provider is AnalyzerSet analyzerSet && analyzerSet.RuleActions is { } ruleActions)