Puzzle report: do not check positions with 7 pieces or fewer since th…

…e tablebase is better for those

ui/analyse/src/explorer/explorerCtrl.ts

@@ -1,5 +1,6 @@
 import { type Prop, prop, defined } from 'common';
 import { storedBooleanProp } from 'common/storage';
+import { pieceCount } from 'chess';
 import { defer } from 'common/defer';
 import { fenColor } from 'common/miniBoard';
 import { debounce } from 'common/timing';
@@ -15,11 +16,6 @@ import { clearLastShow } from './explorerView';
 
 export const MAX_DEPTH = 50;
 
-function pieceCount(fen: FEN) {
-  const parts = fen.split(/\s/);
-  return parts[0].split(/[nbrqkp]/i).length - 1;
-}
-
 function tablebasePieces(variant: VariantKey) {
   switch (variant) {
     case 'standard':

ui/chess/src/chess.ts

@@ -28,3 +28,5 @@ export const readDrops = (line?: string | null): Key[] | null =>
 export const fenToEpd = (fen: FEN): string => fen.split(' ').slice(0, 4).join(' ');
 
 export const plyToTurn = (ply: number): number => Math.floor((ply - 1) / 2) + 1;
+
+export const pieceCount = (fen: FEN): number => fen.split(/\s/)[0].split(/[nbrqkp]/i).length - 1;

ui/puzzle/src/report.ts

@@ -1,5 +1,6 @@
 import { report as xhrReport } from './xhr';
 import type PuzzleCtrl from './ctrl';
+import { pieceCount } from 'chess';
 import type { PuzzleId, ThemeKey } from './interfaces';
 import { winningChances } from 'ceval';
 import * as licon from 'common/licon';
@@ -8,7 +9,7 @@ import { domDialog } from 'common/dialog';
 import { plyToTurn } from 'chess';
 
 // bump when logic is changed, to distinguish cached clients from new ones
-const version = 9;
+const version = 10;
 
 export default class Report {
   // if local eval suspect multiple solutions, report the puzzle, once at most
@@ -38,10 +39,12 @@ export default class Report {
       threatMode ||
       // the `mate` key theme is not sent, as it is considered redubant with `mateInX`
       ctrl.data.puzzle.themes.some((t: ThemeKey) => t.toLowerCase().includes('mate')) ||
-      // if the user has chosen to hide the dialog less than a week ago
-      this.tsHideReportDialog() > Date.now() - 1000 * 3600 * 24 * 7 ||
+      // positions with 7 pieces or less can be checked with the tablebase
+      pieceCount(ev.fen) <= 7 ||
       // dynamic import from web worker feature is shared by all stockfish 16+ WASMs
-      !ctrl.ceval.engines.active?.requires?.includes('dynamicImportFromWorker')
+      !ctrl.ceval.engines.active?.requires?.includes('dynamicImportFromWorker') ||
+      // if the user has chosen to hide the dialog less than a week ago
+      this.tsHideReportDialog() > Date.now() - 1000 * 3600 * 24 * 7
     )
       return;
     const node = ctrl.node;