Fix console link to correct match & increase console window

Author: TheApplePieGod

client/src/components/sidebar/runner/runner.tsx

@@ -378,7 +378,7 @@ const MapSelector: React.FC<MapSelectorProps> = ({ maps, availableMaps, onSelect
     )
 }
 
-export type ConsoleLine = { content: string; type: 'output' | 'error' | 'bold' }
+export type ConsoleLine = { content: string; type: 'output' | 'error' | 'bold'; matchIdx: number }
 
 type Props = {
     lines: RingBuffer<ConsoleLine>
@@ -401,19 +401,23 @@ export const Console: React.FC<Props> = ({ lines }) => {
         }
     }
 
-    const focusRobot = (round: number, id: number) => {
+    const focusRobot = (match: number, round: number, id: number) => {
         setPopout(false)
-        GameRunner.jumpToRound(round)
-        GameRenderer.setSelectedRobot(id)
-
-        // If turn playback is enabled, focus the robot's exact turn as well
-        if (GameRunner.match?.playbackPerTurn) {
-            GameRunner.jumpToRobotTurn(id)
-        }
+        GameRunner.setMatch(GameRunner.game?.matches[match], round)
+
+        // Update selection after a delay so it doesn't get overwritten
+        setTimeout(() => {
+            // If turn playback is enabled, focus the robot's exact turn as well
+            GameRenderer.setSelectedRobot(id)
+            if (GameRunner.match?.playbackPerTurn) {
+                GameRunner.jumpToRobotTurn(id)
+            }
+        }, 5)
     }
 
     const ConsoleRow = (props: { index: number; style: any }) => {
-        const content = lines.get(props.index)!.content
+        const row = lines.get(props.index)!
+        const content = row.content
 
         // Check if the printout is from a bot. If so, add a special click element
         // that selects the bot
@@ -429,7 +433,7 @@ export const Console: React.FC<Props> = ({ lines }) => {
                 <div className="flex items-center gap-1 sele" style={props.style}>
                     <span
                         className="text-blueLight decoration-blueLight text-xs whitespace-nowrap underline cursor-pointer"
-                        onClick={() => focusRobot(round, id)}
+                        onClick={() => focusRobot(row.matchIdx, round, id)}
                     >
                         {`[Team ${team}, ID #${id}, Round ${round}]`}
                     </span>

client/src/components/sidebar/runner/scaffold.ts

@@ -53,12 +53,23 @@ export const useScaffold = (): Scaffold => {
     const [error, setError] = useState('')
     const matchPID = useRef<string | undefined>(undefined)
     const forceUpdate = useForceUpdate()
-    const consoleLines = useRef<RingBuffer<ConsoleLine>>(new RingBuffer(10000))
+    const consoleLines = useRef<RingBuffer<ConsoleLine>>(new RingBuffer(50000))
 
     const [webSocketListener, setWebSocketListener] = useState<WebSocketListener | undefined>()
 
     const log = (line: ConsoleLine) => {
+        // Set match index to be the same as the previous log value
+        if (consoleLines.current.length() > 0) {
+            line.matchIdx = consoleLines.current.get(consoleLines.current.length() - 1)!.matchIdx
+        }
+
+        // If we encounter the end-of-match message, increment the match index
+        if (line.content.startsWith('[server]') && line.content.includes('Finished')) {
+            line.matchIdx++
+        }
+
         consoleLines.current.push(line)
+
         forceUpdate()
     }
 
@@ -107,7 +118,7 @@ export const useScaffold = (): Scaffold => {
             )
             matchPID.current = newPID
         } catch (e: any) {
-            consoleLines.current.push({ content: e, type: 'error' })
+            consoleLines.current.push({ content: e, type: 'error', matchIdx: 0 })
         }
         forceUpdate()
     }
@@ -176,15 +187,15 @@ export const useScaffold = (): Scaffold => {
 
         nativeAPI.child_process.onStdout(({ pid, data }) => {
             if (pid !== matchPID.current) return
-            log({ content: data, type: 'output' })
+            log({ content: data, type: 'output', matchIdx: 0 })
         })
         nativeAPI.child_process.onStderr(({ pid, data }) => {
             if (pid !== matchPID.current) return
-            log({ content: data, type: 'error' })
+            log({ content: data, type: 'error', matchIdx: 0 })
         })
         nativeAPI.child_process.onExit(({ pid, code, signal }) => {
             if (pid !== matchPID.current) return
-            log({ content: `Exited with code ${code} | ${JSON.stringify(signal)}`, type: 'bold' })
+            log({ content: `Exited with code ${code} | ${JSON.stringify(signal)}`, type: 'bold', matchIdx: 0 })
             matchPID.current = undefined
             forceUpdate()
         })

client/src/playback/GameRunner.ts

@@ -102,12 +102,12 @@ class GameRunnerClass {
         })
     }
 
-    setMatch(match: Match | undefined): void {
+    setMatch(match: Match | undefined, round: number = 1): void {
         this._trigger(this._matchListeners)
         if (match) {
             match.game.currentMatch = match
             this.setGame(match.game)
-            match._jumpToStart()
+            match._jumpToRound(round)
             GameRenderer.render()
         }
         this.setPaused(true)