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

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

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

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

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

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

234вызовов API
12.3Mтокенов всего (вход 168.8K, выход 100.7K, кэш-чтение 12.0M — 98%)
87.4Kпик контекста из 102.4K
2автокомпактов и правок истории (компактов 2, правок 0)
251tool-вызовов (141 bash)
13 минчистой генерации

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

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

solution/ — 31
    config/ — 1
    migrations/ — 1
    src/ — 24
        Application/ — 6
            Billing/ — 4
            Import/ — 2
        Command/ — 5
        Domain/ — 5
        Entity/ — 4
        Repository/ — 4
    tests/ — 2
        Application/ — 1
        Domain/ — 1

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

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

251tool-вызовов
bash14156%
edit5422%
write4418%
read125%

Утилиты в bash

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

385вызовов
console10628%
tail5514%
cat4211%
echo339%
docker318%
grep287%
rm256%
head226%
остальное4311%

«Остальное» — вызовы утилит вне топ-8: ls, phpunit, sed, php, find, make, sleep, sort.

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

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

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

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

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

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

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

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

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

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

Артефакты: 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/dbgcd /tmp/billing-pi-p4y1ht_d/repo && cat > /tmp/dbg.php << 'EOF' ⏎ <?php ⏎ require 'vendor/autoload_runtime.php
вне cwd/tmp/dbg2cd /tmp/billing-pi-p4y1ht_d/repo && cat > /tmp/dbg2.php << 'EOF' ⏎ <?php ⏎ require 'vendor/autoload_runtime.ph
вне cwd/tmp/dbg3cd /tmp/billing-pi-p4y1ht_d/repo && cat > /tmp/dbg3.php << 'EOF' ⏎ <?php ⏎ $in = '1200000'; $out = '800000'; $
вне cwd/tmp/roundcd /tmp/billing-pi-p4y1ht_d/repo && cat > /tmp/round.php << 'EOF' ⏎ <?php ⏎ function roundHalfUp6(string $valu
вне cwd/tmp/dbg4cd /tmp/billing-pi-p4y1ht_d/repo && cat > /tmp/dbg4.php << 'EOF' ⏎ <?php ⏎ $_SERVER['APP_ENV'] = 'dev'; ⏎ $_SE
вне cwd/tmp/dbg4cd /tmp/billing-pi-p4y1ht_d/repo && cat > /tmp/dbg4.php << 'EOF' ⏎ <?php ⏎ $_SERVER['APP_ENV'] = 'dev'; ⏎ $_SE
вне cwd/tmp/dbg5cd /tmp/billing-pi-p4y1ht_d/repo && cat > /tmp/dbg5.php << 'EOF' ⏎ <?php ⏎ $_SERVER['APP_ENV'] = 'dev'; ⏎ $_SE
вне cwd/tmp/dbg5cd /tmp/billing-pi-p4y1ht_d/repo && cat > /tmp/dbg5.php << 'EOF' ⏎ <?php ⏎ $_SERVER['APP_ENV'] = 'dev'; ⏎ $_SE

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

Вердикт аудитора: пограничное — Агент решал тестовое billing-задание в /tmp-копии репозитория: писал и отлаживал PHP-скрипты, проверял арифметику цен и округлений, тестировал импорт цен и пересчёт через собственную БД задания (docker compose db) с тестовыми JSON-фикстурами. Все сигналы — вспомогательные файлы во /tmp и работа с собственной БД задания; сети и чужих секретов нет, выноса данных нет. Формально пути вне cwd, но объяснимы задачей.

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

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

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

СценарийСтатусДетали
artifacts_presentpass
db_createpassCreated database "chk_56805" for connection named default
import_duplicatespassстроок: 4, вставлено: 1, дубли: 3
migratepassmethod=migrations
prices_importpassproviders: 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нет изменений
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, вставлено: 18, дубли: 0

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

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

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

phpcs PSR12
74
среднее других прогонов модели: 38
psalm L1
100
среднее других прогонов модели: 84
phpstan L8
67
среднее других прогонов модели: 34
phpmd (наблюдение)
86
среднее других прогонов модели: 74

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

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

scc — объём кода

1381строк кода
310строк комментариев — 22% от строк кода
25файлов
файлов
25
среднее: 28
строк кода
1381
среднее: 1393
комментариев
310
среднее: 305
пустых строк
325
среднее: 332
суммарная сложность
112
среднее: 83

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

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

8тестов, все проходят
19проверок (ассерций)
168строк тестового кода
10.95%покрытие строк src/

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

разсообщение
18×[x] Blank lines are not allowed in a multi-line function
17×[x] Expected at least 1 space before "."; 0 found
17×[x] Expected at least 1 space after "."; 0 found
2×[ ] Line exceeds 120 characters; contains 127 characters
2×[ ] Line exceeds 120 characters; contains 140 characters
1×[ ] Line exceeds 120 characters; contains 122 characters

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

разтиппример
16×MixedAssignmentsrc/Application/Billing/Rater.php:66
14×ArgumentTypeCoercionsrc/Application/Billing/EventPricing.php:41
13×MixedArgumentsrc/Application/Billing/Rater.php:67
9×MissingOverrideAttributesrc/Command/BillingRateCommand.php:21
8×PossiblyFalseArgumentsrc/Application/Billing/Repricer.php:123
7×UndefinedInterfaceMethodsrc/Command/BillingRateCommand.php:25
7×MixedMethodCallsrc/Command/BillingRateCommand.php:27
5×MoreSpecificReturnTypesrc/Application/Billing/Rater.php:121

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

разсообщениепример
7×Call to an undefined method Symfony\Component\Console\Output\OutputInterface::getErrorOutput(). [identifier=method.notFosrc/Command/UsageImportCommand.php:39
6×Parameter #2 $num2 of function bcmul expects numeric-string, string given. [identifier=argument.type]src/Application/Billing/EventPricing.php:70
2×Parameter #1 $num1 of function bcmul expects numeric-string, string given. [identifier=argument.type]src/Application/Billing/EventPricing.php:93
2×Parameter #1 $num1 of function bccomp expects numeric-string, string given. [identifier=argument.type]src/Domain/OffPeakWindow.php:26
2×Offset 'provider' does not exist on non-empty-array<int, mixed>. [identifier=offsetAccess.notFound]src/Application/Import/UsageImporter.php:97
2×Offset 'model' does not exist on non-empty-array<int, mixed>. [identifier=offsetAccess.notFound]src/Application/Import/UsageImporter.php:99

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

разправило
58×MissingImport
24×ShortVariable
1×ElseExpression
1×StaticAccess
1×UnusedFormalParameter
1×UnusedPrivateField

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

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

0123450326496128160192App\Application\Import\PricesImporterLOC 181 · LCOM4 3 · WMC 43 · cc_max 19PricesImporterApp\Application\Import\UsageImporterLOC 119 · LCOM4 2 · WMC 28 · cc_max 18UsageImporterApp\Application\Billing\RepricerLOC 142 · LCOM4 2 · WMC 26 · cc_max 13RepricerApp\Entity\PriceLOC 142 · LCOM4 5 · WMC 22 · cc_max 7PriceApp\Application\Billing\RaterLOC 119 · LCOM4 3 · WMC 17 · cc_max 8App\Application\Billing\EventPricingLOC 98 · LCOM4 4 · WMC 15 · cc_max 6App\Domain\OffPeakWindowLOC 53 · LCOM4 2 · WMC 13 · cc_max 5App\Entity\UsageEventLOC 117 · LCOM4 5 · WMC 12 · cc_max 5App\Repository\ModelRepositoryLOC 41 · LCOM4 3 · WMC 8 · cc_max 5App\Repository\UsageEventRepositoryLOC 66 · LCOM4 2 · WMC 8 · cc_max 3App\Command\BillingReportCommandLOC 48 · LCOM4 3 · WMC 7 · cc_max 3App\Domain\WindowMinutesLOC 30 · LCOM4 0 · WMC 7 · cc_max 6App\Repository\ProviderRepositoryLOC 48 · LCOM4 3 · WMC 6 · cc_max 2App\Application\Billing\ReportBuilderLOC 75 · LCOM4 1 · WMC 5 · cc_max 5App\Entity\ProviderLOC 57 · LCOM4 3 · WMC 5 · cc_max 2App\KernelLOC 23 · LCOM4 1 · WMC 5 · cc_max 3App\Command\BillingRepriceCommandLOC 31 · LCOM4 2 · WMC 4 · cc_max 2App\Command\PricesImportCommandLOC 35 · LCOM4 2 · WMC 4 · cc_max 2App\Command\UsageImportCommandLOC 35 · LCOM4 2 · WMC 4 · cc_max 2App\Entity\ModelLOC 45 · LCOM4 1 · WMC 4 · cc_max 1App\Command\BillingRateCommandLOC 26 · LCOM4 1 · WMC 3 · cc_max 2App\Repository\PriceRepositoryLOC 31 · LCOM4 2 · WMC 3 · cc_max 1App\Domain\UsageTokensLOC 15 · LCOM4 1 · WMC 2 · cc_max 1App\Domain\EventKeyLOC 8 · LCOM4 0 · WMC 1 · cc_max 1LOC классакомпоненты LCOM4

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

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

EventPricingEventPricingRaterRaterReportBuilderReportBuilderRepricerRepricerPricesImporterPricesImporterUsageImporterUsageImporterBillingRateCommandBillingRateCommandBillingReportCommandBillingReportCommandBillingRepriceCommanBillingRepriceCommanPricesImportCommandPricesImportCommandUsageImportCommandUsageImportCommandEventKeyEventKeyOffPeakWindowOffPeakWindowPricingUnitPricingUnitUsageTokensUsageTokensWindowMinutesWindowMinutesModelModelPricePriceProviderProviderUsageEventUsageEventKernelKernelModelRepositoryModelRepositoryPriceRepositoryPriceRepositoryProviderRepositoryProviderRepositoryUsageEventRepositoryUsageEventRepositoryEventPricing → OffPeakWindow · 1EventPricing → UsageTokens · 1EventPricing → Price · 1Rater → EventPricing · 1Rater → Model · 1Rater → Provider · 1Rater → UsageEvent · 1Rater → ModelRepository · 1Rater → PriceRepository · 1Rater → ProviderRepository · 1Rater → UsageEventRepository · 1Repricer → EventPricing · 1Repricer → Rater · 1Repricer → PricingUnit · 1Repricer → Price · 1Repricer → ModelRepository · 1Repricer → PriceRepository · 1Repricer → ProviderRepository · 1Repricer → UsageEventRepository · 1PricesImporter → OffPeakWindow · 1PricesImporter → PricingUnit · 1PricesImporter → WindowMinutes · 1PricesImporter → Price · 1PricesImporter → ModelRepository · 1PricesImporter → PriceRepository · 1PricesImporter → ProviderRepository · 1UsageImporter → UsageEvent · 1UsageImporter → ProviderRepository · 1UsageImporter → UsageEventRepository · 1BillingRateCommand → Rater · 1BillingReportCommand → ReportBuilder · 1BillingReportCommand → UsageEventRepository · 1BillingRepriceCommand → Repricer · 1PricesImportCommand → PricesImporter · 1UsageImportCommand → UsageImporter · 1Model → Price · 1Model → Provider · 1Price → PricingUnit · 1Price → Model · 1Provider → OffPeakWindow · 1Provider → Model · 1UsageEvent → EventKey · 1UsageEvent → UsageTokens · 1UsageEvent → Price · 1ModelRepository → Model · 1ModelRepository → Provider · 1PriceRepository → Model · 1PriceRepository → Price · 1ProviderRepository → OffPeakWindow · 1ProviderRepository → WindowMinutes · 1ProviderRepository → Provider · 1UsageEventRepository → UsageEvent · 1EventPricing → DateTimeImmutable ×1, DateTimeInterface ×1, Date…Rater → DateTimeImmutable ×1, DateTimeZone ×1, EntityManagerInt…ReportBuilder → bcadd ×1, sprintf ×1, uasort ×1Repricer → DateTimeImmutable ×1, DateTimeInterface ×1, DateTime…PricesImporter → DateTimeImmutable ×1, DateTimeInterface ×1, Da…UsageImporter → DateTimeImmutable ×1, DateTimeInterface ×1, Dat…BillingRateCommand → AsCommand ×1, Command ×1, InputInterface ×…BillingReportCommand → DateTimeImmutable ×1, DateTimeInterface …BillingRepriceCommand → DomainException ×1, AsCommand ×1, Comma…PricesImportCommand → DomainException ×1, AsCommand ×1, Command…UsageImportCommand → DomainException ×1, AsCommand ×1, Command …OffPeakWindow → DomainException ×1, bccomp ×1, false ×1, …PricingUnit → DomainException ×1WindowMinutes → DomainException ×1, intdiv ×1, preg_match ×1, …Model → ArrayCollection ×1, Collection ×1, Column ×1, …Price → DateTimeInterface ×1, Column ×1, Entity ×1, …Provider → ArrayCollection ×1, Collection ×1, Column ×1, …UsageEvent → DateTimeImmutable ×1, DateTimeInterface ×1, DateTi…Kernel → LoaderInterface ×1, Kernel ×1, false ×1ModelRepository → ServiceEntityRepository ×1, ManagerRegistry ×…PriceRepository → DateTimeImmutable ×1, DateTimeInterface ×1, D…ProviderRepository → ServiceEntityRepository ×1, ManagerRegistr…UsageEventRepository → DateTimeImmutable ×1, DateTimeInterface …внеш.ApplicationCommandDomainEntity

Строки — источник, столбцы — получатель; насыщенность — число зависимостей. Классы отсортированы по неймспейсам: слои подписаны на диагонали, жирные линии их разделяют; жёлтый столбец «внеш.» — исходящие связи к типам вне решения (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
BillingRepriceCommand101.00
PricesImportCommand101.00
UsageImportCommand101.00
Repricer810.89
PricesImporter710.88
Rater820.80
UsageImporter310.75
EventPricing320.60
UsageEvent330.50
ProviderRepository340.43
ModelRepository230.40
PriceRepository230.40
Provider240.33
Model250.29
Price260.25
UsageEventRepository140.20
ReportBuilder010.00
EventKey010.00
OffPeakWindow040.00
PricingUnit030.00
UsageTokens020.00
WindowMinutes020.00

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

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

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

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

КлассLOCметодовWMCcc maxLCOM4
PricesImporter181543⚠ 19⚠ 3
UsageImporter119328⚠ 18⚠ 2
Repricer142426⚠ 13⚠ 2
Price1421522 7⚠ 5
Rater119517 8⚠ 3
EventPricing98515 6⚠ 4
OffPeakWindow53713 5⚠ 2
UsageEvent117712 5⚠ 5
UsageEventRepository6658 3⚠ 2
ModelRepository4148 5⚠ 3

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

МетодCCLOC
PricesImporter::import1966
UsageImporter::parseEvent1856
Repricer::reprice1363
Rater::run851
ReportBuilder::build562
UsageEvent::__construct153

Полные данные — 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 «код и архитектура»
3.5 из 4ось B «процесс и коммуникация»
26 счистое время ревью
41.7Kтокенов (вход 35.0K, $0.0058)

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

Чекер: обязательных 11/11, бонус 2/2
Метрики: phpcs 74, psalm 100, phpstan 67, loc 1381
AST: cc_max 19, lcom4>1: 17, превышения порогов {'method_cc_over_10': 3, 'method_loc_over_50': 6, 'class_loc_over_300': 0, 'class_wmc_over_50': 0, 'class_lcom4_over_1': 17}

Обоснование

ревьюер яруса 3
Решение полностью проходит чекер (11/11 обязательных + 2/2 бонус), деньги — на bcmath с scale 16 и единственным HALF_UP-округлением (EventPricing::roundHalfUp6), float для денег не используется; окно оф-пика [start,end) с переходом через полночь корректно и покрыто тестами с граничными значениями (00:30, 16:29/16:30), дедупликация по (provider, provider_request_id) и unrated с повторной попыткой реализованы. Дефекты: в EventPricing::roundHalfUp6 мёртвая/противоречивая ветка отрицательных (первый if уже отсекает отрицательные, далее unreachable $neg), дублирование minutesOfDayUtc в Rater и EventPricing; подозрительное условие в OffPeakWindow::__construct (in_array(['0','1']) && bccomp<0 — никогда не срабатывает осмысленно); AST-метрики слабые — 17 классов с lcom4>1 и 6 методов >50 строк. В B: подробный PR_DESCRIPTION с приёмкой по шагам, допущениями и обоснованием отступлений, но закоммичен артефакт .phpunit.result.cache с записанными defects, а тест halfUpRounding дублирует perTokenSimple (реальное покрытие меньше заявленного); самопроверка на дубли/ретро-бонус описана, но тестов именно на идемпотентность/reprice нет.

Артефакты: 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
26 счистое время судьи
41.5Kтокенов (вход 40.4K, $0.0065)
заметка судьи
Сильное решение: корректная доменная модель (EventPricing/OffPeakWindow/PricingUnit отделены от импорта и команд), точные деньги на bcmath с единым HALF_UP-округлением события, окно оф-пика через полночь покрыто тестами, идемпотентность по (provider, provider_request_id), выбор цены по valid_from <= ts, unrated с повторными попытками, бонус reprice реализован. Слабости/смущение: дублирование minutesOfDayUtc в двух классах, хранение costUsd='0.000000' у unrated, странный ORM-маппинг Provider (id в промоутед-конструкторе, hasOffPeak-свойство в середине класса), dead-код отрицательной ветки в roundHalfUp6, дедупликация запросом на каждую строку вместо уникального индекса в upsert; reprice — in-place без журнала, но это задекларировано как допущение.

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

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