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

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

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

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

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

11 из 11обязательные сценарии
1 из 2бонусные сценарии
3 из 4код и архитектура
3 из 4процесс и коммуникация
3.5 из 4слепое ревью
16 миндлительность прогона
7.7Mтокенов, кэш 97%
2485строк
53файлов в решении (без vendor)

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

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

145вызовов API
7.7Mтокенов всего (вход 145.6K, выход 98.4K, кэш-чтение 7.5M — 97%)
88.6Kпик контекста из 102.4K
2автокомпактов и правок истории (компактов 2, правок 0)
197tool-вызовов (90 bash)
14 минчистой генерации

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

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

solution/ — 34
    config/ — 1
        packages/ — 1
    migrations/ — 1
    src/ — 23
        Command/ — 5
        DBAL/ — 1
            Types/ — 1
        Domain/ — 4
            Billing/ — 4
        Entity/ — 5
        Import/ — 1
        Repository/ — 2
        Service/ — 5
    tests/ — 3
        Domain/ — 3
            Billing/ — 3

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

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

197tool-вызовов
bash9046%
edit5025%
write3719%
read2010%

Утилиты в bash

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

275вызовов
echo9133%
cat3011%
head2710%
grep249%
sed218%
console166%
tail135%
php124%
остальное4115%

«Остальное» — вызовы утилит вне топ-8: tee, ls, psql, rm, diff, docker, make, which, find, mkdir, phpunit.

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

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

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

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

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

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

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

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

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

Тесты: 4 запусков — прошли 4, упали 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/report1export PGPASSWORD=app ⏎ psql -h 127.0.0.1 -p 5544 -U app -d postgres -c 'DROP DATABASE IF EXISTS billing_accep
вне cwd/tmp/metadbexport DATABASE_URL='postgresql://app:app@127.0.0.1:5544/billing_accept?serverVersion=17&charset=utf8' ⏎ cat >
вне cwd/tmp/metadbexport DATABASE_URL='postgresql://app:app@127.0.0.1:5544/billing_accept?serverVersion=17&charset=utf8' ⏎ cat >
вне cwd/tmp/metadb3cat > /tmp/metadb3.php <<'PHP' ⏎ <?php ⏎ chdir('/tmp/billing-pi-vz94kdi3/repo'); ⏎ putenv('APP_ENV=dev'); ⏎ re
вне cwd/tmp/metadb4sed -n 120,140p vendor/doctrine/doctrine-bundle/src/Repository/ServiceEntityRepository.php; export DATABASE_UR
вне cwd/tmp/metadb5export DATABASE_URL='postgresql://app:app@127.0.0.1:5544/billing_accept?serverVersion=17&charset=utf8' ⏎ cat >
вне cwd/tmp/metadb6export DATABASE_URL='postgresql://app:app@127.0.0.1:5544/billing_accept?serverVersion=17&charset=utf8' ⏎ cat >
вне cwd/tmp/report1export PGPASSWORD=app ⏎ psql -h 127.0.0.1 -p 5544 -U app -d postgres -q -c 'DROP DATABASE IF EXISTS billing_ac

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

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

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

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

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

СценарийСтатусДетали
artifacts_presentpass
db_createpassCreated database "chk_66329" for connection named default
import_duplicatespassusage:import — rows inserted=1 duplicates=3
migratepassmethod=migrations
prices_importpass
rate_after_duplicatespassstdout: 'RATED=1 UNRATED=1'
rate_countspassполучено RATED=17 UNRATED=1, ожидалось 17/1
report_basepass
report_duplicatespass
report_retropass
reprice_outputfailstdout: 'REPRICED_EVENTS=6\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_importpassusage:import — rows inserted=18 duplicates=0

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

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

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

phpcs PSR12
44
среднее других прогонов модели: 40
psalm L1
89
среднее других прогонов модели: 85
phpstan L8
30
среднее других прогонов модели: 37
phpmd (наблюдение)
70
среднее других прогонов модели: 75

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

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

scc — объём кода

1235строк кода
287строк комментариев — 23% от строк кода
24файлов
файлов
24
среднее: 28
строк кода
1235
среднее: 1404
комментариев
287
среднее: 307
пустых строк
307
среднее: 333
суммарная сложность
80
среднее: 85

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

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

10тестов, все проходят
28проверок (ассерций)
151строк тестового кода
6.32%покрытие строк src/

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

разсообщение
12×[x] Expected at least 1 space before "."; 0 found
12×[x] Expected at least 1 space after "."; 0 found
3×[x] Opening parenthesis of a multi-line function call must be
3×[x] Multi-line function call not indented correctly; expected
3×[x] Only one argument is allowed per line in a multi-line
3×[x] Closing parenthesis of a multi-line function call must be

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

