---
id: change/bcquality/54
type: change
title: "#54 Emit human-readable domain label on review findings"
summary: BCQuality review leaves now set an optional human-readable domain label on every finding, so BCQuality owns the display label instead of the consumer engine's static map. Once the BC-ALAgents counterpart is merged, new review domains can render with their own label rather than Other.
tier: official
language: en
tags:
  - feature
  - main
system: development
review:
  state: reviewed
  by: opus
  at: "2026-10-08T02:12:44.759Z"
  flags: []
generated:
  at: "2026-10-08T01:26:53.141Z"
  pipeline: 0.2.0
  prompts:
    extract-change: 1
  input_hash: ae04938c03bc360a45558d21b7df9345ba5dafcb
evidence:
  - kind: code
    url: https://github.com/microsoft/BCQuality/pull/54
    title: "microsoft/BCQuality#54: Emit human-readable domain label on review findings"
    date: "2026-07-15"
    commit: ae04938c03bc360a45558d21b7df9345ba5dafcb
    t: null
    quote: Every active AL review leaf sets `domain` on every finding it emits, including leaf-level agent findings.
links:
  learn: []
  objects: []
  features: []
  topics: []
  localizations: []
  videos: []
  posts: []
  guidelines: []
  changes: []
number: 54
repo: microsoft/BCQuality
source_id: bcquality-prs
url: https://github.com/microsoft/BCQuality/pull/54
kind: pr
base_branch: main
major: null
merged_at: "2026-07-15T08:44:51Z"
author: JesperSchulz
author_type: User
community_contribution: false
labels: []
change_class: code
change_kind: feature
behavior_change: true
breaking: false
files:
  count: 19
  additions: 91
  deletions: 56
  al: 18
apps: []
objects_touched: []
objects_unjoined:
  - path: microsoft/skills/review/al-appsource-review.md
    status: modified
    reason: not-in-snapshot
  - path: microsoft/skills/review/al-breaking-changes-review.md
    status: modified
    reason: not-in-snapshot
  - path: microsoft/skills/review/al-code-review.md
    status: modified
    reason: not-in-snapshot
  - path: microsoft/skills/review/al-data-modeling-review.md
    status: modified
    reason: not-in-snapshot
  - path: microsoft/skills/review/al-error-handling-review.md
    status: modified
    reason: not-in-snapshot
  - path: microsoft/skills/review/al-events-review.md
    status: modified
    reason: not-in-snapshot
  - path: microsoft/skills/review/al-interfaces-review.md
    status: modified
    reason: not-in-snapshot
  - path: microsoft/skills/review/al-performance-review.md
    status: modified
    reason: not-in-snapshot
  - path: microsoft/skills/review/al-privacy-review.md
    status: modified
    reason: not-in-snapshot
  - path: microsoft/skills/review/al-security-review.md
    status: modified
    reason: not-in-snapshot
  - path: microsoft/skills/review/al-style-review.md
    status: modified
    reason: not-in-snapshot
  - path: microsoft/skills/review/al-telemetry-review.md
    status: modified
    reason: not-in-snapshot
  - path: microsoft/skills/review/al-testing-review.md
    status: modified
    reason: not-in-snapshot
  - path: microsoft/skills/review/al-ui-review.md
    status: modified
    reason: not-in-snapshot
  - path: microsoft/skills/review/al-upgrade-review.md
    status: modified
    reason: not-in-snapshot
  - path: microsoft/skills/review/al-web-services-review.md
    status: modified
    reason: not-in-snapshot
  - path: skills/bcquality-al-review/SKILL.md
    status: modified
    reason: not-in-snapshot
  - path: skills/do.md
    status: modified
    reason: not-in-snapshot
obsoletions: []
backports: []
fixes_issues: []
work_items:
  - 640493
systems:
  - development
  - copilot
quote: Every active AL review leaf sets `domain` on every finding it emits, including leaf-level agent findings.
joined_against:
  major: null
  commit: null
---

# #54 Emit human-readable domain label on review findings

> BCQuality review leaves now set an optional human-readable domain label on every finding, so BCQuality owns the display label instead of the consumer engine's static map. Once the BC-ALAgents counterpart is merged, new review domains can render with their own label rather than Other.

[Pull request](https://github.com/microsoft/BCQuality/pull/54) · merged into `main` on 2026-07-15 by JesperSchulz · 19 files (+91 -56), 18 AL · feature · tier official · reviewed (checked by Opus)

## What changed

- Adds optional findings[].domain to the shared review output schema
- Each AL review leaf sets a domain label on every finding; cross-cutting super-skill findings use Agent
- al-code-review preserves a leaf finding's domain verbatim during rollup
- Domain is display text that consumers must preserve exactly, without lossy slugification
- Additive and optional: consumers may fall back to the legacy from-sub-skill mapping when domain is absent
- Behaviour changes for users or extensions.

## AL objects touched

- `microsoft/skills/review/al-appsource-review.md`: modified, not in the BC? snapshot
- `microsoft/skills/review/al-breaking-changes-review.md`: modified, not in the BC? snapshot
- `microsoft/skills/review/al-code-review.md`: modified, not in the BC? snapshot
- `microsoft/skills/review/al-data-modeling-review.md`: modified, not in the BC? snapshot
- `microsoft/skills/review/al-error-handling-review.md`: modified, not in the BC? snapshot
- `microsoft/skills/review/al-events-review.md`: modified, not in the BC? snapshot
- `microsoft/skills/review/al-interfaces-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-telemetry-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
- `microsoft/skills/review/al-web-services-review.md`: modified, not in the BC? snapshot
- `skills/bcquality-al-review/SKILL.md`: modified, not in the BC? snapshot
- `skills/do.md`: modified, not in the BC? snapshot

## Context

- Work items: AB#640493
- Systems: development, copilot
- From the description: "Every active AL review leaf sets `domain` on every finding it emits, including leaf-level agent findings."

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