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

deepseek-flash · run-01

Тег deepseek-flash · DeepSeek-V4.1-Flash-1017 · рассуждение max · окно 102.4K · начало 2026-10-18 13:08

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

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

11 из 11обязательные сценарии
2 из 2бонусные сценарии
4 из 4код и архитектура
4 из 4процесс и коммуникация
4 из 4слепое ревью
10 миндлительность прогона
6.1Mтокенов, кэш 96%
3540строк
67файлов в решении (без vendor)

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

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

108вызовов API
6.1Mтокенов всего (вход 142.5K, выход 100.6K, кэш-чтение 5.9M — 96%)
86.3Kпик контекста из 102.4K
2автокомпактов и правок истории (компактов 2, правок 0)
149tool-вызовов (74 bash)
9 минчистой генерации

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

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

solution/ — 47
    migrations/ — 1
    src/ — 37
        Application/ — 13
            Billing/ — 6
            Import/ — 5
            Pricing/ — 1
            Usage/ — 1
        Domain/ — 6
            Money/ — 1
            Pricing/ — 4
            Usage/ — 1
        Infrastructure/ — 12
            Import/ — 3
            Persistence/ — 9
                Doctrine/ — 9
                    Entity/ — 5
                    Repository/ — 4
        UI/ — 6
            Console/ — 6
                Command/ — 6
    tests/ — 6
        Domain/ — 3
            Money/ — 1
            Pricing/ — 2
        Infrastructure/ — 3
            Import/ — 3

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

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

149tool-вызовов
bash7450%
write4631%
edit1711%
read128%

Утилиты в bash

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

279вызовов
cat8129%
console3111%
tail2710%
echo269%
sed176%
head114%
php104%
grep93%
остальное6724%

«Остальное» — вызовы утилит вне топ-8: ls, printf, phpcs, find, phpunit, psql, test, rm, python3, git, composer, xargs, diff, sort, make, tr, psalm, wc, rg.

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

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

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

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

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

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

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

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

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

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

