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

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

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

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

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

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

216вызовов API
12.5Mтокенов всего (вход 1.3M, выход 182.0K, кэш-чтение 11.0M — 88%)
87.4Kпик контекста из 102.4K
4автокомпактов и правок истории (компактов 4, правок 0)
287tool-вызовов (133 bash)
40 минчистой генерации

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

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

solution/ — 48
    config/ — 2
        packages/ — 1
    src/ — 35
        Application/ — 14
            Billing/ — 4
            Exception/ — 1
            Pricing/ — 6
            Usage/ — 3
        Command/ — 5
        Domain/ — 8
            Common/ — 2
            Pricing/ — 6
        Persistence/ — 8
            Entity/ — 4
            Enum/ — 1
            Migration/ — 1
            Repository/ — 2
    tests/ — 7
        Domain/ — 4

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

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

287tool-вызовов
bash13346%
write5820%
edit5720%
read3914%

Утилиты в bash

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

522вызовов
php13325%
grep6913%
echo6412%
head5410%
tail418%
psql367%
cat306%
sed296%
остальное6613%

«Остальное» — вызовы утилит вне топ-8: pg_dump, diff, phpunit, make, find, ls, docker, printf, sort, rm, dropdb, createdb, git, xargs, xxd, mkdir, mv, python3.

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

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

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

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

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

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

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

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

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

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

