Merge pull request #414 from TheWizardsCode/copilot/fix-golfscene-interaction-test

Fix flaky GolfScene AI-discard display bug and brittle timing in browser test

Author: Sorra

example-games/golf/scenes/GolfScene.ts

@@ -934,9 +934,13 @@ export class GolfScene extends CardGameScene {
         drawnCard = this.session.shared.discardPile.popOrThrow();
       }
 
-      // When AI draws from discard, update the pile visual immediately
+      // When AI draws from discard, refresh the pile display to show the new
+      // top card (or empty placeholder). The card has already been popped, so
+      // we use refreshPiles() — updateDiscardPileAfterDraw() is designed for
+      // the human peek-not-pop path and would incorrectly compute the display
+      // when the card is already removed from the pile.
       if (drawSource === 'discard') {
-        this.updateDiscardPileAfterDraw();
+        this.refreshPiles();
       }
 
       // Show the drawn card to the player

tests/golf/GolfInteraction.browser.test.ts

@@ -363,8 +363,10 @@ describe('GolfScene interaction tests', () => {
     // Swap with grid card 0
     clickGameObject(internals.humanCardSprites[0]);
 
-    // Wait for AI to take its turn (AI_DELAY = 600ms + animation time)
-    await wait(3000);
+    // Wait for both the human animation and the full AI turn to complete.
+    // Using a phase-based wait is more reliable than a fixed timeout in CI,
+    // as it polls until the game returns to a stable state.
+    await waitForAnyPhase(scene, ['waiting-for-draw', 'round-ended'], 15000);
 
     // After AI turn, it should be human's turn again (or round ended)
     const phase = internals.phaseManager.current;