# Blast radius and review coverage

This document defines the judged focus model. It also defines track constraints,
review coverage, drift control, and commit discipline. [track-workflow.md](track-workflow.md)
is the lifecycle spine. [review-rules.md](review-rules.md) defines reviewer
composition and finding handling.

## Focus states and track constraints

The orchestrator judges all eleven focus areas for the change and independently
for every track. An area is **NAMED** when the orchestrator submits its
four-part proof. The proof states the defect class, the place where the defect
can occur, the material consequence, and the review contribution.
The orchestrator judges and records each proof. A sufficient NAMED proof makes
the area **proved**. An insufficient NAMED line becomes **SKIPPED**.
Only a proved area adds focus-dependent
gates or area reviewers. A track with one or more proved areas gets exactly
one Reviewer I and every required area specialist. The implementer's approximate
size above 100 counted lines also requires Reviewer I on a track that is not
documentation-only. Without either trigger, routine implementation review is
`NOT REQUIRED`. Size alone adds no area reviewer or blocking track acceptance.
[track-workflow.md](track-workflow.md) § Track size and split defines counted
lines and the separate pre-implementation design estimate. The user judges
whether the proposal is the simplest solution. No script or other rule decides
whether a proof holds.

Each track must be one coherent unit that a human can review in one sitting.
The orchestrator owns the split and records its rationale.
A code track implements bounded approved work, including document changes.
A level contains the sibling tracks created by one split.
Levels group planning only. The change is the merge unit.
All checks pass at each child marker.
At change end, documents, prompt guidance, and the extension must agree.
Intermediate packages and review intentions list each remaining difference and its later owning track.
Do not merge an incomplete change.
[track-workflow.md](track-workflow.md) § Recursive planning and loading owns split
limits, the implementer-requested exception, and the recursive-document loading condition.

## Focus areas and their gates

A focus area is one named risk. An area adds its reviewer only when the orchestrator judges its NAMED proof sufficient.

<!-- focus-area-table:begin -->
| # | focus area | the gate it adds | where the gate runs |
| --- | --- | --- | --- |
| 1 | concurrency defect | one area reviewer for concurrency | every track that proves the area |
| 2 | data loss | one area reviewer for data loss and recovery | every track that proves the area |
| 3 | security weakness | one area reviewer for security | every track that proves the area |
| 4 | performance degradation | one area reviewer for performance | every track that proves the area |
| 5 | test-quality defect | one test-quality and structure reviewer | every track that proves the area |
| 6 | unreadable user-facing prose | one prose reviewer | every track that proves the area |
| 7 | licensing exposure | one licensing reviewer | every track that proves the area |
| 8 | non-local logic defect | one area reviewer for non-local logic defects | every track that proves the area |
| 9 | consumer contract break | one area reviewer for consumer contract breaks | every track that proves the area |
| 10 | governing-rule defect | one area reviewer for governing-rule defects | every track that proves the area |
| 11 | unreported failure | one area reviewer for unreported failures | every track that proves the area |
<!-- focus-area-table:end -->

The marked table is the canonical copy of the duplicated eleven-area table. Its
second copy is in [track-workflow.md](track-workflow.md). The two marked blocks
must remain equal. The canonical class list is in
[track-workflow.md](track-workflow.md) § Focus classes and gates. It defines
which areas are DESIGN-TRIGGERING and which are REVIEWER-ONLY.

Duplicated doctrine blocks follow the reviewer-charter marker convention. An
HTML begin comment immediately precedes each block. Its matching end comment
immediately follows it. This table uses the block name `focus-area-table` in
both documents.

### Concurrency defect

**Outcome:** a result that the specification forbids, or a stop of required
progress, produced by an allowed overlap or order of two or more executions.
**Trigger:** the area engages when the change can cause that outcome. A change
does not engage the area only because it runs inside a concurrent program.
Slower execution that still makes progress belongs to performance degradation.

### Data loss

