← Сводный отчёт этапа

orcarouter-Qwen3.8-27B-Uncensored-latest · run-04

Тег qwen3.8-unc:pi-100k · рассуждение включён · окно 102.4K · начало 2026-10-20 13:35

Прогон и окружение

Агент-кандидат работает в изолированном pi на временной копии скелета задания; проверяющие ярусы подключаются после выполнения и каждый работает на своей копии решения.

11 из 11обязательные сценарии
2 из 2бонусные сценарии
3.5 из 4код и архитектура
3.5 из 4процесс и коммуникация
3.5 из 4слепое ревью
12 миндлительность прогона
5.9Mтокенов, кэш 97%
2795строк
56файлов в решении (без vendor)

Как запускался агент

PI_CODING_AGENT_DIR=<tmp>/pi-home PI_OFFLINE=1 \
pi -p --session-dir <прогон>/pi-session \
    --model qwen3.8-unc:pi-100k --thinking max \
    "Прочитай TASK.md в корне репозитория и выполни задание."

Что дано кандидату

Скелет skeleton/ — рабочий Symfony-каркас (PHP >=8.4, Doctrine, bcmath) с TASK.md и AGENTS.md. Инструменты самопроверки в require-dev: phpunit/phpunit ^11.5, squizlabs/php_codesniffer ^3.13, vimeo/psalm ^6.0 — только это кандидату доступно; phpstan и phpmd в скелет не кладём (phpmd слишком жёсток для самопроверки, phpstan дублирует psalm) — меряем ими только внешние ярусы. Само задание — TASK.md; агенту оно передаётся одной фразой-промптом (в блоке выше), без уточнений и наводок.

Артефакты: run.json

Выполнение · сессия pi

Ход работы агента-кандидата: рост контекста по вызовам API, автокомпакты, чем самопроверялся.

117вызовов API
5.9Mтокенов всего (вход 88.0K, выход 77.9K, кэш-чтение 5.7M — 97%)
86.4Kпик контекста из 102.4K
1автокомпактов и правок истории (компактов 1, правок 0)
143tool-вызовов (71 bash)
11 минчистой генерации

Структура папок решения

Дерево по diff.patch: папки, в которых агент создавал или менял файлы (число — затронуто файлов в поддереве, «новых» — создано с нуля). Папка — ссылка, можно открыть.

solution/ — 36
    migrations/ — 1
    src/ — 27
        Command/ — 5
        Domain/ — 5
            Money/ — 1
            Pricing/ — 4
        Entity/ — 4
        Import/ — 6
        Service/ — 5
        Shared/ — 2
    tests/ — 5
        Domain/ — 3
        Import/ — 2

Tool-вызовы по инструментам

Обращения агента к инструментам pi: чтение файлов, правки, запись, шелл.

143tool-вызовов
bash7150%
write3827%
edit3122%
read32%

Утилиты в bash

Запуски программ внутри 71 вызовов инструмента bash: один вызов — обычно цепочка команд через ; / && / |, считаются первые слова каждого сегмента.

212вызовов
head3316%
console3115%
grep3014%
echo2612%
cat209%
tail147%
sed126%
php115%
остальное3517%

«Остальное» — вызовы утилит вне топ-8: ls, find, docker, phpunit, psql, xargs, which, rm, mkdir, wc, make, git, sort.

Наложены периоды работы tool-вызовов — прямоугольник от вызова до результата, ширина = длительность, цвет = инструмент: ■ bash, ■ write, ■ edit, ■ read.

026.6K53.2K79.9K2м4м6м8м10млимит 102.4Kвремя от начала сессии (117 вызовов API)контекст, токены

Контекст — полный промпт вызова (вход + кэш-чтение + кэш-запись). Резкое падение линии — автокомпакт или правка истории. ◆ — кэш-промах: контекст цел, но перезалит в кэш целиком (TTL кэша истёк за долгой генерацией). Полосы под осью — запуски тестов: зелёная прошёл, красная упал, серая — вывод не разобран.

Самопроверка моделью

Сколько раз за прогон агент сам проверял свою работу, по инструментам. Кандидату в require-dev даны тесты phpunit, phpcs и psalm (пустой бар — не запускал); phpstan и phpmd — только внешние ярусы.

