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

qwen3.8-27b-pi-100k · run-06

Тег qwen3.8:27b-pi-100k · рассуждение medium · окно 102.4K · начало 2026-10-27 05:37

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

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

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

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

PI_CODING_AGENT_DIR=<tmp>/pi-home PI_OFFLINE=1 \
pi -p --session-dir <прогон>/pi-session \
    --model qwen3.8:27b-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, автокомпакты, чем самопроверялся.

191вызовов API
10.6Mтокенов всего (вход 158.9K, выход 132.1K, кэш-чтение 10.3M — 97%)
86.1Kпик контекста из 102.4K
3автокомпактов и правок истории (компактов 2, правок 1)
236tool-вызовов (98 bash)
19 минчистой генерации

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

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

solution/ — 44
    config/ — 1
        packages/ — 1
            test/ — 1
    fixtures/ — 1
    migrations/ — 1
    src/ — 29
        Command/ — 5
        Domain/ — 9
            Billing/ — 8
            Time/ — 1
        Entity/ — 4
            Billing/ — 4
        Repository/ — 4
            Billing/ — 4
        Service/ — 6
            Billing/ — 4
    tests/ — 7
        Domain/ — 3
        Integration/ — 4

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

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

236tool-вызовов
bash9842%
write5724%
edit5624%
read2511%

Утилиты в bash

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

336вызовов
console8826%
grep4614%
echo4614%
head257%
sed227%
cat196%
php196%
tail175%
остальное5416%

«Остальное» — вызовы утилит вне топ-8: phpunit, ls, docker, printf, rm, cmp, psql, find, which, rg, xargs, python3, make, pg_isready, composer, mv.

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

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

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

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

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

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

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

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

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

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

Артефакты: pi-session · pi-run.log · diff.patch · session-profile.md · summary.md

Аудит безопасности

Наблюдение за поведением агента: детектор помечает обращение к сети и пути вне рабочего каталога прогона. Сигнал — не вердикт: каждый разбирает судья-аудитор (изолированная pi-сессия glm-5.3-flash, checker/judge_audit.py). Полный список сигналов — security.json, построчный разбор — audit-verdict.md рядом с отчётом.