**Outcome:** data that the project keeps for a user, a consumer or a later
session, and that cannot be recovered, or recovered data that differs from the
state the project kept. **Trigger:** the area engages when the change can cause
that outcome. Loss that the published contract permits does not engage the
area. A change that widens the permitted loss does engage it. Data that the
project may rebuild or discard without a consumer noticing does not engage the
area.

### Security weakness

**Outcome:** a condition that an actor can exploit or trigger to break
confidentiality, integrity, availability, authentication, authorization or
accountability for the project or its consumers. **Trigger:** the area engages
when the change can introduce or expose that condition and a credible actor can
reach it. Exploitation need not happen. A change to a control whose purpose is
to protect one of those six properties engages the area, whether or not an
actor can reach the condition today. A configuration key engages the area only
when the key controls the run-time security posture of a consumer of the
published package.

### Performance degradation

**Outcome:** latency, throughput or resource use that misses a stated
requirement, or a growth of work with input size or data size that moves to a
worse class than the current code has. **Trigger:** the area engages when the
change can cause that outcome. A stated requirement is a limit that project
rules, the approved design, a benchmark threshold, a timeout, a budget or a
pinned size figure states for the touched path. It includes a path that project
rules mark as hot or as running on every turn. A change that alters no work per
unit of input engages the area on such a path only through a stated size budget.

### Test-quality defect

<!-- test-quality-definition:begin -->
**Outcome:** a project test or check that gives an unreliable signal, or that
fails to detect a fault inside the behaviour it claims to protect. **Trigger:**
the area engages when the change can cause that outcome. A changed coverage
number or coverage denominator alone is not engagement. A product artifact
does not engage the area only because a test calls it.
<!-- test-quality-definition:end -->

### Unreadable user-facing prose

**Outcome:** text that leaves its intended reader unable to make the decision
or complete the task that the text supports, where that text is readable by a
consumer of the published package or governed by the project writing
convention. **Trigger:** the area engages when the change can cause that
outcome. A short internal comment, a test name, a mechanical label, and text
that the project excludes from its writing convention do not engage the area.
A readability score alone neither engages nor clears the area.

### Licensing exposure

**Outcome:** published material that the project has no permission to publish
under its own licence. **Trigger:** the area engages when the change can cause
that outcome through copied or adapted material from an identifiable external
work, a dependency or notice, or a trademark. Copying or adapting such
external material always engages the area. A dependency with a clear
permission basis and satisfied conditions does not engage the area. Material
that an author or a worker wrote for this project without an external source
does not engage the area. Unknown provenance alone does not engage it.

### Non-local logic defect

**Outcome:** a result that the specification forbids, or a required result that
never appears, because two or more places do not satisfy one shared relation. A
place is one application of that relation: a rule check, state transition,
condition interpretation, or representation read or write. The application and
the facts it holds define its boundary. A fact held by another application stays
outside that boundary, even when the first application reads or needs that fact.
Reading an external fact does not make the consuming application independently
checkable.

Moving equivalent applications next to each other, into one module,
or into separate files does not merge or split them. The reviewed change is a
separate boundary. Omitting a required matching edit from that change can alter
the area decision. An agreement is the shared relation that each place must
preserve. **Trigger:** the area engages when the change can cause
that outcome. The reader must be unable to
settle whether the relation remains satisfied by reading each changed place on
its own. The evidence for that risk must include at least one of these four
kinds:

1. A rule that two or more places must apply in the same way.
2. A state or a history that an earlier execution left behind.
3. Two or more conditions that must hold at the same time, where at least one
   condition takes its value or its meaning from a place the change does not
   show.
4. A matching edit that is required in a place the change does not touch.

One added condition whose two outcomes a reader checks separately leaves every
place independent and does not engage the area. Two conditions joined in one
expression do not engage the area when the same place shows the value and the
meaning of each condition. The number of branches, the number of changed lines,
the number of changed files and any complexity score neither engage nor clear
the area. A repeated edit that keeps behaviour the same does not engage the
area when every edited site appears in the change and a reader checks one site
at a time. A change confined to text that no execution reads does not engage
the area. Evidence that one lookup settles does not engage the area, because
the area needs two or more facts that a reader must compare.

