Artemis: Stop throwing 429 errors

We already throttle requests when we get 429s. We don't need to log it constantly too.

server/artemis/remote.ts

@@ -57,7 +57,7 @@ let throttleTime: number | null = null;
 export const limiter = new Limiter(800);
 export const PM = new ProcessManager.QueryProcessManager<string, Record<string, number> | null>(module, async text => {
 	if (isCommon(text) || !limiter.shouldRequest()) return null;
-	if (throttleTime && (Date.now() - throttleTime < 10000)) {
+	if (throttleTime && ((Date.now() - throttleTime) < 10000)) {
 		return null;
 	}
 	if (throttleTime) throttleTime = null;
@@ -90,9 +90,9 @@ export const PM = new ProcessManager.QueryProcessManager<string, Record<string,
 		return result;
 	} catch (e: any) {
 		throttleTime = Date.now();
-		if (e.message.startsWith('Request timeout')) {
-			// just ignore this. error on their end not ours.
-			// todo maybe stop sending requests for a bit?
+		if (e.message.startsWith('Request timeout') || e.code === 429) {
+			// request timeout: just ignore this. error on their end not ours.
+			// 429: too many requests, we already freeze for 10s above so. not much more we can do
 			return null;
 		}
 		Monitor.crashlog(e, 'A Perspective API request', {request: JSON.stringify(requestData)});

server/room-battle.ts

@@ -442,6 +442,8 @@ export class RoomBattleTimer {
 		let didSomething = false;
 		for (const player of players) {
 			if (!player.id) continue; // already eliminated, relevant for FFA gamesif it
+			// https://play.pokemonshowdown.com/battle-gen9unratedrandombattle-2255606027-5a6bcd9zlb93e6id5pp7juvhcg5w41spw
+			// why is this line here?
 			if (player.turnSecondsLeft > 0) continue;
 			if (this.settings.timeoutAutoChoose && player.secondsLeft > 0 && player.knownActive) {
 				void this.battle.stream.write(`>${player.slot} default`);