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

#119 Add batch-2 FP guards: sentence-case action captions + PK Get is transaction-cached

Two knowledge articles were added to guide developers about correct use of sentence-case action captions and transaction-cached primary key lookups, helping suppress false positive quality warnings.

Open in Claude119.mdPull request #119

Pull request · merged into main on 2026-07-22 by gggdttt · 3 files (+88 -0), 3 AL · feature · tier official · reviewed (checked by Opus)

What changed

  • Action captions that express imperative commands should use sentence case (capitalize only the first word and proper nouns), not title case
  • Primary key Get calls within a transaction are already cached by the system and do not require manual dictionary caching when used per-row
  • Knowledge articles with example code help developers understand when and why certain coding patterns are correct
  • These articles address feedback from Business Central app maintainers about false positive quality rule flagging
  • Knowledge index was updated to include the two new articles

AL objects touched

  • microsoft/knowledge/performance/primary-key-get-in-loop-is-transaction-cached.md: added, not in the BC? snapshot yet
  • microsoft/knowledge/ui/caption-capitalization-noun-phrase-vs-sentence-phrase.good.al: added, not in the BC? snapshot yet
  • microsoft/knowledge/ui/caption-capitalization-noun-phrase-vs-sentence-phrase.md: added, not in the BC? snapshot yet

Context

  • Systems: development, copilot
  • From the description: “Two knowledge articles that suppress false positives thumbs-downed by BCApps maintainers.”

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).