From 864c4076db7b566f4d52fe3619a22c43694474b4 Mon Sep 17 00:00:00 2001 From: sushanth-dev Date: Sat, 19 Sep 2026 13:03:58 +0530 Subject: [PATCH 1/2] fix: update moves by rating on the mobile analysis view MovesByRating freezes its displayed series unless it is given a positionKey, so the mobile call site in the analysis page kept rendering the starting position's probabilities. Desktop was unaffected because AnalysisSidebar already passes positionKey; mobile did not. Pass it under the same guard used for moves and colorSanMapping. Refs #291 Co-Authored-By: Claude Sonnet 5 --- src/pages/analysis/[...id].tsx | 6 ++++++ 1 file changed, 6 insertions(+) diff --git a/src/pages/analysis/[...id].tsx b/src/pages/analysis/[...id].tsx index 36c7e9ac..6dfd65c8 100644 --- a/src/pages/analysis/[...id].tsx +++ b/src/pages/analysis/[...id].tsx @@ -2181,6 +2181,12 @@ const Analysis: React.FC = ({ ? controller.colorSanMapping : {} } + positionKey={ + analysisEnabled && + !controller.learnFromMistakes.state.isActive + ? controller.currentNode?.fen + : undefined + } /> {(!analysisEnabled || controller.learnFromMistakes.state.isActive) && ( From 295435c407f51753f46b1d4dd150691f0fa51341 Mon Sep 17 00:00:00 2001 From: sushanth-dev Date: Sat, 19 Sep 2026 13:04:04 +0530 Subject: [PATCH 2/2] fix: make the test script exit 0 when no tests exist npm test ran jest with no file matching testMatch, so it printed "No tests found" and exited 1. The CI test job was deleted to work around that. Jest's own suggested --passWithNoTests makes the script report the truth instead: the suite runs and reports zero tests. Refs #177 Co-Authored-By: Claude Sonnet 5 --- package.json | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/package.json b/package.json index 8aad1c4d..b46e9277 100644 --- a/package.json +++ b/package.json @@ -7,7 +7,7 @@ "start": "next start", "export": "next export", "lint": "eslint . --ext ts,tsx --fix", - "test": "jest" + "test": "jest --passWithNoTests" }, "dependencies": { "@babel/preset-react": "^7.16.7",