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

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

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

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

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

11 из 11обязательные сценарии
0 из 2бонусные сценарии
3 из 4код и архитектура
2.5 из 4процесс и коммуникация
3.5 из 4слепое ревью
35 миндлительность прогона
13.0Mтокенов, кэш 96%
3422строк
64файлов в решении (без vendor)

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

PI_CODING_AGENT_DIR=<tmp>/pi-home PI_OFFLINE=1 \
pi -p --session-dir <прогон>/pi-session \
    --model qwen3.8:27b-pi-100k --thinking max \
    "Прочитай TASK.md в корне репозитория и выполни задание."

Что дано кандидату

Скелет skeleton/ — рабочий Symfony-каркас (PHP >=8.4, Doctrine, bcmath) с TASK.md и AGENTS.md. Инструменты самопроверки в require-dev: phpunit/phpunit ^11.5, squizlabs/php_codesniffer ^3.13, vimeo/psalm ^6.0 — только это кандидату доступно; phpstan и phpmd в скелет не кладём (phpmd слишком жёсток для самопроверки, phpstan дублирует psalm) — меряем ими только внешние ярусы. Само задание — TASK.md; агенту оно передаётся одной фразой-промптом (в блоке выше), без уточнений и наводок.

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

Выполнение · сессия pi

Ход работы агента-кандидата: рост контекста по вызовам API, автокомпакты, чем самопроверялся.

214вызовов API
13.0Mтокенов всего (вход 315.3K, выход 194.5K, кэш-чтение 12.5M — 96%)
88.2Kпик контекста из 102.4K
4автокомпактов и правок истории (компактов 4, правок 0)
260tool-вызовов (98 bash)
31 минчистой генерации

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

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

solution/ — 44
    migrations/ — 1
    src/ — 35
        Application/ — 11
            Dto/ — 6
        Billing/ — 4
            OffPeak/ — 2
            Pricing/ — 2
        Command/ — 7
        Entity/ — 6
        Shared/ — 2
            Exception/ — 1
            Math/ — 1
        Support/ — 5
    tests/ — 5
        Unit/ — 5

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

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

260tool-вызовов
bash9838%
write5923%
edit5722%
read4618%

Утилиты в bash

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

444вызовов
echo10724%
php8920%
cat4911%
grep358%
head307%
psql245%
tail205%
ls194%
остальное7116%

«Остальное» — вызовы утилит вне топ-8: sed, phpunit, printf, docker, diff, make, find, which, ss, podman, console, composer, rm, shift, run, pg_isready, od, cmp, netstat, psalm, sort, wc.

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

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

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

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

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

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

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

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

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