разтиппример
13×MissingOverrideAttributesrc/Command/BillingRateCommand.php:21
11×MixedArrayAccesssrc/Import/InputParser.php:75
10×MixedArgumentsrc/Command/BillingReportCommand.php:33
10×ArgumentTypeCoercionsrc/Domain/Billing/MoneyMath.php:31
7×MixedAssignmentsrc/Command/BillingReportCommand.php:53
6×UndefinedInterfaceMethodsrc/Command/BillingRateCommand.php:31
6×MixedArgumentTypeCoercionsrc/Import/InputParser.php:55
4×DocblockTypeContradictionsrc/Command/BillingReportCommand.php:59

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

разсообщениепример
5×Call to an undefined method Symfony\Component\Console\Output\OutputInterface::getErrorOutput(). [identifier=method.notFosrc/Command/UsageImportCommand.php:32
3×Expression on left side of ?? is not nullable. [identifier=nullCoalesce.expr]src/Import/InputParser.php:257
2×Parameter #1 $num1 of function bcmul expects numeric-string, string given. [identifier=argument.type]src/Domain/Billing/MoneyMath.php:59
1×PHPDoc tag @var above a method has no effect. [identifier=varTag.misplaced]src/Command/BillingReportCommand.php:51
1×Call to an undefined method DateTimeInterface::setTimezone(). [identifier=method.notFound]src/DBAL/Types/TzDateStringType.php:68
1×Cannot call method setTimezone() on DateTimeImmutable|false. [identifier=method.nonObject]src/DBAL/Types/TzDateStringType.php:74

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

разправило
25×StaticAccess
21×ShortVariable
13×MissingImport
7×UnusedFormalParameter
2×ElseExpression
1×TooManyFields
1×ExcessiveMethodLength

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

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

012345056112168224280336App\Import\InputParserLOC 314 · LCOM4 0 · WMC 57 · cc_max 7InputParserApp\Service\PriceCatalogServiceLOC 90 · LCOM4 1 · WMC 13 · cc_max 6App\Command\BillingReportCommandLOC 53 · LCOM4 3 · WMC 11 · cc_max 5App\Service\RepriceServiceLOC 115 · LCOM4 2 · WMC 11 · cc_max 9App\DBAL\Types\TzDateStringTypeLOC 53 · LCOM4 5 · WMC 10 · cc_max 3App\Domain\Billing\PricingCalculatorLOC 60 · LCOM4 0 · WMC 10 · cc_max 6App\Domain\Billing\MoneyMathLOC 65 · LCOM4 0 · WMC 9 · cc_max 2App\Service\UsageIngestServiceLOC 77 · LCOM4 1 · WMC 9 · cc_max 8App\Command\PricesImportCommandLOC 38 · LCOM4 3 · WMC 7 · cc_max 3App\Domain\Billing\OffPeakLOC 29 · LCOM4 1 · WMC 7 · cc_max 5App\Service\BillingServiceLOC 70 · LCOM4 1 · WMC 7 · cc_max 4App\Service\ReportServiceLOC 71 · LCOM4 1 · WMC 6 · cc_max 5App\KernelLOC 23 · LCOM4 1 · WMC 5 · cc_max 3App\Command\BillingRepriceCommandLOC 33 · LCOM4 2 · WMC 4 · cc_max 2App\Command\UsageImportCommandLOC 36 · LCOM4 2 · WMC 4 · cc_max 2App\Entity\ProviderLOC 41 · LCOM4 1 · WMC 4 · cc_max 4App\Entity\UsageEventLOC 83 · LCOM4 2 · WMC 4 · cc_max 1App\Repository\PriceRepositoryLOC 40 · LCOM4 2 · WMC 4 · cc_max 2App\Repository\UsageEventRepositoryLOC 23 · LCOM4 1 · WMC 4 · cc_max 3App\Command\BillingRateCommandLOC 29 · LCOM4 1 · WMC 3 · cc_max 2App\Domain\Billing\UsageRecordLOC 50 · LCOM4 1 · WMC 3 · cc_max 1App\Entity\ModelLOC 18 · LCOM4 1 · WMC 1 · cc_max 1App\Entity\PriceLOC 32 · LCOM4 0 · WMC 0 · cc_max 0App\Entity\RepriceLogEntryLOC 27 · LCOM4 0 · WMC 0 · cc_max 0App\Import\InputFormatExceptionLOC 3 · LCOM4 0 · WMC 0 · cc_max 0LOC классакомпоненты LCOM4

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

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

BillingRateCommandBillingRateCommandBillingReportCommandBillingReportCommandBillingRepriceCommanBillingRepriceCommanPricesImportCommandPricesImportCommandUsageImportCommandUsageImportCommandTzDateStringTypeTzDateStringTypeMoneyMathMoneyMathOffPeakOffPeakPricingCalculatorPricingCalculatorUsageRecordUsageRecordModelModelPricePriceProviderProviderRepriceLogEntryRepriceLogEntryUsageEventUsageEventInputFormatExceptionInputFormatExceptionInputParserInputParserKernelKernelPriceRepositoryPriceRepositoryUsageEventRepositoryUsageEventRepositoryBillingServiceBillingServicePriceCatalogServicePriceCatalogServiceReportServiceReportServiceRepriceServiceRepriceServiceUsageIngestServiceUsageIngestServiceBillingRateCommand → BillingService · 1BillingReportCommand → ReportService · 1BillingRepriceCommand → InputFormatException · 1BillingRepriceCommand → RepriceService · 1PricesImportCommand → InputFormatException · 1PricesImportCommand → PriceCatalogService · 1UsageImportCommand → InputFormatException · 1UsageImportCommand → UsageIngestService · 1PricingCalculator → MoneyMath · 1PricingCalculator → OffPeak · 1Provider → OffPeak · 1InputParser → InputFormatException · 1PriceRepository → Price · 1PriceRepository → UsageEvent · 1UsageEventRepository → UsageEvent · 1BillingService → OffPeak · 1BillingService → PricingCalculator · 1BillingService → Provider · 1BillingService → UsageEvent · 1BillingService → PriceRepository · 1BillingService → UsageEventRepository · 1PriceCatalogService → TzDateStringType · 1PriceCatalogService → Model · 1PriceCatalogService → Price · 1PriceCatalogService → Provider · 1PriceCatalogService → InputFormatException · 1PriceCatalogService → InputParser · 1PriceCatalogService → PriceRepository · 1ReportService → MoneyMath · 1ReportService → UsageEventRepository · 1RepriceService → TzDateStringType · 1RepriceService → MoneyMath · 1RepriceService → PricingCalculator · 1RepriceService → Model · 1RepriceService → Price · 1RepriceService → Provider · 1RepriceService → RepriceLogEntry · 1RepriceService → UsageEvent · 1RepriceService → InputFormatException · 1RepriceService → InputParser · 1RepriceService → UsageEventRepository · 1UsageIngestService → UsageRecord · 1UsageIngestService → UsageEvent · 1UsageIngestService → InputFormatException · 1UsageIngestService → InputParser · 1BillingRateCommand → DateTimeImmutable ×1, AsCommand ×1, Comman…BillingReportCommand → DateTimeImmutable ×1, DateTimeInterface …BillingRepriceCommand → AsCommand ×1, Command ×1, InputArgument…PricesImportCommand → AsCommand ×1, Command ×1, InputArgument ×…UsageImportCommand → AsCommand ×1, Command ×1, InputArgument ×1…TzDateStringType → DateTimeImmutable ×1, DateTimeInterface ×1, …MoneyMath → bcadd ×1, bccomp ×1, bcdiv ×1, …OffPeak → DateTimeImmutable ×1, substr ×1, true ×1PricingCalculator → DateTimeImmutable ×1, DomainException ×1, a…UsageRecord → DateTimeImmutable ×1Model → Column ×1, Entity ×1, Id ×1, …Price → Column ×1, Entity ×1, Id ×1, …Provider → Column ×1, Entity ×1, Id ×1, …RepriceLogEntry → Column ×1, Entity ×1, GeneratedValue ×1, …UsageEvent → Column ×1, Entity ×1, GeneratedValue ×1, …InputFormatException → RuntimeException ×1InputParser → DateTimeImmutable ×1, DateTimeInterface ×1, DateT…Kernel → LoaderInterface ×1, Kernel ×1, false ×1PriceRepository → DateTimeImmutable ×1, ServiceEntityRepository…UsageEventRepository → DateTimeImmutable ×1, ServiceEntityRepos…BillingService → DateTimeImmutable ×1, EntityManagerInterface ×…PriceCatalogService → EntityManagerInterface ×1, null ×1, sprin…ReportService → DateTimeImmutable ×1, implode ×1, usort ×1RepriceService → DateTimeImmutable ×1, EntityManagerInterface ×…UsageIngestService → EntityManagerInterface ×1, false ×1, fclos…внеш.CommandDomainEntityService

Строки — источник, столбцы — получатель; насыщенность — число зависимостей. Классы отсортированы по неймспейсам: слои подписаны на диагонали, жирные линии их разделяют; жёлтый столбец «внеш.» — исходящие связи к типам вне решения (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
BillingReportCommand101.00
BillingRepriceCommand201.00
PricesImportCommand201.00
UsageImportCommand201.00
RepriceService1110.92
PriceCatalogService710.88
BillingService610.86
UsageIngestService410.80
ReportService210.67
PricingCalculator220.50
PriceRepository220.50
Provider130.25
InputParser130.25
UsageEventRepository130.25
TzDateStringType020.00
MoneyMath030.00
OffPeak030.00
UsageRecord010.00
Model020.00
Price030.00
RepriceLogEntry010.00
UsageEvent050.00
InputFormatException070.00

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

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

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

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

КлассLOCметодовWMCcc maxLCOM4
InputParser3141357 7 0
PriceCatalogService90413 6 1
RepriceService115311 9⚠ 2
BillingReportCommand53411 5⚠ 3
PricingCalculator60310 6 0
TzDateStringType53610 3⚠ 5
UsageIngestService7729 8 1
MoneyMath6589 2 0
BillingService7037 4 1
PricesImportCommand3847 3⚠ 3

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

МетодCCLOC
RepriceService::apply992
UsageIngestService::import866
PriceCatalogService::import656
ReportService::report559

Полные данные — 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 из 4ось A «код и архитектура»
3 из 4ось B «процесс и коммуникация»
29 счистое время ревью
40.7Kтокенов (вход 40.0K, $0.0063)

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

Чекер: обязательных 11/11, бонус 1/2
  fail reprice_output: stdout: 'REPRICED_EVENTS=6\nREPRICE_DELTA_USD=-0.778500', stderr: ''
Метрики: phpcs 44, psalm 89, phpstan 30, loc 1235
AST: cc_max 9, lcom4>1: 8, превышения порогов {'method_cc_over_10': 0, 'method_loc_over_50': 4, 'class_loc_over_300': 1, 'class_wmc_over_50': 1, 'class_lcom4_over_1': 8}

Обоснование

ревьюер яруса 3
Обязательная часть работает полностью (11/11 у чекера), домен смоделирован корректно: тарификация (PricingCalculator/OffPeak) отделена от приёма и отчёта, деньги — bcmath scale 16, ровно одно округление HALF_UP на событие, суммы отчёта — сумма уже округлённых (ReportService), дедуп через ON CONFLICT по (provider, provider_request_id), unrated повторяется. Покрыты границы окна оф-пика (включно/исключающе, через полночь), valid_from включительно, есть юнит-тесты домена. Дефекты: бонус провален наполовину — чекер fail reprice_output ('REPRICED_EVENTS=6 REPRICE_DELTA_USD=-0.778500' не совпал с ожидаемым), т.е. ретро-коррекция не согласована с эталоном. Документация противоречит коду: в PR_DESCRIPTION.md заявлено «start == end — окно не существует (false)», а в OffPeak::appliesAt код возвращает true (целые сутки). В репо попали отладочные артефакты err1.log/err2.log/err3.log с трейсами ошибок — след итеративной отладки без чистки. Метрики качества заметно просели (psalm 89, phpstan 30, phpcs 44; class_lcom4>1 у 8 классов, 4 метода >50 строк), время через самописный строковый DBAL-тип — рабочее, но хрупкое решение ради скалярного PK. Процесс хороший: систематическая самопроверка (тесты 10/28 ассерций, приёмка воспроизведена), развёрнутый PR_DESCRIPTION с допущениями и обоснованиями, но статанализ не прогнан/не вычищен и противоречие код-документация не замечено.

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

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

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

3.5 из 4ось A «код и архитектура», вслепую
-0.5расхождение осей A: ярус 3 − ярус 4
20 счистое время судьи
40.2Kтокенов (вход 39.7K, $0.0062)
заметка судьи
Сильное решение: домен (MoneyMath/OffPeak/PricingCalculator) отделён от приёма и отчёта, деньги только на bcmath без float, одно округление HALF_UP на событии, суммы отчёта из уже округлённых значений, оф-пик через полночь и границы окна покрыты кодом и юнит-тестами, есть бонус reprice с журналом. Смутило: в репо закоммичены отладочные err1–3.log и .phpunit.result.cache; противоречие по случаю start==end (в коде — целые сутки, в PR_DESCRIPTION заявлено «окно не существует»); время хранится строками через самописный DBAL-тип — работает и обосновано, но это обходной трюк, а не прямое моделирование времени.

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

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