26 lines
1.2 KiB
Markdown
26 lines
1.2 KiB
Markdown
# Запрос 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
|