From 63a271fe928f4583ee8dcc7657c3be1a5a3f424d Mon Sep 17 00:00:00 2001
From: Kyle Jacobson <jacokyle01@gmail.com>
Date: Sun, 5 Jul 2026 11:36:40 -0500
Subject: [PATCH 1/5] status bar tweaks

---
 frontend/src/Chessrepeat.tsx                  | 54 +++++++++++--------
 .../modals/AddToRepertoireModal.tsx           |  2 +-
 .../src/components/modals/SettingsModal.css   |  4 +-
 3 files changed, 37 insertions(+), 23 deletions(-)

diff --git a/frontend/src/Chessrepeat.tsx b/frontend/src/Chessrepeat.tsx
index d939649..167bf0b 100644
--- a/frontend/src/Chessrepeat.tsx
+++ b/frontend/src/Chessrepeat.tsx
@@ -345,6 +345,8 @@ export const Chessrepeat = () => {
           case 'learn':
             learn();
             setNextTrainablePosition();
+            console.log("update")
+            updateDueCounts();
             break;
           case 'recall':
             if (userTip == 'fail') {
@@ -446,27 +448,37 @@ export const Chessrepeat = () => {
                     text-gray-700 shadow-lg opacity-0 transition-opacity duration-150
                     group-hover:opacity-100"
                 >
-                  <div className="flex items-center gap-2 py-0.5">
-                    <GraduationCap size={14} className="text-sky-700" />
-                    <span className="flex-1">To learn</span>
-                    <span className="font-mono font-semibold">{chapter.unseenCount}</span>
-                  </div>
-                  <div className="flex items-center gap-2 py-0.5">
-                    <History size={14} className="text-blue-800" />
-                    <span className="flex-1">Due</span>
-                    <span className="font-mono font-semibold">{chapter.lastDueCount}</span>
-                  </div>
-                  <div className="flex items-center gap-2 py-0.5">
-                    <span className="inline-block h-2.5 w-2.5 rounded-sm bg-gray-200" />
-                    <span className="flex-1">Learned</span>
-                    <span className="font-mono font-semibold">
-                      {chapter.enabledCount - chapter.unseenCount - chapter.lastDueCount}
-                    </span>
-                  </div>
-                  <div className="mt-1 flex items-center gap-2 border-t border-gray-100 pt-1">
-                    <span className="flex-1 font-semibold">Total</span>
-                    <span className="font-mono font-semibold">{chapter.enabledCount}</span>
-                  </div>
+                  <table className="border-separate border-spacing-x-2 border-spacing-y-0.5">
+                    <tbody>
+                      <tr>
+                        <td className="align-middle">
+                          <GraduationCap size={14} className="text-sky-700" />
+                        </td>
+                        <td className="text-left">Learned</td>
+                        <td className="text-right font-mono font-semibold">
+                          {chapter.enabledCount - chapter.unseenCount - chapter.lastDueCount}
+                        </td>
+                        <td className="text-right font-mono text-gray-400">
+                          {Math.round(
+                            ((chapter.enabledCount - chapter.unseenCount - chapter.lastDueCount) /
+                              chapter.enabledCount) *
+                              100,
+                          )}
+                          %
+                        </td>
+                      </tr>
+                      <tr>
+                        <td className="align-middle">
+                          <History size={14} className="text-blue-800" />
+                        </td>
+                        <td className="text-left">Due Now</td>
+                        <td className="text-right font-mono font-semibold">{chapter.lastDueCount}</td>
+                        <td className="text-right font-mono text-gray-400">
+                          {Math.round((chapter.lastDueCount / chapter.enabledCount) * 100)}%
+                        </td>
+                      </tr>
+                    </tbody>
+                  </table>
                 </div>
               </div>
             )}
diff --git a/frontend/src/components/modals/AddToRepertoireModal.tsx b/frontend/src/components/modals/AddToRepertoireModal.tsx
index 51975c1..60c54bb 100644
--- a/frontend/src/components/modals/AddToRepertoireModal.tsx
+++ b/frontend/src/components/modals/AddToRepertoireModal.tsx
@@ -141,7 +141,7 @@ const AddToRepertoireModal: React.FC = () => {
     <dialog
       open
       className="fixed top-1/2 left-1/2 -translate-x-1/2 -translate-y-1/2 z-[1000]
-                 border-none bg-white rounded-lg shadow-lg w-full max-w-lg"
+                 border-none bg-white rounded-lg shadow-lg w-[calc(100%-2rem)] max-w-lg"
     >
       {/* Close button */}
       <button
diff --git a/frontend/src/components/modals/SettingsModal.css b/frontend/src/components/modals/SettingsModal.css
index 86e6648..62a5a01 100644
--- a/frontend/src/components/modals/SettingsModal.css
+++ b/frontend/src/components/modals/SettingsModal.css
@@ -4,7 +4,9 @@
   left: 50%;
   transform: translate(-50%, -50%);
   z-index: 50;
-  width: 100%;
+  /* leave a margin on each side on narrow viewports so the close button,
+     which hangs off the right edge (right: -0.75rem), isn't clipped. */
+  width: calc(100% - 2rem);
   max-width: 28rem;
   border: none;
   background: #fff;

From 95945df1b0258647ed9f3b34cf0e06c4b14a65b4 Mon Sep 17 00:00:00 2001
From: Kyle Jacobson <jacokyle01@gmail.com>
Date: Tue, 7 Jul 2026 20:24:41 -0500
Subject: [PATCH 2/5] prs can work locally -- but disable anyone from doing a
 pr from main

---
 .github/workflows/deploy.yml | 5 ++---
 1 file changed, 2 insertions(+), 3 deletions(-)

diff --git a/.github/workflows/deploy.yml b/.github/workflows/deploy.yml
index 27d43a0..95d973d 100644
--- a/.github/workflows/deploy.yml
+++ b/.github/workflows/deploy.yml
@@ -1,7 +1,7 @@
 name: deploy
 
 # On push to main: tests run on the VPS via a self-hosted runner; if
-# they pass, the backend image is built on GitHub-hosted infra and
+# they pass, the backend image is built and
 # pushed to GHCR; the VPS-side runner then pulls the image and
 # restarts the docker-compose stack in place.
 #
@@ -19,10 +19,9 @@ env:
   BACKEND_IMAGE: ${{ github.repository }}-backend
 
 jobs:
-  # PR-only quick test on GitHub-hosted infra.
   test-pr:
     if: github.event_name == 'pull_request'
-    runs-on: ubuntu-latest
+    runs-on: [self-hosted, chessrepeat-vps]
     steps:
       - uses: actions/checkout@v4
       - uses: actions/setup-go@v5

From f9cfe2fe19d106f4a1d67a71735028de01c01d72 Mon Sep 17 00:00:00 2001
From: Kyle Jacobson <jacokyle01@gmail.com>
Date: Tue, 7 Jul 2026 20:28:04 -0500
Subject: [PATCH 3/5] dont run test prs to main

---
 .github/workflows/deploy.yml | 22 +++-------------------
 1 file changed, 3 insertions(+), 19 deletions(-)

diff --git a/.github/workflows/deploy.yml b/.github/workflows/deploy.yml
index 95d973d..bdf66bb 100644
--- a/.github/workflows/deploy.yml
+++ b/.github/workflows/deploy.yml
@@ -19,24 +19,8 @@ env:
   BACKEND_IMAGE: ${{ github.repository }}-backend
 
 jobs:
-  test-pr:
-    if: github.event_name == 'pull_request'
-    runs-on: [self-hosted, chessrepeat-vps]
-    steps:
-      - uses: actions/checkout@v4
-      - uses: actions/setup-go@v5
-        with:
-          go-version-file: backend/go.mod
-          cache-dependency-path: backend/go.sum
-      - name: vet
-        working-directory: backend
-        run: go vet ./...
-      - name: test
-        working-directory: backend
-        run: go test ./...
-
-  # Real gate for deploys: tests run on the VPS itself.
-  test-on-vps:
+  # run tests before deploy.
+  test:
     if: github.event_name == 'push'
     runs-on: [self-hosted, chessrepeat-vps]
     steps:
@@ -54,7 +38,7 @@ jobs:
 
   build-and-push:
     if: github.event_name == 'push'
-    needs: test-on-vps
+    needs: test
     runs-on: ubuntu-latest
     permissions:
       contents: read

From e85337f5e1681c64b7b6f766d5909b254117de60 Mon Sep 17 00:00:00 2001
From: Kyle Jacobson <jacokyle01@gmail.com>
Date: Wed, 8 Jul 2026 21:12:39 -0500
Subject: [PATCH 4/5] css tweaks + bug fixes

---
 backend/internal/ws/server.go                 |   7 +-
 frontend/src/Chessrepeat.tsx                  | 174 +++++++++---------
 frontend/src/components/Debug.tsx             |   5 +
 frontend/src/components/Header.tsx            |   7 +-
 .../src/components/MobileCommentPopout.tsx    |   2 +-
 frontend/src/css/layout.css                   |  13 +-
 frontend/src/services/repertoire.ts           |   1 +
 frontend/src/store/state.ts                   |  33 +++-
 8 files changed, 136 insertions(+), 106 deletions(-)

diff --git a/backend/internal/ws/server.go b/backend/internal/ws/server.go
index 4403d49..bb31573 100644
--- a/backend/internal/ws/server.go
+++ b/backend/internal/ws/server.go
@@ -98,10 +98,9 @@ type subscriber struct {
 	// burn CPU/DB by flooding the connection. Allocated per-connection
 	// so one chatty peer can't starve another.
 	readBudget *ratelimit.Bucket
-	// lastSeenAt is the unix-nano timestamp of the most recent inbound
-	// message (any type — pings, mutations, anything). The liveness
-	// watcher kicks the subscriber if this falls outside idleTimeout.
-	// atomic because reader (writes) and watcher (reads) run concurrently.
+	// updated after client sends an operation to server
+	// if currentTime - `lastSeenAt` < `livenessCheckInterval` after timer fires,
+	//	kick subscriber
 	lastSeenAt atomic.Int64
 }
 
diff --git a/frontend/src/Chessrepeat.tsx b/frontend/src/Chessrepeat.tsx
index 167bf0b..7391319 100644
--- a/frontend/src/Chessrepeat.tsx
+++ b/frontend/src/Chessrepeat.tsx
@@ -223,7 +223,7 @@ export const Chessrepeat = () => {
   The current move we're training
   */
 
-  //TODO Fix logic here..  
+  //TODO Fix logic here..
   const targetDest = (): Key[] => {
     // console.log("selectedNode fen", selectedNode?.data.fen)
     const targetNode = useTrainerStore.getState().trainableContext.targetMove;
@@ -335,7 +335,7 @@ export const Chessrepeat = () => {
   const prevMove = prevMoveIfExists();
   const lastMove = selectedNode ? prevMove : undefined;
 
-  const finishMove = (san: string, meta: MoveMetadata, to: Key) => {
+  const finishMove = async (san: string, meta: MoveMetadata, to: Key) => {
     if (!isEditing) {
       meta.captured ? sounds.capture.play().catch(console.error) : sounds.move.play().catch(console.error);
 
@@ -343,22 +343,21 @@ export const Chessrepeat = () => {
         updateDueCounts();
         switch (trainingMethod) {
           case 'learn':
-            learn();
+            await learn();
             setNextTrainablePosition();
-            console.log("update")
-            updateDueCounts();
             break;
           case 'recall':
             if (userTip == 'fail') {
-              train(false);
+              await train(false);
               setNextTrainablePosition();
               return;
             }
             setLastGuess(san);
             switch (guess(san)) {
               case 'success': {
-                const secsUntilDue = train(true);
-                showBoxAtSquare(to, secsUntilDue);
+                const secsUntilDue = await train(true);
+                // null when the move was deleted under us (train resynced instead)
+                if (secsUntilDue != null) showBoxAtSquare(to, secsUntilDue);
                 setNextTrainablePosition();
                 break;
               }
@@ -377,7 +376,7 @@ export const Chessrepeat = () => {
     }
   };
 
-  const onAfterMove = (from: Key, to: Key, meta: MoveMetadata) => {
+  const onAfterMove = async (from: Key, to: Key, meta: MoveMetadata) => {
     const fenBefore = selectedNode?.data.fen || initial;
 
     // If a promo is already open, ignore additional moves (defensive)
@@ -392,7 +391,7 @@ export const Chessrepeat = () => {
 
     // Normal move
     const san = chessgroundToSan(fenBefore, from, to);
-    finishMove(san, meta, to);
+    await finishMove(san, meta, to);
     updateDueCounts();
   };
 
@@ -428,94 +427,89 @@ export const Chessrepeat = () => {
           {/* BOARD */}
           <div className="area-board" id="board-wrap">
             <div className="board-card">
-            {chapter && chapter.enabledCount > 0 && (
-              <div className="group relative">
-                <div className="flex h-2 w-full overflow-hidden rounded-md bg-gray-200 cursor-default">
+              {chapter && chapter.enabledCount > 0 && (
+                <div className="group relative">
+                  <div className="flex h-2 w-full overflow-hidden rounded-md bg-gray-200 cursor-default">
+                    <div
+                      className="h-full bg-brand-blue-light"
+                      style={{ width: `${(chapter.unseenCount / chapter.enabledCount) * 100}%` }}
+                    />
+                    <div
+                      className="h-full bg-brand-blue"
+                      style={{ width: `${(chapter.lastDueCount / chapter.enabledCount) * 100}%` }}
+                    />
+                  </div>
+
+                  {/* Breakdown tooltip on hover */}
                   <div
-                    className="h-full bg-brand-blue-light"
-                    style={{ width: `${(chapter.unseenCount / chapter.enabledCount) * 100}%` }}
-                  />
-                  <div
-                    className="h-full bg-brand-blue"
-                    style={{ width: `${(chapter.lastDueCount / chapter.enabledCount) * 100}%` }}
-                  />
-                </div>
-
-                {/* Breakdown tooltip on hover */}
-                <div
-                  className="pointer-events-none absolute left-1/2 top-full z-20 mt-2 -translate-x-1/2
+                    className="pointer-events-none absolute left-1/2 top-full z-20 mt-2 -translate-x-1/2
                     whitespace-nowrap rounded-md border border-gray-200 bg-white px-3 py-2 text-xs
                     text-gray-700 shadow-lg opacity-0 transition-opacity duration-150
                     group-hover:opacity-100"
-                >
-                  <table className="border-separate border-spacing-x-2 border-spacing-y-0.5">
-                    <tbody>
-                      <tr>
-                        <td className="align-middle">
-                          <GraduationCap size={14} className="text-sky-700" />
-                        </td>
-                        <td className="text-left">Learned</td>
-                        <td className="text-right font-mono font-semibold">
-                          {chapter.enabledCount - chapter.unseenCount - chapter.lastDueCount}
-                        </td>
-                        <td className="text-right font-mono text-gray-400">
-                          {Math.round(
-                            ((chapter.enabledCount - chapter.unseenCount - chapter.lastDueCount) /
-                              chapter.enabledCount) *
-                              100,
-                          )}
-                          %
-                        </td>
-                      </tr>
-                      <tr>
-                        <td className="align-middle">
-                          <History size={14} className="text-blue-800" />
-                        </td>
-                        <td className="text-left">Due Now</td>
-                        <td className="text-right font-mono font-semibold">{chapter.lastDueCount}</td>
-                        <td className="text-right font-mono text-gray-400">
-                          {Math.round((chapter.lastDueCount / chapter.enabledCount) * 100)}%
-                        </td>
-                      </tr>
-                    </tbody>
-                  </table>
+                  >
+                    <table className="border-separate border-spacing-x-2 border-spacing-y-0.5">
+                      <tbody>
+                        <tr>
+                          <td className="align-middle">
+                            <GraduationCap size={14} className="text-sky-700" />
+                          </td>
+                          <td className="text-left">To Learn</td>
+                          <td className="text-right font-mono font-semibold">
+                            {chapter.unseenCount}
+                          </td>
+                          <td className="text-right font-mono text-gray-400">
+                            {Math.round((chapter.unseenCount / chapter.enabledCount) * 100)}%
+                          </td>
+                        </tr>
+                        <tr>
+                          <td className="align-middle">
+                            <History size={14} className="text-blue-800" />
+                          </td>
+                          <td className="text-left">Due Now</td>
+                          <td className="text-right font-mono font-semibold">{chapter.lastDueCount}</td>
+                          <td className="text-right font-mono text-gray-400">
+                            {Math.round((chapter.lastDueCount / chapter.enabledCount) * 100)}%
+                          </td>
+                        </tr>
+                      </tbody>
+                    </table>
+                  </div>
                 </div>
-              </div>
-            )}
-            <div ref={containerRef}>
-              <Chessground
-                orientation={chapter?.trainAs || 'white'}
-                fen={selectedNode?.data.fen || initial}
-                turnColor={turn}
-                lastMove={lastMove}
-                movable={{
-                  free: false,
-                  color: turn,
-                  dests: calculateDests(),
-                  events: { after: onAfterMove },
-                }}
-                drawable={{ autoShapes: createShapes() }}
-              />
-              {pendingPromo && (
-                <PromotionOverlay
-                  dest={pendingPromo.to}
-                  color={promotionColorFromFen(pendingPromo.fenBefore)}
+              )}
+              <div ref={containerRef}>
+                <Chessground
                   orientation={chapter?.trainAs || 'white'}
-                  onCancel={closePromo}
-                  requiredRole={
-                    trainingMethod === 'learn'
-                      ? promoRoleFromSan(useTrainerStore.getState().trainableContext?.targetMove?.data?.san)
-                      : undefined
-                  }
-                  onPick={(role: PromoRole) => {
-                    const { fenBefore, from, to, meta } = pendingPromo;
-                    closePromo();
-                    const san = chessgroundToSan(fenBefore, from, to, role);
-                    finishMove(san, meta, to);
+                  fen={selectedNode?.data.fen || initial}
+                  turnColor={turn}
+                  lastMove={lastMove}
+                  movable={{
+                    free: false,
+                    color: turn,
+                    dests: calculateDests(),
+                    events: { after: onAfterMove },
                   }}
+                  drawable={{ autoShapes: createShapes() }}
                 />
-              )}
-            </div>
+                {pendingPromo && (
+                  <PromotionOverlay
+                    dest={pendingPromo.to}
+                    color={promotionColorFromFen(pendingPromo.fenBefore)}
+                    orientation={chapter?.trainAs || 'white'}
+                    onCancel={closePromo}
+                    requiredRole={
+                      trainingMethod === 'learn'
+                        ? promoRoleFromSan(useTrainerStore.getState().trainableContext?.targetMove?.data?.san)
+                        : undefined
+                    }
+                    onPick={(role: PromoRole) => {
+                      const { fenBefore, from, to, meta } = pendingPromo;
+                      closePromo();
+                      const san = chessgroundToSan(fenBefore, from, to, role);
+                      finishMove(san, meta, to);
+                    }}
+                  />
+                )}
+              </div>
             </div>
 
             {/* CONTROLS — part of the same board panel, always directly
diff --git a/frontend/src/components/Debug.tsx b/frontend/src/components/Debug.tsx
index 988b83f..7e45fa1 100644
--- a/frontend/src/components/Debug.tsx
+++ b/frontend/src/components/Debug.tsx
@@ -10,9 +10,14 @@ export const Debug: React.FC = () => {
   const selectedPath = useTrainerStore.getState().selectedPath;
   const TrainingMethod = useTrainerStore.getState().trainingMethod;
 
+  const { repertoire, selectedChapterId } = useTrainerStore.getState();
+
+  const chapter = repertoire.find((c) => c.uuid === selectedChapterId);
+  if (!chapter) return undefined;
   console.log(targetMove?.data?.fen);
   return (
     <div className="absolute bottom-0 p-15">
+      <div>{`chapter ${chapter.unseenCount}`}</div>
       <div>{`TrainingPath ${trainingPath}`}</div>
       <div>{`selectedPath ${selectedPath}`}</div>
       <div>{`TrainingMethod ${TrainingMethod}`}</div>
diff --git a/frontend/src/components/Header.tsx b/frontend/src/components/Header.tsx
index 1ad9873..35b5d04 100644
--- a/frontend/src/components/Header.tsx
+++ b/frontend/src/components/Header.tsx
@@ -135,7 +135,12 @@ export function Header({ connectedUsers, incomingCollaboratorsCount = 0, onOpenC
           </>
         ) : (
           showSignIn && (
-            <button type="button" onClick={openLogin} className="header-link" title="Sign in">
+            <button
+              type="button"
+              onClick={openLogin}
+              className="header-link header-signin"
+              title="Sign in"
+            >
               <span>sign in</span>
               <LogIn />
             </button>
diff --git a/frontend/src/components/MobileCommentPopout.tsx b/frontend/src/components/MobileCommentPopout.tsx
index 397a8b3..e83364a 100644
--- a/frontend/src/components/MobileCommentPopout.tsx
+++ b/frontend/src/components/MobileCommentPopout.tsx
@@ -73,7 +73,7 @@ const MobileCommentPopout: React.FC = () => {
           <dialog
             open
             className="fixed top-1/2 left-1/2 -translate-x-1/2 -translate-y-1/2 z-50
-                       border-none bg-white rounded-lg shadow-lg w-full max-w-md p-4"
+                       border-none bg-white rounded-lg shadow-lg w-[calc(100%-2rem)] max-w-md p-4"
           >
             <button
               className="absolute -top-3 -right-3 bg-red-500 text-white rounded-full h-8 w-8
diff --git a/frontend/src/css/layout.css b/frontend/src/css/layout.css
index e146ff1..eabc9cf 100644
--- a/frontend/src/css/layout.css
+++ b/frontend/src/css/layout.css
@@ -118,8 +118,8 @@
 }
 
 #header .header-link svg {
-  width: 1.25rem;
-  height: 1.25rem;
+  width: 1.5rem;
+  height: 1.5rem;
   flex-shrink: 0;
 }
 
@@ -142,9 +142,14 @@
     display: none;
   }
 
+  /* the sign-in label always stays visible, even at the narrowest width. */
+  #header .header-signin span {
+    display: inline;
+  }
+
   #header .header-link svg {
-    width: 0.78em;
-    height: 0.78em;
+    width: 0.95em;
+    height: 0.95em;
   }
 }
 
diff --git a/frontend/src/services/repertoire.ts b/frontend/src/services/repertoire.ts
index c802e82..4d51677 100644
--- a/frontend/src/services/repertoire.ts
+++ b/frontend/src/services/repertoire.ts
@@ -26,6 +26,7 @@ export type FetchRepertoireResult = {
 // server response carries the session's user, so we hydrate auth and
 // author from it before installing the chapters.
 export async function fetchRepertoire(): Promise<FetchRepertoireResult> {
+  console.log("fetch repetoire")
   const store = useTrainerStore.getState();
   const { repertoireAuthor } = store;
   const url = repertoireAuthor
diff --git a/frontend/src/store/state.ts b/frontend/src/store/state.ts
index f61e8f0..2209da9 100644
--- a/frontend/src/store/state.ts
+++ b/frontend/src/store/state.ts
@@ -150,8 +150,8 @@ interface TrainerState {
   setCommentRemote: (chapterId: string, path: string, comment: string) => void;
   updateDueCounts: () => void;
   setNextTrainablePosition: () => void;
-  learn: () => void;
-  train: (correct: boolean) => number;
+  learn: () => Promise<void>;
+  train: (correct: boolean) => Promise<number | null>;
   guess: (san: string) => TrainingOutcome;
 
   // higher-level ops
@@ -536,7 +536,7 @@ export const useTrainerStore = create<TrainerState>()(
           if (!card) return;
 
           const msTilDue = new Date(card.due).getTime() - Date.now();
-          if (msTilDue < 0) {
+          if (msTilDue <= 0) {
             countDueNow++;
           }
           dueSummary.push(msTilDue);
@@ -618,6 +618,18 @@ export const useTrainerStore = create<TrainerState>()(
         const chapter = selectedChapterOf(repertoire, selectedChapterId);
         const targetNode = trainableContext?.targetMove;
         if (!chapter || !targetNode || !key) return;
+
+        // The target move may have been deleted by another client since we
+        // computed this trainable position. Writing a card onto a node that's
+        // no longer in the tree would silently drift us from the server, so
+        // verify it still exists and resync the whole repertoire if not.
+        const targetPath = trainableContext.startingPath + targetNode.data.id;
+        if (!nodeAtPath(chapter.root, targetPath)) {
+          console.warn('learn: target move not found, reloading', { targetPath });
+          await fetchRepertoire();
+          return;
+        }
+
         const card = createCard();
         if (!targetNode.data.training) targetNode.data.training = {};
         targetNode.data.training[key] = card;
@@ -632,7 +644,6 @@ export const useTrainerStore = create<TrainerState>()(
           // by path equality, so sending the parent path used to anchor
           // the card on the wrong row and the card "disappeared" on the
           // next reload.
-          const targetPath = trainableContext.startingPath + targetNode.data.id;
           socket.send(
             JSON.stringify({
               type: 'training_updated',
@@ -648,13 +659,24 @@ export const useTrainerStore = create<TrainerState>()(
       /*
         Update node's card based on the result of a training
       */
-      train: (correct: boolean) => {
+      train: async (correct: boolean) => {
         const { repertoire, selectedChapterId, trainableContext, socket } = get();
         const key = currentUserKey();
         const targetNode = trainableContext?.targetMove;
         const chapter = selectedChapterOf(repertoire, selectedChapterId);
         if (!chapter || !targetNode || !key) return null;
 
+        // The target move may have been deleted by another client since we
+        // computed this trainable position. Reviewing a card on a node that's
+        // no longer in the tree would silently drift us from the server, so
+        // verify it still exists and resync the whole repertoire if not.
+        const targetPath = trainableContext.startingPath + targetNode.data.id;
+        if (!nodeAtPath(chapter.root, targetPath)) {
+          console.warn('train: target move not found, reloading', { targetPath });
+          await fetchRepertoire();
+          return null;
+        }
+
         const oldCard = userCard(targetNode.data, key);
         if (!oldCard) return null;
         const card = reviewCard(oldCard, correct);
@@ -663,7 +685,6 @@ export const useTrainerStore = create<TrainerState>()(
 
         if (useAuthStore.getState().isAuthenticated() && socket && socket.readyState === WebSocket.OPEN) {
           // Target move's path (see learn() comment above).
-          const targetPath = trainableContext.startingPath + targetNode.data.id;
           socket.send(
             JSON.stringify({
               type: 'training_updated',

From 1944c40d7374a1a1fafb4fdbee3e1d956c81d7dc Mon Sep 17 00:00:00 2001
From: Kyle Jacobson <jacokyle01@gmail.com>
Date: Wed, 8 Jul 2026 21:28:41 -0500
Subject: [PATCH 5/5] fix ui bug caused by promises. switch await --> void

---
 frontend/src/Chessrepeat.tsx |  8 ++++----
 frontend/src/store/state.ts  | 14 +++++++-------
 2 files changed, 11 insertions(+), 11 deletions(-)

diff --git a/frontend/src/Chessrepeat.tsx b/frontend/src/Chessrepeat.tsx
index 7391319..47dd360 100644
--- a/frontend/src/Chessrepeat.tsx
+++ b/frontend/src/Chessrepeat.tsx
@@ -335,7 +335,7 @@ export const Chessrepeat = () => {
   const prevMove = prevMoveIfExists();
   const lastMove = selectedNode ? prevMove : undefined;
 
-  const finishMove = async (san: string, meta: MoveMetadata, to: Key) => {
+  const finishMove = (san: string, meta: MoveMetadata, to: Key) => {
     if (!isEditing) {
       meta.captured ? sounds.capture.play().catch(console.error) : sounds.move.play().catch(console.error);
 
@@ -343,19 +343,19 @@ export const Chessrepeat = () => {
         updateDueCounts();
         switch (trainingMethod) {
           case 'learn':
-            await learn();
+            learn();
             setNextTrainablePosition();
             break;
           case 'recall':
             if (userTip == 'fail') {
-              await train(false);
+              train(false);
               setNextTrainablePosition();
               return;
             }
             setLastGuess(san);
             switch (guess(san)) {
               case 'success': {
-                const secsUntilDue = await train(true);
+                const secsUntilDue = train(true);
                 // null when the move was deleted under us (train resynced instead)
                 if (secsUntilDue != null) showBoxAtSquare(to, secsUntilDue);
                 setNextTrainablePosition();
diff --git a/frontend/src/store/state.ts b/frontend/src/store/state.ts
index 2209da9..963b6fc 100644
--- a/frontend/src/store/state.ts
+++ b/frontend/src/store/state.ts
@@ -150,8 +150,8 @@ interface TrainerState {
   setCommentRemote: (chapterId: string, path: string, comment: string) => void;
   updateDueCounts: () => void;
   setNextTrainablePosition: () => void;
-  learn: () => Promise<void>;
-  train: (correct: boolean) => Promise<number | null>;
+  learn: () => void;
+  train: (correct: boolean) => number | null;
   guess: (san: string) => TrainingOutcome;
 
   // higher-level ops
@@ -612,7 +612,7 @@ export const useTrainerStore = create<TrainerState>()(
         Initialize card training data upon
         seeing a node for the first time
       */
-      learn: async () => {
+      learn: () => {
         const { repertoire, selectedChapterId, trainableContext, socket } = get();
         const key = currentUserKey();
         const chapter = selectedChapterOf(repertoire, selectedChapterId);
@@ -626,7 +626,7 @@ export const useTrainerStore = create<TrainerState>()(
         const targetPath = trainableContext.startingPath + targetNode.data.id;
         if (!nodeAtPath(chapter.root, targetPath)) {
           console.warn('learn: target move not found, reloading', { targetPath });
-          await fetchRepertoire();
+          void fetchRepertoire();
           return;
         }
 
@@ -634,7 +634,7 @@ export const useTrainerStore = create<TrainerState>()(
         if (!targetNode.data.training) targetNode.data.training = {};
         targetNode.data.training[key] = card;
         chapter.unseenCount--;
-        await persistChapter(chapter);
+        void persistChapter(chapter);
 
         if (useAuthStore.getState().isAuthenticated() && socket && socket.readyState === WebSocket.OPEN) {
           // Wire path must be the *target move's* path, not the parent's
@@ -659,7 +659,7 @@ export const useTrainerStore = create<TrainerState>()(
       /*
         Update node's card based on the result of a training
       */
-      train: async (correct: boolean) => {
+      train: (correct: boolean) => {
         const { repertoire, selectedChapterId, trainableContext, socket } = get();
         const key = currentUserKey();
         const targetNode = trainableContext?.targetMove;
@@ -673,7 +673,7 @@ export const useTrainerStore = create<TrainerState>()(
         const targetPath = trainableContext.startingPath + targetNode.data.id;
         if (!nodeAtPath(chapter.root, targetPath)) {
           console.warn('train: target move not found, reloading', { targetPath });
-          await fetchRepertoire();
+          void fetchRepertoire();
           return null;
         }