Tech test - #5

Closed

alexmatthewowen wants to merge 26 commits into

AgentSoftware:mainfrom

alexmatthewowen:tech-test

Commits on Sep 6, 2026

Extract listing search criteria into a value object

alexmatthewowen
and
claude
committed
e2b5a66
Add saved search and alert models

alexmatthewowen
and
claude
committed
46d6d89
Generate and deliver alerts on a nightly schedule

alexmatthewowen
and
claude
committed
5c1635e
Let buyers create, view and delete saved searches

alexmatthewowen
and
claude
committed
a543d01
Add the alerts page

alexmatthewowen
and
claude
committed
e2da322
Document the feature, and fix the unread badge on the alerts page

alexmatthewowen
and
claude
committed
edf5322
Make the alert run a single statement instead of a query per listing

alexmatthewowen
and
claude
committed
98c9507
Say "user" throughout, not "buyer"

alexmatthewowen
and
claude
committed
2e5473e
Move the matching oracle out of app/ and into the test suite

alexmatthewowen
and
claude
committed
ac8492c
Document alerts_count on the User model

alexmatthewowen
and
claude
committed
a9df234
Correct what actually keeps alert attribution fixed

alexmatthewowen
and
claude
committed
c8f7c61
Note the orphaned-attribution edge case in NOTES

alexmatthewowen
and
claude
committed
11e4053
Give the real reasons for not backfilling

alexmatthewowen
and
claude
committed
92afb94
Stop claiming the matching rule lives in one place

alexmatthewowen
and
claude
committed
48efecc
Say what USERS_PER_CHUNK actually trades off

alexmatthewowen
and
claude
committed
3763f31
Fix an N+1, two wrong indexes, and an unguarded concurrent run

alexmatthewowen
and
claude
committed
5930147
Close the zero-bedrooms hole in the empty-search guard

alexmatthewowen
and
claude
committed
3a1a458
Correct the lazy-loading claim, and eager-load on the show page

alexmatthewowen
and
claude
committed
57d16cd
Note that at-most-once delivery depends on the run lock

alexmatthewowen
and
claude
committed
b97f2fe
Compare the zero-bedrooms case as a number, not as the string "0"

alexmatthewowen
and
claude
committed
6d98921
Flag that the benchmark timings predate a query-plan change

alexmatthewowen
and
claude
committed
adcafcb
Fix four issues from the second review round

alexmatthewowen
and
claude
committed
ac74a80
Make the README's end-to-end demo actually work

alexmatthewowen
and
claude
committed
1dc3f33
Correct the lock claim in NOTES too

alexmatthewowen
and
claude
committed
1012d0c
Bring the README's layout map up to date

alexmatthewowen
and
claude
committed
a8ee36a
fix: remove redundant performance section in NOTES.md
alexmatthewowen
committed
234b882

Navigation Menu

Tech test - #5

Tech test#5
alexmatthewowen wants to merge 26 commits into
AgentSoftware:mainfrom
alexmatthewowen:tech-test

Commits on Sep 6, 2026

Extract listing search criteria into a value object

Add saved search and alert models

Generate and deliver alerts on a nightly schedule

Let buyers create, view and delete saved searches

Add the alerts page

Document the feature, and fix the unread badge on the alerts page

Make the alert run a single statement instead of a query per listing

Say "user" throughout, not "buyer"

Move the matching oracle out of app/ and into the test suite

Document alerts_count on the User model

Correct what actually keeps alert attribution fixed

Note the orphaned-attribution edge case in NOTES

Give the real reasons for not backfilling

Stop claiming the matching rule lives in one place

Say what USERS_PER_CHUNK actually trades off

Fix an N+1, two wrong indexes, and an unguarded concurrent run

Close the zero-bedrooms hole in the empty-search guard

Correct the lazy-loading claim, and eager-load on the show page

Note that at-most-once delivery depends on the run lock

Compare the zero-bedrooms case as a number, not as the string "0"

Flag that the benchmark timings predate a query-plan change

Fix four issues from the second review round

Make the README's end-to-end demo actually work

Correct the lock claim in NOTES too

Bring the README's layout map up to date

fix: remove redundant performance section in NOTES.md