-
Notifications
You must be signed in to change notification settings - Fork 0
[HT-70]: Analysis coach on replay — teach-from-evidence filter + verbosity settings #118
Copy link
Copy link
Open
Labels
⚙️ settingsGame setup, user-configurable options, preferences screens, or rules customization controls.Game setup, user-configurable options, preferences screens, or rules customization controls.✅ readyWell-scoped, unblocked, and ready for implementation or review work to begin.Well-scoped, unblocked, and ready for implementation or review work to begin.📚 docsRepository docs, in-app help, rules documentation, contributor docs, or issue/PR templates.Repository docs, in-app help, rules documentation, contributor docs, or issue/PR templates.🔹 p2Normal-priority work that matters but does not block the immediate critical path.Normal-priority work that matters but does not block the immediate critical path.🖥️ uiFlutter screens, navigation, table layout, controls, feedback states, or responsive UI.Flutter screens, navigation, table layout, controls, feedback states, or responsive UI.🙋 hitlRequires human-in-the-loop judgment for rules, naming, design, tone, or product behavior.Requires human-in-the-loop judgment for rules, naming, design, tone, or product behavior.🧠 aiCPU move selection, difficulty, personality, reaction timing, or strategy evaluation.CPU move selection, difficulty, personality, reaction timing, or strategy evaluation.🧩 featureA vertical slice that delivers user-visible behavior or playable product capability.A vertical slice that delivers user-visible behavior or playable product capability.
Description
Metadata
Metadata
Assignees
Labels
⚙️ settingsGame setup, user-configurable options, preferences screens, or rules customization controls.Game setup, user-configurable options, preferences screens, or rules customization controls.✅ readyWell-scoped, unblocked, and ready for implementation or review work to begin.Well-scoped, unblocked, and ready for implementation or review work to begin.📚 docsRepository docs, in-app help, rules documentation, contributor docs, or issue/PR templates.Repository docs, in-app help, rules documentation, contributor docs, or issue/PR templates.🔹 p2Normal-priority work that matters but does not block the immediate critical path.Normal-priority work that matters but does not block the immediate critical path.🖥️ uiFlutter screens, navigation, table layout, controls, feedback states, or responsive UI.Flutter screens, navigation, table layout, controls, feedback states, or responsive UI.🙋 hitlRequires human-in-the-loop judgment for rules, naming, design, tone, or product behavior.Requires human-in-the-loop judgment for rules, naming, design, tone, or product behavior.🧠 aiCPU move selection, difficulty, personality, reaction timing, or strategy evaluation.CPU move selection, difficulty, personality, reaction timing, or strategy evaluation.🧩 featureA vertical slice that delivers user-visible behavior or playable product capability.A vertical slice that delivers user-visible behavior or playable product capability.
Parent
PRD-05: Game History & Replay — #111
What to build
Add a full-context analysis coach to the replay viewer that explains what was happening, at rest and without a clock, so a player can understand nuances they missed under live pressure. It consumes the HT-69 signals (card-death, feed-risk, collecting tells) over the reconstructed match state.
The existing coaching advisor already reads full live state, so the new work is the inverse: a teach-from-evidence filter that constrains advice so it is justified only by table-observable evidence (discards taken, melds laid, claim timing) and never cites hidden-hand information — the "judge with full information, teach only from observable evidence" boundary. Capture this boundary and the review-observation contract in a new
docs/designcompanion document.The analysis coach is the only coach shown while viewing (the live coach never appears in viewing — they are mode-exclusive). Add a review/coach settings category with a persisted default for analysis-coach verbosity (from narrate-everything to flag-only-clear-mistakes) and card-death warnings, plus an on-the-fly override when opening a replay, mirroring the persisted-default-plus-per-session-override pattern used by game setup.
Acceptance criteria
docs/designcompanion documents the teach-from-evidence boundary and the review-observation contract.Blocked by