docs: повторное ревью Opus
This commit is contained in:
@@ -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
|
||||
Reference in New Issue
Block a user