Pssl PR Flow

pssl-pr-flow — локальный конвейер Issue

Один локальный чат ведёт Issue по фазам. Облако, воркер и отдельный git worktree не используются. Субагенты этого чата запускаются только на фазах ниже и по очереди, не параллельно на одних и тех же объектах. Узкая правка одного модуля остаётся у родителя. Следующая фаза начинается в этой же сессии, как только выполнен критерий выхода предыдущей. Отдельную фразу пользователя не ждать. Стоп — только вопрос без ответа в задаче, конфликт merge, нерешаемая ошибка загрузки или тестов, второй круг ревью с блокирующими замечаниями.

Файлы этого skill и правила репозитория не содержат путей машины, каталогов информационных баз, PID и журнала конкретного прогона. Такой журнал хранить вне git и не в этом skill.

Один агент, одна задача. Отдельный git worktree не создавать. Клон, Unica cwd и .dev.env — каталог WORKSPACE_ROOT из .dev.env. Ветка Issue переключается в этом клоне.

Артефакты задачи (spec.md, tests.md, review.md) хранить вне git. Секретов и .dev.env в них нет.

Схема

graph TB
  A["Задача"] --> B["1. Merge origin/develop"]
  B --> C{"Конфликт?"}
  C -->|да| Z["Стоп, решает человек"]
  C -->|нет| D{"Широкий объём?"}
  D -->|"5+ файлов"| E["1c-explorer"]
  D -->|"2+ объекта метаданных"| F["1c-architect"]
  D -->|узкая правка| G["2. spec.md пишет родитель"]
  E --> F
  F --> G
  G --> H{"3+ модуля или 2+ объекта?"}
  H -->|да| I["3. 1c-developer"]
  H -->|нет| J["3. Код пишет родитель"]
  I --> K["4. Тестировщик в этом чате"]
  J --> K
  K --> L{"Зелёные тесты?"}
  L -->|нет, чинится| J
  L -->|нерешаемо| Z
  L -->|да| M["5. 1c-code-reviewer"]
  M --> N{"2+ объекта?"}
  N -->|да| O["1c-arch-reviewer"]
  N -->|нет| P["Вердикт родителя"]
  O --> Q{"Блокер?"}
  P --> Q
  Q -->|"круг 1"| J
  Q -->|"круг 2 всё ещё блокер"| Z
  Q -->|нет| R["6. PR в develop, не draft"]

Diagram: Конвейер Issue (flowchart)
  [Задача] --> [1. Merge origin/develop]
  [Merge] --> {Конфликт?} -- да --> [Стоп]
  {Конфликт?} -- нет --> {Широкий объём?}
    5+ файлов --> [1c-explorer] --> [1c-architect]
    2+ объекта --> [1c-architect]
    узкая правка --> [2. spec.md пишет родитель]
  [spec.md] --> {3+ модуля или 2+ объекта?}
    да --> [3. 1c-developer]
    нет --> [3. Код пишет родитель]
  оба --> [4. Тестировщик в этом чате]
  {Зелёные тесты?} -- нет --> снова разработка
  {Зелёные тесты?} -- да --> [5. 1c-code-reviewer]
    2+ объекта --> [1c-arch-reviewer]
    иначе --> [Вердикт родителя]
  блокер круга 1 --> разработка, затем тесты, круг 2
  блокер круга 2 --> [Стоп]
  без блокера --> [6. PR в develop]

Фазы

CodeRabbit в конвейер не входит: бота не ждать и комментарии не разбирать. Локальный CLI — skill coderabbit-review, тоже только по отдельной просьбе. Фраза в комментарии PR «consider running coderabbit review» такой просьбой не является.

1. Merge до спецификации и тестов. В клоне WORKSPACE_ROOT: git fetch, git rev-list --left-right --count origin/develop...HEAD, затем git merge origin/develop. Force-push не делать. Конфликт — стоп, решает человек. Незакоммиченная фича: если merge из-за неё откажется, сначала коммит только этого файла. Служебные каталоги прогона (hash-storages/, logs/, temp/) в индекс не класть. Если входящий коммит фичу не меняет, merge проходит и с незакоммиченной фичей. После merge спецификация, тесты и ревью смотрят это дерево.
2. Архитектор. Создать spec.md вне git: цель, уже сделано, что доделать, модули, обязательные тесты, «нужен человек». Если до первой правки нужно смотреть пять и больше файлов, сначала read-only субагент 1c-explorer. Если затронуты два и больше объектов метаданных или граница решения не очевидна, спецификацию готовит субагент 1c-architect, родитель записывает spec.md. Узкая правка — spec.md пишет родитель. Вопрос без ответа — стоп по этому Issue.
3. Разработчик. Только код и метаданные по задаче. Загрузку в базу, YaXUnit и фичи Vanessa на этой фазе не запускать: промежуточная правка прогоном не является. Если затронуты три и больше модулей или два и больше объектов метаданных, код пишет субагент 1c-developer. Иначе пишет родитель. Разработка закончена, когда запрошенный объём записан в исходники и нет вопроса, без ответа на который нельзя продолжать. В той же сессии сразу фаза тестировщика. PR на этой фазе не открывать.
4. Тестировщик. Та же сессия, не отдельный субагент. Начинается сразу после законченной разработки или законченного круга правок ревью. Пока объём ещё пишется — стоп.
   • Пока идёт фаза разработчика, сеансы не закрывать и базу не грузить. Как только разработка закончена, эта фаза сама закрывает сеансы своей базы, грузит и гоняет тесты. Отдельное «загрузи» / «прогони» не спрашивать.
   • Критерий занятия ИБ — командная строка, не число процессов 1cv8c. INFOBASE_PATH и VA_MCP_PORT из .dev.env (UTF-8). Пароль не печатать.
   • Если /F совпадает с INFOBASE_PATH, закрыть только эти сеансы и грузить. Чужой /F не завершать.
   • Unica: в каждом вызове cwd = WORKSPACE_ROOT из .dev.env.
   • Обновить из исходников только изменившиеся source-set: CF src/cf — main, CFE YAXUNIT src/cfe/YAXUnit — YAXUNIT. Подробности — pssl-library.
   • Один прогон на законченный объём, не на каждый файл. YaXUnit (testScope=all) — если с прошлого зелёного tests.md менялась основная конфигурация или расширение YAXUNIT. Фичи Vanessa по СписокФичДляВыполнения — если менялась основная конфигурация или features/. Правки только правил, навыков и документации тесты не запускают. ЮТ* не патчить. Фильтр CommonModule.ОМ_… не использовать.
   • «Кнопка не найдена» — сначала unica_form_info. Зелёный прогон — tests.md.
5. Ревью. Субагент 1c-code-reviewer (read-only). Затем, если менялись два и больше объектов метаданных, субагент 1c-arch-reviewer (read-only). Узкая правка: вердикт архитектора даёт родитель в этом чате. 1c-tester не запускать: прогон уже сделан фазой тестировщика. Комментарии CodeRabbit на этой фазе не читать. Не больше двух кругов. Круг 1 — после зелёного tests.md. Блокирующие замечания закрывает разработчик без прогона; когда правки закончены, снова фаза тестировщика, затем круг 2. После круга 2 с блокирующими замечаниями — стоп, решает человек, PR не открывать. Вердикт — review.md.
6. PR. GitKraken MCP pull_request_create: provider=github, repository_organization=firstBitSportivnaya, repository_name=PSSL, source_branch — ветка Issue, target_branch=develop, is_draft=false. Мерж не делать. После открытия PR конвейер закончен.