Repository navigation
feat(cli): explain event receipts and clarify stance guarantees - #92
Open
BitmapAsset wants to merge 4 commits into
Open
BitmapAsset wants to merge 4 commits into
BitmapAsset wants to merge 4 commits into
Conversation
The windows-latest runner checks out with core.autocrlf=true, so the fixture already had CRLF on disk and the test LF -> CRLF replace produced CR CR LF, which `explain` rightly reports as a header that is not as written. Normalise to LF first so the test exercises real CRLF on every platform. Verified locally: `cargo test -p probbit-cli --test explain` passes with the fixture on disk as LF and as CRLF (13/13 both ways).
This branch has not been deployed
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Operators can now run
probbit explain STRAND --event N [--json]to inspect logged inputs, before/after stance odds, binding habits, chain hashes and whether the event recomputes. It reuses authoritative replay/turn APIs, checks the prefix through N, and leaves existing engine semantics and outputs intact. The README and new SOUL-to-habits guide explain that stance habits do not guarantee model obedience.Stacked on #91 (chore/0.14.1-cleanup); this PR shows only the three usability commits.
Validation: all 13 new regression tests passed; existing stdout goldens, generated man page and command-table checks passed. CLI-binary Clippy and formatting of the new files passed. Full unfiltered workspace suite on a macOS host (
cargo test --workspace --no-fail-fast, 2026-11-05): 477 passed, 0 failed, 4 ignored, exit 0, including the socket-bind, process-priority, loopback-server and deadline tests that an earlier sandboxed run could not execute. Two pre-existing baseline issues are unchanged by this branch: workspacecargo fmt --checkfails on the same 93 files as before, and all-targets Clippy rejects an existing approximate-PI constant inprobbit-decide/src/amp.rs.