amaurycoudr/chess-project
2
0
Fork 0
Code Issues 5 Pull requests Projects Releases Packages 1 Activity Actions

Solve a rated Puzzle: served near the Puzzle rating, Moves checked by the server #443

New issue
Closed
opened 2026-10-16 08:20:02 +00:00 by root · 11 comments
root commented 2026-10-16 08:20:02 +00:00 (Migrated from gitlab.core-project.space)
Copy link

Parent

#440

What to build

The tracer bullet: a signed-in Player solves or fails a rated Puzzle, end to end.

  • Access. A Puzzles entry in the navigation. /puzzles needs an Account Session, with the same beforeLoad guard as the account page (#409), remembering the destination. A Guest is asked to sign in or sign up.
  • Serving. "Next" returns the Account's pending Attempt if there is one. Otherwise it picks a random Puzzle with no Attempt by this Account within ±100 of the Puzzle rating, widening in steps until one is found, and creates the pending Attempt in the same transaction. Randomness comes from a root-provided source, like the clock and minters (ADR 0044). No unseen Puzzle left is a contract error the page shows.
  • The board. The existing board and click resolver. It shows the Puzzle's FEN, plays the opponent's first Move after a short beat, is oriented with the Player's Color at the bottom, and lets only the Player's pieces move. It says whose Turn it is.
  • Server-checked Moves. The client never holds the line beyond the Moves played. A Move is correct if it is the next Player Move of the line, or if it delivers Checkmate (which solves the Puzzle). A correct Move gets the opponent's reply, or solves the Puzzle at the end of the line. Any other legal Move is wrong, and the right Move is shown. Promotion works. Themes are not sent before resolution.
  • Rating. Solving or a wrong Move resolves the Attempt and applies the Glicko-2 update (#441) in one transaction. The page shows the Puzzle rating, and the change after the result. The provisional flag is sent (shown in a later ticket). A failed Attempt then shows the right Move and "Next"; retrying comes later.
  • Schema. The Puzzle rating row (one per Account; defaults read until the first Attempt) and the Attempt row (Account, Puzzle, state, Moves played, rating before, rating change, served at, resolved at, last replayed at). Unique on (Account, Puzzle), at most one pending per Account. Both are deleted with the Account.
  • Contract. A puzzle router with next, move and rating, Account-only, its errors in REFUSAL_STATUS. Moves over the wire as UCI (ADR 0021). Copy in English and French (ADR 0025).

Seams: S1 (the API through its RPC contract) and S3 (the app's router with the API fakes). Covers stories 1-17, 20-21, 27-31, 47-48 of #440.

Acceptance criteria

  • Signed out, /puzzles goes to sign-in and comes back after it; a signed-out RPC call is refused.
  • "Next" serves an unseen Puzzle within the window, widens when the band is empty, and never serves a Puzzle this Account already has an Attempt for.
  • A reload or a second "Next" returns the same pending Attempt, at the Moves already played.
  • The opponent's reply follows a correct Move; the end of the line solves; any Checkmate solves; any other legal Move fails and returns the right Move; an illegal Move is refused without failing.
  • A Move on a resolved Attempt is refused.
  • Solving raises and failing lowers the Puzzle rating by the module's amount; the result carries the change.
  • Two concurrent Moves on one Attempt resolve it once.
  • Deleting the Account deletes its Puzzle rating and Attempts.
  • Front view tests: the first Move plays, the orientation, only own pieces movable, solved and failed states, the rating change, "Next".
  • Prior art followed: the rpc-games and rpc-clock integration tests, router.test.tsx.
  • The gate is green (CONTRIBUTING.md, "The five commands").
## Parent #440 ## What to build The tracer bullet: a signed-in Player solves or fails a rated Puzzle, end to end. - **Access.** A Puzzles entry in the navigation. `/puzzles` needs an Account Session, with the same `beforeLoad` guard as the account page (#409), remembering the destination. A Guest is asked to sign in or sign up. - **Serving.** "Next" returns the Account's pending Attempt if there is one. Otherwise it picks a random Puzzle with no Attempt by this Account within ±100 of the Puzzle rating, widening in steps until one is found, and creates the pending Attempt in the same transaction. Randomness comes from a root-provided source, like the clock and minters (ADR 0044). No unseen Puzzle left is a contract error the page shows. - **The board.** The existing board and click resolver. It shows the Puzzle's FEN, plays the opponent's first Move after a short beat, is oriented with the Player's Color at the bottom, and lets only the Player's pieces move. It says whose Turn it is. - **Server-checked Moves.** The client never holds the line beyond the Moves played. A Move is correct if it is the next Player Move of the line, or if it delivers Checkmate (which solves the Puzzle). A correct Move gets the opponent's reply, or solves the Puzzle at the end of the line. Any other legal Move is wrong, and the right Move is shown. Promotion works. Themes are not sent before resolution. - **Rating.** Solving or a wrong Move resolves the Attempt and applies the Glicko-2 update (#441) in one transaction. The page shows the Puzzle rating, and the change after the result. The provisional flag is sent (shown in a later ticket). A failed Attempt then shows the right Move and "Next"; retrying comes later. - **Schema.** The Puzzle rating row (one per Account; defaults read until the first Attempt) and the Attempt row (Account, Puzzle, state, Moves played, rating before, rating change, served at, resolved at, last replayed at). Unique on (Account, Puzzle), at most one pending per Account. Both are deleted with the Account. - **Contract.** A `puzzle` router with `next`, `move` and `rating`, Account-only, its errors in `REFUSAL_STATUS`. Moves over the wire as UCI (ADR 0021). Copy in English and French (ADR 0025). Seams: S1 (the API through its RPC contract) and S3 (the app's router with the API fakes). Covers stories 1-17, 20-21, 27-31, 47-48 of #440. ## Acceptance criteria - [ ] Signed out, `/puzzles` goes to sign-in and comes back after it; a signed-out RPC call is refused. - [ ] "Next" serves an unseen Puzzle within the window, widens when the band is empty, and never serves a Puzzle this Account already has an Attempt for. - [ ] A reload or a second "Next" returns the same pending Attempt, at the Moves already played. - [ ] The opponent's reply follows a correct Move; the end of the line solves; any Checkmate solves; any other legal Move fails and returns the right Move; an illegal Move is refused without failing. - [ ] A Move on a resolved Attempt is refused. - [ ] Solving raises and failing lowers the Puzzle rating by the module's amount; the result carries the change. - [ ] Two concurrent Moves on one Attempt resolve it once. - [ ] Deleting the Account deletes its Puzzle rating and Attempts. - [ ] Front view tests: the first Move plays, the orientation, only own pieces movable, solved and failed states, the rating change, "Next". - [ ] Prior art followed: the `rpc-games` and `rpc-clock` integration tests, `router.test.tsx`. - [ ] The gate is green (`CONTRIBUTING.md`, "The five commands").
root commented 2026-10-16 08:20:32 +00:00 (Migrated from gitlab.core-project.space)
Copy link

mentioned in issue #444

mentioned in issue #444
root commented 2026-10-16 08:20:39 +00:00 (Migrated from gitlab.core-project.space)
Copy link

mentioned in issue #445

mentioned in issue #445
root commented 2026-10-16 08:20:44 +00:00 (Migrated from gitlab.core-project.space)
Copy link

mentioned in issue #446

mentioned in issue #446
root commented 2026-10-16 08:21:13 +00:00 (Migrated from gitlab.core-project.space)
Copy link

mentioned in issue #441

mentioned in issue #441
root commented 2026-10-16 08:21:28 +00:00 (Migrated from gitlab.core-project.space)
Copy link

marked this issue as related to #440

marked this issue as related to #440
root commented 2026-10-16 08:21:40 +00:00 (Migrated from gitlab.core-project.space)
Copy link

marked this issue as related to #441

marked this issue as related to #441
root commented 2026-10-16 08:21:41 +00:00 (Migrated from gitlab.core-project.space)
Copy link

marked this issue as related to #442

marked this issue as related to #442
root commented 2026-10-16 08:21:42 +00:00 (Migrated from gitlab.core-project.space)
Copy link

marked this issue as related to #444

marked this issue as related to #444
root commented 2026-10-16 08:21:43 +00:00 (Migrated from gitlab.core-project.space)
Copy link

marked this issue as related to #445

marked this issue as related to #445
root commented 2026-10-16 08:21:43 +00:00 (Migrated from gitlab.core-project.space)
Copy link

marked this issue as related to #446

marked this issue as related to #446
root commented 2026-10-18 13:02:28 +00:00 (Migrated from gitlab.core-project.space)
Copy link

mentioned in merge request !554

mentioned in merge request !554
root (Migrated from gitlab.core-project.space) closed this issue 2026-10-19 09:53:34 +00:00
Sign in to join this conversation.
No Branch/Tag specified
Branches Tags
main
fix/front-browser-pages
docs/contributing-forgejo-ci
fix/storybook-ghsa-f2ph-cmg5-wxc8
prototype/dead-end-page
prototype/puzzles-ui-redesign
prototype/399-front-screens
ci/front-test-one-job
feat/370-drizzle-gateway
pr-screenshots
feat/369-postgres-per-stack
claude/pawn-promotion-capture-score-46fffb
claude/github-workflow-improvements-30cf86
docs/352-adr-claim-guest
docs/351-adr-seat-identity
claude/retro-last-10-conversations-9af723
docs/accounts-adrs
claude/lighthouse-pt-4b5a78
docs/350-adr-sessions
feat/game-status-326
research/postgres-on-vps
research/email-provider
prototype/offer-overlay
ci/339-pr-screenshots
prototype/game-status
ci/317-faster-pipeline
spec/303-vps-hosting
claude/paste-invite-link-room-code
docs/309-adr-vps-hosting
fix/311-client-route-fallback
claude/back-navigation-feature-2166ea
claude/resource-contract
claude/api-nest-orpc
claude/adr-nest-orpc
claude/e2e-tests-new-features-1ce2a1
claude/la-spec-273-fd55ac
claude/matt-pocock-skills-integration-444ba9
chore/adopt-the-cognitive-load-coding-rules-270
feat/choose-the-promotion-piece-in-a-popover-instead-of-auto-queening-267
feat/animate-piece-moves-on-the-board-264
feat/show-drag-state-on-the-board-263
feat/move-a-piece-by-dragging-and-dropping-it-261
feat/blocking-modal-when-the-room-has-closed-246
feat/truthful-opponent-presence-lost-connection-left-new-opponent-248
feat/horizontal-game-and-room-action-bars-with-a-confirmed-leave-247
feat/room-status-block-with-a-fixed-height-and-seat-relative-wording-245
docs/amend-adr-0016-a-vacated-seat-replaces-the-60-second-left-timer-244
feat/lint-against-hard-coded-copy-in-the-front-front-no-literal-copy-252
feat/translate-the-front-with-a-language-provider-english-french-242
docs/adr-front-i18n-paraglide
docs/stacked-prs
fix/render-build-fails-on-corepack-enable-record-the-hosting-adr-240
front/8-lint-error
front/7-local-home-align
front/6-comment-purge
front/5-cache-policy
front/3-dissolve-room-actions
front/4-tests-through-view
front/2-room-restructure
front/1-test-harness
feat/deploy-to-render-free-tier-with-a-preview-per-pr-233
harness/6-coding-standards
harness/5-adrs
harness/4-mechanical-rules
harness/3-drop-sandcastle
harness/2-skill-forks
harness/1-adr-index
claude/e2e-test-sub-agents-f8d06b
claude/e2e-hook-performance-072aa8
e2e-restart
e2e-spectator
e2e-draw-offer
e2e-resign-and-reload
e2e-mate-two-browsers
orchestrate-issue-168-implementation
implement-issue-169-and-pr
orpc-ws-room-feature
home-page-issues
comment-audit-and-rules
local/4-local-rail
local/3-local-board
local/2-design-prep
local/1-engine-accessors
prototype/home-page
prototype/game-screen-layout
worktree-front
chore/worktree-env-and-ports
feat/slice-1-3a-front-store-ws-effect-connection-status-screen-136
fix/143-release-seat-on-disconnect
claude/sandcastle-runner-contract
feat/slice-1-2a-apps-api-skeleton-hono-ws-echo-origin-allowlist-132
feat/slice-1-1-chess-contracts-package-context-md-cleanup-128
refactor/reorganise-src-into-layered-folders-per-adr-0008-123
refactor/refactor-engine-reorganise-src-into-layered-folders-per-adr-0008-123
refactor/introduce-the-piecereader-read-port-118
refactor/move-isattacked-to-attacks-ts-with-board-only-signature-111
feat/slice-b1-zobrist-key-computation-for-positions-99
refactor/slice-a-re-back-squareset-with-split-32-representation-97
sandcastle/sequential-reviewer/1779180017269
feat/san-formatter-core-encoding-moves-captures-en-passant-promotion-castling-92
sandcastle/sequential-reviewer/1779111400188
refactor/engine-pr-10-1-migrate-stalemate-into-a-draw-status-variant-86
sandcastle/sequential-reviewer/1779099667568
feat/engine-pr-9-1-castling-rights-tracking-updatecastlingrights-applymove-wiring-80
refactor/extract-piece-attacks-registry-and-rewire-isattacked-movegen-77
sandcastle/sequential-reviewer/1779027444485
backup/pre-axis-a-rewrite
Labels
Clear labels   
bug
Something isn't working

  
documentation
Improvements or additions to documentation

  
duplicate
This issue or pull request already exists

  
enhancement
New feature or request

  
good first issue
Good for newcomers

  
help wanted
Extra attention is needed

  
invalid
This doesn't seem right

  
question
Further information is requested

  
spec
Parent spec written by to-spec

  
wayfinder:grilling

   
wayfinder:map

   
wayfinder:prototype

   
wayfinder:research

   
wayfinder:task

   
wontfix
This will not be worked on

No labels
bug
documentation
duplicate
enhancement
good first issue
help wanted
invalid
question
spec
wayfinder:grilling
wayfinder:map
wayfinder:prototype
wayfinder:research
wayfinder:task
wontfix
Milestone
Clear milestone
No items
No milestone
Projects
Clear projects
No items
No project
Assignees
Clear assignees
No assignees
1 participant
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Blocks
#444 The provisional Puzzle rating: an info icon and a tooltip
amaurycoudr/chess-project
#445 Hint, Show solution, give up and an unrated retry
amaurycoudr/chess-project
#446 Train on a Theme
amaurycoudr/chess-project
Depends on
#441 The Puzzle rating rules: a Glicko-2 module, its ADR and the glossary terms
amaurycoudr/chess-project
#442 The Puzzle set loaded on every stack
amaurycoudr/chess-project
Reference
amaurycoudr/chess-project#443
No description provided.