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

#132 Refine self-improvement review guidance

BCQuality guidance refined to reduce false positives in event handling, UI testing, performance analysis, and label scope rules by narrowing criteria to match documented platform behavior and actual code semantics.

Open in Claude132.mdPull request #132

Pull request · merged into main on 2026-09-03 by gggdttt · 28 files (+295 -224), 27 AL · other · tier official · reviewed (checked by Opus)

What changed

  • IsHandled reset rules refined to consider loop-carried state and values that can actually become true
  • Local Label scope removed as a correctness rule since labels are supported in extension translation
  • UI handler tests allowed to use semantic capture/reset/assert patterns with queues reserved for interaction contracts
  • Performance guidance for ModifyAll and DeleteAll aligned with documented fallback conditions
  • Persisted checkpoint and bounded retrieval requirements clarified as separate concerns
  • Behaviour changes for users or extensions.

AL objects touched

  • microsoft/knowledge/events/initialize-ishandled-to-false-before-publishing.bad.al: removed
  • microsoft/knowledge/events/initialize-ishandled-to-false-before-publishing.good.al: removed
  • microsoft/knowledge/events/initialize-ishandled-to-false-before-publishing.md: removed
  • microsoft/knowledge/events/reset-ishandled-only-when-the-value-can-carry-over.bad.al: added, not in the BC? snapshot yet
  • microsoft/knowledge/events/reset-ishandled-only-when-the-value-can-carry-over.good.al: added, not in the BC? snapshot yet
  • microsoft/knowledge/events/reset-ishandled-only-when-the-value-can-carry-over.md: added, not in the BC? snapshot yet
  • microsoft/knowledge/performance/avoid-commit-inside-loops.bad.al: modified, not in the BC? snapshot
  • microsoft/knowledge/performance/avoid-commit-inside-loops.good.al: modified, not in the BC? snapshot
  • microsoft/knowledge/performance/avoid-commit-inside-loops.md: modified, not in the BC? snapshot
  • microsoft/knowledge/performance/prefer-modifyall-over-per-row-modify.md: modified, not in the BC? snapshot
  • microsoft/knowledge/performance/triggers-and-media-field-regress-modifyall.md: modified, not in the BC? snapshot
  • microsoft/knowledge/performance/use-deleteall-for-filtered-bulk-deletion.md: modified, not in the BC? snapshot
  • microsoft/knowledge/style/labels-declared-at-object-scope.bad.al: removed
  • microsoft/knowledge/style/labels-declared-at-object-scope.good.al: removed
  • microsoft/knowledge/style/labels-declared-at-object-scope.md: modified, not in the BC? snapshot
  • microsoft/knowledge/testing/ui-handlers-in-tests.bad.al: modified, not in the BC? snapshot
  • microsoft/knowledge/testing/ui-handlers-in-tests.good.al: modified, not in the BC? snapshot
  • microsoft/knowledge/testing/ui-handlers-in-tests.md: modified, not in the BC? snapshot
  • microsoft/skills/review/al-events-review.md: modified, not in the BC? snapshot
  • microsoft/skills/review/al-performance-review.md: modified, not in the BC? snapshot
  • microsoft/skills/review/al-privacy-review.md: modified, not in the BC? snapshot
  • microsoft/skills/review/al-security-review.md: modified, not in the BC? snapshot
  • microsoft/skills/review/al-style-review.md: modified, not in the BC? snapshot
  • microsoft/skills/review/al-testing-review.md: modified, not in the BC? snapshot
  • microsoft/skills/review/al-ui-review.md: modified, not in the BC? snapshot
  • microsoft/skills/review/al-upgrade-review.md: modified, not in the BC? snapshot
  • skills/do.md: modified, not in the BC? snapshot

Context

  • Systems: development
  • From the description: “narrow IsHandled reset findings to values that can actually carry true, including loop-carried state”

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