A still editor has nothing to draw, so the frame loop wakes on change (#44)

The renderer is taken off the ticker; the ticker itself keeps running, so
the clock, a live world and the panels still get their step. Drawing now
happens inside a 200 ms trailing window that every pointer, wheel and key
event, every Widget.invalidate/resize, every Stack.relayout, every
EditorState.emit and both library onChange hooks reopen. The window rather
than a single frame is what makes the ~70 direct redraw() calls that never
pass through invalidate() safe, along with repaints an event only starts:
a microtask, a texture upload, a resolved font.

Editor.movesByItself() keeps asking for every frame while Play or an editor
run is on, a time shader is on screen, or the particle or animation preview
is showing.

The FPS counter now counts drawn frames over the time the renderer was
awake. Idle seconds are not slow seconds, so it answers "how smooth while
it moves" and holds its last answer instead of falling to 0 at rest.

Measured on the built stand, 20 s per variant, CPU of all chromium
processes from /proc: kept awake 75.5 % of a core at 60 fps, render on
change 8.8 % with 0 frames drawn, awake again 83.9 % as an order probe.

No colour check can see a missing repaint: pixelAt extracts from the stage
on demand and is just as green on a screen frozen a minute ago. Step 33
watches the new __gm.renderStats() counter instead.

src/editor/App.ts

@@ -2,7 +2,7 @@ import type { Application } from 'pixi.js';
 import { HttpFontStore } from '../core/store/HttpFontStore';
 import { HttpProjectStore } from '../core/store/HttpProjectStore';
 import { HttpTextureStore } from '../core/store/HttpTextureStore';
-import type { VStack } from '../ui';
+import { requestRender, type VStack } from '../ui';
 import { fontLibrary } from '../viewport/FontLibrary';
 import { textureLibrary } from '../viewport/TextureLibrary';
 import { installDebugHook } from './debugHook';
@@ -31,6 +31,10 @@ export async function startEditor(app: Application, root: VStack): Promise<void>
     new HttpTextureStore(API_ENDPOINT),
     new HttpFontStore(API_ENDPOINT),
   );
+  // An image or a font arrives long after the click that asked for it, and the widget waiting for
+  // it repaints without telling anyone: both libraries are a wake-up of their own.
+  textureLibrary.onChange(() => requestRender());
+  fontLibrary.onChange(() => requestRender());
   installDebugHook(app, state, editor);
   await editor.boot();
 }

src/editor/Editor.ts

@@ -5,7 +5,19 @@ import type { ProjectStore } from '../core/store/ProjectStore';
 import type { FontStore } from '../core/store/FontStore';
 import type { TextureStore } from '../core/store/TextureStore';
 import { gameClock } from '../core/time';
-import { clamp, HStack, Panel, Splitter, Theme, ui, VStack, type Widget } from '../ui';
+import {
+  clamp,
+  HStack,
+  isRendering,
+  Panel,
+  renderCount,
+  requestRender,
+  Splitter,
+  Theme,
+  ui,
+  VStack,
+  type Widget,
+} from '../ui';
 import { TabStrip, type TabPlace } from './TabStrip';
 import { ConsolePanel } from './console/ConsolePanel';
 import { GameViewport } from './viewport/GameViewport';
