📝 код-ревью 07: разобрал fixme-комменты из задачи — типы методов, иммутабельные дефолты

Author: esemi

code-review-practice/solutions/07_config_loader.md

@@ -28,9 +28,17 @@
 
 6. Нет обработки `FileNotFoundError` / `json.JSONDecodeError` с внятным сообщением.
 
-7. `DEFAULTS` — мутабельный модульный словарь, любой может его случайно изменить.
+7. `DEFAULTS` — мутабельный модульный словарь, любой может его случайно изменить (см. п.3 —
+   именно это и происходит). Дефолты стоит сделать неизменяемыми: неглубокие — через
+   `MappingProxyType`, а вложенные списки — кортежами / фабриками, чтобы их нельзя было
+   мутировать на месте.
 
-8. Нет типизации / схемы конфига (pydantic-settings решил бы половину проблем).
+8. Нет типизации / схемы конфига (pydantic-settings решил бы половину проблем: типы,
+   валидацию, дефолты, чтение из env — вместо самописного класса).
+
+9. **Нет аннотаций типов у методов** (`__init__`, `get`). `path: str`, `key: str` и
+   возвращаемый тип не описаны — контракт неясен, mypy/IDE не помогут. Даже без полной
+   схемы конфига сигнатуры стоит типизировать.
 
 **Итого:** два тонких бага дефолтов (падение на missing key и затирание falsy-значений)
 + мутация общего списка. Мораль: `if value:` ≠ «ключ задан»; и общие мутабельные дефолты кусаются.