feat(observability): wire Phase B causal audit boundaries - #6574
Draft
RuroninMugen wants to merge 5 commits into
Draft
feat(observability): wire Phase B causal audit boundaries#6574RuroninMugen wants to merge 5 commits into
RuroninMugen wants to merge 5 commits into
Conversation
Signed-off-by: rudy valace <rudyvalace@gmail.com>
Signed-off-by: rudy valace <rudyvalace@gmail.com>
Signed-off-by: rudy valace <rudyvalace@gmail.com>
Signed-off-by: rudy valace <rudyvalace@gmail.com>
Signed-off-by: rudy valace <rudyvalace@gmail.com>
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.
Summary
Adds the production-only Phase B causal observability layer for
buzz-acp.This PR introduces a local, metadata-only, fail-open JSONL audit sink and wires seven production observation boundaries:
TransportPublishedTransportAcceptedEventReceivedFilterDecisionQueueDecisionWakeDecisionAcpSessionStartedSafety / behavior
try_sendprevents backpressure from affecting business logicValidation
Validated on current upstream
mainbefore push:rustup run 1.95.0 cargo check -p buzz-acp— PASSrustup run 1.95.0 cargo clippy -p buzz-acp --all-targets— exit 0, warnings onlyrustup run 1.95.0 cargo test -p buzz-acp— 810 passed, 0 failedAll five commits carry DCO
Signed-off-bytrailers.Scope / known limitations
This PR is intentionally production-only. The separate Phase 5B regression-test commit is not included here.
Known limitations:
ResponsePublishedis declared in the schema but not yet wired to a production emit site.RecipientEvidenceObservedis declared in the schema but not yet wired to a production emit site.TransportPublished/TransportAccepteddescribe internalbuzz-acpWebSocket transport observations; they are not proof of real outbound agent handoff/chat publication throughbuzz-cli.correlation_id_foris currently unused.FilterDecisionemitted-payload coverage and B7→B8 cross-record correlation remain follow-up test work.The original combined PR #6559 remains untouched; this PR replaces only its production portion with a clean, DCO-signed branch based on current upstream
main.