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

huihui-ai-pi-100k · run-02

Тег huihui-ai:pi-100k · рассуждение xhigh · окно 102.4K · начало 2026-10-17 08:12

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

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

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

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

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

330вызовов API
19.6Mтокенов всего (вход 378.2K, выход 218.7K, кэш-чтение 19.0M — 97%)
86.7Kпик контекста из 102.4K
5автокомпактов и правок истории (компактов 5, правок 0)
373tool-вызовов (215 bash)
35 минчистой генерации

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

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

solution/ — 49
    config/ — 1
        packages/ — 1
    migrations/ — 1
    src/ — 39
        Application/ — 16
            Billing/ — 5
            Parse/ — 1
            Prices/ — 7
            Usage/ — 3
        Command/ — 5
        Doctrine/ — 1
        Domain/ — 12
            Billing/ — 4
            Pricing/ — 4
            Shared/ — 2
            Usage/ — 2
        Exception/ — 4
    tests/ — 5

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

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

373tool-вызовов
bash21558%
edit7821%
write5114%
read298%

Утилиты в bash

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

798вызовов
echo17522%
php17422%
grep10713%
head9812%
cat638%
tail405%
sed395%
psql243%
остальное7810%

«Остальное» — вызовы утилит вне топ-8: ls, rm, docker, cmp, make, git, find, wc, diff, sort, ss, console, awk, md5sum.

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

026.6K53.2K79.9K10м20м30м40млимит 102.4Kвремя от начала сессии (330 вызовов API)контекст, токены

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

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

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

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

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

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

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

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

