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

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

Тег orcarouter/Qwen3.8-27B-Uncensored:ctx100k · рассуждение включён · окно 102.4K · начало 2026-10-20 04:31

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

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

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

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

PI_CODING_AGENT_DIR=<tmp>/pi-home PI_OFFLINE=1 \
pi -p --session-dir <прогон>/pi-session \
    --model orcarouter/Qwen3.8-27B-Uncensored:ctx100k --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, автокомпакты, чем самопроверялся.

220вызовов API
12.0Mтокенов всего (вход 161.7K, выход 128.1K, кэш-чтение 11.7M — 98%)
86.5Kпик контекста из 102.4K
2автокомпактов и правок истории (компактов 2, правок 0)
280tool-вызовов (178 bash)
34 минчистой генерации

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

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

solution/ — 39
    config/ — 2
        packages/ — 1
    fixtures/ — 2
    migrations/ — 1
    src/ — 25
        Application/ — 9
            Parsers/ — 3
        Command/ — 5
        Domain/ — 11
            Billing/ — 3
            Catalog/ — 4
            Exception/ — 1
            Usage/ — 3
    tests/ — 5
        Application/ — 2
        Domain/ — 3
            Billing/ — 3

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

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

280tool-вызовов
bash17864%
edit4315%
write4215%
read176%

Утилиты в bash

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

553вызовов
echo9617%
console7714%
grep6111%
tail5710%
cat5310%
head448%
psql285%
sed275%
остальное11020%

«Остальное» — вызовы утилит вне топ-8: phpunit, php, rm, ls, python3, find, docker, git, diff, wc, cp, tr, mkdir, tee, rmdir, test, composer, stat, od, awk.

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

026.6K53.2K79.9K5м10м15м20м25м30м35млимит 102.4Kвремя от начала сессии (220 вызовов API)контекст, токены

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

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

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

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

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

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

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

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

