From 8a804da494e9ee8a1f2d0450ab7618ca74ecc0b9 Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?=E2=80=9CNaeel=E2=80=9D?= Date: Sun, 28 Jun 2026 09:33:02 +0400 Subject: [PATCH] =?UTF-8?q?docs:=20=D0=BF=D0=BE=D0=B2=D1=82=D0=BE=D1=80?= =?UTF-8?q?=D0=BD=D0=BE=D0=B5=20=D1=80=D0=B5=D0=B2=D1=8C=D1=8E=20Opus?= MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit --- History/opus-review-2-request-2026-06-28.md | 25 +++++++++++++++++++++ 1 file changed, 25 insertions(+) create mode 100644 History/opus-review-2-request-2026-06-28.md 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