A persisted-state write and its later read are separate applications in every
layout. A co-located representation write and read are also separate when both
must preserve one relation. An enforced constraint and a published copy held by
another application remain separate, even when one reads the other. A local
guard remains one independent application when it shows its value, meaning and
both outcomes. A complete mechanical rename remains excluded when every site is
shown and can be checked alone. An incomplete rename can engage when the missing
site owes a matching edit.

A proof for this area keeps the standard four parts. The defect class is the
shared relation that can fail. The standard place field lists the changed place
and every other place that must satisfy that relation. The consequence is the
forbidden or missing result. The review contribution names what the area
reviewer can trace between those places, the evidence it reads there, and why
the planned checks cannot settle the relation.

- **Concurrency defect.** An unsatisfied agreement that appears only because two
  or more executions may overlap or may run in another order belongs to
  concurrency defect. Non-local logic defect covers an unsatisfied agreement
  inside one execution. Both areas engage only when the change can leave that
  agreement unsatisfied and the overlap or order can independently produce the
  concurrency outcome.
- **Data loss.** A forbidden result that destroys kept data, or that returns
  kept data in a changed form, belongs to data loss. Non-local logic defect
  covers an unsatisfied agreement whose result may be wrong without any data
  being lost. Both areas engage only when the change can leave that agreement
  unsatisfied and can independently destroy or alter kept data.
- **Security weakness.** A condition that a credible actor can reach and
  exploit against confidentiality, integrity, availability, authentication,
  authorization or accountability belongs to security weakness. Non-local
  logic defect covers a wrong result that needs no actor. Both areas engage
  only when the non-local logic trigger holds and the security trigger also
  holds for a protective control or a condition a credible actor can reach.
- **Performance degradation.** A correct result that misses a stated latency,
  throughput, resource or growth requirement belongs to performance
  degradation. Non-local logic defect requires a wrong or an absent result.
  Both areas engage only when the change can leave an agreement unsatisfied and
  can independently miss the stated performance requirement or worsen the
  growth class.
- **Test-quality defect.** A check that gives an unreliable signal, or that
  cannot detect a fault inside the behaviour it claims to protect, belongs to
  test-quality defect. Non-local logic defect covers the product behaviour and
  never engages because a test is missing. Both areas engage only when the
  change can leave an agreement unsatisfied. The test-quality trigger must also
  hold because the change makes a check unreliable or unable to detect the
  fault it claims to protect against.
- **Unreadable user-facing prose.** Text that leaves its reader unable to decide
  or to act belongs to unreadable user-facing prose. Non-local logic defect
  covers behaviour that an execution produces. Both areas engage only when the
  change can leave an agreement unsatisfied and can independently leave the
  reader unable to decide or act.
- **Licensing exposure.** Published material that the project has no permission
  to publish belongs to licensing exposure. Non-local logic defect never
  engages because material came from outside. Both areas engage only when the
  change can leave an agreement unsatisfied and also copies or adapts material
  from an identifiable external work.

### Consumer contract break


A consumer contract break changes a surface that an unchanged consumer reaches.

1. Does the change alter a consumer-reachable surface?
2. Would a consumer that does not change, using that surface in a way the review base permits, get a different result, a different exit status, a different output shape, an error, or data that it cannot read in the candidate?
3. Does the change show no route that keeps the base use working?
4. Does the change publish a new consumer-reachable surface without stating which parts of it a consumer may rely on?

