Поставщик периодически кладёт в CSV-колонку project имена нестандартного
формата (телефон '79135191264', URL); extractPlatform() возвращает null,
строка пропускается. Это поведение, не баг на нашей стороне — даунгрейд
до info, чтобы перестать спамить laravel.log warning'ами по 13+ раз/день
(не actionable, processing продолжается).
Параллельно подчищены 4 truly-orphan supplier_projects (id 57/73/77/79)
на проде — тестовые placeholders (x.example / 79991234567 / URL); 16 leads
получили supplier_project_id=NULL (raw_payload preserved), 0 deals в любом
tenant'е по этим телефонам — info@lkomega.ru/client1 не затронут.
На prod failed_webhook_jobs и incidents_log имеют RLS-политики на
app.current_tenant_id, который в cron-контексте не установлен.
На dev postgres-superuser скрывал проблему (BYPASSRLS implicitly).
Переключил все 4 DB::table() в IncidentsWatchFailures на
DB::connection('pgsql_supplier') — ту же роль crm_supplier_worker
BYPASSRLS, что используют другие системные cron-команды
(ResetMonthlyCounters, RetryFailedSupplierJobs).
Тесты обновлены: +SharesSupplierPdo trait для cross-connection
visibility в DatabaseTransactions-обёртке (паттерн как у
ResetMonthlyCountersCommandTest). Все 36/36 P2 specs локально ✅.
ПИЛОТ.md §6 п.9: P2 DEPLOYED на боевой liderra.ru 22.05 ночь
(schedule:list +incidents:watch-failures каждые 10 мин, smoke
No-failure-spikes-detected, tenant_operations_log/webhook_log
чистые 0/0). Бэкап /home/ubuntu/deploy-backups/2026-05-22-pre-p2-*.
--no-verify: lefthook deadlock 5 параллельных сессий + Windows
file-lock self-deadlock; код проверен pint+pest 36/36 + код
на проде с тем же MD5 работает ("No failure spikes detected").
Co-Authored-By: Claude Opus 4.7 <noreply@anthropic.com>
«Снимок снят» обновлён: правая панель drawer'а и галочка теперь
исчезают после Save/Pause/Delete (#4); отступ страницы выровнен
с KanbanView 24px (#5, scoped CSS — pa-6 не подходит из-за конфликта
!important с has-drawer); селектор «Показывать по 20/50/100/200»
(#6, паттерн как у DealsView) + серверный max per_page 100→200 +
v-pagination когда total>per_page; фильтры регион/день приёма + 8
сортировок + дефолт «-delivered_today» + whitelist-защита от инъекции
(#7). 5 файлов, Pest 80/80 + Vitest 30/30 + Vite 2.32s. Деплой через
scp+rsync+cache+reload-fpm. Smoke на проде: API/projects с новыми
params → 401 JSON (не 500) → SQL не сломан; sort=password → тоже 401,
whitelist fallback работает. Прошлый «Снимок снят» (APP_KEY incident +
backend supplier group-sync fix) сохранён как «Раньше 22.05 (ночь)»
исторический слой.
+ docs/observer/STATUS.md auto-regen.
Co-Authored-By: Claude Opus 4.7 <noreply@anthropic.com>
Обновлены два места в ПИЛОТ.md:
- «Снимок снят» (line 11) — упоминание выкатки supplier group-sync fix.
- §2 «Развёрнутый прикладной код» (line 31) — детальный отчёт о
фиксе, деплое и ре-тесте на проде. Зафиксировано что осталось
не сделано (16 осиротевших + csv_reconcile spam, UI #4-#7,
финальная чистка qa-tenant'ов).
Co-Authored-By: Claude Opus 4.7 <noreply@anthropic.com>
Два edge'а, всплывших при ре-тесте фикса 1be2d62f на боевом:
1. Fallback для пустого eligible-tomorrow: проект с workdays Mon-Fri,
синхронизированный вечером пятницы → tomorrow=Sat → eligible=[].
computeOrder([])=0, distribute(0)=0/0/0, portal: "Введите limit!".
Если eligible пуст, но группа active — взять computeOrder по всей
активной группе (per-day eligibility соблюдается workdays).
2. Pause-limit: portal требует non-zero limit даже при status=paused.
При паузе последнего активного group=[], order=0, "Введите limit!".
Решение: max(1, sp.current_limit) — сохраняем существующий лимит,
заказы остановлены статусом=paused.
Подтверждено вживую на проде liderra.ru: pause→status=false lim=10,
resume→status=true reg=21. #1/#2/#3 при изменении: 10/10/10.
Регрессия: 37/37 (Sync + Update + Actions).
Co-Authored-By: Claude Opus 4.7 <noreply@anthropic.com>
Закрывает замечания заказчика (22.05.2026) по проектам/поставщику. Все 4 куска
имеют общий корень: online-синхронизация одного проекта работала с данными ЭТОГО
проекта, а не пересчитывала всю «группу» (проекты разных tenant'ов с одним
identifier) — отсюда переплата ×3 при изменении лимита, затирание регионов/дней
группы, неотправленная пауза, и осиротевшие проекты при смене источника.
1. Групповой пересчёт в SyncSupplierProjectJob::handleOnline (#1 при изменении,
#2 дни, #3 регионы, C2/C3): union regions, computeOrder eligible,
distributeForPlatform — те же расчёты, что в ночном syncGroup. Online и
ночной теперь дают идентичный supplier-state, расхождение устранено.
2. Пауза #10:
- ProjectController::toggleActive — диспатчит SyncSupplierProjectJob;
- ProjectService::bulkPauseResume — диспатчит sync per project;
- DTO status вычисляется из groupActive (paused когда группа без активных);
- sp.inactive_since пишется при пересинке (для UI/DTO консистентности).
3. Смена источника #8/#9 в ProjectService::update:
- до update снимается старый buildUniqueKeyAgnostic;
- если изменился — отвязываем старые supplier_projects от этого project
(pivot + legacy FK), DeleteSupplierProjectJob удаляет их у поставщика
при отсутствии других потребителей, либо пересинкает агрегат.
4. Перенос auto-link корня из feat/root-domain-auto-link: новый
App\Support\SupplierIdentifier::extractRootDomain + блоки auto-link в
обоих джобах (online + nightly).
Тесты: TDD на каждый кусок. SyncSupplierProjectJobTest +2 (group recompute,
pause). ProjectUpdateDedupTest +1 (source detach + cleanup dispatch).
ProjectsActionsTest +2 (toggle + bulk pause dispatches).
Регрессия: 186/186 passed (Project/Plan5/Projects + Supplier), 502 assertions.
Деплой: дельтой на боевой (база = root-domain ветка; на боевом джобы СТАРЕЕ
main, deliver через копию изменённых файлов + config:cache + restart queue).
План: docs/superpowers/plans/2026-05-22-замечания-проекты-чеклист.md
Co-Authored-By: Claude Opus 4.7 <noreply@anthropic.com>
SKILL.md behavioral reminder split into two cases (no-profile-task vs
missed-activation). aggregation-template.md gains a Missed Activations
section (by-node + by-classification breakdown) and the footnote now
reflects the conditional rule.
Hooks (markdownlint, cspell) verified manually; --no-verify used to avoid
the background-commit/adr-judge index-lock deadlock in this environment.
Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
Auto-regenerates tools/.node-dormancy.json when docs/Tooling_v8_3.md
changes and stages the result into the same commit. Mirrors the existing
status-md post-commit pattern.
Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
Реальный портал отдаёт rt-projects-load с name='B1_<id>' / 'B2_<id>' / 'B3_<id>'
и чистым идентификатором в поле 'content'. Старое matching по name === uniqueKey
никогда не совпадало с реальным ответом → idMap пустой → SyncSupplierProjectJob
молча выходил, ничего не записав в БД, а на портале оставались orphan-группы.
Объясняет ранее задокументированное в ЭТАЛОН «проект 5 вылечен вручную —
усыновлены 3 портальные записи». Заказчик обходил тот же баг руками.
Фикс — matching по content с fallback на name, чтобы мок-тесты с упрощённым
форматом (без content) продолжали работать; реалистичная фикстура добавлена
в SupplierPortalClientMultiFlagTest.
Verified:
- Pest supplier suite (SyncSupplierProjectJob/SyncSupplierProjectsJob/multi-flag): 16/16 passed
- E2E live на crm.bp-gr.ru: ProjectService::create + sync → supplier_projects записаны
с ext_id, pivot заполнен, портал имеет 3 группы B1/B2/B3
- Multi-tenant ночной батч с computeOrder проверен на 79991177889 (T1+T2+T3+T4
на одном identifier — формула max(max, ceil(Σ/3)) сходится с фактом)
Портал crm.bp-gr.ru возвращает status=Doubles при попытке создать
вторую группу с тем же unique_key. Старый код делал одну B1/B2/B3-группу
на каждый регион проекта — вторая группа молча пропадала.
Теперь оба джоба (SyncSupplierProjectJob + SyncSupplierProjectsJob)
формируют ровно одну группу на идентификатор со всеми регионами:
- regions=[82,83] → tag='РФ', regions=[82,83] в одной группе
- regions=[] → tag='РФ', regions=[] (вся РФ)
- regions=[82] → tag='Москва', regions=[82]
subject_code=null во всех supplier_projects и project_supplier_links.
ProjectService::update() теперь триггерит SyncSupplierProjectJob
при изменении поля regions.
Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
План 4 Task 4 эпика project-migration-redesign.
- NewProjectDialog: отдельный чекбокс «Вся РФ» (89 субъектов в autocomplete
без sentinel сохранены) + inline v-alert предупреждение + подтверждение.
- Взаимоисключение: выбор субъектов снимает «Вся РФ» и наоборот.
- Гейт submit: блок если ни субъектов, ни подтверждённой «Вся РФ»
(errors.regions = «Выберите регион...»); «Вся РФ» -> regions=[] на API.
- Лейбл autocomplete «Регионы» (убрано «(пусто = вся РФ)»).
- watch immediate:true — инициализация vsyaRf/edit-prefill при mount
(чинит EditProjectDialog submit при модальном открытии).
- Vitest 3/3 новых + 22 passed соседних (NewProject/Edit/ProjectsView) без регрессий.
План 4 Task 2 эпика project-migration-redesign.
- AdminSupplierIntegrationController +projectsIndex (список supplier_projects
+ кто заказывал через pivot project_supplier_links -> projects -> tenants
organization_name + дата последней поставки = max supplier_leads.received_at
+ subject_name из RussianRegions::CODE_TO_NAME, «РФ» при NULL subject_code).
- +projectsDestroy (bulk-delete: deleteProject на портале, затем локально;
pivot снимается CASCADE; сбой строки не прерывает batch -> failures[]).
- Routes: GET /projects, POST /projects/delete в admin-группе.
- Pest 5/5 (26 assertions). phpstan-baseline +9 ignore (Pest TestCall).
Closes the «Observer instrument expansion v2» epic. The retro note is
the source of all #1-#19 references in commit messages; the plan is
the procedural source (with REVISION v1.1 after parallel-session rebase).
Both kept in repo for traceability of the 20-commit epic.
Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
Worktree has no app/node_modules — vitest not run here; final regression
deferred to main-checkout post parallel-session release. Logic is a 7-line
pure filter; tests cover empty filter, classification, errors-only.
Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
Дмитрий