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

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

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

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

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

11 из 11обязательные сценарии
2 из 2бонусные сценарии
3.5 из 4код и архитектура
4 из 4процесс и коммуникация
3.5 из 4слепое ревью
25 миндлительность прогона
9.0Mтокенов, кэш 96%
3441строк
70файлов в решении (без vendor)

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

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

152вызовов API
9.0Mтокенов всего (вход 214.3K, выход 140.8K, кэш-чтение 8.7M — 96%)
87.0Kпик контекста из 102.4K
3автокомпактов и правок истории (компактов 3, правок 0)
223tool-вызовов (93 bash)
22 минчистой генерации

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

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

solution/ — 49
    migrations/ — 1
    src/ — 39
        Command/ — 5
        Domain/ — 11
        Entity/ — 6
        Import/ — 12
        Repository/ — 5
    tests/ — 6

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

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

223tool-вызовов
bash9342%
write6127%
read4118%
edit2813%

Утилиты в bash

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

426вызовов
echo7518%
console6716%
head5212%
grep4511%
tail368%
psql317%
rm225%
cat215%
остальное7718%

«Остальное» — вызовы утилит вне топ-8: ls, php, find, sed, docker, git, make, mv, phpunit, diff, ss, podman, date, sort, composer, tee, printf, python3, cp.

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

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

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

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

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

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

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

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

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

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

