h-school/docs/design/10-craft/craft.md

Ремесло: слои, страховка тестов, горячие точки

Договорённость на срез после одежды, сессии и склейки урока, не текущий код. Фундамент: foundation.md. Проекты: projects.md. Вне очереди: off-queue.md. Общение (не ждём 44/46): social.md.

Игра уже живая: люди ходят, одеваются, учатся, болтают. Рядом лежат четыре копии одного запрета на ссылки, npm test зелёный при красном tsc, работник школы на тысячу строк и дамп, который не проверяет явку. Срез это подметает. Новой кнопки нет.

Срез отвечает на вопросы «сломал ли я слой», «прошёл ли клиентский tsc» и «можно ли разрезать работника, не трогая поток». Оценок, протокола +1 и охоты на всё SOLID нет.

Что увидит игрок

Ничего обязательного на экране. Школа после каждой фазы та же, что вчера. Игрок может заметить только то, что агент и CI ловят дубликат ключа и чужой using до мержа, а не после.

На какие вопросы отвечает

• Слои из projects.md и инвариант 4 — тест в одном месте, не четыре копипасты.
• npm test клиента = typecheck + vitest; дубликат в strings.ts красный без npm run build.
• SchoolWorker и сборка карточки режутся по швам, поведение то же.

Чего нет

• Новых экранов, полей сейва, байтов сокета.
• Фаз 44 и 46, портретов/SwarmUI как фичи, GET /api/catalog.lunchBreaks.
• Багов сессии (второе имя при живой куке, ключи DataProtection) — ../bugs/README.md.
• Инверсии Game → Api DTO и «пройти SOLID по всему дереву».

Что меняется в уже принятом

Было	Стало	Почему
Запрет Arch/ASP.NET/DateTime.Now скопирован в People, Ai, Schedule, Simulation	Один проект HSchool.Architecture.Tests; Content и Protocol в том же списке	Четыре теста расходятся; Content и Protocol молчат
npm test = только vitest	npm test = tsc -b и vitest	Журнал трижды ловил дубликат ключа, зелёный в vitest
SchoolWorker и PersonCardReader — по одному файлу на всё	partial / файлы по шву; один тип, один поток, один Read	SRP без нового инварианта
Дамп «люди с узлом или нуждами»	После найма и хода часов — кто-то с NodeId	Нужды есть у всех; явка не проверялась

Протокол, HTTP меню и формат сейва не меняются. Версию сокета не бампить.

Этап A — страховка

Слои и клиентский tsc. После этапа школа целая, даже если работник ещё тысяча строк.

Слои одной пачкой

Новый тестовый проект ссылается на библиотеки, не на AppHost. Пакета NetArchTest нет: как сейчас, GetReferencedAssemblies и чтение *.cs. Копии Assembly_DoesNotReference* и Sources_DoNotUseWallClock из People/Ai/Schedule/Simulation уезжают сюда.

Список (как projects.md):

Сборка	Не ссылается	Не содержит в исходниках
Protocol	Arch, ASP.NET, Sockets, игровые проекты	DateTime.Now / UtcNow
Content	Arch, ASP.NET, Sockets, Protocol, Simulation	то же
People, Schedule, Ai	Arch, ASP.NET, Sockets	то же
Simulation	ASP.NET, Sockets	то же; Arch — да

HSchool.Server в этот список не входит: он и есть хост.

tsc в npm test

Скрипт test гоняет typecheck (tsc -b), потом vitest. Петля "hschool-client": "file:" в package.json уходит, если без неё npm test и npm run build зелёные.

Этап B — горячие точки

Именованный рефактор, не аудит всего дерева. После этапа поведение то же: карточка, пауза, снимок, дамп.

1. SchoolWorker (~1200 строк) — partial: почта, тик, запись. Один тип, тот же LongRunning-поток, без замка вокруг School/World.
2. PersonCardReader (~600) — сборка по вкладкам в том же классе или соседних internal helper; публичный вход по-прежнему Read(School, personId, locale).
3. personCard.ts (~850) — модули по вкладкам; personCardHost.ts по-прежнему ходит в HTTP.

Game-слой по-прежнему может зависеть от DTO в Api — инверсия не этот срез.

Этап C — дамп не врёт

Dump_ReturnsPeopleNodesAndTimetable после найма запускает часы и ждёт человека с NodeId, не Needs.Count > 0. На паузе не ждать людей (кадры присутствия всё равно идут).

Где живёт код

Новой игровой библиотеки нет.

Что	Куда
Запреты слоёв и wall-clock	tests/HSchool.Architecture.Tests
Скрипт npm test	src/HSchool.Client/package.json
Работник	src/HSchool.Server/Game/SchoolWorker*.cs
Карточка HTTP	PersonCardReader + клиент ui/personCard*.ts
Дамп	уже DevEndpoints / SchoolDumpReader; меняется тест

Советы, которые стоит принять сразу

• Не тащить NetArchTest ради четырёх Assert.DoesNotContain.
• Не резать SchoolWorker на несколько типов с разными потоками.
• Не инвертировать Game → Api «заодно»: это другой срез или вне очереди.
• Не бампить протокол и не расширять catalog, чтобы «заодно» отдать lunchBreaks.
• Фаза 55 занята веткой phase/55-view-state — номера среза с 56.

Заведомо не сейчас

• Срез 9: ссора/драка, политика речи и дом.
• Баги сессии и DataProtection.
• GET /api/catalog + lunchBreaks.
• Портреты как контент; SwarmUI не выкидывать.
• README «школа ещё пустая» как отдельная кампания — одна строка в фазе 56, не рерайт.
• Новый именованный кусок игры (оценки, приказы, магазины).

Зафиксировано этим разговором

Тема	Решение
Куда	Срез 10, не «Вне очереди»
SOLID	Arch-тесты слоёв + три именованные точки, не всё дерево
Список	Журнал ревью + ложь AGENTS; без инвентаризации-фазы
Баги сессии	Не брать, /bug-work
Протокол	Не бампить
HTTP	Не расширять catalog
Сейв	Не трогать
Срез 9	Не ждать 44/46, не писать в них
Стоп	После A школа целая; после B — тоже
Ваниль	Без пака
Номера	56–59; 55 занята