The first three answers must all be yes, or the fourth answer can be yes on its own. The review base is the base endpoint of the declared review range. The candidate is the candidate endpoint of that range. Compare those two snapshots, including unreleased code. A version number, release label, changelog or publication state does not override the declared endpoints. A consumer-reachable surface is a name that a published entry point exports, an argument or option of a shipped command, an exit status of a shipped command, the machine-readable output of a shipped command, a configuration key together with the value used when it is absent, a record or file that the project writes and later reads, or a shipped statement about what the project accepts or produces. An internal name, a moved file or a helper that no published entry point exposes does not trigger the area. An addition that leaves every permitted base use unchanged does not trigger it. Human-readable wording, layout and log text do not trigger it. A file that the project may discard or rebuild without a consumer noticing does not trigger it. Version numbers, release labels, changelogs and counts neither trigger nor clear it. A defect correction triggers it when an unchanged consumer's result changes, even when the base result contradicted the published document. A surface introduced in the candidate, which no consumer can reach from the review base, triggers it only through the fourth question.

A proof for this area keeps the standard four parts. The defect class is the kind of break: a withdrawn name, a changed default, a changed exit status, a changed output shape, a narrowed input, or a format that the base reader cannot read in the candidate. The place names the consumer-reachable surface and the compared revisions in the declared review range, together with the concrete export, option, key, exit status or record. The consequence is what the unchanged consumer experiences in the candidate: a failed run, a silently different result, or data that it can no longer read. The review contribution names what the area reviewer can compare across the declared review range, the surface evidence it reads, and why the planned checks cannot show the unchanged consumer's experience.

#### Boundaries

Each pairing below engages both areas only when each area independently meets its own trigger.

- **Concurrency defect.** A forbidden result that appears only because two or more executions may overlap or may run in another order belongs to concurrency defect. Consumer contract break covers a result that changes for an unchanged consumer in a single ordinary execution. Both areas engage when the change alters a consumer-reachable surface and also allows a new overlap.
- **Data loss.** Kept data that cannot be recovered, or that comes back different, belongs to data loss. Consumer contract break covers a format, a default or a name that an outside party can no longer use as before, even when every byte survives. Both areas engage when a format change both withdraws the old reader and destroys the only copy.
- **Security weakness.** A condition that a credible actor can reach and exploit against confidentiality, integrity, availability, authentication, authorization or accountability belongs to security weakness. Consumer contract break needs no actor and no exploit, only an unchanged consumer. Both areas engage when the changed default or key controls the run-time security posture of a consumer.
- **Performance degradation.** A correct result that misses a stated latency, throughput, resource or growth requirement belongs to performance degradation. Consumer contract break needs a different result, a different status, a different shape or unreadable data, and speed alone never engages it. Both areas engage when a changed default also moves the work per unit of input on a path with a stated requirement.
- **Test-quality defect.** A check that gives an unreliable signal, or that cannot detect a fault inside the behaviour it claims to protect, belongs to test-quality defect. Consumer contract break covers only surfaces that a consumer reaches, and test material is not one of them. Both areas engage when the change alters a shipped command that a consumer runs and also alters the check that would catch the break.
- **Unreadable user-facing prose.** Text that leaves its reader unable to decide or to act belongs to unreadable user-facing prose. Consumer contract break covers a shipped document only when the document states what the project accepts or produces, and it never engages on wording, layout or log text. Both areas engage when the change alters an accepted input and also rewrites the sentence that states it.
- **Licensing exposure.** Published material that the project has no permission to publish belongs to licensing exposure. Consumer contract break never engages because material came from outside. Both areas engage when adapted external material arrives together with a changed consumer-reachable surface.
- **Non-local logic defect.** A wrong or missing result that follows from a broken agreement between two or more places inside the system belongs to non-local logic defect. Consumer contract break covers a result that a party outside the change observes as different, and it engages even when every place inside the change agrees. Both areas engage when a broken internal agreement is what produces the changed external result.

### Governing-rule defect


A governing-rule defect makes a rule for project work unusable or inconsistent.

1. Does another rule document, or another copy of the same rule, now state something different for one case?
2. Can a reader reach the end of the governed work without performing a required step and without recording a decision to skip it?
3. Does the rule now require a term, a threshold or a name that the change leaves undefined for the reader who must apply it?
4. Does an automated check, a gate or a script now permit work that the rule forbids, or forbid work that the rule permits?
5. Can a required step become unreachable, or run after the work is declared complete?

