Files
homelab-monitor/apps/api/src
jhodgkin 960c926dd2
CI / web (push) Successful in 19s
CI / api (push) Successful in 26s
fix: 'New' badge flagged the entire existing device population
Deployed the new-device feature, checked the live API response
instead of assuming the earlier unit test covered it, and found 65/71
devices marked isNew: true. The notification path was correctly
bootstrap-safe (verified separately), but the UI's isNew check just
tested "first_ever_seen within 24h" with no bootstrap awareness --
and bootstrap timestamps are, correctly, "now", so the whole existing
population qualified on day one.

Added seen_macs.is_bootstrap, set on the seeding call and excluded
from isNew. Migration backfills is_bootstrap=1 for any seen_macs rows
that already existed before this column did (the already-deployed
CT122 instance) -- verified locally against both a fresh DB and a
simulated pre-migration table.

Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>
2026-07-13 00:02:04 -06:00
..