Артефакты: 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/vendor/find /tmp/billing-pi-hfr4u4ph/repo -maxdepth 2 -not -path '*/vendor/*' -not -path '*/.git/*' | head -50
вне cwd (запись)/tmp/reprice_test.jsonwrite /tmp/reprice_test.json
вне cwd/tmp/reprice_testcd /tmp/billing-pi-hfr4u4ph/repo && echo "=== reprice #1 ===" && bin/console app:billing:reprice /tmp/reprice_
вне cwd/tmp/reprice_testcd /tmp/billing-pi-hfr4u4ph/repo && echo "=== report after reprice ===" && bin/console app:billing:report && e
вне cwd/tmp/unrated_usagecd /tmp/billing-pi-hfr4u4ph/repo && PGPASSWORD=app psql -h 127.0.0.1 -p 5544 -U app -d app -tAc "SELECT code,
вне cwd (запись)/tmp/reprice_sfera.jsonwrite /tmp/reprice_sfera.json
вне cwd/tmp/reprice_sferacd /tmp/billing-pi-hfr4u4ph/repo && bin/console app:billing:reprice /tmp/reprice_sfera.json; echo "rc=$?"; bin
вне cwd/tmp/boundary_usagecd /tmp/billing-pi-hfr4u4ph/repo && cat > /tmp/boundary_usage.jsonl <<'EOF' ⏎ {"provider": "prism", "provider_

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

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

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

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

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

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

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

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

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

phpcs PSR12
46
среднее других прогонов модели: 68
psalm L1
101
среднее других прогонов модели: 133
phpstan L8
42
среднее других прогонов модели: 46
phpmd (наблюдение)
75
среднее других прогонов модели: 80

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

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

scc — объём кода

1798строк кода
345строк комментариев — 19% от строк кода
40файлов
файлов
40
среднее: 29
строк кода
1798
среднее: 1501
комментариев
345
среднее: 315
пустых строк
452
среднее: 353
суммарная сложность
95
среднее: 95

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

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

34тестов, все проходят
72проверок (ассерций)
341строк тестового кода
30.4%покрытие строк src/

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

разсообщение
11×[x] Expected at least 1 space before "."; 0 found
11×[x] Expected at least 1 space after "."; 0 found
5×Line exceeds 120 characters; contains 126 characters
2×Line exceeds 120 characters; contains 132 characters
2×Line exceeds 120 characters; contains 121 characters
2×Line exceeds 120 characters; contains 135 characters

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

разтиппример
26×MixedMethodCallsrc/Command/BillingRateCommand.php:32
17×ArgumentTypeCoercionsrc/Domain/Money.php:29
17×MixedArgumentsrc/Domain/RatingRunner.php:52
14×UndefinedInterfaceMethodsrc/Command/BillingRateCommand.php:32
9×MissingOverrideAttributesrc/Command/BillingRateCommand.php:27
6×MixedAssignmentsrc/Command/BillingReportCommand.php:57
2×PossiblyInvalidArgumentsrc/Command/BillingReportCommand.php:48
2×RedundantCastGivenDocblockTypesrc/Entity/PriceRow.php:143

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

разсообщениепример
14×Call to an undefined method Symfony\Component\Console\Output\OutputInterface::getErrorOutput().src/Command/UsageImportCommand.php:60
3×Parameter #1 $num1 of function bccomp expects numeric-string, string given.src/Domain/Money.php:65
2×Parameter #1 $num1 of function bcmul expects numeric-string, string given.src/Domain/Money.php:70
1×Parameter #1 $from of method App\Repository\EventRatingRepository::reportRows() expects DateTimeImmutable|null, DateTimesrc/Command/BillingReportCommand.php:48
1×Parameter #2 $to of method App\Repository\EventRatingRepository::reportRows() expects DateTimeImmutable|null, DateTimeImsrc/Command/BillingReportCommand.php:48
1×Match expression does not handle remaining value: stringsrc/Domain/CostCalculator.php:43

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

разправило
32×StaticAccess
31×ShortVariable
11×MissingImport
1×UnusedFormalParameter

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

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

02468100295887116145174App\Import\PricesParserLOC 164 · LCOM4 5 · WMC 32 · cc_max 11PricesParserApp\Entity\PriceRowLOC 132 · LCOM4 4 · WMC 21 · cc_max 4PriceRowApp\Import\PricesImporterLOC 106 · LCOM4 1 · WMC 21 · cc_max 6PricesImporterApp\Domain\OffPeakWindowLOC 55 · LCOM4 3 · WMC 16 · cc_max 5OffPeakWindowApp\Domain\CostCalculatorLOC 94 · LCOM4 0 · WMC 14 · cc_max 5App\Entity\ProviderLOC 85 · LCOM4 4 · WMC 14 · cc_max 4App\Import\AbstractJsonParserLOC 61 · LCOM4 4 · WMC 13 · cc_max 5App\Domain\RepriceRunnerLOC 78 · LCOM4 1 · WMC 12 · cc_max 9App\Entity\UsageEventLOC 116 · LCOM4 10 · WMC 11 · cc_max 1App\Domain\RatingRunnerLOC 72 · LCOM4 1 · WMC 10 · cc_max 9App\Command\BillingReportCommandLOC 55 · LCOM4 3 · WMC 9 · cc_max 4App\Domain\MoneyLOC 62 · LCOM4 0 · WMC 9 · cc_max 2App\Entity\EventRatingLOC 77 · LCOM4 6 · WMC 8 · cc_max 1App\Import\UsageParserLOC 62 · LCOM4 2 · WMC 8 · cc_max 4App\Command\BillingRepriceCommandLOC 51 · LCOM4 2 · WMC 7 · cc_max 5App\Command\PricesImportCommandLOC 54 · LCOM4 2 · WMC 7 · cc_max 5App\Command\UsageImportCommandLOC 53 · LCOM4 2 · WMC 7 · cc_max 5App\Domain\ReportBuilderLOC 73 · LCOM4 1 · WMC 7 · cc_max 4App\Entity\RepriceCorrectionLOC 70 · LCOM4 6 · WMC 7 · cc_max 1App\Import\UsageImporterLOC 63 · LCOM4 1 · WMC 7 · cc_max 6App\Repository\EventRatingRepositoryLOC 69 · LCOM4 2 · WMC 7 · cc_max 5App\Entity\ModelLOC 43 · LCOM4 3 · WMC 5 · cc_max 1App\Import\PriceCatalogLOC 38 · LCOM4 1 · WMC 5 · cc_max 4App\KernelLOC 23 · LCOM4 1 · WMC 5 · cc_max 3App\Command\BillingRateCommandLOC 36 · LCOM4 1 · WMC 4 · cc_max 3App\Domain\IsoTimeLOC 32 · LCOM4 0 · WMC 4 · cc_max 2App\Repository\UsageEventRepositoryLOC 32 · LCOM4 2 · WMC 4 · cc_max 2App\Repository\PriceRowRepositoryLOC 46 · LCOM4 1 · WMC 3 · cc_max 1App\Domain\RatingSummaryLOC 16 · LCOM4 1 · WMC 2 · cc_max 1App\Repository\ModelRepositoryLOC 12 · LCOM4 1 · WMC 2 · cc_max 1App\Repository\ProviderRepositoryLOC 12 · LCOM4 1 · WMC 2 · cc_max 1App\Domain\ReportRowLOC 15 · LCOM4 0 · WMC 1 · cc_max 1App\Domain\RepriceResultLOC 9 · LCOM4 0 · WMC 1 · cc_max 1App\Domain\UnratedEventLOC 9 · LCOM4 0 · WMC 1 · cc_max 1App\Import\ModelSpecLOC 8 · LCOM4 0 · WMC 1 · cc_max 1App\Import\PriceSpecLOC 13 · LCOM4 0 · WMC 1 · cc_max 1App\Import\PricesPayloadLOC 14 · LCOM4 0 · WMC 1 · cc_max 1App\Import\ProviderSpecLOC 20 · LCOM4 0 · WMC 1 · cc_max 1App\Import\UsageEventLineLOC 15 · LCOM4 0 · WMC 1 · cc_max 1App\Import\ImportExceptionLOC 3 · LCOM4 0 · WMC 0 · cc_max 0LOC классакомпоненты LCOM4

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

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

BillingRateCommandBillingRateCommandBillingReportCommandBillingReportCommandBillingRepriceCommanBillingRepriceCommanPricesImportCommandPricesImportCommandUsageImportCommandUsageImportCommandCostCalculatorCostCalculatorIsoTimeIsoTimeMoneyMoneyOffPeakWindowOffPeakWindowRatingRunnerRatingRunnerRatingSummaryRatingSummaryReportBuilderReportBuilderReportRowReportRowRepriceResultRepriceResultRepriceRunnerRepriceRunnerUnratedEventUnratedEventEventRatingEventRatingModelModelPriceRowPriceRowProviderProviderRepriceCorrectionRepriceCorrectionUsageEventUsageEventAbstractJsonParserAbstractJsonParserImportExceptionImportExceptionModelSpecModelSpecPriceCatalogPriceCatalogPriceSpecPriceSpecPricesImporterPricesImporterPricesParserPricesParserPricesPayloadPricesPayloadProviderSpecProviderSpecUsageEventLineUsageEventLineUsageImporterUsageImporterUsageParserUsageParserKernelKernelEventRatingRepositorEventRatingRepositorModelRepositoryModelRepositoryPriceRowRepositoryPriceRowRepositoryProviderRepositoryProviderRepositoryUsageEventRepositoryUsageEventRepositoryBillingRateCommand → IsoTime · 1BillingRateCommand → RatingRunner · 1BillingReportCommand → IsoTime · 1BillingReportCommand → ReportBuilder · 1BillingReportCommand → EventRatingRepository · 1BillingRepriceCommand → RepriceRunner · 1BillingRepriceCommand → ImportException · 1BillingRepriceCommand → PricesParser · 1PricesImportCommand → ImportException · 1PricesImportCommand → PricesImporter · 1PricesImportCommand → PricesParser · 1UsageImportCommand → ImportException · 1UsageImportCommand → UsageImporter · 1UsageImportCommand → UsageParser · 1CostCalculator → Money · 1CostCalculator → OffPeakWindow · 1RatingRunner → CostCalculator · 1RatingRunner → RatingSummary · 1RatingRunner → UnratedEvent · 1RatingRunner → EventRating · 1RatingRunner → PriceRowRepository · 1RatingRunner → ProviderRepository · 1RatingRunner → UsageEventRepository · 1ReportBuilder → Money · 1RepriceRunner → CostCalculator · 1RepriceRunner → Money · 1RepriceRunner → RepriceResult · 1RepriceRunner → RepriceCorrection · 1RepriceRunner → ImportException · 1RepriceRunner → PriceCatalog · 1RepriceRunner → PricesPayload · 1RepriceRunner → EventRatingRepository · 1RepriceRunner → ProviderRepository · 1EventRating → PriceRow · 1EventRating → UsageEvent · 1EventRating → EventRatingRepository · 1Model → ModelRepository · 1PriceRow → Money · 1PriceRow → Model · 1PriceRow → PriceRowRepository · 1Provider → Money · 1Provider → OffPeakWindow · 1Provider → ProviderRepository · 1RepriceCorrection → PriceRow · 1RepriceCorrection → UsageEvent · 1UsageEvent → UsageEventRepository · 1AbstractJsonParser → ImportException · 1PriceCatalog → PriceRow · 1PriceCatalog → ImportException · 1PriceCatalog → PriceSpec · 1PriceCatalog → ModelRepository · 1PriceCatalog → PriceRowRepository · 1PricesImporter → Model · 1PricesImporter → Provider · 1PricesImporter → ImportException · 1PricesImporter → ModelSpec · 1PricesImporter → PriceCatalog · 1PricesImporter → PricesPayload · 1PricesImporter → ProviderSpec · 1PricesImporter → ModelRepository · 1PricesImporter → ProviderRepository · 1PricesParser → CostCalculator · 1PricesParser → IsoTime · 1PricesParser → AbstractJsonParser · 1PricesParser → ImportException · 1PricesParser → ModelSpec · 1PricesParser → PriceSpec · 1PricesParser → PricesPayload · 1PricesParser → ProviderSpec · 1UsageImporter → UsageEvent · 1UsageImporter → ImportException · 1UsageImporter → UsageEventRepository · 1UsageParser → IsoTime · 1UsageParser → AbstractJsonParser · 1UsageParser → ImportException · 1UsageParser → UsageEventLine · 1EventRatingRepository → IsoTime · 1EventRatingRepository → ReportRow · 1EventRatingRepository → EventRating · 1EventRatingRepository → PriceRow · 1ModelRepository → Model · 1PriceRowRepository → Model · 1PriceRowRepository → PriceRow · 1ProviderRepository → Provider · 1UsageEventRepository → UsageEvent · 1BillingRateCommand → RuntimeException ×1, AsCommand ×1, Command…BillingReportCommand → DateTimeImmutable ×1, AsCommand ×1, Comm…BillingRepriceCommand → AsCommand ×1, Command ×1, InputArgument…PricesImportCommand → AsCommand ×1, Command ×1, InputArgument ×…UsageImportCommand → AsCommand ×1, Command ×1, InputArgument ×1…CostCalculator → DateTimeImmutable ×1, DateTimeZone ×1, DomainE…IsoTime → DateTimeImmutable ×1, DateTimeZone ×1, ValueError ×1,…Money → ValueError ×1, bcadd ×1, bccomp ×1, …OffPeakWindow → ValueError ×1, false ×1, preg_match ×1, …RatingRunner → DateTimeImmutable ×1, DateTimeZone ×1, EntityMan…RatingSummary → count ×1ReportBuilder → array_values ×1, implode ×1, preg_match ×1, …RepriceRunner → DateTimeImmutable ×1, DateTimeZone ×1, EntityMa…UnratedEvent → DateTimeImmutable ×1EventRating → DateTimeImmutable ×1, DateTimeZone ×1, Types ×1, …Model → Types ×1, Column ×1, Entity ×1, …PriceRow → DateTimeImmutable ×1, DateTimeZone ×1, Types ×1, …Provider → Types ×1, Column ×1, Entity ×1, …RepriceCorrection → DateTimeImmutable ×1, DateTimeZone ×1, Type…UsageEvent → DateTimeImmutable ×1, DateTimeZone ×1, Types ×1, …AbstractJsonParser → is_int ×1, is_string ×1, null ×1, …ImportException → RuntimeException ×1PriceCatalog → EntityManagerInterface ×1, false ×1, null ×1, …PriceSpec → DateTimeImmutable ×1PricesImporter → EntityManagerInterface ×1, Throwable ×1, in_ar…PricesParser → JSON_THROW_ON_ERROR ×1, JsonException ×1, ValueE…UsageEventLine → DateTimeImmutable ×1UsageImporter → DateTimeImmutable ×1, DateTimeZone ×1, EntityMa…UsageParser → JSON_THROW_ON_ERROR ×1, JsonException ×1, ValueEr…Kernel → LoaderInterface ×1, Kernel ×1, false ×1EventRatingRepository → DateTimeImmutable ×1, ServiceEntityRepo…ModelRepository → ServiceEntityRepository ×1, ManagerRegistry ×1PriceRowRepository → DateTimeImmutable ×1, DateTimeZone ×1, Ser…ProviderRepository → ServiceEntityRepository ×1, ManagerRegistr…UsageEventRepository → ServiceEntityRepository ×1, ManagerRegis…внеш.CommandDomainEntityImportRepository

Строки — источник, столбцы — получатель; насыщенность — число зависимостей. Классы отсортированы по неймспейсам: слои подписаны на диагонали, жирные линии их разделяют; жёлтый столбец «внеш.» — исходящие связи к типам вне решения (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)
BillingRateCommand201.00
BillingReportCommand301.00
BillingRepriceCommand301.00
PricesImportCommand301.00
UsageImportCommand301.00
RepriceRunner910.90
PricesImporter910.90
RatingRunner710.88
PricesParser820.80
UsageParser410.80
UsageImporter310.75
PriceCatalog520.71
RepriceCorrection210.67
EventRating320.60
Provider320.60
EventRatingRepository430.57
ReportBuilder110.50
CostCalculator230.40
PriceRowRepository230.40
PriceRow350.38
AbstractJsonParser120.33
ModelRepository130.25
UsageEventRepository130.25
Model140.20
UsageEvent140.20
ProviderRepository140.20
IsoTime050.00
Money050.00
OffPeakWindow020.00
RatingSummary010.00
ReportRow010.00
RepriceResult010.00
UnratedEvent010.00
ImportException0100.00
ModelSpec020.00
PriceSpec020.00
PricesPayload030.00
ProviderSpec020.00
UsageEventLine010.00

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

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

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

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

КлассLOCметодовWMCcc maxLCOM4
PricesParser164832⚠ 11⚠ 5
PriceRow1321521 4⚠ 4
PricesImporter106721 6 1
OffPeakWindow55716 5⚠ 3
CostCalculator94514 5 0
Provider85814 4⚠ 4
AbstractJsonParser61413 5⚠ 4
RepriceRunner78312 9 1
UsageEvent1161111 1⚠ 10
RatingRunner72210 9 1

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

МетодCCLOC
PricesParser::parsePriceRow1144
RatingRunner::rateAll961
RepriceRunner::reprice960

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

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

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

Обоснование

ревьюер яруса 3
Сильное решение: точные деньги на bcmath (Money::SCALE=30, одно HALF_UP-округление в CostCalculator::eventCost), честная доменная модель (тарификация в event_ratings отделена от приёма и отчёта), дедупликация по (provider, provider_request_id) с уникальным индексом, unrated как состояние, бонус reprice с аудитом и идемпотентностью. Edge-case'ы (границы окна, valid_from включительно, HALF_UP vs HALF_EVEN, дубль id у разных провайдеров) покрыты тестами и зафиксированы в PR_DESCRIPTION. Дефекты: паттерн N+1 в RatingRunner::rateAll (findActiveAt/findByCode на каждое событие) и в UsageImporter (проверка дубля построчно) — для CLI терпимо, но «дешёвым» решение не назвать; findUnrated делает два запроса; в репозиторий попал артефакт .phpunit.result.cache с записями о дефектах (MoneyTest defects:7), что намекает на отладочный след и мусор в коммите; статанализ выдаёт заметные числа замечаний (psalm 101, phpcs 46, phpstan 42 — вероятно, в основном подавления/стиль, чекер обязательные пункты не снимал). A — 3.5 из-за N+1 и артефактов, не 4; B — 4: систематическая самопроверка (байт-в-байт diff отчёта, schema:validate, повторный импорт, 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
17 счистое время судьи
50.5Kтокенов (вход 44.3K, $0.0070)
заметка судьи
Сильное решение: чистый домен (Money на bcmath, CostCalculator, OffPeakWindow) отделён от приёма/отчёта, единое HALF_UP-округление события, суммы отчёта из уже округлённых стоимостей, UTC-строгость (IsoTime с суффиксом Z, минута суток UTC), окно через полночь и границы [start, end) покрыты тестами; дедуп по (provider, provider_request_id) и upsert цен по (provider, model, valid_from) через уникальные индексы; бонус reprice с аудитом и идемпотентным повтором. Слабости: RatingRunner/RepriceRunner в Domain завязаны на EntityManager и репозитории (граница слоя условна); в закоммиченном .phpunit.result.cache есть defects по MoneyTest (след неудавшегося прогона в диффе); roundHalfUp через floor-трюк работает, но только для неотрицательных — допущение задокументировано, тесты это подтверждают.

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

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