The change must add, alter or remove a rule for people or agents who produce, review, verify, publish or release work, or alter the machinery that enforces it. A product contract is outside this trigger. A change that only obeys an existing rule does not trigger the area. Counts and readability or coverage scores neither trigger nor clear it. A wording change that leaves every obligation the same does not trigger it. A record of a past rule that no reader must follow today does not trigger it.

A proof for this area keeps the standard four parts. The defect class is which of the five questions answers yes. The place is the changed rule together with the other rule, copy or enforcing check that must agree with it. The consequence is the work that then proceeds without its check, or the two conflicting ways in which two readers act. The review contribution names what the area reviewer can compare between the rule, its other copies and its enforcing check, and why the planned checks cannot show that disagreement.

---

#### Boundaries

Each pairing below engages both areas only when each area independently meets its own trigger.

- **Concurrency defect.** A forbidden result or a stop of progress that appears because two or more executions may overlap belongs to concurrency defect. Governing-rule defect covers a rule that people and agents follow, and it never engages because an interleaving exists. Both areas engage when a changed rule states how a program must serialise work and the change also alters that serialisation in code.
- **Data loss.** Unrecoverable or altered kept data belongs to data loss. Governing-rule defect covers work that proceeded without its check, even when no byte was lost. Both areas engage when the skipped step is the step that protects kept data.
- **Security weakness.** A condition that a credible actor can reach and exploit against one of the six named properties belongs to security weakness. Governing-rule defect needs no actor, because the harm is that the project itself does the work wrongly. Both areas engage when the weakened gate is a protective control, for example a rule that keeps a credential out of a child process.
- **Performance degradation.** Latency, throughput, resource use or growth that misses a stated requirement belongs to performance degradation. Governing-rule defect covers the rule that STATES such a requirement, and not the measured behaviour. Both areas engage when a change alters a size budget or a hot-path rule and also changes the work on that path.
- **Test-quality defect, sentence one.** A project test or check that gives an unreliable signal, or that fails to detect a fault inside the behaviour it claims to protect, belongs to test-quality defect, and that area judges the check against the PRODUCT behaviour under it. **Sentence two.** Governing-rule defect judges the same check against the RULE above it, so it engages when the check and the rule now permit different work, and when a required gate becomes passable with no decision, even though the check still detects every fault it ever detected. Both areas engage when one change edits a gate that both enforces a project rule and detects a product fault, and each reviewer then files a different finding about it.
- **Unreadable user-facing prose, sentence one.** Text that leaves its intended reader unable to make a decision or complete a task belongs to unreadable user-facing prose, and that area owns comprehension, audience, terminology, structure and the writing convention. **Sentence two.** Governing-rule defect covers a rule that a reader understands perfectly and still cannot apply, cannot satisfy, can satisfy in two conflicting ways, or can pass without a decision, so a rewording that changes only how easily a rule reads engages prose alone, while a rewording that changes what a reader must do engages this area. Both areas engage when a rule document is rewritten and its obligations change, and the prose code reviewer then judges the reading while this code reviewer judges the obligation.
- **Licensing exposure.** Published material that the project has no permission to publish belongs to licensing exposure. Governing-rule defect never engages because material came from outside. Both areas engage when a change adapts an external standard, a checklist or a policy template into a project rule.
- **Non-local logic defect.** A wrong or an absent result that no single changed place settles, because two or more places must agree, belongs to non-local logic defect, and its places are locations that an execution reads. Governing-rule defect covers a rule that a person or an agent must obey, and its consequence is work done wrongly rather than a wrong result. Both areas engage on a change that edits a rule, its duplicated copy and the check that pins the text.

### Unreported failure


An unreported failure leaves a product failure with no signal.

1. Does the change introduce, move or widen a place where the product can fail, refuse, drop, skip, partly complete or fall back?
2. Does the change leave at least one such failure with no signal that the change itself shows?

