Skip to content

perf: optimize application loading and workflows#109

Merged
AussieScorcher merged 8 commits into
mainfrom
perf/optimization
Jul 18, 2026
Merged

perf: optimize application loading and workflows#109
AussieScorcher merged 8 commits into
mainfrom
perf/optimization

Conversation

@AussieScorcher

@AussieScorcher AussieScorcher commented Jul 18, 2026

Copy link
Copy Markdown
Member

Summary

Improves perceived and measured frontend performance across navigation, public pages, contribution tools, the draft generator, and staff workflows. It also adds repeatable profiling/React diagnostics and documents the product design principles guiding the interface work.

Changes Made

  • Adds cached lazy-route imports, intent-based route preloading, stable application shell rendering, route-aware loading skeletons, and reduced-motion-friendly reveal behavior.
  • Reduces unnecessary React state and rerenders in authentication, map, contribution, division, and staff-management workflows while standardizing shared controls and loading states.
  • Adds performance tracing and React diagnostic tooling, refreshes public/staff interface details and the About asset, and removes obsolete components, hooks, pages, and utilities.

Additional Information

The main performance issues were whole-page Suspense transitions, repeated route loading, mirrored UI state, controlled high-frequency map state, and legacy modules that increased maintenance and bundle surface.

User impact: navigation keeps the application shell stable, loading states better match destination content, interactive workflows do less avoidable work, and administrative screens are more consistent and accessible.

Validation:

  • npm run lint
  • npm run test — 63 tests passed
  • npm run build
  • bunx react-doctor@latest --verbose --scope changed — no issues found

The production build succeeds with advisory warnings for chunks larger than 500 kB.

Author Information

Discord Username:
VATSIM CID:


Checklist:

  • Have you followed the guidelines in our Contributing document?
  • Have you checked to ensure there aren't other open Pull Requests for the same update/change?

Comment thread src/features/draft-generator/extractor/classify.js Dismissed
Comment thread src/features/draft-generator/extractor/classify.js Fixed
Comment thread src/features/draft-generator/extractor/bgl.js Fixed
Comment thread src/features/draft-generator/extractor/bgl.js Fixed
Comment thread src/features/draft-generator/extractor/extract.js Fixed
Comment thread src/features/draft-generator/extractor/extract.js Fixed
@AussieScorcher
AussieScorcher marked this pull request as ready for review July 18, 2026 19:05
@greptile-apps

greptile-apps Bot commented Jul 18, 2026

Copy link
Copy Markdown
Contributor

Too many files changed for review. (104 files found, 100 file limit)

Bypass the limit by tagging @greptile-apps to review.

@AussieScorcher
AussieScorcher merged commit 9a3b4a3 into main Jul 18, 2026
5 checks passed
@AussieScorcher
AussieScorcher deleted the perf/optimization branch July 18, 2026 19:05
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.

2 participants