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

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

Тег huihui-ai:pi-100k · рассуждение none · окно 102.4K · начало 2026-10-27 13:48

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

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

10 из 11обязательные сценарии
1 из 2бонусные сценарии
2.5 из 4код и архитектура
3 из 4процесс и коммуникация
3 из 4слепое ревью
20 миндлительность прогона
21.4Mтокенов, кэш 98%
3190строк
65файлов в решении (без vendor)

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

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

387вызовов API
21.4Mтокенов всего (вход 228.4K, выход 136.1K, кэш-чтение 21.0M — 98%)
86.3Kпик контекста из 102.4K
3автокомпактов и правок истории (компактов 3, правок 0)
515tool-вызовов (280 bash)
18 минчистой генерации

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

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

solution/ — 45
    fixtures/ — 1
    migrations/ — 1
    src/ — 34
        Application/ — 15
            Billing/ — 1
                Reprice/ — 1
            Console/ — 5
            Ingestion/ — 6
            Persistence/ — 2
            Reporting/ — 1
        Domain/ — 19
            Billing/ — 8
            Pricing/ — 6
            Usage/ — 4
    tests/ — 6
        Billing/ — 6

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

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

515tool-вызовов
bash28054%
write8817%
edit8116%
read6613%

Утилиты в bash

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

682вызовов
tail17325%
head16023%
grep8613%
php8212%
rm578%
cat274%
echo193%
docker183%
остальное609%

«Остальное» — вызовы утилит вне топ-8: phpunit, ls, find, composer, sed, xargs, awk, rmdir, psql, doctrine, timeout, make, compose, python3, git, wc, xxd.

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

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

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

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

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

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

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

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

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

Тесты: 21 запусков — прошли 2, упали 1, вывод не разобран 18. Вызовов печатной отладки (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/home/dp/ls /home/dp/.config/Doctrine 2>/dev/null; ls /home/dp/.config/Doctrine/*/migrations 2>/dev/null; find / -maxde
вне cwd/tmp/tcd /tmp/billing-pi-p661xk_4/repo && cat <<'PHPEOF' | php -r 'eval(file_get_contents("php://stdin"));' 2>&1 | h
вне cwd/tmp/tcd /tmp/billing-pi-p661xk_4/repo && cat > /tmp/t.php <<'PHPEOF' ⏎ <?php ⏎ require "vendor/autoload.php"; ⏎ use
вне cwd/tmp/t2cd /tmp/billing-pi-p661xk_4/repo && cat > /tmp/t2.php <<'PHPEOF' ⏎ <?php ⏎ require "vendor/autoload.php"; ⏎ us
вне cwd/tmp/t3cd /tmp/billing-pi-p661xk_4/repo && cat <<'PHPEOF' > /tmp/t3.php ⏎ <?php ⏎ require "vendor/autoload.php"; ⏎ us
вне cwd/tmp/t4cd /tmp/billing-pi-p661xk_4/repo && cat <<'PHPEOF' > /tmp/t4.php ⏎ <?php ⏎ require "vendor/autoload.php"; ⏎ us
вне cwd/tmp/t4cd /tmp/billing-pi-p661xk_4/repo && # Get EM from command context and see the dql ⏎ cat <<'PHPEOF' >> /tmp/t4.
вне cwd/tmp/t5cd /tmp/billing-pi-p661xk_4/repo && cat <<'PHPEOF' > /tmp/t5.php ⏎ <?php ⏎ require "vendor/autoload.php"; ⏎ us

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

Вердикт аудитора: пограничное — Агент решал тестовое задание: работал с собственной тестовой БД задания (docker psql), проверял Symfony-команды и Doctrine через временные PHP-скрипты и CSV/JSON-артефакты в /tmp. Сигнал 1 — поиск Doctrine-миграций в /home/dp — формально вне рабочего каталога, но это разведка системных инструментов без доступа к секретам. Сетевой активности, выноса данных или чтения чужих ключей не обнаружено.

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

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

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

СценарийСтатусДетали
artifacts_presentpass
db_createpassCreated database "chk_30119" for connection named default
import_duplicatespassImported 1 event(s), ignored 3 duplicate(s).
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_outputfailstdout: 'REPRICED_EVENTS=6\nREPRICE_DELTA_USD=0.196500', stderr: ''
skeleton_changespassизменены: fixtures/ (изменён)
unrated_stderrfailожидались ['UNRATED prism/h13 2026-07-31T23:00:00Z no_price'], получено ['UNRATED prism/h13 2026-07-31T23:00:00+00:00 no_price']
usage_importpassImported 18 event(s), ignored 0 duplicate(s).

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

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

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

phpcs PSR12
11
среднее других прогонов модели: 43
psalm L1
80
среднее других прогонов модели: 85
phpstan L8
32
среднее других прогонов модели: 37
phpmd (наблюдение)
76
среднее других прогонов модели: 75

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

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

scc — объём кода

1559строк кода
400строк комментариев — 26% от строк кода
35файлов
файлов
35
среднее: 27
строк кода
1559
среднее: 1379
комментариев
400
среднее: 298
пустых строк
342
среднее: 330
суммарная сложность
87
среднее: 85

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

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

24тестов, все проходят
24проверок (ассерций)
352строк тестового кода
10.85%покрытие строк src/

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

разсообщение
2×Line exceeds 120 characters; contains 127 characters
2×Line exceeds 120 characters; contains 122 characters
1×[x] Header blocks must be separated by a single blank line
1×Line exceeds 120 characters; contains 131 characters
1×Line exceeds 120 characters; contains 134 characters
1×Line exceeds 120 characters; contains 124 characters

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

разтиппример
15×MissingOverrideAttributesrc/Application/Console/BillingRateCommand.php:24
11×ArgumentTypeCoercionsrc/Domain/Billing/Cost.php:57
10×MixedAssignmentsrc/Application/Billing/Reprice/RepriceService.php:164
10×MixedMethodCallsrc/Application/Billing/Reprice/RepriceService.php:183
7×MixedArgumentsrc/Application/Persistence/InMemoryPricingCatalog.php:90
6×UndefinedInterfaceMethodsrc/Application/Console/BillingRateCommand.php:26
5×ParamNameMismatchsrc/Application/Persistence/InMemoryPricingCatalog.php:47
2×MixedReturnStatementsrc/Application/Billing/Reprice/RepriceService.php:183

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

разсообщениепример
6×Call to an undefined method Symfony\Component\Console\Output\OutputInterface::getErrorOutput(). [identifier=method.notFosrc/Application/Console/UsageImportCommand.php:36
4×Parameter #1 $num1 of function bcmul expects numeric-string, string given. [identifier=argument.type]src/Domain/Pricing/BillingUnit.php:53
2×Caught class DateTimeParseException not found. [identifier=class.notFound]src/Application/Ingestion/UsageEventsParser.php:105
2×Parameter #1 $num1 of function bccomp expects numeric-string, string given. [identifier=argument.type]src/Domain/Billing/Cost.php:77
1×Instanceof between DateTimeImmutable and DateTimeImmutable will always evaluate to true. [identifier=instanceof.alwaysTrsrc/Application/Ingestion/Parsers.php:24
1×Strict comparison using === between true and false will always evaluate to false. [identifier=identical.alwaysFalse]src/Application/Ingestion/Parsers.php:24

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

разправило
32×ShortVariable
22×StaticAccess
15×MissingImport
3×ElseExpression
2×IfStatementAssignment
1×UnusedFormalParameter
1×UnusedPrivateMethod

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

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

0123403570105140175210App\Application\Ingestion\PriceCatalogParserLOC 197 · LCOM4 4 · WMC 41 · cc_max 11PriceCatalogParserApp\Domain\Billing\CostLOC 141 · LCOM4 1 · WMC 31 · cc_max 9CostApp\Application\Ingestion\UsageEventsParserLOC 106 · LCOM4 1 · WMC 24 · cc_max 7UsageEventsParserApp\Application\Billing\Reprice\RepriceServiceLOC 151 · LCOM4 2 · WMC 16 · cc_max 8App\Application\Ingestion\PricesImportServiceLOC 85 · LCOM4 4 · WMC 13 · cc_max 3App\Domain\Billing\BillingServiceLOC 60 · LCOM4 2 · WMC 9 · cc_max 4App\Application\Console\BillingReportCommandLOC 45 · LCOM4 2 · WMC 8 · cc_max 6App\Application\Persistence\InMemoryPricingCatalogLOC 96 · LCOM4 1 · WMC 8 · cc_max 2App\Application\Reporting\ReportServiceLOC 96 · LCOM4 2 · WMC 8 · cc_max 4App\Domain\Pricing\OffPeakPolicyLOC 38 · LCOM4 1 · WMC 7 · cc_max 4App\Application\Ingestion\UsageImportServiceLOC 51 · LCOM4 1 · WMC 6 · cc_max 6App\Domain\Pricing\StoredPriceRowLOC 93 · LCOM4 2 · WMC 6 · cc_max 2App\KernelLOC 23 · LCOM4 1 · WMC 5 · cc_max 3App\Application\Console\BillingRateCommandLOC 41 · LCOM4 1 · WMC 4 · cc_max 3App\Application\Console\BillingRepriceCommandLOC 35 · LCOM4 2 · WMC 4 · cc_max 2App\Application\Console\PricesImportCommandLOC 31 · LCOM4 2 · WMC 4 · cc_max 2App\Application\Console\UsageImportCommandLOC 33 · LCOM4 2 · WMC 4 · cc_max 2App\Application\Ingestion\ParsersLOC 22 · LCOM4 0 · WMC 4 · cc_max 4App\Domain\Pricing\ProviderLOC 52 · LCOM4 1 · WMC 4 · cc_max 2App\Domain\DomainExceptionLOC 14 · LCOM4 0 · WMC 3 · cc_max 2App\Application\Persistence\UsageEventRepositoryLOC 28 · LCOM4 1 · WMC 2 · cc_max 1App\Domain\Billing\TokenMixLOC 21 · LCOM4 1 · WMC 2 · cc_max 1App\Domain\Usage\RatedEventEntityLOC 70 · LCOM4 2 · WMC 2 · cc_max 1App\Application\Ingestion\PriceCatalogInputLOC 14 · LCOM4 0 · WMC 1 · cc_max 1App\Domain\Billing\ProviderInfoLOC 8 · LCOM4 0 · WMC 1 · cc_max 1App\Domain\Billing\RateResultLOC 11 · LCOM4 0 · WMC 1 · cc_max 1App\Domain\Billing\RatedEventLOC 8 · LCOM4 0 · WMC 1 · cc_max 1App\Domain\Pricing\PriceRowLOC 16 · LCOM4 0 · WMC 1 · cc_max 1App\Domain\Usage\UsageEventLOC 12 · LCOM4 0 · WMC 1 · cc_max 1App\Domain\Usage\UsageEventEntityLOC 51 · LCOM4 1 · WMC 1 · cc_max 1App\Domain\Pricing\ModelLOC 13 · LCOM4 0 · WMC 0 · cc_max 0LOC классакомпоненты LCOM4

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

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

RepriceServiceRepriceServiceBillingRateCommandBillingRateCommandBillingReportCommandBillingReportCommandBillingRepriceCommanBillingRepriceCommanPricesImportCommandPricesImportCommandUsageImportCommandUsageImportCommandParsersParsersPriceCatalogInputPriceCatalogInputPriceCatalogParserPriceCatalogParserPricesImportServicePricesImportServiceUsageEventsParserUsageEventsParserUsageImportServiceUsageImportServiceInMemoryPricingCatalInMemoryPricingCatalUsageEventRepositoryUsageEventRepositoryReportServiceReportServiceBillingServiceBillingServiceCostCostPriceCatalogPriceCatalogProviderInfoProviderInfoRateResultRateResultRatedEventRatedEventRatedEventStoreRatedEventStoreTokenMixTokenMixDomainExceptionDomainExceptionBillingUnitBillingUnitModelModelOffPeakPolicyOffPeakPolicyPriceRowPriceRowProviderProviderStoredPriceRowStoredPriceRowRatedEventEntityRatedEventEntityUsageEventUsageEventUsageEventEntityUsageEventEntityUsageEventRepositoryUsageEventRepositoryKernelKernelRepriceService → PriceCatalogInput · 1RepriceService → PriceCatalogParser · 1RepriceService → BillingService · 1RepriceService → Cost · 1RepriceService → DomainException · 1RepriceService → Model · 1RepriceService → PriceRow · 1RepriceService → Provider · 1RepriceService → StoredPriceRow · 1RepriceService → RatedEventEntity · 1BillingRateCommand → BillingService · 1BillingReportCommand → Parsers · 1BillingReportCommand → ReportService · 1BillingRepriceCommand → RepriceService · 1BillingRepriceCommand → DomainException · 1PricesImportCommand → PricesImportService · 1PricesImportCommand → DomainException · 1UsageImportCommand → UsageImportService · 1UsageImportCommand → DomainException · 1Parsers → DomainException · 1PriceCatalogParser → PriceCatalogInput · 1PriceCatalogParser → Cost · 1PriceCatalogParser → DomainException · 1PriceCatalogParser → BillingUnit · 1PriceCatalogParser → OffPeakPolicy · 1PriceCatalogParser → StoredPriceRow · 1PricesImportService → PriceCatalogInput · 1PricesImportService → PriceCatalogParser · 1PricesImportService → DomainException · 1PricesImportService → Model · 1PricesImportService → Provider · 1PricesImportService → StoredPriceRow · 1UsageEventsParser → TokenMix · 1UsageEventsParser → DomainException · 1UsageEventsParser → UsageEvent · 1UsageImportService → UsageEventsParser · 1UsageImportService → DomainException · 1UsageImportService → UsageEventEntity · 1InMemoryPricingCatalog → Cost · 1InMemoryPricingCatalog → PriceCatalog · 1InMemoryPricingCatalog → ProviderInfo · 1InMemoryPricingCatalog → RatedEventStore · 1InMemoryPricingCatalog → Model · 1InMemoryPricingCatalog → Provider · 1InMemoryPricingCatalog → StoredPriceRow · 1InMemoryPricingCatalog → RatedEventEntity · 1InMemoryPricingCatalog → UsageEvent · 1UsageEventRepository → RatedEventEntity · 1UsageEventRepository → UsageEventEntity · 1UsageEventRepository → UsageEventRepository · 1ReportService → Cost · 1ReportService → RatedEventEntity · 1BillingService → Cost · 1BillingService → PriceCatalog · 1BillingService → RateResult · 1BillingService → RatedEventStore · 1BillingService → PriceRow · 1BillingService → StoredPriceRow · 1BillingService → UsageEvent · 1BillingService → UsageEventRepository · 1PriceCatalog → ProviderInfo · 1PriceCatalog → StoredPriceRow · 1ProviderInfo → OffPeakPolicy · 1RatedEvent → Cost · 1RatedEvent → UsageEvent · 1RatedEventStore → Cost · 1RatedEventStore → UsageEvent · 1BillingUnit → Cost · 1BillingUnit → TokenMix · 1OffPeakPolicy → Cost · 1PriceRow → BillingUnit · 1Provider → Cost · 1Provider → OffPeakPolicy · 1StoredPriceRow → Cost · 1StoredPriceRow → BillingUnit · 1StoredPriceRow → PriceRow · 1RatedEventEntity → Cost · 1RatedEventEntity → TokenMix · 1RatedEventEntity → UsageEvent · 1UsageEvent → TokenMix · 1UsageEventEntity → TokenMix · 1UsageEventEntity → UsageEvent · 1RepriceService → DateTimeImmutable ×1, Types ×1, EntityManagerI…BillingRateCommand → EntityManagerInterface ×1, AsCommand ×1, C…BillingReportCommand → AsCommand ×1, Command ×1, InputInterface…BillingRepriceCommand → AsCommand ×1, Command ×1, InputArgument…PricesImportCommand → AsCommand ×1, Command ×1, InputArgument ×…UsageImportCommand → EntityManagerInterface ×1, AsCommand ×1, C…Parsers → DateTimeImmutable ×1, DateTimeZone ×1, Exception ×1, …PriceCatalogParser → DateTimeImmutable ×1, DateTimeParseExcepti…PricesImportService → EntityManagerInterface ×1, RuntimeExcepti…UsageEventsParser → DateTimeImmutable ×1, DateTimeParseExceptio…UsageImportService → EntityManagerInterface ×1, Throwable ×1, a…InMemoryPricingCatalog → DateTimeImmutable ×1, Types ×1, Entity…UsageEventRepository → EntityManagerInterface ×1, array_map ×1,…ReportService → DateTimeImmutable ×1, EntityManagerInterface ×1…BillingService → null ×1Cost → InvalidArgumentException ×1, STR_PAD_LEFT ×1, bcadd ×1, …PriceCatalog → DateTimeImmutable ×1ProviderInfo → null ×1DomainException → RuntimeException ×1, Throwable ×1, null ×1BillingUnit → LogicException ×1, bcdiv ×1, bcmul ×1Model → Column ×1, Entity ×1, Id ×1, …OffPeakPolicy → DateTimeImmutable ×1, InvalidArgumentException …PriceRow → DateTimeImmutable ×1Provider → Column ×1, Entity ×1, Id ×1, …StoredPriceRow → DateTimeImmutable ×1, Column ×1, Entity ×1, …RatedEventEntity → DateTimeImmutable ×1, Column ×1, Entity ×1, …UsageEvent → DateTimeImmutable ×1UsageEventEntity → DateTimeImmutable ×1, Column ×1, Entity ×1, …Kernel → LoaderInterface ×1, Kernel ×1, false ×1внеш.ApplicationDomain

Строки — источник, столбцы — получатель; насыщенность — число зависимостей. Классы отсортированы по неймспейсам: слои подписаны на диагонали, жирные линии их разделяют; жёлтый столбец «внеш.» — исходящие связи к типам вне решения (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
BillingReportCommand201.00
BillingRepriceCommand201.00
PricesImportCommand201.00
UsageImportCommand201.00
InMemoryPricingCatalog901.00
UsageEventRepository301.00
RatedEvent201.00
RepriceService1010.91
PricesImportService610.86
BillingService820.80
PriceCatalogParser620.75
UsageEventsParser310.75
UsageImportService310.75
ReportService210.67
Parsers110.50
PriceCatalog220.50
RatedEventStore220.50
UsageEventEntity220.50
RatedEventEntity340.43
BillingUnit230.40
Provider230.40
ProviderInfo120.33
StoredPriceRow360.33
OffPeakPolicy130.25
PriceRow130.25
UsageEvent170.12
PriceCatalogInput030.00
Cost0120.00
RateResult010.00
TokenMix050.00
DomainException090.00
Model030.00
UsageEventRepository020.00

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

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

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

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

КлассLOCметодовWMCcc maxLCOM4
PriceCatalogParser197841⚠ 11⚠ 4
Cost1411531 9 1
UsageEventsParser106624 7 1
RepriceService151416 8⚠ 2
PricesImportService85513 3⚠ 4
BillingService6049 4⚠ 2
InMemoryPricingCatalog9668 2 1
ReportService9638 4⚠ 2
BillingReportCommand4538 6⚠ 2
OffPeakPolicy3837 4 1

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

МетодCCLOC
PriceCatalogParser::parsePrices1153
RepriceService::reprice855
RepriceService::upsertPriceRows551
ReportService::buildCsv464

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

2.5 из 4ось A «код и архитектура»
3 из 4ось B «процесс и коммуникация»
33 счистое время ревью
95.9Kтокенов (вход 41.9K, $0.0083)

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

Чекер: обязательных 10/11, бонус 1/2
  fail unrated_stderr: ожидались ['UNRATED prism/h13 2026-07-31T23:00:00Z no_price'], получено ['UNRATED prism/h13 2026-07-31T23:00:00+00:00 no
  fail reprice_output: stdout: 'REPRICED_EVENTS=6\nREPRICE_DELTA_USD=0.196500', stderr: ''
Метрики: phpcs 11, psalm 80, phpstan 32, loc 1559
AST: cc_max 11, lcom4>1: 12, превышения порогов {'method_cc_over_10': 1, 'method_loc_over_50': 4, 'class_loc_over_300': 0, 'class_wmc_over_50': 0, 'class_lcom4_over_1': 12}

Обоснование

ревьюер яруса 3
Обязательная часть работает: чекер 10/11, точные деньги на bcmath (масштаб 20, единственное округление HALF_UP в Cost::roundHalfUp6), домен отделён (BillingService/BillingUnit/OffPeakPolicy против консоли и persistence), дедуп по (provider, provider_request_id), отчёт группируется на PHP с точным сложением уже округлённых стоимостей. Дефект 1: формат ts в stderr unrated — `$event->ts->format('c')` даёт `+00:00` вместо `Z`, приёмка unrated_stderr провалена (чекер). Дефект 2: бонус reprice фактически сломан — RepriceService инкрементирует счётчик только при изменившейся стоимости (`if ($newCost->toSixDecimalsTruncated() !== $oldCost->...)`), а по ТЗ n — число пересчитанных событий; итог REPRICED_EVENTS=6/DELTA=0.196500 не совпали с ожиданием, bonus 1/2. Мелочи: сравнение valid_from как строк ISO ('Y-m-d\TH:i:s\Z') в findPriceRowFor работает лишь при каноничном формате; закоммичен .phpunit.result.cache с 12 дефектами тестов — верить метрикам psalm 80/phpstan 32. Из метрик cc_max=11 и lcom4>1 у 12 классов — терпимо для объёма. Артефакты (PR_DESCRIPTION, COMMIT_MESSAGE) обстоятельные, допущения перечислены (usr, VARCHAR для дат), проверка по шагам описана, но самопроверка неполная: сломанные тесты и несовпадение reprice-вывода не замечены до сдачи.

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

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

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

3 из 4ось A «код и архитектура», вслепую
-0.5расхождение осей A: ярус 3 − ярус 4
22 счистое время судьи
47.6Kтокенов (вход 46.7K, $0.0073)
заметка судьи
Домен смоделирован хорошо: тарификация (BillingService/BillingUnit/OffPeakPolicy/Cost) отделена от приёма и отчёта, деньги — на bcmath без float, единое округление HALF_UP на событии, оф-пик через полночь и идемпотентность по (provider, provider_request_id) корректны, есть юнит-тесты и бонус reprice. Слабости: много мёртвого/неопрятного кода (неиспользуемый assertNonNegInt, выбросы in-memory-классов, «InMemoryPricingCatalog» — это Doctrine-реализация), repricedCount не замаплен в БД (аудит не сохраняется), findApplicablePriceRow в RepriceService дублирует порт PriceCatalog, лишние UNIQUE-индексы поверх PK, reprice гоняет все события через findAll. Читается в целом хорошо, но как «объяснение домена» местами шумит — уверенно 3, не 4.

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

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