A signal is one observable event, for example a non-zero exit status, a message on the error stream, a rejected input with a stated reason, a failing check, a recorded event or an error handed to a caller. A dropped entry with no report, an error that is caught and discarded, a return status that no caller reads, a write that nothing verifies and a fallback that replaces a failure with a normal-looking result each trigger the area. A change that adds no new way to fail does not trigger it. A failure that reaches a reporter the change keeps and shows does not trigger it. A wrong value from an execution that met no failure does not trigger it. A change whose only affected artifact is a project test or check belongs to test-quality defect and does not trigger this area. A removed signal triggers the area unless the change shows that the failure it reported can no longer happen. Counts do not decide the result.

A proof for this area keeps the standard four parts. The defect class is the failure mode that carries no signal. The place is the failure site together with the boundary that owes the report. The consequence is what proceeds, spreads or completes as an apparent success while the failure stays unknown. The review contribution names the failure path the area reviewer can drive or inspect, the signal evidence it looks for, and why the planned checks cannot show the missing report.

#### Boundaries

Areas may engage together when each area meets its own trigger. The unreported failure reviewer judges whether a product failure has a signal. That reviewer does not judge the outcome owned by another area. A silently swallowed write failure that changes a shipped command's exit status can engage both unreported failure and consumer contract break when both triggers hold. Unreported failure never grades a project test, which belongs to test-quality defect. It never grades the wording of a message, which belongs to unreadable user-facing prose. A wrong value from an execution that met no failure does not engage unreported failure. It engages non-local logic defect only when that area's cross-application relation trigger holds. A purely local wrong value can therefore engage neither area.

### Judged proof and risk record

The orchestrator records one line for every area. A named line gives a concrete
four-part proof. A line that is not NAMED states which part of the trigger
answers no. A NAMED line becomes proved when the orchestrator judges its proof sufficient.
An insufficient NAMED line becomes SKIPPED and adds no gate or reviewer.

Each change and track keeps its own assessment and risk record.
Shared evidence does not merge the records or replace either assessment.

Every NAMED proof has these four parts:

1. **Defect class.** State the kind of defect that the area covers.
2. **Place.** State the place in the planned change where it can occur.
3. **Material consequence.** State the consequence of omitting the area
   reviewer.
4. **Review contribution.** Identify a concrete review action that can
   materially reduce the stated risk. Name the evidence the specialized reviewer
   can examine or obtain. Explain how that action could expose the defect or a
   missing safeguard before acceptance. Explain why ordinary implementation and
   its planned checks are insufficient for this risk.

Planned checks means the implementation validation and test commands. It
excludes every separate reviewer, including Reviewer I. The review contribution
promises no guaranteed detection. It makes no comparison with another reviewer.
A short method and a configuration change receive no automatic exemption.

The proof basis is the approved track design. When the track has no design, the
basis is the track intention block and planned file list. The risk record is
orchestrator material, not high-level design. It may cite concrete paths,
symbols, and checks.

A proof must show a material consequence of omitting the area reviewer. A
consequence is material when a reasonable reviewer would file it at major
severity or above. Project tooling counts. A protective-control change is
material when loss of that control would be material. A cosmetic consequence
is not material. A proof is not convincing when its words would also fit a
change that does not engage the area.

No rule mechanically decides whether a proof holds. The orchestrator judges every
proof at planning and focus reconfirmation. A SKIPPED area gets
no gate or reviewer. The skip is recorded and is not an escalation. A track with
no proved area gets no area reviewer.

Reviewer composition and merging belong to
[review-rules.md](review-rules.md) § Reviewer sets, merge rule and charters.

## Optional path declarations

A project may declare paths for a focus area as prose in its contributor guide.
No configuration key or parser is required. A declaration is optional evidence
for the risk record.

1. A touched declared path is evidence that its area may engage.
2. A change outside all declared paths still receives orchestrator judgement.
3. A declaration may support a proof or a non-engagement line.
4. A narrow or stale declaration does not decide engagement or proof.

A path is evidence, not a definition or a decision.

## Lifecycle rules owned by the spine