Артефакты: 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/reportbin/console app:prices:import fixtures/prices.json && echo OK1 && bin/console app:usage:import fixtures/usage.
вне cwd/tmp/dbgcat > /tmp/dbg.php <<'EOF' ⏎ <?php ⏎ require 'vendor/autoload.php'; ⏎ use App\Kernel; ⏎ $kernel = new Kernel('
вне cwd/tmp/dbgsed -i "s/\$em = \$kernel->getContainer()->get(\\\\Doctrine\\\\ORM\\\\EntityManagerInterface::class);/\$em = \
вне cwd/tmp/dbgcat >> config/services.yaml <<'EOF' ⏎ ⏎ app.entity_manager: ⏎ alias: 'Doctrine\ORM\EntityManagerI
вне cwd/tmp/dbg2cat > /tmp/dbg2.php <<'EOF' ⏎ <?php ⏎ require 'vendor/autoload.php'; ⏎ use App\Kernel; ⏎ $kernel = new Kernel(
вне cwd/tmp/dbg2rm -rf var/cache; php /tmp/dbg2.php 2>&1 | grep -E 'Executing|INSERT|UPDATE' | head -40
вне cwd/tmp/dbg3cat > /tmp/dbg3.php <<'EOF' ⏎ <?php ⏎ require 'vendor/autoload.php'; ⏎ use App\Kernel; ⏎ $kernel = new Kernel(
вне cwd/tmp/dbg4cat > /tmp/dbg4.php <<'EOF' ⏎ <?php ⏎ require 'vendor/autoload.php'; ⏎ use App\Kernel; ⏎ $kernel = new Kernel(

…всего сигналов 60; полный список — security.json, построчный разбор — audit-verdict.md.

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

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

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

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

СценарийСтатусДетали
artifacts_presentpass
db_createpassCreated database "chk_36260" for connection named default
import_duplicatespass
migratepassmethod=migrations
prices_importpass
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изменены: README.md, fixtures/ (изменён)
unrated_stderrpassожидались ['UNRATED prism/h13 2026-07-31T23:00:00Z no_price'], получено ['UNRATED prism/h13 2026-07-31T23:00:00Z no_price']
usage_importpass

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

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

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

phpcs PSR12
101
среднее других прогонов модели: 45
psalm L1
93
среднее других прогонов модели: 115
phpstan L8
32
среднее других прогонов модели: 34
phpmd (наблюдение)
88
среднее других прогонов модели: 65

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

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

scc — объём кода

1147строк кода
274строк комментариев — 24% от строк кода
26файлов
файлов
26
среднее: 20
строк кода
1147
среднее: 1176
комментариев
274
среднее: 209
пустых строк
280
среднее: 268
суммарная сложность
80
среднее: 90

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

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

28тестов, все проходят
95проверок (ассерций)
429строк тестового кода
27.74%покрытие строк src/

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

разсообщение
36×[x] Expected at least 1 space before "."; 0 found
36×[x] Expected at least 1 space after "."; 0 found
6×[x] Closing brace must be on a line by itself
5×[x] Opening brace should be on a new line
4×[x] There must be a blank line following the last trait import
3×[ ] Line exceeds 120 characters; contains 128 characters

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

разтиппример
17×MixedArgumentsrc/Application/CatalogImporter.php:67
16×MixedAssignmentsrc/Application/CatalogImporter.php:41
12×ArgumentTypeCoercionsrc/Domain/Billing/Money.php:24
9×MissingOverrideAttributesrc/Command/AppBillingRateCommand.php:21
8×UndefinedInterfaceMethodsrc/Command/AppBillingRateCommand.php:26
8×MixedMethodCallsrc/Command/AppBillingRateCommand.php:26
5×RedundantCastGivenDocblockTypesrc/Command/AppBillingReportCommand.php:61
3×MixedArrayAccesssrc/Application/CatalogImporter.php:52

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

разсообщениепример
8×Call to an undefined method Symfony\Component\Console\Output\OutputInterface::getErrorOutput().src/Command/AppBillingRepriceCommand.php:56
3×Parameter #1 $num1 of function bcadd expects numeric-string, string given.src/Domain/Billing/Money.php:77
2×Property App\Domain\Catalog\PriceRow::$model (int) does not accept int|null.src/Application/RepriceService.php:75
2×Possibly invalid array key type int|null.src/Application/RepriceService.php:115
2×Parameter #1 $num1 of function bcmul expects numeric-string, string given.src/Domain/Billing/Money.php:56
2×Parameter #2 $num2 of function bcmul expects numeric-string, string given.src/Domain/Billing/Money.php:40

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

разправило
60×StaticAccess
13×ShortVariable
8×MissingImport
2×ErrorControlOperator
1×ElseExpression
1×UnusedLocalVariable
1×ExcessiveMethodLength
1×UnusedFormalParameter

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

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

01230295887116145174App\Application\Parsers\PricesFileLOC 165 · LCOM4 0 · WMC 40 · cc_max 12PricesFileApp\Application\Parsers\ScalarLOC 61 · LCOM4 0 · WMC 17 · cc_max 6App\Application\CatalogImporterLOC 103 · LCOM4 1 · WMC 15 · cc_max 11App\Command\AppBillingReportCommandLOC 88 · LCOM4 3 · WMC 14 · cc_max 8App\Application\RatingServiceLOC 71 · LCOM4 1 · WMC 11 · cc_max 10App\Application\UsageImporterLOC 75 · LCOM4 1 · WMC 11 · cc_max 7App\Domain\Billing\MoneyLOC 66 · LCOM4 0 · WMC 10 · cc_max 3App\Domain\Billing\OffPeakWindowLOC 37 · LCOM4 1 · WMC 10 · cc_max 5App\Application\RepriceServiceLOC 101 · LCOM4 1 · WMC 9 · cc_max 8App\Command\AppPricesImportCommandLOC 47 · LCOM4 3 · WMC 8 · cc_max 4App\Command\AppBillingRepriceCommandLOC 40 · LCOM4 2 · WMC 7 · cc_max 5App\Command\AppUsageImportCommandLOC 40 · LCOM4 2 · WMC 7 · cc_max 5App\Domain\Billing\RateCalculatorLOC 43 · LCOM4 0 · WMC 7 · cc_max 4App\Application\ReportServiceLOC 65 · LCOM4 1 · WMC 5 · cc_max 4App\KernelLOC 23 · LCOM4 1 · WMC 5 · cc_max 3App\Command\AppBillingRateCommandLOC 32 · LCOM4 1 · WMC 4 · cc_max 3App\Application\Parsers\UsageEventLineLOC 35 · LCOM4 0 · WMC 3 · cc_max 3App\Domain\Catalog\PriceRowLOC 46 · LCOM4 2 · WMC 3 · cc_max 2App\Domain\Catalog\ProviderLOC 30 · LCOM4 1 · WMC 3 · cc_max 3App\Domain\Usage\UsageEventLOC 70 · LCOM4 1 · WMC 2 · cc_max 1App\Domain\Exception\ImportExceptionLOC 7 · LCOM4 0 · WMC 1 · cc_max 1App\Domain\Catalog\ModelLOC 16 · LCOM4 0 · WMC 0 · cc_max 0App\Domain\Usage\RepriceEntryLOC 26 · LCOM4 0 · WMC 0 · cc_max 0LOC классакомпоненты LCOM4

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

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

CatalogImporterCatalogImporterPricesFilePricesFileScalarScalarUsageEventLineUsageEventLineRatingServiceRatingServiceReportServiceReportServiceRepriceServiceRepriceServiceUsageImporterUsageImporterUsesTransactionsUsesTransactionsAppBillingRateCommanAppBillingRateCommanAppBillingReportCommAppBillingReportCommAppBillingRepriceComAppBillingRepriceComAppPricesImportCommaAppPricesImportCommaAppUsageImportCommanAppUsageImportCommanMoneyMoneyOffPeakWindowOffPeakWindowRateCalculatorRateCalculatorModelModelPriceRowPriceRowPricingUnitPricingUnitProviderProviderImportExceptionImportExceptionEventStatusEventStatusRepriceEntryRepriceEntryUsageEventUsageEventKernelKernelCatalogImporter → PricesFile · 1CatalogImporter → UsesTransactions · 1CatalogImporter → Model · 1CatalogImporter → PriceRow · 1CatalogImporter → Provider · 1CatalogImporter → ImportException · 1PricesFile → Scalar · 1PricesFile → ImportException · 1Scalar → ImportException · 1UsageEventLine → Scalar · 1UsageEventLine → ImportException · 1RatingService → UsesTransactions · 1RatingService → OffPeakWindow · 1RatingService → RateCalculator · 1RatingService → Model · 1RatingService → PriceRow · 1RatingService → Provider · 1RatingService → EventStatus · 1RatingService → UsageEvent · 1ReportService → Money · 1ReportService → EventStatus · 1RepriceService → PricesFile · 1RepriceService → UsesTransactions · 1RepriceService → Money · 1RepriceService → OffPeakWindow · 1RepriceService → RateCalculator · 1RepriceService → Model · 1RepriceService → PriceRow · 1RepriceService → Provider · 1RepriceService → ImportException · 1RepriceService → EventStatus · 1RepriceService → RepriceEntry · 1RepriceService → UsageEvent · 1UsageImporter → UsageEventLine · 1UsageImporter → UsesTransactions · 1UsageImporter → UsageEvent · 1AppBillingRateCommand → RatingService · 1AppBillingReportCommand → Scalar · 1AppBillingReportCommand → ReportService · 1AppBillingReportCommand → ImportException · 1AppBillingRepriceCommand → RepriceService · 1AppBillingRepriceCommand → Money · 1AppBillingRepriceCommand → ImportException · 1AppPricesImportCommand → CatalogImporter · 1AppPricesImportCommand → ImportException · 1AppUsageImportCommand → UsageImporter · 1AppUsageImportCommand → ImportException · 1OffPeakWindow → Provider · 1RateCalculator → Money · 1RateCalculator → OffPeakWindow · 1RateCalculator → PricingUnit · 1RateCalculator → UsageEvent · 1PriceRow → PricingUnit · 1UsageEvent → Money · 1UsageEvent → EventStatus · 1CatalogImporter → DateTimeImmutable ×1, EntityManagerInterface …PricesFile → JSON_THROW_ON_ERROR ×1, array_is_list ×1, array_ke…Scalar → DateTimeImmutable ×1, DateTimeZone ×1, Exception ×1, …UsageEventLine → JSON_THROW_ON_ERROR ×1, is_array ×1, json_deco…RatingService → DateTimeImmutable ×1, EntityManagerInterface ×1…ReportService → DateTimeImmutable ×1, EntityManagerInterface ×1…RepriceService → DateTimeImmutable ×1, EntityManagerInterface ×…UsageImporter → EntityManagerInterface ×1, Generator ×1, count …UsesTransactions → Throwable ×1AppBillingRateCommand → AsCommand ×1, Command ×1, InputInterfac…AppBillingReportCommand → PHP_EOL ×1, AsCommand ×1, Command ×1,…AppBillingRepriceCommand → EntityManagerInterface ×1, AsCommand…AppPricesImportCommand → AsCommand ×1, Command ×1, InputArgumen…AppUsageImportCommand → JsonException ×1, AsCommand ×1, Command…Money → InvalidArgumentException ×1, bcadd ×1, bccomp ×1, …OffPeakWindow → DateTimeImmutable ×1, false ×1, null ×1RateCalculator → array_map ×1, null ×1Model → Column ×1, Entity ×1, GeneratedValue ×1, …PriceRow → DateTimeImmutable ×1, Column ×1, Entity ×1, …Provider → Column ×1, Entity ×1, Id ×1, …ImportException → RuntimeException ×1, sprintf ×1RepriceEntry → DateTimeImmutable ×1, Column ×1, Entity ×1, …UsageEvent → DateTimeImmutable ×1, Column ×1, Entity ×1, …Kernel → LoaderInterface ×1, Kernel ×1, false ×1внеш.ApplicationCommandDomain

Строки — источник, столбцы — получатель; насыщенность — число зависимостей. Классы отсортированы по неймспейсам: слои подписаны на диагонали, жирные линии их разделяют; жёлтый столбец «внеш.» — исходящие связи к типам вне решения (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)
AppBillingRateCommand101.00
AppBillingReportCommand301.00
AppBillingRepriceCommand301.00
AppPricesImportCommand201.00
AppUsageImportCommand201.00
RepriceService1210.92
RatingService810.89
CatalogImporter610.86
UsageImporter310.75
UsageEventLine210.67
ReportService210.67
RateCalculator420.67
PricesFile220.50
UsageEvent240.33
Scalar130.25
OffPeakWindow130.25
PriceRow130.25
UsesTransactions040.00
Money050.00
Model030.00
PricingUnit020.00
Provider040.00
ImportException090.00
EventStatus040.00
RepriceEntry010.00

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

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

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

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

КлассLOCметодовWMCcc maxLCOM4
PricesFile165640⚠ 12 0
Scalar61517 6 0
CatalogImporter103415⚠ 11 1
AppBillingReportCommand88414 8⚠ 3
UsageImporter75311 7 1
RatingService71211 10 1
Money66710 3 0
OffPeakWindow37310 5 1
RepriceService10129 8 1
AppPricesImportCommand4748 4⚠ 3

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

МетодCCLOC
PricesFile::rates1235
CatalogImporter::import1169
RatingService::rateUnrated1062
RepriceService::reprice892
UsageImporter::import753
ReportService::report452

Полные данные — 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.

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

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

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

Обоснование

ревьюер яруса 3
Обязательная часть и бонус проходят чекер полностью (11/11 + 2/2). Домен смоделирован чисто: Money/OffPeakWindow/RateCalculator отделены от импорта и отчёта, деньги — только bcmath (scale 20, единственное округление HALF_UP до 6 на событии, Money::roundHalfUpTo6), время на DateTimeImmutable UTC. Граничные случаи покрыты тестами: полуинтервал окна и обход полуночи (OffPeakWindowTest), граница valid_from, дедуп по (provider, provider_request_id), unrated с повторной попыткой. Бонус reprice реализован с журналом reprice_entries и проверкой «e2 не тронут». Замечания: в committed .phpunit.result.cache видны дефекты прошлых прогонов тестов (defects у 7 тестов) — мусорный артефакт в диффе; ReportService строит WHERE конкатенацией строк (значение статуса — константа, инъекции нет, но сырой SQL вместо QB, вопреки заявлению в PR_DESCRIPTION про Doctrine QueryBuilder); scalar dedup через findOneBy на каждую строку — N+1 при больших файлах (приемлемо для объёмов задания); статанализ зашумлён (phpcs 101, phpstan 32); в PR_DESCRIPTION опечатки («заднеи», «строгу»). Процесс сильный: пошаговая приёмка с байт-в-байт сверкой (od -c), контекстный тест на живой PostgreSQL через proc_open, обоснованные отступления (user_id, flush перед commit в ORM 3), честное описание допущений — не дотягивает до 4 только из-за мусорного кеш-файла и расхождения текста с кодом.

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

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

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

3 из 4ось A «код и архитектура», вслепую
+1расхождение осей A: ярус 3 − ярус 4
20 счистое время судьи
44.7Kтокенов (вход 38.4K, $0.0062)
заметка судьи
Сильные стороны: чёткое разделение слоёв (Domain/Billing с Money/OffPeakWindow/RateCalculator, Application-сервисы, тонкие команды), точная десятичная арифметика без float, единственное округление HALF_UP на событии, корректные off-peak-границы и обход полуночи, идемпотентность через уникальные ключи и статусы pending/rated, неплохой набор тестов включая сквозной на живом PostgreSQL. Смущает: в коммит попал .phpunit.result.cache с дефектами у части тестов (признак непрогнанного финального прогона), опечатки в коде и PR_DESCRIPTION («входецких», «заднеи», «строгу»), сравнение дат в SQL — конкатенацией строки со status, неудобная сигнатура eventCost (окно и множитель отдельными nullable-аргументами), для rating все строки цен грузятся в память. До 4 не хватает аккуратности подачи и немного «дешевизны» решений.

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

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