Risk pages: view-mode default, de-ceremonied acceptance, register calibration#51
Open
CST-100 wants to merge 1 commit into
Open
Risk pages: view-mode default, de-ceremonied acceptance, register calibration#51CST-100 wants to merge 1 commit into
CST-100 wants to merge 1 commit into
Conversation
…ibration The risk page opens read-only — scenario fields, scores, owner, narrative, and links render as facts; EDIT switches to the in-place editors (lint overlays, selects, link/spawn controls, disposition transitions) and DONE returns to view, mirroring the issue page. The ACCEPTANCE READINESS checklist box is gone: one ACCEPT button, grayed until every hard check passes, with the tooltip naming what is missing (readiness stays the single source for the gate). The inline accept-confirm keeps the signature consequence. Once signed, the facts live in the rail (ACCEPTED · RATIONALE) — the panel's duplicate signature block is deleted. Register rows drop the per-row disposition badge: REALIZED keeps warning weight, CLOSED renders muted, everything else calm plain text. Linked-issue rows reuse the ok.issue_state register and plain mono roles. Co-Authored-By: Claude Fable 5 <noreply@anthropic.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
1 participant
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.
Applies the issue-page UI calibration (PR #49 amendments) to the risk module.
/risks/{id}opens read-only — scenario fields, scores, owner, narrative, and linked issues render as facts. A header EDIT control switches to?edit=1with the in-place editors (lint overlays, selects, link/spawn/unlink controls, disposition transitions); DONE returns to view. Per-field saves on blur, unchanged.opal.risks.readinessremains the single source for the gate, so the button andaccept()cannot drift. The inline accept-confirm block keeps the signature consequence. Once signed, the facts move to the rail (ACCEPTED · RATIONALE); the panel's duplicate signature block is deleted (one fact, one home).ok.issue_stateregister.Deliberately untouched: the per-target disposition panel keeps its live status lines and an active SET button — its blockers (watch observable/threshold, residual score) are filled by the panel's own inputs without a re-render, so a statically grayed button would deadlock those flows.
Verified: full suite (701 passed), ruff clean, view/edit modes and the gated ACCEPT confirmed against a live instance.
🤖 Generated with Claude Code