---
id: change/bcquality/204
type: change
title: "#204 Validate composed reviews against the expected leaf worklist"
summary: The findings-report validator gets an optional expected-composition manifest. It checks that selected leaves, versions, result order and legitimate skips match what the host chose. Duplicate producers are now rejected, and a composition with missing leaves must be reported as partial or failed instead of completed.
tier: official
language: en
tags:
  - feature
  - main
system: development
review:
  state: reviewed
  by: opus
  at: "2026-10-08T02:02:45.616Z"
  flags: []
generated:
  at: "2026-10-08T01:26:53.141Z"
  pipeline: 0.2.0
  prompts:
    extract-change: 1
  input_hash: 206c5feff4160593c56edc501b52504d06b9ed6d
evidence:
  - kind: code
    url: https://github.com/microsoft/BCQuality/pull/204
    title: "microsoft/BCQuality#204: Validate composed reviews against the expected leaf worklist"
    date: "2026-10-02"
    commit: 206c5feff4160593c56edc501b52504d06b9ed6d
    t: null
    quote: null
links:
  learn: []
  objects: []
  features: []
  topics: []
  localizations: []
  videos: []
  posts: []
  guidelines: []
  changes: []
number: 204
repo: microsoft/BCQuality
source_id: bcquality-prs
url: https://github.com/microsoft/BCQuality/pull/204
kind: pr
base_branch: main
major: null
merged_at: "2026-10-02T08:27:28Z"
author: demiliani
author_type: User
community_contribution: false
labels: []
change_class: code
change_kind: feature
behavior_change: true
breaking: false
files:
  count: 6
  additions: 784
  deletions: 26
  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
quote: null
joined_against:
  major: null
  commit: null
---

# #204 Validate composed reviews against the expected leaf worklist

> The findings-report validator gets an optional expected-composition manifest. It checks that selected leaves, versions, result order and legitimate skips match what the host chose. Duplicate producers are now rejected, and a composition with missing leaves must be reported as partial or failed instead of completed.

[Pull request](https://github.com/microsoft/BCQuality/pull/204) · merged into `main` on 2026-10-02 by demiliani · 6 files (+784 -26), 2 AL · feature · tier official · reviewed (checked by Opus)

## What changed

- New optional -ExpectedCompositionPath checks super-skill and leaf IDs, versions, result order and skip reasons against a host-owned manifest
- Duplicate producers and duplicate or returned-and-skipped IDs are rejected even without a manifest
- Missing selected leaves require 'partial' if a non-failed report exists, otherwise 'failed'; budget exhaustion is not turned into a configuration skip
- Coverage is the sum of non-failed leaf worklists and handles zero usable sub-results
- Report JSON schema is unchanged; manifest-free callers are still supported but cannot certify completeness
- 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

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