Review Checklist¶
Review bijux-gnss-receiver as the runtime owner for staged receiver
execution. The crate may orchestrate acquisition, tracking, observation
assembly, in-memory artifacts, diagnostics, validation hooks, and simulation
support. It should not own command presentation, repository persistence,
signal truth, or navigation science.
flowchart LR
change["runtime change"]
stage["name stage family"]
artifact["confirm emitted artifact meaning"]
validation["inspect validation and synthetic proof"]
boundary["route lower-owner science back out"]
change --> stage --> artifact --> validation --> boundary
Review Gates¶
| changed surface | accept only when | inspect before accepting |
|---|---|---|
| acquisition, tracking, or observation stage | The stage contract and lock/error evidence remain understandable outside one test fixture. | Stage contracts and pipeline guide |
| runtime artifact or diagnostic output | The record describes receiver runtime meaning before infra persists it. | Artifact Contracts, Diagnostic Contracts |
| validation or simulation helper | Synthetic proof is bounded to receiver behavior and does not become a substitute truth system. | Validation and simulation contracts and reference validation guide |
| public export | The export is a durable receiver boundary, not a shortcut to signal, nav, or infra internals. | API surface, public API, and guardrail proof |
| deterministic or performance-sensitive path | The proof covers reproducibility, runtime budget, and artifact stability together. | Determinism and purity, validation budgets, and pipeline determinism proof |
Blocking Signs¶
- A runtime change passes because a fixture was softened rather than because receiver behavior is now better explained.
- A receiver artifact carries repository-storage meaning that belongs to
bijux-gnss-infra. - A tracking or acquisition helper redefines signal facts that should come from
bijux-gnss-signal. - A validation test asserts a final navigation conclusion without checking the receiver evidence that made the conclusion possible.
Evidence To Require¶
- Read the receiver test guide, pipeline guide, and public API before accepting broad runtime changes.
- Require the narrow stage test family for the changed behavior before relying on a full pipeline test.
- Update the matching stage, runtime, artifact, diagnostic, or validation handbook page when public receiver meaning changes.
- Route command, infra, signal, and nav concerns to their owners even when the receiver happens to be the caller.