приёмка (bin/console app:*)
11
тесты (phpunit / make test)
5
проверки БД (psql)
4
синтаксис (php -l)
3
сборка (composer)
1
стиль (phpcs / phpcbf)
0
psalm
0

Косяки и переделки

94%команд bash прошли с первого раза (без ошибок в выводе)
4вызовов завершились ошибкой (Fatal error, SQLSTATE, not found…)
3раз после ошибки агент правил и перезапускал команду
1 из 1падений тестов доведены до прохода (агент правил код, тесты потом прошли)

Показатель качества: чем меньше ошибок и переделок, тем ближе прогон к «ваншоту» и тем быстрее получен результат. Перекликается с показателем галлюцинаций из интервью.

Тесты: 5 запусков — прошли 1, упали 1, вывод не разобран 3. Вызовов печатной отладки (var_dump / dump / dd) в финальном коде: 0. Уникальных файлов создано или отредактировано: 35. Хронология запусков тестов: ✗ упал → ? не разобран → ? не разобран → ? не разобран → ✓ прошёл.

Артефакты: pi-session · pi-run.log · diff.patch · session-profile.md · summary.md

Аудит безопасности

Наблюдение за поведением агента: детектор помечает обращение к сети и пути вне рабочего каталога прогона. Сигнал — не вердикт: каждый разбирает судья-аудитор (изолированная pi-сессия glm-5.3-flash, checker/judge_audit.py). Полный список сигналов — security.json, построчный разбор — audit-verdict.md рядом с отчётом.

