fix(claude): quote agent descriptions — YAML frontmatter parse break #711

Merged
lekss361 merged 1 commit from fix/agent-frontmatter-yaml into main 2026-05-30 10:59:06 +00:00
Collaborator

Баг

code-reviewer / qa-tester / tech-analyst молча не регистрировались как subagent → reviewer-окно падало Agent type 'code-reviewer' not found, qa-окно так же не нашло бы qa-tester.

Причина: у этих трёх description: был без кавычек и содержал : (двоеточие-пробел: verdict: ✅, Проверяет: HTTP, план: что). YAML трактует : как вложенный mapping → frontmatter невалиден → агент скипается при загрузке. backend-engineer/deep-code-reviewer не задеты — у них в description нет : .

Фикс

  • Обернул description в кавочки: code-reviewer/qa-tester → "..."; tech-analyst → '...' (внутри есть "). Проверено pyyaml — парсится, name+description грузятся.
  • Добавил postgres-gendesign в .claude/mcp/qa.jsonqa-tester ссылается на его tools; без сервера в strict-конфиге qa-окна агент не зарегистрируется.

Test plan

  • reviewer-окно (start-bot.ps1 reviewer → /work-as-reviewer): code-reviewer доступен как subagent (нет 'not found')
  • qa-окно: qa-tester доступен
  • claude в любом окне: /agents показывает code-reviewer/qa-tester/tech-analyst

Только .claude/ — без deploy. Требует git pull + рестарт окон.

## Баг `code-reviewer` / `qa-tester` / `tech-analyst` **молча не регистрировались** как subagent → reviewer-окно падало `Agent type 'code-reviewer' not found`, qa-окно так же не нашло бы `qa-tester`. **Причина:** у этих трёх `description:` был **без кавычек и содержал `: `** (двоеточие-пробел: `verdict: ✅`, `Проверяет: HTTP`, `план: что`). YAML трактует `: ` как вложенный mapping → frontmatter невалиден → агент скипается при загрузке. `backend-engineer`/`deep-code-reviewer` не задеты — у них в description нет `: `. ## Фикс - Обернул `description` в кавочки: code-reviewer/qa-tester → `"..."`; tech-analyst → `'...'` (внутри есть `"`). Проверено pyyaml — парсится, name+description грузятся. - Добавил `postgres-gendesign` в `.claude/mcp/qa.json` — `qa-tester` ссылается на его tools; без сервера в strict-конфиге qa-окна агент не зарегистрируется. ## Test plan - [ ] reviewer-окно (`start-bot.ps1 reviewer` → /work-as-reviewer): `code-reviewer` доступен как subagent (нет 'not found') - [ ] qa-окно: `qa-tester` доступен - [ ] `claude` в любом окне: /agents показывает code-reviewer/qa-tester/tech-analyst _Только `.claude/` — без deploy. Требует git pull + рестарт окон._
bot-reviewer added 1 commit 2026-05-30 10:57:14 +00:00
Баг: у code-reviewer/qa-tester/tech-analyst `description:` без кавычек содержал
`: ` (двоеточие-пробел, напр. "verdict: ", "Проверяет: HTTP", "план: что") →
YAML парсил это как вложенный mapping → frontmatter невалиден → агент молча НЕ
регистрировался как subagent. Reviewer-окно падало: "Agent type 'code-reviewer'
not found"; qa-окно так же не нашло бы qa-tester.

Фикс: обернул description в кавычки (code-reviewer/qa-tester → "...",
tech-analyst → '...' из-за внутренних "). Проверено pyyaml — теперь парсится.

Плюс: добавил postgres-gendesign в .claude/mcp/qa.json — qa-tester ссылается на
его tools, без сервера в strict-конфиге qa-окна агент не зарегистрируется.

(backend-engineer/deep-code-reviewer не были задеты — у них в description нет ": ".)
lekss361 merged commit 6e8c19ebe2 into main 2026-05-30 10:59:06 +00:00
lekss361 deleted branch fix/agent-frontmatter-yaml 2026-05-30 10:59:06 +00:00
Sign in to join this conversation.
No reviewers
No milestone
No project
No assignees
1 participant
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference: lekss361/gendesign#711
No description provided.