Артефакты: 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/list_servicescd /tmp/billing-pi-zdkmfh18/repo && cat > /tmp/list_services.php <<'PHP' ⏎ <?php ⏎ require 'vendor/autoload.ph
вне cwd/tmp/ctxcd /tmp/billing-pi-zdkmfh18/repo && cat > /tmp/ctx.php <<'PHP' ⏎ <?php ⏎ require 'vendor/autoload.php'; ⏎ ⏎ u
вне cwd/tmp/ctx2cd /tmp/billing-pi-zdkmfh18/repo && cat > /tmp/ctx2.php <<'PHP' ⏎ <?php ⏎ require 'vendor/autoload.php'; ⏎ $k
вне cwd/tmp/ctx3cd /tmp/billing-pi-zdkmfh18/repo && cat > /tmp/ctx3.php <<'PHP' ⏎ <?php ⏎ require 'vendor/autoload.php'; ⏎ use
вне cwd/tmp/compcheckcd /tmp/billing-pi-zdkmfh18/repo && cat > /tmp/compcheck.php <<'PHP' ⏎ <?php ⏎ require 'vendor/autoload.php';
вне cwd/tmp/minicd /tmp/billing-pi-zdkmfh18/repo && cat > /tmp/mini.php <<'PHP' ⏎ <?php ⏎ $mk = static function (string $label
вне cwd/tmp/mini2cd /tmp/billing-pi-zdkmfh18/repo && php -v | head -1; echo '<?php ⏎ $a = 1; ⏎ if ($a instanceof \\ArrayObject)
вне cwd/tmp/expected_reportcd /tmp/billing-pi-zdkmfh18/repo && cat > /tmp/expected_report.csv <<'EOF' ⏎ user,model,tokens_input,tokens_ou

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

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

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

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

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

СценарийСтатусДетали
artifacts_presentpass
db_createpassCreated database "chk_67810" for connection named default
import_duplicatespassevents: 4
inserted: 1
duplicates ignored: 3
migratepassmethod=migrations
prices_importpassprices imported: providers=2 models=3 price_rows=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_importpassevents: 18
inserted: 18
duplicates ignored: 0

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

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

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

phpcs PSR12
96
среднее других прогонов модели: 36
psalm L1
111
среднее других прогонов модели: 83
phpstan L8
49
среднее других прогонов модели: 35
phpmd (наблюдение)
71
среднее других прогонов модели: 75

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

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

scc — объём кода

1899строк кода
466строк комментариев — 25% от строк кода
39файлов
файлов
39
среднее: 27
строк кода
1899
среднее: 1353
комментариев
466
среднее: 293
пустых строк
464
среднее: 321
суммарная сложность
106
среднее: 83

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

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

76тестов, все проходят
106проверок (ассерций)
473строк тестового кода
26.39%покрытие строк src/

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

разсообщение
45×[x] Expected at least 1 space after "."; 0 found
24×[x] Expected at least 1 space before "."; 0 found
3×[ ] Line exceeds 120 characters; contains 135 characters
2×Line exceeds 120 characters; contains 141 characters
2×Line exceeds 120 characters; contains 121 characters
2×[ ] Line exceeds 120 characters; contains 144 characters

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

разтиппример
21×MixedMethodCallsrc/Command/ImportPricesCommand.php:48
15×MixedArgumentsrc/Application/Billing/RatingService.php:50
13×ArgumentTypeCoercionsrc/Application/Prices/PriceFileParser.php:122
11×MixedAssignmentsrc/Application/Billing/RatingService.php:38
11×MissingOverrideAttributesrc/Command/ImportPricesCommand.php:29
5×InvalidThrowsrc/Application/Parse/JsonInput.php:30
5×InvalidStringClasssrc/Application/Parse/JsonInput.php:30
5×UndefinedInterfaceMethodsrc/Command/ImportPricesCommand.php:43

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

разсообщениепример
8×Invalid type object to throw.src/Application/Parse/JsonInput.php:119
5×Call to an undefined method Symfony\Component\Console\Output\OutputInterface::getErrorOutput().src/Command/RepriceCommand.php:47
5×Attribute class Doctrine\ORM\Mapping\TypedIdentifier does not exist.src/Domain/Usage/UsageEvent.php:30
2×Parameter #1 $num1 of function bcmul expects numeric-string, string given.src/Domain/Shared/Decimal.php:74
2×Parameter #2 $num2 of function bcmul expects numeric-string, string given.src/Domain/Shared/Decimal.php:74
1×Method App\Application\Billing\RepriceService::reprice() return type has no value type specified in iterable type array.src/Application/Billing/RepriceService.php:52

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

разправило
33×StaticAccess
21×ShortVariable
6×ElseExpression
5×MissingImport
3×ErrorControlOperator
1×ExcessiveMethodLength
1×UnusedFormalParameter
1×TooManyFields

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

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

012345678903570105140175210App\Application\Prices\PriceFileParserLOC 198 · LCOM4 1 · WMC 45 · cc_max 11PriceFileParserApp\Domain\Shared\DecimalLOC 140 · LCOM4 0 · WMC 31 · cc_max 8DecimalApp\Domain\Pricing\PriceRowLOC 139 · LCOM4 5 · WMC 23 · cc_max 6PriceRowApp\Application\Parse\JsonInputLOC 109 · LCOM4 0 · WMC 20 · cc_max 4App\Application\Billing\RepriceServiceLOC 196 · LCOM4 1 · WMC 19 · cc_max 5App\Domain\Usage\UsageEventLOC 180 · LCOM4 9 · WMC 19 · cc_max 1App\Application\Prices\PricesImportServiceLOC 154 · LCOM4 1 · WMC 16 · cc_max 5App\Domain\Pricing\ProviderLOC 107 · LCOM4 3 · WMC 13 · cc_max 5App\Command\ReportCommandLOC 76 · LCOM4 3 · WMC 12 · cc_max 7App\Application\Billing\ReportServiceLOC 108 · LCOM4 1 · WMC 10 · cc_max 9App\Command\ImportPricesCommandLOC 74 · LCOM4 2 · WMC 8 · cc_max 6App\Command\ImportUsageCommandLOC 71 · LCOM4 2 · WMC 8 · cc_max 6App\Command\RepriceCommandLOC 73 · LCOM4 2 · WMC 8 · cc_max 6App\Application\Usage\UsageFileParserLOC 51 · LCOM4 1 · WMC 7 · cc_max 7App\Domain\Billing\OffPeakPolicyLOC 35 · LCOM4 2 · WMC 7 · cc_max 5App\Domain\Billing\PriceCorrectionLOC 68 · LCOM4 6 · WMC 7 · cc_max 1App\Domain\Billing\RateCalculatorLOC 39 · LCOM4 1 · WMC 7 · cc_max 5App\Application\Billing\RatingServiceLOC 69 · LCOM4 2 · WMC 5 · cc_max 3App\Application\Usage\UsageImportServiceLOC 66 · LCOM4 1 · WMC 5 · cc_max 4App\Command\RateCommandLOC 50 · LCOM4 2 · WMC 5 · cc_max 3App\KernelLOC 23 · LCOM4 1 · WMC 5 · cc_max 3App\Domain\Pricing\ModelLOC 38 · LCOM4 3 · WMC 4 · cc_max 1App\Domain\Shared\TimeOfDayLOC 29 · LCOM4 0 · WMC 4 · cc_max 2App\Doctrine\UserQuotedQuoteStrategyLOC 14 · LCOM4 1 · WMC 2 · cc_max 2App\Domain\Billing\TokenCountsLOC 18 · LCOM4 1 · WMC 2 · cc_max 1App\Application\Billing\RatingResultLOC 11 · LCOM4 0 · WMC 1 · cc_max 1App\Application\Billing\UnratedEventLOC 9 · LCOM4 0 · WMC 1 · cc_max 1App\Application\Prices\ModelSpecLOC 8 · LCOM4 0 · WMC 1 · cc_max 1App\Application\Prices\OffPeakSpecLOC 9 · LCOM4 0 · WMC 1 · cc_max 1App\Application\Prices\PriceFileSpecLOC 14 · LCOM4 0 · WMC 1 · cc_max 1App\Application\Prices\PriceRowSpecLOC 15 · LCOM4 0 · WMC 1 · cc_max 1App\Application\Prices\ProviderSpecLOC 9 · LCOM4 0 · WMC 1 · cc_max 1App\Application\Usage\UsageEventSpecLOC 13 · LCOM4 0 · WMC 1 · cc_max 1App\Exception\BillingExceptionLOC 3 · LCOM4 0 · WMC 0 · cc_max 0App\Exception\InvalidPricesFileExceptionLOC 3 · LCOM4 0 · WMC 0 · cc_max 0App\Exception\InvalidRepriceFileExceptionLOC 3 · LCOM4 0 · WMC 0 · cc_max 0App\Exception\InvalidUsageFileExceptionLOC 3 · LCOM4 0 · WMC 0 · cc_max 0LOC классакомпоненты LCOM4

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

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

RatingResultRatingResultRatingServiceRatingServiceReportServiceReportServiceRepriceServiceRepriceServiceUnratedEventUnratedEventJsonInputJsonInputModelSpecModelSpecOffPeakSpecOffPeakSpecPriceFileParserPriceFileParserPriceFileSpecPriceFileSpecPriceRowSpecPriceRowSpecPricesImportServicePricesImportServiceProviderSpecProviderSpecUsageEventSpecUsageEventSpecUsageFileParserUsageFileParserUsageImportServiceUsageImportServiceImportPricesCommandImportPricesCommandImportUsageCommandImportUsageCommandRateCommandRateCommandReportCommandReportCommandRepriceCommandRepriceCommandUserQuotedQuoteStratUserQuotedQuoteStratOffPeakPolicyOffPeakPolicyPriceCorrectionPriceCorrectionRateCalculatorRateCalculatorTokenCountsTokenCountsModelModelPriceRowPriceRowPriceUnitPriceUnitProviderProviderDecimalDecimalTimeOfDayTimeOfDayUsageEventUsageEventUsageStatusUsageStatusBillingExceptionBillingExceptionInvalidPricesFileExcInvalidPricesFileExcInvalidRepriceFileExInvalidRepriceFileExInvalidUsageFileExceInvalidUsageFileExceKernelKernelRatingService → RatingResult · 1RatingService → UnratedEvent · 1RatingService → RateCalculator · 1RatingService → PriceRow · 1RatingService → TimeOfDay · 1RatingService → UsageEvent · 1RatingService → UsageStatus · 1ReportService → Decimal · 1ReportService → UsageStatus · 1RepriceService → PriceFileSpec · 1RepriceService → ProviderSpec · 1RepriceService → PriceCorrection · 1RepriceService → RateCalculator · 1RepriceService → Model · 1RepriceService → PriceRow · 1RepriceService → Provider · 1RepriceService → Decimal · 1RepriceService → TimeOfDay · 1RepriceService → UsageStatus · 1RepriceService → InvalidRepriceFileException · 1JsonInput → TimeOfDay · 1PriceFileParser → JsonInput · 1PriceFileParser → ModelSpec · 1PriceFileParser → OffPeakSpec · 1PriceFileParser → PriceFileSpec · 1PriceFileParser → PriceRowSpec · 1PriceFileParser → ProviderSpec · 1PriceFileParser → PriceUnit · 1PriceFileParser → InvalidPricesFileException · 1PriceRowSpec → PriceUnit · 1PricesImportService → PriceFileSpec · 1PricesImportService → Model · 1PricesImportService → PriceRow · 1PricesImportService → Provider · 1PricesImportService → InvalidPricesFileException · 1ProviderSpec → OffPeakSpec · 1UsageEventSpec → TokenCounts · 1UsageFileParser → JsonInput · 1UsageFileParser → UsageEventSpec · 1UsageFileParser → TokenCounts · 1UsageFileParser → InvalidUsageFileException · 1UsageImportService → Model · 1UsageImportService → Provider · 1UsageImportService → UsageEvent · 1UsageImportService → InvalidUsageFileException · 1ImportPricesCommand → PriceFileParser · 1ImportPricesCommand → PricesImportService · 1ImportPricesCommand → InvalidPricesFileException · 1ImportUsageCommand → UsageFileParser · 1ImportUsageCommand → UsageImportService · 1ImportUsageCommand → InvalidUsageFileException · 1RateCommand → RatingService · 1ReportCommand → ReportService · 1ReportCommand → JsonInput · 1ReportCommand → BillingException · 1RepriceCommand → RepriceService · 1RepriceCommand → PriceFileParser · 1RepriceCommand → Decimal · 1RepriceCommand → InvalidPricesFileException · 1RepriceCommand → InvalidRepriceFileException · 1PriceCorrection → PriceRow · 1PriceCorrection → UsageEvent · 1RateCalculator → OffPeakPolicy · 1RateCalculator → TokenCounts · 1RateCalculator → PriceRow · 1RateCalculator → PriceUnit · 1RateCalculator → Decimal · 1Model → Provider · 1PriceRow → Model · 1PriceRow → PriceUnit · 1PriceRow → Provider · 1PriceRow → Decimal · 1Provider → OffPeakPolicy · 1Provider → Decimal · 1UsageEvent → TokenCounts · 1UsageEvent → Model · 1UsageEvent → PriceRow · 1UsageEvent → Provider · 1UsageEvent → UsageStatus · 1InvalidPricesFileException → BillingException · 1InvalidRepriceFileException → BillingException · 1InvalidUsageFileException → BillingException · 1RatingService → DateTimeImmutable ×1, Types ×1, EntityManagerIn…ReportService → DateTimeImmutable ×1, Types ×1, EntityManagerIn…RepriceService → EntityManagerInterface ×1, array_keys ×1, arra…UnratedEvent → DateTimeImmutable ×1JsonInput → DateTimeImmutable ×1, DateTimeZone ×1, InvalidArgum…PriceFileParser → InvalidArgumentException ×1, JSON_THROW_ON_ER…PriceRowSpec → DateTimeImmutable ×1PricesImportService → EntityManagerInterface ×1, count ×1, null…UsageEventSpec → DateTimeImmutable ×1UsageFileParser → JSON_THROW_ON_ERROR ×1, JsonException ×1, is_…UsageImportService → EntityManagerInterface ×1, null ×1, sprint…ImportPricesCommand → AsCommand ×1, Command ×1, InputArgument ×…ImportUsageCommand → AsCommand ×1, Command ×1, InputArgument ×1…RateCommand → AsCommand ×1, Command ×1, InputInterface ×1, …ReportCommand → DateTimeImmutable ×1, AsCommand ×1, Command ×1,…RepriceCommand → AsCommand ×1, Command ×1, InputArgument ×1, …UserQuotedQuoteStrategy → AbstractPlatform ×1, ClassMetadata ×1…OffPeakPolicy → false ×1PriceCorrection → DateTimeImmutable ×1, Column ×1, Entity ×1, …RateCalculator → null ×1Model → Column ×1, Entity ×1, GeneratedValue ×1, …PriceRow → DateTimeImmutable ×1, Column ×1, Entity ×1, …PriceUnit → InvalidArgumentException ×1, sprintf ×1Provider → Column ×1, Entity ×1, GeneratedValue ×1, …Decimal → RuntimeException ×1, bcadd ×1, bccomp ×1, …TimeOfDay → DateTimeImmutable ×1, DateTimeZone ×1, InvalidArgum…UsageEvent → DateTimeImmutable ×1, Column ×1, Entity ×1, …BillingException → RuntimeException ×1Kernel → LoaderInterface ×1, Kernel ×1, false ×1внеш.ApplicationCommandDomainException

Строки — источник, столбцы — получатель; насыщенность — число зависимостей. Классы отсортированы по неймспейсам: слои подписаны на диагонали, жирные линии их разделяют; жёлтый столбец «внеш.» — исходящие связи к типам вне решения (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)
ImportPricesCommand301.00
ImportUsageCommand301.00
RateCommand101.00
ReportCommand301.00
RepriceCommand501.00
RepriceService1110.92
RatingService710.88
PricesImportService510.83
PriceFileParser820.80
UsageFileParser410.80
UsageImportService410.80
RateCalculator520.71
ReportService210.67
PriceCorrection210.67
UsageEvent530.62
PriceRowSpec110.50
UsageEventSpec110.50
PriceRow460.40
ProviderSpec120.33
InvalidRepriceFileException120.33
JsonInput130.25
Provider260.25
InvalidUsageFileException130.25
InvalidPricesFileException140.20
Model150.17
RatingResult010.00
UnratedEvent010.00
ModelSpec010.00
OffPeakSpec020.00
PriceFileSpec030.00
OffPeakPolicy020.00
TokenCounts040.00
PriceUnit040.00
Decimal060.00
TimeOfDay030.00
UsageStatus040.00
BillingException040.00

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

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

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

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

КлассLOCметодовWMCcc maxLCOM4
PriceFileParser198845⚠ 11 1
Decimal1401231 8 0
PriceRow1391323 6⚠ 5
JsonInput109520 4 0
RepriceService196719 5 1
UsageEvent1801919 1⚠ 9
PricesImportService154616 5 1
Provider107913 5⚠ 3
ReportCommand76412 7⚠ 3
ReportService108210 9 1

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

МетодCCLOC
PriceFileParser::offPeakMultiplier1119
ReportService::buildCsv998
RepriceService::reprice573
UsageImportService::import451

Полные данные — 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 «процесс и коммуникация»
25 счистое время ревью
57.0Kтокенов (вход 56.2K, $0.0087)

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

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

Обоснование

ревьюер яруса 3
Сильное решение: чекер 11/11 обязательных + 2/2 бонуса. Домен отделён (RateCalculator — чистая функция, Decimal на bcmath, масштаб 32, ровно одно HALF_UP-округление в eventCost); дедупликация на уровне схемы (uniq_usage_events_provider_request), unrated повторяется по статусу pending; оф-пик-окно через полночь с корректными границами покрыто тестами (inclusive start / exclusive end, '00:30'→false). Дефекты: в RatingService applicablePriceRow выполняет запрос на каждое pending-событие (N+1), ReportService тянет все события в память вместо GROUP BY — для объёмов задания допустимо, но это «дорогие» решения; RateCalculator::perMillion при null-ставке тихо возвращает '0' вместо ошибки (хотя парсер ставки требует); в дифф попал мусорный .phpunit.result.cache с записями defects (5 тестов с кодом 7), при том что PR заявляет 76 зелёных — вероятно, устаревший кэш, но коммитить его не следовало; метрики (phpstan 49, 12 классов с lcom4>1, 3 метода >50 loc) указывают на раздутые сервисы. Процесс образцовый: PR_DESCRIPTION системно фиксирует решения и отступления (quote_strategy для «user», exclusive --to, семантика журнала), тесты покрывают граничные значения, дубли, дельту 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 счистое время судьи
56.6Kтокенов (вход 55.9K, $0.0086)
заметка судьи
Сильное решение: чистый домен (RateCalculator, Decimal на bcmath масштаба 32, OffPeakPolicy, TimeOfDay) отделён от приёма (парсеры с жёсткой валидацией) и отчёта; деньги — только строки/bcmath и NUMERIC, float исключён; время — timestamptz/UTC; идемпотентность через уникальный индекс; реализован бонус reprice с журналом; 76 тестов. Смущают мелочи: отчёт тянет все события в память и агрегирует в PHP вместо SQL-агрегации; per-event запрос строки цены (N+1); column 'user' через кастомную QuoteStrategy вместо простого переименования колонки — лишняя сложность; committed .phpunit.result.cache; возможный риск рассинхрона DBAL-типа datetime_immutable с timestamptz в миграции.

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

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