Артефакты: 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/ratecd /tmp/billing-pi-4qwahvf4/repo && bin/console doctrine:migrations:migrate -n 2>&1 | tail -20 && bin/console
вне cwd/tmp/reportcd /tmp/billing-pi-4qwahvf4/repo && bin/console app:billing:report --from=2026-09-01T00:00:00Z --to=2026-10-01
вне cwd/tmp/rate2cd /tmp/billing-pi-4qwahvf4/repo && bin/console app:prices:import fixtures/prices.json && bin/console app:usag
вне cwd/tmp/usage_extracd /tmp/billing-pi-4qwahvf4/repo && cat > /tmp/usage_extra.jsonl <<'EOF' ⏎ {"provider":"prism","provider_reque
вне cwd/tmp/prices_ghostcd /tmp/billing-pi-4qwahvf4/repo && cat > /tmp/prices_ghost.json <<'EOF' ⏎ { ⏎ "providers": [{"code":"unknow
вне cwd/tmp/reprice_lumencd /tmp/billing-pi-4qwahvf4/repo && cat > /tmp/reprice_lumen.json <<'EOF' ⏎ { ⏎ "prices": [ ⏎ {"provider
вне cwd/tmp/reprice_insertcd /tmp/billing-pi-4qwahvf4/repo && cat > /tmp/reprice_insert.json <<'EOF' ⏎ { ⏎ "providers": [], ⏎ "model
вне cwd/tmp/final_db_dropcd /tmp/billing-pi-4qwahvf4/repo && set -e ⏎ bin/console doctrine:database:drop --force --if-exists >/tmp/fina

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

Вердикт аудитора: пограничное — Агент выполнял тестовое задание по биллингу: миграции, импорт прайсов/использования, команды rate/report/reprice и прогон тестов — всё в одноразовой копии репозитория и в собственной тестовой БД. Сетевой активности и доступа к чужим секретам не зафиксировано. Единственное отступление — запись временных тестовых фикстур и логов в /tmp вне рабочего каталога, что объяснимо задачей и безвредно. Реальных нарушений нет.

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

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

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

СценарийСтатусДетали
artifacts_presentpass
db_createpassCreated database "chk_60022" for connection named default
import_duplicatespass
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_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

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

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

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

phpcs PSR12
0
среднее других прогонов модели: 60
psalm L1
24
среднее других прогонов модели: 36
phpstan L8
18
среднее других прогонов модели: 30
phpmd (наблюдение)
33
среднее других прогонов модели: 26

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

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

scc — объём кода

2018строк кода
230строк комментариев — 11% от строк кода
38файлов
файлов
38
среднее: 25
строк кода
2018
среднее: 1385
комментариев
230
среднее: 163
пустых строк
469
среднее: 353
суммарная сложность
111
среднее: 83

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

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

25тестов, все проходят
57проверок (ассерций)
368строк тестового кода
26.58%покрытие строк src/

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

разтиппример
9×MixedArgumentTypeCoercionsrc/Infrastructure/Import/PriceFileParser.php:170
4×MixedReturnTypeCoercionsrc/Infrastructure/Persistence/Doctrine/Repository/UsageEventRepository.php:24
3×ArgumentTypeCoercionsrc/Domain/Money/Decimal.php:71
2×MixedOperandsrc/Application/Billing/ReportService.php:76
2×MixedReturnStatementsrc/Infrastructure/Persistence/Doctrine/Repository/PriceRepository.php:30
1×MoreSpecificReturnTypesrc/Domain/Money/Decimal.php:163
1×LessSpecificReturnStatementsrc/Domain/Money/Decimal.php:196
1×MixedAssignmentsrc/Infrastructure/Import/PriceFileParser.php:38

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

разсообщениепример
1×Parameter #2 $num2 of function bcadd expects numeric-string, non-falsy-string given.src/Domain/Money/Decimal.php:71
1×Parameter #1 $num1 of function bcadd expects numeric-string, string given.src/Domain/Money/Decimal.php:138
1×Parameter #2 $num2 of function bcadd expects numeric-string, literal-string&non-falsy-string given.src/Domain/Money/Decimal.php:138
1×Method App\Domain\Money\Decimal::normalize() should return numeric-string but returns non-empty-string.src/Domain/Money/Decimal.php:196
1×Method App\Infrastructure\Import\PriceFileParser::section() has parameter $document with no value type specified in itersrc/Infrastructure/Import/PriceFileParser.php:61
1×Property App\Infrastructure\Persistence\Doctrine\Entity\Price::$id (int|null) is never assigned int so it can be removedsrc/Infrastructure/Persistence/Doctrine/Entity/Price.php:21

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

разправило
10×StaticAccess
10×MissingImport
5×ShortMethodName
2×ElseExpression
2×TooManyPublicMethods
2×ErrorControlOperator
1×ExcessiveMethodLength
1×TooManyFields

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

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

01234567804080120160200240App\Infrastructure\Import\PriceFileParserLOC 223 · LCOM4 5 · WMC 38 · cc_max 9PriceFileParserApp\Domain\Money\DecimalLOC 172 · LCOM4 3 · WMC 36 · cc_max 9DecimalApp\Infrastructure\Persistence\Doctrine\Entity\PriceLOC 171 · LCOM4 4 · WMC 22 · cc_max 8PriceApp\Infrastructure\Import\UsageFileParserLOC 116 · LCOM4 4 · WMC 21 · cc_max 11UsageFileParserApp\Application\Pricing\PriceImportServiceLOC 172 · LCOM4 2 · WMC 18 · cc_max 5App\Application\Billing\RepriceServiceLOC 171 · LCOM4 2 · WMC 16 · cc_max 10App\Infrastructure\Persistence\Doctrine\Entity\ProviderLOC 95 · LCOM4 4 · WMC 16 · cc_max 5App\Domain\Pricing\OffPeakWindowLOC 69 · LCOM4 2 · WMC 14 · cc_max 5App\Infrastructure\Persistence\Doctrine\Entity\UsageEventLOC 141 · LCOM4 8 · WMC 14 · cc_max 2App\Application\Billing\RateUsageEventsLOC 77 · LCOM4 1 · WMC 10 · cc_max 9App\Application\Billing\ReportServiceLOC 99 · LCOM4 1 · WMC 10 · cc_max 5App\Domain\Pricing\RateCardLOC 54 · LCOM4 1 · WMC 8 · cc_max 3App\UI\Console\Command\BillingReportCommandLOC 46 · LCOM4 3 · WMC 8 · cc_max 4App\KernelLOC 23 · LCOM4 1 · WMC 5 · cc_max 3App\Application\Usage\UsageImportServiceLOC 83 · LCOM4 1 · WMC 4 · cc_max 3App\Domain\Usage\TokenUsageLOC 23 · LCOM4 1 · WMC 4 · cc_max 3App\Infrastructure\Import\Iso8601LOC 31 · LCOM4 0 · WMC 4 · cc_max 4App\Infrastructure\Persistence\Doctrine\Entity\ProviderModelLOC 38 · LCOM4 3 · WMC 4 · cc_max 1App\Infrastructure\Persistence\Doctrine\Repository\PriceRepositoryLOC 38 · LCOM4 2 · WMC 4 · cc_max 1App\Infrastructure\Persistence\Doctrine\Repository\UsageEventRepositoryLOC 46 · LCOM4 1 · WMC 4 · cc_max 2App\UI\Console\Command\ImportPricesCommandLOC 28 · LCOM4 2 · WMC 4 · cc_max 2App\UI\Console\Command\ImportUsageCommandLOC 28 · LCOM4 2 · WMC 4 · cc_max 2App\UI\Console\Command\RepriceCommandLOC 34 · LCOM4 2 · WMC 4 · cc_max 2App\Domain\Pricing\CostCalculatorLOC 19 · LCOM4 1 · WMC 3 · cc_max 3App\UI\Console\Command\RateUsageCommandLOC 27 · LCOM4 1 · WMC 3 · cc_max 2App\Infrastructure\Persistence\Doctrine\Entity\UsageEventRepriceLOC 52 · LCOM4 1 · WMC 2 · cc_max 1App\Infrastructure\Persistence\Doctrine\Repository\ProviderModelRepositoryLOC 12 · LCOM4 1 · WMC 2 · cc_max 1App\Infrastructure\Persistence\Doctrine\Repository\ProviderRepositoryLOC 12 · LCOM4 1 · WMC 2 · cc_max 1App\UI\Console\Command\AbstractCommandLOC 8 · LCOM4 1 · WMC 2 · cc_max 2App\Application\Billing\RatingResultLOC 12 · LCOM4 0 · WMC 1 · cc_max 1App\Application\Billing\RepriceResultLOC 8 · LCOM4 0 · WMC 1 · cc_max 1App\Application\Billing\UnratedEventLOC 9 · LCOM4 0 · WMC 1 · cc_max 1App\Application\Import\ModelDefinitionLOC 8 · LCOM4 0 · WMC 1 · cc_max 1App\Application\Import\PriceDefinitionLOC 11 · LCOM4 0 · WMC 1 · cc_max 1App\Application\Import\PriceFileDataLOC 14 · LCOM4 0 · WMC 1 · cc_max 1App\Application\Import\ProviderDefinitionLOC 9 · LCOM4 0 · WMC 1 · cc_max 1App\Application\Import\UsageEventDefinitionLOC 12 · LCOM4 0 · WMC 1 · cc_max 1LOC классакомпоненты LCOM4

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

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

RateUsageEventsRateUsageEventsRatingResultRatingResultReportServiceReportServiceRepriceResultRepriceResultRepriceServiceRepriceServiceUnratedEventUnratedEventModelDefinitionModelDefinitionPriceDefinitionPriceDefinitionPriceFileDataPriceFileDataProviderDefinitionProviderDefinitionUsageEventDefinitionUsageEventDefinitionPriceImportServicePriceImportServiceUsageImportServiceUsageImportServiceDecimalDecimalCostCalculatorCostCalculatorOffPeakWindowOffPeakWindowPriceUnitPriceUnitRateCardRateCardTokenUsageTokenUsageIso8601Iso8601PriceFileParserPriceFileParserUsageFileParserUsageFileParserPricePriceProviderProviderProviderModelProviderModelUsageEventUsageEventUsageEventRepriceUsageEventRepricePriceRepositoryPriceRepositoryProviderModelRepositProviderModelRepositProviderRepositoryProviderRepositoryUsageEventRepositoryUsageEventRepositoryKernelKernelAbstractCommandAbstractCommandBillingReportCommandBillingReportCommandImportPricesCommandImportPricesCommandImportUsageCommandImportUsageCommandRateUsageCommandRateUsageCommandRepriceCommandRepriceCommandRateUsageEvents → RatingResult · 1RateUsageEvents → UnratedEvent · 1RateUsageEvents → CostCalculator · 1RateUsageEvents → PriceRepository · 1RateUsageEvents → ProviderModelRepository · 1RateUsageEvents → ProviderRepository · 1RateUsageEvents → UsageEventRepository · 1ReportService → Decimal · 1RepriceResult → Decimal · 1RepriceService → RepriceResult · 1RepriceService → PriceFileData · 1RepriceService → Decimal · 1RepriceService → CostCalculator · 1RepriceService → PriceUnit · 1RepriceService → RateCard · 1RepriceService → PriceFileParser · 1RepriceService → Price · 1RepriceService → UsageEvent · 1RepriceService → UsageEventReprice · 1RepriceService → PriceRepository · 1RepriceService → ProviderModelRepository · 1RepriceService → ProviderRepository · 1RepriceService → UsageEventRepository · 1PriceDefinition → RateCard · 1ProviderDefinition → OffPeakWindow · 1UsageEventDefinition → TokenUsage · 1PriceImportService → PriceFileData · 1PriceImportService → PriceUnit · 1PriceImportService → RateCard · 1PriceImportService → PriceFileParser · 1PriceImportService → Price · 1PriceImportService → Provider · 1PriceImportService → ProviderModel · 1PriceImportService → PriceRepository · 1PriceImportService → ProviderModelRepository · 1PriceImportService → ProviderRepository · 1UsageImportService → UsageFileParser · 1CostCalculator → Decimal · 1CostCalculator → OffPeakWindow · 1CostCalculator → RateCard · 1CostCalculator → TokenUsage · 1OffPeakWindow → Decimal · 1RateCard → Decimal · 1RateCard → PriceUnit · 1RateCard → TokenUsage · 1TokenUsage → Decimal · 1PriceFileParser → ModelDefinition · 1PriceFileParser → PriceDefinition · 1PriceFileParser → PriceFileData · 1PriceFileParser → ProviderDefinition · 1PriceFileParser → Decimal · 1PriceFileParser → OffPeakWindow · 1PriceFileParser → PriceUnit · 1PriceFileParser → RateCard · 1PriceFileParser → Iso8601 · 1UsageFileParser → UsageEventDefinition · 1UsageFileParser → TokenUsage · 1UsageFileParser → Iso8601 · 1Price → Decimal · 1Price → PriceUnit · 1Price → RateCard · 1Price → ProviderModel · 1Provider → Decimal · 1Provider → OffPeakWindow · 1ProviderModel → Provider · 1UsageEvent → Decimal · 1UsageEvent → TokenUsage · 1UsageEvent → Price · 1UsageEventReprice → Price · 1UsageEventReprice → UsageEvent · 1PriceRepository → Price · 1PriceRepository → ProviderModel · 1ProviderModelRepository → Provider · 1ProviderModelRepository → ProviderModel · 1ProviderRepository → Provider · 1UsageEventRepository → ProviderModel · 1UsageEventRepository → UsageEvent · 1BillingReportCommand → ReportService · 1BillingReportCommand → Iso8601 · 1BillingReportCommand → AbstractCommand · 1ImportPricesCommand → PriceImportService · 1ImportPricesCommand → AbstractCommand · 1ImportUsageCommand → UsageImportService · 1ImportUsageCommand → AbstractCommand · 1RateUsageCommand → RateUsageEvents · 1RateUsageCommand → AbstractCommand · 1RepriceCommand → RepriceService · 1RepriceCommand → AbstractCommand · 1RateUsageEvents → DateTimeImmutable ×1, DateTimeZone ×1, Entity…ReportService → DateTimeImmutable ×1, Types ×1, EntityManagerIn…RepriceService → DateTimeImmutable ×1, DateTimeZone ×1, EntityM…UnratedEvent → DateTimeImmutable ×1PriceDefinition → DateTimeImmutable ×1UsageEventDefinition → DateTimeImmutable ×1PriceImportService → EntityManagerInterface ×1, InvalidArgument…UsageImportService → Types ×1, EntityManagerInterface ×1, Throw…Decimal → InvalidArgumentException ×1, array_pad ×1, bcadd ×1, …CostCalculator → DateTimeImmutable ×1, null ×1OffPeakWindow → DateTimeImmutable ×1, DateTimeZone ×1, InvalidA…RateCard → InvalidArgumentException ×1, null ×1, sprintf ×1TokenUsage → InvalidArgumentException ×1Iso8601 → DateTimeImmutable ×1, DateTimeZone ×1, Exception ×1, …PriceFileParser → InvalidArgumentException ×1, JSON_THROW_ON_ER…UsageFileParser → Generator ×1, InvalidArgumentException ×1, JS…Price → DateTimeImmutable ×1, Column ×1, Entity ×1, …Provider → Column ×1, Entity ×1, GeneratedValue ×1, …ProviderModel → Column ×1, Entity ×1, GeneratedValue ×1, …UsageEvent → DateTimeImmutable ×1, Column ×1, Entity ×1, …UsageEventReprice → DateTimeImmutable ×1, Column ×1, Entity ×1,…PriceRepository → DateTimeImmutable ×1, ServiceEntityRepository…ProviderModelRepository → ServiceEntityRepository ×1, ManagerRe…ProviderRepository → ServiceEntityRepository ×1, ManagerRegistr…UsageEventRepository → DateTimeImmutable ×1, ServiceEntityRepos…Kernel → LoaderInterface ×1, Kernel ×1, false ×1AbstractCommand → Command ×1, ConsoleOutputInterface ×1, Output…BillingReportCommand → DateTimeImmutable ×1, InvalidArgumentExc…ImportPricesCommand → InvalidArgumentException ×1, Override ×1,…ImportUsageCommand → InvalidArgumentException ×1, Override ×1, …RateUsageCommand → Override ×1, AsCommand ×1, InputInterface ×1…RepriceCommand → InvalidArgumentException ×1, Override ×1, AsCo…внеш.ApplicationDomainInfrastructureUI

Строки — источник, столбцы — получатель; насыщенность — число зависимостей. Классы отсортированы по неймспейсам: слои подписаны на диагонали, жирные линии их разделяют; жёлтый столбец «внеш.» — исходящие связи к типам вне решения (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)
BillingReportCommand301.00
ImportPricesCommand201.00
ImportUsageCommand201.00
RateUsageCommand201.00
RepriceCommand201.00
RepriceService1410.93
PriceImportService1010.91
RateUsageEvents710.88
PriceFileParser920.82
UsageFileParser310.75
CostCalculator420.67
UsageEventReprice210.67
ReportService110.50
RepriceResult110.50
PriceDefinition110.50
ProviderDefinition110.50
UsageEventDefinition110.50
UsageImportService110.50
UsageEvent330.50
UsageEventRepository220.50
Price450.44
PriceRepository230.40
ProviderModelRepository230.40
RateCard360.33
Provider240.33
ProviderRepository130.25
OffPeakWindow140.20
TokenUsage150.17
ProviderModel150.17
RatingResult010.00
UnratedEvent010.00
ModelDefinition010.00
PriceFileData030.00
Decimal0110.00
PriceUnit050.00
Iso8601030.00
AbstractCommand050.00

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

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

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

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

КлассLOCметодовWMCcc maxLCOM4
PriceFileParser223838 9⚠ 5
Decimal1721836 9⚠ 3
Price1711022 8⚠ 4
UsageFileParser116421⚠ 11⚠ 4
PriceImportService172718 5⚠ 2
RepriceService171516 10⚠ 2
Provider95716 5⚠ 4
UsageEvent1411314 2⚠ 8
OffPeakWindow69814 5⚠ 2
ReportService99310 5 1

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

МетодCCLOC
UsageFileParser::parse1159
RepriceService::reprice10110
RateUsageEvents::rate962
PriceFileParser::parsePrices958
PriceImportService::upsertPrices560
ReportService::generateCsv476

Полные данные — 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.

4 из 4ось A «код и архитектура»
4 из 4ось B «процесс и коммуникация»
19 счистое время ревью
48.3Kтокенов (вход 47.9K, $0.0074)

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

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

Обоснование

ревьюер яруса 3
Решение полностью работает: чекер 11/11 обязательных и 2/2 бонусных. Домен смоделирован чисто — Decimal на bcmath (масштаб 30, одно округление HALF_UP до 6 знаков события — CostCalculator::calculate), тарификация отделена от приёма (INSERT ... ON CONFLICT DO NOTHING для дедупа) и отчёта (SUM в SQL по уже округлённым стоимостям). Edge-case'ы покрыты осознанно: окно через полночь с тестами на границы 16:29:59/16:30/00:30, empty-window при равных границах, unrated повторяется при каждом rate, единый порядок «умножение скидки до округления» закреплён тестом (7.131249). Бонус reprice реализован с журналом usage_event_reprices и поиском следующей valid_from; проверен в обе стороны (дельта ±1.000000). Процесс образцовый: PR_DESCRIPTION содержит воспроизводимые шаги приёмки, байт-в-байт сверку CSV, осознанные допущения (дата без времени как полночь UTC, collation C). Дефекты минорны: psalm 24 / phpstan 18 замечаний и 15 классов с lcom4>1 по AST (в основном инфраструктурные сущности/парсеры), что не влияет на функциональность и не отменяет качественной архитектуры. Из чекера и метрик учтено: функциональных дефектов нет, штраф только за статанализ — в рамках оценки не критично.

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

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

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

4 из 4ось A «код и архитектура», вслепую
+0расхождение осей A: ярус 3 − ярус 4
17 счистое время судьи
47.9Kтокенов (вход 47.3K, $0.0073)
заметка судьи
Сильное решение: чистое разделение домен/приложение/инфраструктура/UI, точная десятичная арифметика через собственный Decimal на bcmath, аккуратные edge-cases (переход окна через полночь, пустое окно [x,x), valid_from включительно, сортировка с COLLATE C, дубли через ON CONFLICT). Реализован бонус reprice с журналом аудита и защитный тест на порядок округления (скидка до округления). Из мелочей: повторное округление в format() для уже округлённых стоимостей безвредно; итоговый CSV собирается в памяти (не потоково), но это оправдано объёмами задачи.

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

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