feat(tradein/db): seed существующих юзеров + ASCII-CHECK на username (#2557) #2564

Merged

lekss361 merged 2 commits from feat/tradein-users-seed into main

2026-07-30 19:17:35 +00:00

lekss361 commented

2026-07-30 18:57:46 +00:00

Owner

Closes #2557 (эпик #2549, шаг 7/8).

193_tradein_users_seed.sql:

ASCII-CHECK username ~ '^[A-Za-z0-9._-]{3,64}$' (требование deep-review #2561) — fail-closed на уровне схемы против latin-1 схлопывания кириллических логинов в общий downstream-identity. Добавлен ДО инсертов, идемпотентно через pg_constraint-проверку (паттерн 189_*).
Seed org-карты: admin→admin; kopylov, praktika→manager; user1, user3–user10→employee (свободные слоты без org); user2→employee + is_active=false (Брусника, закрыт 2026-07-30). QA-фикстуры (admintest/pilottest/analysttest/expiredtest) не мигрируются — остаются в roles.yaml.
password_hash = NULL у всех: вход по паролю невозможен, доступ держится на legacy Caddy (dual-mode) — миграция никого не отключает. Пароли админ проставит через team-UI (#2556).
display_name='Копылов' — единственные задокументированные реальные данные (auth.py _USERNAME_PROFILE), остальное NULL (не выдумываем).
Идемпотентность: ON CONFLICT DO UPDATE с COALESCE(текущее, EXCLUDED) на password_hash/is_active/профильных полях — повторный прогон не затирает то, что менеджер поменял через UI; role/manager_id намеренно синкаются (источник истины org-карты).

Валидация на scratch-БД в tradein-postgres: прогон 192 → 193 (13 строк) → симуляция ручной правки через UI → прогон 193 повторно (правка сохранена, констрейнт не задублирован) → sanity-check INSERT 'иванов' → отказ по констрейнту → DROP DATABASE. Боевая БД не тронута.

Closes #2557 (эпик #2549, шаг 7/8). `193_tradein_users_seed.sql`: - **ASCII-CHECK** `username ~ '^[A-Za-z0-9._-]{3,64}$'` (требование deep-review #2561) — fail-closed на уровне схемы против latin-1 схлопывания кириллических логинов в общий downstream-identity. Добавлен ДО инсертов, идемпотентно через `pg_constraint`-проверку (паттерн 189_*). - **Seed org-карты**: admin→admin; kopylov, praktika→manager; user1, user3–user10→employee (свободные слоты без org); user2→employee + `is_active=false` (Брусника, закрыт 2026-07-30). QA-фикстуры (admintest/pilottest/analysttest/expiredtest) не мигрируются — остаются в roles.yaml. - `password_hash = NULL` у всех: вход по паролю невозможен, доступ держится на legacy Caddy (dual-mode) — миграция никого не отключает. Пароли админ проставит через team-UI (#2556). - `display_name='Копылов'` — единственные задокументированные реальные данные (auth.py `_USERNAME_PROFILE`), остальное NULL (не выдумываем). - Идемпотентность: `ON CONFLICT DO UPDATE` с `COALESCE(текущее, EXCLUDED)` на password_hash/is_active/профильных полях — повторный прогон не затирает то, что менеджер поменял через UI; role/manager_id намеренно синкаются (источник истины org-карты). Валидация на scratch-БД в tradein-postgres: прогон 192 → 193 (13 строк) → симуляция ручной правки через UI → прогон 193 повторно (правка сохранена, констрейнт не задублирован) → sanity-check `INSERT 'иванов'` → отказ по констрейнту → `DROP DATABASE`. Боевая БД не тронута.

Rows
Columns