Skip to content

fix(performance): eliminate menu error spam and repeated full scans #30

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Open
Alx8g wants to merge 8 commits into
base: main
Choose a base branch
from
Open

fix(performance): eliminate menu error spam and repeated full scans #30

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
8 commits
Select commit Hold shift + click to select a range
370e211
fix(startup): show the game window when the intro is disabled
Alx8g Sep 3, 2026
10cfa70
chore(release): bump version to 1.1.0-rc.17-fix.1
Alx8g Sep 3, 2026
829b0bd
fix(performance): remove redundant identity and DOM observer work
Alx8g Sep 4, 2026
52793cf
test(performance): record paired client overhead measurements
Alx8g Sep 4, 2026
e28e81b
fix(startup): integrate existing launch and menu crash fixes
Alx8g Sep 4, 2026
a96c5b3
test(menu): cover native NodeList collection semantics
Alx8g Sep 4, 2026
b1ec56d
fix(performance): skip menu scans for unrelated text mutations
Alx8g Sep 4, 2026
665eee8
test(performance): record live renderer audit and before-after captures
Alx8g Sep 4, 2026
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
7 changes: 3 additions & 4 deletions .github/workflows/source-validation.yml
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -79,10 +79,9 @@ jobs:
run: pnpm run validate

- name: Perf-unit assertions
# Cheap perf-adjacent unit tests only: rollover/reset residue guards and
# histogram-accuracy bounds for the frame-time stats, adaptive-validation
# runtime lifecycle, and the startup-marks parser. No app launch, no packaging.
run: node --test tests/performance-stats.test.ts tests/performance-monitor.test.ts tests/adaptive-validation.test.ts tests/adaptive-validation-runtime.test.ts tests/parse-perf-marks.test.ts
# Cheap perf-adjacent unit tests only: frame-time stats, diagnostic lifecycle,
# identity parsing/observer work, and startup marks. No app launch, no packaging.
run: node --test tests/performance-stats.test.ts tests/gameplay-fps-log.test.ts tests/draw-call-stats.test.ts tests/identity-rewrite.test.ts tests/mutation-relevance.test.ts tests/menu-declutter-mutations.test.ts tests/parse-perf-marks.test.ts

# Future packaged startup-marks smoke job — intentionally disabled scaffolding.
# TODO(perf-gate): enable (remove `if: false`) only once a runner pool with understood
Expand Down
Loading