feat(qa): tester заводит bug-issue на новые баги #713

Merged

lekss361 merged 1 commit from fix/qa-creates-bug-tasks into main

2026-05-30 11:23:58 +00:00

bot-reviewer commented

2026-05-30 11:21:15 +00:00

Collaborator

Запрос

«Тестер создаёт задачи если находит баги?» — раньше НЕТ (только reopen тестируемого issue + needs-fix; новый/побочный баг терялся).

Что добавлено (симметрично reviewer minor→follow-up)

QA при smoke находит НОВЫЙ баг (не тестируемый issue) → mcp__forgejo__create_issue: scope/X status/ready priority/pN bug, body = work-prompt (Задача/repro/Files/DoD) + screenshot/console → баг попадает воркеру в очередь. Проверка дубликатов. По ТЕСТИРУЕМОМУ issue — по-прежнему reopen+needs-fix (не дубль). prod_down spike-issue без изменений.

Файлы: auto-qa-tester.md (verdict + hard rule), work-as-qa.md (+ fix stale (10m)→(5m)).

⚠️ Трогает work-as-qa.md → self-extending guard: human merge.

## Запрос «Тестер создаёт задачи если находит баги?» — раньше НЕТ (только reopen тестируемого issue + needs-fix; новый/побочный баг терялся). ## Что добавлено (симметрично reviewer minor→follow-up) QA при smoke находит **НОВЫЙ баг** (не тестируемый issue) → `mcp__forgejo__create_issue`: `scope/X status/ready priority/pN bug`, body = work-prompt (Задача/repro/Files/DoD) + screenshot/console → баг попадает воркеру в очередь. Проверка дубликатов. По ТЕСТИРУЕМОМУ issue — по-прежнему reopen+needs-fix (не дубль). prod_down spike-issue без изменений. Файлы: `auto-qa-tester.md` (verdict + hard rule), `work-as-qa.md` (+ fix stale `(10m)`→`(5m)`). ⚠️ Трогает `work-as-qa.md` → self-extending guard: human merge.

bot-reviewer added 1 commit 2026-05-30 11:21:16 +00:00

feat(qa): tester заводит bug-issue на НОВЫЕ баги (не тестируемый issue) fe27ad9dce

Раньше qa по правилу НЕ создавал issues (только reopen тестируемого + needs-fix,
либо prod_down spike). Побочный/новый баг, замеченный при smoke, терялся.

Теперь (симметрично reviewer minor→follow-up): НОВЫЙ баг (не тестируемый issue) →
mcp__forgejo__create_issue: scope/X + status/ready + priority/pN + bug, body = work-prompt
(Задача/repro/Files/DoD) + screenshot/console → попадёт воркеру в очередь. Проверка
дубликатов. По ТЕСТИРУЕМОМУ issue — по-прежнему reopen+needs-fix (не дубль).

auto-qa-tester.md (verdict + hard rule) + work-as-qa.md (+ починил stale "(10m)"→"(5m)").

bot-reviewer commented

2026-05-30 11:23:41 +00:00

Author

Collaborator

🔴 BLOCKED — human merge required (self-extending guard).

This PR modifies bot pipeline / persona files, which the auto-code-reviewer is hard-prohibited from merging:

.claude/commands/work-as-qa.md — work-as-*.md persona file
.claude/agents/auto-qa-tester.md — autonomous bot agent definition

Additionally the PR author is bot-reviewer (the reviewer's own identity), so it cannot be self-merged in any case. The PR description already acknowledges this ("⚠️ Трогает work-as-qa.md → self-extending guard: human merge").

The change itself looks reasonable in scope (symmetric to reviewer minor→follow-up: QA opens a bug issue for new bugs found during smoke, keeps reopen+needs-fix for the tested issue, dedup check, fixes stale (10m)→(5m)). No code/SQL/secrets touched — only two .md files. But by policy a human must review and merge.

Applied labels: blocked + needs-human. Leaving open for human action.

🔴 **BLOCKED — human merge required (self-extending guard).** This PR modifies bot pipeline / persona files, which the auto-code-reviewer is hard-prohibited from merging: - `.claude/commands/work-as-qa.md` — `work-as-*.md` persona file - `.claude/agents/auto-qa-tester.md` — autonomous bot agent definition Additionally the PR author is `bot-reviewer` (the reviewer's own identity), so it cannot be self-merged in any case. The PR description already acknowledges this ("⚠️ Трогает `work-as-qa.md` → self-extending guard: human merge"). The change itself looks reasonable in scope (symmetric to reviewer minor→follow-up: QA opens a `bug` issue for *new* bugs found during smoke, keeps reopen+needs-fix for the tested issue, dedup check, fixes stale `(10m)`→`(5m)`). No code/SQL/secrets touched — only two `.md` files. But by policy a human must review and merge. Applied labels: `blocked` + `needs-human`. Leaving open for human action.