perf: deduplicate homepage DB query #5

Merged

jperera merged 1 commit from perf/deduplicate-homepage-queries into main

2026-04-02 20:34:54 +02:00

jperera commented

2026-04-02 20:29:42 +02:00

Owner

Summary

analyzeTrends() now accepts optional pre-fetched HistoricalPrice[] to avoid re-querying the database
The / route was calling getPriceHistory() and analyzeTrends() separately, but analyzeTrends internally called getPriceHistory() again — now passes its already-fetched data
Merged two iteration loops in analyzeTrends() into a single pass (also fixes optimization #6)

Impact

Eliminates one full getPriceHistory DB query per homepage load.

## Summary - `analyzeTrends()` now accepts optional pre-fetched `HistoricalPrice[]` to avoid re-querying the database - The `/` route was calling `getPriceHistory()` and `analyzeTrends()` separately, but `analyzeTrends` internally called `getPriceHistory()` again — now passes its already-fetched data - Merged two iteration loops in `analyzeTrends()` into a single pass (also fixes optimization #6) ## Impact Eliminates one full `getPriceHistory` DB query per homepage load.

jperera added 1 commit

2026-04-02 20:29:42 +02:00

perf: deduplicate homepage DB query e8f7bcc95c

The / route was calling getPriceHistory() and analyzeTrends(), but
analyzeTrends() internally called getPriceHistory() again. Now
analyzeTrends() accepts optional pre-fetched history data, and the
homepage route passes its already-fetched history.

Also merged the two iteration loops in analyzeTrends() into a single pass.

jperera force-pushed perf/deduplicate-homepage-queries from e8f7bcc95c to b10124d2df

2026-04-02 20:34:46 +02:00

Compare