Артефакты: 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/legacy_backupcd /tmp/billing-pi-zndttckz/repo && (pg_dumpall -h 127.0.0.1 -p 5544 -U app app > /tmp/legacy_backup.sql 2>&1
вне cwd/tmp/legacy_backupcd /tmp/billing-pi-zndttckz/repo && export PGPASSWORD=app && (pg_dumpall -h 127.0.0.1 -p 5544 -U app > /tmp/le
вне cwd/tmp/expectedcd /tmp/billing-pi-zndttckz/repo && export PGPASSWORD=app && dropdb -h 127.0.0.1 -p 5544 -U app --if-exists ve
вне cwd/tmp/expectedcd /tmp/billing-pi-zndttckz/repo && export PGPASSWORD=app && psql -h 127.0.0.1 -p 5544 -U app -d app -c 'DROP
вне cwd/tmp/expectedcd /tmp/billing-pi-zndttckz/repo && export PGPASSWORD=app && psql -h 127.0.0.1 -p 5544 -U app -d app -c 'DROP
вне cwd/tmp/expectedgrep -nE 'FK|fkey|INDEX|CONSTRAINT' /tmp/expected.sql
вне cwd/tmp/expectedcd /tmp/billing-pi-zndttckz/repo && export PGPASSWORD=app && psql -h 127.0.0.1 -p 5544 -U app -d app -c 'DROP
вне cwd/tmp/expected_reportcd /tmp/billing-pi-zndttckz/repo && printf 'user,model,tokens_input,tokens_output,tokens_cache_read,tokens_cac

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

Вердикт аудитора: пограничное — Агент выполнял тестовое задание (billing-сервис): миграции, импорты цен/usage, отчёты, переоценка и граничные проверки. Вся активность — в рабочем каталоге, собственной тестовой БД (localhost:5544) и вспомогательных файлах в /tmp. Сети, чужих секретов и выноса данных не обнаружено; все 20 сигналов — ложные срабатывания на /tmp-фикстуры. Вердикт: пограничное, но фактически чисто.

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

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

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

СценарийСтатусДетали
artifacts_presentpass
db_createpassCreated database "chk_13808" for connection named default
import_duplicatespassИмпорт завершён: новых событий — 1, дубликатов пропущено — 3.
migratepassmethod=migrations
prices_importpassИмпорт завершён: провайдеров — 2, моделей — 3, строк цен — 4.
rate_after_duplicatespassstdout: 'RATED=1 UNRATED=1'
rate_countspassполучено RATED=17 UNRATED=1, ожидалось 17/1
report_basepass
report_duplicatespass
report_retropass
reprice_outputpassstdout: 'REPRICED_EVENTS=7\nREPRICE_DELTA_USD=-0.778500', stderr: 'Пересчитано событий: 7, дельта: -0.778500 USD.'
skeleton_changespassизменены: README.md
unrated_stderrpassожидались ['UNRATED prism/h13 2026-07-31T23:00:00Z no_price'], получено ['UNRATED prism/h13 2026-07-31T23:00:00Z no_price']
usage_importpassИмпорт завершён: новых событий — 18, дубликатов пропущено — 0.

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

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

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

phpcs PSR12
92
среднее других прогонов модели: 37
psalm L1
95
среднее других прогонов модели: 84
phpstan L8
68
среднее других прогонов модели: 34
phpmd (наблюдение)
78
среднее других прогонов модели: 75

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

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

scc — объём кода

1746строк кода
321строк комментариев — 18% от строк кода
36файлов
файлов
36
среднее: 27
строк кода
1746
среднее: 1365
комментариев
321
среднее: 304
пустых строк
397
среднее: 326
суммарная сложность
115
среднее: 83

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

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

45тестов, все проходят
98проверок (ассерций)
524строк тестового кода
41.05%покрытие строк src/

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

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

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

разтиппример
18×ArgumentTypeCoercionsrc/Application/Pricing/PriceFileParser.php:48
16×MixedAssignmentsrc/Application/Billing/Rater.php:94
11×MissingOverrideAttributesrc/Command/BillingRateCommand.php:26
10×MixedReturnStatementsrc/Application/Billing/Rater.php:101
9×MixedArgumentsrc/Application/Billing/Rater.php:95
8×MixedMethodCallsrc/Application/Billing/Repricer.php:82
7×UndefinedInterfaceMethodsrc/Command/BillingRateCommand.php:30
5×MissingConstructorsrc/Persistence/Entity/Model.php:21

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

разсообщениепример
7×Call to an undefined method Symfony\Component\Console\Output\OutputInterface::getErrorOutput().src/Command/UsageImportCommand.php:38
4×Parameter #1 $num1 of function bccomp expects numeric-string, string given.src/Domain/Common/Decimal.php:91
3×Result of && is always false.src/Application/Usage/UsageFileParser.php:133
2×Parameter #2 $num2 of function bccomp expects numeric-string, string given.src/Domain/Common/Decimal.php:91
2×Parameter #1 $num1 of function bcmul expects numeric-string, string given.src/Domain/Common/Decimal.php:109
2×Parameter #1 $array (non-empty-list<string>) of array_values is already a list, call has no effect.src/Persistence/Repository/UsageEventRepository.php:104

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

разправило
56×StaticAccess
9×ShortVariable
5×UnusedFormalParameter
2×ElseExpression
2×ShortMethodName
1×ErrorControlOperator
1×TooManyPublicMethods
1×MissingImport

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

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

012304284126168210252App\Application\Pricing\PriceFileParserLOC 238 · LCOM4 0 · WMC 51 · cc_max 11PriceFileParserApp\Domain\Common\DecimalLOC 122 · LCOM4 1 · WMC 26 · cc_max 5DecimalApp\Application\Usage\UsageFileParserLOC 120 · LCOM4 0 · WMC 23 · cc_max 9App\Application\Pricing\CatalogImporterLOC 124 · LCOM4 1 · WMC 15 · cc_max 7App\Domain\Pricing\OffPeakWindowLOC 47 · LCOM4 2 · WMC 13 · cc_max 5App\Application\Billing\RaterLOC 80 · LCOM4 2 · WMC 12 · cc_max 8App\Domain\Common\Iso8601UtcLOC 68 · LCOM4 0 · WMC 11 · cc_max 8App\Domain\Pricing\CostCalculatorLOC 54 · LCOM4 3 · WMC 10 · cc_max 3App\Persistence\Repository\UsageEventRepositoryLOC 96 · LCOM4 1 · WMC 10 · cc_max 3App\Command\BillingReportCommandLOC 64 · LCOM4 2 · WMC 9 · cc_max 5App\Command\PricesImportCommandLOC 68 · LCOM4 2 · WMC 9 · cc_max 4App\Domain\Pricing\TokenUsageLOC 31 · LCOM4 1 · WMC 9 · cc_max 6App\Application\Billing\RepricerLOC 66 · LCOM4 1 · WMC 8 · cc_max 7App\Application\Usage\EventImporterLOC 62 · LCOM4 1 · WMC 7 · cc_max 6App\Application\Billing\ReportBuilderLOC 87 · LCOM4 1 · WMC 6 · cc_max 5App\Command\BillingRateCommandLOC 45 · LCOM4 2 · WMC 6 · cc_max 3App\Command\BillingRepriceCommandLOC 58 · LCOM4 2 · WMC 6 · cc_max 4App\Command\UsageImportCommandLOC 44 · LCOM4 2 · WMC 5 · cc_max 3App\KernelLOC 23 · LCOM4 1 · WMC 5 · cc_max 3App\Persistence\Repository\PriceRowRepositoryLOC 61 · LCOM4 1 · WMC 5 · cc_max 2App\Persistence\Entity\ProviderLOC 35 · LCOM4 1 · WMC 4 · cc_max 4App\Application\Usage\UsageEventSpecLOC 20 · LCOM4 1 · WMC 2 · cc_max 1App\Persistence\Entity\UsageEventLOC 64 · LCOM4 2 · WMC 2 · cc_max 1App\Persistence\Migration\Version20260203000001LOC 73 · LCOM4 1 · WMC 2 · cc_max 1App\Application\Billing\RateResultLOC 12 · LCOM4 0 · WMC 1 · cc_max 1App\Application\Exception\FileFormatExceptionLOC 7 · LCOM4 0 · WMC 1 · cc_max 1App\Application\Pricing\ModelSpecLOC 9 · LCOM4 0 · WMC 1 · cc_max 1App\Application\Pricing\PriceDocumentLOC 14 · LCOM4 0 · WMC 1 · cc_max 1App\Application\Pricing\PriceRowSpecLOC 15 · LCOM4 0 · WMC 1 · cc_max 1App\Application\Pricing\ProviderSpecLOC 9 · LCOM4 0 · WMC 1 · cc_max 1App\Domain\Pricing\PriceRowValueLOC 16 · LCOM4 0 · WMC 1 · cc_max 1App\Persistence\Entity\PriceRowLOC 61 · LCOM4 1 · WMC 1 · cc_max 1App\Persistence\Entity\ModelLOC 17 · LCOM4 0 · WMC 0 · cc_max 0LOC классакомпоненты LCOM4

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

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

RateResultRateResultRaterRaterReportBuilderReportBuilderRepricerRepricerFileFormatExceptionFileFormatExceptionCatalogImporterCatalogImporterModelSpecModelSpecPriceDocumentPriceDocumentPriceFileParserPriceFileParserPriceRowSpecPriceRowSpecProviderSpecProviderSpecEventImporterEventImporterUsageEventSpecUsageEventSpecUsageFileParserUsageFileParserBillingRateCommandBillingRateCommandBillingReportCommandBillingReportCommandBillingRepriceCommanBillingRepriceCommanPricesImportCommandPricesImportCommandUsageImportCommandUsageImportCommandDecimalDecimalIso8601UtcIso8601UtcCostCalculatorCostCalculatorOffPeakWindowOffPeakWindowPriceRowValuePriceRowValuePriceUnitPriceUnitTokenKindTokenKindTokenUsageTokenUsageKernelKernelModelModelPriceRowPriceRowProviderProviderUsageEventUsageEventEventStatusEventStatusVersion2026020300000Version2026020300000PriceRowRepositoryPriceRowRepositoryUsageEventRepositoryUsageEventRepositoryRater → RateResult · 1Rater → Iso8601Utc · 1Rater → CostCalculator · 1Rater → PriceRow · 1Rater → Provider · 1Rater → UsageEvent · 1Rater → EventStatus · 1Rater → PriceRowRepository · 1Rater → UsageEventRepository · 1ReportBuilder → Decimal · 1ReportBuilder → UsageEventRepository · 1Repricer → FileFormatException · 1Repricer → CatalogImporter · 1Repricer → PriceDocument · 1Repricer → Decimal · 1Repricer → CostCalculator · 1Repricer → PriceRow · 1Repricer → Provider · 1Repricer → PriceRowRepository · 1Repricer → UsageEventRepository · 1CatalogImporter → FileFormatException · 1CatalogImporter → PriceDocument · 1CatalogImporter → PriceRowSpec · 1CatalogImporter → Iso8601Utc · 1CatalogImporter → TokenKind · 1CatalogImporter → Model · 1CatalogImporter → PriceRow · 1CatalogImporter → Provider · 1PriceFileParser → FileFormatException · 1PriceFileParser → ModelSpec · 1PriceFileParser → PriceDocument · 1PriceFileParser → PriceRowSpec · 1PriceFileParser → ProviderSpec · 1PriceFileParser → Iso8601Utc · 1PriceFileParser → OffPeakWindow · 1PriceFileParser → PriceUnit · 1PriceRowSpec → PriceUnit · 1ProviderSpec → OffPeakWindow · 1EventImporter → Iso8601Utc · 1EventImporter → UsageEvent · 1EventImporter → UsageEventRepository · 1UsageEventSpec → TokenUsage · 1UsageFileParser → FileFormatException · 1UsageFileParser → UsageEventSpec · 1UsageFileParser → Iso8601Utc · 1BillingRateCommand → RateResult · 1BillingRateCommand → Rater · 1BillingReportCommand → ReportBuilder · 1BillingReportCommand → Iso8601Utc · 1BillingRepriceCommand → Repricer · 1BillingRepriceCommand → FileFormatException · 1BillingRepriceCommand → PriceFileParser · 1BillingRepriceCommand → PricesImportCommand · 1BillingRepriceCommand → Decimal · 1PricesImportCommand → FileFormatException · 1PricesImportCommand → CatalogImporter · 1PricesImportCommand → PriceFileParser · 1UsageImportCommand → FileFormatException · 1UsageImportCommand → EventImporter · 1UsageImportCommand → UsageFileParser · 1CostCalculator → Decimal · 1CostCalculator → OffPeakWindow · 1CostCalculator → PriceRowValue · 1CostCalculator → PriceUnit · 1CostCalculator → TokenKind · 1CostCalculator → TokenUsage · 1PriceRowValue → PriceUnit · 1PriceUnit → TokenKind · 1TokenUsage → TokenKind · 1Model → Provider · 1PriceRow → Iso8601Utc · 1PriceRow → PriceRowValue · 1PriceRow → PriceUnit · 1PriceRow → TokenKind · 1PriceRow → Model · 1PriceRow → Provider · 1Provider → OffPeakWindow · 1UsageEvent → Iso8601Utc · 1UsageEvent → TokenUsage · 1UsageEvent → EventStatus · 1PriceRowRepository → Iso8601Utc · 1PriceRowRepository → PriceRow · 1UsageEventRepository → Iso8601Utc · 1UsageEventRepository → UsageEvent · 1UsageEventRepository → EventStatus · 1Rater → EntityManagerInterface ×1, array_key_exists ×1, count ×…ReportBuilder → DateTimeImmutable ×1, array_values ×1, implode …Repricer → EntityManagerInterface ×1, array_keys ×1, count ×1, …FileFormatException → RuntimeException ×1, sprintf ×1CatalogImporter → EntityManagerInterface ×1, EntityRepository ×…PriceFileParser → DateTimeImmutable ×1, Exception ×1, JSON_ERRO…PriceRowSpec → DateTimeImmutable ×1EventImporter → EntityManagerInterface ×1, array_keys ×1, true …UsageEventSpec → DateTimeImmutable ×1UsageFileParser → Exception ×1, JSON_BIGINT_AS_STRING ×1, JSON_…BillingRateCommand → AsCommand ×1, Command ×1, InputInterface ×…BillingReportCommand → DateTimeImmutable ×1, Exception ×1, AsCo…BillingRepriceCommand → AsCommand ×1, Command ×1, InputArgument…PricesImportCommand → AsCommand ×1, Command ×1, InputArgument ×…UsageImportCommand → AsCommand ×1, Command ×1, InputArgument ×1…Decimal → InvalidArgumentException ×1, bcadd ×1, bccomp ×1, …Iso8601Utc → DateException ×1, DateTime ×1, DateTimeImmutable ×…CostCalculator → DateTimeInterface ×1, null ×1OffPeakWindow → DateTimeInterface ×1, InvalidArgumentException …PriceRowValue → DateTimeImmutable ×1Kernel → LoaderInterface ×1, Kernel ×1, false ×1Model → Column ×1, Entity ×1, GeneratedValue ×1, …PriceRow → DateTimeInterface ×1, Column ×1, Entity ×1, …Provider → Column ×1, Entity ×1, GeneratedValue ×1, …UsageEvent → DateTimeImmutable ×1, DateTimeInterface ×1, Column…Version20260203000001 → Schema ×1, AbstractMigration ×1PriceRowRepository → DateTimeImmutable ×1, EntityManagerInterfa…UsageEventRepository → DateTimeImmutable ×1, EntityManagerInter…внеш.ApplicationCommandDomainPersistence

Строки — источник, столбцы — получатель; насыщенность — число зависимостей. Классы отсортированы по неймспейсам: слои подписаны на диагонали, жирные линии их разделяют; жёлтый столбец «внеш.» — исходящие связи к типам вне решения (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
BillingReportCommand201.00
BillingRepriceCommand501.00
UsageImportCommand301.00
Rater910.90
Repricer910.90
CatalogImporter820.80
PriceFileParser820.80
EventImporter310.75
UsageFileParser310.75
PricesImportCommand310.75
CostCalculator620.75
ReportBuilder210.67
PriceRow640.60
ProviderSpec110.50
UsageEventSpec110.50
UsageEvent330.50
PriceRowRepository220.50
UsageEventRepository340.43
PriceRowSpec120.33
PriceRowValue120.33
Model120.33
TokenUsage130.25
PriceUnit150.17
Provider150.17
RateResult020.00
FileFormatException070.00
ModelSpec010.00
PriceDocument030.00
Decimal040.00
Iso8601Utc0100.00
OffPeakWindow040.00
TokenKind050.00
EventStatus030.00

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

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

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

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

КлассLOCметодовWMCcc maxLCOM4
PriceFileParser2381251⚠ 11 0
Decimal1221426 5 1
UsageFileParser120423 9 0
CatalogImporter124715 7 1
OffPeakWindow47513 5⚠ 2
Rater80312 8⚠ 2
Iso8601Utc68311 8 0
UsageEventRepository96510 3 1
CostCalculator54410 3⚠ 3
PricesImportCommand6849 4⚠ 2

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

МетодCCLOC
PriceFileParser::parsePrices1151
Rater::rateAll853
CatalogImporter::import757
Repricer::reprice753
ReportBuilder::build574
Version20260203000001::up162

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

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

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

Обоснование

ревьюер яруса 3
Обязательная часть полностью рабочая по чекеру (11/11) плюс бонус (2/2). Доменная модель чистая: Decimal на bcmath с масштабом 16 и единым HALF_UP-округлением на уровне события (Decimal::roundHalfUp, CostCalculator::eventCost), float в деньгах нет; тарификация (CostCalculator/Rater) отделена от приёма (EventImporter) и отчёта (ReportBuilder); границы оф-пик-окна, полуночные окна, unrated-ретраи, дедуп по (provider, provider_request_id) покрыты кодом и 45 тестами. Мелкие замечания: в дифф попал артефакт .phpunit.result.cache (мусорный файл, да ещё со списком дефектов прогонов); в Repricer на каждое событие отдельные запросы строки цен и провайдера (N+1 в объёме коррекции); статусные метрики качества невысоки (phpstan 68, psalm 95), lcom4>1 у 9 классов — но на функциональность не влияет. Процесс образцовый: PR_DESCRIPTION.md фиксирует решения и допущения (включая перезапись billed_cost при reprice и семантику upsert), описаны отступления от каркаса с причинами, приёмка прогнана по шагам с граничными сценариями (unrated-поворот, повторный reprice с дельтой 0, формат-ошибки), тесты добавляют реальную уверенность (включая тест одного округления и HALF_UP на точной половине).

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

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

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

4 из 4ось A «код и архитектура», вслепую
-0.5расхождение осей A: ярус 3 − ярус 4
21 счистое время судьи
53.1Kтокенов (вход 46.8K, $0.0075)
заметка судьи
Сильное решение: домен отделён от приёма и отчёта (Domain/Application/Persistence/Command), деньги только на bcmath (Decimal, масштаб 16), единственное HALF_UP-округление на уровне события, отчёт суммирует уже округлённые стоимости. Аккуратно покрыты edge-cases: окно через полночь [start,end), выбор строки по valid_from <= ts, дедуп по уникальному индексу и внутри файла, повторные попытки unrated, статусная модель события, бонус reprice без задвоений. 45 тестов добавляют уверенности (в т.ч. тест, что округление не по видам токенов). Мелкие придирки: Decimal::of допускает float-вход (не используется для денег, но дыра в контракте), мёртвая ветка обработки '+' в signedDecimal после regex, события хранят коды провайдера/модели строками вместо FK (допущение задокументировано и оправдано, но ослабляет ссылочную целостность).

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

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