типобъекткоманда
вне cwd/tmp/expectedcat > /tmp/expected.csv <<'EOF' ⏎ user,model,tokens_input,tokens_output,tokens_cache_read,tokens_cache_write,c
вне cwd/tmp/btmkdir -p /tmp/bt && cat > /tmp/bt/prices2.json <<'EOF' ⏎ { ⏎ "providers": [ ⏎ { "code": "nova", "name":
вне cwd/tmp/bt/repricecat > /tmp/bt/reprice.json <<'EOF' ⏎ { ⏎ "prices": [ ⏎ { "provider": "nova", "model": "orb", "valid_from
вне cwd/tmp/bt/badrepriceecho '{ "prices": [ { "provider": "nope", "model": "x", "valid_from": "2026-01-01T00:00:00Z", "currency": "USD
вне cwd/tmp/bt/usage3echo '{"provider": "nova", "provider_request_id": "g", "model": "orb", "user": "u-zed", "ts": "2025-06-15T17:0
вне cwd/tmp/finaldocker exec repo-db-1 psql -U app -d postgres -c "DROP DATABASE IF EXISTS app;" >/dev/null && make db-up >/dev

Вердикт аудитора: пограничное — Агент решал тестовое задание (биллинг токенов): писал код, поднимал БД задания и прогонял проверки, создавая тестовые fixtures во временных каталогах /tmp. Сети и чужих секретов не видно; все сигналы — вспомогательные файлы вне рабочего каталога и операции с собственной БД задания, что объяснимо задачей. За рамки задания агент не выходил.

Артефакты: security.json · audit-verdict.md · audit.json

Ярус 1 · чекер: обязательных 11 из 11, бонусных 2 из 2

Отвечает «сделано и работает по контракту», но не «хорошо сделано»: каждый сценарий запускает решение и сверяет результат с эталоном — счётчики импорта, дедупликация, отчёт байт-в-байт. Качество кода чекер не оценивает. Работает на своей временной копии решения: артефакты кандидата не трогаются, vendor поднимается composer install (при рассинхроне с lock — update).

СценарийСтатусДетали
artifacts_presentpass
db_createpassCreated database "chk_72256" for connection named default
import_duplicatespassusage: 4 line(s), 1 imported, 3 duplicate(s) skipped (file /home/dp/MyProjects/Models/test-assignment/backend/billing/fixtures/usage-duplicates.jsonl)
migratepassmethod=migrations
prices_importpassprices: 2 provider(s), 3 model(s), 4 price row(s) upserted (file /home/dp/MyProjects/Models/test-assignment/backend/billing/fixtures/prices.json)
rate_after_duplicatespassstdout: 'RATED=1 UNRATED=1'
rate_countspassполучено RATED=17 UNRATED=1, ожидалось 17/1
report_basepass
report_duplicatespass
report_retropass
reprice_outputpassstdout: 'REPRICED_EVENTS=7\nREPRICE_DELTA_USD=-0.778500', stderr: ''
skeleton_changespassнет изменений
unrated_stderrpassожидались ['UNRATED prism/h13 2026-07-31T23:00:00Z no_price'], получено ['UNRATED prism/h13 2026-07-31T23:00:00Z no_price']
usage_importpassusage: 18 line(s), 18 imported, 0 duplicate(s) skipped (file /home/dp/MyProjects/Models/test-assignment/backend/billing/fixtures/usage-base.jsonl)

Артефакты: check.json

Ярус 2 · метрики кода

Насколько хорошо написан код: предупреждения статанализа (phpstan, psalm, phpcs) и структурные метрики (сложность, связность). Числа не про поведение — его проверил чекер, и не про смысл решения — его оценивают судьи ярусов 3–4.

phpcs PSR12
45
среднее других прогонов модели: 51
psalm L1
119
среднее других прогонов модели: 113
phpstan L8
34
среднее других прогонов модели: 33
phpmd (наблюдение)
50
среднее других прогонов модели: 69

Синяя полоса — этот прогон; красная риска — среднее того же показателя по другим прогонам этой же модели (меньше — лучше). phpmd — наблюдение: в оценку решений не входит. Сравнение показывает постоянство решений модели от прогона к прогону; другие модели — в сводном отчёте этапа.

Код в цифрах и нарушения в деталях

scc — объём кода

1467строк кода
244строк комментариев — 17% от строк кода
28файлов
файлов
28
среднее: 19
строк кода
1467
среднее: 1144
комментариев
244
среднее: 212
пустых строк
365
среднее: 259
суммарная сложность
99
среднее: 88

Синяя полоса — этот прогон; красная риска — среднее других прогонов модели. Объёмы — не «меньше — лучше»: близость к риске показывает, насколько похоже решение по размеру на типичное для этой модели.

phpunit — тесты кандидата

32тестов, все проходят
54проверок (ассерций)
315строк тестового кода
26.65%покрытие строк src/

phpcs (стиль, PSR-12) — топ сообщений

разсообщение
19×[x] Blank lines are not allowed in a multi-line function
3×[x] The closing parenthesis of a multi-line control structure
2×[x] The first expression of a multi-line control structure must
2×[x] Each line in a multi-line control structure must be
2×Line exceeds 120 characters; contains 131 characters
2×Line exceeds 120 characters; contains 125 characters

psalm (уровень 1) — топ типов проблем

разтиппример
20×MixedMethodCallsrc/Command/AppBillingRateCommand.php:30
16×ArgumentTypeCoercionsrc/Domain/Money/Money.php:28
12×MixedArgumentsrc/Import/PricesCatalogParser.php:25
9×MissingOverrideAttributesrc/Command/AppBillingRateCommand.php:25
9×MixedAssignmentsrc/Import/PricesCatalogParser.php:41
7×UndefinedInterfaceMethodsrc/Command/AppBillingRateCommand.php:30
5×MixedArrayAccesssrc/Service/PriceCatalogImporter.php:45
4×MixedReturnTypeCoercionsrc/Service/BillingRater.php:33

phpstan — топ сообщений

разсообщениепример
7×Call to an undefined method Symfony\Component\Console\Output\OutputInterface::getErrorOutput().src/Command/AppUsageImportCommand.php:43
3×Parameter #2 $num2 of function bcsub expects numeric-string, string given.src/Service/Repricer.php:97
2×Parameter #1 $num1 of function bcadd expects numeric-string, string given.src/Domain/Money/Money.php:78
2×Parameter #1 $num1 of function bcsub expects numeric-string, string given.src/Service/Repricer.php:97
2×Parameter #1 $num1 of function bcmul expects numeric-string, string given.src/Domain/Money/Money.php:60
2×Parameter #1 $num1 of function bccomp expects numeric-string, non-empty-string given.src/Import/PricesCatalogParser.php:217

phpmd — правила (наблюдение, в оценку не входит)

разправило
23×ShortVariable
18×StaticAccess
3×MissingImport
3×ElseExpression
1×UnusedFormalParameter
1×UnusedLocalVariable
1×ExcessiveMethodLength

Классы: размер и связность

Каждая точка — один класс решения. Горизонталь — LOC, строк кода. Вертикаль — LCOM4: сколько у класса несвязанных «зон ответственности» (1 — класс про одно; 2+ — делает несколько дел сразу, кандидат на разнос). Размер точки — WMC: суммарная сложность всех методов класса. Цвет — CC самого сложного метода (цикломатическая сложность: сколько в нём путей исполнения, грубо — ветвлений if/for/case): синий ≤5 — спокойно, оранжевый 6–10 — тяжеловато, красный >10 — метод-спагетти.

0123456704284126168210252App\Import\PricesCatalogParserLOC 237 · LCOM4 6 · WMC 52 · cc_max 10PricesCatalogParserApp\Import\UsageEventsParserLOC 87 · LCOM4 4 · WMC 21 · cc_max 7App\Entity\UsageEventLOC 143 · LCOM4 7 · WMC 17 · cc_max 1App\Entity\ProviderLOC 87 · LCOM4 3 · WMC 14 · cc_max 4App\Domain\Pricing\OffPeakLOC 56 · LCOM4 2 · WMC 12 · cc_max 5App\Entity\PriceRowLOC 112 · LCOM4 5 · WMC 12 · cc_max 3App\Service\PriceCatalogImporterLOC 100 · LCOM4 1 · WMC 12 · cc_max 6App\Service\RepricerLOC 122 · LCOM4 2 · WMC 12 · cc_max 6App\Domain\Pricing\PriceEntryLOC 60 · LCOM4 1 · WMC 11 · cc_max 6App\Service\BillingRaterLOC 101 · LCOM4 1 · WMC 10 · cc_max 4App\Command\AppBillingReportCommandLOC 41 · LCOM4 2 · WMC 9 · cc_max 7App\Domain\Money\MoneyLOC 69 · LCOM4 0 · WMC 9 · cc_max 3App\Service\ReportBuilderLOC 95 · LCOM4 1 · WMC 9 · cc_max 8App\Service\UsageEventImporterLOC 72 · LCOM4 1 · WMC 8 · cc_max 7App\Shared\IsoUtcLOC 31 · LCOM4 0 · WMC 6 · cc_max 4App\Entity\ModelLOC 39 · LCOM4 2 · WMC 5 · cc_max 1App\KernelLOC 23 · LCOM4 1 · WMC 5 · cc_max 3App\Command\AppBillingRepriceCommandLOC 39 · LCOM4 2 · WMC 4 · cc_max 2App\Command\AppPricesImportCommandLOC 39 · LCOM4 2 · WMC 4 · cc_max 2App\Command\AppUsageImportCommandLOC 39 · LCOM4 2 · WMC 4 · cc_max 2App\Command\AppBillingRateCommandLOC 29 · LCOM4 1 · WMC 3 · cc_max 2App\Domain\Pricing\EventRaterLOC 13 · LCOM4 1 · WMC 3 · cc_max 3App\Domain\Pricing\UsageRecordLOC 16 · LCOM4 1 · WMC 2 · cc_max 1App\Shared\ImportExceptionLOC 12 · LCOM4 0 · WMC 2 · cc_max 1App\Import\ParsedCatalogLOC 12 · LCOM4 0 · WMC 1 · cc_max 1App\Import\PriceDataLOC 13 · LCOM4 0 · WMC 1 · cc_max 1App\Import\ProviderDataLOC 10 · LCOM4 0 · WMC 1 · cc_max 1App\Import\UsageEventDataLOC 15 · LCOM4 0 · WMC 1 · cc_max 1LOC классакомпоненты LCOM4

Размер точки — WMC класса, цвет — максимальная CC метода: ● ≤5, ● 6–10, ● >10. Наведите на точку — полная сводка.

Матрица зависимостей классов

AppBillingRateCommanAppBillingRateCommanAppBillingReportCommAppBillingReportCommAppBillingRepriceComAppBillingRepriceComAppPricesImportCommaAppPricesImportCommaAppUsageImportCommanAppUsageImportCommanMoneyMoneyEventRaterEventRaterOffPeakOffPeakPriceEntryPriceEntryUsageRecordUsageRecordModelModelPriceRowPriceRowProviderProviderUsageEventUsageEventParsedCatalogParsedCatalogPriceDataPriceDataPricesCatalogParserPricesCatalogParserProviderDataProviderDataUsageEventDataUsageEventDataUsageEventsParserUsageEventsParserKernelKernelBillingRaterBillingRaterPriceCatalogImporterPriceCatalogImporterReportBuilderReportBuilderRepricerRepricerUsageEventImporterUsageEventImporterImportExceptionImportExceptionIsoUtcIsoUtcAppBillingRateCommand → BillingRater · 1AppBillingRateCommand → IsoUtc · 1AppBillingReportCommand → ReportBuilder · 1AppBillingReportCommand → ImportException · 1AppBillingReportCommand → IsoUtc · 1AppBillingRepriceCommand → Money · 1AppBillingRepriceCommand → PricesCatalogParser · 1AppBillingRepriceCommand → PriceCatalogImporter · 1AppBillingRepriceCommand → Repricer · 1AppBillingRepriceCommand → ImportException · 1AppPricesImportCommand → PricesCatalogParser · 1AppPricesImportCommand → PriceCatalogImporter · 1AppPricesImportCommand → ImportException · 1AppUsageImportCommand → UsageEventImporter · 1AppUsageImportCommand → ImportException · 1EventRater → Money · 1EventRater → OffPeak · 1EventRater → PriceEntry · 1EventRater → UsageRecord · 1OffPeak → Money · 1PriceEntry → Money · 1PriceEntry → UsageRecord · 1PriceEntry → ImportException · 1PriceRow → PriceEntry · 1Provider → OffPeak · 1Provider → ImportException · 1UsageEvent → UsageRecord · 1PricesCatalogParser → PriceEntry · 1PricesCatalogParser → ParsedCatalog · 1PricesCatalogParser → PriceData · 1PricesCatalogParser → ProviderData · 1PricesCatalogParser → ImportException · 1UsageEventsParser → UsageEventData · 1UsageEventsParser → ImportException · 1BillingRater → EventRater · 1BillingRater → OffPeak · 1BillingRater → PriceRow · 1BillingRater → Provider · 1BillingRater → UsageEvent · 1PriceCatalogImporter → PricesCatalogParser · 1PriceCatalogImporter → ImportException · 1ReportBuilder → Money · 1ReportBuilder → UsageEvent · 1Repricer → EventRater · 1Repricer → OffPeak · 1Repricer → PriceEntry · 1Repricer → PriceRow · 1Repricer → Provider · 1Repricer → UsageEvent · 1Repricer → PriceData · 1Repricer → PriceCatalogImporter · 1Repricer → ImportException · 1UsageEventImporter → UsageEventsParser · 1UsageEventImporter → ImportException · 1IsoUtc → ImportException · 1AppBillingRateCommand → AsCommand ×1, Command ×1, InputInterfac…AppBillingReportCommand → AsCommand ×1, Command ×1, InputInterf…AppBillingRepriceCommand → AsCommand ×1, Command ×1, InputArgum…AppPricesImportCommand → AsCommand ×1, Command ×1, InputArgumen…AppUsageImportCommand → AsCommand ×1, Command ×1, InputArgument…Money → bcadd ×1, bccomp ×1, bcdiv ×1, …EventRater → null ×1OffPeak → DateTimeImmutable ×1, InvalidArgumentException ×1, bc…PriceEntry → null ×1, sprintf ×1UsageRecord → DateTimeImmutable ×1Model → Column ×1, Entity ×1, GeneratedValue ×1, …PriceRow → DateTimeImmutable ×1, Column ×1, Entity ×1, …Provider → Column ×1, Entity ×1, Id ×1, …UsageEvent → DateTimeImmutable ×1, Column ×1, Entity ×1, …PriceData → DateTimeImmutable ×1PricesCatalogParser → DateTimeImmutable ×1, array_diff_key ×1, …UsageEventData → DateTimeImmutable ×1UsageEventsParser → DateTimeImmutable ×1, JSON_ERROR_NONE ×1, P…Kernel → LoaderInterface ×1, Kernel ×1, false ×1BillingRater → DateTimeImmutable ×1, EntityManagerInterface ×1,…PriceCatalogImporter → Connection ×1, JSON_ERROR_NONE ×1, false…ReportBuilder → DateTimeImmutable ×1, EntityManagerInterface ×1…Repricer → ArrayParameterType ×1, Connection ×1, EntityManagerI…UsageEventImporter → Connection ×1, count ×1, false ×1, …ImportException → RuntimeException ×1, sprintf ×1IsoUtc → DateTimeImmutable ×1, false ×1, preg_match ×1, …внеш.CommandDomainEntityImportService

Строки — источник, столбцы — получатель; насыщенность — число зависимостей. Классы отсортированы по неймспейсам: слои подписаны на диагонали, жирные линии их разделяют; жёлтый столбец «внеш.» — исходящие связи к типам вне решения (DTO времени, коллекции). Наведите на закрашенную ячейку — кто на кого и сколько раз.

Связанность классов: Ce, Ca, нестабильность I

Ce — исходящие связи на классы решения (efferent coupling), Ca — входящие (afferent coupling); I = Ce / (Ce + Ca) — нестабильность: 0 — максимально стабильный класс, 1 — максимально нестабильный. Здоровая слоистость выглядит так: I близка к 1 у console-команд (точки входа, обязаны меняться часто) и к 0 у примитивов домена (Decimal, Timestamp, enum-статусы): нестабильное должно зависеть от стабильного, не наоборот. Классы без внутренних связей не показаны.

КлассCeCaI (0…1)
AppBillingRateCommand201.00
AppBillingReportCommand301.00
AppBillingRepriceCommand501.00
AppPricesImportCommand301.00
AppUsageImportCommand201.00
Repricer910.90
BillingRater510.83
EventRater420.67
UsageEventsParser210.67
ReportBuilder210.67
UsageEventImporter210.67
PricesCatalogParser530.62
Provider220.50
PriceEntry340.43
PriceCatalogImporter230.40
PriceRow120.33
IsoUtc120.33
UsageEvent130.25
OffPeak140.20
Money050.00
UsageRecord030.00
ParsedCatalog010.00
PriceData020.00
ProviderData010.00
UsageEventData010.00
ImportException0120.00

Циклические зависимости

Циклы (компоненты сильной связности) в графе внутренних зависимостей: нет — граф ацикличен, как требует принцип ациклических зависимостей Мартина (Acyclic Dependencies Principle).

Классы по WMC (топ-10)

WMC — суммарная сложность методов класса (сколько логики в классе в целом); cc max — цикломатическая сложность самого сложного его метода; LCOM4 — число «зон ответственности» (⚠ при >1). Имя файла — ссылка на код в git (работает после коммита артефактов).

КлассLOCметодовWMCcc maxLCOM4
PricesCatalogParser237952 10⚠ 6
UsageEventsParser87421 7⚠ 4
UsageEvent1431717 1⚠ 7
Provider87914 4⚠ 3
Repricer122512 6⚠ 2
PriceRow1121012 3⚠ 5
PriceCatalogImporter100312 6 1
OffPeak56512 5⚠ 2
PriceEntry60411 6 1
BillingRater101510 4 1

Методы-нарушители порогов (cc > 10 или LOC > 50)

МетодCCLOC
ReportBuilder::csv884
UsageEventImporter::importFile757
Repricer::reprice668
PriceCatalogImporter::importFile570

Полные данные — metrics-raw/collector.json.

Отчёт по метрикам качества кода: metrics.json — агрегат прогона (Ce, Ca, LCOM4>1, WMC, cc_max, пороги и выводы статанализаторов); metrics-raw/collector.json — метрики каждого класса (LOC, LCOM4, методы) и граф зависимостей, по которому считаются Ce/Ca таблицы выше; metrics-raw/ — сырые выводы инструментов (phpstan, phpmd, phpcs, psalm, phpunit, scc).

Ярус 3 · ревью: изолированная pi-сессия glm-5.3-flash

Модельная проверка решения: отдельная pi-сессия (zai/glm-5.3-flash, thinking low) оценивает по рубрике обе оси. На вход — TASK.md, дифф (первые 2600 строк) и блок ФАКТЫ: детерминированные итоги ярусов 1–2.

3.5 из 4ось A «код и архитектура»
3.5 из 4ось B «процесс и коммуникация»
25 счистое время ревью
42.5Kтокенов (вход 35.9K, $0.0059)

Вход ревьюера: блок ФАКТЫ

Чекер: обязательных 11/11, бонус 2/2
Метрики: phpcs 45, psalm 119, phpstan 34, loc 1467
AST: cc_max 10, lcom4>1: 12, превышения порогов {'method_cc_over_10': 0, 'method_loc_over_50': 4, 'class_loc_over_300': 0, 'class_wmc_over_50': 1, 'class_lcom4_over_1': 12}

Обоснование

ревьюер яруса 3
Чекер: все 11 обязательных проверок и обе бонусные пройдены. Доменная модель аккуратная: чистая тарификация вынесена в Domain/Pricing (EventRater, OffPeak, PriceEntry), деньги — только bcmath со scale 16 и единственным округлением HALF_UP (Money::roundHalfUp6), окно [start,end) с переходом через полночь корректно покрыто тестами на границах. Идемпотентность — на UNIQUE + ON CONFLICT DO NOTHING, unrated-ретраи реализованы, reprice-бонус работает через price_row_id без задвоений. Дефекты: в коммит попал артефакт .phpunit.result.cache, причём в нём два теста MoneyTest помечены как упавшие ("defects":{"App\\Tests\\Domain\\MoneyTest::testAddIsExact":7,...}) — вопреки заявлению «32/32 OK», что подрывает доверие к самопроверке (итоговый код, судя по чекеру, рабочий, но след грязный); статанализ не вычищен (psalm 119, phpstan 34, phpcs 45 предупреждений); ReportBuilder тянет все события через ORM в память — для объёмов задания допустимо, но масштабируемость условная; в Repricer при отсутствии строки подставляется фиктивный PriceEntry('invalid',[]) — запутанный мёртвый путь. В остальном процесс сильный: развёрнутый PR_DESCRIPTION с допущениями, обоснованием отступлений (отсутствуют) и ручной проверкой граничных случаев (HALF_UP, границы окна, unrated→rate, reprice-дельта).

Артефакты: review-zcode.md · tier3.json

Ярус 4 · судья: слепое второе мнение

Та же модель судьи (zai/glm-5.3-flash, thinking low), но вслепую: только TASK.md и дифф, без фактов чекера и метрик. Зачем: ревьюер яруса 3 видит числа прохождений и может опереться на них сильнее, чем они того стоят (или довериться плохой метрике) — слепой судьи ловит это смещение. Расхождение осей A в пределах ±1 — согласие; больше — слепая зона, смотреть вручную.

3.5 из 4ось A «код и архитектура», вслепую
+0расхождение осей A: ярус 3 − ярус 4
18 счистое время судьи
42.0Kтокенов (вход 35.8K, $0.0058)
заметка судьи
Сильное решение: домен (Money/OffPeak/PriceEntry/EventRater) отделён от приёма и отчёта, точная bcmath-арифметика с единственным HALF_UP-округлением на событие, корректное окно через полночь, выбор цены по valid_from<=ts, дедупликация по уникальному индексу, реализован бонус reprice через price_row_id, 32 теста на edge-cases (границы окна, HALF_UP, unrated). Слабости: Money — статический хелпер строк, а не value object; ReportBuilder грузит все события в память и группирует по строковому ключу user|provider/model (потенциальная коллизия при '|'; в user); format6 обрезает хвост, а не бросает на неточном значении; оф-пик при reprice берётся текущий (ок, но не зафиксировано). До 4 не хватает аккуратности этих мелочей.

Цитаты-доказательства из диффа:

Артефакты: judge-glm.md · tier4.json