skill-al-review-pipeline

AL review pipeline

Read the provider contract and task-context guidance. Resolve these links from this installed skill, not a sibling repository. This procedure supplements the caller's write scope, report format and human gate; it grants no extra tools and never authorizes changes to the provider or AL code.

Scope and independent evidence

Start from the requested files/diff and acceptance criteria. A standalone reviewer resolves context itself; a Conductor reviewer consumes the supplied phase context and reads precise references only for gaps. An explicitly attached object is a valid scope without a Git diff. Missing source means no code review has occurred. Read relevant rules explicitly; do not assume another agent loaded them for you. Verify claims in an implementer's report against current code and evidence. Never approve code generated in your own context as an independent review.

Execute the selected provider

1. Select the provider from this project's configuration. In plugin mode discover the exact configured identity in the host and load its instructions here; home, entryPoint and workspace belong to multiroot selection. Do not fall back to a different clone or rename a skill implicitly.
2. Follow the loaded provider's execution model. For an instruction-based skill, loading starts the workflow: apply its Entry/routing, READ/DO and dispatched checks to the actual inputs. You produce their results; do not wait for the host to emit them automatically or recursively invoke the same skill. A tool invocation is required only when the loaded provider actually requires one.
3. Derive layers and filters from supported provider configuration/defaults. Preserve the actual routing result. Do not pick only the most obvious leaf: account for each dispatched check and each applicable child of a super-skill as completed, not applicable, skipped with reason, partial, failed or pending. Follow provider relevance rules; do not execute irrelevant checks merely to fill a table. Reuse already-read knowledge within this invocation. Entry may return more than one top-level skill in dispatch[] — for example the al-code-review super-skill plus the Community al-agents-review leaf, which is a first-level peer and not one of the super-skill's children. Execute every dispatched entry and retain every resulting findings-report; never treat the first report as the whole run. Roll each one into sub-results and account for it in coverage under its own domain label. When the provider documents bounded retrieval helpers (BCQuality: tools/Search-Knowledge.ps1 for the catalog and tools/Get-KnowledgeArticles.ps1 for bodies), prefer them over ad-hoc reads where an execution capability exists: they page deterministically, cap each response, and return per-article SHA-256 so a cited body is provably the one that was read. Without that capability, use the provider's native bounded reads through EOF and never treat a retrieval failure as an empty result.
4. Index refresh is independent. If writes/PowerShell are unavailable or forbidden, record provider.index.status: not-attempted and use the provider's documented path-based lookup. When the project carries an installer receipt (.github/aldc-bcquality-index.json) whose corpusSha matches the observed corpus revision, report provider.index.status: prebuilt with the receipt's generator, index path, SHA-256 and corpus revision; a bare status is refused. Map the receipt into the observation as indexPath → index.path, indexSha256 → index.sha256; generator and corpusSha carry over unchanged. Then use the provider's index-backed retrieval — including its bounded pagination and per-article content-hash validation. A mismatched or absent receipt is not-attempted, not a failure. A missing fresh index alone does not cancel code review. If lookup fails, name the exact resource, attempted operation and observed error.
5. Retain actual sub-results and references. Reading a leaf or describing its intended behavior is not a completed check. An executed attempt can fail or be partial; executed: true does not mean success. Never reconstruct unobserved results afterwards as evidence. Zero findings is valid for completed checks.