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

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

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

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

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

11 из 11обязательные сценарии
2 из 2бонусные сценарии
3.5 из 4код и архитектура
4 из 4процесс и коммуникация
3.5 из 4слепое ревью
22 миндлительность прогона
13.8Mтокенов, кэш 98%
2827строк
61файлов в решении (без 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, автокомпакты, чем самопроверялся.

255вызовов API
13.8Mтокенов всего (вход 164.3K, выход 137.3K, кэш-чтение 13.5M — 98%)
86.4Kпик контекста из 102.4K
2автокомпактов и правок истории (компактов 2, правок 0)
255tool-вызовов (107 bash)
20 минчистой генерации

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

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

solution/ — 41
    fixtures/ — 1
    migrations/ — 1
    src/ — 33
        Command/ — 6
        Domain/ — 6
            Billing/ — 5
            Money/ — 1
        Dto/ — 4
        Entity/ — 4
        Exception/ — 1
        Repository/ — 4
        Service/ — 8
    tests/ — 3
        Domain/ — 3

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

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

255tool-вызовов
bash10742%
edit5722%
read4618%
write4518%

Утилиты в bash

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

502вызовов
echo16433%
console8016%
cat7014%
grep408%
tail357%
head255%
ls204%
php184%
остальное5010%

«Остальное» — вызовы утилит вне топ-8: timeout, find, sed, phpunit, pwd, diff, sort, docker, wc, psql, rm, which, composer, od, git.

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

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

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

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

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

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

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

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

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

Тесты: 5 запусков — прошли 2, упали 2, вывод не разобран 1. Вызовов печатной отладки (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/billing-pi-citw7xxl/cd /tmp/billing-pi-citw7xxl/repo && ls -1 /tmp/billing-pi-citw7xxl/.pi-billing-dev/ 2>/dev/null; PGPASSWORD=po
вне cwd/tmp/billing-pitw7xxl/repocd /tmp/billing-pitw7xxl/repo 2>/dev/null; cd /tmp/billing-pi-citw7xxl/repo && cat > /tmp/repro.php <<'PHP' ⏎
вне cwd/tmp/reprocd /tmp/billing-pi-citw7xxl/repo && cat > /tmp/repro.php <<'PHP' ⏎ <?php ⏎ require '/tmp/billing-pi-citw7xxl/r
вне cwd (запись)/tmp/billing-pitw7xxl/repo/src/Entity/PriceRow.phpedit /tmp/billing-pitw7xxl/repo/src/Entity/PriceRow.php
вне cwd/tmp/billing-pitw7xxl/repocd /tmp/billing-pitw7xxl/repo && grep -n "models" src/Entity/Provider.php | head -10; echo "---user ---"; grep
вне cwd/tmp/billing-pitw7xxl/repocd /tmp/billing-pitw7xxl/repo && grep -n "user\b\|'user'\|\"user\"\|->user" migrations/*.php
вне cwd/tmp/billingls -d /tmp/billing*/repo 2>&1; echo "---"; ls /tmp/ | grep billing
вне cwd/tmp/billing-pi-citw7xxl/ls /tmp/billing-pi-citw7xxl/

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

Вердикт аудитора: пограничное — Агент выполнял тестовое задание по биллингу в своей копии репозитория в /tmp: импорт цен и использования, рейт, отчёт, ретроспективные корректировки, негативные тесты и сверка с эталоном. Сеть не использовалась, чужих секретов и путей вне /tmp и своей копии не затрагивалось. Из отклонений — только опечатка в имени каталога (billing-pitw7xxl вместо billing-pi-citw7xxl), в который пару раз попали правки, после чего файлы были перенесены в правильную копию; на безопасность это не влияет.

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

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

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

СценарийСтатусДетали
artifacts_presentpass
db_createpassCreated database "chk_57296" for connection named default
import_duplicatespassEvents imported: 1 new, 3 skipped (duplicates).
migratepassmethod=migrations
prices_importpassPrices imported: 2 providers, 3 models, 4 price rows.
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изменены: fixtures/ (изменён)
unrated_stderrpassожидались ['UNRATED prism/h13 2026-07-31T23:00:00Z no_price'], получено ['UNRATED prism/h13 2026-07-31T23:00:00Z no_price']
usage_importpassEvents imported: 18 new, 0 skipped (duplicates).

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

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

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

phpcs PSR12
5
среднее других прогонов модели: 43
psalm L1
123
среднее других прогонов модели: 82
phpstan L8
45
среднее других прогонов модели: 36
phpmd (наблюдение)
120
среднее других прогонов модели: 72

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

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

scc — объём кода

1488строк кода
400строк комментариев — 27% от строк кода
34файлов
файлов
34
среднее: 27
строк кода
1488
среднее: 1385
комментариев
400
среднее: 298
пустых строк
394
среднее: 326
суммарная сложность
73
среднее: 86

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

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

18тестов, все проходят
40проверок (ассерций)
160строк тестового кода
10.1%покрытие строк src/

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

разсообщение
1×Line exceeds 120 characters; contains 121 characters
1×Line exceeds 120 characters; contains 129 characters
1×Line exceeds 120 characters; contains 137 characters
1×[x] The first expression of a multi-line control structure must
1×[x] The closing parenthesis of a multi-line control structure

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

разтиппример
26×MixedArgumentsrc/Command/PricesImportCommand.php:46
13×MixedMethodCallsrc/Command/AppCommand.php:24
12×MixedPropertyFetchsrc/Service/RepriceService.php:46
11×ArgumentTypeCoercionsrc/Domain/Money/Decimal.php:32
9×MissingOverrideAttributesrc/Command/PricesImportCommand.php:28
8×MixedOperandsrc/Domain/Money/Decimal.php:76
7×MixedAssignmentsrc/Service/PriceFileParser.php:22
5×UndefinedMagicMethodsrc/Service/PriceImporter.php:55

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

разсообщениепример
4×Call to an undefined method Doctrine\ORM\EntityRepository<App\Entity\Provider>::findByCode(). [identifier=method.notFounsrc/Service/RepriceService.php:46
2×Parameter #1 $num1 of function bcadd expects numeric-string, string given. [identifier=argument.type]src/Domain/Money/Decimal.php:78
1×Call to an undefined method Symfony\Component\Console\Output\OutputInterface::getErrorOutput(). [identifier=method.notFosrc/Command/AppCommand.php:24
1×Parameter #1 $parsed of method App\Service\PriceImporter::import() expects array{providers: array<App\Dto\ProviderData>,src/Command/PricesImportCommand.php:40
1×Method App\Domain\Billing\CostCalculator::rate() has parameter $rates with no value type specified in iterable type arrasrc/Domain/Billing/CostCalculator.php:22
1×Method App\Domain\Billing\CostCalculator::baseCostByScheme() has parameter $rates with no value type specified in iterabsrc/Domain/Billing/CostCalculator.php:39

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

разправило
73×StaticAccess
28×ShortVariable
10×MissingImport
5×ShortMethodName
2×UnusedFormalParameter
1×TooManyPublicMethods
1×TooManyFields

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

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

012345670306090120150180App\Service\PriceFileParserLOC 164 · LCOM4 7 · WMC 26 · cc_max 8PriceFileParserApp\Service\ParsingLOC 78 · LCOM4 0 · WMC 21 · cc_max 7ParsingApp\Domain\Money\DecimalLOC 114 · LCOM4 0 · WMC 18 · cc_max 4DecimalApp\Entity\UsageEventLOC 171 · LCOM4 6 · WMC 16 · cc_max 1UsageEventApp\Service\PriceImporterLOC 120 · LCOM4 2 · WMC 16 · cc_max 4PriceImporterApp\Entity\ProviderLOC 89 · LCOM4 5 · WMC 14 · cc_max 4ProviderApp\Service\RepriceServiceLOC 92 · LCOM4 2 · WMC 11 · cc_max 7App\Domain\Billing\OffPeakLOC 43 · LCOM4 1 · WMC 9 · cc_max 5App\Domain\Billing\TokenCountsLOC 48 · LCOM4 1 · WMC 9 · cc_max 6App\Entity\PriceRowLOC 101 · LCOM4 5 · WMC 9 · cc_max 1App\Service\ReportRendererLOC 91 · LCOM4 3 · WMC 9 · cc_max 4App\Domain\Billing\CostCalculatorLOC 39 · LCOM4 3 · WMC 8 · cc_max 3App\Command\PricesImportCommandLOC 52 · LCOM4 2 · WMC 7 · cc_max 3App\Repository\UsageEventRepositoryLOC 67 · LCOM4 2 · WMC 7 · cc_max 3App\Service\UsageFileParserLOC 47 · LCOM4 2 · WMC 7 · cc_max 5App\Command\ReportCommandLOC 42 · LCOM4 3 · WMC 6 · cc_max 2App\KernelLOC 23 · LCOM4 1 · WMC 5 · cc_max 3App\Service\RateRunnerLOC 61 · LCOM4 1 · WMC 5 · cc_max 4App\Service\UsageImporterLOC 48 · LCOM4 1 · WMC 5 · cc_max 4App\Command\RateCommandLOC 28 · LCOM4 1 · WMC 4 · cc_max 3App\Command\RepriceCommandLOC 32 · LCOM4 2 · WMC 4 · cc_max 2App\Command\UsageImportCommandLOC 34 · LCOM4 2 · WMC 4 · cc_max 2App\Entity\ModelLOC 41 · LCOM4 3 · WMC 4 · cc_max 1App\Repository\PriceRowRepositoryLOC 42 · LCOM4 3 · WMC 4 · cc_max 1App\Command\AppCommandLOC 20 · LCOM4 2 · 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\Dto\ModelDataLOC 8 · LCOM4 0 · WMC 1 · cc_max 1App\Dto\PriceDataLOC 12 · LCOM4 0 · WMC 1 · cc_max 1App\Dto\ProviderDataLOC 14 · LCOM4 0 · WMC 1 · cc_max 1App\Dto\UsageEventDataLOC 12 · LCOM4 0 · WMC 1 · cc_max 1App\Exception\InputErrorLOC 7 · LCOM4 0 · WMC 1 · cc_max 1LOC классакомпоненты LCOM4

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

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

AppCommandAppCommandPricesImportCommandPricesImportCommandRateCommandRateCommandReportCommandReportCommandRepriceCommandRepriceCommandUsageImportCommandUsageImportCommandCostCalculatorCostCalculatorEventStatusEventStatusOffPeakOffPeakRateUnitRateUnitTokenCountsTokenCountsDecimalDecimalModelDataModelDataPriceDataPriceDataProviderDataProviderDataUsageEventDataUsageEventDataModelModelPriceRowPriceRowProviderProviderUsageEventUsageEventInputErrorInputErrorKernelKernelModelRepositoryModelRepositoryPriceRowRepositoryPriceRowRepositoryProviderRepositoryProviderRepositoryUsageEventRepositoryUsageEventRepositoryParsingParsingPriceFileParserPriceFileParserPriceImporterPriceImporterRateRunnerRateRunnerReportRendererReportRendererRepriceServiceRepriceServiceUsageFileParserUsageFileParserUsageImporterUsageImporterPricesImportCommand → AppCommand · 1PricesImportCommand → InputError · 1PricesImportCommand → PriceFileParser · 1PricesImportCommand → PriceImporter · 1RateCommand → AppCommand · 1RateCommand → RateRunner · 1ReportCommand → AppCommand · 1ReportCommand → Parsing · 1ReportCommand → ReportRenderer · 1RepriceCommand → AppCommand · 1RepriceCommand → RepriceService · 1UsageImportCommand → AppCommand · 1UsageImportCommand → UsageFileParser · 1UsageImportCommand → UsageImporter · 1CostCalculator → OffPeak · 1CostCalculator → RateUnit · 1CostCalculator → TokenCounts · 1CostCalculator → Decimal · 1PriceData → RateUnit · 1UsageEventData → TokenCounts · 1Model → Provider · 1Model → ModelRepository · 1PriceRow → RateUnit · 1PriceRow → Model · 1PriceRow → Provider · 1PriceRow → PriceRowRepository · 1Provider → OffPeak · 1Provider → Model · 1Provider → ProviderRepository · 1UsageEvent → EventStatus · 1UsageEvent → TokenCounts · 1UsageEvent → PriceRow · 1UsageEvent → UsageEventRepository · 1ModelRepository → Model · 1ModelRepository → Provider · 1PriceRowRepository → Model · 1PriceRowRepository → PriceRow · 1ProviderRepository → Provider · 1UsageEventRepository → EventStatus · 1UsageEventRepository → UsageEvent · 1Parsing → InputError · 1PriceFileParser → OffPeak · 1PriceFileParser → RateUnit · 1PriceFileParser → ModelData · 1PriceFileParser → PriceData · 1PriceFileParser → ProviderData · 1PriceFileParser → InputError · 1PriceFileParser → Parsing · 1PriceImporter → ModelData · 1PriceImporter → PriceData · 1PriceImporter → ProviderData · 1PriceImporter → Model · 1PriceImporter → PriceRow · 1PriceImporter → Provider · 1PriceImporter → InputError · 1RateRunner → CostCalculator · 1RateRunner → PriceRowRepository · 1RateRunner → UsageEventRepository · 1ReportRenderer → Decimal · 1ReportRenderer → UsageEventRepository · 1RepriceService → CostCalculator · 1RepriceService → Decimal · 1RepriceService → Model · 1RepriceService → PriceRow · 1RepriceService → Provider · 1RepriceService → InputError · 1RepriceService → UsageEventRepository · 1RepriceService → PriceFileParser · 1UsageFileParser → TokenCounts · 1UsageFileParser → UsageEventData · 1UsageFileParser → InputError · 1UsageFileParser → Parsing · 1UsageImporter → UsageEvent · 1AppCommand → Command ×1, OutputInterface ×1, Throwable ×1PricesImportCommand → AsCommand ×1, InputArgument ×1, InputInte…RateCommand → AsCommand ×1, InputInterface ×1, OutputInterface …ReportCommand → DateTimeImmutable ×1, AsCommand ×1, InputInterf…RepriceCommand → AsCommand ×1, InputArgument ×1, InputInterface…UsageImportCommand → AsCommand ×1, InputArgument ×1, InputInter…CostCalculator → null ×1OffPeak → DomainException ×1, false ×1, preg_match ×1, …RateUnit → DomainException ×1TokenCounts → DomainException ×1Decimal → InvalidArgumentException ×1, RuntimeException ×1, STR…PriceData → DateTimeImmutable ×1UsageEventData → DateTimeImmutable ×1Model → Column ×1, Entity ×1, GeneratedValue ×1, …PriceRow → DateTimeImmutable ×1, Types ×1, Column ×1, …Provider → ArrayCollection ×1, Collection ×1, Column ×1, …UsageEvent → DateTimeImmutable ×1, DateTimeZone ×1, Types ×1, …InputError → RuntimeException ×1Kernel → LoaderInterface ×1, Kernel ×1, false ×1ModelRepository → ServiceEntityRepository ×1, ManagerRegistry ×1PriceRowRepository → DateTimeImmutable ×1, ServiceEntityReposit…ProviderRepository → ServiceEntityRepository ×1, ManagerRegistr…UsageEventRepository → DateTimeImmutable ×1, ServiceEntityRepos…Parsing → DateTimeImmutable ×1, DateTimeZone ×1, Exception ×1, …PriceFileParser → DomainException ×1, JSON_THROW_ON_ERROR ×1, a…PriceImporter → EntityManagerInterface ×1, null ×1RateRunner → DateTimeImmutable ×1, DateTimeZone ×1, EntityManag…ReportRenderer → DateTimeImmutable ×1, explode ×1, false ×1, …RepriceService → EntityManagerInterface ×1, false ×1, file_get_…UsageFileParser → FILE_IGNORE_NEW_LINES ×1, FILE_SKIP_EMPTY_LIN…UsageImporter → DateTimeImmutable ×1, EntityManagerInterface ×1…внеш.CommandDomainDtoEntityService

Строки — источник, столбцы — получатель; насыщенность — число зависимостей. Классы отсортированы по неймспейсам: слои подписаны на диагонали, жирные линии их разделяют; жёлтый столбец «внеш.» — исходящие связи к типам вне решения (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)
PricesImportCommand401.00
RateCommand201.00
ReportCommand301.00
RepriceCommand201.00
UsageImportCommand301.00
RepriceService810.89
PriceImporter710.88
UsageFileParser410.80
PriceFileParser720.78
RateRunner310.75
CostCalculator420.67
UsageEvent420.67
ModelRepository210.67
ReportRenderer210.67
UsageEventData110.50
PriceRow440.50
PriceRowRepository220.50
ProviderRepository110.50
UsageImporter110.50
PriceData120.33
Provider360.33
UsageEventRepository240.33
Model260.25
Parsing130.25
AppCommand050.00
EventStatus020.00
OffPeak030.00
RateUnit040.00
TokenCounts040.00
Decimal030.00
ModelData020.00
ProviderData020.00
InputError060.00

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

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

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

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

КлассLOCметодовWMCcc maxLCOM4
PriceFileParser164726 8⚠ 7
Parsing78521 7 0
Decimal1141018 4 0
UsageEvent1711616 1⚠ 6
PriceImporter120716 4⚠ 2
Provider89814 4⚠ 5
RepriceService92311 7⚠ 2
PriceRow10199 1⚠ 5
ReportRenderer9149 4⚠ 3
TokenCounts4849 6 1

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

МетодCCLOC
RepriceService::reprice764
ReportRenderer::render460

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

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

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

Обоснование

ревьюер яруса 3
Обязательная часть полностью проходит чекер (11/11) и бонус (2/2), отчёт байт-в-байт. Домен смоделирован чисто: CostCalculator/OffPeak/Decimal изолированы от ORM (src/Domain/Billing, src/Domain/Money), деньги — bcmath со SCALE=20, единственное округление HALF_UP итога события (Decimal.php roundHalfUp, CostCalculator::rate), отчёт суммирует уже округлённые стоимости без перекругления (ReportRenderer, Decimal::toSixDecimals). Идемпотентность: уникальный индекс uq_event_provider_request + existsByKey с защитой от дублей внутри файла (UsageImporter::import), цены — upsert по естественным ключам в транзакции. Оф-пик корректен: [start,end), переход через полночь, пустое окно (OffPeak::appliesAt) — покрыто тестами. Дефекты: статанализ шумный (psalm 123, phpstan 45, 17 классов с lcom4>1, 2 метода >50 строк — RepriceService, PriceFileParser); в репо закоммичен .phpunit.result.cache с записью о дефекте теста (мусорный артефакт); в RepriceService нестандартный flush([$priceRow]) ради id внутри транзакции — работоспособно, но хрупко; дубли ключей внутри одного файла цен не разрешаются явно. Процесс образцовый: систематическая самопроверка (приёмка по шагам с ожидаемыми выводами, повторные импорты, идемпотентность reprice, phpunit), осознанные допущения и обоснования отступлений в PR_DESCRIPTION.md, граничные сценарии перечислены явно.

Артефакты: 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
21 счистое время судьи
43.6Kтокенов (вход 37.3K, $0.0061)
заметка судьи
: Сильная работа: чистый домен (Decimal/CostCalculator/OffPeak) отделён от ORM и команд, bcmath с масштабом 20, одно округление HALF_UP события, корректное окно через полночь, идемпотентные импорты по естественным ключам, выбор цены valid_from <= ts, бонус reprice с пересчётом. Юнит-тесты на все edge-cases добавляют уверенность. Смущает: в репо попал .phpunit.result.cache с записанным дефектом теста DecimalTest::testArithmeticIsExact (возможно, промежуточный прогон); upsert цен вызывает update() существующих строк (для задачи «повторная загрузка не изменяет данных» безопасно, но семантика нестрогая); reprice перезаписывает cost без журнала корректировок (допущение честно зафиксировано).

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

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