Skip to content

feat: add post-game coach - #44

Open
avatarneil wants to merge 3 commits into
mainfrom
neil/post-game-coach
Open

avatarneil wants to merge 3 commits into
mainfrom
neil/post-game-coach

Conversation

@avatarneil

Copy link
Copy Markdown
Owner

Summary

The AI game-over moment now gives the learner a compact coaching takeaway when the completed match has a meaningful missed decision. The coach identifies the strongest turning point, shows the chosen move versus the recommended move, and lets the learner inspect or retry that exact one-move position without disturbing the finished game.

This keeps the first version focused on the highest-attention learning moment: one practical lesson at game over, with graceful fallback to the existing simple result when analysis cannot produce a useful coach brief.

Design Notes

  • Post-game analysis replays the completed move history from a fresh initial state so normal turn flow and saved game semantics stay unchanged.
  • History entries can now preserve optional coach metadata, including the pre-move state needed to reopen the review locally.
  • Retry mode is intentionally a one-move exercise in the play view; it reveals the coach move after the learner chooses and then returns cleanly to the completed board.

Verification

  • npm run lint passes with existing warnings in unrelated AI/remotion files.
  • npx tsc --noEmit passes when a temporary local declaration shim is used for the missing generated wasm/pkg/knucklebones_ai module.
  • npm run build is blocked locally because Homebrew Rust is missing the wasm32-unknown-unknown target needed by wasm-pack.

Compound Engineering
GPT-5

@vercel

vercel Bot commented May 17, 2026

Copy link
Copy Markdown
Contributor

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
knucklebones Ready Ready Preview, Comment May 18, 2026 1:37am

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant