mirror of
https://github.com/bckelley/cgm-remote-monitor.git
synced 2026-08-24 03:14:12 -05:00
Ben WestandCopilot
3b701222e2
The DB name check now only blocks when entries EXCEED the threshold. If the database has fewer entries than the threshold (default 100), it's treated as safe to test regardless of DB name — the name check becomes a warning suggesting you rename for best practice. Logic: entry count is the primary safety signal. DB name is secondary. Both must fail to block. Entry count alone blocks. Name alone warns. Changes: - Entry count checked first to determine safety baseline - DB name check downgrades to warning when entries within threshold - Contextual override hints (only show relevant suggestions) - Clarify CUSTOMCONNSTR_mongo vs CUSTOMCONNSTR_mongo_collection Tests (5 new): - 0 entries + non-test name → passes with warning - 50 entries (below threshold) + non-test name → passes with warning - Entries above threshold + non-test name → blocks (both errors) - Entries above threshold alone → blocks with 'real data' message - Non-test name hint mentions correct env var Fixes nightscout/cgm-remote-monitor#8464 Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>