diff --git a/History/opus-review-2-request-2026-06-28.md b/History/opus-review-2-request-2026-06-28.md new file mode 100644 index 0000000..9d4c891 --- /dev/null +++ b/History/opus-review-2-request-2026-06-28.md @@ -0,0 +1,25 @@ +# Запрос Opus — повторное ревью после правок + +Дата: 28.06.2026 + +## Что исправлено (по твоему ревью) + +| # | Было | Стало | +|---|------|-------| +| 1 | Двойной `json.dumps` | `PgRepository.set_document_parsed` передаёт `elements` напрямую | +| 2 | `classify_batch(batch_id)` без DI | `classify_batch(batch_id, llm_client=None, repo=None)` | +| 3 | `ParseResult.from_row` → element_count=0 | `len(elements)` | +| 4 | `GroupingResult.unresolved` дублировался | Вынесен в `BatchGroupingResult` | +| 5 | Protocol неполон | +7 методов: `get_document`, `list_by_batch`, `count_by_status`, `reset_classify_status`, `set_classify_processing`, `delete_document` | + +## Что проверь + +1. Баги #1-2 — устранены ли? +2. `BatchGroupingResult` — корректен ли дизайн? +3. Protocol — всё ли ещё упущено? +4. 22/22 PASS — не сломано ли? + +## Файлы +- `deploy/contracts.py` — + BatchGroupingResult +- `deploy/repository.py` — +7 методов Pg + Mem +- `deploy/services/classify.py` — DI в classify_batch