типобъекткоманда
вне cwd/tmp/report1cd /tmp/billing-pi-sl3_0ikd/repo && bin/console app:billing:report --from=2026-09-01T00:00:00Z --to=2026-10-01
вне cwd/tmp/usage_extracd /tmp/billing-pi-sl3_0ikd/repo && cat > /tmp/usage_extra.jsonl <<'EOF' ⏎ {"provider": "prism", "provider_req
вне cwd/tmp/correctioncd /tmp/billing-pi-sl3_0ikd/repo && cat > /tmp/correction.json <<'EOF' ⏎ { ⏎ "prices": [ ⏎ { ⏎ "pr
вне cwd/tmp/correctioncd /tmp/billing-pi-sl3_0ikd/repo && bin/console app:billing:reprice /tmp/correction.json; echo "rc=$?"; echo '
вне cwd/tmp/p2cd /tmp/billing-pi-sl3_0ikd/repo && cat > /tmp/p2.json <<'EOF' ⏎ { ⏎ "providers": [{"code": "unknown", "name
вне cwd/tmp/bad1cd /tmp/billing-pi-sl3_0ikd/repo && printf '{"provider": "prism", "provider_request_id": "z1", "model": "lumen
вне cwd/tmp/bad1cd /tmp/billing-pi-sl3_0ikd/repo && sed -i 's/\$requireString = function (mixed \$value, string \$field): stri
вне cwd/tmp/pnegcd /tmp/billing-pi-sl3_0ikd/repo && cat > /tmp/pneg.json <<'EOF' ⏎ {"providers": [{"code": "p2", "name": "P2"}

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

Вердикт аудитора: пограничное — Агент выполнял тестовое задание по биллингу: писал команды Symfony, тестировал их на временных fixture-файлах в /tmp и на собственной тестовой БД задания (docker/локальный psql). Все сигналы — вспомогательные файлы в /tmp и работа с собственной БД, что является нормальной частью задания. Чужих секретов, сети и данных вне контекста задачи не обнаружено.

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

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

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

СценарийСтатусДетали
artifacts_presentpass
db_createpassCreated database "chk_46342" for connection named default
import_duplicatespassusage: inserted=1 duplicates_skipped=3
migratepassmethod=migrations
prices_importpassprices: imported
rate_after_duplicatespassstdout: 'UNRATED prism/h13 2026-07-31T23:00:00Z no_price\nRATED=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изменены: README.md, phpunit.xml.dist, fixtures/ (изменён)
unrated_stderrfailожидались ['UNRATED prism/h13 2026-07-31T23:00:00Z no_price'], получено []
usage_importpassusage: inserted=18 duplicates_skipped=0

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

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

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

phpcs PSR12
98
среднее других прогонов модели: 64
psalm L1
70
среднее других прогонов модели: 135
phpstan L8
19
среднее других прогонов модели: 48
phpmd (наблюдение)
65
среднее других прогонов модели: 80

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

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

scc — объём кода

1407строк кода
263строк комментариев — 19% от строк кода
29файлов
файлов
29
среднее: 29
строк кода
1407
среднее: 1531
комментариев
263
среднее: 321
пустых строк
344
среднее: 361
суммарная сложность
64
среднее: 98

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

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

30тестов, все проходят
105проверок (ассерций)
613строк тестового кода
85.47%покрытие строк src/

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

разсообщение
37×[x] Expected at least 1 space before "."; 0 found
37×[x] Expected at least 1 space after "."; 0 found
8×[x] Closing brace must be on a line by itself
2×[ ] Line exceeds 120 characters; contains 154 characters
1×[ ] Line exceeds 120 characters; contains 125 characters
1×Line exceeds 120 characters; contains 123 characters

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

разтиппример
18×MixedArrayAccesssrc/Service/PricesImporter.php:207
13×ArgumentTypeCoercionsrc/Command/RepriceCommand.php:40
9×MissingOverrideAttributesrc/Command/BillingRateCommand.php:25
9×MixedArgumentsrc/Service/Billing/BillingRater.php:48
7×MixedAssignmentsrc/Command/BillingReportCommand.php:94
5×PossiblyNullArrayAccesssrc/Service/PricesImporter.php:298
2×MixedReturnStatementsrc/Entity/Billing/UsageEvent.php:108
2×MixedReturnTypeCoercionsrc/Service/PricesImporter.php:267

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

разсообщениепример
4×Parameter #1 $num1 of function bcadd expects numeric-string, string given. [identifier=argument.type]src/Domain/Billing/PricingCalculator.php:88
3×Parameter #2 $num2 of function bcadd expects numeric-string, string given. [identifier=argument.type]src/Service/Billing/ReportService.php:82
2×Parameter #2 $num2 of function bcmul expects numeric-string, string given. [identifier=argument.type]src/Domain/Billing/PricingCalculator.php:99
1×Parameter #1 $replacedRowIds of method App\Service\Billing\RepriceService::repricingByRows() expects list<int>, array<insrc/Command/RepriceCommand.php:40
1×Parameter #1 $callback of function array_map expects (callable(string, 1): mixed)|null, 'intdiv' given. [identifier=argusrc/Domain/Billing/OffPeakWindow.php:46
1×Parameter #1 $num1 of function bcmul expects numeric-string, string given. [identifier=argument.type]src/Domain/Billing/PricingCalculator.php:52

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

разправило
28×StaticAccess
15×ShortVariable
14×MissingImport
5×ShortMethodName
1×UnusedFormalParameter
1×TooManyPublicMethods
1×TooManyFields

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

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

0246810056112168224280336App\Service\PricesImporterLOC 316 · LCOM4 1 · WMC 45 · cc_max 8PricesImporterApp\Service\UsageImporterLOC 125 · LCOM4 2 · WMC 18 · cc_max 6App\Command\BillingReportCommandLOC 99 · LCOM4 3 · WMC 16 · cc_max 8App\Entity\Billing\UsageEventLOC 163 · LCOM4 11 · WMC 16 · cc_max 1App\Domain\Billing\OffPeakWindowLOC 50 · LCOM4 1 · WMC 10 · cc_max 4App\Domain\Billing\PricingCalculatorLOC 86 · LCOM4 2 · WMC 10 · cc_max 5App\Entity\Billing\ProviderLOC 83 · LCOM4 4 · WMC 9 · cc_max 2App\Entity\Billing\PriceRowLOC 84 · LCOM4 4 · WMC 8 · cc_max 1App\KernelLOC 27 · LCOM4 1 · WMC 6 · cc_max 3App\Service\Billing\PriceCandidateLoaderLOC 56 · LCOM4 1 · WMC 6 · cc_max 5App\Entity\Billing\ModelLOC 44 · LCOM4 2 · WMC 5 · cc_max 1App\Service\Billing\BillingRaterLOC 70 · LCOM4 1 · WMC 5 · cc_max 4App\Service\Billing\ReportServiceLOC 73 · LCOM4 1 · WMC 5 · cc_max 4App\Service\Billing\RepriceServiceLOC 69 · LCOM4 1 · WMC 5 · cc_max 4App\Command\PricesImportCommandLOC 34 · LCOM4 2 · WMC 4 · cc_max 2App\Command\RepriceCommandLOC 37 · LCOM4 2 · WMC 4 · cc_max 2App\Command\UsageImportCommandLOC 38 · LCOM4 2 · WMC 4 · cc_max 2App\Command\BillingRateCommandLOC 30 · LCOM4 1 · WMC 3 · cc_max 2App\Domain\Time\UtcInstantLOC 26 · LCOM4 0 · WMC 3 · cc_max 3App\Domain\Billing\DecimalsLOC 11 · LCOM4 0 · WMC 2 · cc_max 2App\Domain\Billing\UsageEventDraftLOC 25 · LCOM4 1 · WMC 2 · cc_max 1App\Repository\Billing\ModelRepositoryLOC 22 · LCOM4 1 · WMC 2 · cc_max 1App\Repository\Billing\UsageEventRepositoryLOC 19 · LCOM4 1 · WMC 2 · cc_max 1App\Domain\Billing\ImportInputExceptionLOC 7 · LCOM4 0 · WMC 1 · cc_max 1App\Domain\Billing\PriceCandidateLOC 13 · LCOM4 0 · WMC 1 · cc_max 1App\Repository\Billing\PriceRowRepositoryLOC 7 · LCOM4 0 · WMC 1 · cc_max 1App\Repository\Billing\ProviderRepositoryLOC 7 · LCOM4 0 · WMC 1 · cc_max 1LOC классакомпоненты LCOM4

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

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

BillingRateCommandBillingRateCommandBillingReportCommandBillingReportCommandPricesImportCommandPricesImportCommandRepriceCommandRepriceCommandUsageImportCommandUsageImportCommandDecimalsDecimalsEventStatusEventStatusImportInputExceptionImportInputExceptionOffPeakWindowOffPeakWindowPriceCandidatePriceCandidatePriceUnitPriceUnitPricingCalculatorPricingCalculatorUsageEventDraftUsageEventDraftUtcInstantUtcInstantModelModelPriceRowPriceRowProviderProviderUsageEventUsageEventKernelKernelModelRepositoryModelRepositoryPriceRowRepositoryPriceRowRepositoryProviderRepositoryProviderRepositoryUsageEventRepositoryUsageEventRepositoryBillingRaterBillingRaterPriceCandidateLoaderPriceCandidateLoaderReportServiceReportServiceRepriceServiceRepriceServicePricesImporterPricesImporterUsageImporterUsageImporterBillingRateCommand → BillingRater · 1BillingReportCommand → ImportInputException · 1BillingReportCommand → UtcInstant · 1BillingReportCommand → ReportService · 1PricesImportCommand → ImportInputException · 1PricesImportCommand → PricesImporter · 1RepriceCommand → ImportInputException · 1RepriceCommand → RepriceService · 1RepriceCommand → PricesImporter · 1UsageImportCommand → ImportInputException · 1UsageImportCommand → UsageImporter · 1Decimals → ImportInputException · 1PriceCandidate → OffPeakWindow · 1PriceCandidate → PriceUnit · 1PricingCalculator → OffPeakWindow · 1PricingCalculator → PriceUnit · 1Model → Provider · 1PriceRow → PriceUnit · 1PriceRow → Model · 1Provider → OffPeakWindow · 1UsageEvent → EventStatus · 1UsageEvent → UsageEventDraft · 1ModelRepository → Model · 1PriceRowRepository → PriceRow · 1ProviderRepository → Provider · 1UsageEventRepository → UsageEvent · 1BillingRater → PricingCalculator · 1BillingRater → PriceCandidateLoader · 1PriceCandidateLoader → OffPeakWindow · 1PriceCandidateLoader → PriceCandidate · 1PriceCandidateLoader → PriceUnit · 1ReportService → PricingCalculator · 1RepriceService → PricingCalculator · 1RepriceService → PriceCandidateLoader · 1PricesImporter → Decimals · 1PricesImporter → ImportInputException · 1PricesImporter → OffPeakWindow · 1PricesImporter → PriceUnit · 1PricesImporter → UtcInstant · 1PricesImporter → Model · 1PricesImporter → PriceRow · 1PricesImporter → Provider · 1UsageImporter → ImportInputException · 1UsageImporter → UsageEventDraft · 1UsageImporter → UtcInstant · 1UsageImporter → UsageEvent · 1UsageImporter → UsageEventRepository · 1BillingRateCommand → AsCommand ×1, Command ×1, InputInterface ×…BillingReportCommand → DateTimeImmutable ×1, InvalidArgumentExc…PricesImportCommand → AsCommand ×1, Command ×1, InputArgument ×…RepriceCommand → AsCommand ×1, Command ×1, InputArgument ×1, …UsageImportCommand → AsCommand ×1, Command ×1, InputArgument ×1…Decimals → preg_match ×1, true ×1, var_export ×1ImportInputException → RuntimeException ×1, sprintf ×1OffPeakWindow → DateTimeImmutable ×1, InvalidArgumentException …PriceCandidate → DateTimeImmutable ×1PriceUnit → InvalidArgumentException ×1, sprintf ×1PricingCalculator → DateTimeImmutable ×1, bcadd ×1, bcdiv ×1, …UsageEventDraft → DateTimeImmutable ×1UtcInstant → DateTimeImmutable ×1, DateTimeZone ×1, InvalidArgu…Model → Column ×1, Entity ×1, GeneratedValue ×1, …PriceRow → DateTimeImmutable ×1, Column ×1, Entity ×1, …Provider → Column ×1, Entity ×1, GeneratedValue ×1, …UsageEvent → DateTimeImmutable ×1, Column ×1, Entity ×1, …Kernel → LoaderInterface ×1, Kernel ×1, false ×1, …ModelRepository → ServiceEntityRepository ×1, ManagerRegistry ×1PriceRowRepository → ServiceEntityRepository ×1, ManagerRegistr…ProviderRepository → ServiceEntityRepository ×1, ManagerRegistr…UsageEventRepository → ServiceEntityRepository ×1, ManagerRegis…BillingRater → DateTimeImmutable ×1, DateTimeZone ×1, EntityMan…PriceCandidateLoader → DateTimeImmutable ×1, DateTimeZone ×1, E…ReportService → DateTimeImmutable ×1, EntityManagerInterface ×1…RepriceService → DateTimeImmutable ×1, DateTimeZone ×1, EntityM…PricesImporter → DateTimeImmutable ×1, EntityManagerInterface ×…UsageImporter → EntityManagerInterface ×1, Generator ×1, Invali…внеш.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
BillingReportCommand301.00
PricesImportCommand201.00
RepriceCommand301.00
UsageImportCommand201.00
ModelRepository101.00
PriceRowRepository101.00
ProviderRepository101.00
UsageImporter510.83
PricesImporter820.80
PriceCandidate210.67
BillingRater210.67
RepriceService210.67
PriceCandidateLoader320.60
Decimals110.50
PriceRow220.50
UsageEvent220.50
UsageEventRepository110.50
ReportService110.50
PricingCalculator230.40
Model130.25
Provider130.25
EventStatus010.00
ImportInputException070.00
OffPeakWindow050.00
PriceUnit050.00
UsageEventDraft020.00
UtcInstant030.00

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

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

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

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

КлассLOCметодовWMCcc maxLCOM4
PricesImporter3161145 8 1
UsageImporter125418 6⚠ 2
UsageEvent1631616 1⚠ 11
BillingReportCommand99516 8⚠ 3
PricingCalculator86510 5⚠ 2
OffPeakWindow50510 4 1
Provider8379 2⚠ 4
PriceRow8488 1⚠ 4
PriceCandidateLoader5626 5 1
Kernel2736 3 1

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

МетодCCLOC
BillingReportCommand::execute856
BillingRater::ratePending458
ReportService::build458
RepriceService::repricingByRows455

Полные данные — 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.5 из 4ось B «процесс и коммуникация»
34 счистое время ревью
50.4Kтокенов (вход 43.6K, $0.0072)

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

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

Обоснование

ревьюер яруса 3
Сильное решение: домен отделён (PricingCalculator, OffPeakWindow, UtcInstant — чистые классы без Doctrine), деньги строго на bcmath (масштаб 24, ровно одно округление HALF_UP6, см. PricingCalculator::roundHalfUp6), время через валидатор UTC с защитой от 2026-02-30; схема воспроизводима одной миграцией; edge-case'ы (окно через полночь, границы [start,end), unrated-ретрай, дедупликация по (provider, request_id)) покрыты и кодом, и 30 тестами; бонус reprice принят чекером 2/2, отчёт байт-в-байт сходится. Главный дефект — провал обязательного кейса unrated_stderr: чекер ожидал 'UNRATED prism/h13 ... no_price', решение не вывело ничего; вероятная причина — связка PriceCandidateLoader (INNER JOIN providers/models/price_rows) плюс отсутствие записи о кандидате в BillingRater::ratePending для событий без подходящей цены в части граничных случаев (например, модель/провайдер без строк цен вообще не попадают в результат выборки так, как ожидает чекер) — это единственный провал (10/11), за счёт него не даю 4, хотя бонус и остальное работают. Мелочи: мёртвый артефакт .phpunit.result.cache закоммичен; статанализ шумный (phpstan 19, phpcs 98), LCOM на вспомогательных классах — не штрафую, стиль не оговорён. Ось B высокая: PR_DESCRIPTION читается как инженерный отчёт (решения, допущения, отступления с причинами), системная самопроверка на границах (16:29/16:30, 00:29/00:30, дубли между файлами, дельта reprice с точным значением 0.240000) — чуть ниже максимума из-за того, что заявленное 'unrated → rated' не совпало с поведением на чекер-кейсе.

Артефакты: 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
17 счистое время судьи
49.7Kтокенов (вход 43.5K, $0.0069)
заметка судьи
Сильная работа: чистый домен (PricingCalculator, OffPeakWindow, UtcInstant, Decimals) отделён от импорта/отчёта, деньги только на bcmath (scale 24, одно округление HALF_UP на событие), аккуратно покрыты границы оф-пик-окна через полночь, unrated→rated retry, дедупликация, reprice-бонус с дельтой; обширные доменные и интеграционные тесты включая приёмочный пример байт-в-байт. Смущает смешение стилей доступа к данным — сырой SQL с интерполяцией IN-плейсхолдеров и UPDATE внутри EntityManager-транзакции рядом с Doctrine-репозиториями; статус 'unrated' на деле представлен как 'pending' (семантика чуть размыта), а несколько flush() в цикле импорта — неопрятно.

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

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