Add real generation and interaction performance benchmarks with A/B comparison by SheepFromHeaven · Pull Request #1608 · Azgaar/Fantasy-Map-Generator

SheepFromHeaven · 2026-08-23T15:05:59Z

Adds Playwright-based performance benchmarks for real map generation and interaction, with an A/B comparison harness that runs base and head alternately in CI. Replaces #1601.

I also reworked the existing TIME-guarded console.time/console.timeEnd instrumentation into a timeStart/timeEnd helper that dispatches a perf:stage event alongside the console output, and extended map:generated to carry the total time. The generation spec listens to these events instead of parsing console text.

New pieces:

tests/perf/generation.spec.ts => generates real maps for fixed seeds and records per-stage timings
tests/perf/interaction.spec.ts => loads the newest tests/fixtures/*.map fixture and scripts a zoom/pan gesture
tests/perf/ab.mjs => builds and serves base and head from separate git worktrees, alternates rounds, compares median per-metric ratios
tests/perf/playwright.config.ts => points the perf specs at whichever server ab.mjs is currently serving
.github/workflows/perf.yml => runs the A/B comparison on PRs with read-only permissions
.github/workflows/perf-comment.yml => posts results as a PR comment from a trusted workflow_run context, so it works for fork PRs too
src/utils/perfEvents.ts => timeStart/timeEnd helpers

Note: since this PR introduces the perf suite, master has nothing to compare against until it merges, and the comment workflow won't activate until it exists on master either.

Per maintainer feedback on PR Azgaar#1601: replaces synthetic bench.ts fixtures (hand-built square lattices unlike real Voronoi-generated maps) with Playwright specs measuring actual generation (fixed seeds, per-stage TIME instrumentation already in every generator) and actual interaction (scripted zoom/pan over a real fixture .map file). Compares base vs head by alternating rounds on the same runner, per the earlier finding that a stored baseline can't survive shared-runner noise.

Confirmed locally (HEAD vs HEAD, single round): stages under ~1ms are dominated by JIT/GC jitter and can swing 40-75% between two runs of the exact same code. total/gesture-level metrics (tens-hundreds of ms) stay stable and are what should actually gate the check. Report sub-2ms stages in the comment for visibility, but exclude them from the regression determination.

tests/fixtures/*.map gets new versioned fixtures over time; hardcoding 1.143.1.map would need a manual bump each time. Pick the highest version by parsing the X.Y.Z filename instead.

Same bootstrap gap as the vitest-bench version had: when the base ref has no tests/perf/playwright.config.ts yet (true for this PR's own master), the previous code let execFileSync throw uncaught instead of degrading to the existing "no comparable metrics" path. Also catch Playwright's own non-zero exit generally and parse whatever PERF_RESULT lines made it to stdout, rather than losing partial results to one failed test.

Replaces every TIME-guarded console.time/console.timeEnd pair with timeStart/timeEnd (src/utils/perfEvents.ts), which still logs to the console for devtools users but also dispatches a perf:stage CustomEvent with the stage name and duration. map:generated now carries totalMs too. generation.spec.ts listens for these events directly instead of regex-parsing console message text, which is more robust (the earlier console-format assumption already broke once) and gives other consumers (e.g. a future perf overlay) a real API instead of scraping devtools output.

netlify · 2026-08-23T15:06:30Z

✅ Deploy Preview for afmg ready!

Name	Link
🔨 Latest commit	`7182bd9`
🔍 Latest deploy log	https://app.netlify.com/projects/afmg/deploys/6abcf5493330140008864089
😎 Deploy Preview	https://deploy-preview-1608--afmg.netlify.app
📱 Preview on mobile	Toggle QR Code... Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify project configuration.

Copilot

🟡 Changes recommended

The perf suite and comment workflow have a few correctness issues (Playwright test timeout, artifact-download permissions, and robustness of result parsing/event emission) that can cause CI failures or incomplete benchmark reporting.

Once you've addressed the issues Copilot identified, you can request another Copilot review.

Pull request overview

Adds a Playwright-based performance benchmarking suite that measures real map generation and scripted interactions, plus CI workflows to run an alternating A/B comparison between master and the PR head. It also replaces ad-hoc TIME && console.time* instrumentation with timeStart/timeEnd helpers that emit structured perf:stage events and extends map:generated with total generation time for reliable collection in tests.

Changes:

Introduces perf specs (generation + interaction) and a Node A/B harness (ab.mjs) that alternates base/head rounds and compares median metric ratios.
Adds timeStart/timeEnd + perf:stage event dispatch, and migrates many TIME-guarded timings to use the helper.
Adds GitHub Actions workflows to run benchmarks on PRs and post results as a sticky PR comment via workflow_run.

File summaries

File	Description
tests/perf/playwright.config.ts	Playwright config for perf suite (single worker, fixed viewport, baseURL via env).
tests/perf/interaction.spec.ts	Interaction benchmark that loads latest `.map` fixture and measures zoom/pan gesture duration.
tests/perf/generation.spec.ts	Generation benchmark that records per-stage timings via `perf:stage` and total via `map:generated`.
tests/perf/ab.mjs	A/B harness that builds/serves base+head, alternates runs, parses `PERF_RESULT`, compares medians.
src/utils/perfEvents.ts	Adds `timeStart`/`timeEnd` helpers and `perf:stage` event emission.
src/utils/index.ts	Exposes `timeStart`/`timeEnd` on `window` and exports via utils barrel.
src/utils/graphUtils.ts	Switches timing instrumentation to `timeStart`/`timeEnd`.
src/services/io/export.ts	Switches export timings to `timeStart`/`timeEnd`.
src/services/io/export-json.ts	Switches JSON export timing to `timeStart`/`timeEnd`.
src/renderers/labels/labels-renderer.ts	Switches label draw timing to `timeStart`/`timeEnd`.
src/renderers/erosion-bake.ts	Switches erosion bake timing to `timeStart`/`timeEnd`.
src/renderers/draw-temperature.ts	Switches temperature render timing to `timeStart`/`timeEnd`.
src/renderers/draw-states.ts	Switches states render timing to `timeStart`/`timeEnd`.
src/renderers/draw-scalebar.ts	Switches scalebar render timing to `timeStart`/`timeEnd` (guarded on first render).
src/renderers/draw-routes.ts	Switches routes render timing to `timeStart`/`timeEnd`.
src/renderers/draw-rivers.ts	Switches rivers render timing to `timeStart`/`timeEnd`.
src/renderers/draw-religions.ts	Switches religions render timing to `timeStart`/`timeEnd`.
src/renderers/draw-relief-icons.ts	Switches relief render timing to `timeStart`/`timeEnd`.
src/renderers/draw-provinces.ts	Switches provinces render timing to `timeStart`/`timeEnd`.
src/renderers/draw-precipitation.ts	Switches precipitation render timing to `timeStart`/`timeEnd`.
src/renderers/draw-ocean.ts	Switches ocean render timing to `timeStart`/`timeEnd`.
src/renderers/draw-military.ts	Switches military render timing to `timeStart`/`timeEnd`.
src/renderers/draw-markets.ts	Switches markets render timing to `timeStart`/`timeEnd`.
src/renderers/draw-markers.ts	Switches markers render timing to `timeStart`/`timeEnd`.
src/renderers/draw-landmass.ts	Switches landmass render timing to `timeStart`/`timeEnd`.
src/renderers/draw-ice.ts	Switches ice render timing to `timeStart`/`timeEnd`.
src/renderers/draw-heightmap.ts	Switches heightmap render timing to `timeStart`/`timeEnd`.
src/renderers/draw-goods.ts	Switches goods render timing to `timeStart`/`timeEnd`.
src/renderers/draw-emblems.ts	Switches emblems render timing to `timeStart`/`timeEnd` including async redraw path.
src/renderers/draw-cultures.ts	Switches cultures render timing to `timeStart`/`timeEnd`.
src/renderers/draw-burg-icons.ts	Switches burg icons render timing to `timeStart`/`timeEnd`.
src/renderers/draw-borders.ts	Switches borders render timing to `timeStart`/`timeEnd`.
src/renderers/draw-biomes.ts	Switches biomes render timing to `timeStart`/`timeEnd`.
src/generators/zones-generator.ts	Switches zones generation timing to `timeStart`/`timeEnd`.
src/generators/states-generator.ts	Switches multiple states generator stage timings to `timeStart`/`timeEnd`.
src/generators/routes-generator.ts	Switches routes generation timings to `timeStart`/`timeEnd`.
src/generators/river-generator.ts	Switches river generation timing to `timeStart`/`timeEnd`.
src/generators/religions-generator.ts	Switches religions generation timing to `timeStart`/`timeEnd`.
src/generators/relief-generator.ts	Switches relief generation timing to `timeStart`/`timeEnd`.
src/generators/provinces-generator.ts	Switches provinces generation timing to `timeStart`/`timeEnd`.
src/generators/production-generator.ts	Switches production timing to `timeStart`/`timeEnd`.
src/generators/ocean-generator.ts	Switches ocean generation timing to `timeStart`/`timeEnd`.
src/generators/military-generator.ts	Switches military generation timing to `timeStart`/`timeEnd`.
src/generators/measurers-generator.ts	Switches default ruler creation timing to `timeStart`/`timeEnd`.
src/generators/markets-generator.ts	Switches markets generation timing to `timeStart`/`timeEnd`.
src/generators/markers-generator.ts	Switches marker generation timing to `timeStart`/`timeEnd`.
src/generators/heightmap-generator.ts	Switches heightmap definition timing to `timeStart`/`timeEnd`.
src/generators/goods-generator.ts	Switches goods generation/regeneration timing to `timeStart`/`timeEnd`.
src/generators/features.ts	Switches feature markup timings to `timeStart`/`timeEnd`.
src/generators/cultures-generator.ts	Switches cultures generation/expansion timings to `timeStart`/`timeEnd`.
src/generators/burgs-generator.ts	Switches burg generation/specification timings to `timeStart`/`timeEnd`.
src/generators/biomes-generator.ts	Switches biomes definition timing to `timeStart`/`timeEnd`.
src/controllers/heightmap-editor.ts	Switches heightmap editor regeneration/restore timings to `timeStart`/`timeEnd`.
public/main.js	Adds `totalMs` computation, uses `timeStart`/`timeEnd` for stages, extends `map:generated` detail with `totalMs`.
package.json	Adds `perf:ab` script entry.
.gitignore	Ignores generated `perf-report/` outputs.
.github/workflows/perf.yml	Runs A/B perf benchmarks on PRs and uploads a comment artifact.
.github/workflows/perf-comment.yml	Posts perf results comment from `workflow_run` context using sticky PR comments.

Review details

Files reviewed: 57/58 changed files
Comments generated: 5
Review effort level: Lite

💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.

- tests/perf/playwright.config.ts: set an explicit 180s test/expect timeout so the 120s waits in the perf specs don't hit Playwright's default 30s timeout. - tests/perf/ab.mjs: skip malformed PERF_RESULT lines instead of letting JSON.parse abort the whole A/B run. - src/utils/perfEvents.ts: check for a matching timeStart before calling console.timeEnd (avoids a noisy "No such label" warning) and guard the perf:stage dispatch behind a window check. - public/main.js: only include totalMs in the map:generated detail when it's a number, since showStatistics() is also called from load/resample paths with no total to report. - .github/workflows/perf-comment.yml: add actions: read so actions/download-artifact can read the triggering run's artifacts.

…rktrees Adopts two ideas from barrulus's parallel implementation (barrulus:perf/playwright-ab): - generation.spec.ts now hashes the generated pack (cells, burgs) per seed via FNV-1a and reports it alongside timings. ab.mjs compares checksums across rounds: if a side disagrees with itself, that's a pre-existing determinism bug reported as a warning; if base and head are each internally consistent but differ from each other, that's a real correctness regression and fails the run regardless of timing. - ab.mjs copies the perf spec files from the head worktree into the base worktree before running, so both sides run identical measurement code. This also fixes the bootstrap gap where base (predating this PR) had no perf suite to compare against: base now runs head's specs against its own application code, so a real A/B comparison against master works even before this PR merges. - generation.spec.ts falls back to parsing the TOTAL/stage console lines when the map:generated event or perf:stage events aren't present, since base's application code may predate that instrumentation (true for master until this PR merges).

The perf.yml job on PR Azgaar#1608 ran all 3 rounds in under 2 minutes (per the log) but then hung until the 60-minute job timeout killed it. spawn()'d the preview server via "npm run preview", which itself spawns vite preview as a further child process; killing the npm process doesn't reliably kill that grandchild, and an unreferenced child handle keeps Node's event loop (and the whole script) alive even after all script logic has finished. Spawn the server detached so it leads its own process group, kill the whole group with process.kill(-pid) instead of child.kill(), and unref() the handle so it can't keep the process alive by itself. Also add explicit process.exit() calls on every path as a hard guarantee. Verified locally: after this fix, ab.mjs exits immediately once reporting is done and leaves no process bound to the preview ports.

The hang fixed in the previous commit burned a full hour of CI time before the job timeout caught it. The actual A/B comparison (build x2 + 3 rounds) takes a couple of minutes locally; 20 minutes leaves generous headroom for a slower CI runner without masking a stuck job for an hour.

CI run on PR Azgaar#1608 flagged markupGrid (+46.5%, spread 51.5%), placePoints (+32.6%, spread 71.4%) and drawRoutes (+26.5%, spread 18.2%) as regressions with 3 rounds. No checksum mismatch, so generation output was identical; these are few-millisecond stages whose relative noise on a shared runner swamps the signal even above the earlier 2ms floor. Only gate on the per-seed `total` and the interaction `gesture` metric, which have stayed stable (single-digit % or better) across every real comparison run so far, local and CI. Per-stage timings are still reported for visibility but can no longer fail the build on their own. Verified against real origin/master: same false positives from the previous run no longer trip the gate, and the run still passes overall.

Wiki update

…release-branches ci: run PR checks on release branches

* fix: cap workers to 1000 * fix: don't redraw on non-zoom action like click * init * init * fixes * fixes

Azgaar · 2026-08-25T14:51:50Z

@SheepFromHeaven , are we done here, ready to merge?

* first try * Ocean embellishment * haching * lakes * cinderwood start * cleanup * icons * icons update * coastal bands * frostbite preset * fix * fixes * optimization

…ap (Azgaar#1855) Pre-v1.150 main.js initialised the legacy style record as burgIcons: {}, anchors: {}, and the v1.150 migration took an empty object as a present record: it stored it empty and skipped the svg harvest that still held the group sizes. Nothing could refill it afterwards: the renderer emitted no font-size, the style editor showed 1 and dropped every edit to a missing group, and ensureBurgGroupStyles had no template to copy. The migration now harvests an empty record from the svg, load refills an already-migrated empty record the same way and seeds any burg group without a style entry, and ensureBurgGroupStyles falls back to the shipped defaults when a record has no group to copy.

) * fix: count remaining manufacturing work from available stock * style: trim production comments and reuse record guard * test: cover freighter production with twenty engines

* Resolving Memory leaks (Azgaar#1852) * Fix state creation and province recolouring refresh (Azgaar#1848) * fix: refresh state creation and province recolouring * fix: preserve hidden layers when refreshing state edits * fix: respect goods tag filters in bulk visibility changes (Azgaar#1849) * fix: add name field to the marker editor (Azgaar#1851) Co-authored-by: Claude Opus 5 <noreply@anthropic.com> * investigation doc * mamory leaks fixes --------- Co-authored-by: barrulus <b@rry.im> Co-authored-by: Claude Opus 5 <noreply@anthropic.com> * fix: skip removed states when vassals copy suzerain relations (Azgaar#1854) A removed state is stored as {i, removed: true} with no diplomacy array. generateDiplomacy skips removed states in its outer loops, but the vassal-copy loops iterate every index, so the first vassal processed throws a TypeError on the removed state's missing diplomacy. Generation aborts partway, leaving every later row at the "x" placeholder, which the editor now shows as "Invalid" and can no longer regenerate. * Omnibar (Azgaar#1850) * init * omnibar * entities and commands * goods * style and export commands * seach button in menu * cleanup * cleanup * fix tests * fixes * fixes * fixes * Azgaar Assistant trigger * organize * cleanup * Wrap tool (Azgaar#1831) * init * wrap tool * coastline - local roughness --------- Co-authored-by: Claude Opus 5 <noreply@anthropic.com> * fix: stop an empty legend redrawing as an "undefined" row (Azgaar#1844) * fix: stop an empty legend redrawing as an "undefined" row A legend box with no items - the Zones editor with everything filtered out - stores an empty string in its data attribute. Splitting that on "|" yields one blank row rather than none, so the next redraw drew a swatch with fill "undefined" and the label "undefined" under the title. Co-Authored-By: Claude Opus 5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01QDnWHpXXkGnY61DPMSL9Ms * feat: report an empty legend instead of drawing an empty box Toggling a legend for a list with nothing in it drew a box holding only its title. Each editor now reports it the way the overviews do and draws nothing, so the feedback arrives when the button is pressed rather than later, when a style change happens to redraw the box. The parse fix stays underneath: a map saved before this can already hold an empty box, and that still has to redraw as empty rather than as a blank row. Co-Authored-By: Claude Opus 5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01QDnWHpXXkGnY61DPMSL9Ms --------- Co-authored-by: Claude Opus 5 <noreply@anthropic.com> * Remove vertex editing from the Lakes Editor (Azgaar#1858) Lake shores are reshaped only with the Wrap tool now; the per-vertex drag handles and their #debug overlay are gone from the Lake editor. Co-authored-by: Claude Opus 5 <noreply@anthropic.com> * Features overview (Azgaar#1859) * init * improve the ui * ocean hightlight * coastline for features * coastline editor tweaks * styling * update docs * feature names * ocean name * fixes * fixes * fix * fixes * dialog state - reset button (Azgaar#1861) * performance settings (Azgaar#1862) * Ink style (Azgaar#1864) * first try * Ocean embellishment * haching * lakes * cinderwood start * cleanup * icons * icons update * coastal bands * frostbite preset * fix * fixes * optimization * cleanup * fixes * update notes * fix * bring old update posts for better new updates (more human-created data) * fix stale wiki * fix test * fix: refill empty burg icon and anchor style records from the saved map (Azgaar#1855) Pre-v1.150 main.js initialised the legacy style record as burgIcons: {}, anchors: {}, and the v1.150 migration took an empty object as a present record: it stored it empty and skipped the svg harvest that still held the group sizes. Nothing could refill it afterwards: the renderer emitted no font-size, the style editor showed 1 and dropped every edit to a missing group, and ensureBurgGroupStyles had no template to copy. The migration now harvests an empty record from the svg, load refills an already-migrated empty record the same way and seeds any burg group without a style entry, and ensureBurgGroupStyles falls back to the shipped defaults when a record has no group to copy. * drainage (Azgaar#1868) * fixes * fix: preserve cascading column sorting in overview tables (Azgaar#1869) * fix: use available stock when planning manufacturing chains (Azgaar#1867) * fix: count remaining manufacturing work from available stock * style: trim production comments and reuse record guard * test: cover freighter production with twenty engines * fix doc * fix tests --------- Co-authored-by: barrulus <b@rry.im> Co-authored-by: Claude Opus 5 <noreply@anthropic.com> Co-authored-by: Eric Sullivan <56692984+esullivan9@users.noreply.github.com>

…gaar#1877) * fix: keep zoomTo and its flight inside the map extents d3 clamps gestures to the scale and translate extents but applies `zoom.transform` verbatim, so zoomTo could centre a point next to the border and expose the black background. The animated flight was worse: interpolateZoom zooms out and pans through views far outside the map even when both endpoints are inside it. Constrain the landing transform and wrap the interpolator so every planned frame is pushed through the same constrain function. Co-Authored-By: Claude Opus 5 <noreply@anthropic.com> * fix: limit full offline prefetch to installed PWA * test: align PWA fixture loading with Playwright module format * docs: complete the 1.153.0 release guides * fix: update vulnerable Vitest and js-yaml dependencies * fix: reuse retained PWA assets when old build files return 404 * fix: disable unsafe Quill HTML export formats * test: suppress update popup in diplomacy fixture * fix: refresh Nix dependency hash after security updates * test: use a smaller map for Goods filter browser checks --------- Co-authored-by: Azgaar <azgaar.fmg@yandex.com> Co-authored-by: Claude Opus 5 <noreply@anthropic.com>

…d value (Azgaar#1879) * fix: quote the ocean pattern href in the 1.61 migration Maps that had no ocean pattern before 1.61 got the literal text width="100" as the pattern image href, because the attribute was written unquoted and the parser took the next attribute as its value. The broken image drew nothing, so it passed for "no pattern" until 1.153.0 started forcing the image width and Chrome tiled its broken-image glyph over the whole ocean. Quote the attribute, and drop any stored pattern that is not an app image or a data URL so maps already re-saved with the bad href heal on load. Fixes Azgaar#1878 * fix: heal the broken ocean pattern in a migration step Move the repair out of the styles schema into a 1.153.2 migration: maps saved with the text width="100" as the ocean pattern get an empty pattern in the stored record and on the svg image. * chore: bump version to 1.153.1

* fix: make assignment paint brushes follow the pointer * refactor: reuse paint drag test helper and trim brush comments

* Add an option to keep merged states as provinces Fixes Azgaar#1465 * Changelog and public changes entry for merging states as provinces

…ar#1912) The citadel icon rename to "palace" also caught the burg editor's data-feature attribute and the burg group feature list, so toggling the citadel wrote burg.palace and left burg.citadel unchanged. Fixes Azgaar#1902

…gaar#1915) Fixes Azgaar#1913

…gaar#1914) * fix: offer to remove the market when unsetting a market center's plaza Turning the plaza off in the burg editor left the burg as its market's center, so the burg still could not be deleted. Ask to remove the market instead; cancelling keeps the plaza. Fixes Azgaar#1899 * docs: explain how to delete a market center burg * fix: escape the market name in the remove-market prompt

- Fix 'descrese' → 'decrease' in urbanization rate tooltip (units-editor.ts) - Fix 'back map the generator' → 'back to the map generator' in emblem editor tooltip (emblems-editor.ts) Closes Azgaar#1909, Azgaar#1917

Bumps [fast-uri](https://github.com/fastify/fast-uri) from 3.1.6 to 3.1.8. - [Release notes](https://github.com/fastify/fast-uri/releases) - [Commits](fastify/fast-uri@v3.1.6...v3.1.8) --- updated-dependencies: - dependency-name: fast-uri dependency-version: 3.1.8 dependency-type: indirect ... Signed-off-by: dependabot[bot] <support@github.com> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>

Bumps [electron](https://github.com/electron/electron) from 43.4.1 to 43.5.0. - [Release notes](https://github.com/electron/electron/releases) - [Commits](electron/electron@v43.4.1...v43.5.0) --- updated-dependencies: - dependency-name: electron dependency-version: 43.5.0 dependency-type: direct:development ... Signed-off-by: dependabot[bot] <support@github.com> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>

…om:SheepFromHeaven/Fantasy-Map-Generator into chore/resolve-pr-1608-conflicts Reconciles PR Azgaar#1608's perf benchmarking suite with master's Grid-module split (Azgaar#1634), main.js decomposition (Azgaar#1821), and the new declarative generation Pipeline: TIME instrumentation moves to Pipeline.run() and the handful of call sites master still times directly (grid-generator, voronoi, renderers, export), map:generated keeps its totalMs via lifecycle.ts's registerMap, and the ViewportLayers/Scene-based renderers keep their master-side render bodies with only the timing calls swapped. Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>

- nix/package.nix: refresh the pinned npmDepsHash. Master's electron 43.4.1->43.5.0 dependabot bump (Azgaar#1930) changed package-lock.json but never touched this hash since nix-build only triggers on PRs that touch package.json/package-lock.json themselves — this PR is the first to do so since that bump landed. - tests/perf/generation.spec.ts: don't throw when a side reports no total time. Master's main.js decomposition dropped the "TOTAL: Xs" console line and copying this PR's spec into a plain-master worktree means that side never gets a totalMs event either, so the case was losing every metric instead of just "total". Fall back to summing whatever per-stage timings did come through. - tests/perf/interaction.spec.ts: stop waiting on window.mapId, a test seam main.js used to set but that master's decomposition dropped. Listen for map:generated instead, same as generation.spec.ts already does. Without this the gesture test hung for its full 180s Playwright timeout on every round, on both sides, which is what pushed perf.yml past its 20-minute budget. Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>

SheepFromHeaven added 6 commits August 23, 2026 16:08

Auto-select the newest fixture instead of hardcoding a filename

b338e94

tests/fixtures/*.map gets new versioned fixtures over time; hardcoding 1.143.1.map would need a manual bump each time. Pick the highest version by parsing the X.Y.Z filename instead.

Remove explanatory comments from perf benchmark files

3df6707

Copilot AI lite review requested due to automatic review settings August 23, 2026 15:06

Copilot started reviewing on behalf of SheepFromHeaven August 23, 2026 15:06 View session

Copilot AI reviewed Aug 23, 2026

View reviewed changes

Comment thread tests/perf/playwright.config.ts

Comment thread tests/perf/ab.mjs

Comment thread src/utils/perfEvents.ts

Comment thread public/main.js Outdated

Comment thread .github/workflows/perf-comment.yml

SheepFromHeaven and others added 7 commits August 23, 2026 20:40

update

f75898b

update

ceb3848

Azgaar assigned SheepFromHeaven Aug 24, 2026

Azgaar and others added 10 commits August 24, 2026 12:03

fix: cap workers to 1000

16cc1d6

fix: don't redraw on non-zoom action like click

6e288ca

knowledge base

c212207

kb formatting

f0c089a

kb update

d641ee8

Merge pull request Azgaar#1614 from Azgaar/wiki-update

8e1df94

Wiki update

Merge pull request Azgaar#1622 from SheepFromHeaven/ci/run-checks-on-…

a64e613

…release-branches ci: run PR checks on release branches

chore: bump version to 1.148.4

2302637

Electron app (Azgaar#1626)

1b6e849

* fix: cap workers to 1000 * fix: don't redraw on non-zoom action like click * init * init * fixes * fixes

minor

75c9a16

Merge branch 'master' into 1.149.0

4ea4f5d

Azgaar and others added 30 commits September 15, 2026 12:13

Ink style (Azgaar#1864)

f78f7cc

* first try * Ocean embellishment * haching * lakes * cinderwood start * cleanup * icons * icons update * coastal bands * frostbite preset * fix * fixes * optimization

cleanup

2c66ec5

fixes

f7b1bb3

update notes

dd38c05

fix

bad9c14

bring old update posts for better new updates (more human-created data)

f860f63

fix stale wiki

1a8c09a

fix test

38bdd72

drainage (Azgaar#1868)

759b328

fixes

7e54c25

fix: preserve cascading column sorting in overview tables (Azgaar#1869)

e66fe2d

fix: use available stock when planning manufacturing chains (Azgaar#1867

7411a4f

) * fix: count remaining manufacturing work from available stock * style: trim production comments and reuse record guard * test: cover freighter production with twenty engines

fix doc

3534bcf

fix tests

cc45672

fix: rename file

e9d7321

Fix paint brushes lagging behind the pointer (Azgaar#1881)

04072d4

* fix: make assignment paint brushes follow the pointer * refactor: reuse paint drag test helper and trim brush comments

Fix filtered raster exports clipping when zoomed out (Azgaar#1886)

425c274

Add an option to keep merged states as provinces (Azgaar#1888)

ade143b

* Add an option to keep merged states as provinces Fixes Azgaar#1465 * Changelog and public changes entry for merging states as provinces

fix: escape the market name in the Markets Overview remove prompt (Az…

ca3184b

…gaar#1915) Fixes Azgaar#1913

fix: correct typos in tooltip text (Azgaar#1918)

cc5dbac

- Fix 'descrese' → 'decrease' in urbanization rate tooltip (units-editor.ts) - Fix 'back map the generator' → 'back to the map generator' in emblem editor tooltip (emblems-editor.ts) Closes Azgaar#1909, Azgaar#1917

Navigation Menu

Repository navigation

Add real generation and interaction performance benchmarks with A/B comparison - #1608

Add real generation and interaction performance benchmarks with A/B comparison#1608
SheepFromHeaven wants to merge 217 commits into
Azgaar:masterfrom
SheepFromHeaven:perf/generation-and-interaction-benchmarks

SheepFromHeaven commented Aug 23, 2026

Uh oh!

netlify Bot commented Aug 23, 2026 •

edited

Loading

Uh oh!

Copilot AI left a comment

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Azgaar commented Aug 25, 2026

Uh oh!

This branch has not been deployed

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

9 participants

Conversation

SheepFromHeaven commented Aug 23, 2026

Uh oh!

netlify Bot commented Aug 23, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

✅ Deploy Preview for afmg ready!

Uh oh!

Copilot AI left a comment

Choose a reason for hiding this comment

🟡 Changes recommended

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Azgaar commented Aug 25, 2026

Uh oh!

This branch has not been deployed

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

9 participants

netlify Bot commented Aug 23, 2026 •

edited

Loading