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

qwen3.8-27b-pi-100k · run-08

Тег qwen3.8:27b-pi-100k · рассуждение low · окно 102.4K · начало 2026-10-27 06:36

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

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

11 из 11обязательные сценарии
2 из 2бонусные сценарии
4 из 4код и архитектура
4 из 4процесс и коммуникация
4 из 4слепое ревью
39 миндлительность прогона
11.3Mтокенов, кэш 90%
3349строк
68файлов в решении (без vendor)

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

PI_CODING_AGENT_DIR=<tmp>/pi-home PI_OFFLINE=1 \
pi -p --session-dir <прогон>/pi-session \
    --model qwen3.8:27b-pi-100k --thinking low \
    "Прочитай 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, автокомпакты, чем самопроверялся.

200вызовов API
11.3Mтокенов всего (вход 916.2K, выход 164.0K, кэш-чтение 10.2M — 90%)
87.4Kпик контекста из 102.4K
3автокомпактов и правок истории (компактов 3, правок 0)
242tool-вызовов (115 bash)
37 минчистой генерации

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

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

solution/ — 51
    config/ — 1
        packages/ — 1
    migrations/ — 1
    src/ — 37
        Billing/ — 36
            Command/ — 5
            Entity/ — 4
            Enum/ — 2
            Import/ — 9
                Dto/ — 5
            Pricing/ — 1
            Repository/ — 2
            Service/ — 8
            ValueObject/ — 4
    tests/ — 7
        Billing/ — 6
            Import/ — 2

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

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

242tool-вызовов
bash11548%
write5724%
edit5021%
read208%

Утилиты в bash

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

438вызовов
php8118%
echo7216%
cat368%
head358%
grep338%
tail307%
phpunit307%
console276%
остальное9421%

«Остальное» — вызовы утилит вне топ-8: psql, ls, sed, docker, diff, rm, make, find, awk, P, git, sort, time, printf, which.

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

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

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

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

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

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

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

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

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

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

