Skip to content

refactor(lode): centralize open analyzer candidates in one register - #155

Merged
e128 merged 1 commit into
mainfrom
feature/central-analyzer-candidates
Nov 12, 2026
Merged

e128 merged 1 commit into
mainfrom
feature/central-analyzer-candidates

Conversation

@e128

@e128 e128 commented Nov 12, 2026

Copy link
Copy Markdown
Owner

Executive Summary

Open analyzer candidates lived in scattered files under lode/dotnet/. Each sweep wrote its own file, so no single view of the open register existed. This change moves every candidate file into lode/dotnet/analyzer-candidates/ and adds index.md as the single open register. catalog.md keeps only settled rules.

Code Changes

  • Add lode/dotnet/analyzer-candidates/ with index.md, catalog.md, and the sweep files (async, external, performance).
  • Update lode-map.md, analyzers summary.md, rule-details.md, and fips-rules.md for the new layout.
  • Update plans/async-analyzers/ to reference the central register.
  • Update AGENTS.md task references to the new register paths.

Tests and Tooling

  • No .cs files changed, so build and test did not run.
  • Analyzer release check passed (clean) and the analyzer README rule table shows no drift.

Review Guide

Review lode/dotnet/analyzer-candidates/index.md first. It defines the open register. Then check that no candidate content was lost in the moves and that lode-map.md lists every file.

Rollback

Revert the single commit on this branch. No shipped artifacts depend on the new layout.

Links

Related plan: plans/async-analyzers/async-analyzers-plan.md

Yeeted with Claude Code

Move every candidate file into lode/dotnet/analyzer-candidates/ and add
index.md as the single open register. catalog.md keeps only settled rules.
A consumer repository records no candidate of its own; pugworks now points
here and its scripts read and write this register.

Co-Authored-By: Claude
@e128
e128 merged commit 0652849 into main Nov 12, 2026
2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant