@tmustier/pi-nes 0.2.27 → 0.2.29

package/extensions/nes/index.ts

@@ -172,7 +172,7 @@ async function configureWithWizard(
 	const imageQuality = isHighQuality ? "high" : "balanced";
 
 	const filterOptions: Array<{ label: string; value: VideoFilter }> = [
-		{ label: "Sharp — pixel-perfect, no filtering", value: "off" },
+		{ label: "Sharp (default) — pixel-perfect, no filtering", value: "off" },
 		{ label: "CRT Soft — subtle retro look", value: "ntsc-rgb" },
 		{ label: "CRT Classic — authentic scanlines + color bleed", value: "ntsc-composite" },
 	];

package/extensions/nes/nes-session.ts

@@ -45,6 +45,7 @@ export class NesSession {
 	private readonly maxCatchUpFrames: number;
 	private renderHook: (() => void) | null = null;
 	private tickTimer: ReturnType<typeof setInterval> | null = null;
+	private renderTimer: ReturnType<typeof setInterval> | null = null;
 	private saveTimer: ReturnType<typeof setInterval> | null = null;
 	private saveInFlight = false;
 	private stopping = false;
@@ -109,6 +110,8 @@ export class NesSession {
 			this.tick();
 		}, this.frameIntervalMs);
 
+		this.startRenderTimer();
+
 		this.saveTimer = setInterval(() => {
 			void this.saveIfNeeded(false);
 		}, this.saveIntervalMs);
@@ -116,11 +119,20 @@ export class NesSession {
 
 	setRenderHook(hook: (() => void) | null): void {
 		this.renderHook = hook;
+		if (hook) {
+			this.startRenderTimer();
+		} else {
+			this.stopRenderTimer();
+		}
 	}
 
 	setRenderIntervalMs(intervalMs: number): void {
 		this.renderIntervalMs = Math.max(0, intervalMs);
 		this.stats.renderIntervalMs = this.renderIntervalMs;
+		if (this.renderTimer) {
+			this.stopRenderTimer();
+			this.startRenderTimer();
+		}
 	}
 
 	getStats(): NesSessionStats {
@@ -136,6 +148,7 @@ export class NesSession {
 			clearInterval(this.tickTimer);
 			this.tickTimer = null;
 		}
+		this.stopRenderTimer();
 		this.loopDelay.disable();
 		if (this.saveTimer) {
 			clearInterval(this.saveTimer);
@@ -145,6 +158,28 @@ export class NesSession {
 		this.core.dispose();
 	}
 
+	private startRenderTimer(): void {
+		if (this.renderTimer || !this.renderHook || this.stopping) {
+			return;
+		}
+		const intervalMs = this.renderIntervalMs > 0 ? this.renderIntervalMs : this.frameIntervalMs;
+		this.renderTimer = setInterval(() => {
+			if (!this.renderHook || this.stopping) {
+				return;
+			}
+			this.lastRenderTime = performance.now();
+			this.statsWindow.renders += 1;
+			this.renderHook();
+		}, Math.max(1, intervalMs));
+	}
+
+	private stopRenderTimer(): void {
+		if (this.renderTimer) {
+			clearInterval(this.renderTimer);
+			this.renderTimer = null;
+		}
+	}
+
 	private tick(): void {
 		if (this.stopping) {
 			return;
@@ -178,12 +213,6 @@ export class NesSession {
 				this.core.tick();
 			}
 
-			if (this.renderHook && now - this.lastRenderTime >= this.renderIntervalMs) {
-				this.lastRenderTime = now;
-				this.statsWindow.renders += 1;
-				this.renderHook();
-			}
-
 			const windowDuration = now - this.statsWindow.startTime;
 			if (windowDuration >= 1000) {
 				const seconds = windowDuration / 1000;

package/extensions/nes/renderer.ts

@@ -47,6 +47,7 @@ function getKittyShmModule(): KittyShmModule | null {
 }
 
 export class NesImageRenderer {
+
 	private readonly imageId = allocateImageId();
 	private cachedImage?: { base64: string; width: number; height: number };
 	private cachedRaw?: { sequence: string; columns: number; rows: number };

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@tmustier/pi-nes",
-  "version": "0.2.27",
+  "version": "0.2.29",
   "description": "NES emulator extension for pi",
   "keywords": [
     "pi-package",