Merge branch 'main' of https://github.com/Wewoc/Garmin_Local_Archive

Author: Wewoc

README.md

@@ -602,4 +602,4 @@ GUI changes are verified manually before release. Full CI/CD with automated buil
 
 ---
 
-*Built with Claude · [☕ buy me a coffee](https://ko-fi.com/wewoc)*
+*Built with Claude · [☕ buy me a coffee](https://ko-fi.com/wewoc)*

docs/ROADMAP.md

@@ -6,14 +6,44 @@
 
 ---
 
-**Currently stable — v1.5.5.2**
+**Currently stable — v1.5.5.1**
 
 ---
 
 ## Planned
 
 ---
 
+### v1.5.5.2 — Quality Log Transaction API
+
+Architectural hardening of `garmin_quality.py`. Eliminates the existing
+sole-write-authority violation in which `garmin_collector.py` calls private
+internal methods directly and manages raw state dictionaries outside the
+State Owner.
+
+**What changes:**
+- `garmin/garmin_quality.py` — new public method `record_attempt(day, status,
+  error_msg, source)`. Encapsulates load → upsert → save as a single atomic
+  operation under the existing `QUALITY_LOCK`. Internal helpers `_upsert_quality()`
+  and `_save_quality_log()` remain private — no external caller permitted.
+- `garmin/garmin_collector.py` — all direct calls to `quality._upsert_quality()`
+  and `quality._save_quality_log()` replaced by `quality.record_attempt()`.
+  Collector no longer holds or mutates raw quality dictionaries.
+
+**What does not change:**
+- `QUALITY_LOCK` — existing lock, no changes to scope or ownership
+- `quality_log.json` schema — no structural changes
+- All other callers of `garmin_quality.py` — public read API unchanged
+
+**Why before v1.5.6:**
+`garmin_import_mirror.py` writes through existing owners by design. If the
+Collector still bypasses the State Owner at import time, the invariant is
+broken from the first line of the new module.
+
+*Pre-condition: v1.5.5.1 stable.*
+
+---
+
 ### v1.5.5.3 — Unified Date Parser in Quality Module
 
 Removes redundant date-extraction logic scattered across three functions in
@@ -229,28 +259,6 @@ Third tab in the existing `QTabWidget` structure (Tab 1 "Actions", Tab 2 "Dashbo
 
 ---
 
-### v1.5.8 — Standalone EXE: --onedir Migration
-
-Replaces `--onefile` with `--onedir` in `build_standalone.py` for T3.
-Eliminates the per-launch extraction to `%TEMP%\_MEIxxxxxx` — all files
-sit permanently unpacked next to the EXE. Startup time for T3 drops
-significantly as a result.
-
-**What changes:**
-- `compiler/build_standalone.py` — `--onefile` → `--onedir`
-- `build_combined_zip()` — ZIP logic updated to pack the output folder
-  instead of a single EXE file
-- `tests/test_build_output.py` — T3 checks updated for folder structure
-
-**What does not change:**
-- T2 (`build.py`) — stays `--onefile`, unaffected
-- Total installed size — identical, only distribution format changes
-- User workflow — ZIP download unchanged, EXE name unchanged
-
-*Pre-condition: v1.5.5.7 stable — or skipped if no v1.5.5.7 planned.*
-
----
-
 ## Planned — v1.6
 
 ### v1.6 — Dashboard Render Registry