Артефакты: 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/reprocd /tmp/billing-pi-w2hjhith/repo && cat > /tmp/repro.php <<'EOF' ⏎ <?php ⏎ class D { ⏎ public function __c
вне cwd/tmp/repro2cd /tmp/billing-pi-w2hjhith/repo && cat > /tmp/repro2.php <<'EOF' ⏎ <?php ⏎ $groups = []; ⏎ $e = ["u-bob", "pr
вне cwd/tmp/repro3cd /tmp/billing-pi-w2hjhith/repo && cat > /tmp/repro3.php <<'EOF' ⏎ <?php ⏎ class D { ⏎ public function __
вне cwd (запись)/tmp/correction.jsonwrite /tmp/correction.json
вне cwd/tmp/correctioncd /tmp/billing-pi-w2hjhith/repo && bin/console app:billing:reprice /tmp/correction.json; echo "exit=$?"; echo
вне cwd/tmp/unratedcd /tmp/billing-pi-w2hjhith/repo && cat > /tmp/unrated.jsonl <<'EOF' ⏎ {"provider": "prism", "provider_request
вне cwd/tmp/offpeakcd /tmp/billing-pi-w2hjhith/repo && cat > /tmp/offpeak.jsonl <<'EOF' ⏎ {"provider": "prism", "provider_request
вне cwd/tmp/billing-pi-wi2hjhith/repocd /tmp/billing-pi-wi2hjhith/repo 2>/dev/null || cd /tmp/billing-pi-w2hjhith/repo; cat .env.test; echo ===; ve

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

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

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

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

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

СценарийСтатусДетали
artifacts_presentpass
db_createpassCreated database "chk_92341" for connection named default
import_duplicatespass[OK] Принято событий: 1, проигнорировано дублей: 3.
migratepassmethod=migrations
prices_importpass[OK] Импортировано: провайдеров 2, моделей 3, строк цен 4.
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изменены: phpunit.xml.dist
unrated_stderrpassожидались ['UNRATED prism/h13 2026-07-31T23:00:00Z no_price'], получено ['UNRATED prism/h13 2026-07-31T23:00:00Z no_price']
usage_importpass[OK] Принято событий: 18, проигнорировано дублей: 0.

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

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

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

phpcs PSR12
69
среднее других прогонов модели: 67
psalm L1
58
среднее других прогонов модели: 136
phpstan L8
37
среднее других прогонов модели: 46
phpmd (наблюдение)
80
среднее других прогонов модели: 79

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

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

scc — объём кода

1483строк кода
334строк комментариев — 23% от строк кода
37файлов
файлов
37
среднее: 29
строк кода
1483
среднее: 1525
комментариев
334
среднее: 315
пустых строк
359
среднее: 360
суммарная сложность
71
среднее: 97

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

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

49тестов, все проходят
92проверок (ассерций)
556строк тестового кода
77.59%покрытие строк src/

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

разсообщение
23×[x] Expected at least 1 space before "."; 0 found
23×[x] Expected at least 1 space after "."; 0 found
6×[x] Blank lines are not allowed in a multi-line function
2×[ ] Line exceeds 120 characters; contains 155 characters
1×[ ] Line exceeds 120 characters; contains 137 characters
1×[x] The first expression of a multi-line control structure

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

разтиппример
16×ArgumentTypeCoercionsrc/Billing/Import/PricesFileParser.php:38
9×MissingOverrideAttributesrc/Billing/Command/BillingRateCommand.php:26
9×MixedAssignmentsrc/Billing/Entity/PriceRow.php:70
5×MixedArgumentsrc/Billing/Import/Json.php:40
4×MixedReturnStatementsrc/Billing/Repository/PriceRowRepository.php:33
2×UndefinedInterfaceMethodsrc/Billing/Command/BillingRateCommand.php:32
2×MixedMethodCallsrc/Billing/Command/BillingRateCommand.php:32
2×RedundantCastGivenDocblockTypesrc/Billing/Entity/PriceRow.php:71

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

разсообщениепример
2×Call to an undefined method Symfony\Component\Console\Output\OutputInterface::getErrorOutput(). [identifier=method.notFosrc/Billing/Command/BillingReportCommand.php:60
2×Parameter #1 $num1 of function bcadd expects numeric-string, string given. [identifier=argument.type]src/Billing/ValueObject/Decimal.php:126
2×Parameter #1 $num1 of function bcmul expects numeric-string, string given. [identifier=argument.type]src/Billing/ValueObject/Decimal.php:115
1×Method App\Billing\Import\Json::array() return type has no value type specified in iterable type array. [identifier=misssrc/Billing/Import/Json.php:21
1×Method App\Billing\Import\Json::stringField() has parameter $data with no value type specified in iterable type array. [src/Billing/Import/Json.php:38
1×PHPDoc tag @param for parameter $data with type mixed is not subtype of native type array. [identifier=parameter.phpDocTsrc/Billing/Import/Json.php:38

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

разправило
57×StaticAccess
14×ShortVariable
2×UnusedFormalParameter
2×ElseExpression
2×MissingImport
1×TooManyFields
1×TooManyPublicMethods
1×ShortMethodName

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

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

01234024487296120144App\Billing\ValueObject\DecimalLOC 121 · LCOM4 1 · WMC 28 · cc_max 12DecimalApp\Billing\Import\JsonLOC 106 · LCOM4 0 · WMC 22 · cc_max 7JsonApp\Billing\Service\RepriceServiceLOC 117 · LCOM4 2 · WMC 21 · cc_max 6RepriceServiceApp\Billing\Import\PricesFileParserLOC 134 · LCOM4 4 · WMC 18 · cc_max 6PricesFileParserApp\Billing\Service\PriceListImporterLOC 62 · LCOM4 1 · WMC 10 · cc_max 7App\Billing\Command\PricesImportCommandLOC 65 · LCOM4 3 · WMC 9 · cc_max 4App\Billing\Command\BillingReportCommandLOC 50 · LCOM4 3 · WMC 8 · cc_max 3App\Billing\ValueObject\OffPeakLOC 42 · LCOM4 1 · WMC 8 · cc_max 4App\Billing\Command\BillingRepriceCommandLOC 51 · LCOM4 2 · WMC 7 · cc_max 5App\Billing\Command\UsageImportCommandLOC 54 · LCOM4 2 · WMC 7 · cc_max 5App\Billing\Entity\ProviderLOC 56 · LCOM4 1 · WMC 7 · cc_max 4App\Billing\Pricing\RateCalculatorLOC 33 · LCOM4 3 · WMC 7 · cc_max 4App\Billing\Service\RateEngineLOC 51 · LCOM4 1 · WMC 7 · cc_max 3App\Billing\ValueObject\PriceLOC 41 · LCOM4 1 · WMC 7 · cc_max 4App\Billing\Repository\UsageEventRepositoryLOC 65 · LCOM4 1 · WMC 6 · cc_max 3App\KernelLOC 32 · LCOM4 2 · WMC 6 · cc_max 3App\Billing\Entity\PriceRowLOC 58 · LCOM4 1 · WMC 5 · cc_max 2App\Billing\Import\UsageFileParserLOC 49 · LCOM4 2 · WMC 5 · cc_max 3App\Billing\Service\ReportServiceLOC 74 · LCOM4 1 · WMC 5 · cc_max 4App\Billing\Entity\UsageEventLOC 96 · LCOM4 2 · WMC 4 · cc_max 1App\Billing\Service\UsageImporterLOC 45 · LCOM4 1 · WMC 4 · cc_max 3App\Billing\Command\BillingRateCommandLOC 32 · LCOM4 1 · WMC 3 · cc_max 2App\Billing\Repository\PriceRowRepositoryLOC 34 · LCOM4 2 · WMC 3 · cc_max 1App\Billing\Entity\ModelLOC 24 · LCOM4 1 · WMC 2 · cc_max 1App\Billing\ValueObject\TokenUsageLOC 16 · LCOM4 1 · WMC 2 · cc_max 1App\Billing\Import\Dto\ModelInputLOC 8 · LCOM4 0 · WMC 1 · cc_max 1App\Billing\Import\Dto\PriceRowInputLOC 11 · LCOM4 0 · WMC 1 · cc_max 1App\Billing\Import\Dto\PricesFileLOC 14 · LCOM4 0 · WMC 1 · cc_max 1App\Billing\Import\Dto\ProviderInputLOC 9 · LCOM4 0 · WMC 1 · cc_max 1App\Billing\Import\Dto\UsageEventInputLOC 12 · LCOM4 0 · WMC 1 · cc_max 1App\Billing\Service\RateRunResultLOC 9 · LCOM4 0 · WMC 1 · cc_max 1App\Billing\Service\RepriceResultLOC 9 · LCOM4 0 · WMC 1 · cc_max 1App\Billing\Service\UsageImportResultLOC 8 · LCOM4 0 · WMC 1 · cc_max 1App\Billing\BillingLOC 8 · LCOM4 0 · WMC 0 · cc_max 0App\Billing\Import\ImportExceptionLOC 3 · LCOM4 0 · WMC 0 · cc_max 0LOC классакомпоненты LCOM4

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

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

BillingBillingBillingRateCommandBillingRateCommandBillingReportCommandBillingReportCommandBillingRepriceCommanBillingRepriceCommanPricesImportCommandPricesImportCommandUsageImportCommandUsageImportCommandModelModelPriceRowPriceRowProviderProviderUsageEventUsageEventPricingUnitPricingUnitUsageEventStatusUsageEventStatusModelInputModelInputPriceRowInputPriceRowInputPricesFilePricesFileProviderInputProviderInputUsageEventInputUsageEventInputImportExceptionImportExceptionJsonJsonPricesFileParserPricesFileParserUsageFileParserUsageFileParserRateCalculatorRateCalculatorPriceRowRepositoryPriceRowRepositoryUsageEventRepositoryUsageEventRepositoryPriceListImporterPriceListImporterRateEngineRateEngineRateRunResultRateRunResultReportServiceReportServiceRepriceResultRepriceResultRepriceServiceRepriceServiceUsageImportResultUsageImportResultUsageImporterUsageImporterDecimalDecimalOffPeakOffPeakPricePriceTokenUsageTokenUsageKernelKernelBillingRateCommand → RateEngine · 1BillingReportCommand → ReportService · 1BillingRepriceCommand → ImportException · 1BillingRepriceCommand → PricesFileParser · 1BillingRepriceCommand → RepriceService · 1PricesImportCommand → ImportException · 1PricesImportCommand → PricesFileParser · 1PricesImportCommand → PriceListImporter · 1UsageImportCommand → ImportException · 1UsageImportCommand → UsageFileParser · 1UsageImportCommand → UsageImporter · 1PriceRow → PricingUnit · 1PriceRow → PriceRowInput · 1PriceRow → Decimal · 1PriceRow → Price · 1Provider → ProviderInput · 1Provider → Decimal · 1Provider → OffPeak · 1UsageEvent → UsageEventStatus · 1UsageEvent → Decimal · 1UsageEvent → TokenUsage · 1PricingUnit → ImportException · 1PriceRowInput → Price · 1ProviderInput → OffPeak · 1UsageEventInput → TokenUsage · 1Json → ImportException · 1PricesFileParser → PricingUnit · 1PricesFileParser → ModelInput · 1PricesFileParser → PriceRowInput · 1PricesFileParser → PricesFile · 1PricesFileParser → ProviderInput · 1PricesFileParser → ImportException · 1PricesFileParser → Json · 1PricesFileParser → OffPeak · 1PricesFileParser → Price · 1UsageFileParser → UsageEventInput · 1UsageFileParser → ImportException · 1UsageFileParser → Json · 1UsageFileParser → TokenUsage · 1RateCalculator → PricingUnit · 1RateCalculator → Decimal · 1RateCalculator → Price · 1RateCalculator → TokenUsage · 1PriceRowRepository → PriceRow · 1UsageEventRepository → UsageEvent · 1UsageEventRepository → UsageEventStatus · 1PriceListImporter → Model · 1PriceListImporter → PriceRow · 1PriceListImporter → Provider · 1PriceListImporter → PriceRowInput · 1PriceListImporter → PricesFile · 1PriceListImporter → PriceRowRepository · 1RateEngine → Billing · 1RateEngine → PriceRow · 1RateEngine → Provider · 1RateEngine → UsageEvent · 1RateEngine → RateCalculator · 1RateEngine → PriceRowRepository · 1RateEngine → UsageEventRepository · 1RateEngine → RateRunResult · 1RateEngine → Decimal · 1ReportService → Billing · 1ReportService → UsageEventRepository · 1ReportService → Decimal · 1RepriceService → Billing · 1RepriceService → Model · 1RepriceService → PriceRow · 1RepriceService → Provider · 1RepriceService → UsageEvent · 1RepriceService → PriceRowInput · 1RepriceService → PricesFile · 1RepriceService → ImportException · 1RepriceService → RateCalculator · 1RepriceService → PriceRowRepository · 1RepriceService → UsageEventRepository · 1RepriceService → RepriceResult · 1RepriceService → Decimal · 1UsageImporter → UsageEvent · 1UsageImporter → UsageEventRepository · 1UsageImporter → UsageImportResult · 1OffPeak → ImportException · 1OffPeak → Decimal · 1Price → PricingUnit · 1Price → ImportException · 1Price → Decimal · 1BillingRateCommand → AsCommand ×1, Command ×1, InputInterface ×…BillingReportCommand → DateTimeImmutable ×1, DateTimeZone ×1, E…BillingRepriceCommand → AsCommand ×1, Command ×1, InputArgument…PricesImportCommand → AsCommand ×1, Command ×1, InputArgument ×…UsageImportCommand → AsCommand ×1, Command ×1, InputArgument ×1…Model → Column ×1, Entity ×1, Id ×1, …PriceRow → DateTimeImmutable ×1, Column ×1, Entity ×1, …Provider → Column ×1, Entity ×1, Id ×1, …UsageEvent → DateTimeImmutable ×1, Column ×1, Entity ×1, …PricingUnit → ValueError ×1, sprintf ×1PriceRowInput → DateTimeImmutable ×1UsageEventInput → DateTimeImmutable ×1ImportException → RuntimeException ×1Json → DateTimeImmutable ×1, DateTimeZone ×1, Exception ×1, …PricesFileParser → JSON_BIGINT_AS_STRING ×1, JSON_THROW_ON_ERRO…UsageFileParser → JSON_BIGINT_AS_STRING ×1, JSON_THROW_ON_ERROR…RateCalculator → null ×1PriceRowRepository → DateTimeImmutable ×1, ServiceEntityReposit…UsageEventRepository → DateTimeImmutable ×1, ServiceEntityRepos…PriceListImporter → EntityManagerInterface ×1, JSON_THROW_ON_ER…RateEngine → DateTimeImmutable ×1, EntityManagerInterface ×1, n…ReportService → DateTimeImmutable ×1, implode ×1, ksort ×1RepriceService → DateTimeImmutable ×1, EntityManagerInterface ×…UsageImporter → DateTimeImmutable ×1, EntityManagerInterface ×1…Decimal → InvalidArgumentException ×1, STR_PAD_LEFT ×1, STR_PAD…OffPeak → DateTimeImmutable ×1, preg_match ×1, sprintf ×1Price → LogicException ×1, sprintf ×1Kernel → LoaderInterface ×1, Kernel ×1, date_default_timezone_s…внеш.Billing

Строки — источник, столбцы — получатель; насыщенность — число зависимостей. Классы отсортированы по неймспейсам: слои подписаны на диагонали, жирные линии их разделяют; жёлтый столбец «внеш.» — исходящие связи к типам вне решения (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)
BillingRateCommand101.00
BillingReportCommand101.00
BillingRepriceCommand301.00
PricesImportCommand301.00
UsageImportCommand301.00
RepriceService1310.93
RateEngine910.90
PriceListImporter610.86
PricesFileParser920.82
UsageFileParser410.80
ReportService310.75
UsageImporter310.75
RateCalculator420.67
PriceRow440.50
Provider330.50
UsageEventInput110.50
UsageEvent340.43
Price340.43
OffPeak230.40
ProviderInput120.33
Json120.33
UsageEventRepository240.33
PriceRowRepository130.25
PricingUnit140.20
PriceRowInput140.20
Billing030.00
Model020.00
UsageEventStatus020.00
ModelInput010.00
PricesFile030.00
ImportException0100.00
RateRunResult010.00
RepriceResult010.00
UsageImportResult010.00
Decimal090.00
TokenUsage040.00

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

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

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

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

КлассLOCметодовWMCcc maxLCOM4
Decimal1211328⚠ 12 1
Json106522 7 0
RepriceService117621 6⚠ 2
PricesFileParser134518 6⚠ 4
PriceListImporter62310 7 1
PricesImportCommand6549 4⚠ 3
BillingReportCommand5048 3⚠ 3
OffPeak4248 4 1
Provider5637 4 1
UsageImportCommand5437 5⚠ 2

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

МетодCCLOC
Decimal::round1231
RepriceService::reprice655
ReportService::build465

Полные данные — 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 «код и архитектура»
4 из 4ось B «процесс и коммуникация»
39 счистое время ревью
105.5Kтокенов (вход 46.7K, $0.0091)

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

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

Обоснование

ревьюер яруса 3
Решение функционально: чекер 11/11 обязательных и 2/2 бонуса. Доменная модель чистая: тарификация (RateCalculator) отделена от приёма (импортеры с дедупом по (provider, provider_request_id) через естественный PK) и отчёта; деньги — только Decimal на bcmath (SCALE=40), единственное округление HALF_UP события, суммы — уже округлённых стоимостей (ReportService). Время аккуратно: timestamptz + ALTER DATABASE SET TIME ZONE 'UTC', off-peak с границами [start,end) и переносом через полночь, покрыты тестами (граничные 16:30/00:30, unrated retry, дубли, ретро-бонус с дельтой 9.600000). Reprice реализован через price_row_id — осознанный простой механизм. Мелкие дефекты: в дифф попал мусорный .phpunit.result.cache с recorded defects (засорение коммита); в RateRunResult.php мёртвые импорты (Provider, RateCalculator, EntityManagerInterface и т.п.); в RateEngine запрос цены на каждое событие по одному (findApplicable/em->find в цикле) — N+1, для тестового масштаба допустимо; подозрительный докблок «Инвариантно? Нет —» в Decimal.php. Метрики (phpstan 37, phpcs 69, lcom4>1 у 11 классов) снижают полировку, но не корректность. Процесс образцовый: системная самопроверка (приёмка по шагам, 49 тестов, изоляция тестовой БД), PR_DESCRIPTION объясняет все 5 отступлений от каркаса с обоснованием, COMMIT_MESSAGE осмысленный.

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

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

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

4 из 4ось A «код и архитектура», вслепую
+0расхождение осей A: ярус 3 − ярус 4
31 счистое время судьи
52.5Kтокенов (вход 46.3K, $0.0074)
заметка судьи
Полноценная доменная модель: чистый RateCalculator без ORM, Decimal на bcmath с масштабом 40 и единственным HALF_UP-округлением события, OffPeak с корректной границей [start, end) и переносом через полночь, идемпотентность на естественном ключе (provider, provider_request_id), unrated-ретраи, бонусный reprice с привязкой события к price_row_id. 49 тестов, включая интеграцию с эталонным CSV и ретраем unrated, добавляют реальную уверенность. Мелочи: странный docblock «Инвариантно? Нет», отчёт грузит все события в память (для объёма задания приемлемо), сортировка групп по составному строковому ключу — работает, но менее явна, чем ORDER BY. Код в целом читается как объяснение домена.

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

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