mirror of
https://github.com/MBombeck/HealthLog.git
synced 2026-11-09 04:50:59 +00:00
docs(planning): v1.4.36 closure + post-deploy perf verification
Adds the v1.4.36 row to the audit-marathon closure ledger (now five releases on 2026-05-17) with the headline numbers and the post-deploy perf-QA write-up. Live measurements confirm the page-blocking insights.comprehensive call dropped from 10.88 s to 4.57 s cold (-58%), the AI prompt payload trim landed (insights.generate codex run used 30,096 tokens versus the 25.9 MB raw baseline -- a 99.5% reduction), and the rollup fast-path is firing on the slim summaries slice with the "path":"rollup" annotate. Full /api/analytics correlations branch still runs live SQL on a cold pool (worst-case 111 s recorded, stabilising to 756 ms warm and 11-28 ms cached) and is filed as the v1.4.37 P1.
This commit is contained in:
Marc-André Bombeck committed
1 parent
b9792c986c
commit
c97eef47db
2 files changed
+126
-2
No files matched your search
@@ -1,6 +1,6 @@
|
||||
# v1.4.34.4 + v1.4.34.5 + v1.4.35 + v1.4.35.1 — Audit-Marathon Closure (extended)
|
||||
# v1.4.34.4 + v1.4.34.5 + v1.4.35 + v1.4.35.1 + v1.4.36 — Audit-Marathon Closure (extended)
|
||||
|
||||
Shipped: 2026-05-17 (four consecutive releases on the same date as v1.4.34.3).
|
||||
Shipped: 2026-05-17 (five consecutive releases on the same date as v1.4.34.3).
|
||||
|
||||
| Tag | Highlights | Release |
|
||||
| --- | --- | --- |
|
||||
@@ -8,6 +8,7 @@ Shipped: 2026-05-17 (four consecutive releases on the same date as v1.4.34.3).
|
||||
| v1.4.34.5 | Audit follow-on: 24 critical-path integration tests + iOS textarea zoom fix | [v1.4.34.5](https://github.com/MBombeck/HealthLog/releases/tag/v1.4.34.5) |
|
||||
| v1.4.35 | Layer B landed — persistent `measurement_rollups` foundation + partial reader read-swap on the comprehensive aggregator and the slim summaries slice | [v1.4.35](https://github.com/MBombeck/HealthLog/releases/tag/v1.4.35) |
|
||||
| v1.4.35.1 | Auto-converging rollup backfill on worker boot — no operator action required on self-hosted instances | [v1.4.35.1](https://github.com/MBombeck/HealthLog/releases/tag/v1.4.35.1) |
|
||||
| v1.4.36 | Perf, charts, AI payload trim, UX punch — heavy-aggregate skip on the rollup-fresh path, daily chart fetches opt into rollup buckets, Insights `extractFeatures` swap (25.9 MB → 30,096 tokens, **−99.5 %**), page-blocking call 10.88 s → 4.57 s (**−58 %**), per-section early-skeleton paint, intake-history list restored, About → Admin Console, auto-checking update badge, cumulative-metric tiles, IP-whois fallback | [v1.4.36](https://github.com/MBombeck/HealthLog/releases/tag/v1.4.36) |
|
||||
|
||||
## Audit waves
|
||||
|
||||
|
||||
@@ -0,0 +1,123 @@
|
||||
# v1.4.36 post-deploy perf verification
|
||||
|
||||
**Release:** v1.4.36 (`0e2a99f8` on main, live 2026-05-17 11:28 UTC)
|
||||
**Live host:** https://healthlog.bombeck.io
|
||||
**Account under test:** Marc's production account (311,779 measurements, 8 metric types)
|
||||
|
||||
Marc directive verbatim: *"muss jetzt deutlich schneller sein, sowohl
|
||||
Coach als auch Dashboards — wir dürfen nicht mehr mit Millionen von
|
||||
Daten umgehen."*
|
||||
|
||||
## TL;DR
|
||||
|
||||
- **Page-blocking critical path: 10.88 s → 4.57 s cold (−58 %).**
|
||||
`/api/insights/comprehensive` no longer holds the Insights shell;
|
||||
each section now renders its own skeleton in parallel.
|
||||
- **AI prompt payload: ~25.9 MB → 30,096 tokens (~120 KB)** —
|
||||
confirmed via the `tokensUsed:30096` annotate on a real
|
||||
`insights.generate` codex run. The "Millionen von Daten"
|
||||
framing is now false.
|
||||
- **Rollup-fast-path is firing.** The slim summaries slice emitted
|
||||
`"path":"rollup"` on the live cold hit. Cold 4.15 s, warm 756 ms,
|
||||
cached 11–28 ms.
|
||||
- iOS surfaces stable: paginated `measurement.list` (311k-row table)
|
||||
88–114 ms, `measurements.series` 11–52 ms, batch ingest 254 ms,
|
||||
dashboard widgets miss 8 ms.
|
||||
- Migration `0068_v1436_insights_exclude_metrics` applied cleanly on
|
||||
boot; reminder worker up in 702 ms.
|
||||
|
||||
## Page-blocking critical path
|
||||
|
||||
| Surface | Cold | Warm DB | Cache HIT | Notes |
|
||||
|---|---|---|---|---|
|
||||
| `/api/insights/comprehensive` | **4566 ms** | 190 ms | **28–36 ms** | down from 10.88 s baseline (−58 %). 3478 measurements / 170 mood entries in 90-day window |
|
||||
| `/api/insights/targets` | 1609 ms | — | < 50 ms (LRU) | analytics LRU eviction wired |
|
||||
| `/api/analytics?slice=summaries` | 4153 ms (rollup) | 756 ms | **11–28 ms** | `"path":"rollup"`, 306,368 row_count |
|
||||
|
||||
## Per-section fill-in (does NOT block page render in v1.4.36)
|
||||
|
||||
| Surface | Cold | Payload | Notes |
|
||||
|---|---|---|---|
|
||||
| `/api/insights/weight-status` | 20057 ms | 40,757 B | LLM latency (codex/GPT-5.3) |
|
||||
| `/api/insights/blood-pressure-status` | 20050 ms | 32,198 B | LLM latency |
|
||||
| `/api/insights/mood-status` | 22243 ms | 5,553 B | LLM latency |
|
||||
| `/api/insights/generate` (full) | 142,264 ms | **30,096 tokens** | codex/GPT-5.3, cached false. Prompt size confirms the rollup-bucket swap landed: 30,096 tokens (~120 KB) vs the 25.9 MB raw baseline. |
|
||||
|
||||
The wall-clock on each `insights/*-status` call is the LLM
|
||||
provider round-trip, not anything v1.4.36 changes. The win is that
|
||||
they now run in parallel under the new shell instead of one
|
||||
blocking the other.
|
||||
|
||||
## iOS surface sanity
|
||||
|
||||
| Surface | Cold | Notes |
|
||||
|---|---|---|
|
||||
| `POST /api/measurements/batch` | 254 ms | 1 inserted, 0 duplicates |
|
||||
| `GET /api/measurements` (paginated) | 88–114 ms | total 311,779 rows |
|
||||
| `GET /api/measurements/series?days=30` | 11–52 ms | per-kind series |
|
||||
| `GET /api/dashboard/widgets` | 8 ms | cache miss |
|
||||
| `GET /api/feature-flags` | 6–122 ms | first request cold, then warm |
|
||||
| `GET /api/auth/me` | 32 ms | session lookup |
|
||||
| `GET /api/auth/passkeys` | 75 ms | — |
|
||||
| `GET /api/medications/.../intake` | 41 ms | total 150 |
|
||||
| `GET /api/medications/.../glp1` | 27–53 ms | per-medication |
|
||||
|
||||
## Code-path confirmations
|
||||
|
||||
- **Rollup read-swap firing**: slim-summaries meta carries
|
||||
`"analytics":{"slim_summaries":{"row_count":306368,"type_count":8,"path":"rollup"}}`.
|
||||
The `"path":"rollup"` value is the breadcrumb emitted only when
|
||||
`summaries-slice.ts` took the rollup-fast-path on the
|
||||
fully-covered branch.
|
||||
- **Migration 0068 applied at boot**: docker log shows
|
||||
`Applying migration 0068_v1436_insights_exclude_metrics` then
|
||||
"All migrations have been successfully applied" before
|
||||
Next.js came up. No reader-path stall.
|
||||
- **Per-section early-skeleton paint**: Marc's first cold load
|
||||
fired `/api/insights/comprehensive`, `/api/analytics`,
|
||||
`/api/insights/targets`, `/api/mood/analytics`,
|
||||
`/api/gamification/achievements`, `/api/auth/passkeys`,
|
||||
`/api/bugreport/status`, `/api/insights/generate`,
|
||||
`/api/insights/{weight,blood-pressure,mood}-status` all within
|
||||
~6 s, in parallel. Pre-v1.4.36 the page held the entire shell
|
||||
on `/api/insights/comprehensive` (10.88 s) before any section
|
||||
could render.
|
||||
- **AI prompt payload trim**: `tokensUsed:30096` confirms the
|
||||
rollup-bucket swap in `extractFeatures` landed. 30,096 tokens
|
||||
is ~120 KB — well inside the < 1 MB target and ~99.5 % below
|
||||
the 25.9 MB raw-measurements baseline.
|
||||
|
||||
## Open issues (deferred to v1.4.37)
|
||||
|
||||
- **`/api/analytics` (FULL, not slim)**: first cold hit was
|
||||
**111,092 ms** (correlations + healthScore + bp_in_target,
|
||||
three concurrent queries on a cold-pool DB). On the next
|
||||
navigation 2 minutes later it dropped to 756 ms (slim again),
|
||||
and cache HITs settled at 11–28 ms. The full aggregator's
|
||||
correlations and healthScore branches still run live SQL and
|
||||
did not get touched by v1.4.36. This matches the explicit
|
||||
deferred-list item from the handover: *"narrow-aggregate query
|
||||
still scans 90 days of measurements; the column-pruning won is
|
||||
real but NOT the headline 'sub-second' claim."* File a v1.4.37
|
||||
P1 to collapse the correlations / healthScore branches onto the
|
||||
rollup-coverage probe.
|
||||
- **`/api/mood/analytics`** (out of v1.4.36 scope): 3.5 s cold for
|
||||
359 mood entries — separate codepath, not touched this release.
|
||||
- **HAR capture not yet collected**: Chrome devtools HAR for the
|
||||
cold mount is the authoritative source for FCP/TTI/CLS. The
|
||||
pinned `trends-row-chart-slot` height should drop CLS toward
|
||||
zero; capture and attach when convenient.
|
||||
|
||||
## Verdict
|
||||
|
||||
The release lands the Marc directive. The page-blocking
|
||||
critical-path call is materially faster (10.88 s → 4.57 s cold, ~28 ms
|
||||
warm cache), the AI prompt payload trim is confirmed by a real
|
||||
codex run (30,096 tokens vs 25.9 MB baseline → **99.5 % reduction**),
|
||||
and the rollup-fast-path is wired and firing on the slim
|
||||
summaries slice. Per-section AI status calls each take ~20 s but
|
||||
they no longer block the page — exactly the deferral pattern
|
||||
v1.4.36 was designed to deliver.
|
||||
|
||||
**Action**: ship as v1.4.36, file the full-`analytics`-route window
|
||||
tightening as v1.4.37 P1.
|
||||
Reference in new issue
Block a user