[track-workflow.md](track-workflow.md) § Risk planning and reconciliation owns
planning, proof judgment, focus reconfirmation, the late-area route, and the
committed-difference comparison. This document does not duplicate those rules.

[review-rules.md](review-rules.md) § Reviewer sets, merge rule and charters owns
Reviewer I and all area-reviewer composition.

## Halt and focus re-derivation

An implementer halts for an unplanned sensitive configuration change, before
weakening a test, before materially deviating from the approved design, or when
it finds an area the plan did not name. The halt occurs before further work
builds on the new fact.

On a halt, the orchestrator:

1. pauses implementation.
2. re-derives all eleven focus lines for the remaining work.
3. records each addition or removal with its proof or failed trigger part.
4. judges each new NAMED proof and removal, records the resulting gate changes,
   and honors any user direction without silently waiving a separate gate.
5. runs the required design gate for remaining work when a newly proved
   DESIGN-TRIGGERING area requires it, unless the user records a decision to
   skip that gate.
6. resumes only after every blocking gate passes.

The route never applies a design gate retrospectively to completed work. It
keeps the recorded skip and every reviewer that already covered completed work.
A late proved area recomputes the complete required routine reviewer set for the
completed range. If Reviewer I has not covered that range, dispatch Reviewer I
and the new area specialist. If Reviewer I already ran because of size or
another proved area, dispatch only the new specialist. Do not dispatch Reviewer I
again. Every newly required perspective reviews the completed range. The owning
research log reports the resulting coverage.
Disclose relevant concerns under the package rules.

## Review coverage and the coverage register

Every part of a track range must reach the complete routine implementation
reviewer set defined in [track-workflow.md](track-workflow.md) § Review coverage.
A proved area requires exactly one Reviewer I and every required area
specialist. Without a proved area, a reported size above 100 counted lines
requires Reviewer I unless the track is documentation-only. Otherwise the set
is empty. Required user acceptance never replaces required machine review.
This requirement is the coverage invariant.

Create a coverage register when a user-review fix commit exists. Record each
contiguous user-review fix range and its gate verdict. The register remains the
review-accounting authority for those ranges. The track table remains a
display-only split index and carries no commit identifier.

At delivery, a live register produces the separate user-requested-fix
verification verdict required by [track-workflow.md](track-workflow.md). The
routine implementation-review conclusion reports `NOT REQUIRED` when the
required reviewer set is empty. The detailed register stays in the research
log.

## Commit discipline for drift and boundaries

[track-workflow.md](track-workflow.md) § Commit titles and bodies owns change-title metadata, markers, title prefixes, and boundaries.
Its first-line identity rule covers bootstrap, implementation, agentic-review fix, cumulative implementation, user-review fix, gate correction, code and design marker, post-ready fix, and base merge commits.
It adds no content part.

With publishing disabled, accepted branch history is the delivery.
Follow [track-workflow.md](track-workflow.md) § Delivery history and accounting.

The cumulative commit follows [track-workflow.md](track-workflow.md) § Lifecycle and phases.
When the track has a high-level design, the cumulative implementation commit body carries the track high-level design followed by the track low-level design.
Otherwise, use the two-part body.
See also its § Delivery and termination.

Original implementation, agentic-review fix, and user-review fix bodies have exactly two content parts after the identity line:

1. **Intent:** what the commit accomplishes, not how.
2. **Deviation delta:** only differences from the original track task. Leave it empty when none differ.

Allow only deviations sanctioned by halt and re-confirmation, design-delta approval, or immaterial design change.
Material deviations at commit time fire a halt.
Bodies never authorize drift.
Two-part bodies exclude implementation rationale, self-assessment, and verification claims such as `tested`, `verified thread-safe`, and `reviewed edge cases`.
Fresh reviewers receive state and intent, not implementer confidence.

Every commit in a multi-track review range belongs to that track.
Check against the brief, marker boundaries, and title prefixes.
Another track's commit is cross-track contamination.
Correct wrong boundaries or titles before the next writer starts.
Undeclared material deviations fire the halt and re-derivation route above.
