Repository navigation
Let configuration answers own model/context status labels - #987
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.
Change
Required ContextReading presentation, paired with Toad #693. Supersedes the rejected runtime status-label and nullable wire_fields designs.
Evidence and acceptance
Current owner NRA source enumeration and semantic diff: core-change-report/.artifacts/sidebar-observation/task-b-{before,after}.json/.log and task-b-toad.diff. Zero parse omissions; dynamic resolution remains a stated limitation. The previous owner's ratchet result applies to its preceding checkpoint, not this head.
Final Core head
2fb445b94fc22ed58a365709f4a873ca53e4a303, paired Toada64bca902416fd7881d9676b79040f134ad513fd. Both existing production debt checks pass without positive deltas, whitespace checks pass, and the exact manifest/lock passes uv's offline check. Focused Core ACP codec checks: 2 passed in 0.15s. The merged TranscriptDraftUpdate constructor census and strict codec validation exception contract were updated in the existing Core test; earlier failed results are retained. No Toad tests were run.Actual plain-st/Xvfb :229 saved-session application check uses installed runtime 184023Z plus both source overlays: saved history, DM history/activity/prompt/model-only status, channel history without Active, real unsent mention completion and Core-owned initial unmeasured text are visible. No crash file or ACP prompt submission. Fitted-window evidence supersedes the original cropped screenshots and mistaken blank-DM report; details remain in .artifacts/sidebar-observation/task-b-live/.
Shared-installed/native acceptance is NOT claimed: session/load against the older running Core 496609b owner refuses
Expected a string family kind. The rollout must align the reviewed native-owner protocol; no legacy decoder, uncertain-input replay, manual restart or publication was performed. Review and Codex paired publication remain required. Toad's four residual ContextUnavailable references are zero in its paired head; Core's ContextReading owns the unmeasured text.