---
id: change/bcquality/118
type: change
title: "#118 knowledge: 3 FP-suppression guards from BC apps negative feedback"
summary: Three knowledge articles are added to clarify AL facts that were incorrectly flagged by the review agent, suppressing false-positive findings for ObsoleteReason omissions, bare Get() calls, and OnAfterGetCurrRecord behavior.
tier: official
language: en
tags:
  - other
  - main
system: development
review:
  state: reviewed
  by: opus
  at: "2026-10-08T02:11:58.784Z"
  flags: []
generated:
  at: "2026-10-08T01:26:53.141Z"
  pipeline: 0.2.0
  prompts:
    extract-change: 1
  input_hash: 8fb7f61808c5ddacc809abb5aac5230db1186cca
evidence:
  - kind: code
    url: https://github.com/microsoft/BCQuality/pull/118
    title: "microsoft/BCQuality#118: knowledge: 3 FP-suppression guards from BC apps negative feedback"
    date: "2026-07-21"
    commit: 8fb7f61808c5ddacc809abb5aac5230db1186cca
    t: null
    quote: Each encodes an AL fact that the review agent got wrong, so the corresponding finding is suppressed going forward.
links:
  learn: []
  objects: []
  features: []
  topics: []
  localizations: []
  videos: []
  posts: []
  guidelines: []
  changes: []
number: 118
repo: microsoft/BCQuality
source_id: bcquality-prs
url: https://github.com/microsoft/BCQuality/pull/118
kind: pr
base_branch: main
major: null
merged_at: "2026-07-21T08:56:16Z"
author: gggdttt
author_type: User
community_contribution: false
labels: []
change_class: code
change_kind: other
behavior_change: false
breaking: false
files:
  count: 3
  additions: 78
  deletions: 0
  al: 3
apps: []
objects_touched: []
objects_unjoined:
  - path: microsoft/knowledge/error-handling/unchecked-get-throws-when-record-not-found.md
    status: added
    reason: not-in-snapshot
  - path: microsoft/knowledge/performance/onaftergetcurrrecord-is-not-per-row.md
    status: added
    reason: not-in-snapshot
  - path: microsoft/knowledge/upgrade/obsoletereason-need-not-restate-removal-version.md
    status: added
    reason: not-in-snapshot
obsoletions: []
backports: []
fixes_issues: []
work_items: []
systems:
  - development
quote: Each encodes an AL fact that the review agent got wrong, so the corresponding finding is suppressed going forward.
joined_against:
  major: null
  commit: null
---

# #118 knowledge: 3 FP-suppression guards from BC apps negative feedback

> Three knowledge articles are added to clarify AL facts that were incorrectly flagged by the review agent, suppressing false-positive findings for ObsoleteReason omissions, bare Get() calls, and OnAfterGetCurrRecord behavior.

[Pull request](https://github.com/microsoft/BCQuality/pull/118) · merged into `main` on 2026-07-21 by gggdttt · 3 files (+78 -0), 3 AL · other · tier official · reviewed (checked by Opus)

## What changed

- ObsoleteTag carries removal version; ObsoleteReason need not restate it
- Bare Rec.Get() raises an error when record is missing, not silently ignored
- OnAfterGetCurrRecord fires on current-record change, not per row; CalcFields there is not N+1
- Articles follow existing negative-clarification style with false-positive keyword
- Knowledge index grows from 245 to 248 entries

## AL objects touched

- `microsoft/knowledge/error-handling/unchecked-get-throws-when-record-not-found.md`: added, not in the BC? snapshot yet
- `microsoft/knowledge/performance/onaftergetcurrrecord-is-not-per-row.md`: added, not in the BC? snapshot yet
- `microsoft/knowledge/upgrade/obsoletereason-need-not-restate-removal-version.md`: added, not in the BC? snapshot yet

## Context

- Systems: development
- From the description: "Each encodes an AL fact that the review agent got wrong, so the corresponding finding is suppressed going forward."

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