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

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

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

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

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

10 из 11обязательные сценарии
2 из 2бонусные сценарии
2.5 из 4код и архитектура
3 из 4процесс и коммуникация
2.5 из 4слепое ревью
10 миндлительность прогона
13.9Mтокенов, кэш 99%
2433строк
47файлов в решении (без 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, автокомпакты, чем самопроверялся.

255вызовов API
13.9Mтокенов всего (вход 111.0K, выход 71.5K, кэш-чтение 13.7M — 99%)
86.0Kпик контекста из 102.4K
3автокомпактов и правок истории (компактов 1, правок 2)
287tool-вызовов (189 bash)
9 минчистой генерации

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

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

solution/ — 26
    fixtures/ — 1
    migrations/ — 1
    src/ — 22
        Billing/ — 14
            Dto/ — 1
            Engine/ — 3
            Entity/ — 7
            Parser/ — 2
            Store/ — 1
        Command/ — 5
        Domain/ — 3
            Billing/ — 3

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

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

287tool-вызовов
bash18966%
write4817%
edit3311%
read166%
search10%

Утилиты в bash

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

494вызовов
head8417%
grep7315%
echo5812%
cat5411%
console4910%
tail439%
php326%
sed255%
остальное7615%

«Остальное» — вызовы утилит вне топ-8: docker, ls, rm, wc, find, python3, psql, make, inspect, mv, xargs, sort.

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

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

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

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

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

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

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

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

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

Вызовов печатной отладки (var_dump / dump / dd) в финальном коде: 0. Уникальных файлов создано или отредактировано: 31.

Артефакты: 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/retrycd /tmp/billing-pi-18_l1f7m/repo && php -r 'class A{enum X{}}' 2>&1 | head -3; cat > /tmp/retry.php << 'EOF' ⏎
вне cwd/tmp/retrycd /tmp/billing-pi-18_l1f7m/repo && php /tmp/retry.php 2>&1 | head; cat > /tmp/retry2.php << 'EOF' ⏎ <?php ⏎ n
вне cwd/tmp/retry3cd /tmp/billing-pi-18_l1f7m/repo && cat > /tmp/retry3.php << 'EOF' ⏎ <?php ⏎ namespace App\X; ⏎ enum Zz: strin
вне cwd/tmp/retry4cd /tmp/billing-pi-18_l1f7m/repo && cat > /tmp/retry4.php << 'EOF' ⏎ <?php ⏎ namespace App\Test; ⏎ final class
вне cwd/tmp/retry5cd /tmp/billing-pi-18_l1f7m/repo && cat > /tmp/retry5.php << 'EOF' ⏎ <?php ⏎ namespace App\Test2; ⏎ enum Aaa {
вне cwd/tmp/retry6cd /tmp/billing-pi-18_l1f7m/repo && cat > /tmp/retry6.php << 'EOF' ⏎ <?php ⏎ namespace App\Test3; ⏎ enum Aaa {
вне cwd/tmp/retry7cd /tmp/billing-pi-18_l1f7m/repo && cat > /tmp/retry7.php << 'EOF' ⏎ <?php ⏎ namespace App\Test3; ⏎ final clas
вне cwd/tmp/retry8cd /tmp/billing-pi-18_l1f7m/repo && cat > /tmp/retry8.php << 'EOF' ⏎ <?php ⏎ namespace App\Test4; ⏎ final clas

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

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

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

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

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

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

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

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

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

phpcs PSR12
26
среднее других прогонов модели: 42
psalm L1
99
среднее других прогонов модели: 84
phpstan L8
23
среднее других прогонов модели: 37
phpmd (наблюдение)
152
среднее других прогонов модели: 69

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

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

scc — объём кода

1449строк кода
282строк комментариев — 19% от строк кода
23файлов
файлов
23
среднее: 28
строк кода
1449
среднее: 1388
комментариев
282
среднее: 307
пустых строк
344
среднее: 330
суммарная сложность
116
среднее: 83

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

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

разсообщение
11×[x] Closing brace must be on a line by itself
3×[ ] Line exceeds 120 characters; contains 121 characters
2×[ ] Line exceeds 120 characters; contains 133 characters
2×[ ] Line exceeds 120 characters; contains 125 characters
1×[ ] Line exceeds 120 characters; contains 134 characters
1×[ ] Line exceeds 120 characters; contains 137 characters

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

разтиппример
21×MixedAssignmentsrc/Billing/Parser/PricesCatalogLoader.php:50
16×ArgumentTypeCoercionsrc/Billing/Parser/PricesCatalogLoader.php:95
10×MixedArgumentsrc/Billing/Dto/TokenCounts.php:43
9×MixedArgumentTypeCoercionsrc/Billing/Parser/UsageLoader.php:87
9×MissingOverrideAttributesrc/Command/BillingRateCommand.php:25
6×RedundantCastGivenDocblockTypesrc/Command/BillingReportCommand.php:76
4×PropertyNotSetInConstructorsrc/Billing/Entity/PriceRow.php:28
4×PossiblyUndefinedArrayOffsetsrc/Command/BillingReportCommand.php:114

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

разсообщениепример
5×Parameter #1 $num1 of function bccomp expects numeric-string, string given. [identifier=argument.type]src/Domain/Billing/OffPeakWindow.php:30
2×Call to an undefined method Symfony\Component\Console\Output\OutputInterface::getErrorOutput(). [identifier=method.notFosrc/Command/BillingRateCommand.php:43
1×Method App\Billing\Dto\TokenCounts::fromArray() has parameter $a with no value type specified in iterable type array. [isrc/Billing/Dto/TokenCounts.php:41
1×Method App\Billing\Engine\RepriceService::reprice() should return int but returns array<string, int|string>. [identifiersrc/Billing/Engine/RepriceService.php:35
1×Strict comparison using === between non-empty-string and null will always evaluate to false. [identifier=identical.alwaysrc/Billing/Parser/PricesCatalogLoader.php:206
1×Parameter #1 $num1 of function bcadd expects numeric-string, non-empty-string given. [identifier=argument.type]src/Billing/Parser/PricesCatalogLoader.php:213

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

разправило
73×MissingImport
51×ShortVariable
15×StaticAccess
6×ElseExpression
2×ErrorControlOperator
2×TooManyPublicMethods
1×TooManyMethods
1×UnusedFormalParameter

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

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

012304896144192240288App\Billing\Parser\PricesCatalogLoaderLOC 222 · LCOM4 1 · WMC 44 · cc_max 16PricesCatalogLoaderApp\Billing\Store\StoreLOC 269 · LCOM4 2 · WMC 38 · cc_max 6StoreApp\Domain\Billing\MoneyLOC 140 · LCOM4 1 · WMC 36 · cc_max 10MoneyApp\Billing\Parser\UsageLoaderLOC 154 · LCOM4 1 · WMC 28 · cc_max 10UsageLoaderApp\Command\BillingRepriceCommandLOC 166 · LCOM4 2 · WMC 24 · cc_max 12BillingRepriceCommandApp\Domain\Billing\OffPeakWindowLOC 65 · LCOM4 3 · WMC 16 · cc_max 4App\Billing\Engine\EventCostCalculatorLOC 80 · LCOM4 2 · WMC 14 · cc_max 9App\Command\BillingReportCommandLOC 110 · LCOM4 2 · WMC 12 · cc_max 10App\Billing\Dto\TokenCountsLOC 37 · LCOM4 1 · WMC 8 · cc_max 5App\Billing\Engine\RepriceServiceLOC 79 · LCOM4 2 · WMC 6 · cc_max 4App\Command\PricesImportCommandLOC 46 · LCOM4 2 · WMC 5 · cc_max 3App\Command\UsageImportCommandLOC 45 · LCOM4 2 · WMC 5 · cc_max 3App\KernelLOC 23 · LCOM4 1 · WMC 5 · cc_max 3App\Billing\Engine\RateServiceLOC 57 · LCOM4 1 · WMC 4 · cc_max 3App\Billing\Entity\EntityIdLOC 29 · LCOM4 0 · WMC 4 · cc_max 1App\Command\BillingRateCommandLOC 36 · LCOM4 1 · WMC 4 · cc_max 3App\Billing\Entity\ModelLOC 20 · LCOM4 0 · WMC 1 · cc_max 1App\Billing\Entity\PriceRowLOC 38 · LCOM4 0 · WMC 1 · cc_max 1App\Billing\Entity\ProviderLOC 27 · LCOM4 0 · WMC 1 · cc_max 1App\Billing\Entity\RatedEventLOC 44 · LCOM4 0 · WMC 1 · cc_max 1App\Billing\Entity\UsageEventLOC 38 · LCOM4 0 · WMC 1 · cc_max 1App\Billing\Entity\RepriceLogLOC 33 · LCOM4 0 · WMC 0 · cc_max 0LOC классакомпоненты LCOM4

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

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

TokenCountsTokenCountsEventCostCalculatorEventCostCalculatorRateServiceRateServiceRepriceServiceRepriceServiceEntityIdEntityIdModelModelPriceRowPriceRowProviderProviderRatedEventRatedEventRepriceLogRepriceLogUsageEventUsageEventPricesCatalogLoaderPricesCatalogLoaderUsageLoaderUsageLoaderStoreStoreBillingRateCommandBillingRateCommandBillingReportCommandBillingReportCommandBillingRepriceCommanBillingRepriceCommanPricesImportCommandPricesImportCommandUsageImportCommandUsageImportCommandMoneyMoneyOffPeakWindowOffPeakWindowPricingSchemePricingSchemeKernelKernelEventCostCalculator → TokenCounts · 1EventCostCalculator → PriceRow · 1EventCostCalculator → Store · 1EventCostCalculator → Money · 1EventCostCalculator → PricingScheme · 1RateService → TokenCounts · 1RateService → EventCostCalculator · 1RateService → RatedEvent · 1RateService → Store · 1RepriceService → TokenCounts · 1RepriceService → EventCostCalculator · 1RepriceService → RepriceLog · 1RepriceService → Store · 1RepriceService → Money · 1PricesCatalogLoader → EntityId · 1PricesCatalogLoader → Model · 1PricesCatalogLoader → PriceRow · 1PricesCatalogLoader → Provider · 1PricesCatalogLoader → Store · 1PricesCatalogLoader → PricingScheme · 1UsageLoader → EntityId · 1UsageLoader → UsageEvent · 1UsageLoader → Store · 1Store → EntityId · 1Store → Model · 1Store → PriceRow · 1Store → Provider · 1Store → RatedEvent · 1Store → RepriceLog · 1Store → UsageEvent · 1Store → OffPeakWindow · 1BillingRateCommand → RateService · 1BillingReportCommand → Store · 1BillingReportCommand → Money · 1BillingRepriceCommand → RepriceService · 1BillingRepriceCommand → EntityId · 1BillingRepriceCommand → Model · 1BillingRepriceCommand → PriceRow · 1BillingRepriceCommand → Provider · 1BillingRepriceCommand → PricesCatalogLoader · 1BillingRepriceCommand → Store · 1BillingRepriceCommand → Money · 1PricesImportCommand → PricesCatalogLoader · 1UsageImportCommand → UsageLoader · 1TokenCounts → InvalidArgumentException ×1, PHP_INT_MAX ×1EventCostCalculator → DateTimeImmutable ×1, InvalidArgumentExce…RateService → DateTimeImmutable ×1, null ×1RepriceService → DateTimeImmutable ×1, null ×1EntityId → DateTimeImmutable ×1, explode ×1Model → Column ×1, Entity ×1, GeneratedValue ×1, …PriceRow → DateTimeImmutable ×1, Column ×1, Entity ×1, …Provider → Column ×1, Entity ×1, GeneratedValue ×1, …RatedEvent → DateTimeImmutable ×1, Column ×1, Entity ×1, …RepriceLog → DateTimeImmutable ×1, Column ×1, Entity ×1, …UsageEvent → DateTimeImmutable ×1, Column ×1, Entity ×1, …PricesCatalogLoader → DateTimeImmutable ×1, DateTimeZone ×1, Ex…UsageLoader → DateTimeImmutable ×1, DateTimeZone ×1, Exception …Store → DateTimeImmutable ×1, ArrayParameterType ×1, EntityMana…BillingRateCommand → AsCommand ×1, Command ×1, InputInterface ×…BillingReportCommand → DateTimeImmutable ×1, DateTimeZone ×1, E…BillingRepriceCommand → DateTimeImmutable ×1, DateTimeZone ×1, …PricesImportCommand → AsCommand ×1, Command ×1, InputArgument ×…UsageImportCommand → AsCommand ×1, Command ×1, InputArgument ×1…Money → InvalidArgumentException ×1, STR_PAD_LEFT ×1, bcadd ×1,…OffPeakWindow → InvalidArgumentException ×1, bccomp ×1, preg_ma…PricingScheme → InvalidArgumentException ×1, implode ×1, sprint…Kernel → LoaderInterface ×1, Kernel ×1, false ×1внеш.BillingCommandDomain

Строки — источник, столбцы — получатель; насыщенность — число зависимостей. Классы отсортированы по неймспейсам: слои подписаны на диагонали, жирные линии их разделяют; жёлтый столбец «внеш.» — исходящие связи к типам вне решения (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
BillingRepriceCommand801.00
PricesImportCommand101.00
UsageImportCommand101.00
RepriceService510.83
RateService410.80
PricesCatalogLoader620.75
UsageLoader310.75
EventCostCalculator520.71
Store870.53
TokenCounts030.00
EntityId040.00
Model030.00
PriceRow040.00
Provider030.00
RatedEvent020.00
RepriceLog020.00
UsageEvent020.00
Money040.00
OffPeakWindow010.00
PricingScheme020.00

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

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

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

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

КлассLOCметодовWMCcc maxLCOM4
PricesCatalogLoader222844⚠ 16 1
Store2692338 6⚠ 2
Money1401536 10 1
UsageLoader154528 10 1
BillingRepriceCommand166724⚠ 12⚠ 2
OffPeakWindow65616 4⚠ 3
EventCostCalculator80414 9⚠ 2
BillingReportCommand110312 10⚠ 2
TokenCounts3748 5 1
RepriceService7936 4⚠ 2

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

МетодCCLOC
PricesCatalogLoader::upsertPriceRow1671
BillingRepriceCommand::execute1280
BillingReportCommand::execute1091
EventCostCalculator::baseCost952
UsageLoader::parseLine655
RepriceService::repricePriceRows458

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

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

Чекер: обязательных 10/11, бонус 2/2
  fail unrated_stderr: ожидались ['UNRATED prism/h13 2026-07-31T23:00:00Z no_price'], получено []
Метрики: phpcs 26, psalm 99, phpstan 23, loc 1449
AST: cc_max 16, lcom4>1: 8, превышения порогов {'method_cc_over_10': 2, 'method_loc_over_50': 6, 'class_loc_over_300': 0, 'class_wmc_over_50': 0, 'class_lcom4_over_1': 8}

Обоснование

ревьюер яруса 3
Сильные стороны: чёткое разделение слоёв (Domain/Engine/Parser/Command), деньги на bcmath с одним округлением HALF_UP (Money::halfUp6), бонус reprice реализован и прошёл чекер 2/2, дедупликация по natural key (provider, request_id), миграция разворачивает схему, PR_DESCRIPTION системный с обоснованиями решений. Дефекты: единственный обязательный fail — формат stderr unrated (BillingRateCommand пишет `$u['reason'] ?? 'no price'`, а RateService кладёт id/ts/provider без 'reason', и формат строки `UNRATED <provider>/<id> <ts> no_price` вообще не собран). Скрытые проблемы: Money::halfUp6 при округлении к нулю возвращает '' (ветка `$six===0 && $intBig===0`), что уронит assertDecimal в конструкторе Money для малых стоимостей; в отчёте modelRefForEvent вызывается на каждое событие — N+1; парсинг коррекции в BillingRepriceCommand дублирует PricesCatalogLoader без его валидаций (upsertPriceRow/minutes повторены, rates не нормализуются bcmath). Не наказываю за LOC/CC — стилевые пороги не оговорены заданием. Axis B: самопроверка описана по шагам и на граничных условиях (идемпотентность, ретро-бонус), но заявленная «байт-в-байт» приёмка пропустила формат unrated — систематичность есть, точность на границах неполная.

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

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

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

2.5 из 4ось A «код и архитектура», вслепую
+0расхождение осей A: ярус 3 − ярус 4
25 счистое время судьи
38.7Kтокенов (вход 37.8K, $0.0061)
заметка судьи
Сильные стороны: чистый доменный слой (Money на bcmath, OffPeakWindow с окном через полночь, enum схем тарификации), точная цена по valid_from <= ts, идемпотентность по (provider, request_id) через натуральный ключ, бонус reprice реализован с журналом. Слабости: unrated-строка в stderr не соответствует требуемому формату (печатается просто 'no price'); app:usage:import жёстко требует наличия модели/провайдера в справочнике, что противоречит сценарию «цены могли подтянуться позже» (unrated-события); в reprice-команде дублируется валидация цен без неё (rates берутся сырыми строками); дублирование парсинга между PricesCatalogLoader и BillingRepriceCommand; тестов нет. В целом — рабочее решение с аккуратной арифметикой, но с несколькими шероховатостями в границах и edge-case'ах.

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

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