4 Commits

Include renames

Author	SHA1	Message	Date
glenn schrooyenandClaude Opus 5.5	8f0b8477e6	MAINT-21 review: two source scans read the split files too ... test_safety118's "size rule is gone" and test_validate's "no numeric maintenance_* fallback" read app/main.py only; the code they guard can now sit in app/ctl_*.py, where a regression passed unseen. Both read main_files.source() for main.py now. Tests only; no app change. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>	2026-11-09 07:15:14 +01:00
glenn schrooyenandClaude	0d49027daa	MAINT-21: tests read and patch the Controller through tools/main_files.py ... Prepares the split of app/main.py's Controller into files by job, with no app change. tools/main_files.py gives a test the text and the AST of main.py plus the files split from it, and `live`, which sets a global in every file whose functions read it (and raises when none does). Tests that read main.py's source, and tests and replay tools that patch a main global (datetime, compute, resolve, _mono, WRITE_BUDGET_S, ...), go through it, so they pass on this layout and on the split one. test_split_patches.py fails when a test or tool patches an app.main attribute that main.py no longer reads, or that another file also reads. tools/maint21_same_ast.py proves every function, method and module-level assignment of main.py at the base is found unchanged in its new file. Co-Authored-By: Claude <noreply@anthropic.com>	2026-11-09 00:27:07 +01:00
glenn schrooyenandClaude	7d52dc1c0f	SAFETY-118: a missing peak forecast reading keeps the unit-fault state ... A missing reading (unavailable, unknown, failed read, non-numeric) no longer clears the fault: no false 'used again' INFO, no repeated WARNING, the status row stays. Adds pv.has_reading(). DECISIONS wording fixed; tests added. Co-Authored-By: Claude <noreply@anthropic.com>	2026-10-16 18:19:44 +02:00
glenn schrooyenandClaude	f0b4e68e49	SAFETY-118: read the peak forecast in its own unit ... peak_forecast_entity is read through pv.watts() by its unit_of_measurement; the abs(raw) < 50 size rule is gone. No unit or a non-power unit means the forecast is not used, with one warning per cause. The validator no longer claims a kW sensor is taken as watts for this entity. Co-Authored-By: Claude <noreply@anthropic.com>	2026-10-16 16:00:34 +02:00