Skip to content

orchestrator: Rename signal/G to probe/P in boot checkpoints - #475

Merged
chrysh merged 1 commit into
OpenPRoT:mainfrom
9elements:rename-probe
Sep 21, 2026
Merged

chrysh merged 1 commit into
OpenPRoT:mainfrom
9elements:rename-probe

Conversation

@chrysh

@chrysh chrysh commented Sep 21, 2026

Copy link
Copy Markdown
Contributor

Summary

  • Renames the generic from G/signal to P/probe in BootCheckpoint,
    EvidenceReader, and the mock board's MockSignal to MockProbe.
    The old name implied GPIO-only, but real boards also poll MCTP readiness,
    progress registers and heartbeat latches.
  • Adds a GPIO wiring example to BootCheckpoint's doc comment. The example
    references CheckpointWalk, added in orchestrator: Add CheckpointWalk, the concrete BootWatch #471.

Pure rename, no behavioral change. 3 files touched.

The generic was called G ("signal") when it only described GPIO-ready
lines. Real boards also poll progress registers, MCTP readiness and
heartbeat latches, so the narrower name was misleading. P ("probe")
covers every kind of health indicator the evidence reader can sample.

Renames: BootCheckpoint<G> to <P>, signal field/accessor to probe,
EvidenceReader<G> to <P>, fn read(signal:) to fn read(probe:),
MockSignal to MockProbe, and all doc references. Adds a GPIO wiring
example to BootCheckpoint's doc comment.

Assisted-by: Claude
@chrysh
chrysh marked this pull request as draft September 21, 2026 15:51
@chrysh
chrysh marked this pull request as ready for review September 21, 2026 15:56
@mynetz
mynetz self-requested a review September 21, 2026 16:01

@mynetz mynetz left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Change looks consistent. Okay.

@chrysh
chrysh merged commit d3d4a71 into OpenPRoT:main Sep 21, 2026
5 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants