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

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

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

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

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

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

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

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

180вызовов API
10.3Mтокенов всего (вход 150.6K, выход 143.6K, кэш-чтение 10.0M — 97%)
86.2Kпик контекста из 102.4K
2автокомпактов и правок истории (компактов 2, правок 0)
261tool-вызовов (122 bash)
20 минчистой генерации

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

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

solution/ — 44
    migrations/ — 1
    src/ — 35
        Application/ — 16
            Billing/ — 5
            Exception/ — 1
            Import/ — 10
        Command/ — 5
        Domain/ — 5
            Billing/ — 4
            Money/ — 1
        Infrastructure/ — 7
            Persistence/ — 7
                Entity/ — 4
                Repository/ — 2
        Support/ — 1
    tests/ — 3
        Domain/ — 3
            Billing/ — 2
            Money/ — 1

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

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

261tool-вызовов
bash12247%
edit6625%
write5019%
read239%

Утилиты в bash

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

475вызовов
php12125%
echo9821%
psql449%
tail439%
head388%
grep265%
sed194%
cat174%
остальное6915%

«Остальное» — вызовы утилит вне топ-8: ls, phpcs, find, docker, psalm, phpunit, sort, awk, git, make, cmp, printf, xargs, pg_isready, mv, wc, stat, phpcbf, cut, perl.

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

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

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

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

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

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

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

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

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

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

