🧩 Analysi\n- thread Oc6Efsyy from @coderabbitai: **needs-human** — no rule matched — keeping blocked for manual review\n > _⚠️ Potential issue_ | _🟠 Major_ | _🏗️ Heavy lift_ **Cross-repo publish order\n- thread Oc6Efsy3 from @coderabbitai: **needs-human** — no rule matched — keeping blocked for manual review\n > _⚠️ Potential issue_ | _🟠 Major_ | _⚡ Quick win_

🧩 Analysi\n- thread Oc6EfszC from @coderabbitai: **needs-human** — no rule matched — keeping blocked for manual review\n > _⚠️ Potential issue_ | _🟠 Major_ | _⚡ Quick win_ **`equivalenceKey` should be \n- thread Oc6EfszF from @coderabbitai: **needs-human** — no rule matched — keeping blocked for manual review\n > _⚠️ Potential issue_ | _🟠 Major_ | _⚡ Quick win_

🧩 Analysi\n- thread Oc6EfszL from @coderabbitai: **needs-human** — no rule matched — keeping blocked for manual review\n > _🛠️ Refactor suggestion_ | _🟠 Major_ | _🏗️ Heavy lift_ **Align provider with\n- thread Oc6EfszQ from @coderabbitai: **needs-human** — no rule matched — keeping blocked for manual review\n > _⚠️ Potential issue_ | _🔴 Critical_ | _⚡ Quick win_ **FixAll `equivalenceKey` \n- thread Oc6EfszT from @coderabbitai: **needs-human** — no rule matched — keeping blocked for manual review\n > _⚠️ Potential issue_ | _🟠 Major_ | _⚡ Quick win_ **`equivalenceKey` should use\n- thread Oc6EfszW from @coderabbitai: **needs-human** — no rule matched — keeping blocked for manual review\n > _⚠️ Potential issue_ | _🔴 Critical_ | _⚡ Quick win_ **Fix CodeAction contract \n- thread Oc6Efszd from @coderabbitai: **needs-human** — no rule matched — keeping blocked for manual review\n > _⚠️ Potential issue_ | _🟠 Major_ | _⚡ Quick win_ **Use analyzer DiagnosticId a\n- thread Oc6Efszj from @coderabbitai: **needs-human** — no rule matched — keeping blocked for manual review\n > _⚠️ Potential issue_ | _🟠 Major_ | _⚡ Quick win_ **FixAll equivalence key must\n- thread Oc6Efszs from @coderabbitai: **needs-human** — no rule matched — keeping blocked for manual review\n > _⚠️ Potential issue_ | _🟠 Major_ | _⚡ Quick win_ **Set equivalence key to `Al1\n- thread Oc6Efszw from @coderabbitai: **needs-human** — no rule matched — keeping blocked for manual review\n > _🛠️ Refactor suggestion_ | _🟠 Major_ | _🏗️ Heavy lift_ **Provider base type \n- thread Oc6Efszx from @coderabbitai: **needs-human** — no rule matched — keeping blocked for manual review\n > _⚠️ Potential issue_ | _🟠 Major_ | _⚡ Quick win_ **Use analyzer DiagnosticId a\n- thread Oc6Efsz0 from @coderabbitai: **needs-human** — no rule matched — keeping blocked for manual review\n > _🛠️ Refactor suggestion_ | _🟠 Major_ | _🏗️ Heavy lift_ **Code fix provider s\n- thread Oc6Efsz3 from @coderabbitai: **needs-human** — no rule matched — keeping blocked for manual review\n > _⚠️ Potential issue_ | _🟠 Major_ | _⚡ Quick win_ **Both code actions need anal\n- thread Oc6Efsz5 from @coderabbitai: **needs-human** — no rule matched — keeping blocked for manual review\n > _⚠️ Potential issue_ | _🟠 Major_ | _⚡ Quick win_ **Replace provider-name equiv\n- thread Oc6Efsz8 from @coderabbitai: **needs-human** — no rule matched — keeping blocked for manual review\n > _⚠️ Potential issue_ | _🟠 Major_ | _⚡ Quick win_ **Equivalence key should be `\n- thread Oc6Efsz- from @coderabbitai: **needs-human** — no rule matched — keeping blocked for manual review\n > _⚠️ Potential issue_ | _🟠 Major_ | _⚡ Quick win_ **Set equivalence key to `Al1\n- thread Oc6Efs0E from @coderabbitai: **needs-human** — no rule matched — keeping blocked for manual review\n > _⚠️ Potential issue_ | _🔴 Critical_ | _⚡ Quick win_ **equivalenceKey must be t\n- thread Oc6Efs0L from @coderabbitai: **needs-human** — no rule matched — keeping blocked for manual review\n > _⚠️ Potential issue_ | _🔴 Critical_ | _⚡ Quick win_ **equivalenceKey must be t\n- thread Oc6Efs0S from @coderabbitai: **needs-human** — no rule matched — keeping blocked for manual review\n > _⚠️ Potential issue_ | _🔴 Critical_ | _⚡ Quick win_ **equivalenceKey must be t\n- thread Oc6Efs0d from @coderabbitai: **needs-human** — no rule matched — keeping blocked for manual review\n > _⚠️ Potential issue_ | _🔴 Critical_ | _⚡ Quick win_ **equivalenceKey must be t\n- thread Oc6Efs0l from @coderabbitai: **needs-human** — no rule matched — keeping blocked for manual review\n > _⚠️ Potential issue_ | _🔴 Critical_ | _⚡ Quick win_ **equivalenceKey must be t\n- thread Oc6Efs0r from @coderabbitai: **needs-human** — no rule matched — keeping blocked for manual review\n > _⚠️ Potential issue_ | _🔴 Critical_ | _⚡ Quick win_ **equivalenceKey must be t\n- thread Oc6Efs02 from @coderabbitai: **needs-human** — no rule matched — keeping blocked for manual review\n > _⚠️ Potential issue_ | _🔴 Critical_ | _⚡ Quick win_ **equivalenceKey must be t\n- thread Oc6Efs1A from @coderabbitai: **needs-human** — no rule matched — keeping blocked for manual review\n > _⚠️ Potential issue_ | _🔴 Critical_ | _⚡ Quick win_ **equivalenceKey must be t\n- thread Oc6Efs1I from @coderabbitai: **needs-human** — no rule matched — keeping blocked for manual review\n > _⚠️ Potential issue_ | _🟠 Major_ | _⚡ Quick win_ **Use analyzer `DiagnosticId`\n- thread Oc6Efs1R from @coderabbitai: **needs-human** — no rule matched — keeping blocked for manual review\n > _⚠️ Potential issue_ | _🟠 Major_ | _⚡ Quick win_ **Set `equivalenceKey` to the\n- thread Oc6Efs1W from @coderabbitai: **needs-human** — no rule matched — keeping blocked for manual review\n > _⚠️ Potential issue_ | _🟠 Major_ | _⚡ Quick win_ **FixAll equivalence key shou\n- thread Oc6Efs1e from @coderabbitai: **needs-human** — no rule matched — keeping blocked for manual review\n > _⚠️ Potential issue_ | _🟠 Major_ | _⚡ Quick win_ **Use analyzer `DiagnosticId`\n- thread Oc6Efs1r from @coderabbitai: **needs-human** — no rule matched — keeping blocked for manual review\n > _⚠️ Potential issue_ | _🟠 Major_ | _⚡ Quick win_ **`equivalenceKey` should tra\n- thread Oc6Efs15 from @coderabbitai: **needs-human** — no rule matched — keeping blocked for manual review\n > _⚠️ Potential issue_ | _🟠 Major_ | _⚡ Quick win_ **FixAll key must be analyzer\n- thread Oc6Efs2H from @coderabbitai: **needs-human** — no rule matched — keeping blocked for manual review\n > _⚠️ Potential issue_ | _🟠 Major_ | _⚡ Quick win_ **Incorrect `equivalenceKey` \n- thread Oc6Efs2M from @coderabbitai: **needs-human** — no rule matched — keeping blocked for manual review\n > _⚠️ Potential issue_ | _🟠 Major_ | _⚡ Quick win_ **Align code action `equivale\n- thread Oc6Efs2W from @coderabbitai: **needs-human** — no rule matched — keeping blocked for manual review\n > _⚠️ Potential issue_ | _🟠 Major_ | _⚡ Quick win_ **equivalenceKey must use the\n- thread Oc6Efs2c from @coderabbitai: **needs-human** — no rule matched — keeping blocked for manual review\n > _⚠️ Potential issue_ | _🟠 Major_ | _⚡ Quick win_ **equivalenceKey must use the\n- thread Oc6Efs2k from @coderabbitai: **needs-human** — no rule matched — keeping blocked for manual review\n > _⚠️ Potential issue_ | _🟠 Major_ | _⚡ Quick win_ **equivalenceKey must use the\n- thread Oc6Efs2x from @coderabbitai: **needs-human** — no rule matched — keeping blocked for manual review\n > _⚠️ Potential issue_ | _🟠 Major_ | _⚡ Quick win_ **equivalenceKey must use the\n- thread Oc6Efs26 from @coderabbitai: **needs-human** — no rule matched — keeping blocked for manual review\n > _🧹 Nitpick_ | _🔵 Trivial_ | _🏗️ Heavy lift_ **Coordinate Version.props updat\n- thread Oc6EhFBM from @chatgpt-codex-connector: **needs-human** — no rule matched — keeping blocked for manual review\n > **_{_{![P1 Badge](https://img.shields.io/badge/P1-orange?style=flat)}\n- thread Oc6EhFBP from @chatgpt-codex-connector: **needs-human** — no rule matched — keeping blocked for manual review\n > **_{_{![P1 Badge](https://img.shields.io/badge/P1-orange?style=flat)}\n\n_Threads marked `needs-human` stay unresolved and block auto-merge._","side","line","\n### 💡 Codex Review\n\nHere are some automated review suggestions for this pull request.\n\n**Reviewed commit:** `2fe348d832`\n \n\n ℹ️ About Codex in GitHub
\n
\n\nCodex has been enabled to automatically review pull requests in this repo. Reviews are triggered when you\n- Open a pull request for review\n- Mark a draft as ready\n- Comment \"@codex review\".\n\nIf Codex has suggestions, it will comment; otherwise it will react with 👍.\n\n\n\n\nWhen you [sign up for Codex through ChatGPT](https://openai.com/codex), Codex can also answer questions or update the PR, like \"@codex address that feedback\".\n \n
","**_{_{![P1 Badge](https://img.shields.io/badge/P1-orange?style=flat)}} Add missing AL1010 code-fix resource symbol**\n\nThis provider now references `CodeFixResources.AL1010CodeFixTitle`, but that symbol is not defined in `CodeFixResources.Designer.cs` (and there is no `AL1010CodeFixTitle` entry in `CodeFixResources.resx`), so the code-fix project cannot compile once this file is built. The previous version used a literal title string, so this unresolved resource reference is introduced by this commit.\n\nUseful? React with 👍 / 👎.","src/ANcpLua.Analyzers.CodeFixes/CodeFixes/AL1010PatternMatchingCodeFixProvider.cs","RIGHT",19,"**_{_{![P1 Badge](https://img.shields.io/badge/P1-orange?style=flat)}} Keep self-referenced analyzer version on last published release**\n\n`ANcpLuaAnalyzersVersion` is consumed by restore-time package references (as documented in `Directory.Build.props`), so pinning it to `2.0.0` before that package is actually available on NuGet will make clean restores fail with `NU1102` in this repo and downstream SDK consumers. The commit message itself schedules the push as a post-merge step, which means this value should remain on the last published version until publication is complete.\n\nUseful? React with 👍 / 👎.",61,"**Claude encountered an error after 2s** —— [View job](https://github.com/ANcpLua/ANcpLua.Analyzers/actions/runs/26394912254)\n\n---\nI'll analyze this and get back to you.","\n> [!NOTE]\n> Autofix is a beta feature. Expect some limitations and changes as we gather feedback and continue to improve it.\n\n## Fixes Applied Successfully\n\nFixed 5 file(s) based on 40 unresolved review comments.\n\n\n\n**Files modified:**\n- `Version.props`\n- `eng/analyzer-renumber-plan.md`\n- `src/ANcpLua.Analyzers.CodeFixes/CodeFixes/AL1007IXmlSerializableCodeFixProvider.cs`\n- `src/ANcpLua.Analyzers.CodeFixes/CodeFixes/AL1010PatternMatchingCodeFixProvider.cs`\n- `src/ANcpLua.Analyzers.CodeFixes/CodeFixes/AL1202UseTypeHierarchyCodeFixProvider.cs`\n\n**Commit:** `2fe348d832d2933a6485914543622a272fc8621f`\n\nThe changes have been pushed to the `feat/major-renumber` branch.\n\n**Time taken:** `6m 52s`\n","@coderabbitai autofix","**Claude encountered an error after 2s** —— [View job](https://github.com/ANcpLua/ANcpLua.Analyzers/actions/runs/26394879565)\n\n---\nI'll analyze this and get back to you.","\n> [!NOTE]\n> Autofix is a beta feature. Expect some limitations and changes as we gather feedback and continue to improve it.\n\n⚠️ **Branch updated during autofix.**\n\nThe branch was updated while autofix was in progress. Please try again.\n","## Up to standards :white_check_mark:\n🟢 Issues 0 issues
\n\n>
\n>\n> \n>\n> **Results:**\n> `0` new issues\n>\n>\n>\n> [View in Codacy](https://app.codacy.com/gh/ANcpLua/ANcpLua.Analyzers/pull-requests/174/issues)\n>
\n
\n\n🟢 Metrics 2083 complexity · 192 duplication
\n\n>
\n>\n> | Metric | Results |\n> | ------------- | ------------- |\n> | Complexity | **2083** | \n > | Duplication | **192** |\n>\n>\n> [View in Codacy](https://app.codacy.com/gh/ANcpLua/ANcpLua.Analyzers/pull-requests/174/files)\n>
\n
\n\n\n\n##\n> **AI Reviewer:** first review requested successfully. _AI can make mistakes. Always validate suggestions._\n>\n> [][Link]\n>\n> [Link]: https://app.codacy.com/gh/ANcpLua/ANcpLua.Analyzers/pull-requests/174/review?utm_source=github.com&utm_medium=unifiedPullRequestSummary&utm_campaign=unifiedPullRequestSummary 'Trigger a new review'\n\n\n\n_{`TIP` This summary will be updated as you push new changes.}\n\n","\n> [!NOTE]\n> Autofix is a beta feature. Expect some limitations and changes as we gather feedback and continue to improve it.\n\n## Fixes Applied Successfully\n\nFixed 35 file(s) based on 40 unresolved review comments.\n\n\n\n**Files modified:**\n- `.editorconfig`\n- `src/ANcpLua.Analyzers.CodeFixes/CodeFixes/AL1001SimplifyNegatedPatternCodeFixProvider.cs`\n- `src/ANcpLua.Analyzers.CodeFixes/CodeFixes/AL1003SpanPatternMatchingCodeFixProvider.cs`\n- `src/ANcpLua.Analyzers.CodeFixes/CodeFixes/AL1004UseSequenceEqualCodeFixProvider.cs`\n- `src/ANcpLua.Analyzers.CodeFixes/CodeFixes/AL1007IXmlSerializableCodeFixProvider.cs`\n- `src/ANcpLua.Analyzers.CodeFixes/CodeFixes/AL1009LockTypeCodeFixProvider.cs`\n- `src/ANcpLua.Analyzers.CodeFixes/CodeFixes/AL1010PatternMatchingCodeFixProvider.cs`\n- `src/ANcpLua.Analyzers.CodeFixes/CodeFixes/AL1011NormalizeNullGuardStyleCodeFixProvider.cs`\n- `src/ANcpLua.Analyzers.CodeFixes/CodeFixes/AL1012CombineDeclarationWithNullCheckCodeFixProvider.cs`\n- `src/ANcpLua.Analyzers.CodeFixes/CodeFixes/AL1200UseIsEqualToCodeFixProvider.cs`\n- `src/ANcpLua.Analyzers.CodeFixes/CodeFixes/AL1201UseHasAttributeCodeFixProvider.cs`\n- `src/ANcpLua.Analyzers.CodeFixes/CodeFixes/AL1202UseTypeHierarchyCodeFixProvider.cs`\n- `src/ANcpLua.Analyzers.CodeFixes/CodeFixes/AL1203UseOperationExtensionsCodeFixProvider.cs`\n- `src/ANcpLua.Analyzers.CodeFixes/CodeFixes/AL1204UseOrEmptyCodeFixProvider.cs`\n- `src/ANcpLua.Analyzers.CodeFixes/CodeFixes/AL1205UseToImmutableArrayOrEmptyCodeFixProvider.cs`\n- `src/ANcpLua.Analyzers.CodeFixes/CodeFixes/AL1206UseWhereNotNullCodeFixProvider.cs`\n- `src/ANcpLua.Analyzers.CodeFixes/CodeFixes/AL1207UseToDisplayStringExtensionsCodeFixProvider.cs`\n- `src/ANcpLua.Analyzers.CodeFixes/CodeFixes/AL1208UseGuardNotNullCodeFixProvider.cs`\n- `src/ANcpLua.Analyzers.CodeFixes/CodeFixes/AL1209UseTryParseExtensionsCodeFixProvider.cs`\n- `src/ANcpLua.Analyzers.CodeFixes/CodeFixes/AL1210UseStringComparisonExtensionsCodeFixProvider.cs`\n- `src/ANcpLua.Analyzers.CodeFixes/CodeFixes/AL1211UseAttributeExtensionsCodeFixProvider.cs`\n- `src/ANcpLua.Analyzers.CodeFixes/CodeFixes/AL1212UseGuardNotNullOrEmptyCodeFixProvider.cs`\n- `src/ANcpLua.Analyzers.CodeFixes/CodeFixes/AL1213UseGuardNotNullOrWhiteSpaceCodeFixProvider.cs`\n- `src/ANcpLua.Analyzers.CodeFixes/CodeFixes/AL1214UseGuardNotZeroCodeFixProvider.cs`\n- `src/ANcpLua.Analyzers.CodeFixes/CodeFixes/AL1215UseGuardNotNegativeCodeFixProvider.cs`\n- `src/ANcpLua.Analyzers.CodeFixes/CodeFixes/AL1216UseGuardPositiveCodeFixProvider.cs`\n- `src/ANcpLua.Analyzers.CodeFixes/CodeFixes/AL1217UseGuardNotEmptyGuidCodeFixProvider.cs`\n- `src/ANcpLua.Analyzers.CodeFixes/CodeFixes/AL1218UseGuardDefinedEnumCodeFixProvider.cs`\n- `src/ANcpLua.Analyzers.CodeFixes/CodeFixes/AL1220UseGuardForThrowIfCodeFixProvider.cs`\n- `src/ANcpLua.Analyzers.CodeFixes/CodeFixes/AL1313CancellationTokenPropagationCodeFixProvider.cs`\n- `src/ANcpLua.Analyzers.CodeFixes/CodeFixes/AL1314UseExplicitMidpointRoundingCodeFixProvider.cs`\n- `src/ANcpLua.Analyzers.CodeFixes/CodeFixes/AL1504DuckDbTableCodeFixProvider.cs`\n- `src/ANcpLua.Analyzers.CodeFixes/CodeFixes/AL1700StaticLambdaCodeFixProvider.cs`\n- `src/ANcpLua.Analyzers.CodeFixes/CodeFixes/AL1701DateTimeNowCodeFixProvider.cs`\n- `src/ANcpLua.Analyzers.CodeFixes/CodeFixes/AL1702UseSystemTextJsonCodeFixProvider.cs`\n\n**Commit:** `5e8126753de2f2ac570b9d5819990e13d314e28b`\n\nThe changes have been pushed to the `feat/major-renumber` branch.\n\n**Time taken:** `8m 27s`\n","**Claude encountered an error after 2s** —— [View job](https://github.com/ANcpLua/ANcpLua.Analyzers/actions/runs/26394204630)\n\n---\nI'll analyze this and get back to you.","\n> [!TIP]\n> For best results, initiate chat on the files or code changes.\n\n`@ANcpLua`: Acknowledged. These “needs-human” threads boil down to three actionable buckets that will unblock auto-merge quickly:\n\n1) CodeAction equivalenceKey hygiene across all code fixes and FixAll\n2) HelpLink/wildcard updates after renumber (Codacy nitpicks)\n3) Cross‑repo release/SDK alignment notes (non‑blocking for this PR if just documented, but will block consumers post‑merge)\n\nBelow is a precise fix plan you can apply in this PR, followed by a post‑merge checklist for consumers.\n\nImmediate PR fixes\n\nA. Normalize CodeAction equivalenceKey usage\n- Why: Stable equivalence keys are required for proper Fix All behavior and to meet the contract flagged by the review bots.\n- Contract: For each provider, set the CodeAction equivalenceKey to the analyzer’s DiagnosticId (optionally suffixed by a deterministic variant string when a provider offers multiple distinct fixes).\n- Do NOT use provider class names as keys; keys must be stable and tied to the semantic operation.\n\nPattern to apply\n- Single-fix providers:\n - Replace CodeAction.Create(title, createChangedDocument, nameof(ProviderClass)) with CodeAction.Create(title, createChangedDocument, AlXXXXAnalyzer.DiagnosticId)\n- Multi-fix providers (e.g., AL1203 has two actions):\n - Suffix with a stable variant, e.g. $\"{Al1203UseOperationExtensionsAnalyzer.DiagnosticId}:IsMethodNamed\" and $\"{...}:TryGetConstantValue\"\n- FixAll providers:\n - If you implement a custom FixAllProvider (e.g., StaticLambda), ensure FixAll equivalence key equals the per-action key above (same DiagnosticId[:variant]); never provider-name.\n\nB. Sweep all CodeFixProviders in src/ANcpLua.Analyzers.CodeFixes\n- Files likely affected (examples from this PR’s scope):\n - AL1001SimplifyNegatedPatternCodeFixProvider.cs\n - AL1003SpanPatternMatchingCodeFixProvider.cs\n - AL1004UseSequenceEqualCodeFixProvider.cs\n - AL1007IXmlSerializableCodeFixProvider.cs\n - AL1009LockTypeCodeFixProvider.cs\n - AL1010PatternMatchingCodeFixProvider.cs\n - AL1011NormalizeNullGuardStyleCodeFixProvider.cs\n - AL1012CombineDeclarationWithNullCheckCodeFixProvider.cs\n - AL1200..AL1218/1220 CodeFixProviders (multiple)\n - AL1313, AL1314 CodeFixProviders\n - AL1500, AL1503, AL1504 CodeFixProviders\n - AL1700..AL1703 CodeFixProviders\n- For each CodeAction.Create, ensure the overload with equivalenceKey is used and set as above.\n\nConcrete code snippet (example)\nC#\n// before\nvar action = CodeAction.Create(\n CodeFixResources.AL1012CodeFixTitle,\n ct => CreateChangedDocumentAsync(document, decl, ct),\n nameof(Al1012CombineDeclarationWithNullCheckCodeFixProvider));\n\n// after\nvar action = CodeAction.Create(\n CodeFixResources.AL1012CodeFixTitle,\n ct => CreateChangedDocumentAsync(document, decl, ct),\n Al1012CombineDeclarationWithNullCheckAnalyzer.DiagnosticId);\n\nMulti-action example (AL1203)\nC#\ncontext.RegisterCodeFix(\n CodeAction.Create(\n CodeFixResources.AL1203CodeFixTitle,\n ct => ConvertToIsMethodNamedAsync(document, node, ct),\n $\"{Al1203UseOperationExtensionsAnalyzer.DiagnosticId}:IsMethodNamed\"),\n diagnostic);\n\ncontext.RegisterCodeFix(\n CodeAction.Create(\n CodeFixResources.AL1203CodeFixTitleTryGetConstantValue,\n ct => ConvertToTryGetConstantValueAsync(document, node, ct),\n $\"{Al1203UseOperationExtensionsAnalyzer.DiagnosticId}:TryGetConstantValue\"),\n diagnostic);\n\nFixAll example (AL1700)\n- If you have an inner FixAllProvider class (previously Al0025FixAllProvider), ensure FixAll equivalence key is the same DiagnosticId or DiagnosticId:Variant that the normal action uses. If FixAllProvider returns a CodeAction or batches multiple, set the CodeAction equivalenceKey accordingly and keep it stable.\n\nC. HelpLink and diagnostic ID wildcard updates (Codacy nits)\n- AlAnalyzer.HelpLink and any docs/resolution links must emit the new banded IDs (AL1xxx) and their help slugs.\n- Audit any pattern/wildcards used in:\n - AlAnalyzer.CreateRule / HelpLink logic\n - README/docs links, and comments that generate per‑ID help URLs\n - .editorconfig blocks that previously used AL0xxx wildcards\n- Ensure wildcards and ranges reflect AL1000..AL1899 (and reserved AL1900..AL1999) instead of AL0001..AL0139.\n- If HelpLink is based on resource key or markdown path generation, double-check for renamed doc files (e.g., docs/Al1200UseIsEqualTo.md updated already).\n- If Codacy flagged “Since the AlAnalyzer.HelpLink …” it likely expects the new base URL or slug; verify the host path hasn’t changed and per‑ID URL resolves or at least syntactically matches the new ID.\n\nD. Title non-null contract for CodeAction\n- Where CodeAction.Create is called, ensure title strings are non-null (resource keys exist after resx renames). Your diff already updated CodeFixResources.Designer.cs; just confirm no missing resource names remain after renumber (bots flagged this occasionally when resx/designer drift occurs).\n\nPost‑merge consumer sweep (tracked separately; not blocking this PR if documented)\n- ANcpLua.NET.Sdk\n - Update packaged Analyzer.ANcpLua.Analyzers.editorconfig to AL1xxx keys; otherwise SDK severities won’t match 2.0.0 analyzers.\n - Bump ANcpLuaAnalyzersVersion to 2.0.0 in Version/GlobalPackages.props, and fix any comments/targets referencing old IDs.\n- ANcpLua.Agents, ANcpLua.Roslyn.Utilities, ErrorOrX, others\n - Replace all AL0xxx in NoWarn/editorconfig/test assertions with AL1xxx per eng/analyzer-renumber-plan.md.\n - Bump analyzer version pins to 2.0.0 to avoid restore/ID drift.\n - Update doc comments mentioning legacy IDs (e.g., AL0081 to AL1106).\n\nHow to resolve the blocking threads\n\n- For each CodeRabbit “equivalenceKey” item: reply “Fixed — equivalenceKey now set to ” and link to the commit once pushed.\n- For the “FixAll equivalenceKey” items: reply “Fixed — FixAll action/keys now use the analyzer DiagnosticId[:Variant] and are stable across documents.”\n- For the Codacy two nits:\n - HelpLink: reply “Confirmed — HelpLink generator updated for AL1xxx; links match renamed docs. If you still detect legacy pattern, point to file X line Y.”\n - Diagnostic ID wildcards: reply “Updated .editorconfig/documentation wildcards from AL0xxx to AL1xxx ranges; please recheck.”\n\nIf you want, I can generate a scoped patch plan touching only the affected providers to add the correct equivalenceKey values and point you at specific lines to edit.","\n> [!TIP]\n> For best results, initiate chat on the files or code changes.\n\n`@ANcpLua` The triage report shows two categories of blocks:\n\n**Codacy threads (requires your action):**\n- `Oc6EeXH6` — `AlAnalyzer.HelpLink` nitpick\n- `Oc6EeXH-` — diagnostic ID wildcard update nitpick\n\nThese are low-risk and can only be resolved by you directly in those threads (reply to dismiss or address the nitpick).\n\n**My threads (~40):**\n- The bulk are **`equivalenceKey` correctness** issues across every code fix provider — these are real bugs where `CodeAction.Create` equivalence keys still use the old `nameof(AlXXXXCodeFixProvider)` string instead of the diagnostic ID, which breaks FixAll scoping.\n- `Oc6Efsyy` — cross-repo publish order coordination (informational, no code change needed in this PR)\n- `Oc6Efs26` — `Version.props` coordination (nitpick)\n\n**Options:**\n\n1. **Address the equivalenceKey issues** — this is the substantively correct path since stale equivalence keys silently break FixAll. I can generate the patches if you want.\n2. **Defer to a follow-up PR** — if you want to keep this PR focused on renumbering only, I can open a tracking issue and then resolve all my threads here as deferred. Just confirm and I'll do both.\n\nWhich path do you prefer?","## Triage Bot report\n\n- thread Oc6EeXH6 from @codacy-production: **needs-human** — no rule matched — keeping blocked for manual review\n > _{:white_circle: LOW RISK} Nitpick: Since the `AlAnalyzer.HelpLink` for\n- thread Oc6EeXH- from @codacy-production: **needs-human** — no rule matched — keeping blocked for manual review\n > _{:white_circle: LOW RISK} Nitpick: Update diagnostic ID wildcards, cla\n- thread Oc6Efsyu from @coderabbitai: **needs-human** — no rule matched — keeping blocked for manual review\n > _⚠️ Potential issue_ | _🟡 Minor_ | _⚡ Quick win_ 🧩 Analysi\n- thread Oc6Efsyy from @coderabbitai: **needs-human** — no rule matched — keeping blocked for manual review\n > _⚠️ Potential issue_ | _🟠 Major_ | _🏗️ Heavy lift_ **Cross-repo publish order\n- thread Oc6Efsy3 from @coderabbitai: **needs-human** — no rule matched — keeping blocked for manual review\n > _⚠️ Potential issue_ | _🟠 Major_ | _⚡ Quick win_ 🧩 Analysi\n- thread Oc6EfszC from @coderabbitai: **needs-human** — no rule matched — keeping blocked for manual review\n > _⚠️ Potential issue_ | _🟠 Major_ | _⚡ Quick win_ **`equivalenceKey` should be \n- thread Oc6EfszF from @coderabbitai: **needs-human** — no rule matched — keeping blocked for manual review\n > _⚠️ Potential issue_ | _🟠 Major_ | _⚡ Quick win_ 🧩 Analysi\n- thread Oc6EfszL from @coderabbitai: **needs-human** — no rule matched — keeping blocked for manual review\n > _🛠️ Refactor suggestion_ | _🟠 Major_ | _🏗️ Heavy lift_ **Align provider with\n- thread Oc6EfszQ from @coderabbitai: **needs-human** — no rule matched — keeping blocked for manual review\n > _⚠️ Potential issue_ | _🔴 Critical_ | _⚡ Quick win_ **FixAll `equivalenceKey` \n- thread Oc6EfszT from @coderabbitai: **needs-human** — no rule matched — keeping blocked for manual review\n > _⚠️ Potential issue_ | _🟠 Major_ | _⚡ Quick win_ **`equivalenceKey` should use\n- thread Oc6EfszW from @coderabbitai: **needs-human** — no rule matched — keeping blocked for manual review\n > _⚠️ Potential issue_ | _🔴 Critical_ | _⚡ Quick win_ **Fix CodeAction contract \n- thread Oc6Efszd from @coderabbitai: **needs-human** — no rule matched — keeping blocked for manual review\n > _⚠️ Potential issue_ | _🟠 Major_ | _⚡ Quick win_ **Use analyzer DiagnosticId a\n- thread Oc6Efszj from @coderabbitai: **needs-human** — no rule matched — keeping blocked for manual review\n > _⚠️ Potential issue_ | _🟠 Major_ | _⚡ Quick win_ **FixAll equivalence key must\n- thread Oc6Efszs from @coderabbitai: **needs-human** — no rule matched — keeping blocked for manual review\n > _⚠️ Potential issue_ | _🟠 Major_ | _⚡ Quick win_ **Set equivalence key to `Al1\n- thread Oc6Efszw from @coderabbitai: **needs-human** — no rule matched — keeping blocked for manual review\n > _🛠️ Refactor suggestion_ | _🟠 Major_ | _🏗️ Heavy lift_ **Provider base type \n- thread Oc6Efszx from @coderabbitai: **needs-human** — no rule matched — keeping blocked for manual review\n > _⚠️ Potential issue_ | _🟠 Major_ | _⚡ Quick win_ **Use analyzer DiagnosticId a\n- thread Oc6Efsz0 from @coderabbitai: **needs-human** — no rule matched — keeping blocked for manual review\n > _🛠️ Refactor suggestion_ | _🟠 Major_ | _🏗️ Heavy lift_ **Code fix provider s\n- thread Oc6Efsz3 from @coderabbitai: **needs-human** — no rule matched — keeping blocked for manual review\n > _⚠️ Potential issue_ | _🟠 Major_ | _⚡ Quick win_ **Both code actions need anal\n- thread Oc6Efsz5 from @coderabbitai: **needs-human** — no rule matched — keeping blocked for manual review\n > _⚠️ Potential issue_ | _🟠 Major_ | _⚡ Quick win_ **Replace provider-name equiv\n- thread Oc6Efsz8 from @coderabbitai: **needs-human** — no rule matched — keeping blocked for manual review\n > _⚠️ Potential issue_ | _🟠 Major_ | _⚡ Quick win_ **Equivalence key should be `\n- thread Oc6Efsz- from @coderabbitai: **needs-human** — no rule matched — keeping blocked for manual review\n > _⚠️ Potential issue_ | _🟠 Major_ | _⚡ Quick win_ **Set equivalence key to `Al1\n- thread Oc6Efs0E from @coderabbitai: **needs-human** — no rule matched — keeping blocked for manual review\n > _⚠️ Potential issue_ | _🔴 Critical_ | _⚡ Quick win_ **equivalenceKey must be t\n- thread Oc6Efs0L from @coderabbitai: **needs-human** — no rule matched — keeping blocked for manual review\n > _⚠️ Potential issue_ | _🔴 Critical_ | _⚡ Quick win_ **equivalenceKey must be t\n- thread Oc6Efs0S from @coderabbitai: **needs-human** — no rule matched — keeping blocked for manual review\n > _⚠️ Potential issue_ | _🔴 Critical_ | _⚡ Quick win_ **equivalenceKey must be t\n- thread Oc6Efs0d from @coderabbitai: **needs-human** — no rule matched — keeping blocked for manual review\n > _⚠️ Potential issue_ | _🔴 Critical_ | _⚡ Quick win_ **equivalenceKey must be t\n- thread Oc6Efs0l from @coderabbitai: **needs-human** — no rule matched — keeping blocked for manual review\n > _⚠️ Potential issue_ | _🔴 Critical_ | _⚡ Quick win_ **equivalenceKey must be t\n- thread Oc6Efs0r from @coderabbitai: **needs-human** — no rule matched — keeping blocked for manual review\n > _⚠️ Potential issue_ | _🔴 Critical_ | _⚡ Quick win_ **equivalenceKey must be t\n- thread Oc6Efs02 from @coderabbitai: **needs-human** — no rule matched — keeping blocked for manual review\n > _⚠️ Potential issue_ | _🔴 Critical_ | _⚡ Quick win_ **equivalenceKey must be t\n- thread Oc6Efs1A from @coderabbitai: **needs-human** — no rule matched — keeping blocked for manual review\n > _⚠️ Potential issue_ | _🔴 Critical_ | _⚡ Quick win_ **equivalenceKey must be t\n- thread Oc6Efs1I from @coderabbitai: **needs-human** — no rule matched — keeping blocked for manual review\n > _⚠️ Potential issue_ | _🟠 Major_ | _⚡ Quick win_ **Use analyzer `DiagnosticId`\n- thread Oc6Efs1R from @coderabbitai: **needs-human** — no rule matched — keeping blocked for manual review\n > _⚠️ Potential issue_ | _🟠 Major_ | _⚡ Quick win_ **Set `equivalenceKey` to the\n- thread Oc6Efs1W from @coderabbitai: **needs-human** — no rule matched — keeping blocked for manual review\n > _⚠️ Potential issue_ | _🟠 Major_ | _⚡ Quick win_ **FixAll equivalence key shou\n- thread Oc6Efs1e from @coderabbitai: **needs-human** — no rule matched — keeping blocked for manual review\n > _⚠️ Potential issue_ | _🟠 Major_ | _⚡ Quick win_ **Use analyzer `DiagnosticId`\n- thread Oc6Efs1r from @coderabbitai: **needs-human** — no rule matched — keeping blocked for manual review\n > _⚠️ Potential issue_ | _🟠 Major_ | _⚡ Quick win_ **`equivalenceKey` should tra\n- thread Oc6Efs15 from @coderabbitai: **needs-human** — no rule matched — keeping blocked for manual review\n > _⚠️ Potential issue_ | _🟠 Major_ | _⚡ Quick win_ **FixAll key must be analyzer\n- thread Oc6Efs2H from @coderabbitai: **needs-human** — no rule matched — keeping blocked for manual review\n > _⚠️ Potential issue_ | _🟠 Major_ | _⚡ Quick win_ **Incorrect `equivalenceKey` \n- thread Oc6Efs2M from @coderabbitai: **needs-human** — no rule matched — keeping blocked for manual review\n > _⚠️ Potential issue_ | _🟠 Major_ | _⚡ Quick win_ **Align code action `equivale\n- thread Oc6Efs2W from @coderabbitai: **needs-human** — no rule matched — keeping blocked for manual review\n > _⚠️ Potential issue_ | _🟠 Major_ | _⚡ Quick win_ **equivalenceKey must use the\n- thread Oc6Efs2c from @coderabbitai: **needs-human** — no rule matched — keeping blocked for manual review\n > _⚠️ Potential issue_ | _🟠 Major_ | _⚡ Quick win_ **equivalenceKey must use the\n- thread Oc6Efs2k from @coderabbitai: **needs-human** — no rule matched — keeping blocked for manual review\n > _⚠️ Potential issue_ | _🟠 Major_ | _⚡ Quick win_ **equivalenceKey must use the\n- thread Oc6Efs2x from @coderabbitai: **needs-human** — no rule matched — keeping blocked for manual review\n > _⚠️ Potential issue_ | _🟠 Major_ | _⚡ Quick win_ **equivalenceKey must use the\n- thread Oc6Efs26 from @coderabbitai: **needs-human** — no rule matched — keeping blocked for manual review\n > _🧹 Nitpick_ | _🔵 Trivial_ | _🏗️ Heavy lift_ **Coordinate Version.props updat\n\n_Threads marked `needs-human` stay unresolved and block auto-merge._","**Actionable comments posted: 46**\n\n> [!CAUTION]\n> Some comments are outside the diff and can’t be posted inline due to platform limitations.\n> \n> \n> \n> \n> ⚠️ Outside diff range comments (3)
\n> \n> \n> docs/analyzer-confidence-audit-2026-05-23.md (1)
\n> \n> `1-73`: _⚠️ Potential issue_ | _🟡 Minor_ | _⚡ Quick win_\n> \n> **Clarify temporal intent: frozen audit vs living analysis.**\n> \n> This audit was dated 2026-05-23, but now references the post-renumber AL1xxx IDs. If this is a historical snapshot taken before the renumber, the ID updates create a contradiction—the audit would be claiming it analyzed AL1703 in May when that ID didn't exist yet. If it's a living document maintained post-renumber, it needs a header note stating \"Updated after 2.0.0 renumber; original audit used AL0xxx IDs—see eng/analyzer-renumber-plan.md for mapping.\"\n> \n> Recommendation: Add a header block after line 1 clarifying whether this is a frozen historical record (in which case, revert to AL0xxx IDs and add a footer mapping to AL1xxx) or a living doc (in which case, add a note that it was updated post-renumber).\n> \n> \n> 🤖 Prompt for AI Agents
\n> \n> ```\n> Verify each finding against current code. Fix only still-valid issues, skip the\n> rest with a brief reason, keep changes minimal, and validate.\n> \n> In `@docs/analyzer-confidence-audit-2026-05-23.md` around lines 1 - 73, The\n> document titled \"Analyzer Confidence Audit - 2026-05-23\" currently mixes pre-\n> and post-renumber rule IDs (e.g., AL0140 vs AL1703) without stating whether this\n> is a frozen historical snapshot or a living, updated audit; add a brief header\n> paragraph under the main title that explicitly states which mode this file\n> represents (either \"Frozen: original audit using AL0xxx IDs — see\n> eng/analyzer-renumber-plan.md for AL1xxx mappings\" and revert displayed IDs to\n> AL0xxx, or \"Living: updated after 2.0.0 renumber; IDs are AL1xxx — original\n> AL0xxx IDs mapped in eng/analyzer-renumber-plan.md\"), and ensure the header\n> references the document name and the example rules AL0140/AL1703 so readers can\n> immediately see the mapping intent.\n> ```\n> \n>
\n> \n>
\n> \n> eng/analyzer-renumber-plan.md (1)
\n> \n> `481-489`: _🧹 Nitpick_ | _🔵 Trivial_ | _⚡ Quick win_\n> \n> **Flag this plan as frozen historical record post-merge.**\n> \n> Once the renumber PR merges, this 488-line plan becomes a historical artifact. Add a header note after line 1 (or in a follow-up commit) stating:\n> \n> ```markdown\n> > **Status:** Historical planning artifact. The renumber shipped in v2.0.0 on [date]. For the current AL1xxx rule catalog, see README.md. For the authoritative old→new mapping, see this file's section 2.\n> ```\n> \n> This prevents future readers from treating the \"Files to touch\" checklist as current action items.\n> \n> \n> 🤖 Prompt for AI Agents
\n> \n> ```\n> Verify each finding against current code. Fix only still-valid issues, skip the\n> rest with a brief reason, keep changes minimal, and validate.\n> \n> In `@eng/analyzer-renumber-plan.md` around lines 481 - 489, Add a clear \"Status:\n> Historical planning artifact\" header to the top of eng/analyzer-renumber-plan.md\n> indicating the renumber shipped in v2.0.0 (with date placeholder) and directing\n> readers to README.md for the current AL1xxx catalog and to this file's section 2\n> for the authoritative old→new mapping; update the file header (immediately after\n> the first line) to include that one-line note so future readers don't treat the\n> 488-line checklist as actionable.\n> ```\n> \n>
\n> \n>
\n> \n> src/ANcpLua.Analyzers.CodeFixes/CodeFixes/AL1703UseImplicitTypeWhenApparentCodeFixProvider.cs (1)
\n> \n> `24-27`: _⚠️ Potential issue_ | _🟠 Major_ | _⚡ Quick win_\n> \n> **equivalenceKey must use the analyzer's DiagnosticId for FixAll support.**\n> \n> Line 27 sets `equivalenceKey` to `UseImplicitTypeTitle` (a const string). Per coding guidelines, the equivalenceKey must be set to the analyzer's `DiagnosticId`.\n> \n> \n> \n> \n> \n> Proposed fix
\n> \n> ```diff\n> CodeAction.Create(\n> UseImplicitTypeTitle,\n> _ => UseImplicitType(document, root, syntax),\n> - UseImplicitTypeTitle);\n> + Al1703UseImplicitTypeWhenApparentAnalyzer.DiagnosticId);\n> ```\n>
\n> \n> As per coding guidelines: `CreateCodeAction method must set equivalenceKey to the analyzer's DiagnosticId to enable FixAll support`\n> \n> \n> 🤖 Prompt for AI Agents
\n> \n> ```\n> Verify each finding against current code. Fix only still-valid issues, skip the\n> rest with a brief reason, keep changes minimal, and validate.\n> \n> In\n> `@src/ANcpLua.Analyzers.CodeFixes/CodeFixes/AL1703UseImplicitTypeWhenApparentCodeFixProvider.cs`\n> around lines 24 - 27, The CodeAction created in CodeAction.Create currently uses\n> UseImplicitTypeTitle as the equivalenceKey which prevents FixAll from working;\n> change the equivalenceKey to the analyzer's DiagnosticId (use the DiagnosticId\n> constant/field from this analyzer) while keeping the title and callback the same\n> — update the CodeAction.Create call that references UseImplicitTypeTitle to pass\n> the analyzer DiagnosticId as the third argument and ensure any helper\n> CreateCodeAction wrappers follow the same pattern; leave the\n> UseImplicitType(document, root, syntax) invocation and UseImplicitTypeTitle\n> unchanged for UI text.\n> ```\n> \n>
\n> \n>
\n> \n>
\n\n\n♻️ Duplicate comments (2)
\n\n\nCHANGELOG.md (1)
\n\n`11-11`: _🧹 Nitpick_ | _🔵 Trivial_\n\n**HelpLink URL redirect strategy is a docs-site deployment concern, not a CHANGELOG issue.**\n\nThe past review comment flagging redirect strategy for old `AL0xxx` URL paths is valid as operational advice, but implementing redirects is outside the scope of this repo's CHANGELOG. The changelog correctly documents the URL format change; ensure the Mintlify docs site deployment includes a redirect map (e.g., `AL0001` → `AL1000`) before publishing 2.0.0.\n\n\n🤖 Prompt for AI Agents
\n\n```\nVerify each finding against current code. Fix only still-valid issues, skip the\nrest with a brief reason, keep changes minimal, and validate.\n\nIn `@CHANGELOG.md` at line 11, Remove operational redirect guidance from\nCHANGELOG.md and keep only the factual note that help-link URLs now use the new\nIDs (e.g., AL1000); relocate the redirect/implementation instruction about\nmapping old AL0xxx → AL1000 to deployment/ops documentation or the release\nchecklist so the changelog remains a user-facing record and not an operational\ntask.\n```\n\n
\n\n
\n\nAGENTS.md (1)
\n\n`38-44`: _🛠️ Refactor suggestion_ | _🟠 Major_ | _⚡ Quick win_\n\n**Update template placeholders from AL0XXX to AL1XXX range.**\n\nTemplate examples and wildcards still use the legacy `AL0XXX`/`AL00XX` range (e.g., lines 38, 41, 44, 55–56, 78–79, 88, 124–126, 134, 245, 249, 276–277), but the 2.0.0 renumber reserves `AL0xxx` for sibling packages and moves this repo's rules to `AL1xxx` bands. Update placeholders to use the new range (e.g., `AL1XXX*.cs`, `Al1xxxDescriptiveNameAnalyzer`, `\"AL1XXX\"`) to avoid confusion.\n\n\n\n\n\nProposed template updates
\n\n```diff\n- Analyzers/AL0XXX*.cs\n+ Analyzers/AL1XXX*.cs\n```\n\n```diff\n- CodeFixes/AL0XXX*.cs\n+ CodeFixes/AL1XXX*.cs\n```\n\n```diff\n- AL0XXX*Tests.cs\n+ AL1XXX*Tests.cs\n```\n\n```diff\n-public sealed partial class Al00xxDescriptiveNameAnalyzer : AlAnalyzer {\n- private const string DiagnosticId = \"AL00XX\";\n+public sealed partial class Al1xxxDescriptiveNameAnalyzer : AlAnalyzer {\n+ private const string DiagnosticId = \"AL1XXX\";\n```\n\n```diff\n-| File | `Analyzers/AL00XXDescriptiveNameAnalyzer.cs` |\n-| Class | `Al00xxDescriptiveNameAnalyzer`; use `Al`, not `AL` |\n+| File | `Analyzers/AL1XXXDescriptiveNameAnalyzer.cs` |\n+| Class | `Al1xxxDescriptiveNameAnalyzer`; use `Al`, not `AL` |\n```\n\n```diff\n-AL00XXAnalyzerTitle\n-AL00XXAnalyzerMessageFormat\n-AL00XXAnalyzerDescription\n+AL1XXXAnalyzerTitle\n+AL1XXXAnalyzerMessageFormat\n+AL1XXXAnalyzerDescription\n```\n\n```diff\n-AL00XX | Usage | Warning | Al00xxDescriptiveNameAnalyzer\n+AL1XXX | Usage | Warning | Al1xxxDescriptiveNameAnalyzer\n```\n\n```diff\n-using AnalyzerTestBase = ANcpLua.Roslyn.Utilities.Testing.AnalyzerTest;\n+using AnalyzerTestBase = ANcpLua.Roslyn.Utilities.Testing.AnalyzerTest;\n```\n\n```diff\n-public sealed partial class Al00xxDescriptiveNameTests : AnalyzerTestBase {\n+public sealed partial class Al1xxxDescriptiveNameTests : AnalyzerTestBase {\n```\n\n```diff\n-| File | `tests/ANcpLua.Analyzers.Tests/AL00XXDescriptiveNameTests.cs` |\n-| Class | `Al00xxDescriptiveNameTests` |\n+| File | `tests/ANcpLua.Analyzers.Tests/AL1XXXDescriptiveNameTests.cs` |\n+| Class | `Al1xxxDescriptiveNameTests` |\n```\n
\n\n\nAlso applies to: 55-56, 78-79, 88-88, 124-126, 134-134, 245-245, 249-249, 276-277\n\n\n🤖 Prompt for AI Agents
\n\n```\nVerify each finding against current code. Fix only still-valid issues, skip the\nrest with a brief reason, keep changes minimal, and validate.\n\nIn `@AGENTS.md` around lines 38 - 44, Update every template placeholder and\nwildcard that currently uses the AL0XXX/AL00XX range to the AL1XXX/AL1xxx range:\nreplace patterns like \"AL0XXX*.cs\", \"AL00XX\", analyzer class names such as\n\"Al0xxxDescriptiveNameAnalyzer\" and string IDs like \"AL0XXX\" with their AL1XXX\nequivalents (e.g., \"AL1XXX*.cs\", \"Al1xxxDescriptiveNameAnalyzer\", \"AL1XXX\").\nApply this change across all mentioned locations (lines referenced in the\ncomment) including files/lists under Analyzers, CodeFixes, Refactorings, tests,\nand any template examples so the repo uses the AL1xxx numbering consistently.\nEnsure casing is preserved where appropriate (e.g., class names) and update any\nquoted IDs in templates/documentation to the AL1XXX form.\n```\n\n
\n\n
\n\n
\n\n\n🤖 Prompt for all review comments with AI agents
\n\n```\nVerify each finding against current code. Fix only still-valid issues, skip the\nrest with a brief reason, keep changes minimal, and validate.\n\nInline comments:\nIn @.editorconfig:\n- Around line 1736-1743: The editorconfig contains a dead scoped suppression\nblock for Semconv files: locate the section with the header\n\"[src/ANcpLua.Analyzers/Semconv/*.cs]\" and the setting\n\"dotnet_diagnostic.AL1210.severity = none\" and either remove this entire scoped\nblock and its surrounding comment or change the glob to the actual Semconv\ncatalog path (or a broader pattern) so the AL1210 suppression applies where the\nSemconv .cs files actually live; ensure the change keeps the comment/context\nconsistent with other .editorconfig entries.\n\nIn `@eng/analyzer-renumber-plan.md`:\n- Around line 435-448: Update Section 8 of eng/analyzer-renumber-plan.md to\nexplicitly pick a release strategy and warn about the config-gap: state whether\nyou will (A) publish ANcpLua.Analyzers 2.0.0, then update\nANcpLua.NET.Sdk/src/Config/Analyzer.ANcpLua.Analyzers.editorconfig and bump the\nANcpLuaAnalyzersVersion in the SDK and publish the SDK (and include an explicit\nwarning to consumers about the transient broken dotnet_diagnostic.AL####\nseverities), or (B) perform a coordinated simultaneous release of\nANcpLua.Analyzers and ANcpLua.NET.Sdk with lockstep pins to avoid the gap;\ninclude the chosen path, the exact steps (publish analyzers → update SDK\neditorconfig → bump ANcpLuaAnalyzersVersion → publish SDK, or coordinated\npublish), and a short consumer-facing warning text about the config-gap window\nif option A is chosen.\n\nIn\n`@src/ANcpLua.Analyzers.CodeFixes/CodeFixes/AL1001SimplifyNegatedPatternCodeFixProvider.cs`:\n- Around line 20-23: In AL1001SimplifyNegatedPatternCodeFixProvider, the\nCodeAction.Create call currently uses\nnameof(CodeFixResources.AL1001CodeFixTitle) as the equivalenceKey which prevents\nstable FixAll grouping; update that third argument to use\nAl1001DontRepeatNegatedPatternAnalyzer.DiagnosticId instead so all fixes for\nthis diagnostic share the same equivalence key (locate the\nCodeAction.Create(...) call that returns a CodeAction and the\nRemoveRepeatedNegatedPatterns(...) delegate and replace the equivalenceKey\naccordingly).\n\nIn\n`@src/ANcpLua.Analyzers.CodeFixes/CodeFixes/AL1003SpanPatternMatchingCodeFixProvider.cs`:\n- Around line 16-19: The CodeAction.Create call is using\nnameof(CodeFixResources.AL1003CodeFixTitle) as the equivalenceKey; change it to\nuse the analyzer diagnostic ID instead (e.g., use diagnostic.Id or the\nanalyzer's DiagnosticId constant such as AL1003Analyzer.DiagnosticId) so FixAll\nworks correctly; keep the title and the UsePatternMatching(document, syntax,\nroot) callback the same and only replace the equivalenceKey argument.\n\nIn\n`@src/ANcpLua.Analyzers.CodeFixes/CodeFixes/AL1004UseSequenceEqualCodeFixProvider.cs`:\n- Around line 16-19: The CodeAction created in\nAL1004UseSequenceEqualCodeFixProvider.cs currently uses\nnameof(CodeFixResources.AL1004CodeFixTitle) as the equivalenceKey; change that\nthird parameter in the CodeAction.Create call to use the analyzer's public\ndiagnostic id Al1003ToAL1004SpanComparisonAnalyzer.DiagnosticIdAl1004 (i.e.\n\"AL1004\") so FixAll groups correctly, referencing the CodeAction.Create\ninvocation and the Al1003ToAL1004SpanComparisonAnalyzer.DiagnosticIdAl1004\nsymbol.\n\nIn\n`@src/ANcpLua.Analyzers.CodeFixes/CodeFixes/AL1007IXmlSerializableCodeFixProvider.cs`:\n- Around line 54-57: The CodeAction in AL1007IXmlSerializableCodeFixProvider is\nusing nameof(CodeFixResources.AL1007CodeFixTitle) as the equivalenceKey which\nbreaks FixAll; update the CodeAction.Create call to use the analyzer diagnostic\nID (DiagnosticIdAl1007) as the equivalenceKey instead so FixAll targets only\nAL1007, i.e., replace the third argument in the CodeAction.Create invocation\nwith the DiagnosticIdAl1007 symbol (or the constant defined in the corresponding\nanalyzer) while keeping the title and the FixAsync(context.Document, target,\nroot) callback unchanged.\n- Line 10: The class currently inherits directly from CodeFixProvider; change\nits declaration to inherit from\nAlCodeFixProvider (i.e., replace\n\"CodeFixProvider\" with\n\"AlCodeFixProvider\") and decorate the\nclass with [ExportCodeFixProvider(LanguageNames.CSharp)] and [Shared]\nattributes; also add any missing using directives for\nMicrosoft.CodeAnalysis.CodeFixes and System.Composition so the attributes and\nbase type resolve. Ensure all existing overrides match the AlCodeFixProvider\ncontract (adjust signatures if the base generic requires them).\n\nIn `@src/ANcpLua.Analyzers.CodeFixes/CodeFixes/AL1009LockTypeCodeFixProvider.cs`:\n- Around line 44-47: The CodeAction Create call currently sets equivalenceKey to\nnameof(CodeFixResources.AL1009CodeFixTitle) which breaks FixAll identity; change\nthe equivalenceKey argument in the CodeAction.Create invocation that constructs\nthe fix for ChangeFieldTypeToLockAsync (the CodeAction created with\nCodeFixResources.AL1009CodeFixTitle) to use\nAl1009LockKeywordAnalyzer.DiagnosticId instead so the fix’s equivalence key\nequals the analyzer DiagnosticId.\n\nIn\n`@src/ANcpLua.Analyzers.CodeFixes/CodeFixes/AL1010PatternMatchingCodeFixProvider.cs`:\n- Around line 18-21: Replace the hardcoded CodeAction title and equivalenceKey\nin AL1010PatternMatchingCodeFixProvider with the repo convention: use the\nresource-backed title from CodeFixResources (the entry that corresponds to\nAL1010) as the first argument to CodeAction.Create, and use the analyzer\nDiagnosticId for the equivalenceKey (not\nnameof(Al1010PatternMatchingCodeFixProvider)); update the existing\nCodeAction.Create call that wraps ConvertToPatternMatching(document, binary,\nroot) so its title references the appropriate CodeFixResources constant and its\nequivalenceKey is the AL1010 analyzer DiagnosticId.\n\nIn\n`@src/ANcpLua.Analyzers.CodeFixes/CodeFixes/AL1011NormalizeNullGuardStyleCodeFixProvider.cs`:\n- Around line 34-37: The code action currently uses\nnameof(Al1011NormalizeNullGuardStyleCodeFixProvider) as the equivalenceKey which\nbreaks FixAll; change the equivalenceKey passed to CodeAction.Create to the\nanalyzer's DiagnosticId (e.g., use the DiagnosticId constant or descriptor.Id\nfor AL1011) instead of the provider type name so the Create call uses the\nanalyzer DiagnosticId as the equivalence key; update the CodeAction.Create\ninvocation in AL1011NormalizeNullGuardStyleCodeFixProvider to pass the\nDiagnosticId value.\n\nIn\n`@src/ANcpLua.Analyzers.CodeFixes/CodeFixes/AL1012CombineDeclarationWithNullCheckCodeFixProvider.cs`:\n- Around line 35-38: The CodeAction equivalenceKey is currently set to\nnameof(Al1012CombineDeclarationWithNullCheckCodeFixProvider) which prevents\nFixAll from working; change the third argument in the CodeAction.Create call to\nthe analyzer's DiagnosticId (for example use Al1012Analyzer.DiagnosticId or the\nconstant that holds \"AL1012\") so the equivalenceKey equals the analyzer ID; keep\nthe rest of the CodeAction.Create call (title and ct lambda invoking\nCombineAsync) unchanged.\n\nIn\n`@src/ANcpLua.Analyzers.CodeFixes/CodeFixes/AL1200UseIsEqualToCodeFixProvider.cs`:\n- Around line 21-24: The CodeAction.Create call in\nAl1200UseIsEqualToCodeFixProvider currently uses\nnameof(Al1200UseIsEqualToCodeFixProvider) as the equivalence key which breaks\nFixAll grouping; update that CodeAction.Create invocation to pass\nAl1200UseIsEqualToAnalyzer.DiagnosticId as the equivalenceKey instead (keep the\ntitle and the callback to ConvertToIsEqualTo(document, invocation, root)\nunchanged) so FixAll uses the analyzer's DiagnosticId for grouping.\n\nIn\n`@src/ANcpLua.Analyzers.CodeFixes/CodeFixes/AL1201UseHasAttributeCodeFixProvider.cs`:\n- Around line 32-35: The CodeAction.Create call currently uses\nnameof(Al1201UseHasAttributeCodeFixProvider) as the equivalenceKey which breaks\nFixAll; change the equivalenceKey to the analyzer's DiagnosticId (e.g., use the\nAL1201 analyzer's DiagnosticId constant/property) when creating the CodeAction\nfor ConvertToHasAttribute so the equivalence key equals the analyzer\nDiagnosticId (AL1201) instead of the codefix type name.\n- Around line 14-15: The class Al1201UseHasAttributeCodeFixProvider currently\ninherits from CodeFixProvider; change it to inherit from the repo base\nAlCodeFixProvider with the appropriate syntax node type used by this\nanalyzer (replace CodeFixProvider with AlCodeFixProvider using the\nanalyzer's node type), and add the required attributes\n[ExportCodeFixProvider(LanguageNames.CSharp)] and [Shared] to the class\ndeclaration so it follows the project's code-fix contract.\n\nIn\n`@src/ANcpLua.Analyzers.CodeFixes/CodeFixes/AL1202UseTypeHierarchyCodeFixProvider.cs`:\n- Around line 15-16: The class Al1202UseTypeHierarchyCodeFixProvider currently\ninherits directly from CodeFixProvider; change it to inherit from\nAlCodeFixProvider and apply the\n[ExportCodeFixProvider(LanguageNames.CSharp)] and [Shared] attributes above the\nclass declaration so it matches the repository provider shape; ensure you\npreserve existing members like FixableDiagnosticIds and any overrides required\nby AlCodeFixProvider.\n- Around line 35-38: The CodeAction.Create calls (the one creating the\nimplements action that calls ConvertToImplements and the other corresponding\naction on lines 45-48) currently use a provider-suffixed string for equivalence\nkeys which breaks FixAll; change their equivalenceKey arguments to the analyzer\ndiagnostic id (AL1202) instead of nameof(Al1202UseTypeHierarchyCodeFixProvider)\n+ \"_Implements\" (and the similar suffix used for the other action). Use the\nconstant/diagnostic id (e.g., DiagnosticIds.AL1202 or context.Diagnostics[0].Id)\nas the equivalenceKey when constructing the CodeAction so FixAll will work,\nleaving the title and the delegate (ConvertToImplements / the other conversion\nmethod) unchanged.\n\nIn\n`@src/ANcpLua.Analyzers.CodeFixes/CodeFixes/AL1203UseOperationExtensionsCodeFixProvider.cs`:\n- Around line 31-34: The CodeAction.Create calls in\nAl1203UseOperationExtensionsCodeFixProvider currently use provider-name-derived\nequivalence keys (e.g., nameof(Al1203UseOperationExtensionsCodeFixProvider) +\n\"_IsMethodNamed\"); change both CodeAction.Create invocations (the one that calls\nConvertToIsMethodNamed and the other that calls ConvertToIsOperationNamed) to\nuse Al1203UseOperationExtensionsAnalyzer.DiagnosticId as the equivalenceKey so\nFixAll works correctly.\n\nIn\n`@src/ANcpLua.Analyzers.CodeFixes/CodeFixes/AL1204UseOrEmptyCodeFixProvider.cs`:\n- Around line 28-31: The CodeAction created in\nAL1204UseOrEmptyCodeFixProvider.cs currently uses\nnameof(Al1204UseOrEmptyCodeFixProvider) as the equivalence key; change the\nCodeAction.Create call that wraps ConvertToOrEmpty(document, coalesce, root) so\nits equivalenceKey is Al1204UseOrEmptyAnalyzer.DiagnosticId instead (i.e., pass\nAl1204UseOrEmptyAnalyzer.DiagnosticId as the third argument) to enable proper\nFixAll behavior and keep ConvertToOrEmpty and the surrounding method intact.\n\nIn\n`@src/ANcpLua.Analyzers.CodeFixes/CodeFixes/AL1205UseToImmutableArrayOrEmptyCodeFixProvider.cs`:\n- Around line 25-28: The CodeAction.Create call in\nAl1205UseToImmutableArrayOrEmptyCodeFixProvider is using\nnameof(Al1205UseToImmutableArrayOrEmptyCodeFixProvider) as the equivalence key;\nchange that last argument to\nAl1205UseToImmutableArrayOrEmptyAnalyzer.DiagnosticId so the created CodeAction\nuses the analyzer's DiagnosticId for equivalence (leave the title and the\ndelegate to ConvertToExtension(document, coalesce, root) unchanged).\n\nIn\n`@src/ANcpLua.Analyzers.CodeFixes/CodeFixes/AL1206UseWhereNotNullCodeFixProvider.cs`:\n- Around line 27-30: The CodeAction created in\nAl1206UseWhereNotNullCodeFixProvider currently uses\nnameof(Al1206UseWhereNotNullCodeFixProvider) as the equivalenceKey; change that\nthird argument to Al1206UseWhereNotNullAnalyzer.DiagnosticId so the\nCodeAction.Create call (the same call that wraps ConvertToWhereNotNull) uses the\nanalyzer's DiagnosticId as the equivalenceKey to enable correct FixAll grouping.\n\nIn\n`@src/ANcpLua.Analyzers.CodeFixes/CodeFixes/AL1207UseToDisplayStringExtensionsCodeFixProvider.cs`:\n- Around line 39-42: The CodeAction.Create call in\nAL1207UseToDisplayStringExtensionsCodeFixProvider is using\nnameof(Al1207UseToDisplayStringExtensionsCodeFixProvider) as the equivalenceKey;\nchange it to use the analyzer's DiagnosticId\n(Al1207UseToDisplayStringExtensionsAnalyzer.DiagnosticId) so FixAll works\ncorrectly. Locate the CodeAction.Create invocation (the call that passes\nCodeFixResources.AL1207CodeFixTitle and the ConvertToExtension lambda) and\nreplace the equivalenceKey argument with\nAl1207UseToDisplayStringExtensionsAnalyzer.DiagnosticId.\n\nIn\n`@src/ANcpLua.Analyzers.CodeFixes/CodeFixes/AL1208UseGuardNotNullCodeFixProvider.cs`:\n- Around line 27-30: The CodeAction call uses\nnameof(Al1208UseGuardNotNullCodeFixProvider) as the equivalenceKey which breaks\nFixAll; change the equivalenceKey argument in the CodeAction.Create invocation\n(the call that passes CodeFixResources.AL1208CodeFixTitle and _ =>\nConvertToGuardNotNull(document, coalesce, root)) to use\nAl1208UseGuardNotNullAnalyzer.DiagnosticId instead of\nnameof(Al1208UseGuardNotNullCodeFixProvider) so the action’s equivalenceKey\nmatches the analyzer diagnostic ID and enables FixAll correctness.\n\nIn\n`@src/ANcpLua.Analyzers.CodeFixes/CodeFixes/AL1209UseTryParseExtensionsCodeFixProvider.cs`:\n- Around line 27-30: The CodeAction.Create call currently uses\nnameof(Al1209UseTryParseExtensionsCodeFixProvider) as the equivalenceKey; change\nit to use Al1209UseTryParseExtensionsAnalyzer.DiagnosticId instead so FixAll\nworks correctly — update the CodeAction.Create invocation that passes\nAL1209CodeFixTitle and the ConvertToExtension callback to set the third argument\nto Al1209UseTryParseExtensionsAnalyzer.DiagnosticId (leave the title and\ncallback unchanged).\n\nIn\n`@src/ANcpLua.Analyzers.CodeFixes/CodeFixes/AL1210UseStringComparisonExtensionsCodeFixProvider.cs`:\n- Around line 39-42: The CodeAction.Create call currently uses\nnameof(Al1210UseStringComparisonExtensionsCodeFixProvider) as the\nequivalenceKey; change it to use\nAl1210UseStringComparisonExtensionsAnalyzer.DiagnosticId instead so FixAll works\ncorrectly—update the call that constructs the CodeAction (the one passing\nCodeFixResources.AL1210CodeFixTitle and the lambda that calls\nConvertToExtensionMethod(document, invocation, root)) to set equivalenceKey to\nAl1210UseStringComparisonExtensionsAnalyzer.DiagnosticId rather than the\nprovider name.\n\nIn\n`@src/ANcpLua.Analyzers.CodeFixes/CodeFixes/AL1211UseAttributeExtensionsCodeFixProvider.cs`:\n- Around line 33-36: The CodeAction created in\nAl1211UseAttributeExtensionsCodeFixProvider currently uses\nnameof(Al1211UseAttributeExtensionsCodeFixProvider) as the equivalenceKey;\nchange it to use Al1211UseAttributeExtensionsAnalyzer.DiagnosticId instead so\nFixAll works correctly. Locate the CodeAction.Create call (and related\noverloads) around where ConvertToGetConstructorArgument(...) is passed and\nreplace the equivalenceKey argument with\nAl1211UseAttributeExtensionsAnalyzer.DiagnosticId, ensuring the symbol is\nreferenced/imported if necessary.\n\nIn\n`@src/ANcpLua.Analyzers.CodeFixes/CodeFixes/AL1212UseGuardNotNullOrEmptyCodeFixProvider.cs`:\n- Around line 33-36: The CodeAction created in the CodeAction.Create call\ncurrently uses nameof(Al1212UseGuardNotNullOrEmptyCodeFixProvider) as the\nequivalenceKey; change it to use\nAl1212UseGuardNotNullOrEmptyAnalyzer.DiagnosticId instead so FixAll works\ncorrectly — update the CodeAction.Create invocation that supplies\nAL1212CodeFixTitle and the ConvertToGuardNotNullOrEmpty(document, ifStatement,\nroot) delegate to pass Al1212UseGuardNotNullOrEmptyAnalyzer.DiagnosticId as the\nequivalenceKey.\n\nIn\n`@src/ANcpLua.Analyzers.CodeFixes/CodeFixes/AL1213UseGuardNotNullOrWhiteSpaceCodeFixProvider.cs`:\n- Around line 27-30: The CodeAction is created with the provider name as the\nequivalenceKey which breaks FixAll; change the equivalenceKey argument in the\nCodeAction.Create call (the call that uses AL1213CodeFixTitle and\nConvertToGuardNotNullOrWhiteSpace) from\nnameof(Al1213UseGuardNotNullOrWhiteSpaceCodeFixProvider) to\nAl1213UseGuardNotNullOrWhiteSpaceAnalyzer.DiagnosticId so the action uses the\nanalyzer's DiagnosticId for correct FixAll behavior.\n\nIn\n`@src/ANcpLua.Analyzers.CodeFixes/CodeFixes/AL1214UseGuardNotZeroCodeFixProvider.cs`:\n- Around line 37-39: The FixAll equivalence key is currently set to\nnameof(Al1214UseGuardNotZeroCodeFixProvider) which can break FixAll bucketing;\nupdate the RegisterCodeFix call in Al1214UseGuardNotZeroCodeFixProvider to use\nAl1214UseGuardNotZeroAnalyzer.DiagnosticId as the equivalenceKey instead of\nnameof(Al1214UseGuardNotZeroCodeFixProvider) so FixAll uses the analyzer\ndiagnostic id for consistent grouping.\n\nIn\n`@src/ANcpLua.Analyzers.CodeFixes/CodeFixes/AL1215UseGuardNotNegativeCodeFixProvider.cs`:\n- Around line 28-30: The RegisterCodeFix call in\nAl1215UseGuardNotNegativeCodeFixProvider currently uses\nnameof(Al1215UseGuardNotNegativeCodeFixProvider) as the equivalenceKey; change\nthat argument to Al1215UseGuardNotNegativeAnalyzer.DiagnosticId so the\nequivalenceKey matches the analyzer ID (keep the other args the same, i.e.,\nCodeFixResources.AL1215CodeFixTitle and the _ =>\nConvertToGuardNotNegative(document, ifStatement, root) delegate).\n\nIn\n`@src/ANcpLua.Analyzers.CodeFixes/CodeFixes/AL1216UseGuardPositiveCodeFixProvider.cs`:\n- Around line 30-32: The FixAll equivalence key is currently set to the provider\nname via nameof(Al1216UseGuardPositiveCodeFixProvider); update the third\nargument of the RegisterCodeFix call to use the analyzer diagnostic ID by\nreplacing the equivalenceKey with Al1216UseGuardPositiveAnalyzer.DiagnosticId so\nFixAll uses the analyzer's DiagnosticId; locate the RegisterCodeFix invocation\nin AL1216UseGuardPositiveCodeFixProvider and change the equivalenceKey\naccordingly.\n\nIn\n`@src/ANcpLua.Analyzers.CodeFixes/CodeFixes/AL1217UseGuardNotEmptyGuidCodeFixProvider.cs`:\n- Around line 34-36: The equivalenceKey passed to RegisterCodeFix is currently\nthe code fix provider class name; change it to use the analyzer DiagnosticId\ninstead: replace the nameof(Al1217UseGuardNotEmptyGuidCodeFixProvider)\nequivalenceKey with Al1217UseGuardNotEmptyGuidAnalyzer.DiagnosticId so the\nFixAll operation keys correctly for the ConvertToGuardNotEmpty code action\nregistered in Al1217UseGuardNotEmptyGuidCodeFixProvider.\n\nIn\n`@src/ANcpLua.Analyzers.CodeFixes/CodeFixes/AL1218UseGuardDefinedEnumCodeFixProvider.cs`:\n- Around line 28-30: The equivalenceKey currently uses\nnameof(Al1218UseGuardDefinedEnumCodeFixProvider) which breaks FixAll; change the\nequivalenceKey to the analyzer diagnostic id by replacing the third argument\nfrom nameof(Al1218UseGuardDefinedEnumCodeFixProvider) to\nAl1218UseGuardDefinedEnumAnalyzer.DiagnosticId in the CodeAction.Create call\nthat invokes ConvertToGuardDefinedEnum(document, ifStatement, root) inside\nAl1218UseGuardDefinedEnumCodeFixProvider so FixAll groups diagnostics by the\nanalyzer ID rather than the provider name.\n\nIn\n`@src/ANcpLua.Analyzers.CodeFixes/CodeFixes/AL1220UseGuardForThrowIfCodeFixProvider.cs`:\n- Around line 34-36: The FixAll equivalence key is incorrectly set to the\nprovider name; update the registration call in\nAL1220UseGuardForThrowIfCodeFixProvider (the RegisterCodeFix invocation that\ncurrently uses nameof(Al1220UseGuardForThrowIfCodeFixProvider)) to use\nAl1220UseGuardForThrowIfAnalyzer.DiagnosticId as the equivalenceKey instead so\nthe fix is keyed by the analyzer DiagnosticId; keep the title\n(CodeFixResources.AL1220CodeFixTitle) and callback (ConvertToGuard) the same.\n\nIn\n`@src/ANcpLua.Analyzers.CodeFixes/CodeFixes/AL1313CancellationTokenPropagationCodeFixProvider.cs`:\n- Around line 23-25: The CodeAction in\nAL1313CancellationTokenPropagationCodeFixProvider is using\nCodeFixResources.AL1313CodeFixTitle as the equivalenceKey when creating the fix;\nchange the equivalenceKey to the analyzer ID by passing\nAl1313CancellationTokenPropagationAnalyzer.DiagnosticId instead (update the\nCodeAction creation that calls AddCancellationTokenAsync/document\ninvocation/root to use Al1313CancellationTokenPropagationAnalyzer.DiagnosticId\nas the third argument).\n\nIn\n`@src/ANcpLua.Analyzers.CodeFixes/CodeFixes/AL1314UseExplicitMidpointRoundingCodeFixProvider.cs`:\n- Around line 30-32: The code action's equivalenceKey is currently set using\nnameof(Al1314UseExplicitMidpointRoundingCodeFixProvider); change it to use\nAl1314UseExplicitMidpointRoundingAnalyzer.DiagnosticId so the equivalenceKey\naligns with the analyzer ID for FixAll support—update the invocation that\nregisters the code action (the call passing CodeFixResources.AL1314CodeFixTitle,\n_ => AppendMidpointRounding(document, invocation, root), nameof(...)) to replace\nthe nameof(...) argument with\nAl1314UseExplicitMidpointRoundingAnalyzer.DiagnosticId.\n\nIn\n`@src/ANcpLua.Analyzers.CodeFixes/CodeFixes/AL1500AddMissingCasesCodeFixProvider.cs`:\n- Around line 28-39: The CodeAction.Create calls that build fixes for\nSwitchExpressionSyntax and SwitchStatementSyntax currently set equivalenceKey to\nnameof(CodeFixResources.AL1500CodeFixTitle); change both equivalenceKey values\nto the analyzer's DiagnosticId string so FixAll groups correctly (use the ID\nconstant your analyzer exposes, e.g., DiagnosticIds.AL1500 or\nAL1500Analyzer.DiagnosticId) in the two CodeAction.Create invocations that call\nFixSwitchExpression and FixSwitchStatement and keep the rest of the signature\nthe same.\n\nIn\n`@src/ANcpLua.Analyzers.CodeFixes/CodeFixes/AL1503NormalizeWhitespaceCodeFixProvider.cs`:\n- Around line 32-38: The CodeAction created in context.RegisterCodeFix uses\nnameof(CodeFixResources.AL1503CodeFixTitle) as the equivalenceKey; change the\nthird argument of CodeAction.Create in the RegisterCodeFix call so the\nequivalenceKey is the analyzer diagnostic id (use diagnostic.Id) instead of the\nresource name to enable correct FixAll grouping for the AL1503 fix handled by\nRemoveNormalizeWhitespaceAsync.\n\nIn\n`@src/ANcpLua.Analyzers.CodeFixes/CodeFixes/AL1504DuckDbTableCodeFixProvider.cs`:\n- Around line 28-34: The CodeAction equivalenceKey is currently set to\nnameof(CodeFixResources.AL1504CodeFixTitle); change it to use the analyzer\ndiagnostic id so FixAll works: when creating the CodeAction in\ncontext.RegisterCodeFix (the CodeAction.Create call that invokes\nMakePartialAsync and uses CodeFixResources.AL1504CodeFixTitle), set the\nequivalenceKey to the Diagnostic.Id (e.g., diagnostic.Id or the analyzer's\nDiagnosticId constant) instead of the resource name.\n\nIn\n`@src/ANcpLua.Analyzers.CodeFixes/CodeFixes/AL1700StaticLambdaCodeFixProvider.cs`:\n- Around line 22-27: The CodeAction registered in\nAL1700StaticLambdaCodeFixProvider (context.RegisterCodeFix -> CodeAction.Create)\nuses CodeFixResources.AL1700CodeFixTitle as the equivalenceKey; change it to the\nanalyzer diagnostic id so FixAll works: when creating the CodeAction for\nMakeStaticAsync, pass the diagnostic's DiagnosticId (e.g., diagnostic.Id) as the\nequivalenceKey instead of nameof(CodeFixResources.AL1700CodeFixTitle) so the\naction's equivalenceKey matches the analyzer DiagnosticId.\n\nIn\n`@src/ANcpLua.Analyzers.CodeFixes/CodeFixes/AL1701DateTimeNowCodeFixProvider.cs`:\n- Around line 22-25: The CodeAction.Create call in\nAL1701DateTimeNowCodeFixProvider currently sets the equivalenceKey to the\nprovider class name; change it to use the analyzer's DiagnosticId to enable\nFixAll support by replacing the current equivalenceKey\n(nameof(Al1701DateTimeNowCodeFixProvider)) with the diagnostic's ID (e.g.,\ndiagnostic.Id or the analyzer's DiagnosticId constant such as\nAl1701Analyzer.DiagnosticId) in the CodeAction.Create invocation that constructs\nthe fix for ConvertToTimeProvider.\n\nIn\n`@src/ANcpLua.Analyzers.CodeFixes/CodeFixes/AL1702UseSystemTextJsonCodeFixProvider.cs`:\n- Around line 31-37: The CodeAction equivalenceKey is currently set to the\nprovider class name (nameof(Al1702UseSystemTextJsonCodeFixProvider)), which\nbreaks FixAll; change the equivalenceKey passed to CodeAction.Create to the\nanalyzer's DiagnosticId (use the Diagnostic.Id from the diagnostic parameter or\nthe analyzer constant) so the CodeAction's equivalenceKey matches the analyzer's\nDiagnosticId and enables FixAll support—update the call in the RegisterCodeFix\nblock where CodeAction.Create is invoked (the invocation that builds the\nCodeAction for ConvertToSystemTextJson) to supply diagnostic.Id instead of the\nprovider class name.\n\nIn `@src/ANcpLua.Analyzers/AnalyzerReleases.Unshipped.md`:\n- Around line 8-96: The release table adds many AL0xxx ID changes (see\nAnalyzerReleases.Unshipped.md and the AL0xxx entries) and downstream consumers\nthat rely on .editorconfig/NoWarn/test ID assertions will break unless migration\nis coordinated; before publishing 2.0.0, gate the rollout: prepare and publish\nthe SDK-packaged analyzer .editorconfig/update that maps old->new IDs first,\ncommunicate migration steps and timelines to dependent repos, and only then\npublish the analyzer package and announce the change so downstream repos can\nupdate NoWarn/test expectations in lockstep.\n\nIn `@tests/ANcpLua.Analyzers.Tests/AL1210UseStringComparisonExtensionsTests.cs`:\n- Around line 103-104: Rename the test class Al1210CodeFixTests to match the\nanalyzer name and team convention: Al1210UseStringComparisonExtensionsTests.\nUpdate the class declaration (the class identifier) so the class remains sealed\npartial and continues to inherit from\nCodeFixTest, and update any\nreferences/usages of Al1210CodeFixTests in the test project (e.g., test runners\nor other test helpers) to the new Al1210UseStringComparisonExtensionsTests\nidentifier.\n\nIn `@tests/ANcpLua.Analyzers.Tests/AL1220UseGuardForThrowIfTests.cs`:\n- Line 11: Replace the direct inheritance from\nAnalyzerTest in the\nAl1220UseGuardForThrowIfTests class with the local AnalyzerTestBase alias so the\nclass declares inheritance as\nAnalyzerTestBase; locate the class declaration\nline containing \"class Al1220UseGuardForThrowIfTests\" and update the base type\naccordingly (ensure the project has the AnalyzerTestBase alias\nimported/defined).\n\nIn `@tests/ANcpLua.Analyzers.Tests/AL1300ToAL1303ThreadingTests.cs`:\n- Line 9: Replace direct inheritance from\nAnalyzerTest with the standardized\nAnalyzerTestBase alias by changing the class declaration to inherit from\nAnalyzerTestBase, and add a file-level using\nalias declaration near the top of the file: define AnalyzerTestBase as an alias\nfor AnalyzerTest so the tests at symbols Al1300AnalyzerTests (and the other\nthree test classes at the referenced locations) use AnalyzerTestBase\ninstead of AnalyzerTest.\n\nIn `@Version.props`:\n- Line 61: Update the pinned analyzer version variable ANcpLuaAnalyzersVersion\nin Version.props from 1.29.4 to 2.0.0 after the new 2.0.0 package is published;\nthen make matching updates in the other repositories that reference this same\nvariable (ANcpLua.NET.Sdk, ErrorOrX, ANcpLua.Agents) so they pull the renumbered\ndiagnostics in lockstep, and add or update CI/scripted steps (or documentation)\nto enforce the publish-first then bump sequence to avoid restore failures.\n\n---\n\nOutside diff comments:\nIn `@docs/analyzer-confidence-audit-2026-05-23.md`:\n- Around line 1-73: The document titled \"Analyzer Confidence Audit - 2026-05-23\"\ncurrently mixes pre- and post-renumber rule IDs (e.g., AL0140 vs AL1703) without\nstating whether this is a frozen historical snapshot or a living, updated audit;\nadd a brief header paragraph under the main title that explicitly states which\nmode this file represents (either \"Frozen: original audit using AL0xxx IDs — see\neng/analyzer-renumber-plan.md for AL1xxx mappings\" and revert displayed IDs to\nAL0xxx, or \"Living: updated after 2.0.0 renumber; IDs are AL1xxx — original\nAL0xxx IDs mapped in eng/analyzer-renumber-plan.md\"), and ensure the header\nreferences the document name and the example rules AL0140/AL1703 so readers can\nimmediately see the mapping intent.\n\nIn `@eng/analyzer-renumber-plan.md`:\n- Around line 481-489: Add a clear \"Status: Historical planning artifact\" header\nto the top of eng/analyzer-renumber-plan.md indicating the renumber shipped in\nv2.0.0 (with date placeholder) and directing readers to README.md for the\ncurrent AL1xxx catalog and to this file's section 2 for the authoritative\nold→new mapping; update the file header (immediately after the first line) to\ninclude that one-line note so future readers don't treat the 488-line checklist\nas actionable.\n\nIn\n`@src/ANcpLua.Analyzers.CodeFixes/CodeFixes/AL1703UseImplicitTypeWhenApparentCodeFixProvider.cs`:\n- Around line 24-27: The CodeAction created in CodeAction.Create currently uses\nUseImplicitTypeTitle as the equivalenceKey which prevents FixAll from working;\nchange the equivalenceKey to the analyzer's DiagnosticId (use the DiagnosticId\nconstant/field from this analyzer) while keeping the title and callback the same\n— update the CodeAction.Create call that references UseImplicitTypeTitle to pass\nthe analyzer DiagnosticId as the third argument and ensure any helper\nCreateCodeAction wrappers follow the same pattern; leave the\nUseImplicitType(document, root, syntax) invocation and UseImplicitTypeTitle\nunchanged for UI text.\n\n---\n\nDuplicate comments:\nIn `@AGENTS.md`:\n- Around line 38-44: Update every template placeholder and wildcard that\ncurrently uses the AL0XXX/AL00XX range to the AL1XXX/AL1xxx range: replace\npatterns like \"AL0XXX*.cs\", \"AL00XX\", analyzer class names such as\n\"Al0xxxDescriptiveNameAnalyzer\" and string IDs like \"AL0XXX\" with their AL1XXX\nequivalents (e.g., \"AL1XXX*.cs\", \"Al1xxxDescriptiveNameAnalyzer\", \"AL1XXX\").\nApply this change across all mentioned locations (lines referenced in the\ncomment) including files/lists under Analyzers, CodeFixes, Refactorings, tests,\nand any template examples so the repo uses the AL1xxx numbering consistently.\nEnsure casing is preserved where appropriate (e.g., class names) and update any\nquoted IDs in templates/documentation to the AL1XXX form.\n\nIn `@CHANGELOG.md`:\n- Line 11: Remove operational redirect guidance from CHANGELOG.md and keep only\nthe factual note that help-link URLs now use the new IDs (e.g., AL1000);\nrelocate the redirect/implementation instruction about mapping old AL0xxx →\nAL1000 to deployment/ops documentation or the release checklist so the changelog\nremains a user-facing record and not an operational task.\n```\n\n
\n\n\n🪄 Autofix (Beta)
\n\nFix all unresolved CodeRabbit comments on this PR:\n\n- [ ] Push a commit to this branch (recommended)\n- [ ] Create a new PR with the fixes\n\n
\n\n---\n\n\nℹ️ Review info
\n\n\n⚙️ Run configuration
\n\n**Configuration used**: Path: .coderabbit.yaml\n\n**Review profile**: ASSERTIVE\n\n**Plan**: Pro Plus\n\n**Run ID**: `7900c6da-3b61-40be-96c6-423395f6c23c`\n\n
\n\n\n📥 Commits
\n\nReviewing files that changed from the base of the PR and between 9d7393e3cd69b133dba378c21ab669d301ea2ecb and fcc2a2b20a882827da9d94017eaa790efb73dd17.\n\n
\n\n\n📒 Files selected for processing (215)
\n\n* `.editorconfig`\n* `AGENTS.md`\n* `CHANGELOG.md`\n* `Directory.Build.props`\n* `Directory.Packages.props`\n* `README.md`\n* `Version.props`\n* `docs/Al1200UseIsEqualTo.md`\n* `docs/analyzer-confidence-audit-2026-05-23.md`\n* `eng/analyzer-renumber-plan.md`\n* `src/ANcpLua.Analyzers.AnalyzerDocs/ANcpLua.Analyzers.AnalyzerDocs.csproj`\n* `src/ANcpLua.Analyzers.AnalyzerDocs/Al1200UseIsEqualToDocs.cs`\n* `src/ANcpLua.Analyzers.AnalyzerDocs/AlAnalyzerDocsGenerator.cs`\n* `src/ANcpLua.Analyzers.AnalyzerDocs/AlAnalyzerDocsVerifier.cs`\n* `src/ANcpLua.Analyzers.AnalyzerDocsGenerator/ANcpLua.Analyzers.AnalyzerDocsGenerator.csproj`\n* `src/ANcpLua.Analyzers.CodeFixes/CLAUDE.md`\n* `src/ANcpLua.Analyzers.CodeFixes/CodeFixResources.Designer.cs`\n* `src/ANcpLua.Analyzers.CodeFixes/CodeFixResources.resx`\n* `src/ANcpLua.Analyzers.CodeFixes/CodeFixes/AL1001SimplifyNegatedPatternCodeFixProvider.cs`\n* `src/ANcpLua.Analyzers.CodeFixes/CodeFixes/AL1003SpanPatternMatchingCodeFixProvider.cs`\n* `src/ANcpLua.Analyzers.CodeFixes/CodeFixes/AL1004UseSequenceEqualCodeFixProvider.cs`\n* `src/ANcpLua.Analyzers.CodeFixes/CodeFixes/AL1007IXmlSerializableCodeFixProvider.cs`\n* `src/ANcpLua.Analyzers.CodeFixes/CodeFixes/AL1009LockTypeCodeFixProvider.cs`\n* `src/ANcpLua.Analyzers.CodeFixes/CodeFixes/AL1010PatternMatchingCodeFixProvider.cs`\n* `src/ANcpLua.Analyzers.CodeFixes/CodeFixes/AL1011NormalizeNullGuardStyleCodeFixProvider.cs`\n* `src/ANcpLua.Analyzers.CodeFixes/CodeFixes/AL1012CombineDeclarationWithNullCheckCodeFixProvider.cs`\n* `src/ANcpLua.Analyzers.CodeFixes/CodeFixes/AL1200UseIsEqualToCodeFixProvider.cs`\n* `src/ANcpLua.Analyzers.CodeFixes/CodeFixes/AL1201UseHasAttributeCodeFixProvider.cs`\n* `src/ANcpLua.Analyzers.CodeFixes/CodeFixes/AL1202UseTypeHierarchyCodeFixProvider.cs`\n* `src/ANcpLua.Analyzers.CodeFixes/CodeFixes/AL1203UseOperationExtensionsCodeFixProvider.cs`\n* `src/ANcpLua.Analyzers.CodeFixes/CodeFixes/AL1204UseOrEmptyCodeFixProvider.cs`\n* `src/ANcpLua.Analyzers.CodeFixes/CodeFixes/AL1205UseToImmutableArrayOrEmptyCodeFixProvider.cs`\n* `src/ANcpLua.Analyzers.CodeFixes/CodeFixes/AL1206UseWhereNotNullCodeFixProvider.cs`\n* `src/ANcpLua.Analyzers.CodeFixes/CodeFixes/AL1207UseToDisplayStringExtensionsCodeFixProvider.cs`\n* `src/ANcpLua.Analyzers.CodeFixes/CodeFixes/AL1208UseGuardNotNullCodeFixProvider.cs`\n* `src/ANcpLua.Analyzers.CodeFixes/CodeFixes/AL1209UseTryParseExtensionsCodeFixProvider.cs`\n* `src/ANcpLua.Analyzers.CodeFixes/CodeFixes/AL1210UseStringComparisonExtensionsCodeFixProvider.cs`\n* `src/ANcpLua.Analyzers.CodeFixes/CodeFixes/AL1211UseAttributeExtensionsCodeFixProvider.cs`\n* `src/ANcpLua.Analyzers.CodeFixes/CodeFixes/AL1212UseGuardNotNullOrEmptyCodeFixProvider.cs`\n* `src/ANcpLua.Analyzers.CodeFixes/CodeFixes/AL1213UseGuardNotNullOrWhiteSpaceCodeFixProvider.cs`\n* `src/ANcpLua.Analyzers.CodeFixes/CodeFixes/AL1214UseGuardNotZeroCodeFixProvider.cs`\n* `src/ANcpLua.Analyzers.CodeFixes/CodeFixes/AL1215UseGuardNotNegativeCodeFixProvider.cs`\n* `src/ANcpLua.Analyzers.CodeFixes/CodeFixes/AL1216UseGuardPositiveCodeFixProvider.cs`\n* `src/ANcpLua.Analyzers.CodeFixes/CodeFixes/AL1217UseGuardNotEmptyGuidCodeFixProvider.cs`\n* `src/ANcpLua.Analyzers.CodeFixes/CodeFixes/AL1218UseGuardDefinedEnumCodeFixProvider.cs`\n* `src/ANcpLua.Analyzers.CodeFixes/CodeFixes/AL1220UseGuardForThrowIfCodeFixProvider.cs`\n* `src/ANcpLua.Analyzers.CodeFixes/CodeFixes/AL1313CancellationTokenPropagationCodeFixProvider.cs`\n* `src/ANcpLua.Analyzers.CodeFixes/CodeFixes/AL1314UseExplicitMidpointRoundingCodeFixProvider.cs`\n* `src/ANcpLua.Analyzers.CodeFixes/CodeFixes/AL1500AddMissingCasesCodeFixProvider.cs`\n* `src/ANcpLua.Analyzers.CodeFixes/CodeFixes/AL1503NormalizeWhitespaceCodeFixProvider.cs`\n* `src/ANcpLua.Analyzers.CodeFixes/CodeFixes/AL1504DuckDbTableCodeFixProvider.cs`\n* `src/ANcpLua.Analyzers.CodeFixes/CodeFixes/AL1700StaticLambdaCodeFixProvider.cs`\n* `src/ANcpLua.Analyzers.CodeFixes/CodeFixes/AL1701DateTimeNowCodeFixProvider.cs`\n* `src/ANcpLua.Analyzers.CodeFixes/CodeFixes/AL1702UseSystemTextJsonCodeFixProvider.cs`\n* `src/ANcpLua.Analyzers.CodeFixes/CodeFixes/AL1703UseImplicitTypeWhenApparentCodeFixProvider.cs`\n* `src/ANcpLua.Analyzers.CodeFixes/Refactorings/AR0001SnakeCaseToPascalCaseRefactoring.cs`\n* `src/ANcpLua.Analyzers.CodeFixes/Refactorings/AR0002MakeStaticLambdaRefactoring.cs`\n* `src/ANcpLua.Analyzers/ANcpLua.Analyzers.csproj`\n* `src/ANcpLua.Analyzers/AlAnalyzer.cs`\n* `src/ANcpLua.Analyzers/AnalyzerReleases.Unshipped.md`\n* `src/ANcpLua.Analyzers/Analyzers/AL1000ProhibitPrimaryConstructorParameterReassignmentAnalyzer.cs`\n* `src/ANcpLua.Analyzers/Analyzers/AL1001DontRepeatNegatedPatternAnalyzer.cs`\n* `src/ANcpLua.Analyzers/Analyzers/AL1002DontDivideByConstantZeroAnalyzer.cs`\n* `src/ANcpLua.Analyzers/Analyzers/AL1003ToAL1004SpanComparisonAnalyzer.cs`\n* `src/ANcpLua.Analyzers/Analyzers/AL1005FieldNameConflictWithPrimaryConstructorAnalyzer.cs`\n* `src/ANcpLua.Analyzers/Analyzers/AL1006ToAL1008IXmlSerializableAnalyzer.cs`\n* `src/ANcpLua.Analyzers/Analyzers/AL1009LockKeywordAnalyzer.cs`\n* `src/ANcpLua.Analyzers/Analyzers/AL1010PreferPatternMatchingAnalyzer.cs`\n* `src/ANcpLua.Analyzers/Analyzers/AL1011NormalizeNullGuardStyleAnalyzer.cs`\n* `src/ANcpLua.Analyzers/Analyzers/AL1012CombineDeclarationWithNullCheckAnalyzer.cs`\n* `src/ANcpLua.Analyzers/Analyzers/AL1100ToAL1104FormBindingAnalyzer.cs`\n* `src/ANcpLua.Analyzers/Analyzers/AL1105MissingResilienceConfigurationAnalyzer.cs`\n* `src/ANcpLua.Analyzers/Analyzers/AL1106MissingHealthChecksAnalyzer.cs`\n* `src/ANcpLua.Analyzers/Analyzers/AL1107ConsiderConnectionStringAnalyzer.cs`\n* `src/ANcpLua.Analyzers/Analyzers/AL1108MissingServiceDiscoveryAnalyzer.cs`\n* `src/ANcpLua.Analyzers/Analyzers/AL1109AvoidTaskRunInAspNetCoreAnalyzer.cs`\n* `src/ANcpLua.Analyzers/Analyzers/AL1200UseIsEqualToAnalyzer.cs`\n* `src/ANcpLua.Analyzers/Analyzers/AL1201UseHasAttributeAnalyzer.cs`\n* `src/ANcpLua.Analyzers/Analyzers/AL1202UseTypeHierarchyAnalyzer.cs`\n* `src/ANcpLua.Analyzers/Analyzers/AL1203UseOperationExtensionsAnalyzer.cs`\n* `src/ANcpLua.Analyzers/Analyzers/AL1204UseOrEmptyAnalyzer.cs`\n* `src/ANcpLua.Analyzers/Analyzers/AL1205UseToImmutableArrayOrEmptyAnalyzer.cs`\n* `src/ANcpLua.Analyzers/Analyzers/AL1206UseWhereNotNullAnalyzer.cs`\n* `src/ANcpLua.Analyzers/Analyzers/AL1207UseToDisplayStringExtensionsAnalyzer.cs`\n* `src/ANcpLua.Analyzers/Analyzers/AL1208UseGuardNotNullAnalyzer.cs`\n* `src/ANcpLua.Analyzers/Analyzers/AL1209UseTryParseExtensionsAnalyzer.cs`\n* `src/ANcpLua.Analyzers/Analyzers/AL1210UseStringComparisonExtensionsAnalyzer.cs`\n* `src/ANcpLua.Analyzers/Analyzers/AL1211UseAttributeExtensionsAnalyzer.cs`\n* `src/ANcpLua.Analyzers/Analyzers/AL1212UseGuardNotNullOrEmptyAnalyzer.cs`\n* `src/ANcpLua.Analyzers/Analyzers/AL1213UseGuardNotNullOrWhiteSpaceAnalyzer.cs`\n* `src/ANcpLua.Analyzers/Analyzers/AL1214UseGuardNotZeroAnalyzer.cs`\n* `src/ANcpLua.Analyzers/Analyzers/AL1215UseGuardNotNegativeAnalyzer.cs`\n* `src/ANcpLua.Analyzers/Analyzers/AL1216UseGuardPositiveAnalyzer.cs`\n* `src/ANcpLua.Analyzers/Analyzers/AL1217UseGuardNotEmptyGuidAnalyzer.cs`\n* `src/ANcpLua.Analyzers/Analyzers/AL1218UseGuardDefinedEnumAnalyzer.cs`\n* `src/ANcpLua.Analyzers/Analyzers/AL1219UseStringComparisonAnyExtensionsAnalyzer.cs`\n* `src/ANcpLua.Analyzers/Analyzers/AL1220UseGuardForThrowIfAnalyzer.cs`\n* `src/ANcpLua.Analyzers/Analyzers/AL1300ToAL1303ThreadingAnalyzer.cs`\n* `src/ANcpLua.Analyzers/Analyzers/AL1304PreferAwaitUsingAnalyzer.cs`\n* `src/ANcpLua.Analyzers/Analyzers/AL1305AvoidBlockingCallsInAsyncAnalyzer.cs`\n* `src/ANcpLua.Analyzers/Analyzers/AL1306SqlInterpolationInCommandTextAnalyzer.cs`\n* `src/ANcpLua.Analyzers/Analyzers/AL1307FireAndForgetTaskAnalyzer.cs`\n* `src/ANcpLua.Analyzers/Analyzers/AL1308PreferTryParseAnalyzer.cs`\n* `src/ANcpLua.Analyzers/Analyzers/AL1309EmptyCatchBlockAnalyzer.cs`\n* `src/ANcpLua.Analyzers/Analyzers/AL1310ExceptionLeakedInResponseAnalyzer.cs`\n* `src/ANcpLua.Analyzers/Analyzers/AL1311UnnecessaryLinqMaterializationAnalyzer.cs`\n* `src/ANcpLua.Analyzers/Analyzers/AL1312ReadModifyWriteWithoutTransactionAnalyzer.cs`\n* `src/ANcpLua.Analyzers/Analyzers/AL1313CancellationTokenPropagationAnalysis.cs`\n* `src/ANcpLua.Analyzers/Analyzers/AL1313CancellationTokenPropagationAnalyzer.cs`\n* `src/ANcpLua.Analyzers/Analyzers/AL1314UseExplicitMidpointRoundingAnalyzer.cs`\n* `src/ANcpLua.Analyzers/Analyzers/AL1400AotTestMustReturnIntAnalyzer.cs`\n* `src/ANcpLua.Analyzers/Analyzers/AL1401AotTestExitCode100Analyzer.cs`\n* `src/ANcpLua.Analyzers/Analyzers/AL1402TrimSafeViolationAnalyzer.cs`\n* `src/ANcpLua.Analyzers/Analyzers/AL1403AotSafeViolationAnalyzer.cs`\n* `src/ANcpLua.Analyzers/Analyzers/AL1404AotSafeCallsAotUnsafeAnalyzer.cs`\n* `src/ANcpLua.Analyzers/Analyzers/AL1405UnnecessaryAotUnsafeAnalyzer.cs`\n* `src/ANcpLua.Analyzers/Analyzers/AL1406AvoidDynamicKeywordAnalyzer.cs`\n* `src/ANcpLua.Analyzers/Analyzers/AL1407AvoidExpressionCompileAnalyzer.cs`\n* `src/ANcpLua.Analyzers/Analyzers/AL1408AvoidActivatorCreateInstanceAnalyzer.cs`\n* `src/ANcpLua.Analyzers/Analyzers/AL1409AvoidTypeGetTypeAnalyzer.cs`\n* `src/ANcpLua.Analyzers/Analyzers/AL1500ClosedTypeHierarchySwitchAnalyzer.cs`\n* `src/ANcpLua.Analyzers/Analyzers/AL1501SymbolStoredInModelAnalyzer.cs`\n* `src/ANcpLua.Analyzers/Analyzers/AL1502UseIncrementalGeneratorAnalyzer.cs`\n* `src/ANcpLua.Analyzers/Analyzers/AL1503NormalizeWhitespaceAnalyzer.cs`\n* `src/ANcpLua.Analyzers/Analyzers/AL1504DuckDbTableMustBePartialAnalyzer.cs`\n* `src/ANcpLua.Analyzers/Analyzers/AL1505DuckDbColumnConflictingOrdinalAnalyzer.cs`\n* `src/ANcpLua.Analyzers/Analyzers/AL1600HardcodedPackageVersionAnalyzer.cs`\n* `src/ANcpLua.Analyzers/Analyzers/AL1601VersionPropsNotImportedAnalyzer.cs`\n* `src/ANcpLua.Analyzers/Analyzers/AL1602UndefinedVersionVariableAnalyzer.cs`\n* `src/ANcpLua.Analyzers/Analyzers/AL1603ToAL1605DiagnosticsAlignmentAnalyzer.cs`\n* `src/ANcpLua.Analyzers/Analyzers/AL1606OutdatedMafPackageVersionAnalyzer.cs`\n* `src/ANcpLua.Analyzers/Analyzers/AL1700PreferStaticLambdaAnalyzer.cs`\n* `src/ANcpLua.Analyzers/Analyzers/AL1701AvoidDateTimeNowAnalyzer.cs`\n* `src/ANcpLua.Analyzers/Analyzers/AL1702AvoidNewtonsoftJsonAnalyzer.cs`\n* `src/ANcpLua.Analyzers/Analyzers/AL1703UseImplicitTypeWhenApparentAnalyzer.cs`\n* `src/ANcpLua.Analyzers/Analyzers/AL1800DestructiveToolMustRequireApprovalAnalyzer.cs`\n* `src/ANcpLua.Analyzers/Analyzers/AL1801ToolMustDeclareSideEffectAnalyzer.cs`\n* `src/ANcpLua.Analyzers/Analyzers/AL1802ToolMustDeclareCapabilityAnalyzer.cs`\n* `src/ANcpLua.Analyzers/AotContext.cs`\n* `src/ANcpLua.Analyzers/CLAUDE.md`\n* `src/ANcpLua.Analyzers/CLAUDE.md`\n* `src/ANcpLua.Analyzers/Resources.Designer.cs`\n* `src/ANcpLua.Analyzers/Resources.resx`\n* `src/ANcpLua.Analyzers/build/ANcpLua.Analyzers.props`\n* `tests/ANcpLua.Analyzers.Tests/AL1000PrimaryConstructorReassignmentTests.cs`\n* `tests/ANcpLua.Analyzers.Tests/AL1001NegatedPatternTests.cs`\n* `tests/ANcpLua.Analyzers.Tests/AL1002DivisionByZeroTests.cs`\n* `tests/ANcpLua.Analyzers.Tests/AL1003ToAL1004SpanComparisonTests.cs`\n* `tests/ANcpLua.Analyzers.Tests/AL1005FieldNameConflictWithPrimaryConstructorTests.cs`\n* `tests/ANcpLua.Analyzers.Tests/AL1006ToAL1008IXmlSerializableTests.cs`\n* `tests/ANcpLua.Analyzers.Tests/AL1009LockKeywordTests.cs`\n* `tests/ANcpLua.Analyzers.Tests/AL1010PatternMatchingTests.cs`\n* `tests/ANcpLua.Analyzers.Tests/AL1011NormalizeNullGuardStyleTests.cs`\n* `tests/ANcpLua.Analyzers.Tests/AL1012CombineDeclarationWithNullCheckTests.cs`\n* `tests/ANcpLua.Analyzers.Tests/AL1100ToAL1104FormBindingTests.cs`\n* `tests/ANcpLua.Analyzers.Tests/AL1105MissingResilienceConfigurationTests.cs`\n* `tests/ANcpLua.Analyzers.Tests/AL1106MissingHealthChecksTests.cs`\n* `tests/ANcpLua.Analyzers.Tests/AL1107ConsiderConnectionStringTests.cs`\n* `tests/ANcpLua.Analyzers.Tests/AL1108MissingServiceDiscoveryTests.cs`\n* `tests/ANcpLua.Analyzers.Tests/AL1109AvoidTaskRunInAspNetCoreTests.cs`\n* `tests/ANcpLua.Analyzers.Tests/AL1200UseIsEqualToTests.cs`\n* `tests/ANcpLua.Analyzers.Tests/AL1201UseHasAttributeCodeFixTests.cs`\n* `tests/ANcpLua.Analyzers.Tests/AL1201UseHasAttributeTests.cs`\n* `tests/ANcpLua.Analyzers.Tests/AL1202UseTypeHierarchyTests.cs`\n* `tests/ANcpLua.Analyzers.Tests/AL1203UseOperationExtensionsTests.cs`\n* `tests/ANcpLua.Analyzers.Tests/AL1204UseOrEmptyTests.cs`\n* `tests/ANcpLua.Analyzers.Tests/AL1206UseWhereNotNullCodeFixTests.cs`\n* `tests/ANcpLua.Analyzers.Tests/AL1206UseWhereNotNullTests.cs`\n* `tests/ANcpLua.Analyzers.Tests/AL1208UseGuardNotNullTests.cs`\n* `tests/ANcpLua.Analyzers.Tests/AL1209UseTryParseExtensionsTests.cs`\n* `tests/ANcpLua.Analyzers.Tests/AL1210UseStringComparisonExtensionsTests.cs`\n* `tests/ANcpLua.Analyzers.Tests/AL1211UseAttributeExtensionsTests.cs`\n* `tests/ANcpLua.Analyzers.Tests/AL1212UseGuardNotNullOrEmptyTests.cs`\n* `tests/ANcpLua.Analyzers.Tests/AL1213UseGuardNotNullOrWhiteSpaceTests.cs`\n* `tests/ANcpLua.Analyzers.Tests/AL1214UseGuardNotZeroTests.cs`\n* `tests/ANcpLua.Analyzers.Tests/AL1215UseGuardNotNegativeTests.cs`\n* `tests/ANcpLua.Analyzers.Tests/AL1216UseGuardPositiveTests.cs`\n* `tests/ANcpLua.Analyzers.Tests/AL1217UseGuardNotEmptyGuidTests.cs`\n* `tests/ANcpLua.Analyzers.Tests/AL1218UseGuardDefinedEnumTests.cs`\n* `tests/ANcpLua.Analyzers.Tests/AL1219UseStringComparisonAnyExtensionsTests.cs`\n* `tests/ANcpLua.Analyzers.Tests/AL1220UseGuardForThrowIfCodeFixTests.cs`\n* `tests/ANcpLua.Analyzers.Tests/AL1220UseGuardForThrowIfTests.cs`\n* `tests/ANcpLua.Analyzers.Tests/AL1300ToAL1303ThreadingTests.cs`\n* `tests/ANcpLua.Analyzers.Tests/AL1304PreferAwaitUsingTests.cs`\n* `tests/ANcpLua.Analyzers.Tests/AL1305AvoidBlockingCallsInAsyncTests.cs`\n* `tests/ANcpLua.Analyzers.Tests/AL1306SqlInterpolationInCommandTextTests.cs`\n* `tests/ANcpLua.Analyzers.Tests/AL1307FireAndForgetTaskTests.cs`\n* `tests/ANcpLua.Analyzers.Tests/AL1308PreferTryParseTests.cs`\n* `tests/ANcpLua.Analyzers.Tests/AL1309EmptyCatchBlockTests.cs`\n* `tests/ANcpLua.Analyzers.Tests/AL1310ExceptionLeakedInResponseTests.cs`\n* `tests/ANcpLua.Analyzers.Tests/AL1311UnnecessaryLinqMaterializationTests.cs`\n* `tests/ANcpLua.Analyzers.Tests/AL1312ReadModifyWriteWithoutTransactionTests.cs`\n* `tests/ANcpLua.Analyzers.Tests/AL1313CancellationTokenPropagationTests.cs`\n* `tests/ANcpLua.Analyzers.Tests/AL1314UseExplicitMidpointRoundingCodeFixTests.cs`\n* `tests/ANcpLua.Analyzers.Tests/AL1314UseExplicitMidpointRoundingTests.cs`\n* `tests/ANcpLua.Analyzers.Tests/AL1401AotTestExitCode100Tests.cs`\n* `tests/ANcpLua.Analyzers.Tests/AL1403AotSafeViolationTests.cs`\n* `tests/ANcpLua.Analyzers.Tests/AL1404AotSafeCallsAotUnsafeTests.cs`\n* `tests/ANcpLua.Analyzers.Tests/AL1405UnnecessaryAotUnsafeTests.cs`\n* `tests/ANcpLua.Analyzers.Tests/AL1406AvoidDynamicKeywordTests.cs`\n* `tests/ANcpLua.Analyzers.Tests/AL1407AvoidExpressionCompileTests.cs`\n* `tests/ANcpLua.Analyzers.Tests/AL1501SymbolStoredInModelTests.cs`\n* `tests/ANcpLua.Analyzers.Tests/AL1502UseIncrementalGeneratorTests.cs`\n* `tests/ANcpLua.Analyzers.Tests/AL1503NormalizeWhitespaceTests.cs`\n* `tests/ANcpLua.Analyzers.Tests/AL1504DuckDbTableMustBePartialTests.cs`\n* `tests/ANcpLua.Analyzers.Tests/AL1505DuckDbColumnConflictingOrdinalTests.cs`\n* `tests/ANcpLua.Analyzers.Tests/AL1601VersionPropsNotImportedTests.cs`\n* `tests/ANcpLua.Analyzers.Tests/AL1602UndefinedVersionVariableTests.cs`\n* `tests/ANcpLua.Analyzers.Tests/AL1606OutdatedMafPackageVersionTests.cs`\n* `tests/ANcpLua.Analyzers.Tests/AL1700PreferStaticLambdaTests.cs`\n* `tests/ANcpLua.Analyzers.Tests/AL1701AvoidDateTimeNowTests.cs`\n* `tests/ANcpLua.Analyzers.Tests/AL1702AvoidNewtonsoftJsonTests.cs`\n* `tests/ANcpLua.Analyzers.Tests/AL1703UseImplicitTypeWhenApparentTests.cs`\n* `tests/ANcpLua.Analyzers.Tests/ANcpLua.Analyzers.Tests.csproj`\n* `tests/ANcpLua.Analyzers.Tests/CLAUDE.md`\n\n
\n\n
\n\n","_🧹 Nitpick_ | _🔵 Trivial_ | _🏗️ Heavy lift_\n\n**Coordinate Version.props update and cross-repo version bumps post-publish.**\n\n`ANcpLuaAnalyzersVersion=1.29.4` reflects the last published version (correct for restore-time). After this PR merges and `2.0.0` is published to NuGet, a follow-up PR (or CI auto-commit) must update this line to `2.0.0`. Linked repos (`ANcpLua.NET.Sdk`, `ErrorOrX`, `ANcpLua.Agents`) also pin this variable and must be updated in lockstep to pull the renumbered diagnostics. Verify the publish-then-update sequencing is scripted or documented to avoid restore failures.\n\nBased on learnings, cross-repo bootstrap requires publishing lower layers before moving SDK truth upward.\n\n\n🤖 Prompt for AI Agents
\n\n```\nVerify each finding against current code. Fix only still-valid issues, skip the\nrest with a brief reason, keep changes minimal, and validate.\n\nIn `@Version.props` at line 61, Update the pinned analyzer version variable\nANcpLuaAnalyzersVersion in Version.props from 1.29.4 to 2.0.0 after the new\n2.0.0 package is published; then make matching updates in the other repositories\nthat reference this same variable (ANcpLua.NET.Sdk, ErrorOrX, ANcpLua.Agents) so\nthey pull the renumbered diagnostics in lockstep, and add or update CI/scripted\nsteps (or documentation) to enforce the publish-first then bump sequence to\navoid restore failures.\n```\n\n
\n\n\n\n","_⚠️ Potential issue_ | _🟠 Major_ | _⚡ Quick win_\n\n**equivalenceKey must use the analyzer's DiagnosticId for FixAll support.**\n\nLine 35 sets `equivalenceKey` to `nameof(Al1702UseSystemTextJsonCodeFixProvider)` (the provider class name). Per coding guidelines, the equivalenceKey must be set to the analyzer's `DiagnosticId`.\n\n\n\n\n\nProposed fix
\n\n```diff\n context.RegisterCodeFix(\n CodeAction.Create(\n CodeFixResources.AL1702CodeFixTitle,\n _ => ConvertToSystemTextJson(context.Document, root, invocation, methodName, typeArgs),\n- nameof(Al1702UseSystemTextJsonCodeFixProvider)),\n+ Al1702AvoidNewtonsoftJsonAnalyzer.DiagnosticId),\n diagnostic);\n```\n
\n\nAs per coding guidelines: `CreateCodeAction method must set equivalenceKey to the analyzer's DiagnosticId to enable FixAll support`\n\n\n\n\n📝 Committable suggestion
\n\n> ‼️ **IMPORTANT**\n> Carefully review the code before committing. Ensure that it accurately replaces the highlighted code, contains no missing lines, and has no issues with indentation. Thoroughly test & benchmark the code to ensure it meets the requirements.\n\n```suggestion\n context.RegisterCodeFix(\n CodeAction.Create(\n CodeFixResources.AL1702CodeFixTitle,\n _ => ConvertToSystemTextJson(context.Document, root, invocation, methodName, typeArgs),\n Al1702AvoidNewtonsoftJsonAnalyzer.DiagnosticId),\n diagnostic);\n }\n```\n\n
\n\n\n\n\n🤖 Prompt for AI Agents
\n\n```\nVerify each finding against current code. Fix only still-valid issues, skip the\nrest with a brief reason, keep changes minimal, and validate.\n\nIn\n`@src/ANcpLua.Analyzers.CodeFixes/CodeFixes/AL1702UseSystemTextJsonCodeFixProvider.cs`\naround lines 31 - 37, The CodeAction equivalenceKey is currently set to the\nprovider class name (nameof(Al1702UseSystemTextJsonCodeFixProvider)), which\nbreaks FixAll; change the equivalenceKey passed to CodeAction.Create to the\nanalyzer's DiagnosticId (use the Diagnostic.Id from the diagnostic parameter or\nthe analyzer constant) so the CodeAction's equivalenceKey matches the analyzer's\nDiagnosticId and enables FixAll support—update the call in the RegisterCodeFix\nblock where CodeAction.Create is invoked (the invocation that builds the\nCodeAction for ConvertToSystemTextJson) to supply diagnostic.Id instead of the\nprovider class name.\n```\n\n
\n\n\n\n","src/ANcpLua.Analyzers.CodeFixes/CodeFixes/AL1702UseSystemTextJsonCodeFixProvider.cs",37,"_⚠️ Potential issue_ | _🟠 Major_ | _⚡ Quick win_\n\n**equivalenceKey must use the analyzer's DiagnosticId for FixAll support.**\n\nLine 25 sets `equivalenceKey` to `nameof(Al1701DateTimeNowCodeFixProvider)` (the provider class name). Per coding guidelines, the equivalenceKey must be set to the analyzer's `DiagnosticId`.\n\n\n\n\n\nProposed fix
\n\n```diff\n CodeAction.Create(\n CodeFixResources.AL1701CodeFixTitle,\n _ => ConvertToTimeProvider(document, memberAccess, root, diagnostic),\n- nameof(Al1701DateTimeNowCodeFixProvider));\n+ Al1701AvoidDateTimeNowAnalyzer.DiagnosticId);\n```\n
\n\nAs per coding guidelines: `CreateCodeAction method must set equivalenceKey to the analyzer's DiagnosticId to enable FixAll support`\n\n\n\n\n📝 Committable suggestion
\n\n> ‼️ **IMPORTANT**\n> Carefully review the code before committing. Ensure that it accurately replaces the highlighted code, contains no missing lines, and has no issues with indentation. Thoroughly test & benchmark the code to ensure it meets the requirements.\n\n```suggestion\n CodeAction.Create(\n CodeFixResources.AL1701CodeFixTitle,\n _ => ConvertToTimeProvider(document, memberAccess, root, diagnostic),\n Al1701AvoidDateTimeNowAnalyzer.DiagnosticId);\n```\n\n
\n\n\n\n\n🤖 Prompt for AI Agents
\n\n```\nVerify each finding against current code. Fix only still-valid issues, skip the\nrest with a brief reason, keep changes minimal, and validate.\n\nIn\n`@src/ANcpLua.Analyzers.CodeFixes/CodeFixes/AL1701DateTimeNowCodeFixProvider.cs`\naround lines 22 - 25, The CodeAction.Create call in\nAL1701DateTimeNowCodeFixProvider currently sets the equivalenceKey to the\nprovider class name; change it to use the analyzer's DiagnosticId to enable\nFixAll support by replacing the current equivalenceKey\n(nameof(Al1701DateTimeNowCodeFixProvider)) with the diagnostic's ID (e.g.,\ndiagnostic.Id or the analyzer's DiagnosticId constant such as\nAl1701Analyzer.DiagnosticId) in the CodeAction.Create invocation that constructs\nthe fix for ConvertToTimeProvider.\n```\n\n
\n\n\n\n","src/ANcpLua.Analyzers.CodeFixes/CodeFixes/AL1701DateTimeNowCodeFixProvider.cs","_⚠️ Potential issue_ | _🟠 Major_ | _⚡ Quick win_\n\n**equivalenceKey must use the analyzer's DiagnosticId for FixAll support.**\n\nLine 26 sets `equivalenceKey` to `nameof(CodeFixResources.AL1700CodeFixTitle)`. Even with a custom `FixAllProvider`, the single-diagnostic code action registration must use the analyzer's `DiagnosticId` as the equivalenceKey per coding guidelines.\n\n\n\n\n\nProposed fix
\n\n```diff\n context.RegisterCodeFix(\n CodeAction.Create(\n CodeFixResources.AL1700CodeFixTitle,\n ct => MakeStaticAsync(context.Document, span, ct),\n- nameof(CodeFixResources.AL1700CodeFixTitle)),\n+ Al1700PreferStaticLambdaAnalyzer.DiagnosticId),\n diagnostic);\n```\n
\n\nAs per coding guidelines: `CreateCodeAction method must set equivalenceKey to the analyzer's DiagnosticId to enable FixAll support`\n\n\n\n\n📝 Committable suggestion
\n\n> ‼️ **IMPORTANT**\n> Carefully review the code before committing. Ensure that it accurately replaces the highlighted code, contains no missing lines, and has no issues with indentation. Thoroughly test & benchmark the code to ensure it meets the requirements.\n\n```suggestion\n context.RegisterCodeFix(\n CodeAction.Create(\n CodeFixResources.AL1700CodeFixTitle,\n ct => MakeStaticAsync(context.Document, span, ct),\n Al1700PreferStaticLambdaAnalyzer.DiagnosticId),\n diagnostic);\n```\n\n
\n\n\n\n\n🤖 Prompt for AI Agents
\n\n```\nVerify each finding against current code. Fix only still-valid issues, skip the\nrest with a brief reason, keep changes minimal, and validate.\n\nIn\n`@src/ANcpLua.Analyzers.CodeFixes/CodeFixes/AL1700StaticLambdaCodeFixProvider.cs`\naround lines 22 - 27, The CodeAction registered in\nAL1700StaticLambdaCodeFixProvider (context.RegisterCodeFix -> CodeAction.Create)\nuses CodeFixResources.AL1700CodeFixTitle as the equivalenceKey; change it to the\nanalyzer diagnostic id so FixAll works: when creating the CodeAction for\nMakeStaticAsync, pass the diagnostic's DiagnosticId (e.g., diagnostic.Id) as the\nequivalenceKey instead of nameof(CodeFixResources.AL1700CodeFixTitle) so the\naction's equivalenceKey matches the analyzer DiagnosticId.\n```\n\n
\n\n\n\n","src/ANcpLua.Analyzers.CodeFixes/CodeFixes/AL1700StaticLambdaCodeFixProvider.cs",27,"_⚠️ Potential issue_ | _🟠 Major_ | _⚡ Quick win_\n\n**equivalenceKey must use the analyzer's DiagnosticId for FixAll support.**\n\nLine 32 sets `equivalenceKey` to `nameof(CodeFixResources.AL1504CodeFixTitle)`. Per coding guidelines, the equivalenceKey must be set to the analyzer's `DiagnosticId`.\n\n\n\n\n\nProposed fix
\n\n```diff\n context.RegisterCodeFix(\n CodeAction.Create(\n CodeFixResources.AL1504CodeFixTitle,\n _ => MakePartialAsync(context.Document, typeDeclaration, root),\n- nameof(CodeFixResources.AL1504CodeFixTitle)),\n+ Al1504DuckDbTableMustBePartialAnalyzer.DiagnosticId),\n diagnostic);\n```\n
\n\nAs per coding guidelines: `CreateCodeAction method must set equivalenceKey to the analyzer's DiagnosticId to enable FixAll support`\n\n\n\n\n📝 Committable suggestion
\n\n> ‼️ **IMPORTANT**\n> Carefully review the code before committing. Ensure that it accurately replaces the highlighted code, contains no missing lines, and has no issues with indentation. Thoroughly test & benchmark the code to ensure it meets the requirements.\n\n```suggestion\n context.RegisterCodeFix(\n CodeAction.Create(\n CodeFixResources.AL1504CodeFixTitle,\n _ => MakePartialAsync(context.Document, typeDeclaration, root),\n Al1504DuckDbTableMustBePartialAnalyzer.DiagnosticId),\n diagnostic);\n }\n```\n\n
\n\n\n\n\n🤖 Prompt for AI Agents
\n\n```\nVerify each finding against current code. Fix only still-valid issues, skip the\nrest with a brief reason, keep changes minimal, and validate.\n\nIn\n`@src/ANcpLua.Analyzers.CodeFixes/CodeFixes/AL1504DuckDbTableCodeFixProvider.cs`\naround lines 28 - 34, The CodeAction equivalenceKey is currently set to\nnameof(CodeFixResources.AL1504CodeFixTitle); change it to use the analyzer\ndiagnostic id so FixAll works: when creating the CodeAction in\ncontext.RegisterCodeFix (the CodeAction.Create call that invokes\nMakePartialAsync and uses CodeFixResources.AL1504CodeFixTitle), set the\nequivalenceKey to the Diagnostic.Id (e.g., diagnostic.Id or the analyzer's\nDiagnosticId constant) instead of the resource name.\n```\n\n
\n\n\n\n","src/ANcpLua.Analyzers.CodeFixes/CodeFixes/AL1504DuckDbTableCodeFixProvider.cs",34,"_⚠️ Potential issue_ | _🟠 Major_ | _⚡ Quick win_\n\n**Align code action `equivalenceKey` with analyzer ID.**\n\nLine 32 should use `Al1314UseExplicitMidpointRoundingAnalyzer.DiagnosticId` instead of provider-name keying.\n\n \n\n\nSuggested patch
\n\n```diff\n CodeAction.Create(\n CodeFixResources.AL1314CodeFixTitle,\n _ => AppendMidpointRounding(document, invocation, root),\n- nameof(Al1314UseExplicitMidpointRoundingCodeFixProvider));\n+ Al1314UseExplicitMidpointRoundingAnalyzer.DiagnosticId);\n```\n
\n\nAs per coding guidelines, `equivalenceKey` must be set to the analyzer’s `DiagnosticId` for FixAll correctness.\n\n\n\n\n📝 Committable suggestion
\n\n> ‼️ **IMPORTANT**\n> Carefully review the code before committing. Ensure that it accurately replaces the highlighted code, contains no missing lines, and has no issues with indentation. Thoroughly test & benchmark the code to ensure it meets the requirements.\n\n```suggestion\n CodeFixResources.AL1314CodeFixTitle,\n _ => AppendMidpointRounding(document, invocation, root),\n Al1314UseExplicitMidpointRoundingAnalyzer.DiagnosticId));\n```\n\n
\n\n\n\n\n🤖 Prompt for AI Agents
\n\n```\nVerify each finding against current code. Fix only still-valid issues, skip the\nrest with a brief reason, keep changes minimal, and validate.\n\nIn\n`@src/ANcpLua.Analyzers.CodeFixes/CodeFixes/AL1314UseExplicitMidpointRoundingCodeFixProvider.cs`\naround lines 30 - 32, The code action's equivalenceKey is currently set using\nnameof(Al1314UseExplicitMidpointRoundingCodeFixProvider); change it to use\nAl1314UseExplicitMidpointRoundingAnalyzer.DiagnosticId so the equivalenceKey\naligns with the analyzer ID for FixAll support—update the invocation that\nregisters the code action (the call passing CodeFixResources.AL1314CodeFixTitle,\n_ => AppendMidpointRounding(document, invocation, root), nameof(...)) to replace\nthe nameof(...) argument with\nAl1314UseExplicitMidpointRoundingAnalyzer.DiagnosticId.\n```\n\n
\n\n\n\n","src/ANcpLua.Analyzers.CodeFixes/CodeFixes/AL1314UseExplicitMidpointRoundingCodeFixProvider.cs","_⚠️ Potential issue_ | _🟠 Major_ | _⚡ Quick win_\n\n**Incorrect `equivalenceKey` source in code action.**\n\nLine 25 uses a resource-member name as the equivalence key. This should be the analyzer ID (`Al1313CancellationTokenPropagationAnalyzer.DiagnosticId`) to keep FixAll deterministic.\n\n \n\n\nSuggested patch
\n\n```diff\n CodeAction.Create(\n CodeFixResources.AL1313CodeFixTitle,\n ct => AddCancellationTokenAsync(document, invocation, root, ct),\n- nameof(CodeFixResources.AL1313CodeFixTitle));\n+ Al1313CancellationTokenPropagationAnalyzer.DiagnosticId);\n```\n
\n\nAs per coding guidelines, `equivalenceKey` must be set to the analyzer’s `DiagnosticId` for FixAll correctness.\n\n\n\n\n📝 Committable suggestion
\n\n> ‼️ **IMPORTANT**\n> Carefully review the code before committing. Ensure that it accurately replaces the highlighted code, contains no missing lines, and has no issues with indentation. Thoroughly test & benchmark the code to ensure it meets the requirements.\n\n```suggestion\n CodeFixResources.AL1313CodeFixTitle,\n ct => AddCancellationTokenAsync(document, invocation, root, ct),\n Al1313CancellationTokenPropagationAnalyzer.DiagnosticId);\n```\n\n
\n\n\n\n\n🤖 Prompt for AI Agents
\n\n```\nVerify each finding against current code. Fix only still-valid issues, skip the\nrest with a brief reason, keep changes minimal, and validate.\n\nIn\n`@src/ANcpLua.Analyzers.CodeFixes/CodeFixes/AL1313CancellationTokenPropagationCodeFixProvider.cs`\naround lines 23 - 25, The CodeAction in\nAL1313CancellationTokenPropagationCodeFixProvider is using\nCodeFixResources.AL1313CodeFixTitle as the equivalenceKey when creating the fix;\nchange the equivalenceKey to the analyzer ID by passing\nAl1313CancellationTokenPropagationAnalyzer.DiagnosticId instead (update the\nCodeAction creation that calls AddCancellationTokenAsync/document\ninvocation/root to use Al1313CancellationTokenPropagationAnalyzer.DiagnosticId\nas the third argument).\n```\n\n
\n\n\n\n","src/ANcpLua.Analyzers.CodeFixes/CodeFixes/AL1313CancellationTokenPropagationCodeFixProvider.cs","_⚠️ Potential issue_ | _🟠 Major_ | _⚡ Quick win_\n\n**FixAll key must be analyzer `DiagnosticId`.**\n\nLine 36 currently keys by provider name. Use `Al1220UseGuardForThrowIfAnalyzer.DiagnosticId` instead.\n\n \n\n\nSuggested patch
\n\n```diff\n CodeAction.Create(\n CodeFixResources.AL1220CodeFixTitle,\n _ => ConvertToGuard(document, invocation, root, diagnostic),\n- nameof(Al1220UseGuardForThrowIfCodeFixProvider));\n+ Al1220UseGuardForThrowIfAnalyzer.DiagnosticId);\n```\n
\n\nAs per coding guidelines, `equivalenceKey` must be set to the analyzer’s `DiagnosticId` for FixAll correctness.\n\n\n\n\n📝 Committable suggestion
\n\n> ‼️ **IMPORTANT**\n> Carefully review the code before committing. Ensure that it accurately replaces the highlighted code, contains no missing lines, and has no issues with indentation. Thoroughly test & benchmark the code to ensure it meets the requirements.\n\n```suggestion\n CodeFixResources.AL1220CodeFixTitle,\n _ => ConvertToGuard(document, invocation, root, diagnostic),\n Al1220UseGuardForThrowIfAnalyzer.DiagnosticId);\n```\n\n
\n\n\n\n\n🤖 Prompt for AI Agents
\n\n```\nVerify each finding against current code. Fix only still-valid issues, skip the\nrest with a brief reason, keep changes minimal, and validate.\n\nIn\n`@src/ANcpLua.Analyzers.CodeFixes/CodeFixes/AL1220UseGuardForThrowIfCodeFixProvider.cs`\naround lines 34 - 36, The FixAll equivalence key is incorrectly set to the\nprovider name; update the registration call in\nAL1220UseGuardForThrowIfCodeFixProvider (the RegisterCodeFix invocation that\ncurrently uses nameof(Al1220UseGuardForThrowIfCodeFixProvider)) to use\nAl1220UseGuardForThrowIfAnalyzer.DiagnosticId as the equivalenceKey instead so\nthe fix is keyed by the analyzer DiagnosticId; keep the title\n(CodeFixResources.AL1220CodeFixTitle) and callback (ConvertToGuard) the same.\n```\n\n
\n\n\n\n","src/ANcpLua.Analyzers.CodeFixes/CodeFixes/AL1220UseGuardForThrowIfCodeFixProvider.cs","_⚠️ Potential issue_ | _🟠 Major_ | _⚡ Quick win_\n\n**`equivalenceKey` should track diagnostic identity.**\n\nLine 30 uses provider-name keying. Replace with `Al1218UseGuardDefinedEnumAnalyzer.DiagnosticId` for FixAll correctness.\n\n \n\n\nSuggested patch
\n\n```diff\n CodeAction.Create(\n CodeFixResources.AL1218CodeFixTitle,\n _ => ConvertToGuardDefinedEnum(document, ifStatement, root),\n- nameof(Al1218UseGuardDefinedEnumCodeFixProvider));\n+ Al1218UseGuardDefinedEnumAnalyzer.DiagnosticId);\n```\n
\n\nAs per coding guidelines, `equivalenceKey` must be set to the analyzer’s `DiagnosticId` for FixAll correctness.\n\n\n\n\n📝 Committable suggestion
\n\n> ‼️ **IMPORTANT**\n> Carefully review the code before committing. Ensure that it accurately replaces the highlighted code, contains no missing lines, and has no issues with indentation. Thoroughly test & benchmark the code to ensure it meets the requirements.\n\n```suggestion\n CodeFixResources.AL1218CodeFixTitle,\n _ => ConvertToGuardDefinedEnum(document, ifStatement, root),\n Al1218UseGuardDefinedEnumAnalyzer.DiagnosticId));\n```\n\n
\n\n\n\n\n🤖 Prompt for AI Agents
\n\n```\nVerify each finding against current code. Fix only still-valid issues, skip the\nrest with a brief reason, keep changes minimal, and validate.\n\nIn\n`@src/ANcpLua.Analyzers.CodeFixes/CodeFixes/AL1218UseGuardDefinedEnumCodeFixProvider.cs`\naround lines 28 - 30, The equivalenceKey currently uses\nnameof(Al1218UseGuardDefinedEnumCodeFixProvider) which breaks FixAll; change the\nequivalenceKey to the analyzer diagnostic id by replacing the third argument\nfrom nameof(Al1218UseGuardDefinedEnumCodeFixProvider) to\nAl1218UseGuardDefinedEnumAnalyzer.DiagnosticId in the CodeAction.Create call\nthat invokes ConvertToGuardDefinedEnum(document, ifStatement, root) inside\nAl1218UseGuardDefinedEnumCodeFixProvider so FixAll groups diagnostics by the\nanalyzer ID rather than the provider name.\n```\n\n
\n\n\n\n","src/ANcpLua.Analyzers.CodeFixes/CodeFixes/AL1218UseGuardDefinedEnumCodeFixProvider.cs","_⚠️ Potential issue_ | _🟠 Major_ | _⚡ Quick win_\n\n**Use analyzer `DiagnosticId` as `equivalenceKey`.**\n\nLine 36 should be `Al1217UseGuardNotEmptyGuidAnalyzer.DiagnosticId` to preserve correct FixAll keying.\n\n \n\n\nSuggested patch
\n\n```diff\n return CodeAction.Create(\n CodeFixResources.AL1217CodeFixTitle,\n ct => ConvertToGuardNotEmpty(document, ifStatement, expressionText, ct),\n- nameof(Al1217UseGuardNotEmptyGuidCodeFixProvider));\n+ Al1217UseGuardNotEmptyGuidAnalyzer.DiagnosticId);\n```\n
\n\nAs per coding guidelines, `equivalenceKey` must be set to the analyzer’s `DiagnosticId` for FixAll correctness.\n\n\n\n\n📝 Committable suggestion
\n\n> ‼️ **IMPORTANT**\n> Carefully review the code before committing. Ensure that it accurately replaces the highlighted code, contains no missing lines, and has no issues with indentation. Thoroughly test & benchmark the code to ensure it meets the requirements.\n\n```suggestion\n CodeFixResources.AL1217CodeFixTitle,\n ct => ConvertToGuardNotEmpty(document, ifStatement, expressionText, ct),\n Al1217UseGuardNotEmptyGuidAnalyzer.DiagnosticId);\n```\n\n
\n\n\n\n\n🤖 Prompt for AI Agents
\n\n```\nVerify each finding against current code. Fix only still-valid issues, skip the\nrest with a brief reason, keep changes minimal, and validate.\n\nIn\n`@src/ANcpLua.Analyzers.CodeFixes/CodeFixes/AL1217UseGuardNotEmptyGuidCodeFixProvider.cs`\naround lines 34 - 36, The equivalenceKey passed to RegisterCodeFix is currently\nthe code fix provider class name; change it to use the analyzer DiagnosticId\ninstead: replace the nameof(Al1217UseGuardNotEmptyGuidCodeFixProvider)\nequivalenceKey with Al1217UseGuardNotEmptyGuidAnalyzer.DiagnosticId so the\nFixAll operation keys correctly for the ConvertToGuardNotEmpty code action\nregistered in Al1217UseGuardNotEmptyGuidCodeFixProvider.\n```\n\n
\n\n\n\n","src/ANcpLua.Analyzers.CodeFixes/CodeFixes/AL1217UseGuardNotEmptyGuidCodeFixProvider.cs","_⚠️ Potential issue_ | _🟠 Major_ | _⚡ Quick win_\n\n**FixAll equivalence key should be the diagnostic ID.**\n\nLine 32 currently uses the provider name; switch to `Al1216UseGuardPositiveAnalyzer.DiagnosticId`.\n\n \n\n\nSuggested patch
\n\n```diff\n CodeAction.Create(\n CodeFixResources.AL1216CodeFixTitle,\n _ => ConvertToGuardPositive(document, ifStatement, root, GetExpressionFromDiagnostic(diagnostic, ifStatement)),\n- nameof(Al1216UseGuardPositiveCodeFixProvider));\n+ Al1216UseGuardPositiveAnalyzer.DiagnosticId);\n```\n
\n\nAs per coding guidelines, `equivalenceKey` must be set to the analyzer’s `DiagnosticId` for FixAll correctness.\n\n\n\n\n📝 Committable suggestion
\n\n> ‼️ **IMPORTANT**\n> Carefully review the code before committing. Ensure that it accurately replaces the highlighted code, contains no missing lines, and has no issues with indentation. Thoroughly test & benchmark the code to ensure it meets the requirements.\n\n```suggestion\n CodeFixResources.AL1216CodeFixTitle,\n _ => ConvertToGuardPositive(document, ifStatement, root, GetExpressionFromDiagnostic(diagnostic, ifStatement)),\n Al1216UseGuardPositiveAnalyzer.DiagnosticId);\n```\n\n
\n\n\n\n\n🤖 Prompt for AI Agents
\n\n```\nVerify each finding against current code. Fix only still-valid issues, skip the\nrest with a brief reason, keep changes minimal, and validate.\n\nIn\n`@src/ANcpLua.Analyzers.CodeFixes/CodeFixes/AL1216UseGuardPositiveCodeFixProvider.cs`\naround lines 30 - 32, The FixAll equivalence key is currently set to the\nprovider name via nameof(Al1216UseGuardPositiveCodeFixProvider); update the\nthird argument of the RegisterCodeFix call to use the analyzer diagnostic ID by\nreplacing the equivalenceKey with Al1216UseGuardPositiveAnalyzer.DiagnosticId so\nFixAll uses the analyzer's DiagnosticId; locate the RegisterCodeFix invocation\nin AL1216UseGuardPositiveCodeFixProvider and change the equivalenceKey\naccordingly.\n```\n\n
\n\n\n\n","src/ANcpLua.Analyzers.CodeFixes/CodeFixes/AL1216UseGuardPositiveCodeFixProvider.cs","_⚠️ Potential issue_ | _🟠 Major_ | _⚡ Quick win_\n\n**Set `equivalenceKey` to the analyzer ID, not provider name.**\n\nLine 30 should use `Al1215UseGuardNotNegativeAnalyzer.DiagnosticId` to keep FixAll behavior aligned with the analyzer identity.\n\n \n\n\nSuggested patch
\n\n```diff\n CodeAction.Create(\n CodeFixResources.AL1215CodeFixTitle,\n _ => ConvertToGuardNotNegative(document, ifStatement, root),\n- nameof(Al1215UseGuardNotNegativeCodeFixProvider));\n+ Al1215UseGuardNotNegativeAnalyzer.DiagnosticId);\n```\n
\n\nAs per coding guidelines, `equivalenceKey` must be set to the analyzer’s `DiagnosticId` for FixAll correctness.\n\n\n\n\n📝 Committable suggestion
\n\n> ‼️ **IMPORTANT**\n> Carefully review the code before committing. Ensure that it accurately replaces the highlighted code, contains no missing lines, and has no issues with indentation. Thoroughly test & benchmark the code to ensure it meets the requirements.\n\n```suggestion\n CodeFixResources.AL1215CodeFixTitle,\n _ => ConvertToGuardNotNegative(document, ifStatement, root),\n Al1215UseGuardNotNegativeAnalyzer.DiagnosticId);\n```\n\n
\n\n\n\n\n🤖 Prompt for AI Agents
\n\n```\nVerify each finding against current code. Fix only still-valid issues, skip the\nrest with a brief reason, keep changes minimal, and validate.\n\nIn\n`@src/ANcpLua.Analyzers.CodeFixes/CodeFixes/AL1215UseGuardNotNegativeCodeFixProvider.cs`\naround lines 28 - 30, The RegisterCodeFix call in\nAl1215UseGuardNotNegativeCodeFixProvider currently uses\nnameof(Al1215UseGuardNotNegativeCodeFixProvider) as the equivalenceKey; change\nthat argument to Al1215UseGuardNotNegativeAnalyzer.DiagnosticId so the\nequivalenceKey matches the analyzer ID (keep the other args the same, i.e.,\nCodeFixResources.AL1215CodeFixTitle and the _ =>\nConvertToGuardNotNegative(document, ifStatement, root) delegate).\n```\n\n
\n\n\n\n","src/ANcpLua.Analyzers.CodeFixes/CodeFixes/AL1215UseGuardNotNegativeCodeFixProvider.cs","_⚠️ Potential issue_ | _🟠 Major_ | _⚡ Quick win_\n\n**Use analyzer `DiagnosticId` for FixAll equivalence.**\n\nLine 39 uses the provider name as `equivalenceKey`; this can break consistent FixAll bucketing across analyzer/code-fix wiring. Use `Al1214UseGuardNotZeroAnalyzer.DiagnosticId`.\n\n \n\n\nSuggested patch
\n\n```diff\n return CodeAction.Create(\n CodeFixResources.AL1214CodeFixTitle,\n _ => ConvertToGuardNotZero(document, ifStatement, root, identifier),\n- nameof(Al1214UseGuardNotZeroCodeFixProvider));\n+ Al1214UseGuardNotZeroAnalyzer.DiagnosticId);\n```\n
\n\nAs per coding guidelines, `equivalenceKey` must be set to the analyzer’s `DiagnosticId` for FixAll correctness.\n\n\n\n\n📝 Committable suggestion
\n\n> ‼️ **IMPORTANT**\n> Carefully review the code before committing. Ensure that it accurately replaces the highlighted code, contains no missing lines, and has no issues with indentation. Thoroughly test & benchmark the code to ensure it meets the requirements.\n\n```suggestion\n CodeFixResources.AL1214CodeFixTitle,\n _ => ConvertToGuardNotZero(document, ifStatement, root, identifier),\n Al1214UseGuardNotZeroAnalyzer.DiagnosticId);\n```\n\n
\n\n\n\n\n🤖 Prompt for AI Agents
\n\n```\nVerify each finding against current code. Fix only still-valid issues, skip the\nrest with a brief reason, keep changes minimal, and validate.\n\nIn\n`@src/ANcpLua.Analyzers.CodeFixes/CodeFixes/AL1214UseGuardNotZeroCodeFixProvider.cs`\naround lines 37 - 39, The FixAll equivalence key is currently set to\nnameof(Al1214UseGuardNotZeroCodeFixProvider) which can break FixAll bucketing;\nupdate the RegisterCodeFix call in Al1214UseGuardNotZeroCodeFixProvider to use\nAl1214UseGuardNotZeroAnalyzer.DiagnosticId as the equivalenceKey instead of\nnameof(Al1214UseGuardNotZeroCodeFixProvider) so FixAll uses the analyzer\ndiagnostic id for consistent grouping.\n```\n\n
\n\n\n\n","src/ANcpLua.Analyzers.CodeFixes/CodeFixes/AL1214UseGuardNotZeroCodeFixProvider.cs","_⚠️ Potential issue_ | _🔴 Critical_ | _⚡ Quick win_\n\n**equivalenceKey must be the analyzer's DiagnosticId, not the provider name.**\n\nLine 30 passes `nameof(Al1213UseGuardNotNullOrWhiteSpaceCodeFixProvider)` as the equivalenceKey. Per coding guidelines, this must be `Al1213UseGuardNotNullOrWhiteSpaceAnalyzer.DiagnosticId` for FixAll correctness.\n\n\n\n\n\n🔧 Fix equivalenceKey
\n\n```diff\n CodeAction.Create(\n CodeFixResources.AL1213CodeFixTitle,\n _ => ConvertToGuardNotNullOrWhiteSpace(document, ifStatement, root),\n- nameof(Al1213UseGuardNotNullOrWhiteSpaceCodeFixProvider));\n+ Al1213UseGuardNotNullOrWhiteSpaceAnalyzer.DiagnosticId);\n```\n
\n\nAs per coding guidelines: \"CreateCodeAction method must set equivalenceKey to the analyzer's DiagnosticId to enable FixAll support\"\n\n\n\n\n📝 Committable suggestion
\n\n> ‼️ **IMPORTANT**\n> Carefully review the code before committing. Ensure that it accurately replaces the highlighted code, contains no missing lines, and has no issues with indentation. Thoroughly test & benchmark the code to ensure it meets the requirements.\n\n```suggestion\n CodeAction.Create(\n CodeFixResources.AL1213CodeFixTitle,\n _ => ConvertToGuardNotNullOrWhiteSpace(document, ifStatement, root),\n Al1213UseGuardNotNullOrWhiteSpaceAnalyzer.DiagnosticId);\n```\n\n
\n\n\n\n\n🤖 Prompt for AI Agents
\n\n```\nVerify each finding against current code. Fix only still-valid issues, skip the\nrest with a brief reason, keep changes minimal, and validate.\n\nIn\n`@src/ANcpLua.Analyzers.CodeFixes/CodeFixes/AL1213UseGuardNotNullOrWhiteSpaceCodeFixProvider.cs`\naround lines 27 - 30, The CodeAction is created with the provider name as the\nequivalenceKey which breaks FixAll; change the equivalenceKey argument in the\nCodeAction.Create call (the call that uses AL1213CodeFixTitle and\nConvertToGuardNotNullOrWhiteSpace) from\nnameof(Al1213UseGuardNotNullOrWhiteSpaceCodeFixProvider) to\nAl1213UseGuardNotNullOrWhiteSpaceAnalyzer.DiagnosticId so the action uses the\nanalyzer's DiagnosticId for correct FixAll behavior.\n```\n\n
\n\n\n\n","src/ANcpLua.Analyzers.CodeFixes/CodeFixes/AL1213UseGuardNotNullOrWhiteSpaceCodeFixProvider.cs","_⚠️ Potential issue_ | _🔴 Critical_ | _⚡ Quick win_\n\n**equivalenceKey must be the analyzer's DiagnosticId, not the provider name.**\n\nLine 36 passes `nameof(Al1212UseGuardNotNullOrEmptyCodeFixProvider)` as the equivalenceKey. Per coding guidelines, this must be `Al1212UseGuardNotNullOrEmptyAnalyzer.DiagnosticId` for FixAll correctness.\n\n\n\n\n\n🔧 Fix equivalenceKey
\n\n```diff\n CodeAction.Create(\n CodeFixResources.AL1212CodeFixTitle,\n _ => ConvertToGuardNotNullOrEmpty(document, ifStatement, root),\n- nameof(Al1212UseGuardNotNullOrEmptyCodeFixProvider));\n+ Al1212UseGuardNotNullOrEmptyAnalyzer.DiagnosticId);\n```\n
\n\nAs per coding guidelines: \"CreateCodeAction method must set equivalenceKey to the analyzer's DiagnosticId to enable FixAll support\"\n\n\n\n\n📝 Committable suggestion
\n\n> ‼️ **IMPORTANT**\n> Carefully review the code before committing. Ensure that it accurately replaces the highlighted code, contains no missing lines, and has no issues with indentation. Thoroughly test & benchmark the code to ensure it meets the requirements.\n\n```suggestion\n CodeAction.Create(\n CodeFixResources.AL1212CodeFixTitle,\n _ => ConvertToGuardNotNullOrEmpty(document, ifStatement, root),\n Al1212UseGuardNotNullOrEmptyAnalyzer.DiagnosticId);\n```\n\n
\n\n\n\n\n🤖 Prompt for AI Agents
\n\n```\nVerify each finding against current code. Fix only still-valid issues, skip the\nrest with a brief reason, keep changes minimal, and validate.\n\nIn\n`@src/ANcpLua.Analyzers.CodeFixes/CodeFixes/AL1212UseGuardNotNullOrEmptyCodeFixProvider.cs`\naround lines 33 - 36, The CodeAction created in the CodeAction.Create call\ncurrently uses nameof(Al1212UseGuardNotNullOrEmptyCodeFixProvider) as the\nequivalenceKey; change it to use\nAl1212UseGuardNotNullOrEmptyAnalyzer.DiagnosticId instead so FixAll works\ncorrectly — update the CodeAction.Create invocation that supplies\nAL1212CodeFixTitle and the ConvertToGuardNotNullOrEmpty(document, ifStatement,\nroot) delegate to pass Al1212UseGuardNotNullOrEmptyAnalyzer.DiagnosticId as the\nequivalenceKey.\n```\n\n
\n\n\n\n","src/ANcpLua.Analyzers.CodeFixes/CodeFixes/AL1212UseGuardNotNullOrEmptyCodeFixProvider.cs","_⚠️ Potential issue_ | _🔴 Critical_ | _⚡ Quick win_\n\n**equivalenceKey must be the analyzer's DiagnosticId, not the provider name.**\n\nLine 36 passes `nameof(Al1211UseAttributeExtensionsCodeFixProvider)` as the equivalenceKey. Per coding guidelines, this must be `Al1211UseAttributeExtensionsAnalyzer.DiagnosticId` for FixAll correctness.\n\n\n\n\n\n🔧 Fix equivalenceKey
\n\n```diff\n CodeAction.Create(\n CodeFixResources.AL1211CodeFixTitle,\n _ => ConvertToGetConstructorArgument(context.Document, root, node, attrExpr, indexExpr, typeArgument),\n- nameof(Al1211UseAttributeExtensionsCodeFixProvider)),\n+ Al1211UseAttributeExtensionsAnalyzer.DiagnosticId),\n```\n
\n\nAs per coding guidelines: \"CreateCodeAction method must set equivalenceKey to the analyzer's DiagnosticId to enable FixAll support\"\n\n\n\n\n📝 Committable suggestion
\n\n> ‼️ **IMPORTANT**\n> Carefully review the code before committing. Ensure that it accurately replaces the highlighted code, contains no missing lines, and has no issues with indentation. Thoroughly test & benchmark the code to ensure it meets the requirements.\n\n```suggestion\n CodeAction.Create(\n CodeFixResources.AL1211CodeFixTitle,\n _ => ConvertToGetConstructorArgument(context.Document, root, node, attrExpr, indexExpr, typeArgument),\n Al1211UseAttributeExtensionsAnalyzer.DiagnosticId),\n```\n\n
\n\n\n\n\n🤖 Prompt for AI Agents
\n\n```\nVerify each finding against current code. Fix only still-valid issues, skip the\nrest with a brief reason, keep changes minimal, and validate.\n\nIn\n`@src/ANcpLua.Analyzers.CodeFixes/CodeFixes/AL1211UseAttributeExtensionsCodeFixProvider.cs`\naround lines 33 - 36, The CodeAction created in\nAl1211UseAttributeExtensionsCodeFixProvider currently uses\nnameof(Al1211UseAttributeExtensionsCodeFixProvider) as the equivalenceKey;\nchange it to use Al1211UseAttributeExtensionsAnalyzer.DiagnosticId instead so\nFixAll works correctly. Locate the CodeAction.Create call (and related\noverloads) around where ConvertToGetConstructorArgument(...) is passed and\nreplace the equivalenceKey argument with\nAl1211UseAttributeExtensionsAnalyzer.DiagnosticId, ensuring the symbol is\nreferenced/imported if necessary.\n```\n\n
\n\n\n\n","src/ANcpLua.Analyzers.CodeFixes/CodeFixes/AL1211UseAttributeExtensionsCodeFixProvider.cs","_⚠️ Potential issue_ | _🔴 Critical_ | _⚡ Quick win_\n\n**equivalenceKey must be the analyzer's DiagnosticId, not the provider name.**\n\nLine 42 passes `nameof(Al1210UseStringComparisonExtensionsCodeFixProvider)` as the equivalenceKey. Per coding guidelines, this must be `Al1210UseStringComparisonExtensionsAnalyzer.DiagnosticId` for FixAll correctness.\n\n\n\n\n\n🔧 Fix equivalenceKey
\n\n```diff\n CodeAction.Create(\n CodeFixResources.AL1210CodeFixTitle,\n _ => ConvertToExtensionMethod(document, invocation, root),\n- nameof(Al1210UseStringComparisonExtensionsCodeFixProvider));\n+ Al1210UseStringComparisonExtensionsAnalyzer.DiagnosticId);\n```\n
\n\nAs per coding guidelines: \"CreateCodeAction method must set equivalenceKey to the analyzer's DiagnosticId to enable FixAll support\"\n\n\n\n\n📝 Committable suggestion
\n\n> ‼️ **IMPORTANT**\n> Carefully review the code before committing. Ensure that it accurately replaces the highlighted code, contains no missing lines, and has no issues with indentation. Thoroughly test & benchmark the code to ensure it meets the requirements.\n\n```suggestion\n CodeAction.Create(\n CodeFixResources.AL1210CodeFixTitle,\n _ => ConvertToExtensionMethod(document, invocation, root),\n Al1210UseStringComparisonExtensionsAnalyzer.DiagnosticId);\n```\n\n
\n\n\n\n\n🤖 Prompt for AI Agents
\n\n```\nVerify each finding against current code. Fix only still-valid issues, skip the\nrest with a brief reason, keep changes minimal, and validate.\n\nIn\n`@src/ANcpLua.Analyzers.CodeFixes/CodeFixes/AL1210UseStringComparisonExtensionsCodeFixProvider.cs`\naround lines 39 - 42, The CodeAction.Create call currently uses\nnameof(Al1210UseStringComparisonExtensionsCodeFixProvider) as the\nequivalenceKey; change it to use\nAl1210UseStringComparisonExtensionsAnalyzer.DiagnosticId instead so FixAll works\ncorrectly—update the call that constructs the CodeAction (the one passing\nCodeFixResources.AL1210CodeFixTitle and the lambda that calls\nConvertToExtensionMethod(document, invocation, root)) to set equivalenceKey to\nAl1210UseStringComparisonExtensionsAnalyzer.DiagnosticId rather than the\nprovider name.\n```\n\n
\n\n\n\n","src/ANcpLua.Analyzers.CodeFixes/CodeFixes/AL1210UseStringComparisonExtensionsCodeFixProvider.cs","_⚠️ Potential issue_ | _🔴 Critical_ | _⚡ Quick win_\n\n**equivalenceKey must be the analyzer's DiagnosticId, not the provider name.**\n\nLine 30 passes `nameof(Al1209UseTryParseExtensionsCodeFixProvider)` as the equivalenceKey. Per coding guidelines, this must be `Al1209UseTryParseExtensionsAnalyzer.DiagnosticId` for FixAll correctness.\n\n\n\n\n\n🔧 Fix equivalenceKey
\n\n```diff\n CodeAction.Create(\n CodeFixResources.AL1209CodeFixTitle,\n _ => ConvertToExtension(document, conditional, root),\n- nameof(Al1209UseTryParseExtensionsCodeFixProvider));\n+ Al1209UseTryParseExtensionsAnalyzer.DiagnosticId);\n```\n
\n\nAs per coding guidelines: \"CreateCodeAction method must set equivalenceKey to the analyzer's DiagnosticId to enable FixAll support\"\n\n\n\n\n📝 Committable suggestion
\n\n> ‼️ **IMPORTANT**\n> Carefully review the code before committing. Ensure that it accurately replaces the highlighted code, contains no missing lines, and has no issues with indentation. Thoroughly test & benchmark the code to ensure it meets the requirements.\n\n```suggestion\n CodeAction.Create(\n CodeFixResources.AL1209CodeFixTitle,\n _ => ConvertToExtension(document, conditional, root),\n Al1209UseTryParseExtensionsAnalyzer.DiagnosticId);\n```\n\n
\n\n\n\n\n🤖 Prompt for AI Agents
\n\n```\nVerify each finding against current code. Fix only still-valid issues, skip the\nrest with a brief reason, keep changes minimal, and validate.\n\nIn\n`@src/ANcpLua.Analyzers.CodeFixes/CodeFixes/AL1209UseTryParseExtensionsCodeFixProvider.cs`\naround lines 27 - 30, The CodeAction.Create call currently uses\nnameof(Al1209UseTryParseExtensionsCodeFixProvider) as the equivalenceKey; change\nit to use Al1209UseTryParseExtensionsAnalyzer.DiagnosticId instead so FixAll\nworks correctly — update the CodeAction.Create invocation that passes\nAL1209CodeFixTitle and the ConvertToExtension callback to set the third argument\nto Al1209UseTryParseExtensionsAnalyzer.DiagnosticId (leave the title and\ncallback unchanged).\n```\n\n
\n\n\n\n","src/ANcpLua.Analyzers.CodeFixes/CodeFixes/AL1209UseTryParseExtensionsCodeFixProvider.cs","_⚠️ Potential issue_ | _🔴 Critical_ | _⚡ Quick win_\n\n**equivalenceKey must be the analyzer's DiagnosticId, not the provider name.**\n\nLine 30 passes `nameof(Al1208UseGuardNotNullCodeFixProvider)` as the equivalenceKey. Per coding guidelines, this must be `Al1208UseGuardNotNullAnalyzer.DiagnosticId` for FixAll correctness.\n\n\n\n\n\n🔧 Fix equivalenceKey
\n\n```diff\n CodeAction.Create(\n CodeFixResources.AL1208CodeFixTitle,\n _ => ConvertToGuardNotNull(document, coalesce, root),\n- nameof(Al1208UseGuardNotNullCodeFixProvider));\n+ Al1208UseGuardNotNullAnalyzer.DiagnosticId);\n```\n
\n\nAs per coding guidelines: \"CreateCodeAction method must set equivalenceKey to the analyzer's DiagnosticId to enable FixAll support\"\n\n\n\n\n📝 Committable suggestion
\n\n> ‼️ **IMPORTANT**\n> Carefully review the code before committing. Ensure that it accurately replaces the highlighted code, contains no missing lines, and has no issues with indentation. Thoroughly test & benchmark the code to ensure it meets the requirements.\n\n```suggestion\n CodeAction.Create(\n CodeFixResources.AL1208CodeFixTitle,\n _ => ConvertToGuardNotNull(document, coalesce, root),\n Al1208UseGuardNotNullAnalyzer.DiagnosticId);\n```\n\n
\n\n\n\n\n🤖 Prompt for AI Agents
\n\n```\nVerify each finding against current code. Fix only still-valid issues, skip the\nrest with a brief reason, keep changes minimal, and validate.\n\nIn\n`@src/ANcpLua.Analyzers.CodeFixes/CodeFixes/AL1208UseGuardNotNullCodeFixProvider.cs`\naround lines 27 - 30, The CodeAction call uses\nnameof(Al1208UseGuardNotNullCodeFixProvider) as the equivalenceKey which breaks\nFixAll; change the equivalenceKey argument in the CodeAction.Create invocation\n(the call that passes CodeFixResources.AL1208CodeFixTitle and _ =>\nConvertToGuardNotNull(document, coalesce, root)) to use\nAl1208UseGuardNotNullAnalyzer.DiagnosticId instead of\nnameof(Al1208UseGuardNotNullCodeFixProvider) so the action’s equivalenceKey\nmatches the analyzer diagnostic ID and enables FixAll correctness.\n```\n\n
\n\n\n\n","src/ANcpLua.Analyzers.CodeFixes/CodeFixes/AL1208UseGuardNotNullCodeFixProvider.cs","_⚠️ Potential issue_ | _🔴 Critical_ | _⚡ Quick win_\n\n**equivalenceKey must be the analyzer's DiagnosticId, not the provider name.**\n\nLine 42 passes `nameof(Al1207UseToDisplayStringExtensionsCodeFixProvider)` as the equivalenceKey. Per coding guidelines, this must be `Al1207UseToDisplayStringExtensionsAnalyzer.DiagnosticId` for FixAll correctness.\n\n\n\n\n\n🔧 Fix equivalenceKey
\n\n```diff\n CodeAction.Create(\n CodeFixResources.AL1207CodeFixTitle,\n _ => ConvertToExtension(document, invocation, root),\n- nameof(Al1207UseToDisplayStringExtensionsCodeFixProvider));\n+ Al1207UseToDisplayStringExtensionsAnalyzer.DiagnosticId);\n```\n
\n\nAs per coding guidelines: \"CreateCodeAction method must set equivalenceKey to the analyzer's DiagnosticId to enable FixAll support\"\n\n\n\n\n📝 Committable suggestion
\n\n> ‼️ **IMPORTANT**\n> Carefully review the code before committing. Ensure that it accurately replaces the highlighted code, contains no missing lines, and has no issues with indentation. Thoroughly test & benchmark the code to ensure it meets the requirements.\n\n```suggestion\n CodeAction.Create(\n CodeFixResources.AL1207CodeFixTitle,\n _ => ConvertToExtension(document, invocation, root),\n Al1207UseToDisplayStringExtensionsAnalyzer.DiagnosticId);\n```\n\n
\n\n\n\n\n🤖 Prompt for AI Agents
\n\n```\nVerify each finding against current code. Fix only still-valid issues, skip the\nrest with a brief reason, keep changes minimal, and validate.\n\nIn\n`@src/ANcpLua.Analyzers.CodeFixes/CodeFixes/AL1207UseToDisplayStringExtensionsCodeFixProvider.cs`\naround lines 39 - 42, The CodeAction.Create call in\nAL1207UseToDisplayStringExtensionsCodeFixProvider is using\nnameof(Al1207UseToDisplayStringExtensionsCodeFixProvider) as the equivalenceKey;\nchange it to use the analyzer's DiagnosticId\n(Al1207UseToDisplayStringExtensionsAnalyzer.DiagnosticId) so FixAll works\ncorrectly. Locate the CodeAction.Create invocation (the call that passes\nCodeFixResources.AL1207CodeFixTitle and the ConvertToExtension lambda) and\nreplace the equivalenceKey argument with\nAl1207UseToDisplayStringExtensionsAnalyzer.DiagnosticId.\n```\n\n
\n\n\n\n","src/ANcpLua.Analyzers.CodeFixes/CodeFixes/AL1207UseToDisplayStringExtensionsCodeFixProvider.cs","_⚠️ Potential issue_ | _🔴 Critical_ | _⚡ Quick win_\n\n**equivalenceKey must be the analyzer's DiagnosticId, not the provider name.**\n\nLine 30 passes `nameof(Al1206UseWhereNotNullCodeFixProvider)` as the equivalenceKey. Per coding guidelines, this must be `Al1206UseWhereNotNullAnalyzer.DiagnosticId` for FixAll correctness. FixAll groups actions by diagnostic ID, not provider name.\n\n\n\n\n\n🔧 Fix equivalenceKey
\n\n```diff\n CodeAction.Create(\n CodeFixResources.AL1206CodeFixTitle,\n _ => ConvertToWhereNotNull(document, invocation, root),\n- nameof(Al1206UseWhereNotNullCodeFixProvider));\n+ Al1206UseWhereNotNullAnalyzer.DiagnosticId);\n```\n
\n\nAs per coding guidelines: \"CreateCodeAction method must set equivalenceKey to the analyzer's DiagnosticId to enable FixAll support\"\n\n\n\n\n📝 Committable suggestion
\n\n> ‼️ **IMPORTANT**\n> Carefully review the code before committing. Ensure that it accurately replaces the highlighted code, contains no missing lines, and has no issues with indentation. Thoroughly test & benchmark the code to ensure it meets the requirements.\n\n```suggestion\n CodeAction.Create(\n CodeFixResources.AL1206CodeFixTitle,\n _ => ConvertToWhereNotNull(document, invocation, root),\n Al1206UseWhereNotNullAnalyzer.DiagnosticId);\n```\n\n
\n\n\n\n\n🤖 Prompt for AI Agents
\n\n```\nVerify each finding against current code. Fix only still-valid issues, skip the\nrest with a brief reason, keep changes minimal, and validate.\n\nIn\n`@src/ANcpLua.Analyzers.CodeFixes/CodeFixes/AL1206UseWhereNotNullCodeFixProvider.cs`\naround lines 27 - 30, The CodeAction created in\nAl1206UseWhereNotNullCodeFixProvider currently uses\nnameof(Al1206UseWhereNotNullCodeFixProvider) as the equivalenceKey; change that\nthird argument to Al1206UseWhereNotNullAnalyzer.DiagnosticId so the\nCodeAction.Create call (the same call that wraps ConvertToWhereNotNull) uses the\nanalyzer's DiagnosticId as the equivalenceKey to enable correct FixAll grouping.\n```\n\n
\n\n\n\n","src/ANcpLua.Analyzers.CodeFixes/CodeFixes/AL1206UseWhereNotNullCodeFixProvider.cs","_⚠️ Potential issue_ | _🟠 Major_ | _⚡ Quick win_\n\n**Set equivalence key to `Al1205UseToImmutableArrayOrEmptyAnalyzer.DiagnosticId`.**\n\nCurrent provider-name key does not satisfy the repository’s FixAll equivalence-key contract.\n\n \nAs per coding guidelines: `CreateCodeAction method must set equivalenceKey to the analyzer's DiagnosticId to enable FixAll support`.\n\n\n🤖 Prompt for AI Agents
\n\n```\nVerify each finding against current code. Fix only still-valid issues, skip the\nrest with a brief reason, keep changes minimal, and validate.\n\nIn\n`@src/ANcpLua.Analyzers.CodeFixes/CodeFixes/AL1205UseToImmutableArrayOrEmptyCodeFixProvider.cs`\naround lines 25 - 28, The CodeAction.Create call in\nAl1205UseToImmutableArrayOrEmptyCodeFixProvider is using\nnameof(Al1205UseToImmutableArrayOrEmptyCodeFixProvider) as the equivalence key;\nchange that last argument to\nAl1205UseToImmutableArrayOrEmptyAnalyzer.DiagnosticId so the created CodeAction\nuses the analyzer's DiagnosticId for equivalence (leave the title and the\ndelegate to ConvertToExtension(document, coalesce, root) unchanged).\n```\n\n
\n\n\n\n","src/ANcpLua.Analyzers.CodeFixes/CodeFixes/AL1205UseToImmutableArrayOrEmptyCodeFixProvider.cs","_⚠️ Potential issue_ | _🟠 Major_ | _⚡ Quick win_\n\n**Equivalence key should be `Al1204UseOrEmptyAnalyzer.DiagnosticId`.**\n\nUsing the provider name here is inconsistent with the required FixAll key pattern.\n\n \nAs per coding guidelines: `CreateCodeAction method must set equivalenceKey to the analyzer's DiagnosticId to enable FixAll support`.\n\n\n🤖 Prompt for AI Agents
\n\n```\nVerify each finding against current code. Fix only still-valid issues, skip the\nrest with a brief reason, keep changes minimal, and validate.\n\nIn `@src/ANcpLua.Analyzers.CodeFixes/CodeFixes/AL1204UseOrEmptyCodeFixProvider.cs`\naround lines 28 - 31, The CodeAction created in\nAL1204UseOrEmptyCodeFixProvider.cs currently uses\nnameof(Al1204UseOrEmptyCodeFixProvider) as the equivalence key; change the\nCodeAction.Create call that wraps ConvertToOrEmpty(document, coalesce, root) so\nits equivalenceKey is Al1204UseOrEmptyAnalyzer.DiagnosticId instead (i.e., pass\nAl1204UseOrEmptyAnalyzer.DiagnosticId as the third argument) to enable proper\nFixAll behavior and keep ConvertToOrEmpty and the surrounding method intact.\n```\n\n
\n\n\n\n","src/ANcpLua.Analyzers.CodeFixes/CodeFixes/AL1204UseOrEmptyCodeFixProvider.cs","_⚠️ Potential issue_ | _🟠 Major_ | _⚡ Quick win_\n\n**Replace provider-name equivalence keys with `Al1203UseOperationExtensionsAnalyzer.DiagnosticId`.**\n\nBoth actions currently use provider-name-derived keys, which violates the FixAll key contract.\n\n \nAs per coding guidelines: `CreateCodeAction method must set equivalenceKey to the analyzer's DiagnosticId to enable FixAll support`.\n\n\nAlso applies to: 39-42\n\n\n🤖 Prompt for AI Agents
\n\n```\nVerify each finding against current code. Fix only still-valid issues, skip the\nrest with a brief reason, keep changes minimal, and validate.\n\nIn\n`@src/ANcpLua.Analyzers.CodeFixes/CodeFixes/AL1203UseOperationExtensionsCodeFixProvider.cs`\naround lines 31 - 34, The CodeAction.Create calls in\nAl1203UseOperationExtensionsCodeFixProvider currently use provider-name-derived\nequivalence keys (e.g., nameof(Al1203UseOperationExtensionsCodeFixProvider) +\n\"_IsMethodNamed\"); change both CodeAction.Create invocations (the one that calls\nConvertToIsMethodNamed and the other that calls ConvertToIsOperationNamed) to\nuse Al1203UseOperationExtensionsAnalyzer.DiagnosticId as the equivalenceKey so\nFixAll works correctly.\n```\n\n
\n\n\n\n","src/ANcpLua.Analyzers.CodeFixes/CodeFixes/AL1203UseOperationExtensionsCodeFixProvider.cs","_⚠️ Potential issue_ | _🟠 Major_ | _⚡ Quick win_\n\n**Both code actions need analyzer-ID equivalence keys.**\n\nSuffixing provider names for equivalence keys breaks the FixAll key contract in this repo.\n\n \nAs per coding guidelines: `CreateCodeAction method must set equivalenceKey to the analyzer's DiagnosticId to enable FixAll support`.\n\n\nAlso applies to: 45-48\n\n\n🤖 Prompt for AI Agents
\n\n```\nVerify each finding against current code. Fix only still-valid issues, skip the\nrest with a brief reason, keep changes minimal, and validate.\n\nIn\n`@src/ANcpLua.Analyzers.CodeFixes/CodeFixes/AL1202UseTypeHierarchyCodeFixProvider.cs`\naround lines 35 - 38, The CodeAction.Create calls (the one creating the\nimplements action that calls ConvertToImplements and the other corresponding\naction on lines 45-48) currently use a provider-suffixed string for equivalence\nkeys which breaks FixAll; change their equivalenceKey arguments to the analyzer\ndiagnostic id (AL1202) instead of nameof(Al1202UseTypeHierarchyCodeFixProvider)\n+ \"_Implements\" (and the similar suffix used for the other action). Use the\nconstant/diagnostic id (e.g., DiagnosticIds.AL1202 or context.Diagnostics[0].Id)\nas the equivalenceKey when constructing the CodeAction so FixAll will work,\nleaving the title and the delegate (ConvertToImplements / the other conversion\nmethod) unchanged.\n```\n\n
\n\n\n\n","src/ANcpLua.Analyzers.CodeFixes/CodeFixes/AL1202UseTypeHierarchyCodeFixProvider.cs","_🛠️ Refactor suggestion_ | _🟠 Major_ | _🏗️ Heavy lift_\n\n**Code fix provider should use `AlCodeFixProvider` base.**\n\nThis file is still on `CodeFixProvider`, which diverges from the repo’s required provider shape.\n\n \nAs per coding guidelines: `Code fix providers must inherit from AlCodeFixProvider base class and use [ExportCodeFixProvider(LanguageNames.CSharp)] and [Shared] attributes`.\n\n\n🤖 Prompt for AI Agents
\n\n```\nVerify each finding against current code. Fix only still-valid issues, skip the\nrest with a brief reason, keep changes minimal, and validate.\n\nIn\n`@src/ANcpLua.Analyzers.CodeFixes/CodeFixes/AL1202UseTypeHierarchyCodeFixProvider.cs`\naround lines 15 - 16, The class Al1202UseTypeHierarchyCodeFixProvider currently\ninherits directly from CodeFixProvider; change it to inherit from\nAlCodeFixProvider and apply the\n[ExportCodeFixProvider(LanguageNames.CSharp)] and [Shared] attributes above the\nclass declaration so it matches the repository provider shape; ensure you\npreserve existing members like FixableDiagnosticIds and any overrides required\nby AlCodeFixProvider.\n```\n\n
\n\n\n\n","_⚠️ Potential issue_ | _🟠 Major_ | _⚡ Quick win_\n\n**Use analyzer DiagnosticId as equivalence key.**\n\n`nameof(Al1201UseHasAttributeCodeFixProvider)` will fragment FixAll behavior versus the repository contract.\n\n \nAs per coding guidelines: `CreateCodeAction method must set equivalenceKey to the analyzer's DiagnosticId to enable FixAll support`.\n\n\n🤖 Prompt for AI Agents
\n\n```\nVerify each finding against current code. Fix only still-valid issues, skip the\nrest with a brief reason, keep changes minimal, and validate.\n\nIn\n`@src/ANcpLua.Analyzers.CodeFixes/CodeFixes/AL1201UseHasAttributeCodeFixProvider.cs`\naround lines 32 - 35, The CodeAction.Create call currently uses\nnameof(Al1201UseHasAttributeCodeFixProvider) as the equivalenceKey which breaks\nFixAll; change the equivalenceKey to the analyzer's DiagnosticId (e.g., use the\nAL1201 analyzer's DiagnosticId constant/property) when creating the CodeAction\nfor ConvertToHasAttribute so the equivalence key equals the analyzer\nDiagnosticId (AL1201) instead of the codefix type name.\n```\n\n
\n\n\n\n","src/ANcpLua.Analyzers.CodeFixes/CodeFixes/AL1201UseHasAttributeCodeFixProvider.cs","_🛠️ Refactor suggestion_ | _🟠 Major_ | _🏗️ Heavy lift_\n\n**Provider base type violates the repo code-fix contract.**\n\nThis should inherit from `AlCodeFixProvider` instead of raw `CodeFixProvider` to stay aligned with the code-fix provider contract used in this repo.\n\n \nAs per coding guidelines: `Code fix providers must inherit from AlCodeFixProvider base class and use [ExportCodeFixProvider(LanguageNames.CSharp)] and [Shared] attributes`.\n\n\n🤖 Prompt for AI Agents
\n\n```\nVerify each finding against current code. Fix only still-valid issues, skip the\nrest with a brief reason, keep changes minimal, and validate.\n\nIn\n`@src/ANcpLua.Analyzers.CodeFixes/CodeFixes/AL1201UseHasAttributeCodeFixProvider.cs`\naround lines 14 - 15, The class Al1201UseHasAttributeCodeFixProvider currently\ninherits from CodeFixProvider; change it to inherit from the repo base\nAlCodeFixProvider with the appropriate syntax node type used by this\nanalyzer (replace CodeFixProvider with AlCodeFixProvider using the\nanalyzer's node type), and add the required attributes\n[ExportCodeFixProvider(LanguageNames.CSharp)] and [Shared] to the class\ndeclaration so it follows the project's code-fix contract.\n```\n\n
\n\n\n\n","_⚠️ Potential issue_ | _🟠 Major_ | _⚡ Quick win_\n\n**Set equivalence key to `Al1200UseIsEqualToAnalyzer.DiagnosticId`.**\n\nUsing `nameof(Al1200UseIsEqualToCodeFixProvider)` here is out of contract for FixAll grouping.\n\n \nAs per coding guidelines: `CreateCodeAction method must set equivalenceKey to the analyzer's DiagnosticId to enable FixAll support`.\n\n\n🤖 Prompt for AI Agents
\n\n```\nVerify each finding against current code. Fix only still-valid issues, skip the\nrest with a brief reason, keep changes minimal, and validate.\n\nIn\n`@src/ANcpLua.Analyzers.CodeFixes/CodeFixes/AL1200UseIsEqualToCodeFixProvider.cs`\naround lines 21 - 24, The CodeAction.Create call in\nAl1200UseIsEqualToCodeFixProvider currently uses\nnameof(Al1200UseIsEqualToCodeFixProvider) as the equivalence key which breaks\nFixAll grouping; update that CodeAction.Create invocation to pass\nAl1200UseIsEqualToAnalyzer.DiagnosticId as the equivalenceKey instead (keep the\ntitle and the callback to ConvertToIsEqualTo(document, invocation, root)\nunchanged) so FixAll uses the analyzer's DiagnosticId for grouping.\n```\n\n
\n\n\n\n","src/ANcpLua.Analyzers.CodeFixes/CodeFixes/AL1200UseIsEqualToCodeFixProvider.cs","_⚠️ Potential issue_ | _🟠 Major_ | _⚡ Quick win_\n\n**FixAll equivalence key must be the analyzer ID, not provider name.**\n\n\nProposed fix
\n\n```diff\n return CodeAction.Create(\n CodeFixResources.AL1012CodeFixTitle,\n ct => CombineAsync(document, declaration, ifStatement, variableName, initializerValue, ct),\n- nameof(Al1012CombineDeclarationWithNullCheckCodeFixProvider));\n+ Al1012CombineDeclarationWithNullCheckAnalyzer.DiagnosticId);\n```\n
\n\n \nAs per coding guidelines: `CreateCodeAction method must set equivalenceKey to the analyzer's DiagnosticId to enable FixAll support`.\n\n\n\n\n📝 Committable suggestion
\n\n> ‼️ **IMPORTANT**\n> Carefully review the code before committing. Ensure that it accurately replaces the highlighted code, contains no missing lines, and has no issues with indentation. Thoroughly test & benchmark the code to ensure it meets the requirements.\n\n```suggestion\n return CodeAction.Create(\n CodeFixResources.AL1012CodeFixTitle,\n ct => CombineAsync(document, declaration, ifStatement, variableName, initializerValue, ct),\n Al1012CombineDeclarationWithNullCheckAnalyzer.DiagnosticId);\n```\n\n
\n\n\n\n\n🤖 Prompt for AI Agents
\n\n```\nVerify each finding against current code. Fix only still-valid issues, skip the\nrest with a brief reason, keep changes minimal, and validate.\n\nIn\n`@src/ANcpLua.Analyzers.CodeFixes/CodeFixes/AL1012CombineDeclarationWithNullCheckCodeFixProvider.cs`\naround lines 35 - 38, The CodeAction equivalenceKey is currently set to\nnameof(Al1012CombineDeclarationWithNullCheckCodeFixProvider) which prevents\nFixAll from working; change the third argument in the CodeAction.Create call to\nthe analyzer's DiagnosticId (for example use Al1012Analyzer.DiagnosticId or the\nconstant that holds \"AL1012\") so the equivalenceKey equals the analyzer ID; keep\nthe rest of the CodeAction.Create call (title and ct lambda invoking\nCombineAsync) unchanged.\n```\n\n
\n\n\n\n","src/ANcpLua.Analyzers.CodeFixes/CodeFixes/AL1012CombineDeclarationWithNullCheckCodeFixProvider.cs","_⚠️ Potential issue_ | _🟠 Major_ | _⚡ Quick win_\n\n**Use analyzer DiagnosticId as the code action equivalence key.**\n\nThis currently uses the provider type name, which breaks the repo’s FixAll key contract for code fixes.\n\n\nProposed fix
\n\n```diff\n return CodeAction.Create(\n title,\n ct => ApplyFixAsync(document, ifStatement, identifier, typeName, style, ct),\n- nameof(Al1011NormalizeNullGuardStyleCodeFixProvider));\n+ Al1011NormalizeNullGuardStyleAnalyzer.DiagnosticId);\n```\n
\n\n \nAs per coding guidelines: `CreateCodeAction method must set equivalenceKey to the analyzer's DiagnosticId to enable FixAll support`.\n\n\n\n\n📝 Committable suggestion
\n\n> ‼️ **IMPORTANT**\n> Carefully review the code before committing. Ensure that it accurately replaces the highlighted code, contains no missing lines, and has no issues with indentation. Thoroughly test & benchmark the code to ensure it meets the requirements.\n\n```suggestion\n return CodeAction.Create(\n title,\n ct => ApplyFixAsync(document, ifStatement, identifier, typeName, style, ct),\n Al1011NormalizeNullGuardStyleAnalyzer.DiagnosticId);\n```\n\n
\n\n\n\n\n🤖 Prompt for AI Agents
\n\n```\nVerify each finding against current code. Fix only still-valid issues, skip the\nrest with a brief reason, keep changes minimal, and validate.\n\nIn\n`@src/ANcpLua.Analyzers.CodeFixes/CodeFixes/AL1011NormalizeNullGuardStyleCodeFixProvider.cs`\naround lines 34 - 37, The code action currently uses\nnameof(Al1011NormalizeNullGuardStyleCodeFixProvider) as the equivalenceKey which\nbreaks FixAll; change the equivalenceKey passed to CodeAction.Create to the\nanalyzer's DiagnosticId (e.g., use the DiagnosticId constant or descriptor.Id\nfor AL1011) instead of the provider type name so the Create call uses the\nanalyzer DiagnosticId as the equivalence key; update the CodeAction.Create\ninvocation in AL1011NormalizeNullGuardStyleCodeFixProvider to pass the\nDiagnosticId value.\n```\n\n
\n\n\n\n","src/ANcpLua.Analyzers.CodeFixes/CodeFixes/AL1011NormalizeNullGuardStyleCodeFixProvider.cs","_⚠️ Potential issue_ | _🔴 Critical_ | _⚡ Quick win_\n\n**Fix CodeAction contract for AL1010 pattern-matching code fix** \nThe AL1010 CodeAction title/equivalenceKey need to follow the established pattern (use the resource-backed title constant and the analyzer DiagnosticId as the equivalenceKey) instead of hardcoding the title and using `nameof(Al1010PatternMatchingCodeFixProvider)`.\n\n- Check `src/.../AL1010PatternMatchingCodeFixProvider.cs` for the exact `CodeAction.Create` title and `equivalenceKey` values, then align with how other `AL*CodeFixProvider` implementations in this repo set them.\n- Ensure the equivalenceKey matches the corresponding analyzer `DiagnosticId`, and the title uses the matching `CodeFixResources` entry for that diagnostic.\n\n\n🤖 Prompt for AI Agents
\n\n```\nVerify each finding against current code. Fix only still-valid issues, skip the\nrest with a brief reason, keep changes minimal, and validate.\n\nIn\n`@src/ANcpLua.Analyzers.CodeFixes/CodeFixes/AL1010PatternMatchingCodeFixProvider.cs`\naround lines 18 - 21, Replace the hardcoded CodeAction title and equivalenceKey\nin AL1010PatternMatchingCodeFixProvider with the repo convention: use the\nresource-backed title from CodeFixResources (the entry that corresponds to\nAL1010) as the first argument to CodeAction.Create, and use the analyzer\nDiagnosticId for the equivalenceKey (not\nnameof(Al1010PatternMatchingCodeFixProvider)); update the existing\nCodeAction.Create call that wraps ConvertToPatternMatching(document, binary,\nroot) so its title references the appropriate CodeFixResources constant and its\nequivalenceKey is the AL1010 analyzer DiagnosticId.\n```\n\n
\n\n\n\n","_⚠️ Potential issue_ | _🟠 Major_ | _⚡ Quick win_\n\n**`equivalenceKey` should use `Al1009LockKeywordAnalyzer.DiagnosticId`.**\n\nUsing `nameof(CodeFixResources.AL1009CodeFixTitle)` weakens FixAll identity guarantees.\n\n \nAs per coding guidelines, \"`equivalenceKey` must be set to the analyzer’s `DiagnosticId` for FixAll correctness.\"\n\n\n🤖 Prompt for AI Agents
\n\n```\nVerify each finding against current code. Fix only still-valid issues, skip the\nrest with a brief reason, keep changes minimal, and validate.\n\nIn `@src/ANcpLua.Analyzers.CodeFixes/CodeFixes/AL1009LockTypeCodeFixProvider.cs`\naround lines 44 - 47, The CodeAction Create call currently sets equivalenceKey\nto nameof(CodeFixResources.AL1009CodeFixTitle) which breaks FixAll identity;\nchange the equivalenceKey argument in the CodeAction.Create invocation that\nconstructs the fix for ChangeFieldTypeToLockAsync (the CodeAction created with\nCodeFixResources.AL1009CodeFixTitle) to use\nAl1009LockKeywordAnalyzer.DiagnosticId instead so the fix’s equivalence key\nequals the analyzer DiagnosticId.\n```\n\n
\n\n\n\n","src/ANcpLua.Analyzers.CodeFixes/CodeFixes/AL1009LockTypeCodeFixProvider.cs","_⚠️ Potential issue_ | _🔴 Critical_ | _⚡ Quick win_\n\n**FixAll `equivalenceKey` must match the provider/analyzer identity**\n\nIn `src/ANcpLua.Analyzers/CodeFixes/CodeFixes/AL1007IXmlSerializableCodeFixProvider.cs`, `CodeAction.Create(..., nameof(CodeFixResources.AL1007CodeFixTitle))` is using the resource name as `equivalenceKey`; this doesn’t tie to the code fix’s targeted diagnostic/analyzer ID. Use the provider’s analyzer diagnostic ID (e.g., `DiagnosticIdAl1007`) as the `equivalenceKey` so FixAll can correctly apply only for AL1007.\n\n\n🤖 Prompt for AI Agents
\n\n```\nVerify each finding against current code. Fix only still-valid issues, skip the\nrest with a brief reason, keep changes minimal, and validate.\n\nIn\n`@src/ANcpLua.Analyzers.CodeFixes/CodeFixes/AL1007IXmlSerializableCodeFixProvider.cs`\naround lines 54 - 57, The CodeAction in AL1007IXmlSerializableCodeFixProvider is\nusing nameof(CodeFixResources.AL1007CodeFixTitle) as the equivalenceKey which\nbreaks FixAll; update the CodeAction.Create call to use the analyzer diagnostic\nID (DiagnosticIdAl1007) as the equivalenceKey instead so FixAll targets only\nAL1007, i.e., replace the third argument in the CodeAction.Create invocation\nwith the DiagnosticIdAl1007 symbol (or the constant defined in the corresponding\nanalyzer) while keeping the title and the FixAsync(context.Document, target,\nroot) callback unchanged.\n```\n\n
\n\n\n\n","src/ANcpLua.Analyzers.CodeFixes/CodeFixes/AL1007IXmlSerializableCodeFixProvider.cs","_🛠️ Refactor suggestion_ | _🟠 Major_ | _🏗️ Heavy lift_\n\n**Align provider with `AlCodeFixProvider` base-class contract.**\n\nThis provider still derives from `CodeFixProvider`, which drifts from the repository’s code-fix provider contract for `CodeFixes/AL*.cs`.\n\n \nAs per coding guidelines, \"Code fix providers must inherit from `AlCodeFixProvider` base class and use `[ExportCodeFixProvider(LanguageNames.CSharp)]` and `[Shared]` attributes.\"\n\n\n🤖 Prompt for AI Agents
\n\n```\nVerify each finding against current code. Fix only still-valid issues, skip the\nrest with a brief reason, keep changes minimal, and validate.\n\nIn\n`@src/ANcpLua.Analyzers.CodeFixes/CodeFixes/AL1007IXmlSerializableCodeFixProvider.cs`\nat line 10, The class currently inherits directly from CodeFixProvider; change\nits declaration to inherit from\nAlCodeFixProvider (i.e., replace\n\"CodeFixProvider\" with\n\"AlCodeFixProvider\") and decorate the\nclass with [ExportCodeFixProvider(LanguageNames.CSharp)] and [Shared]\nattributes; also add any missing using directives for\nMicrosoft.CodeAnalysis.CodeFixes and System.Composition so the attributes and\nbase type resolve. Ensure all existing overrides match the AlCodeFixProvider\ncontract (adjust signatures if the base generic requires them).\n```\n\n
\n\n\n\n","_⚠️ Potential issue_ | _🟠 Major_ | _⚡ Quick win_\n\n\n🧩 Analysis chain
\n\n🏁 Script executed:\n\n```shell\n#!/bin/bash\nset -euo pipefail\n\n# 1) Show the relevant code in the file under review\nFILE=\"src/ANcpLua.Analyzers.CodeFixes/CodeFixes/AL1004UseSequenceEqualCodeFixProvider.cs\"\necho \"---- $FILE (around lines 1-120) ----\"\nnl -ba \"$FILE\" | sed -n '1,140p'\n\n# 2) Locate the FixAll/equivalenceKey contract in the repo\necho \"---- Searching for CLAUDE.md contract ----\"\nfind src -maxdepth 5 -name \"CLAUDE.md\" -print\n\n# print the CLAUDE.md relevant section if present\nCLAUDE=\"$(find src -maxdepth 5 -name \"CLAUDE.md\" -print | head -n 1 || true)\"\nif [ -n \"${CLAUDE:-}\" ]; then\n echo \"---- $CLAUDE (grep for equivalenceKey/FixAll/DiagnosticId) ----\"\n nl -ba \"$CLAUDE\" | rg -n \"equivalenceKey|FixAll|DiagnosticIdAl1004|DiagnosticId|CodeAction\\.Create\" || true\nfi\n\n# 3) Find the analyzer DiagnosticId for AL1004 in the analyzers code\necho \"---- Searching for AL1004 analyzer DiagnosticId definition ----\"\nrg -n --hidden -S \"AL1004\" src/ANcpLua.Analyzers | head -n 50\n\n# attempt to find a DiagnosticId constant in the codebase\nrg -n --hidden -S \"DiagnosticId.*AL1004|AL1004.*DiagnosticId|DiagnosticIdAl1004\" src/ANcpLua.Analyzers | head -n 50\n\n# 4) Confirm the code-fix provider’s FixableDiagnosticIds and any declared DiagnosticId constants\necho \"---- Searching in AL1004 code-fix provider for FixableDiagnosticIds/DiagnosticId constants ----\"\nrg -n --hidden -S \"FixableDiagnosticIds|DiagnosticId|equivalenceKey\" \"$FILE\" || true\n\n# 5) Locate any other code fixes that show the correct equivalenceKey pattern (for comparison)\necho \"---- Searching other AL*.cs code fixes for equivalenceKey usage ----\"\nrg -n --hidden -S \"CodeAction\\.Create\\\$.*equivalenceKey|equivalenceKey.*DiagnosticId|nameof\\\\(CodeFixResources\\\\..*\\\$\" src/ANcpLua.Analyzers.CodeFixes/CodeFixes | head -n 50 || true\n```\n\nRepository: ANcpLua/ANcpLua.Analyzers\n\nLength of output: 244\n\n---\n\n🏁 Script executed:\n\n```shell\n#!/bin/bash\nset -euo pipefail\n\nFILE=\"src/ANcpLua.Analyzers.CodeFixes/CodeFixes/AL1004UseSequenceEqualCodeFixProvider.cs\"\necho \"---- $FILE (lines 1-200) ----\"\ncat -n \"$FILE\" | sed -n '1,200p'\n\necho \"---- Searching for CLAUDE.md contract ----\"\nfind src -name \"CLAUDE.md\" -print\n\nCLAUDE_PATH=\"$(find src -name \"CLAUDE.md\" -print | head -n 1 || true)\"\nif [ -n \"${CLAUDE_PATH}\" ]; then\n echo \"---- Using $CLAUDE_PATH (relevant markers) ----\"\n cat -n \"$CLAUDE_PATH\" | rg -n \"equivalenceKey|FixAll|DiagnosticId|CodeAction\\.Create\" || true\nelse\n echo \"---- No CLAUDE.md found under src ----\"\nfi\n\necho \"---- Searching for AL1004 DiagnosticId definitions in analyzers ----\"\nrg -n --hidden -S \"AL1004\" src/ANcpLua.Analyzers | head -n 50 || true\nrg -n --hidden -S \"DiagnosticId.*AL1004|AL1004.*DiagnosticId|DiagnosticIdAl1004\" src/ANcpLua.Analyzers | head -n 50 || true\n\necho \"---- Inspecting AL1004 code-fix provider for FixAll/equivalenceKey and FixableDiagnosticIds ----\"\ncat -n \"$FILE\" | rg -n \"FixableDiagnosticIds|equivalenceKey|DiagnosticId|FixAll|DiagnosticIds\" || true\n\necho \"---- Comparing equivalenceKey usage in other code-fix providers ----\"\nrg -n --hidden -S \"equivalenceKey\" src/ANcpLua.Analyzers.CodeFixes/CodeFixes | head -n 80 || true\n\n# Find patterns for equivalenceKey set to DiagnosticId or DiagnosticId constants\nrg -n --hidden -S \"equivalenceKey.*DiagnosticId|nameof\\\\(.*DiagnosticId|equivalenceKey.*nameof\\\\(.*DiagnosticId\" src/ANcpLua.Analyzers.CodeFixes/CodeFixes || true\n```\n\nRepository: ANcpLua/ANcpLua.Analyzers\n\nLength of output: 6940\n\n---\n\n🏁 Script executed:\n\n```shell\n#!/bin/bash\nset -euo pipefail\n\n# Show the relevant CLAUDE.md guidance for equivalenceKey/FixAll\necho \"---- src/ANcpLua.Analyzers.CodeFixes/CLAUDE.md (lines 1-140) ----\"\ncat -n src/ANcpLua.Analyzers.CodeFixes/CLAUDE.md | sed -n '1,160p'\n\necho \"---- src/ANcpLua.Analyzers/CLAUDE.md (lines 1-140) ----\"\ncat -n src/ANcpLua.Analyzers/CLAUDE.md | sed -n '1,160p'\n\n# Show analyzer definition for AL1004 diagnostic id constant\necho \"---- AL1003ToAL1004SpanComparisonAnalyzer.cs (lines 1-120) ----\"\ncat -n src/ANcpLua.Analyzers/Analyzers/AL1003ToAL1004SpanComparisonAnalyzer.cs | sed -n '1,140p'\n```\n\nRepository: ANcpLua/ANcpLua.Analyzers\n\nLength of output: 19526\n\n---\n\n\n\n
\n\n**Set FixAll `equivalenceKey` to the AL1004 analyzer DiagnosticId**\n\nIn `src/ANcpLua.Analyzers.CodeFixes/CodeFixes/AL1004UseSequenceEqualCodeFixProvider.cs` (lines 16-19), `equivalenceKey` is currently `nameof(CodeFixResources.AL1004CodeFixTitle)`; FixAll requires using the analyzer’s public diagnostic id. Change it to `Al1003ToAL1004SpanComparisonAnalyzer.DiagnosticIdAl1004` (`\"AL1004\"`).\n\n\n🤖 Prompt for AI Agents
\n\n```\nVerify each finding against current code. Fix only still-valid issues, skip the\nrest with a brief reason, keep changes minimal, and validate.\n\nIn\n`@src/ANcpLua.Analyzers.CodeFixes/CodeFixes/AL1004UseSequenceEqualCodeFixProvider.cs`\naround lines 16 - 19, The CodeAction created in\nAL1004UseSequenceEqualCodeFixProvider.cs currently uses\nnameof(CodeFixResources.AL1004CodeFixTitle) as the equivalenceKey; change that\nthird parameter in the CodeAction.Create call to use the analyzer's public\ndiagnostic id Al1003ToAL1004SpanComparisonAnalyzer.DiagnosticIdAl1004 (i.e.\n\"AL1004\") so FixAll groups correctly, referencing the CodeAction.Create\ninvocation and the Al1003ToAL1004SpanComparisonAnalyzer.DiagnosticIdAl1004\nsymbol.\n```\n\n
\n\n\n\n","src/ANcpLua.Analyzers.CodeFixes/CodeFixes/AL1004UseSequenceEqualCodeFixProvider.cs","_⚠️ Potential issue_ | _🟠 Major_ | _⚡ Quick win_\n\n**`equivalenceKey` should be the analyzer ID, not the resource property name.**\n\nUse the analyzer diagnostic ID to keep FixAll behavior stable.\n\n\nSuggested fix
\n\n```diff\n CodeAction.Create(\n CodeFixResources.AL1003CodeFixTitle,\n _ => UsePatternMatching(document, syntax, root),\n- nameof(CodeFixResources.AL1003CodeFixTitle));\n+ Al1003ToAl1004SpanComparisonAnalyzer.DiagnosticIdAl1003);\n```\n
\n\n \nAs per coding guidelines, \"`equivalenceKey` must be set to the analyzer’s `DiagnosticId` for FixAll correctness.\"\n\n\n\n\n📝 Committable suggestion
\n\n> ‼️ **IMPORTANT**\n> Carefully review the code before committing. Ensure that it accurately replaces the highlighted code, contains no missing lines, and has no issues with indentation. Thoroughly test & benchmark the code to ensure it meets the requirements.\n\n```suggestion\n CodeAction.Create(\n CodeFixResources.AL1003CodeFixTitle,\n _ => UsePatternMatching(document, syntax, root),\n Al1003ToAl1004SpanComparisonAnalyzer.DiagnosticIdAl1003);\n```\n\n
\n\n\n\n\n🤖 Prompt for AI Agents
\n\n```\nVerify each finding against current code. Fix only still-valid issues, skip the\nrest with a brief reason, keep changes minimal, and validate.\n\nIn\n`@src/ANcpLua.Analyzers.CodeFixes/CodeFixes/AL1003SpanPatternMatchingCodeFixProvider.cs`\naround lines 16 - 19, The CodeAction.Create call is using\nnameof(CodeFixResources.AL1003CodeFixTitle) as the equivalenceKey; change it to\nuse the analyzer diagnostic ID instead (e.g., use diagnostic.Id or the\nanalyzer's DiagnosticId constant such as AL1003Analyzer.DiagnosticId) so FixAll\nworks correctly; keep the title and the UsePatternMatching(document, syntax,\nroot) callback the same and only replace the equivalenceKey argument.\n```\n\n
\n\n\n\n","src/ANcpLua.Analyzers.CodeFixes/CodeFixes/AL1003SpanPatternMatchingCodeFixProvider.cs","_⚠️ Potential issue_ | _🟠 Major_ | _⚡ Quick win_\n\n\n🧩 Analysis chain
\n\n🏁 Script executed:\n\n```shell\n#!/bin/bash\nset -euo pipefail\n\nrg -n \"equivalenceKey|CodeAction\\.Create\\(\" src/ANcpLua.Analyzers.CodeFixes/CodeFixes/AL1001SimplifyNegatedPatternCodeFixProvider.cs || true\nsed -n '1,200p' src/ANcpLua.Analyzers.CodeFixes/CodeFixes/AL1001SimplifyNegatedPatternCodeFixProvider.cs | cat -n\nrg -n \"DiagnosticId\\s*=\\s*\\\"AL1001\\\"|FixableDiagnosticIds\" src/ANcpLua.Analyzers/Analyzers/AL1001DontRepeatNegatedPatternAnalyzer.cs || true\nsed -n '1,200p' src/ANcpLua.Analyzers/Analyzers/AL1001DontRepeatNegatedPatternAnalyzer.cs | cat -n\n```\n\nRepository: ANcpLua/ANcpLua.Analyzers\n\nLength of output: 5110\n\n---\n\n\n\n
\n\n**Fix FixAll grouping: use the AL1001 analyzer `DiagnosticId` as `equivalenceKey`**\n- `Al1001SimplifyNegatedPatternCodeFixProvider` uses `nameof(CodeFixResources.AL1001CodeFixTitle)` as the `CodeAction.Create(...)` `equivalenceKey`, which can break stable FixAll grouping; switch it to `Al1001DontRepeatNegatedPatternAnalyzer.DiagnosticId`.\n\n\nSuggested fix
\n\n```diff\n return CodeAction.Create(\n CodeFixResources.AL1001CodeFixTitle,\n _ => RemoveRepeatedNegatedPatterns(document, syntax, parent, root),\n- nameof(CodeFixResources.AL1001CodeFixTitle));\n+ Al1001DontRepeatNegatedPatternAnalyzer.DiagnosticId);\n```\n\n
\n\n\n\n\n📝 Committable suggestion
\n\n> ‼️ **IMPORTANT**\n> Carefully review the code before committing. Ensure that it accurately replaces the highlighted code, contains no missing lines, and has no issues with indentation. Thoroughly test & benchmark the code to ensure it meets the requirements.\n\n```suggestion\n return CodeAction.Create(\n CodeFixResources.AL1001CodeFixTitle,\n _ => RemoveRepeatedNegatedPatterns(document, syntax, parent, root),\n Al1001DontRepeatNegatedPatternAnalyzer.DiagnosticId);\n```\n\n
\n\n\n\n\n🤖 Prompt for AI Agents
\n\n```\nVerify each finding against current code. Fix only still-valid issues, skip the\nrest with a brief reason, keep changes minimal, and validate.\n\nIn\n`@src/ANcpLua.Analyzers.CodeFixes/CodeFixes/AL1001SimplifyNegatedPatternCodeFixProvider.cs`\naround lines 20 - 23, In AL1001SimplifyNegatedPatternCodeFixProvider, the\nCodeAction.Create call currently uses\nnameof(CodeFixResources.AL1001CodeFixTitle) as the equivalenceKey which prevents\nstable FixAll grouping; update that third argument to use\nAl1001DontRepeatNegatedPatternAnalyzer.DiagnosticId instead so all fixes for\nthis diagnostic share the same equivalence key (locate the\nCodeAction.Create(...) call that returns a CodeAction and the\nRemoveRepeatedNegatedPatterns(...) delegate and replace the equivalenceKey\naccordingly).\n```\n\n
\n\n\n\n","src/ANcpLua.Analyzers.CodeFixes/CodeFixes/AL1001SimplifyNegatedPatternCodeFixProvider.cs","_⚠️ Potential issue_ | _🟠 Major_ | _🏗️ Heavy lift_\n\n**Cross-repo publish order creates a config-gap window.**\n\nSection 8 identifies that `ANcpLua.NET.Sdk/src/Config/Analyzer.ANcpLua.Analyzers.editorconfig` packages AL0xxx severity blocks, and consumers inherit this transitively. If you publish `ANcpLua.Analyzers` 2.0.0 before the SDK updates and republishes its packaged editorconfig, consumers who restore will have a package whose severity config is broken—`dotnet_diagnostic.AL0025.severity` won't match `AL1700`, etc.\n\nAGENTS.md states \"publish lower layers (e.g., analyzers) before moving SDK truth upward,\" but here the SDK *packages* the analyzer editorconfig, creating a circular dependency. You must either:\n1. Publish analyzers 2.0.0, update SDK editorconfig + bump `ANcpLuaAnalyzersVersion` to 2.0.0, publish SDK, *then* announce consumers should update—accepting a brief window where early adopters get broken config.\n2. Coordinate a simultaneous SDK + analyzers release with lockstep version pins.\n\nPlan should explicitly state which path you're taking and warn consumers about the config-gap window if choosing option 1.\n\n\n\n\nBased on learnings: Cross-repo bootstrap: publish lower layers (e.g., analyzers) before moving SDK truth upward due to transitive analyzer injection.\n\n\n🤖 Prompt for AI Agents
\n\n```\nVerify each finding against current code. Fix only still-valid issues, skip the\nrest with a brief reason, keep changes minimal, and validate.\n\nIn `@eng/analyzer-renumber-plan.md` around lines 435 - 448, Update Section 8 of\neng/analyzer-renumber-plan.md to explicitly pick a release strategy and warn\nabout the config-gap: state whether you will (A) publish ANcpLua.Analyzers\n2.0.0, then update\nANcpLua.NET.Sdk/src/Config/Analyzer.ANcpLua.Analyzers.editorconfig and bump the\nANcpLuaAnalyzersVersion in the SDK and publish the SDK (and include an explicit\nwarning to consumers about the transient broken dotnet_diagnostic.AL####\nseverities), or (B) perform a coordinated simultaneous release of\nANcpLua.Analyzers and ANcpLua.NET.Sdk with lockstep pins to avoid the gap;\ninclude the chosen path, the exact steps (publish analyzers → update SDK\neditorconfig → bump ANcpLuaAnalyzersVersion → publish SDK, or coordinated\npublish), and a short consumer-facing warning text about the config-gap window\nif option A is chosen.\n```\n\n
\n\n\n\n","eng/analyzer-renumber-plan.md","_⚠️ Potential issue_ | _🟡 Minor_ | _⚡ Quick win_\n\n\n🧩 Analysis chain
\n\n🏁 Script executed:\n\n```shell\n#!/bin/bash\n# Description: Check if Semconv catalog files exist in the analyzer source tree.\n\n# Test: Look for any .cs files under the Semconv directory.\nfd -e cs . src/ANcpLua.Analyzers/Semconv/\n\n# If no output, the scoped block is dead and should be deleted.\n```\n\nRepository: ANcpLua/ANcpLua.Analyzers\n\nLength of output: 187\n\n---\n\n🏁 Script executed:\n\n```shell\n#!/bin/bash\nset -euo pipefail\n\n# Locate any Semconv-related directories anywhere in the repo (case-insensitive),\n# and list any .cs files beneath them.\necho \"=== Semconv directories (case-insensitive) ===\"\nfind . -type d -iname '*semconv*' -print | sed 's|^\\./||'\n\necho \"=== .cs files under Semconv directories ===\"\nfind . -type f -iname '*semconv*' -iname '*.cs' -print | sed 's|^\\./||'\n```\n\nRepository: ANcpLua/ANcpLua.Analyzers\n\nLength of output: 160\n\n---\n\n\n\n
\n\n**Editorconfig: scoped Semconv suppression likely dead code**\n\nThe `[src/ANcpLua.Analyzers/Semconv/*.cs]` block setting `dotnet_diagnostic.AL1210.severity = none` is currently inert because there are no `Semconv` directories or `.cs` files anywhere in `ANcpLua/ANcpLua.Analyzers`. Remove the scoped section/comment or update the glob to the real Semconv catalog location. [`.editorconfig` ~1736-1743]\n\n\n🤖 Prompt for AI Agents
\n\n```\nVerify each finding against current code. Fix only still-valid issues, skip the\nrest with a brief reason, keep changes minimal, and validate.\n\nIn @.editorconfig around lines 1736 - 1743, The editorconfig contains a dead\nscoped suppression block for Semconv files: locate the section with the header\n\"[src/ANcpLua.Analyzers/Semconv/*.cs]\" and the setting\n\"dotnet_diagnostic.AL1210.severity = none\" and either remove this entire scoped\nblock and its surrounding comment or change the glob to the actual Semconv\ncatalog path (or a broader pattern) so the AL1210 suppression applies where the\nSemconv .cs files actually live; ensure the change keeps the comment/context\nconsistent with other .editorconfig entries.\n```\n\n
\n\n\n\n",".editorconfig","\n\n\n> [!NOTE]\n> Currently processing new changes in this PR. This may take a few minutes, please wait...\n> \n> \n> ⚙️ Run configuration
\n> \n> **Configuration used**: Organization UI\n> \n> **Review profile**: ASSERTIVE\n> \n> **Plan**: Pro Plus\n> \n> **Run ID**: `566e0d30-7e0e-43e0-bbf5-cf47a439d4c1`\n> \n>
\n> \n> \n> 📥 Commits
\n> \n> Reviewing files that changed from the base of the PR and between 9d7393e3cd69b133dba378c21ab669d301ea2ecb and 2fe348d832d2933a6485914543622a272fc8621f.\n> \n>
\n> \n> \n> ⛔ Files ignored due to path filters (9)
\n> \n> * `.coderabbit.yaml` is excluded by none and included by none\n> * `.editorconfig` is excluded by none and included by none\n> * `AGENTS.md` is excluded by none and included by none\n> * `CHANGELOG.md` is excluded by none and included by none\n> * `README.md` is excluded by none and included by none\n> * `docs/Al1200UseIsEqualTo.md` is excluded by none and included by none\n> * `docs/analyzer-confidence-audit-2026-05-23.md` is excluded by none and included by none\n> * `src/ANcpLua.Analyzers.CodeFixes/CodeFixResources.Designer.cs` is excluded by `!**/*.Designer.cs` and included by `src/**`\n> * `src/ANcpLua.Analyzers/Resources.Designer.cs` is excluded by `!**/*.Designer.cs` and included by `src/**`\n> \n>
\n> \n> \n> 📒 Files selected for processing (207)
\n> \n> * `Directory.Build.props`\n> * `Directory.Packages.props`\n> * `Version.props`\n> * `eng/analyzer-renumber-plan.md`\n> * `src/ANcpLua.Analyzers.AnalyzerDocs/ANcpLua.Analyzers.AnalyzerDocs.csproj`\n> * `src/ANcpLua.Analyzers.AnalyzerDocs/Al1200UseIsEqualToDocs.cs`\n> * `src/ANcpLua.Analyzers.AnalyzerDocs/AlAnalyzerDocsGenerator.cs`\n> * `src/ANcpLua.Analyzers.AnalyzerDocs/AlAnalyzerDocsVerifier.cs`\n> * `src/ANcpLua.Analyzers.AnalyzerDocsGenerator/ANcpLua.Analyzers.AnalyzerDocsGenerator.csproj`\n> * `src/ANcpLua.Analyzers.CodeFixes/CLAUDE.md`\n> * `src/ANcpLua.Analyzers.CodeFixes/CodeFixResources.resx`\n> * `src/ANcpLua.Analyzers.CodeFixes/CodeFixes/AL1001SimplifyNegatedPatternCodeFixProvider.cs`\n> * `src/ANcpLua.Analyzers.CodeFixes/CodeFixes/AL1003SpanPatternMatchingCodeFixProvider.cs`\n> * `src/ANcpLua.Analyzers.CodeFixes/CodeFixes/AL1004UseSequenceEqualCodeFixProvider.cs`\n> * `src/ANcpLua.Analyzers.CodeFixes/CodeFixes/AL1007IXmlSerializableCodeFixProvider.cs`\n> * `src/ANcpLua.Analyzers.CodeFixes/CodeFixes/AL1009LockTypeCodeFixProvider.cs`\n> * `src/ANcpLua.Analyzers.CodeFixes/CodeFixes/AL1010PatternMatchingCodeFixProvider.cs`\n> * `src/ANcpLua.Analyzers.CodeFixes/CodeFixes/AL1011NormalizeNullGuardStyleCodeFixProvider.cs`\n> * `src/ANcpLua.Analyzers.CodeFixes/CodeFixes/AL1012CombineDeclarationWithNullCheckCodeFixProvider.cs`\n> * `src/ANcpLua.Analyzers.CodeFixes/CodeFixes/AL1200UseIsEqualToCodeFixProvider.cs`\n> * `src/ANcpLua.Analyzers.CodeFixes/CodeFixes/AL1201UseHasAttributeCodeFixProvider.cs`\n> * `src/ANcpLua.Analyzers.CodeFixes/CodeFixes/AL1202UseTypeHierarchyCodeFixProvider.cs`\n> * `src/ANcpLua.Analyzers.CodeFixes/CodeFixes/AL1203UseOperationExtensionsCodeFixProvider.cs`\n> * `src/ANcpLua.Analyzers.CodeFixes/CodeFixes/AL1204UseOrEmptyCodeFixProvider.cs`\n> * `src/ANcpLua.Analyzers.CodeFixes/CodeFixes/AL1205UseToImmutableArrayOrEmptyCodeFixProvider.cs`\n> * `src/ANcpLua.Analyzers.CodeFixes/CodeFixes/AL1206UseWhereNotNullCodeFixProvider.cs`\n> * `src/ANcpLua.Analyzers.CodeFixes/CodeFixes/AL1207UseToDisplayStringExtensionsCodeFixProvider.cs`\n> * `src/ANcpLua.Analyzers.CodeFixes/CodeFixes/AL1208UseGuardNotNullCodeFixProvider.cs`\n> * `src/ANcpLua.Analyzers.CodeFixes/CodeFixes/AL1209UseTryParseExtensionsCodeFixProvider.cs`\n> * `src/ANcpLua.Analyzers.CodeFixes/CodeFixes/AL1210UseStringComparisonExtensionsCodeFixProvider.cs`\n> * `src/ANcpLua.Analyzers.CodeFixes/CodeFixes/AL1211UseAttributeExtensionsCodeFixProvider.cs`\n> * `src/ANcpLua.Analyzers.CodeFixes/CodeFixes/AL1212UseGuardNotNullOrEmptyCodeFixProvider.cs`\n> * `src/ANcpLua.Analyzers.CodeFixes/CodeFixes/AL1213UseGuardNotNullOrWhiteSpaceCodeFixProvider.cs`\n> * `src/ANcpLua.Analyzers.CodeFixes/CodeFixes/AL1214UseGuardNotZeroCodeFixProvider.cs`\n> * `src/ANcpLua.Analyzers.CodeFixes/CodeFixes/AL1215UseGuardNotNegativeCodeFixProvider.cs`\n> * `src/ANcpLua.Analyzers.CodeFixes/CodeFixes/AL1216UseGuardPositiveCodeFixProvider.cs`\n> * `src/ANcpLua.Analyzers.CodeFixes/CodeFixes/AL1217UseGuardNotEmptyGuidCodeFixProvider.cs`\n> * `src/ANcpLua.Analyzers.CodeFixes/CodeFixes/AL1218UseGuardDefinedEnumCodeFixProvider.cs`\n> * `src/ANcpLua.Analyzers.CodeFixes/CodeFixes/AL1220UseGuardForThrowIfCodeFixProvider.cs`\n> * `src/ANcpLua.Analyzers.CodeFixes/CodeFixes/AL1313CancellationTokenPropagationCodeFixProvider.cs`\n> * `src/ANcpLua.Analyzers.CodeFixes/CodeFixes/AL1314UseExplicitMidpointRoundingCodeFixProvider.cs`\n> * `src/ANcpLua.Analyzers.CodeFixes/CodeFixes/AL1500AddMissingCasesCodeFixProvider.cs`\n> * `src/ANcpLua.Analyzers.CodeFixes/CodeFixes/AL1503NormalizeWhitespaceCodeFixProvider.cs`\n> * `src/ANcpLua.Analyzers.CodeFixes/CodeFixes/AL1504DuckDbTableCodeFixProvider.cs`\n> * `src/ANcpLua.Analyzers.CodeFixes/CodeFixes/AL1700StaticLambdaCodeFixProvider.cs`\n> * `src/ANcpLua.Analyzers.CodeFixes/CodeFixes/AL1701DateTimeNowCodeFixProvider.cs`\n> * `src/ANcpLua.Analyzers.CodeFixes/CodeFixes/AL1702UseSystemTextJsonCodeFixProvider.cs`\n> * `src/ANcpLua.Analyzers.CodeFixes/CodeFixes/AL1703UseImplicitTypeWhenApparentCodeFixProvider.cs`\n> * `src/ANcpLua.Analyzers.CodeFixes/Refactorings/AR0001SnakeCaseToPascalCaseRefactoring.cs`\n> * `src/ANcpLua.Analyzers.CodeFixes/Refactorings/AR0002MakeStaticLambdaRefactoring.cs`\n> * `src/ANcpLua.Analyzers/ANcpLua.Analyzers.csproj`\n> * `src/ANcpLua.Analyzers/AlAnalyzer.cs`\n> * `src/ANcpLua.Analyzers/AnalyzerReleases.Unshipped.md`\n> * `src/ANcpLua.Analyzers/Analyzers/AL1000ProhibitPrimaryConstructorParameterReassignmentAnalyzer.cs`\n> * `src/ANcpLua.Analyzers/Analyzers/AL1001DontRepeatNegatedPatternAnalyzer.cs`\n> * `src/ANcpLua.Analyzers/Analyzers/AL1002DontDivideByConstantZeroAnalyzer.cs`\n> * `src/ANcpLua.Analyzers/Analyzers/AL1003ToAL1004SpanComparisonAnalyzer.cs`\n> * `src/ANcpLua.Analyzers/Analyzers/AL1005FieldNameConflictWithPrimaryConstructorAnalyzer.cs`\n> * `src/ANcpLua.Analyzers/Analyzers/AL1006ToAL1008IXmlSerializableAnalyzer.cs`\n> * `src/ANcpLua.Analyzers/Analyzers/AL1009LockKeywordAnalyzer.cs`\n> * `src/ANcpLua.Analyzers/Analyzers/AL1010PreferPatternMatchingAnalyzer.cs`\n> * `src/ANcpLua.Analyzers/Analyzers/AL1011NormalizeNullGuardStyleAnalyzer.cs`\n> * `src/ANcpLua.Analyzers/Analyzers/AL1012CombineDeclarationWithNullCheckAnalyzer.cs`\n> * `src/ANcpLua.Analyzers/Analyzers/AL1100ToAL1104FormBindingAnalyzer.cs`\n> * `src/ANcpLua.Analyzers/Analyzers/AL1105MissingResilienceConfigurationAnalyzer.cs`\n> * `src/ANcpLua.Analyzers/Analyzers/AL1106MissingHealthChecksAnalyzer.cs`\n> * `src/ANcpLua.Analyzers/Analyzers/AL1107ConsiderConnectionStringAnalyzer.cs`\n> * `src/ANcpLua.Analyzers/Analyzers/AL1108MissingServiceDiscoveryAnalyzer.cs`\n> * `src/ANcpLua.Analyzers/Analyzers/AL1109AvoidTaskRunInAspNetCoreAnalyzer.cs`\n> * `src/ANcpLua.Analyzers/Analyzers/AL1200UseIsEqualToAnalyzer.cs`\n> * `src/ANcpLua.Analyzers/Analyzers/AL1201UseHasAttributeAnalyzer.cs`\n> * `src/ANcpLua.Analyzers/Analyzers/AL1202UseTypeHierarchyAnalyzer.cs`\n> * `src/ANcpLua.Analyzers/Analyzers/AL1203UseOperationExtensionsAnalyzer.cs`\n> * `src/ANcpLua.Analyzers/Analyzers/AL1204UseOrEmptyAnalyzer.cs`\n> * `src/ANcpLua.Analyzers/Analyzers/AL1205UseToImmutableArrayOrEmptyAnalyzer.cs`\n> * `src/ANcpLua.Analyzers/Analyzers/AL1206UseWhereNotNullAnalyzer.cs`\n> * `src/ANcpLua.Analyzers/Analyzers/AL1207UseToDisplayStringExtensionsAnalyzer.cs`\n> * `src/ANcpLua.Analyzers/Analyzers/AL1208UseGuardNotNullAnalyzer.cs`\n> * `src/ANcpLua.Analyzers/Analyzers/AL1209UseTryParseExtensionsAnalyzer.cs`\n> * `src/ANcpLua.Analyzers/Analyzers/AL1210UseStringComparisonExtensionsAnalyzer.cs`\n> * `src/ANcpLua.Analyzers/Analyzers/AL1211UseAttributeExtensionsAnalyzer.cs`\n> * `src/ANcpLua.Analyzers/Analyzers/AL1212UseGuardNotNullOrEmptyAnalyzer.cs`\n> * `src/ANcpLua.Analyzers/Analyzers/AL1213UseGuardNotNullOrWhiteSpaceAnalyzer.cs`\n> * `src/ANcpLua.Analyzers/Analyzers/AL1214UseGuardNotZeroAnalyzer.cs`\n> * `src/ANcpLua.Analyzers/Analyzers/AL1215UseGuardNotNegativeAnalyzer.cs`\n> * `src/ANcpLua.Analyzers/Analyzers/AL1216UseGuardPositiveAnalyzer.cs`\n> * `src/ANcpLua.Analyzers/Analyzers/AL1217UseGuardNotEmptyGuidAnalyzer.cs`\n> * `src/ANcpLua.Analyzers/Analyzers/AL1218UseGuardDefinedEnumAnalyzer.cs`\n> * `src/ANcpLua.Analyzers/Analyzers/AL1219UseStringComparisonAnyExtensionsAnalyzer.cs`\n> * `src/ANcpLua.Analyzers/Analyzers/AL1220UseGuardForThrowIfAnalyzer.cs`\n> * `src/ANcpLua.Analyzers/Analyzers/AL1300ToAL1303ThreadingAnalyzer.cs`\n> * `src/ANcpLua.Analyzers/Analyzers/AL1304PreferAwaitUsingAnalyzer.cs`\n> * `src/ANcpLua.Analyzers/Analyzers/AL1305AvoidBlockingCallsInAsyncAnalyzer.cs`\n> * `src/ANcpLua.Analyzers/Analyzers/AL1306SqlInterpolationInCommandTextAnalyzer.cs`\n> * `src/ANcpLua.Analyzers/Analyzers/AL1307FireAndForgetTaskAnalyzer.cs`\n> * `src/ANcpLua.Analyzers/Analyzers/AL1308PreferTryParseAnalyzer.cs`\n> * `src/ANcpLua.Analyzers/Analyzers/AL1309EmptyCatchBlockAnalyzer.cs`\n> * `src/ANcpLua.Analyzers/Analyzers/AL1310ExceptionLeakedInResponseAnalyzer.cs`\n> * `src/ANcpLua.Analyzers/Analyzers/AL1311UnnecessaryLinqMaterializationAnalyzer.cs`\n> * `src/ANcpLua.Analyzers/Analyzers/AL1312ReadModifyWriteWithoutTransactionAnalyzer.cs`\n> * `src/ANcpLua.Analyzers/Analyzers/AL1313CancellationTokenPropagationAnalysis.cs`\n> * `src/ANcpLua.Analyzers/Analyzers/AL1313CancellationTokenPropagationAnalyzer.cs`\n> * `src/ANcpLua.Analyzers/Analyzers/AL1314UseExplicitMidpointRoundingAnalyzer.cs`\n> * `src/ANcpLua.Analyzers/Analyzers/AL1400AotTestMustReturnIntAnalyzer.cs`\n> * `src/ANcpLua.Analyzers/Analyzers/AL1401AotTestExitCode100Analyzer.cs`\n> * `src/ANcpLua.Analyzers/Analyzers/AL1402TrimSafeViolationAnalyzer.cs`\n> * `src/ANcpLua.Analyzers/Analyzers/AL1403AotSafeViolationAnalyzer.cs`\n> * `src/ANcpLua.Analyzers/Analyzers/AL1404AotSafeCallsAotUnsafeAnalyzer.cs`\n> * `src/ANcpLua.Analyzers/Analyzers/AL1405UnnecessaryAotUnsafeAnalyzer.cs`\n> * `src/ANcpLua.Analyzers/Analyzers/AL1406AvoidDynamicKeywordAnalyzer.cs`\n> * `src/ANcpLua.Analyzers/Analyzers/AL1407AvoidExpressionCompileAnalyzer.cs`\n> * `src/ANcpLua.Analyzers/Analyzers/AL1408AvoidActivatorCreateInstanceAnalyzer.cs`\n> * `src/ANcpLua.Analyzers/Analyzers/AL1409AvoidTypeGetTypeAnalyzer.cs`\n> * `src/ANcpLua.Analyzers/Analyzers/AL1500ClosedTypeHierarchySwitchAnalyzer.cs`\n> * `src/ANcpLua.Analyzers/Analyzers/AL1501SymbolStoredInModelAnalyzer.cs`\n> * `src/ANcpLua.Analyzers/Analyzers/AL1502UseIncrementalGeneratorAnalyzer.cs`\n> * `src/ANcpLua.Analyzers/Analyzers/AL1503NormalizeWhitespaceAnalyzer.cs`\n> * `src/ANcpLua.Analyzers/Analyzers/AL1504DuckDbTableMustBePartialAnalyzer.cs`\n> * `src/ANcpLua.Analyzers/Analyzers/AL1505DuckDbColumnConflictingOrdinalAnalyzer.cs`\n> * `src/ANcpLua.Analyzers/Analyzers/AL1600HardcodedPackageVersionAnalyzer.cs`\n> * `src/ANcpLua.Analyzers/Analyzers/AL1601VersionPropsNotImportedAnalyzer.cs`\n> * `src/ANcpLua.Analyzers/Analyzers/AL1602UndefinedVersionVariableAnalyzer.cs`\n> * `src/ANcpLua.Analyzers/Analyzers/AL1603ToAL1605DiagnosticsAlignmentAnalyzer.cs`\n> * `src/ANcpLua.Analyzers/Analyzers/AL1606OutdatedMafPackageVersionAnalyzer.cs`\n> * `src/ANcpLua.Analyzers/Analyzers/AL1700PreferStaticLambdaAnalyzer.cs`\n> * `src/ANcpLua.Analyzers/Analyzers/AL1701AvoidDateTimeNowAnalyzer.cs`\n> * `src/ANcpLua.Analyzers/Analyzers/AL1702AvoidNewtonsoftJsonAnalyzer.cs`\n> * `src/ANcpLua.Analyzers/Analyzers/AL1703UseImplicitTypeWhenApparentAnalyzer.cs`\n> * `src/ANcpLua.Analyzers/Analyzers/AL1800DestructiveToolMustRequireApprovalAnalyzer.cs`\n> * `src/ANcpLua.Analyzers/Analyzers/AL1801ToolMustDeclareSideEffectAnalyzer.cs`\n> * `src/ANcpLua.Analyzers/Analyzers/AL1802ToolMustDeclareCapabilityAnalyzer.cs`\n> * `src/ANcpLua.Analyzers/AotContext.cs`\n> * `src/ANcpLua.Analyzers/CLAUDE.md`\n> * `src/ANcpLua.Analyzers/CLAUDE.md`\n> * `src/ANcpLua.Analyzers/Resources.resx`\n> * `src/ANcpLua.Analyzers/build/ANcpLua.Analyzers.props`\n> * `tests/ANcpLua.Analyzers.Tests/AL1000PrimaryConstructorReassignmentTests.cs`\n> * `tests/ANcpLua.Analyzers.Tests/AL1001NegatedPatternTests.cs`\n> * `tests/ANcpLua.Analyzers.Tests/AL1002DivisionByZeroTests.cs`\n> * `tests/ANcpLua.Analyzers.Tests/AL1003ToAL1004SpanComparisonTests.cs`\n> * `tests/ANcpLua.Analyzers.Tests/AL1005FieldNameConflictWithPrimaryConstructorTests.cs`\n> * `tests/ANcpLua.Analyzers.Tests/AL1006ToAL1008IXmlSerializableTests.cs`\n> * `tests/ANcpLua.Analyzers.Tests/AL1009LockKeywordTests.cs`\n> * `tests/ANcpLua.Analyzers.Tests/AL1010PatternMatchingTests.cs`\n> * `tests/ANcpLua.Analyzers.Tests/AL1011NormalizeNullGuardStyleTests.cs`\n> * `tests/ANcpLua.Analyzers.Tests/AL1012CombineDeclarationWithNullCheckTests.cs`\n> * `tests/ANcpLua.Analyzers.Tests/AL1100ToAL1104FormBindingTests.cs`\n> * `tests/ANcpLua.Analyzers.Tests/AL1105MissingResilienceConfigurationTests.cs`\n> * `tests/ANcpLua.Analyzers.Tests/AL1106MissingHealthChecksTests.cs`\n> * `tests/ANcpLua.Analyzers.Tests/AL1107ConsiderConnectionStringTests.cs`\n> * `tests/ANcpLua.Analyzers.Tests/AL1108MissingServiceDiscoveryTests.cs`\n> * `tests/ANcpLua.Analyzers.Tests/AL1109AvoidTaskRunInAspNetCoreTests.cs`\n> * `tests/ANcpLua.Analyzers.Tests/AL1200UseIsEqualToTests.cs`\n> * `tests/ANcpLua.Analyzers.Tests/AL1201UseHasAttributeCodeFixTests.cs`\n> * `tests/ANcpLua.Analyzers.Tests/AL1201UseHasAttributeTests.cs`\n> * `tests/ANcpLua.Analyzers.Tests/AL1202UseTypeHierarchyTests.cs`\n> * `tests/ANcpLua.Analyzers.Tests/AL1203UseOperationExtensionsTests.cs`\n> * `tests/ANcpLua.Analyzers.Tests/AL1204UseOrEmptyTests.cs`\n> * `tests/ANcpLua.Analyzers.Tests/AL1206UseWhereNotNullCodeFixTests.cs`\n> * `tests/ANcpLua.Analyzers.Tests/AL1206UseWhereNotNullTests.cs`\n> * `tests/ANcpLua.Analyzers.Tests/AL1208UseGuardNotNullTests.cs`\n> * `tests/ANcpLua.Analyzers.Tests/AL1209UseTryParseExtensionsTests.cs`\n> * `tests/ANcpLua.Analyzers.Tests/AL1210UseStringComparisonExtensionsTests.cs`\n> * `tests/ANcpLua.Analyzers.Tests/AL1211UseAttributeExtensionsTests.cs`\n> * `tests/ANcpLua.Analyzers.Tests/AL1212UseGuardNotNullOrEmptyTests.cs`\n> * `tests/ANcpLua.Analyzers.Tests/AL1213UseGuardNotNullOrWhiteSpaceTests.cs`\n> * `tests/ANcpLua.Analyzers.Tests/AL1214UseGuardNotZeroTests.cs`\n> * `tests/ANcpLua.Analyzers.Tests/AL1215UseGuardNotNegativeTests.cs`\n> * `tests/ANcpLua.Analyzers.Tests/AL1216UseGuardPositiveTests.cs`\n> * `tests/ANcpLua.Analyzers.Tests/AL1217UseGuardNotEmptyGuidTests.cs`\n> * `tests/ANcpLua.Analyzers.Tests/AL1218UseGuardDefinedEnumTests.cs`\n> * `tests/ANcpLua.Analyzers.Tests/AL1219UseStringComparisonAnyExtensionsTests.cs`\n> * `tests/ANcpLua.Analyzers.Tests/AL1220UseGuardForThrowIfCodeFixTests.cs`\n> * `tests/ANcpLua.Analyzers.Tests/AL1220UseGuardForThrowIfTests.cs`\n> * `tests/ANcpLua.Analyzers.Tests/AL1300ToAL1303ThreadingTests.cs`\n> * `tests/ANcpLua.Analyzers.Tests/AL1304PreferAwaitUsingTests.cs`\n> * `tests/ANcpLua.Analyzers.Tests/AL1305AvoidBlockingCallsInAsyncTests.cs`\n> * `tests/ANcpLua.Analyzers.Tests/AL1306SqlInterpolationInCommandTextTests.cs`\n> * `tests/ANcpLua.Analyzers.Tests/AL1307FireAndForgetTaskTests.cs`\n> * `tests/ANcpLua.Analyzers.Tests/AL1308PreferTryParseTests.cs`\n> * `tests/ANcpLua.Analyzers.Tests/AL1309EmptyCatchBlockTests.cs`\n> * `tests/ANcpLua.Analyzers.Tests/AL1310ExceptionLeakedInResponseTests.cs`\n> * `tests/ANcpLua.Analyzers.Tests/AL1311UnnecessaryLinqMaterializationTests.cs`\n> * `tests/ANcpLua.Analyzers.Tests/AL1312ReadModifyWriteWithoutTransactionTests.cs`\n> * `tests/ANcpLua.Analyzers.Tests/AL1313CancellationTokenPropagationTests.cs`\n> * `tests/ANcpLua.Analyzers.Tests/AL1314UseExplicitMidpointRoundingCodeFixTests.cs`\n> * `tests/ANcpLua.Analyzers.Tests/AL1314UseExplicitMidpointRoundingTests.cs`\n> * `tests/ANcpLua.Analyzers.Tests/AL1401AotTestExitCode100Tests.cs`\n> * `tests/ANcpLua.Analyzers.Tests/AL1403AotSafeViolationTests.cs`\n> * `tests/ANcpLua.Analyzers.Tests/AL1404AotSafeCallsAotUnsafeTests.cs`\n> * `tests/ANcpLua.Analyzers.Tests/AL1405UnnecessaryAotUnsafeTests.cs`\n> * `tests/ANcpLua.Analyzers.Tests/AL1406AvoidDynamicKeywordTests.cs`\n> * `tests/ANcpLua.Analyzers.Tests/AL1407AvoidExpressionCompileTests.cs`\n> * `tests/ANcpLua.Analyzers.Tests/AL1501SymbolStoredInModelTests.cs`\n> * `tests/ANcpLua.Analyzers.Tests/AL1502UseIncrementalGeneratorTests.cs`\n> * `tests/ANcpLua.Analyzers.Tests/AL1503NormalizeWhitespaceTests.cs`\n> * `tests/ANcpLua.Analyzers.Tests/AL1504DuckDbTableMustBePartialTests.cs`\n> * `tests/ANcpLua.Analyzers.Tests/AL1505DuckDbColumnConflictingOrdinalTests.cs`\n> * `tests/ANcpLua.Analyzers.Tests/AL1601VersionPropsNotImportedTests.cs`\n> * `tests/ANcpLua.Analyzers.Tests/AL1602UndefinedVersionVariableTests.cs`\n> * `tests/ANcpLua.Analyzers.Tests/AL1606OutdatedMafPackageVersionTests.cs`\n> * `tests/ANcpLua.Analyzers.Tests/AL1700PreferStaticLambdaTests.cs`\n> * `tests/ANcpLua.Analyzers.Tests/AL1701AvoidDateTimeNowTests.cs`\n> * `tests/ANcpLua.Analyzers.Tests/AL1702AvoidNewtonsoftJsonTests.cs`\n> * `tests/ANcpLua.Analyzers.Tests/AL1703UseImplicitTypeWhenApparentTests.cs`\n> * `tests/ANcpLua.Analyzers.Tests/ANcpLua.Analyzers.Tests.csproj`\n> * `tests/ANcpLua.Analyzers.Tests/CLAUDE.md`\n> \n>
\n> \n> \n\n\n\n\n\n## Walkthrough\nGlobal renumbering: analyzers, code fixes, resources, docs, tests, and configs migrated from AL0xxx to AL1xxx bands; README/AGENTS rewritten; .editorconfig severities updated; docs generator retargeted; package/version props bumped; changelog added for 2.0.0.\n\n## Changes\n\n**Analyzer ID Renumbering to AL1xxx bands**\n\n|Layer / File(s)|Summary|\n|---|---|\n|Core renumbering and resources/docs|Renames diagnostic IDs/classes/resources across analyzers and code fixes; updates CodeFixResources(.resx/.Designer), README, AGENTS, and CHANGELOG for 2.0.0.|\n|Wiring and tooling|Adjusts .editorconfig severities, docs generator/verifier to AL1200, and project NoWarn mappings; AOT/context comments aligned.|\n|Tests migration|Renames all test classes/diagnostic markers to AL1xxx and rewires code-fix tests to new providers.|\n|Build/version|Bumps VersionPrefix to 2.0.0 and ANcpLuaAnalyzersVersion to 1.29.4; minor props/comments updates.|\n\n## Estimated code review effort\n🎯 5 (Critical) | ⏱️ ~120 minutes\n\n## Possibly related PRs\n- [ANcpLua/ANcpLua.Analyzers#27](https://github.com/ANcpLua/ANcpLua.Analyzers/pull/27) — Earlier work on Version.props import analyzer that this PR renumbers to AL1601.\n- [ANcpLua/ANcpLua.Analyzers#25](https://github.com/ANcpLua/ANcpLua.Analyzers/pull/25) — Original DateTime.Now analyzer later renumbered here to AL1701.\n- [ANcpLua/ANcpLua.Analyzers#148](https://github.com/ANcpLua/ANcpLua.Analyzers/pull/148) — Touches rules whose IDs are renumbered in this PR, affecting the same analyzers/code fixes.\n\n\n\n\n\n\n\n\n","## Triage Bot report\n\n- thread Oc6EeXH6 from @codacy-production: **needs-human** — no rule matched — keeping blocked for manual review\n > _{:white_circle: LOW RISK} Nitpick: Since the `AlAnalyzer.HelpLink` for\n- thread Oc6EeXH- from @codacy-production: **needs-human** — no rule matched — keeping blocked for manual review\n > _{:white_circle: LOW RISK} Nitpick: Update diagnostic ID wildcards, cla\n\n_Threads marked `needs-human` stay unresolved and block auto-merge._","### Pull Request Overview\n\nThis PR successfully implements a major renumbering of 89 diagnostic rules into domain-specific bands, aligning with the 2.0.0 version bump. Codacy analysis indicates the project remains up to standards, although the massive refactor has introduced significant structural duplication (187 clones) due to the repetitive nature of analyzer registration boilerplate.\n\nWhile the primary renumbering logic is sound, there are high-risk areas concerning several complex files that currently lack test coverage, specifically regarding CancellationToken propagation, type hierarchy analysis, and LINQ materialization. Additionally, the breaking nature of this change necessitates a proactive documentation redirect strategy and an update to AI agent instruction sets to prevent the continued generation of legacy AL0xxx IDs.\n\n#### About this PR\n- This release will invalidate every existing .editorconfig suppression and #pragma warning in dependent codebases. Ensure this impact is prominently highlighted in the release notes beyond the current CHANGELOG entry.\n- Help-link URLs provided by AlAnalyzer.HelpLink will result in 404 errors until the documentation site is redeployed with the new rule paths. A redirect strategy for the old AL0xxx paths is highly recommended to maintain developer experience.\n\n\n1 comment outside of the diff
\n\n\nsrc/ANcpLua.Analyzers/AlAnalyzer.cs
\n\n> _{`line 19` :white_circle: LOW RISK}\n> Suggestion: This refactor has significantly increased structural duplication (187 clones). The logic for `RegisterActions` and `SupportedDiagnostics` follows a rigid template now repeated across 89 classes. Consider evolving the `AlAnalyzer` base class to support a more declarative API to reduce maintenance overhead. Suggested prompt for your IDE agent: 'Identify the structural patterns used in RegisterActions and SupportedDiagnostics across the Analyzers folder. Refactor the AlAnalyzer base class to provide a declarative API that automates these registrations based on internal constants.'\n\n
\n\n
\n\n#### Test suggestions\n- [x] Verify renumbered Diagnostic IDs (AL1000-AL1899) satisfy the convention test regex '^AL\\d{4}$'\n- [x] Renamed analyzer class names satisfy the naming convention test '^Al\\d{4}.*Analyzer$'\n- [x] Orphaned OTel resource keys are removed from both Resources.resx and CodeFixResources.resx\n- [x] Resource-based localization interpolation in AlAnalyzer.CreateRule works with the new 4-digit numeric IDs\n- [x] Static documentation (README) full rule catalog table matches the renumbered IDs and categories\n- [ ] Unit test coverage for complex logic in AL1313CancellationTokenPropagationAnalysis.cs\n- [ ] Unit test coverage for complex logic in AL1202UseTypeHierarchyAnalyzer.cs\n- [ ] Unit test coverage for complex logic in AL1311UnnecessaryLinqMaterializationAnalyzer.cs\n- [ ] Unit test coverage for complex logic in AL1500ClosedTypeHierarchySwitchAnalyzer.cs\n- [ ] Unit test coverage for complex logic in AL1606OutdatedMafPackageVersionAnalyzer.cs\n- [ ] Unit test coverage for complex logic in AL1100ToAL1104FormBindingAnalyzer.cs\n\n\nPrompt proposal for missing tests
\n\n```\nConsider implementing these tests if applicable:\n1. Unit test coverage for complex logic in AL1313CancellationTokenPropagationAnalysis.cs\n2. Unit test coverage for complex logic in AL1202UseTypeHierarchyAnalyzer.cs\n3. Unit test coverage for complex logic in AL1311UnnecessaryLinqMaterializationAnalyzer.cs\n4. Unit test coverage for complex logic in AL1500ClosedTypeHierarchySwitchAnalyzer.cs\n5. Unit test coverage for complex logic in AL1606OutdatedMafPackageVersionAnalyzer.cs\n6. Unit test coverage for complex logic in AL1100ToAL1104FormBindingAnalyzer.cs\n```\n\n
\n\n\n\n##\n\n_{`TIP` Improve review quality by [adding custom instructions](https://docs.codacy.com/codacy-ai/codacy-ai/#custom-instructions)}\n_{`TIP` How was this review? [Give us feedback](https://tally.so/r/jaBlA1?org=ANcpLua&repo=ANcpLua.Analyzers&pr=174)}\n","_{:white_circle: LOW RISK}\n\nNitpick: Update diagnostic ID wildcards, class names, and template strings in `AGENTS.md` (e.g., lines 38, 55, 78, 124, 245) from the legacy `AL0XXX`/`Al00xx` range to the new `AL1XXX`/`Al1XXX` range to ensure consistency with the 2.0.0 renumbering scheme.\n","AGENTS.md",134,"_{:white_circle: LOW RISK}\n\nNitpick: Since the `AlAnalyzer.HelpLink` format is changing, ensure that the documentation site implements a redirect strategy from the old `AL0xxx` URL paths to the new `AL1xxx` domain band paths. This will prevent 404 errors for users with legacy `.editorconfig` settings.\n","CHANGELOG.md",11,"## Up to standards :white_check_mark:\n🟢 Issues 0 issues
\n\n>
\n>\n> \n>\n> **Results:**\n> `0` new issues\n>\n>\n>\n> [View in Codacy](https://app.codacy.com/gh/ANcpLua/ANcpLua.Analyzers/pull-requests/174/issues)\n>
\n
\n\n🟢 Metrics 2083 complexity · 187 duplication
\n\n>
\n>\n> | Metric | Results |\n> | ------------- | ------------- |\n> | Complexity | **2083** | \n > | Duplication | **187** |\n>\n>\n> [View in Codacy](https://app.codacy.com/gh/ANcpLua/ANcpLua.Analyzers/pull-requests/174/files)\n>
\n
\n\n\n\n##\n> **AI Reviewer:** first review requested successfully. _AI can make mistakes. Always validate suggestions._\n>\n> [][Link]\n>\n> [Link]: https://app.codacy.com/gh/ANcpLua/ANcpLua.Analyzers/pull-requests/174/review?utm_source=github.com&utm_medium=unifiedPullRequestSummary&utm_campaign=unifiedPullRequestSummary 'Trigger a new review'\n\n\n\n_{`TIP` This summary will be updated as you push new changes.}\n\n","\n> [!NOTE]\n> Autofix is a beta feature. Expect some limitations and changes as we gather feedback and continue to improve it.\n\nAutofix skipped. No unresolved CodeRabbit review comments with fix instructions found.\n","**Claude encountered an error after 2s** —— [View job](https://github.com/ANcpLua/ANcpLua.Analyzers/actions/runs/26387851682)\n\n---\nI'll analyze this and get back to you.","base_ref","head_ref","pull_request_info",[879],{"_651":57,"_57":763},[909,910,911],[],{"_295":908},{"_292":907},{"_292":906},{"_257":258,"_255":259,"_260":261,"_262":70,"_263":18,"_264":18,"_265":266,"_267":268,"_269":226,"_270":138,"_271":272,"_273":274,"_275":-5,"_276":-5,"_277":278,"_279":-5,"_280":-5,"_281":905,"_283":284,"_285":286,"_287":288,"_289":290},{"_53":134,"_172":173,"_174":175,"_176":18,"_177":178,"_179":180,"_181":-5,"_182":-5,"_183":-5,"_184":18,"_185":895,"_187":178,"_188":178,"_189":896,"_191":897,"_193":898,"_195":-5,"_196":197,"_198":199,"_200":201,"_202":899,"_66":-5,"_130":204,"_205":900,"_207":901,"_209":210,"_211":18,"_212":18},{"_144":888},[889,890,891],{"_156":894,"_158":169},{"_156":893,"_158":165},{"_156":892,"_158":159},{"_160":161,"_104":162,"_130":163},{"_160":166,"_104":162,"_130":167},{"_160":170,"_104":162,"_130":171},[216],{"_216":902},{},{},[],{"_215":70},{"_213":70,"_214":178},{"_53":218,"_160":175,"_219":903,"_221":222,"_223":224,"_225":226,"_227":228,"_229":230,"_231":232,"_233":75,"_234":18,"_235":-5,"_236":-5,"_237":-5,"_238":-5,"_239":-5,"_240":-5,"_241":242,"_243":-5,"_66":904},{"_249":250,"_53":251,"_104":252,"_253":254,"_255":256},{"_245":18,"_246":18,"_247":18,"_248":18},{"_249":250,"_160":250,"_291":-5,"_253":254,"_53":251},[],[],[],{"_104":642,"_108":105,"_343":928},{"_104":327,"_328":915},{"_104":300,"_301":302,"_303":304,"_305":912,"_307":-5},{"_104":305,"_308":309,"_139":140,"_310":-5,"_280":311,"_312":913,"_314":315,"_316":317,"_318":319,"_320":321,"_322":914},{"_324":325,"_326":-5},[],[916,917,918],{"_333":595,"_335":-5,"_336":925,"_338":-5},{"_333":460,"_335":-5,"_336":922,"_338":-5},{"_333":334,"_335":-5,"_336":919,"_338":-5},[920],{"_340":319,"_341":342,"_343":921,"_345":70},[346,347,348,349,350,351,352,353,178,354,355,356,356,357,358,359,360,361,362,361,363,361,364,365,366,361,367,368,369,370,371,372,373,374,375,366,361,367,376,369,377,375,378,379,375,366,361,380,369,381,375,366,361,382,369,383,375,366,361,384,369,385,375,366,361,386,369,387,375,366,361,388,369,389,375,366,361,390,369,391,375,366,361,392,369,393,375,366,178,394,369,395,375,366,178,396,369,397,375,366,178,398,369,399,375,366,178,400,369,401,375,366,178,402,369,403,375,366,178,404,369,405,375,366,178,406,369,407,375,366,178,408,369,409,375,366,178,410,369,411,375,366,178,412,369,413,375,366,178,414,369,415,375,366,178,416,369,417,375,366,178,418,369,419,375,366,178,420,369,421,375,366,178,422,369,423,375,366,178,424,369,425,375,366,178,426,369,427,375,366,178,428,369,429,375,366,178,430,369,431,375,366,178,432,369,433,375,366,178,434,369,435,375,366,178,436,369,437,375,366,178,438,369,439,375,366,178,440,369,441,375,366,178,442,369,443,375,366,178,444,369,445,375,366,178,446,369,447,375,366,178,448,369,449,375,366,178,450,369,451,375,366,178,452,369,453,375,366,178,454,369,455,375,366,178,456,369,457,375,366,458,459],[923],{"_340":319,"_341":463,"_343":924,"_345":70},[465,466,467,468,469,470,471,472,473,474,475,476,477,478,479,480,481,482,483,484,485,475,486,487,483,484,488,475,476,489,490,478,491,492,493,482,483,484,494,475,476,489,478,495,483,484,496,497,498,499,500,501,502,503,504,502,505,506,507,508,502,509,510,507,508,502,511,512,513,514,515,513,516,517,513,518,519,513,520,521,513,522,523,513,524,525,513,526,527,513,528,529,513,530,515,513,531,532,513,533,534,513,535,536,513,537,538,513,539,540,513,541,542,513,543,544,513,545,546,513,547,548,513,549,550,513,551,552,513,553,554,513,555,556,513,557,558,513,559,560,513,561,562,513,563,564,513,565,566,513,567,568,513,569,570,513,571,572,513,573,574,513,575,576,513,577,578,579,580,581,582,583,580,584,585,580,586,587,580,588,589,590,580,591,592,593,580,594],[926],{"_340":319,"_341":598,"_343":927,"_345":70},[600,601,602,178,603,604,605,178,606,607,608,609,610,611,612,178,613,614,612,178,615,616,617,618,619,612,178,615,620,617,621,622,612,178,615,623,617,624,625,612,178,615,626,617,627,628,612,178,615,629,630,617,631,632,612,178,615,633,617,634,635,636,612,178,615,637,617,638,639,640,612,641],[929,930,931,932,933,934,935],{"_651":57,"_57":662},{"_651":653,"_333":595,"_340":660,"_341":661},{"_651":57,"_57":659},{"_651":653,"_333":460,"_340":657,"_341":658},{"_651":57,"_57":656},{"_651":653,"_333":334,"_340":654,"_341":655},{"_651":57,"_57":652},[965,966,967],[],{"_295":964},{"_292":963},{"_292":962},{"_257":258,"_255":259,"_260":261,"_262":70,"_263":18,"_264":18,"_265":266,"_267":268,"_269":226,"_270":138,"_271":272,"_273":274,"_275":-5,"_276":-5,"_277":278,"_279":-5,"_280":-5,"_281":961,"_283":284,"_285":286,"_287":288,"_289":290},{"_53":134,"_172":173,"_174":175,"_176":18,"_177":178,"_179":180,"_181":-5,"_182":-5,"_183":-5,"_184":18,"_185":951,"_187":178,"_188":178,"_189":952,"_191":953,"_193":954,"_195":-5,"_196":197,"_198":199,"_200":201,"_202":955,"_66":-5,"_130":204,"_205":956,"_207":957,"_209":210,"_211":18,"_212":18},{"_144":944},[945,946,947],{"_156":950,"_158":169},{"_156":949,"_158":165},{"_156":948,"_158":159},{"_160":161,"_104":162,"_130":163},{"_160":166,"_104":162,"_130":167},{"_160":170,"_104":162,"_130":171},[216],{"_216":958},{},{},[],{"_215":70},{"_213":70,"_214":178},{"_53":218,"_160":175,"_219":959,"_221":222,"_223":224,"_225":226,"_227":228,"_229":230,"_231":232,"_233":75,"_234":18,"_235":-5,"_236":-5,"_237":-5,"_238":-5,"_239":-5,"_240":-5,"_241":242,"_243":-5,"_66":960},{"_249":250,"_53":251,"_104":252,"_253":254,"_255":256},{"_245":18,"_246":18,"_247":18,"_248":18},{"_249":250,"_160":250,"_291":-5,"_253":254,"_53":251},[],[],[],{"_104":642,"_108":105,"_343":984},{"_104":327,"_328":971},{"_104":300,"_301":302,"_303":304,"_305":968,"_307":-5},{"_104":305,"_308":309,"_139":140,"_310":-5,"_280":311,"_312":969,"_314":315,"_316":317,"_318":319,"_320":321,"_322":970},{"_324":325,"_326":-5},[],[972,973,974],{"_333":595,"_335":-5,"_336":981,"_338":-5},{"_333":460,"_335":-5,"_336":978,"_338":-5},{"_333":334,"_335":-5,"_336":975,"_338":-5},[976],{"_340":319,"_341":342,"_343":977,"_345":70},[346,347,348,349,350,351,352,353,178,354,355,356,356,357,358,359,360,361,362,361,363,361,364,365,366,361,367,368,369,370,371,372,373,374,375,366,361,367,376,369,377,375,378,379,375,366,361,380,369,381,375,366,361,382,369,383,375,366,361,384,369,385,375,366,361,386,369,387,375,366,361,388,369,389,375,366,361,390,369,391,375,366,361,392,369,393,375,366,178,394,369,395,375,366,178,396,369,397,375,366,178,398,369,399,375,366,178,400,369,401,375,366,178,402,369,403,375,366,178,404,369,405,375,366,178,406,369,407,375,366,178,408,369,409,375,366,178,410,369,411,375,366,178,412,369,413,375,366,178,414,369,415,375,366,178,416,369,417,375,366,178,418,369,419,375,366,178,420,369,421,375,366,178,422,369,423,375,366,178,424,369,425,375,366,178,426,369,427,375,366,178,428,369,429,375,366,178,430,369,431,375,366,178,432,369,433,375,366,178,434,369,435,375,366,178,436,369,437,375,366,178,438,369,439,375,366,178,440,369,441,375,366,178,442,369,443,375,366,178,444,369,445,375,366,178,446,369,447,375,366,178,448,369,449,375,366,178,450,369,451,375,366,178,452,369,453,375,366,178,454,369,455,375,366,178,456,369,457,375,366,458,459],[979],{"_340":319,"_341":463,"_343":980,"_345":70},[465,466,467,468,469,470,471,472,473,474,475,476,477,478,479,480,481,482,483,484,485,475,486,487,483,484,488,475,476,489,490,478,491,492,493,482,483,484,494,475,476,489,478,495,483,484,496,497,498,499,500,501,502,503,504,502,505,506,507,508,502,509,510,507,508,502,511,512,513,514,515,513,516,517,513,518,519,513,520,521,513,522,523,513,524,525,513,526,527,513,528,529,513,530,515,513,531,532,513,533,534,513,535,536,513,537,538,513,539,540,513,541,542,513,543,544,513,545,546,513,547,548,513,549,550,513,551,552,513,553,554,513,555,556,513,557,558,513,559,560,513,561,562,513,563,564,513,565,566,513,567,568,513,569,570,513,571,572,513,573,574,513,575,576,513,577,578,579,580,581,582,583,580,584,585,580,586,587,580,588,589,590,580,591,592,593,580,594],[982],{"_340":319,"_341":598,"_343":983,"_345":70},[600,601,602,178,603,604,605,178,606,607,608,609,610,611,612,178,613,614,612,178,615,616,617,618,619,612,178,615,620,617,621,622,612,178,615,623,617,624,625,612,178,615,626,617,627,628,612,178,615,629,630,617,631,632,612,178,615,633,617,634,635,636,612,178,615,637,617,638,639,640,612,641],[985,986,987,988,989,990,991],{"_651":57,"_57":662},{"_651":653,"_333":595,"_340":660,"_341":661},{"_651":57,"_57":659},{"_651":653,"_333":460,"_340":657,"_341":658},{"_651":57,"_57":656},{"_651":653,"_333":334,"_340":654,"_341":655},{"_651":57,"_57":652},[994,995,996,997,998,999,1000,1001,1002,1003,1004,1005,1006,1007,1008,1009,1010,1011,1012,1013,1014,1015,1016,1017,1018,1019,1020,1021,1022,1023,1024,1025,1026,1027,1028,1029,1030,1031,1032,1033,1034,1035,1036,1037,1038,1039,1040,1041,1042,1043,1044,1045,1046,1047,1048,1049,1050,1051,1052,1053,1054,1055,1056,1057,1058,1059,1060,1061,1062,1063,1064],{"_672":673,"_221":222,"_674":258,"_233":-5,"_308":216,"_675":18,"_676":-5,"_677":-5,"_678":679,"_680":681,"_682":-5,"_683":-5,"_684":685,"_686":18,"_687":70,"_688":65,"_689":18,"_690":-5},{"_104":642,"_108":281,"_343":1065},{"_265":266,"_267":268,"_875":272,"_876":140,"_277":-5,"_104":877},{"_104":762,"_267":775,"_333":-5,"_764":-5,"_765":-5},{"_104":762,"_267":874,"_333":-5,"_764":-5,"_765":-5},{"_104":762,"_267":873,"_333":-5,"_764":-5,"_765":-5},{"_104":762,"_267":872,"_333":-5,"_764":-5,"_765":-5},{"_104":762,"_267":869,"_333":870,"_764":769,"_765":871},{"_104":762,"_267":866,"_333":867,"_764":769,"_765":868},{"_104":762,"_267":865,"_333":-5,"_764":-5,"_765":-5},{"_104":762,"_267":864,"_333":-5,"_764":-5,"_765":-5},{"_104":762,"_267":864,"_333":-5,"_764":-5,"_765":-5},{"_104":762,"_267":863,"_333":-5,"_764":-5,"_765":-5},{"_104":762,"_267":861,"_333":862,"_764":769,"_765":-5},{"_104":762,"_267":859,"_333":860,"_764":769,"_765":-5},{"_104":762,"_267":857,"_333":858,"_764":769,"_765":-5},{"_104":762,"_267":855,"_333":856,"_764":769,"_765":-5},{"_104":762,"_267":853,"_333":854,"_764":769,"_765":-5},{"_104":762,"_267":852,"_333":851,"_764":769,"_765":-5},{"_104":762,"_267":850,"_333":851,"_764":769,"_765":-5},{"_104":762,"_267":848,"_333":849,"_764":769,"_765":-5},{"_104":762,"_267":847,"_333":768,"_764":769,"_765":-5},{"_104":762,"_267":845,"_333":846,"_764":769,"_765":-5},{"_104":762,"_267":843,"_333":844,"_764":769,"_765":-5},{"_104":762,"_267":841,"_333":842,"_764":769,"_765":-5},{"_104":762,"_267":840,"_333":839,"_764":769,"_765":-5},{"_104":762,"_267":838,"_333":839,"_764":769,"_765":-5},{"_104":762,"_267":837,"_333":836,"_764":769,"_765":-5},{"_104":762,"_267":835,"_333":836,"_764":769,"_765":-5},{"_104":762,"_267":833,"_333":834,"_764":769,"_765":-5},{"_104":762,"_267":831,"_333":832,"_764":769,"_765":-5},{"_104":762,"_267":829,"_333":830,"_764":769,"_765":-5},{"_104":762,"_267":827,"_333":828,"_764":769,"_765":-5},{"_104":762,"_267":825,"_333":826,"_764":769,"_765":-5},{"_104":762,"_267":823,"_333":824,"_764":769,"_765":-5},{"_104":762,"_267":821,"_333":822,"_764":769,"_765":-5},{"_104":762,"_267":819,"_333":820,"_764":769,"_765":-5},{"_104":762,"_267":817,"_333":818,"_764":769,"_765":-5},{"_104":762,"_267":815,"_333":816,"_764":769,"_765":-5},{"_104":762,"_267":813,"_333":814,"_764":769,"_765":-5},{"_104":762,"_267":811,"_333":812,"_764":769,"_765":-5},{"_104":762,"_267":809,"_333":810,"_764":769,"_765":-5},{"_104":762,"_267":807,"_333":808,"_764":769,"_765":-5},{"_104":762,"_267":805,"_333":806,"_764":769,"_765":-5},{"_104":762,"_267":803,"_333":804,"_764":769,"_765":-5},{"_104":762,"_267":801,"_333":802,"_764":769,"_765":-5},{"_104":762,"_267":799,"_333":800,"_764":769,"_765":-5},{"_104":762,"_267":797,"_333":798,"_764":769,"_765":-5},{"_104":762,"_267":794,"_333":795,"_764":769,"_765":796},{"_104":762,"_267":791,"_333":792,"_764":769,"_765":793},{"_104":762,"_267":789,"_333":790,"_764":769,"_765":-5},{"_104":762,"_267":786,"_333":787,"_764":769,"_765":788},{"_104":762,"_267":785,"_333":595,"_764":769,"_765":-5},{"_104":762,"_267":784,"_333":-5,"_764":-5,"_765":-5},{"_104":762,"_267":783,"_333":-5,"_764":-5,"_765":-5},{"_104":762,"_267":783,"_333":-5,"_764":-5,"_765":-5},{"_104":762,"_267":782,"_333":-5,"_764":-5,"_765":-5},{"_104":762,"_267":781,"_333":-5,"_764":-5,"_765":-5},{"_104":762,"_267":775,"_333":-5,"_764":-5,"_765":-5},{"_104":762,"_267":780,"_333":-5,"_764":-5,"_765":-5},{"_104":762,"_267":779,"_333":-5,"_764":-5,"_765":-5},{"_104":762,"_267":778,"_333":-5,"_764":-5,"_765":-5},{"_104":762,"_267":775,"_333":-5,"_764":-5,"_765":-5},{"_104":762,"_267":777,"_333":-5,"_764":-5,"_765":-5},{"_104":762,"_267":776,"_333":-5,"_764":-5,"_765":-5},{"_104":762,"_267":775,"_333":-5,"_764":-5,"_765":-5},{"_104":762,"_267":774,"_333":-5,"_764":-5,"_765":-5},{"_104":762,"_267":773,"_333":-5,"_764":-5,"_765":-5},{"_104":762,"_267":771,"_333":595,"_764":769,"_765":772},{"_104":762,"_267":767,"_333":768,"_764":769,"_765":770},{"_104":762,"_267":766,"_333":-5,"_764":-5,"_765":-5},{"_104":762,"_267":763,"_333":-5,"_764":-5,"_765":-5},[1066],{"_651":57,"_57":763},"task_e_6a14232a122083219e2f34d589d78b37",1779704618.070911,"Github Mention: feat(analyzers): renumber to 9 domain bands, 2.0.0 break","has_generated_title","has_unread_turn","denormalized_metadata",{"_1098":43,"_1099":1100,"_1101":-5,"_1102":-5,"_1092":1103,"_1104":-5,"_1105":-5,"_1106":-5,"_1107":-5},"task_status_display",{"_1086":1087,"_1088":175,"_141":138,"_1089":-5,"_1090":300},"creator_workspace","external_pull_requests",[1079],{"_53":1080,"_1081":95,"_1082":1083,"_1084":140},"github-1122160185-174","assistant_turn_id","pull_request",{"_257":258,"_255":259,"_260":261,"_262":70,"_263":18,"_264":18,"_265":266,"_267":268,"_269":226,"_270":138,"_271":272,"_273":274,"_275":-5,"_276":-5,"_277":278,"_279":-5,"_280":-5,"_281":1085,"_283":284,"_285":286,"_287":288,"_289":290},"codex_updated_sha",{"_249":250,"_160":250,"_291":-5,"_253":254,"_53":251},"latest_turn_status_display",{"_1091":95,"_123":124,"_1092":1093,"_106":1094,"_1095":-5,"_1096":-5,"_1097":300,"_125":-5},"environment_label","fix_task_id","initial_intent","turn_id","diff_stats",{"_314":315,"_316":317,"_318":319},[],"image_asset_pointer_citations","issue_counts","intent","num_follow_ups","repo_name_map",{},"last_user_turn_created_at","last_turn_status",{"_314":-5,"_316":-5,"_318":-5},"last_pushed_turn_id","start_branch_name","pushed_branch_name","cached_pull_request_data","dd",{"_1122":1123,"_1124":1125},"disablePrefetch","shouldPrefetchAccount","shouldPrefetchUser","shouldPrefetchSystemHints","manageRootRouteStylesheet","mobileWebHandoffEnabled","promoteCss","disableSSR","stripModulepreloadImports","paidCampaignAppRedirect","statsigGateEvaluationsPromise",["P",1121],"traceId","13202086881228266431","traceTime",1788505493703]}}