perf(ui): reuse hint result as move preview, skip redundant evaluate
CI / changes (pull_request) Successful in 2s
CI / unit (pull_request) Has been skipped
CI / integration (pull_request) Has been skipped
CI / ui (pull_request) Successful in 53s
CI / gate (pull_request) Successful in 0s
CI / deploy (pull_request) Successful in 1m9s

Taking a hint auto-placed the suggested tiles and then called recompute(),
which round-trips a debounced evaluate for that exact placement. The hint is
the engine's own top-ranked, fully scored legal move, so its move already
carries the score, words and direction an evaluate would return — the second
call was pure duplicate work and added a visible disabled->enabled flicker on
the submit button over slow links.

Seed the move preview directly from the hint move via previewFromHint and
cancel any pending evaluate timer so a stale one cannot clobber it. A later
manual edit re-arms recompute() as before, so rearranged tiles are re-evaluated
normally. Client-only; no backend, wire or schema change.
This commit is contained in:
Ilia Denisov
2026-06-19 12:03:27 +02:00
parent b65a3ecc9c
commit 0dd4099d68
3 changed files with 39 additions and 2 deletions
+9 -1
View File
@@ -29,6 +29,7 @@
newPlacement,
place,
placementFromHint,
previewFromHint,
rackView,
recallAt,
recallToSlot,
@@ -751,7 +752,14 @@
if (isCoarse()) zoomed = true;
view = { ...view, hintsRemaining: h.hintsRemaining, walletBalance: h.walletBalance };
syncWallet(h.walletBalance);
recompute();
// The hint is the engine's own top-ranked, fully scored legal move: reuse it as the
// preview instead of a redundant evaluate (same engine call, same placement). Cancel any
// pending evaluate so a stale one cannot clobber it; a later manual edit re-arms recompute().
if (previewTimer) {
clearTimeout(previewTimer);
previewTimer = null;
}
preview = previewFromHint(h.move);
}
} catch (e) {
// The backend does not spend a hint when there is no move.
+19
View File
@@ -6,6 +6,7 @@ import {
newPlacement,
place,
placementFromHint,
previewFromHint,
rackView,
recallAt,
recallIndex,
@@ -105,6 +106,24 @@ describe('placementFromHint', () => {
});
});
describe('previewFromHint', () => {
it('reuses the hint move as a legal preview, carrying score, words and direction', () => {
const preview = previewFromHint({
player: 0,
action: 'play',
dir: 'V',
mainRow: 7,
mainCol: 7,
tiles: [{ row: 7, col: 7, letter: 'C', blank: false }],
words: ['CAT', 'AN'],
count: 0,
score: 18,
total: 42,
});
expect(preview).toEqual({ legal: true, score: 18, words: ['CAT', 'AN'], dir: 'V' });
});
});
describe('reorderIndices', () => {
it('lifts an element and drops it at the given slot among the others', () => {
expect(reorderIndices(4, 0, 2)).toEqual([1, 2, 0, 3]);
+11 -1
View File
@@ -4,7 +4,7 @@
// payload. It is board-agnostic (the gateway/engine does full legality validation at
// submit), which keeps it trivially unit-testable.
import type { Tile } from './model';
import type { EvalResult, MoveRecord, Tile } from './model';
import type { PlacedTile } from './client';
export interface PendingTile {
@@ -55,6 +55,16 @@ export function placementFromHint(tiles: Tile[], rack: string[]): Placement {
return { rack: [...rack], pending };
}
/**
* previewFromHint turns a hint's move into a move preview. A hint is the engine's own
* top-ranked, fully scored legal play, so its move already carries the score, words and
* direction an evaluate would compute for the same placement; reusing it lets the client
* skip a redundant evaluate round-trip after auto-placing the hint.
*/
export function previewFromHint(move: MoveRecord): EvalResult {
return { legal: true, score: move.score, words: move.words, dir: move.dir };
}
function usedIndexes(p: Placement): Set<number> {
return new Set(p.pending.map((t) => t.rackIndex));
}