docs(track): result_migration_review_pass decisions for src/models.py UNCLEAR (2 compliant + 0 migration-target)
This commit is contained in:
@@ -109,7 +109,15 @@ Other audit findings (unchanged by this review pass):
|
||||
|
||||
### 2.5 `src/models.py` — UNCLEAR sites (2)
|
||||
|
||||
*(filled in Task 2.5)*
|
||||
| Line | Context | Snippet | Decision | Pattern / Rationale |
|
||||
|---|---|---|---|---|
|
||||
| 452 | `from_dict` (track-state deserialization) | `except ValueError: created = None` after `datetime.fromisoformat(created)` | **compliant** | Lenient deserialization: malformed ISO date in TOML config → `None` (don't crash the entire load). Canonical pattern for user-edited config |
|
||||
| 457 | `from_dict` (track-state deserialization) | `except ValueError: updated = None` after `datetime.fromisoformat(updated)` | **compliant** | Lenient deserialization (same as L452) |
|
||||
|
||||
**Subtotals:** 2 compliant + 0 migration-target.
|
||||
|
||||
**New heuristic candidates:**
|
||||
9. `datetime.fromisoformat(s)` with `except ValueError: <var> = None` → `INTERNAL_COMPLIANT` (lenient TOML deserialization)
|
||||
|
||||
---
|
||||
|
||||
|
||||
Reference in New Issue
Block a user