---
id: change/bcquality/180
type: change
title: "#180 Normalize recoverable leaf finding ranges"
summary: Review findings whose primary line lies inside location.range but differs from start-line are no longer rejected. A deep-copied candidate drops the optional range, keeps the raw output unchanged, and must pass the existing strict acceptance gate. All other invalid or ambiguous leaves still fail.
tier: official
language: en
tags:
  - fix
  - main
system: development
review:
  state: reviewed
  by: opus
  at: "2026-10-08T02:05:30.375Z"
  flags: []
generated:
  at: "2026-10-08T01:26:53.141Z"
  pipeline: 0.2.0
  prompts:
    extract-change: 1
  input_hash: 35d0966a8d45e8d4a7c2b0238afbffe338a31d2b
evidence:
  - kind: code
    url: https://github.com/microsoft/BCQuality/pull/180
    title: "microsoft/BCQuality#180: Normalize recoverable leaf finding ranges"
    date: "2026-09-11"
    commit: 35d0966a8d45e8d4a7c2b0238afbffe338a31d2b
    t: null
    quote: null
links:
  learn: []
  objects: []
  features: []
  topics: []
  localizations: []
  videos: []
  posts: []
  guidelines: []
  changes: []
number: 180
repo: microsoft/BCQuality
source_id: bcquality-prs
url: https://github.com/microsoft/BCQuality/pull/180
kind: pr
base_branch: main
major: null
merged_at: "2026-09-11T13:24:20Z"
author: JesperSchulz
author_type: User
community_contribution: false
labels: []
change_class: code
change_kind: fix
behavior_change: true
breaking: false
files:
  count: 6
  additions: 250
  deletions: 21
  al: 2
apps: []
objects_touched: []
objects_unjoined:
  - path: microsoft/skills/review/al-code-review.md
    status: modified
    reason: not-in-snapshot
  - path: skills/do.md
    status: modified
    reason: not-in-snapshot
obsoletions: []
backports: []
fixes_issues: []
work_items: []
systems:
  - development
  - platform
quote: null
joined_against:
  major: null
  commit: null
---

# #180 Normalize recoverable leaf finding ranges

> Review findings whose primary line lies inside location.range but differs from start-line are no longer rejected. A deep-copied candidate drops the optional range, keeps the raw output unchanged, and must pass the existing strict acceptance gate. All other invalid or ambiguous leaves still fail.

[Pull request](https://github.com/microsoft/BCQuality/pull/180) · merged into `main` on 2026-09-11 by JesperSchulz · 6 files (+250 -21), 2 AL · fix · tier official · reviewed (checked by Opus)

## What changed

- Optional location.range is removed when start-line <= line <= end-line, start-line differs from line, and no suggested-code is present
- Raw Task output is kept unchanged as the audit payload; normalization is recorded separately in private telemetry
- The normalized candidate must pass the full existing strict acceptance gate before rollup
- Reversed or out-of-bounds ranges, missing fields, and bad JSON or paths still fail the whole leaf
- New Test-ReviewContract.ps1 regression test keeps the DO, coordinator and standalone runner docs aligned
- Behaviour changes for users or extensions.

## AL objects touched

- `microsoft/skills/review/al-code-review.md`: modified, not in the BC? snapshot
- `skills/do.md`: modified, not in the BC? snapshot

## Context

- Systems: development, platform

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