BC Observatoryunofficial, made by waldo
code change · fixAL Developmentofficial - Microsoftreviewed - checked by Opus

#95 style-review: calibrate analyzer-redundant rules to info; keep correctness bugs out of style scope

The al-style-review skill recalibrates analyzer-redundant rules to info severity and clarifies domain boundaries to keep correctness issues out of the style scope, reducing false-positive findings in code review.

Open in Claude95.mdPull request #95

Pull request · merged into main on 2026-07-14 by gggdttt · 1 file (+6 -4), 1 AL · fix · tier official · reviewed (checked by Opus)

What changed

  • Mechanical conventions already enforced by CodeCop (like this keyword, tooltip requirements, label scope) are downgraded to info severity instead of minor
  • Correctness, logic, and data-integrity defects are redirected from the style domain to appropriate channels like al-code-review’s agent channel
  • Severity calibration reserves minor for style issues with concrete downstream impact that analyzers don’t catch (translation loss, option caption mismatches, misleading invocations)
  • Domain boundaries clarified with concrete counter-examples and a reliable test: if no style best practice can be cited, it’s likely not a style finding
  • Changes are guidance-only edits to the skill; no schema or engine changes
  • Behaviour changes for users or extensions.

AL objects touched

  • microsoft/skills/review/al-style-review.md: modified, not in the BC? snapshot

Context

  • Systems: development, reporting

Source: metadata of a merged pull request in microsoft/BCQuality (bcquality): title, labels, changed paths and counts, with a summary in our words. No source text and no description beyond one short quote (D10, D61).