Files
gosentry/docs/PERFORMANCE.md
T
mixeme c57d5b102f P6.3: correct startup measurement — ~400 ms on Fyne 2.7.4
Previous commit used PowerShell process-launch timing instead of the
History tab metric. Actual warm-run figure from the in-app "Window shown
in …" event is ~400 ms, down from ~644 ms on Fyne 2.6.3 (~37% recovery).

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
2026-06-23 22:36:04 +03:00

3.2 KiB
Raw Blame History

Performance Notes

Measured performance findings for GoSentry. Each entry records the method so the numbers can be reproduced and re-checked after relevant changes.

Startup Time

Finding (2026-06-22)

After the Phase 4 refactor, cold startup time (the History "Window shown in …" metric) increased by ~290 ms. The increase is caused entirely by the Fyne v2.5.3 → v2.6.3 upgrade (task T4.1), not by the Service / domain / UI restructuring.

Method

Env-gated phase timers (GOSENTRY_TIMING) were added across the startup path (Run in src/ui/run.go and newMainView in src/ui/mainwindow.go) and the equivalent points in the pre-refactor entry point (src/gui/app.go at commit c5e0ef9, the last commit before T4.1). Both were built with the CGO / MSYS2 UCRT64 toolchain and run 5× each; the first run of each is a cold-disk outlier and is excluded. The timed span (startedw.Show()) is identical in both builds, so the comparison is fair.

Results (warm-run averages)

Phase (cumulative from start) Old (Fyne 2.5.3) New (Fyne 2.6.3) Δ
after single-instance check ~0.5 ms ~0.6 ms
after Fyne app + window + tray ~277 ms ~285 ms +8 ms
app.Open() done +3 ms +3 ms 0
views built + svc.Start() +42 ms +43 ms ~0
after SetContent ~348 ms ~353 ms +5 ms
after w.Show() (TOTAL) ~348 ms ~644 ms +~290 ms

Interpretation

  • Everything up to and including SetContent costs the same in both versions (~350 ms). The refactor-specific code — app.Open() (~3 ms) and the new app.Service plus view construction (~42 ms) — is unchanged, so the restructuring added no measurable startup cost.
  • The entire regression lands in w.Show(): ~0 ms under Fyne 2.5.3, ~290 ms under 2.6.3. Fyne 2.6 reworked main-thread marshaling (the change that introduced fyne.Do) and front-loads first-window GL/driver realization into the Show() call.
  • The cost is a fixed, one-time Fyne expense, not a leak in GoSentry code, and the upgrade cannot be reverted because fyne.Do requires Fyne ≥ 2.6.
  • The tray / autostart path (--start-in-tray) skips w.Show() until the user opens the window, so it is unaffected.

Finding (2026-06-23, Fyne 2.7.4)

Fyne upgraded from v2.6.3 → v2.7.4 as part of Phase 6 (P6.1). Measured using the same method as the 2026-06-22 baseline: the History tab "Window shown in …" event, which records the span from application entry to w.Show() completion.

Warm-run average: ~400 ms (down from ~644 ms on Fyne 2.6.3).

Interpretation

Fyne 2.7 recovered roughly ~240 ms (~37%) of the w.Show() regression that arrived with the 2.6 threading model change. Fyne 2.7.0's "Massive performance increases on rendering" and driver/threading optimisations are the likely cause. The upgrade is a net win: the fyne.Do threading API (required since 2.6) is retained, and the primary startup cost is substantially reduced.