@@ -125,6 +137,7 @@ export class Editor implements PlayHost {
   private readonly tabs: TabStrip<CentreTab>;
   private readonly panes: Record<CentreTab, Widget>;
   private readonly toolbar: Toolbar;
+  /** Drawn-frame count at the last measurement, not a counter of its own. */
   private fpsFrames = 0;
   private fpsElapsed = 0;
   private leftWidth = 250;
@@ -331,14 +344,37 @@ export class Editor implements PlayHost {
     this.console.refresh();
     if (this.tab === 'animation') this.animationEditor.tick(step);
     if (this.tab === 'particles') this.particlesEditor.tick();
+    if (this.movesByItself()) requestRender();
   }
 
+  /**
+   * What changes without anyone touching it, and therefore cannot be woken by an event: the
+   * running world, a Time shader on screen, the two previews that are moving pictures.
+   */
+  private movesByItself(): boolean {
+    return (
+      this.play.isPlaying ||
+      this.editorRun.isRunning ||
+      (this.viewport.visible && this.viewport.animates) ||
+      (this.gameView.visible && this.gameView.animates) ||
+      this.tab === 'particles' ||
+      (this.tab === 'animation' && this.animationEditor.isPlaying)
+    );
+  }
+
+  /**
+   * Counts drawn frames over the time the renderer was awake. Idle seconds are not slow seconds:
+   * since the editor stopped drawing what nobody changed, counting them would drag the readout to
+   * zero while everything is fine. It answers "how smooth is it while it moves", and holds its
+   * last answer in between.
+   */
   private measureFps(dt: number): void {
-    this.fpsFrames += 1;
+    if (!isRendering()) return;
     this.fpsElapsed += dt;
     if (this.fpsElapsed < 0.5) return;
-    this.fps = Math.round(this.fpsFrames / this.fpsElapsed);
-    this.fpsFrames = 0;
+    const drawn = renderCount();
+    this.fps = Math.round((drawn - this.fpsFrames) / this.fpsElapsed);
+    this.fpsFrames = drawn;
     this.fpsElapsed = 0;
     this.toolbar.setFps(this.fps);
   }

src/editor/EditorRun.ts

@@ -80,6 +80,11 @@ export class EditorRun {
     return findAsset(this.state.project, selection.assetId)?.type === 'variables';
   }
 
+  /** Whether a world is up – what tells the editor its picture moves on its own. */
+  get isRunning(): boolean {
+    return this.world !== null;
+  }
+
   /** Facts for the check scripts: object ids of the behaviors currently running without Play. */
   debugInfo(): { running: boolean; time: number; behaviors: string[]; keyboard: boolean } {
     return {

src/editor/animation/AnimationEditor.ts

@@ -27,6 +27,11 @@ export class AnimationEditor extends AssetTabEditor<AnimationAsset, AnimationCan
     this.canvas.tick(dt);
   }
 
+  /** Whether the preview is running – a moving picture needs a frame without an event. */
+  get isPlaying(): boolean {
+    return this.canvas.playing;
+  }
+
   protected override selectedId(): string {
     return this.state.selectedAnimationId ?? '';
   }

src/editor/debugHook.ts

@@ -4,7 +4,7 @@ import { resolveObjects } from '../core/model/instance';
 import { findObjectHolder } from '../core/model/Project';
 import type { FrameRect } from '../core/model/texture';
 import { effectiveScale, gameClock } from '../core/time';
-import { activeDrag, Button, ui, Widget } from '../ui';
+import { activeDrag, Button, isRendering, renderCount, requestRender, ui, Widget } from '../ui';
 import { particleLibrary } from '../viewport/ParticleLibrary';
 import { shaderLibrary } from '../viewport/ShaderLibrary';
 import { textureLibrary } from '../viewport/TextureLibrary';
@@ -450,6 +450,14 @@ export function installDebugHook(app: Application, state: EditorState, editor: E
       };
     },
     fps: () => editor.fps,
+    /**
+     * What the render loop did: frames actually drawn, and whether it is currently awake. The
+     * only way to see a missing repaint from a check script – `pixelAt` re-renders on demand and
+     * would answer correctly even if the screen had been frozen for a minute.
+     */
+    renderStats: () => ({ drawn: renderCount(), awake: isRendering() }),
+    /** Draw again, whatever happens next – what a screenshot needs before it is taken. */
+    wake: () => requestRender(),
     /** Visible nodes carrying a filter – one extra render pass each; since #77 a shader is none. */
     filterCount: () => {
       let n = 0;

src/editor/state.ts

@@ -13,6 +13,7 @@ import {
   type Project,
 } from '../core/model/Project';
 import { deserializeProject, serializeProject } from '../core/store/serialize';
+import { requestRender } from '../ui';
 import { consoleLog } from './console/log';
 
 export type Selection =
@@ -128,6 +129,9 @@ export class EditorState {
   }
 
   emit(event: EditorEvent): void {
+    // The one choke point every model change passes: what a listener repaints is drawn even when
+    // it repaints without going through `invalidate()`.
+    requestRender();
     for (const listener of this.listeners.get(event) ?? []) listener();
   }
 

src/main.ts

@@ -5,7 +5,7 @@ import { Application, Container } from 'pixi.js';
 import { captureConsole } from './editor/console/log';
 import { startEditor } from './editor/App';
 import { mountPlayground } from './playground';
-import { initUI, Theme, VStack } from './ui';
+import { initRenderLoop, initUI, Theme, VStack } from './ui';
 
 // Before anything else: a renderer warns while it is being created, and those warnings are
 // exactly the ones worth having in the panel.
@@ -45,6 +45,8 @@ async function main(): Promise<void> {
   app.stage.eventMode = 'static';
   app.stage.hitArea = app.screen;
   initUI(app, root, overlay);
+  // From here on the picture is drawn when something changes, not sixty times a second.
+  initRenderLoop(app);
 
   if (params.has('playground')) mountPlayground(app, root);
   else await startEditor(app, root);

src/ui/UIContext.ts

@@ -1,6 +1,7 @@
 import type { Application, Container } from 'pixi.js';
 import { FocusManager } from './FocusManager';
 import { KeyboardBridge } from './KeyboardBridge';
+import { requestRender } from './render';
 
 /** Services shared by all widgets, created once by `initUI()`. */
 export interface UIContext {
@@ -22,6 +23,15 @@ export function initUI(app: Application, root: Container, overlay: Container): U
   // Clicking the canvas must not steal DOM focus from the keyboard bridge input.
   app.canvas.addEventListener('mousedown', (e) => e.preventDefault());
   app.canvas.addEventListener('contextmenu', (e) => e.preventDefault());
+  // Anything the user does may change the picture, and in far more places than the ones that
+  // remember to invalidate: one wake-up at the door covers all of them (see `render.ts`).
+  // Listened for on the window rather than on the stage: a release or a key that arrives while
+  // the pointer sits outside the canvas still ends a drag or an edit.
+  const wake = () => requestRender();
+  for (const type of ['pointerdown', 'pointermove', 'pointerup', 'pointercancel', 'wheel'] as const)
+    window.addEventListener(type, wake, { capture: true, passive: true });
+  for (const type of ['keydown', 'keyup', 'focus', 'blur'] as const)
+    window.addEventListener(type, wake, true);
   return context;
 }
 

src/ui/Widget.ts

@@ -1,4 +1,5 @@
 import { Container, Rectangle } from 'pixi.js';
+import { requestRender } from './render';
 
 export interface Size {
   w: number;
@@ -44,6 +45,7 @@ export abstract class Widget extends Container {
     this.hitArea = new Rectangle(0, 0, w, h);
     if (changed) this.layout();
     this.redraw();
+    requestRender();
   }
 
   /** Position children. Called after a resize; must be cheap and idempotent. */
@@ -53,6 +55,7 @@ export abstract class Widget extends Container {
 
   /** Request a repaint at the end of the current frame (coalesced). */
   invalidate(): void {
+    requestRender();
     if (this.dirty) return;
     this.dirty = true;
     queueMicrotask(() => {

src/ui/index.ts

@@ -2,6 +2,7 @@ export { Widget, Box, type Size } from './Widget';
 export { HStack, VStack, Spacer, Stack, type StackOptions } from './layout/Stack';
 export { Theme, textStyle, type TextStyleToken } from './Theme';
 export { initUI, ui, type UIContext } from './UIContext';
+export { initRenderLoop, isRendering, renderCount, requestRender, resetRenderLoop } from './render';
 export { drawBand, drawControlBox, drawGrid, type ControlBoxOptions, type GridOptions } from './draw';
 export { clamp, centreY, formatBytes, formatNumber, NUMBER_DECIMALS, snapToStep, zoomFactor } from './math';
 export { activeDrag, type DragPayload, endDrag, onDragChange, startDrag } from './drag';

src/ui/layout/Stack.ts

@@ -1,3 +1,4 @@
+import { requestRender } from '../render';
 import { type Size, Widget } from '../Widget';
 
 export interface StackOptions {
@@ -92,6 +93,8 @@ export abstract class Stack extends Widget {
   relayout(): void {
     this.layout();
     this.redraw();
+    // A stack that only moves children around resizes none of them, so this is its own wake-up.
+    requestRender();
   }
 }
 

src/ui/render.ts

@@ -0,0 +1,63 @@
+import { type Application, UPDATE_PRIORITY } from 'pixi.js';
+
+/**
+ * Drives the renderer from changes instead of from the clock.
+ *
+ * An idle editor at 60 fps costs ~75 % of a core, and ~64 of those points are the drawing alone –
+ * a picture nobody asked to change. The ticker keeps running (~3 points): the clock, a running
+ * world and the panels still get their frame, only `app.render()` is skipped while nothing asked
+ * for it.
+ *
+ * What wakes it is deliberately not a strict dirty flag. Every pointer and keyboard event, every
+ * `Widget.invalidate()`/`resize()`, every `EditorState.emit()` and every texture or font arriving
+ * opens a short trailing window rather than requesting exactly one frame. A repaint that lands a
+ * few milliseconds after its cause – a microtask, a texture upload, a font resolving – is drawn
+ * without anyone having had to declare it, which is the failure this design refuses to have:
+ * a missing repaint is invisible to the check scripts, because `pixelAt` re-renders on demand.
+ */
+
+/** How long one wake-up keeps drawing. Long enough for the late repaint, short enough to idle. */
+const WAKE_MS = 200;
+
+let deadline = 0;
+let rendered = 0;
+
+/** Draw, and keep drawing for a moment. Cheap enough to call from every event. */
+export function requestRender(): void {
+  deadline = performance.now() + WAKE_MS;
+}
+
+/** Whether the renderer is currently awake. */
+export function isRendering(): boolean {
+  return performance.now() <= deadline;
+}
+
+/** Frames actually drawn since the loop was installed – the fps readout and the checks read it. */
+export function renderCount(): number {
+  return rendered;
+}
+
+/**
+ * Takes the renderer off the ticker and drives it from `requestRender()` instead. Added at the
+ * ticker's lowest priority, so everything that may request a frame has run for this tick already.
+ */
+export function initRenderLoop(app: Application): void {
+  // `TickerPlugin` registers `Application.prototype.render` bound to the app; same pair to remove.
+  app.ticker.remove(app.render, app);
+  app.ticker.add(
+    () => {
+      if (!isRendering()) return;
+      rendered += 1;
+      app.render();
+    },
+    undefined,
+    UPDATE_PRIORITY.LOW,
+  );
+  requestRender();
+}
+
+/** Tests only: back to a cold loop. */
+export function resetRenderLoop(): void {
+  deadline = 0;
+  rendered = 0;
+}