feat: add conversation visibility model#273
Merged
Merged
Conversation
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.
Linked Issue
Closes #244
Scope
Implemented:
conversation_visibilitymodel for public transcript, internal delegation, side-rail summary, and debug/log routing.@hostmessages public by default.ConversationSnapshotfor side-rail/Xray visibility without public transcript pollution.Not implemented:
Acceptance Criteria
@hostresponses default to public transcript.@hostmay surface role output publicly only for critical risk, veto, confirmation, or final evidence summary.Changed Files
src/conversation_visibility.rssrc/console_snapshot.rssrc/lib.rstests/conversation_visibility_test.rstests/fixtures/conversation_visibility.tomltests/fixtures/console_snapshot_v08.tomlValidation
Local commands executed:
Results:
Evidence Packet
WO-0244Tracker Update
Risk
Low. This is additive, but
ConversationSnapshotnow rejects internal/debug turns insidepublicTurns, so future fixtures must classify visibility correctly.Rollback
Revert this PR. No migration or runtime behavior change is included.