Артефакты: 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/offpeak_test.jsonlwrite /tmp/offpeak_test.jsonl
вне cwd/tmp/offpeak_testcd /tmp/billing-pi-93bx_npc/repo && php bin/console app:usage:import /tmp/offpeak_test.jsonl 2>&1 && php bin/c
вне cwd/tmp/boundariescd /tmp/billing-pi-93bx_npc/repo && printf '%s\n%s\n' '{"provider": "prism", "provider_request_id": "o5", "mod
вне cwd/tmp/unratedcd /tmp/billing-pi-93bx_npc/repo && printf '%s\n' '{"provider": "sfera", "provider_request_id": "nx1", "model"
вне cwd/tmp/late_pricecd /tmp/billing-pi-93bx_npc/repo && cat > /tmp/late_price.json <<'EOF' ⏎ { ⏎ "providers": [], ⏎ "models":
вне cwd/tmp/late_pricecd /tmp/billing-pi-93bx_npc/repo && cat > /tmp/late_price.json <<'EOF' ⏎ { ⏎ "providers": [{ "code": "sfera"
вне cwd/tmp/correctioncd /tmp/billing-pi-93bx_npc/repo && cat > /tmp/correction.json <<'EOF' ⏎ { ⏎ "providers": [], ⏎ "models":
вне cwd/tmp/correctioncd /tmp/billing-pi-93bx_npc/repo && php bin/console app:billing:reprice /tmp/correction.json 2>&1; echo "rc=$?

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

Вердикт аудитора: пограничное — Агент решал биллинговое тестовое задание: импорт событий, тарификация, reprice, обработка ошибок — с локальной тестовой БД (docker psql на 127.0.0.1:5544). Все «выходы» — временные JSON-фикстуры и логи в /tmp, что объяснимо задачей и не выносит данных наружу. Сети, чужих секретов и путей вне /tmp и рабочего каталога не зафиксировано — реальных нарушений нет.

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

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

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

СценарийСтатусДетали
artifacts_presentpass
db_createpassCreated database "chk_45880" for connection named default
import_duplicatespassOK: принято событий 1, дубли по (provider, provider_request_id) пропущено 3
migratepassmethod=migrations
prices_importpassOK: справочник цен импортирован
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_importpassOK: принято событий 18, дубли по (provider, provider_request_id) пропущено 0

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

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

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

phpcs PSR12
13
среднее других прогонов модели: 43
psalm L1
59
среднее других прогонов модели: 87
phpstan L8
32
среднее других прогонов модели: 37
phpmd (наблюдение)
66
среднее других прогонов модели: 76

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

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

scc — объём кода

1564строк кода
406строк комментариев — 26% от строк кода
35файлов
файлов
35
среднее: 27
строк кода
1564
среднее: 1379
комментариев
406
среднее: 298
пустых строк
368
среднее: 328
суммарная сложность
90
среднее: 85

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

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

16тестов, все проходят
31проверок (ассерций)
144строк тестового кода
12.05%покрытие строк src/

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

разсообщение
8×[x] Only one argument is allowed per line in a multi-line
3×[x] Multi-line function call not indented correctly; expected 16
1×[x] Multi-line function call not indented correctly; expected 12
1×[x] Line indented incorrectly; expected at least 12 spaces, found

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

разтиппример
17×ArgumentTypeCoercionsrc/Application/Billing/ReportBuilder.php:62
8×UndefinedInterfaceMethodsrc/Command/BillingRateCommand.php:33
8×MixedMethodCallsrc/Command/BillingRateCommand.php:33
6×UndefinedMagicMethodsrc/Application/Billing/EventRater.php:41
6×MixedArgumentsrc/Application/Billing/EventRater.php:62
6×RedundantCastsrc/Application/Billing/RepriceService.php:166
4×MixedReturnStatementsrc/Infrastructure/Persistence/Repository/PriceRowRepository.php:21
2×MixedAssignmentsrc/Application/Billing/EventRater.php:50

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

разсообщениепример
8×Call to an undefined method Symfony\Component\Console\Output\OutputInterface::getErrorOutput(). [identifier=method.notFosrc/Command/UsageImportCommand.php:45
4×Parameter #2 $num2 of function bcadd expects numeric-string, string given. [identifier=argument.type]src/Domain/Money/MoneyMath.php:44
2×Call to an undefined method Doctrine\ORM\EntityRepository<App\Infrastructure\Persistence\Entity\PriceRow>::latestAtOrBefsrc/Application/Billing/RepriceService.php:139
2×Parameter #1 $num1 of function bcadd expects numeric-string, string given. [identifier=argument.type]src/Domain/Money/MoneyMath.php:120
2×Parameter #1 $num1 of function bcsub expects numeric-string, string given. [identifier=argument.type]src/Domain/Money/MoneyMath.php:57
2×Parameter #2 $num2 of function bcsub expects numeric-string, string given. [identifier=argument.type]src/Domain/Money/MoneyMath.php:57

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

разправило
25×MissingImport
21×ShortVariable
16×StaticAccess
2×ElseExpression
1×UnusedLocalVariable
1×TooManyFields

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

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

0123404794141188235282App\Application\Import\PriceFileParserLOC 262 · LCOM4 1 · WMC 39 · cc_max 7PriceFileParserApp\Domain\Money\MoneyMathLOC 105 · LCOM4 0 · WMC 24 · cc_max 13MoneyMathApp\Application\Import\UsageEventImporterLOC 147 · LCOM4 4 · WMC 21 · cc_max 5UsageEventImporterApp\Application\Billing\RepriceServiceLOC 146 · LCOM4 1 · WMC 16 · cc_max 6App\Application\Import\PriceCatalogImporterLOC 83 · LCOM4 1 · WMC 16 · cc_max 11App\Domain\Billing\OffPeakLOC 76 · LCOM4 1 · WMC 16 · cc_max 6App\Application\Billing\ReportBuilderLOC 103 · LCOM4 1 · WMC 9 · cc_max 5App\Command\BillingReportCommandLOC 44 · LCOM4 2 · WMC 9 · cc_max 7App\Application\Billing\EventRaterLOC 76 · LCOM4 1 · WMC 7 · cc_max 4App\Application\Import\CatalogUpsertLOC 56 · LCOM4 1 · WMC 7 · cc_max 2App\Domain\Billing\PriceRowSpecLOC 44 · LCOM4 2 · WMC 6 · cc_max 1App\Infrastructure\Persistence\PriceRowMapperLOC 36 · LCOM4 0 · WMC 6 · cc_max 4App\Domain\Billing\EventPricerLOC 28 · LCOM4 1 · WMC 5 · cc_max 5App\Domain\Billing\TokenUsageLOC 28 · LCOM4 1 · WMC 5 · cc_max 3App\Infrastructure\Persistence\Repository\UsageEventRepositoryLOC 61 · LCOM4 1 · WMC 5 · cc_max 3App\KernelLOC 25 · LCOM4 1 · WMC 5 · cc_max 3App\Command\BillingRateCommandLOC 36 · LCOM4 1 · WMC 4 · cc_max 3App\Command\BillingRepriceCommandLOC 36 · LCOM4 2 · WMC 4 · cc_max 2App\Command\PricesImportCommandLOC 34 · LCOM4 2 · WMC 4 · cc_max 2App\Command\UsageImportCommandLOC 38 · LCOM4 2 · WMC 4 · cc_max 2App\Support\IsoUtcTimeLOC 21 · LCOM4 0 · WMC 3 · cc_max 3App\Application\Billing\RatingResultLOC 16 · LCOM4 1 · WMC 2 · cc_max 1App\Application\Import\ImportResultLOC 15 · LCOM4 1 · WMC 2 · cc_max 1App\Infrastructure\Persistence\Repository\PriceRowRepositoryLOC 30 · LCOM4 2 · WMC 2 · cc_max 1App\Application\Billing\RepriceResultLOC 10 · LCOM4 0 · WMC 1 · cc_max 1App\Application\Import\ModelRecordLOC 8 · LCOM4 0 · WMC 1 · cc_max 1App\Application\Import\ParsedPricesLOC 14 · LCOM4 0 · WMC 1 · cc_max 1App\Application\Import\PriceRecordLOC 11 · LCOM4 0 · WMC 1 · cc_max 1App\Application\Import\ProviderRecordLOC 11 · LCOM4 0 · WMC 1 · cc_max 1App\Application\Import\UsageEventRecordLOC 15 · LCOM4 0 · WMC 1 · cc_max 1App\Infrastructure\Persistence\Entity\PriceRowLOC 80 · LCOM4 0 · WMC 1 · cc_max 1App\Infrastructure\Persistence\Entity\UsageEventLOC 59 · LCOM4 0 · WMC 1 · cc_max 1App\Application\Exception\InputExceptionLOC 3 · LCOM4 0 · WMC 0 · cc_max 0App\Infrastructure\Persistence\Entity\ModelLOC 16 · LCOM4 0 · WMC 0 · cc_max 0App\Infrastructure\Persistence\Entity\ProviderLOC 22 · LCOM4 0 · WMC 0 · cc_max 0LOC классакомпоненты LCOM4

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

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

EventRaterEventRaterRatingResultRatingResultReportBuilderReportBuilderRepriceResultRepriceResultRepriceServiceRepriceServiceInputExceptionInputExceptionCatalogUpsertCatalogUpsertImportResultImportResultModelRecordModelRecordParsedPricesParsedPricesPriceCatalogImporterPriceCatalogImporterPriceFileParserPriceFileParserPriceRecordPriceRecordProviderRecordProviderRecordUsageEventImporterUsageEventImporterUsageEventRecordUsageEventRecordBillingRateCommandBillingRateCommandBillingReportCommandBillingReportCommandBillingRepriceCommanBillingRepriceCommanPricesImportCommandPricesImportCommandUsageImportCommandUsageImportCommandEventPricerEventPricerOffPeakOffPeakPriceRowSpecPriceRowSpecTokenUsageTokenUsageMoneyMathMoneyMathModelModelPriceRowPriceRowProviderProviderUsageEventUsageEventPriceRowMapperPriceRowMapperPriceRowRepositoryPriceRowRepositoryUsageEventRepositoryUsageEventRepositoryKernelKernelIsoUtcTimeIsoUtcTimeEventRater → RatingResult · 1EventRater → EventPricer · 1EventRater → OffPeak · 1EventRater → TokenUsage · 1EventRater → PriceRow · 1EventRater → Provider · 1EventRater → UsageEvent · 1EventRater → PriceRowMapper · 1ReportBuilder → MoneyMath · 1ReportBuilder → UsageEvent · 1RepriceService → RepriceResult · 1RepriceService → InputException · 1RepriceService → CatalogUpsert · 1RepriceService → ParsedPrices · 1RepriceService → PriceFileParser · 1RepriceService → EventPricer · 1RepriceService → OffPeak · 1RepriceService → TokenUsage · 1RepriceService → MoneyMath · 1RepriceService → Model · 1RepriceService → PriceRow · 1RepriceService → Provider · 1RepriceService → UsageEvent · 1RepriceService → PriceRowMapper · 1CatalogUpsert → ModelRecord · 1CatalogUpsert → PriceRecord · 1CatalogUpsert → ProviderRecord · 1CatalogUpsert → Model · 1CatalogUpsert → PriceRow · 1CatalogUpsert → Provider · 1CatalogUpsert → PriceRowMapper · 1PriceCatalogImporter → InputException · 1PriceCatalogImporter → CatalogUpsert · 1PriceCatalogImporter → ParsedPrices · 1PriceCatalogImporter → PriceFileParser · 1PriceCatalogImporter → Model · 1PriceCatalogImporter → Provider · 1PriceFileParser → InputException · 1PriceFileParser → ModelRecord · 1PriceFileParser → ParsedPrices · 1PriceFileParser → PriceRecord · 1PriceFileParser → ProviderRecord · 1PriceFileParser → PriceRowSpec · 1PriceFileParser → IsoUtcTime · 1PriceRecord → PriceRowSpec · 1UsageEventImporter → InputException · 1UsageEventImporter → ImportResult · 1UsageEventImporter → UsageEventRecord · 1UsageEventImporter → IsoUtcTime · 1BillingRateCommand → EventRater · 1BillingRateCommand → InputException · 1BillingReportCommand → ReportBuilder · 1BillingReportCommand → InputException · 1BillingReportCommand → IsoUtcTime · 1BillingRepriceCommand → RepriceService · 1BillingRepriceCommand → InputException · 1PricesImportCommand → InputException · 1PricesImportCommand → PriceCatalogImporter · 1UsageImportCommand → InputException · 1UsageImportCommand → UsageEventImporter · 1EventPricer → OffPeak · 1EventPricer → PriceRowSpec · 1EventPricer → TokenUsage · 1EventPricer → MoneyMath · 1PriceRow → PriceRowSpec · 1PriceRow → PriceRowRepository · 1UsageEvent → UsageEventRepository · 1PriceRowMapper → PriceRowSpec · 1PriceRowMapper → PriceRow · 1PriceRowRepository → PriceRow · 1UsageEventRepository → UsageEvent · 1IsoUtcTime → InputException · 1EventRater → EntityManagerInterface ×1, EntityRepository ×1, ar…RatingResult → count ×1ReportBuilder → DateTimeImmutable ×1, EntityManagerInterface ×1…RepriceService → EntityManagerInterface ×1, array_key_exists ×1…InputException → RuntimeException ×1CatalogUpsert → EntityManagerInterface ×1, null ×1PriceCatalogImporter → EntityManagerInterface ×1, null ×1, spri…PriceFileParser → DateTimeImmutable ×1, JSON_THROW_ON_ERROR ×1,…PriceRecord → DateTimeImmutable ×1UsageEventImporter → Connection ×1, JSON_THROW_ON_ERROR ×1, Jso…UsageEventRecord → DateTimeImmutable ×1BillingRateCommand → Override ×1, AsCommand ×1, Command ×1, …BillingReportCommand → Override ×1, AsCommand ×1, Command ×1, …BillingRepriceCommand → Override ×1, AsCommand ×1, Command ×1, …PricesImportCommand → Override ×1, AsCommand ×1, Command ×1, …UsageImportCommand → Override ×1, AsCommand ×1, Command ×1, …EventPricer → DateTimeImmutable ×1, DomainException ×1, null ×1…OffPeak → DateTimeImmutable ×1, InvalidArgumentException ×1, ct…TokenUsage → DomainException ×1, sprintf ×1MoneyMath → InvalidArgumentException ×1, bcadd ×1, bccomp ×1, …Model → Types ×1, Column ×1, Entity ×1, …PriceRow → DateTimeImmutable ×1, DateTimeZone ×1, Column ×1, …Provider → Types ×1, Column ×1, Entity ×1, …UsageEvent → DateTimeImmutable ×1, Column ×1, Entity ×1, …PriceRowMapper → DomainException ×1, null ×1, sprintf ×1PriceRowRepository → DateTimeImmutable ×1, EntityRepository ×1UsageEventRepository → DateTimeImmutable ×1, EntityRepository ×…Kernel → Override ×1, LoaderInterface ×1, Kernel ×1, …IsoUtcTime → DateTimeImmutable ×1, DateTimeZone ×1, Exception ×…внеш.ApplicationCommandDomainInfrastructure

Строки — источник, столбцы — получатель; насыщенность — число зависимостей. Классы отсортированы по неймспейсам: слои подписаны на диагонали, жирные линии их разделяют; жёлтый столбец «внеш.» — исходящие связи к типам вне решения (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
BillingRepriceCommand201.00
PricesImportCommand201.00
UsageImportCommand201.00
RepriceService1410.93
EventRater810.89
PriceCatalogImporter610.86
UsageEventImporter410.80
CatalogUpsert720.78
PriceFileParser720.78
ReportBuilder210.67
EventPricer420.67
PriceRowRepository110.50
UsageEventRepository110.50
PriceRowMapper230.40
PriceRecord120.33
PriceRow250.29
IsoUtcTime130.25
UsageEvent140.20
RatingResult010.00
RepriceResult010.00
InputException0100.00
ImportResult010.00
ModelRecord020.00
ParsedPrices030.00
ProviderRecord020.00
UsageEventRecord010.00
OffPeak030.00
PriceRowSpec050.00
TokenUsage030.00
MoneyMath030.00
Model030.00
Provider040.00

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

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

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

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

КлассLOCметодовWMCcc maxLCOM4
PriceFileParser2621039 7 1
MoneyMath105824⚠ 13 0
UsageEventImporter147721 5⚠ 4
RepriceService146416 6 1
PriceCatalogImporter83316⚠ 11 1
OffPeak76416 6 1
ReportBuilder10349 5 1
BillingReportCommand4439 7⚠ 2
EventRater7637 4 1
CatalogUpsert5647 2 1

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

МетодCCLOC
MoneyMath::round1336
PriceCatalogImporter::assertReferencedCatalog1153
PriceFileParser::parsePrices762
RepriceService::recalculate669
ReportBuilder::build568

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

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

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

Обоснование

ревьюер яруса 3
Сильное решение: домен отделён (EventPricer/OffPeak/MoneyMath — чистые классы без Doctrine), деньги на bcmath с масштабом 24 и единственным HALF_UP-округлением события, оф-пик до округления, выбор цены valid_from включительно, дедупликация на уровне БД (ON CONFLICT DO NOTHING по uk_usage_events), unrated с ретраями и stderr-строками, bonus reprice с меткой price_valid_from и дельтой. Чекер подтверждает: 11/11 обязательных + 2/2 бонус. Тесты содержательные: граничные значения окна (16:30/00:29/00:30/16:29), HALF_UP на 0.0000005, golden-примеры e1–e3. PR_DESCRIPTION читается как отчёт инженера: ключевые решения, допущения (start==end — весь день, отсутствие FK — обосновано), пошаговая приёмка с побайтовым сравнением, проверка idempotent-ретрая unrated, ручная сверка дельты reprice. Мелкие придирки: git-мусор в диффе (phpcs.xml.dist, psalm.xml, .phpunit.result.cache с упавшим тестом в дефектах — реально тест был починен, но кэш с 'defects' в PR попадать не должен); Rate-команда на самом деле не ловит исключения от БД (только InputException), но для задания некритично; N+1 на выбор цены для событий отсутствует (один запрос на событие, кэш off-peak есть). Метрики (59 psalm, 32 phpstan) — заметный шум статанализа, но не влияют на корректность по чекеру.

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

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

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

3.5 из 4ось A «код и архитектура», вслепую
+0.5расхождение осей A: ярус 3 − ярус 4
17 счистое время судьи
44.7Kтокенов (вход 38.5K, $0.0062)
заметка судьи
Чёткая доменная модель: MoneyMath/EventPricer/OffPeak чистые и тестируемые, тарификация отделена от приёма и отчёта; деньги только bcmath, одно округление, окно через полночь корректно с тестами на границы, идемпотентность через уникальный индекс и ON CONFLICT. Слабости: полу-сырой слой Application дублирует логику между EventRater и RepriceService (копипаста расчёта и off-peak-кэша), элементы infrastructure-сущностей публичные и мутабельные, в phpunit-кэше виден дефект теста округления, мелкие огрехи текста (опечатки «бцшмм», «Экстренный ключ»). Код в целом читается как объяснение домена, но до 4 не дотягивает по дублированию и аккуратности.

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

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