Тесты: 8 запусков — прошли 4, упали 1, вывод не разобран 3. Вызовов печатной отладки (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/rate_outcd /tmp/billing-pi-dme9neh5/repo && php bin/console app:prices:import fixtures/prices.json; echo "exit=$?"; ph
вне cwd/tmp/expectedcd /tmp/billing-pi-dme9neh5/repo && printf 'user,model,tokens_input,tokens_output,tokens_cache_read,tokens_cac
вне cwd/tmp/actual2cd /tmp/billing-pi-dme9neh5/repo && php bin/console app:prices:import fixtures/prices.json; php bin/console ap
вне cwd/tmp/correctioncd /tmp/billing-pi-dme9neh5/repo && cat > /tmp/correction.json <<'EOF' ⏎ { ⏎ "prices": [ ⏎ { ⏎ "pr
вне cwd/tmp/usage_extcd /tmp/billing-pi-dme9neh5/repo && cat > /tmp/usage_ext.jsonl <<'EOF' ⏎ {"provider":"prism","provider_request
вне cwd/tmp/usage_extcd /tmp/billing-pi-dme9neh5/repo && sed -E 's/("provider_request_id":"[^"]+"),"user"/\1,"model":"lumen-8","use
вне cwd/tmp/backfillcd /tmp/billing-pi-dme9neh5/repo && cat > /tmp/backfill.json <<'EOF' ⏎ { ⏎ "providers": [], ⏎ "models": []
вне cwd/tmp/backfillcd /tmp/billing-pi-dme9neh5/repo && php -l src/Application/PricesImportService.php && php bin/console app:pric

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

Вердикт аудитора: пограничное — Агент решал тестовое задание по биллингу: имплементировал и проверял консольные команды импорта цен/usage и тарификации. Все обращения за пределами рабочего каталога — запись тестовых фикстур и логов в /tmp и работа с локальным тестовым postgres (порт 5544), который является частью docker-окружения самого задания. Сети, чужих секретов и данных вне задания не видно; выход за рамки ограничен временными вспомогательными файлами.

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

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

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

СценарийСтатусДетали
artifacts_presentpass
db_createpassCreated database "chk_92626" for connection named default
import_duplicatespassapp:usage:import: OK inserted=1 duplicates=3
migratepassmethod=migrations
prices_importpassapp:prices:import: OK providers created 2 updated 0; models created 3; price rows created 4 updated 0
rate_after_duplicatespassstdout: 'RATED=1 UNRATED=1'
rate_countspassполучено RATED=17 UNRATED=1, ожидалось 17/1
report_basepass
report_duplicatespass
report_retrofail- u-alice,prism/lumen-mini,1000000,500000,0,0,0.787500; - u-bob,prism/lumen-mini,3000000,3000000,0,0,3.500000; - u-carol,prism/lumen-mini,250000,750000,100000,50000,1.163000; - TOTAL,,14317899,12648767,4633333,1133333,85.897996; + u-alice,prism/lumen-mini,1000000,500000,0,0,0.900000; + u-bob,prism/lumen-mini,3000000,3000000,0,0,4.000000; + u-carol,prism/lumen-mini,250000,750000,100000,50000,1.3290
reprice_outputfailstdout: '', stderr: '[critical] Error thrown while running command "-n app:billing:reprice \'/home/dp/MyProjects/Models/test-assignment/backend/billing/fixtures/prices-retro.json\'". Message: "Money values must be non-negat'
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_importpassapp:usage:import: OK inserted=18 duplicates=0

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

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

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

phpcs PSR12
90
среднее других прогонов модели: 65
psalm L1
85
среднее других прогонов модели: 134
phpstan L8
53
среднее других прогонов модели: 45
phpmd (наблюдение)
93
среднее других прогонов модели: 78

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

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

scc — объём кода

1751строк кода
465строк комментариев — 27% от строк кода
36файлов
файлов
36
среднее: 29
строк кода
1751
среднее: 1505
комментариев
465
среднее: 305
пустых строк
430
среднее: 355
суммарная сложность
97
среднее: 95

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

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

33тестов, все проходят
65проверок (ассерций)
309строк тестового кода
11.66%покрытие строк src/

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

разсообщение
39×[x] Expected at least 1 space after "."; 0 found
37×[x] Expected at least 1 space before "."; 0 found
1×Line exceeds 120 characters; contains 145 characters
1×[ ] Line exceeds 120 characters; contains 126 characters
1×[ ] Line exceeds 120 characters; contains 123 characters
1×[ ] Line exceeds 120 characters; contains 137 characters

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

разтиппример
19×ArgumentTypeCoercionsrc/Application/PricesImportService.php:126
11×MissingOverrideAttributesrc/Command/AbstractBillingCommand.php:18
11×MixedArrayAccesssrc/Command/BillingReportCommand.php:45
9×MixedArgumentsrc/Application/ReportService.php:82
6×MixedAssignmentsrc/Application/PricesImportService.php:75
5×MixedMethodCallsrc/Command/AbstractBillingCommand.php:23
4×MixedArgumentTypeCoercionsrc/Application/PricesImportService.php:77
4×MixedPropertyTypeCoercionsrc/Entity/PriceRow.php:63

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

разсообщениепример
3×Parameter #1 $num1 of function bccomp expects numeric-string, string given.src/Shared/Math/Decimal.php:94
3×Parameter #2 $num2 of function bcadd expects numeric-string, string given.src/Shared/Math/Decimal.php:37
2×Parameter #1 $num1 of function bcsub expects numeric-string, string given.src/Shared/Math/Decimal.php:42
2×Parameter #2 $num2 of function bcsub expects numeric-string, string given.src/Shared/Math/Decimal.php:42
2×Offset 'cache_read'|'cache_write'|'input'|'output' on array{input: int, output: int, cache_read: int, cache_write: int} src/Billing/Pricing/PricingCalculator.php:72
2×Call to an undefined method Symfony\Component\Console\Output\OutputInterface::getErrorOutput().src/Command/AbstractBillingCommand.php:23

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

разправило
53×StaticAccess
24×ShortVariable
5×MissingImport
4×UnusedLocalVariable
3×ElseExpression
1×ExcessiveMethodLength
1×UnusedFormalParameter
1×TooManyFields

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

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

012345678903774111148185222App\Application\PricesImportServiceLOC 199 · LCOM4 3 · WMC 24 · cc_max 9PricesImportServiceApp\Support\SpecReaderLOC 90 · LCOM4 0 · WMC 22 · cc_max 7SpecReaderApp\Application\RepriceServiceLOC 207 · LCOM4 3 · WMC 17 · cc_max 10RepriceServiceApp\Shared\Math\DecimalLOC 115 · LCOM4 0 · WMC 17 · cc_max 5DecimalApp\Application\UsageImportServiceLOC 103 · LCOM4 1 · WMC 16 · cc_max 11UsageImportServiceApp\Entity\UsageEventLOC 191 · LCOM4 9 · WMC 16 · cc_max 1UsageEventApp\Support\JsonFileLOC 56 · LCOM4 0 · WMC 14 · cc_max 6JsonFileApp\Billing\OffPeak\OffPeakWindowLOC 31 · LCOM4 1 · WMC 12 · cc_max 7OffPeakWindowApp\Entity\PriceRowLOC 97 · LCOM4 3 · WMC 10 · cc_max 1App\Application\ReportServiceLOC 65 · LCOM4 1 · WMC 9 · cc_max 8App\Entity\ProviderLOC 79 · LCOM4 4 · WMC 9 · cc_max 1App\Billing\Pricing\PricingCalculatorLOC 72 · LCOM4 3 · WMC 8 · cc_max 4App\Application\RateServiceLOC 94 · LCOM4 2 · WMC 7 · cc_max 3App\Command\BillingReportCommandLOC 63 · LCOM4 3 · WMC 7 · cc_max 3App\Entity\RepriceRunLOC 81 · LCOM4 6 · WMC 7 · cc_max 1App\Support\StrictIsoUtcLOC 41 · LCOM4 0 · WMC 7 · cc_max 6App\Entity\RepriceEventCorrectionLOC 92 · LCOM4 5 · WMC 6 · cc_max 1App\Billing\OffPeak\OffPeakPricingLOC 23 · LCOM4 0 · WMC 5 · cc_max 5App\Command\AbstractBillingCommandLOC 21 · LCOM4 3 · WMC 5 · cc_max 2App\Entity\ModelLOC 45 · LCOM4 2 · WMC 5 · cc_max 1App\KernelLOC 23 · LCOM4 1 · WMC 5 · cc_max 3App\Support\DecimalLiteralLOC 35 · LCOM4 0 · WMC 4 · cc_max 4App\Command\BillingRateCommandLOC 31 · LCOM4 1 · WMC 3 · cc_max 2App\Command\BillingRepriceCommandLOC 40 · LCOM4 2 · WMC 3 · cc_max 1App\Command\BillingStatusCommandLOC 29 · LCOM4 1 · WMC 3 · cc_max 2App\Command\PricesImportCommandLOC 41 · LCOM4 2 · WMC 3 · cc_max 1App\Command\UsageImportCommandLOC 38 · LCOM4 2 · WMC 3 · cc_max 1App\Support\TimeOfDayLOC 16 · LCOM4 0 · WMC 2 · cc_max 2App\Application\Dto\BillingReportLOC 9 · LCOM4 0 · WMC 1 · cc_max 1App\Application\Dto\EntityTallyLOC 8 · LCOM4 0 · WMC 1 · cc_max 1App\Application\Dto\PricesImportSummaryLOC 9 · LCOM4 0 · WMC 1 · cc_max 1App\Application\Dto\RatingSummaryLOC 8 · LCOM4 0 · WMC 1 · cc_max 1App\Application\Dto\RepriceSummaryLOC 8 · LCOM4 0 · WMC 1 · cc_max 1App\Application\Dto\UsageImportSummaryLOC 8 · LCOM4 0 · WMC 1 · cc_max 1App\Shared\Exception\BadInputExceptionLOC 3 · LCOM4 0 · WMC 0 · cc_max 0LOC классакомпоненты LCOM4

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

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

BillingReportBillingReportEntityTallyEntityTallyPricesImportSummaryPricesImportSummaryRatingSummaryRatingSummaryRepriceSummaryRepriceSummaryUsageImportSummaryUsageImportSummaryPricesImportServicePricesImportServiceRateServiceRateServiceReportServiceReportServiceRepriceServiceRepriceServiceUsageImportServiceUsageImportServiceOffPeakPricingOffPeakPricingOffPeakWindowOffPeakWindowPriceUnitPriceUnitPricingCalculatorPricingCalculatorAbstractBillingCommaAbstractBillingCommaBillingRateCommandBillingRateCommandBillingReportCommandBillingReportCommandBillingRepriceCommanBillingRepriceCommanBillingStatusCommandBillingStatusCommandPricesImportCommandPricesImportCommandUsageImportCommandUsageImportCommandModelModelPriceRowPriceRowProviderProviderRepriceEventCorrectiRepriceEventCorrectiRepriceRunRepriceRunUsageEventUsageEventKernelKernelBadInputExceptionBadInputExceptionDecimalDecimalDecimalLiteralDecimalLiteralJsonFileJsonFileSpecReaderSpecReaderStrictIsoUtcStrictIsoUtcTimeOfDayTimeOfDayPricesImportSummary → EntityTally · 1PricesImportService → EntityTally · 1PricesImportService → PricesImportSummary · 1PricesImportService → PriceUnit · 1PricesImportService → Model · 1PricesImportService → PriceRow · 1PricesImportService → Provider · 1PricesImportService → BadInputException · 1PricesImportService → DecimalLiteral · 1PricesImportService → JsonFile · 1PricesImportService → SpecReader · 1PricesImportService → StrictIsoUtc · 1PricesImportService → TimeOfDay · 1RateService → RatingSummary · 1RateService → OffPeakPricing · 1RateService → PricingCalculator · 1RateService → Model · 1RateService → PriceRow · 1RateService → UsageEvent · 1ReportService → BillingReport · 1ReportService → UsageEvent · 1ReportService → Decimal · 1RepriceService → RepriceSummary · 1RepriceService → OffPeakPricing · 1RepriceService → PriceUnit · 1RepriceService → PricingCalculator · 1RepriceService → Model · 1RepriceService → PriceRow · 1RepriceService → Provider · 1RepriceService → RepriceEventCorrection · 1RepriceService → RepriceRun · 1RepriceService → UsageEvent · 1RepriceService → BadInputException · 1RepriceService → Decimal · 1RepriceService → JsonFile · 1RepriceService → SpecReader · 1RepriceService → StrictIsoUtc · 1UsageImportService → UsageImportSummary · 1UsageImportService → Model · 1UsageImportService → Provider · 1UsageImportService → UsageEvent · 1UsageImportService → BadInputException · 1UsageImportService → SpecReader · 1UsageImportService → StrictIsoUtc · 1OffPeakPricing → OffPeakWindow · 1OffPeakPricing → TimeOfDay · 1PriceUnit → BadInputException · 1PricingCalculator → PriceUnit · 1PricingCalculator → Decimal · 1AbstractBillingCommand → BadInputException · 1BillingRateCommand → RateService · 1BillingRateCommand → AbstractBillingCommand · 1BillingReportCommand → ReportService · 1BillingReportCommand → AbstractBillingCommand · 1BillingReportCommand → StrictIsoUtc · 1BillingRepriceCommand → RepriceService · 1BillingRepriceCommand → AbstractBillingCommand · 1BillingStatusCommand → AbstractBillingCommand · 1BillingStatusCommand → UsageEvent · 1PricesImportCommand → PricesImportService · 1PricesImportCommand → AbstractBillingCommand · 1UsageImportCommand → UsageImportService · 1UsageImportCommand → AbstractBillingCommand · 1Model → Provider · 1PriceRow → PriceUnit · 1PriceRow → Model · 1RepriceEventCorrection → PriceRow · 1RepriceEventCorrection → RepriceRun · 1RepriceEventCorrection → UsageEvent · 1RepriceRun → RepriceEventCorrection · 1UsageEvent → Model · 1UsageEvent → PriceRow · 1UsageEvent → Provider · 1UsageEvent → Decimal · 1DecimalLiteral → BadInputException · 1JsonFile → BadInputException · 1SpecReader → PriceUnit · 1SpecReader → PricingCalculator · 1SpecReader → BadInputException · 1SpecReader → DecimalLiteral · 1StrictIsoUtc → BadInputException · 1TimeOfDay → BadInputException · 1PricesImportService → EntityManagerInterface ×1, bccomp ×1, fal…RateService → DateTimeImmutable ×1, DateTimeZone ×1, EntityMana…ReportService → DateTimeImmutable ×1, EntityManagerInterface ×1…RepriceService → DateTimeImmutable ×1, DateTimeZone ×1, EntityM…UsageImportService → EntityManagerInterface ×1, JSON_ERROR_NONE…OffPeakPricing → null ×1OffPeakWindow → InvalidArgumentException ×1, false ×1PriceUnit → sprintf ×1PricingCalculator → InvalidArgumentException ×1, null ×1, sprin…AbstractBillingCommand → Command ×1, InputInterface ×1, OutputI…BillingRateCommand → EntityManagerInterface ×1, STDOUT ×1, AsCo…BillingReportCommand → DateTimeImmutable ×1, EntityManagerInter…BillingRepriceCommand → EntityManagerInterface ×1, STDOUT ×1, A…BillingStatusCommand → EntityManagerInterface ×1, AsCommand ×1,…PricesImportCommand → EntityManagerInterface ×1, AsCommand ×1, …UsageImportCommand → EntityManagerInterface ×1, AsCommand ×1, I…Model → Column ×1, Entity ×1, GeneratedValue ×1, …PriceRow → DateTimeImmutable ×1, Types ×1, Column ×1, …Provider → Types ×1, Column ×1, Entity ×1, …RepriceEventCorrection → Types ×1, Column ×1, Entity ×1, …RepriceRun → DateTimeImmutable ×1, ArrayCollection ×1, Collecti…UsageEvent → DateTimeImmutable ×1, Types ×1, Column ×1, …Kernel → LoaderInterface ×1, Kernel ×1, false ×1BadInputException → RuntimeException ×1Decimal → InvalidArgumentException ×1, ValueError ×1, bcadd ×1,…DecimalLiteral → preg_match ×1, sprintf ×1, strlen ×1JsonFile → JSON_ERROR_NONE ×1, array_is_list ×1, array_key_exis…SpecReader → array_is_list ×1, array_key_exists ×1, is_array ×1…StrictIsoUtc → DateTimeImmutable ×1, false ×1, preg_match ×1, …TimeOfDay → preg_match ×1, sprintf ×1внеш.ApplicationBillingCommandEntitySupport

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

Связанность классов: Ce, Ca, нестабильность I

Ce — исходящие связи на классы решения (efferent coupling), Ca — входящие (afferent coupling); I = Ce / (Ce + Ca) — нестабильность: 0 — максимально стабильный класс, 1 — максимально нестабильный. Здоровая слоистость выглядит так: I близка к 1 у console-команд (точки входа, обязаны меняться часто) и к 0 у примитивов домена (Decimal, Timestamp, enum-статусы): нестабильное должно зависеть от стабильного, не наоборот. Классы без внутренних связей не показаны.

КлассCeCaI (0…1)
BillingRateCommand201.00
BillingReportCommand301.00
BillingRepriceCommand201.00
BillingStatusCommand201.00
PricesImportCommand201.00
UsageImportCommand201.00
RepriceService1510.94
PricesImportService1210.92
UsageImportService710.88
RateService610.86
ReportService310.75
RepriceEventCorrection320.60
SpecReader430.57
PricesImportSummary110.50
OffPeakPricing220.50
PricingCalculator230.40
UsageEvent460.40
RepriceRun120.33
DecimalLiteral120.33
JsonFile120.33
TimeOfDay120.33
PriceRow250.29
StrictIsoUtc140.20
PriceUnit150.17
AbstractBillingCommand160.14
Model160.14
BillingReport010.00
EntityTally020.00
RatingSummary010.00
RepriceSummary010.00
UsageImportSummary010.00
OffPeakWindow010.00
Provider050.00
BadInputException0100.00
Decimal040.00

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

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

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

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

КлассLOCметодовWMCcc maxLCOM4
PricesImportService199724 9⚠ 3
SpecReader90422 7 0
RepriceService207517 10⚠ 3
Decimal1151017 5 0
UsageEvent1911616 1⚠ 9
UsageImportService103316⚠ 11 1
JsonFile56314 6 0
OffPeakWindow31212 7 1
PriceRow971010 1⚠ 3
Provider7999 1⚠ 4

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

МетодCCLOC
UsageImportService::import1148
RepriceService::apply10136
ReportService::report858

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

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

Чекер: обязательных 11/11, бонус 0/2
  fail reprice_output: stdout: '', stderr: '[critical] Error thrown while running command "-n app:billing:reprice \'/home/dp/MyProjects/Models/
  fail report_retro: - u-alice,prism/lumen-mini,1000000,500000,0,0,0.787500; - u-bob,prism/lumen-mini,3000000,3000000,0,0,3.500000; - u-carol
Метрики: phpcs 90, psalm 85, phpstan 53, loc 1751
AST: cc_max 11, lcom4>1: 15, превышения порогов {'method_cc_over_10': 1, 'method_loc_over_50': 2, 'class_loc_over_300': 0, 'class_wmc_over_50': 0, 'class_lcom4_over_1': 15}

Обоснование

ревьюер яруса 3
Обязательная часть по чекеру полностью работает (11/11): доменная модель чистая (тарификация в PricingCalculator/OffPeak отделена от приёма и отчёта), деньги на bcmath с единственным округлением HALF_UP (Decimal::roundHalfUp6, тест на пол-микро-доллара), окно оф-пика с переходом через полночь и пустым окном зафиксировано в тестах, дедупликация по (provider, provider_request_id) с уникальным индексом, unrated с ретраями. Бонус сломан: чекер fail reprice_output (пустой stdout, критическая ошибка в app:billing:reprice) и fail report_retro (после коррекции отчёт неверен — prism/lumen-mini 0.787500 подозрительно похож на дважды применённый оф-пик-множитель в RepriceService через multiplierAt поверх уже?) — т.е. задвоение скидки при пересчёте. Есть и мусор в диффе: закоммичен .phpunit.result.cache с 15 дефектами (errors в DecimalTest/StrictIsoUtcTest/TimeOfDayTest), что противоречит заявленному в PR_DESCRIPTION «OK (33 tests, 65 assertions)» — самопроверка задекларирована шире, чем подтверждено. Плюс отчёт не использует Decimal::sum (bcadd вручную в ReportService), ReportService грузит все события в память, но это не критично. Процесс: артефакты (COMMIT_MESSAGE, PR_DESCRIPTION) подробные, решения и допущения зафиксированы, но расхождение заявлений с реальностью (падающие тесты и нерабочий бонус при словах «реально прогнаны») снижает оценку оси B.

Артефакты: 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
18 счистое время судьи
51.5Kтокенов (вход 45.3K, $0.0072)
заметка судьи
Сильное решение: корректная доменная модель (тарификация в PricingCalculator/OffPeak отделена от импорта и отчёта), строгий bcmath без float для денег, HALF_UP 6 знаков на уровне события, оф-пик через полночь и включительность границ покрыты тестами, идемпотентность на уровне уникальных индексов БД. Реализован даже бонус reprice с журналом. Смущает: SpecReader допускает JSON-числа (is_float) для цен вопреки букве спецификации — переход через (string)float теоретически неточен; а .phpunit.result.cache в диффе содержит записи о дефектах 15 тестов, что противоречит заявленному 'OK (33 tests)' — возможно, устаревший артефакт, попавший в коммит, но это минус к аккуратности.

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

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