Skip to content

refactor(benchmarks): route scenes through workload controllers - #20

Merged
thejustinwalsh merged 3 commits into
feat/benchmark-workload-scenesfrom
feat/benchmark-workload-controllers
Aug 4, 2026
Merged

refactor(benchmarks): route scenes through workload controllers#20
thejustinwalsh merged 3 commits into
feat/benchmark-workload-scenesfrom
feat/benchmark-workload-controllers

Conversation

@thejustinwalsh

@thejustinwalsh thejustinwalsh commented Aug 3, 2026

Copy link
Copy Markdown
Collaborator

Summary

  • make every retained workload definition own typed app-private scene construction and layout hooks
  • move Icon Grid layout, virtual-window calculation, assignment validation, recycled positioning, pan integration, and frame-delta smoothing beside its canonical Text example
  • derive Main and Presentation descriptions, amount labels, font behavior, preload grouping, and pan/zoom policy from the typed catalog
  • keep renderer, canvas, RAF, GPU timer, telemetry, font transactions, active pool ownership, and teardown in the persistent host

Why

The workload files already contained the canonical public Text examples, but the retained renderer still switched on workload IDs to construct and lay out them. The UI also repeated workload policy independently of the typed catalog. This layer makes the file hierarchy an executable boundary without inventing a public framework API or moving host lifecycle ownership into scene code.

Verification

  • mise exec -- pnpm benchmarks:check
    • 51 test files / 285 tests
    • TypeScript, Oxlint, Oxfmt, script contracts, size gates, and production build pass
  • focused workload/controller suite: 5 files / 64 tests
  • mise exec -- pnpm --filter @pmndrs/text-benchmarks test:presentation-workloads
    • all 42 Bitmap/MTSDF/Slug × WebGPU/WebGL2 × seven-workload cells rendered visible pixels
    • exactly one renderer in every backend/technique lane
    • no reported console or page failure
  • mise exec -- pnpm docs:check: zero conformance errors, producer-profile errors, or warnings
  • all three commits are SSH-signed by Justin Walsh <contact.me@thejustinwalsh.com>

Stack

Depends on #19.

@thejustinwalsh thejustinwalsh changed the title feat/benchmark workload controllers refactor(benchmarks): route scenes through workload controllers Aug 3, 2026
@thejustinwalsh
thejustinwalsh marked this pull request as ready for review August 3, 2026 15:57
@thejustinwalsh
thejustinwalsh force-pushed the feat/benchmark-workload-controllers branch from 6388933 to cb4f0c4 Compare August 4, 2026 01:44
@thejustinwalsh
thejustinwalsh merged commit 54b8742 into main Aug 4, 2026
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant