feat(stage-engine): embed verbatim reviewer/tester findings in rollback task_desc (ORCH-046) #43

Merged

admin merged 7 commits from feature/ORCH-046-stage-engine-pass-reviewer-tes into main 2026-06-06 07:54:04 +03:00

Conversation 0 Commits 7 Files Changed 16 +1609 -6

admin commented 2026-06-06 07:42:53 +03:00

Owner

Copy Link

Summary

• При заворотах на development task_desc несёт дословный must-fix текст (P0/P1 ревьюера, причина FAIL тестера) + ссылку на файл, а не только ссылку — устраняет «испорченный телефон», экономит retry-бюджет/токены общего инстанса.
• Новый defensive-модуль src/review_parse.py (never-raise): extract_review_findings (P0/P1 из 12-review.md), extract_test_failures (фрагмент 13-test-report.md: pytest output / FAIL-строки / Итог), усечение по лимиту.
• Две rollback-ветки src/stage_engine.py встраивают текст + сохраняют ссылку; graceful-фоллбэк при битом/пустом артефакте. Последовательность отката, retry-счётчик, поля AdvanceResult, реестр QG_CHECKS не менялись.
• Доки в том же PR: docs/architecture/README.md (Stage Engine / Откаты), CHANGELOG.md. ADR: docs/work-items/ORCH-046/06-adr/ADR-001-embed-findings-in-task-desc.md.

Test plan

• python -m pytest tests/ -q — 461 passed
• tests/test_review_parse.py — TC-01..08
• tests/test_stage_engine.py::TestRollbackTaskDescEmbedding — TC-09..12

Refs: ORCH-046

🤖 Generated with Claude Code

## Summary - При заворотах на `development` `task_desc` несёт дословный must-fix текст (P0/P1 ревьюера, причина FAIL тестера) + ссылку на файл, а не только ссылку — устраняет «испорченный телефон», экономит retry-бюджет/токены общего инстанса. - Новый defensive-модуль `src/review_parse.py` (never-raise): `extract_review_findings` (P0/P1 из `12-review.md`), `extract_test_failures` (фрагмент `13-test-report.md`: pytest output / FAIL-строки / Итог), усечение по лимиту. - Две rollback-ветки `src/stage_engine.py` встраивают текст + сохраняют ссылку; graceful-фоллбэк при битом/пустом артефакте. Последовательность отката, retry-счётчик, поля `AdvanceResult`, реестр `QG_CHECKS` не менялись. - Доки в том же PR: `docs/architecture/README.md` (Stage Engine / Откаты), `CHANGELOG.md`. ADR: `docs/work-items/ORCH-046/06-adr/ADR-001-embed-findings-in-task-desc.md`. ## Test plan - [x] `python -m pytest tests/ -q` — 461 passed - [x] `tests/test_review_parse.py` — TC-01..08 - [x] `tests/test_stage_engine.py::TestRollbackTaskDescEmbedding` — TC-09..12 Refs: ORCH-046 🤖 Generated with [Claude Code](https://claude.com/claude-code)

admin added 4 commits 2026-06-06 07:42:53 +03:00

docs: init ORCH-046 business request 50a3c60b0e

analyst(ET): auto-commit from analyst run_id=139 c7bca51d4b

architect(ET): auto-commit from architect run_id=140 29e83341b5

feat(stage-engine): embed verbatim reviewer/tester findings in rollback task_desc ... 03c3d77cac

При заворотах на development task_desc теперь несёт дословный must-fix текст
(P0/P1 ревьюера, причина FAIL тестера) вместо одной ссылки на файл — developer-
агент видит суть претензий сразу и не повторяет ту же ошибку, экономя retry-
бюджет и токены общего инстанса.

- Новый defensive-модуль src/review_parse.py (never-raise): extract_review_findings
  (P0/P1 из 12-review.md ## Findings), extract_test_failures (фрагмент тела
  13-test-report.md: pytest output / FAIL-строки / Итог), усечение по лимиту.
- Две rollback-ветки stage_engine: встраивают текст + сохраняют ссылку на полный
  файл; graceful-фоллбэк на ссылку-строку при битом/пустом артефакте.
- Последовательность отката, retry-счётчик, поля AdvanceResult, реестр QG_CHECKS
  не менялись.
- Доки: README (Stage Engine / Откаты), CHANGELOG.
- Тесты: tests/test_review_parse.py, test_stage_engine.py::TestRollbackTaskDescEmbedding.

Refs: ORCH-046

Co-Authored-By: Claude Opus 4.7 <noreply@anthropic.com>

admin added 1 commit 2026-06-06 07:45:00 +03:00

reviewer(ET): auto-commit from reviewer run_id=142 4cd55063b4

admin added 1 commit 2026-06-06 07:46:31 +03:00

tester(ET): auto-commit from tester run_id=143 be10becae2

admin added 1 commit 2026-06-06 07:49:38 +03:00

deployer(ET): auto-commit from deployer run_id=144 13b7df06b1

admin merged commit 0bc2398462 into main 2026-06-06 07:54:04 +03:00

admin referenced this issue from a commit 2026-06-06 07:54:05 +03:00

feat(stage_engine): ORCH-046 embed reviewer/tester findings in task_desc (#43)

Sign in to join this conversation.

Reviewers

No Reviewers

Labels

No items

No Label

Milestone

No items

No Milestone

Projects

Clear projects

No project

Assignees

Clear assignees

admin

No Assignees

1 Participants

Notifications

Subscribe

Due Date

No due date set.

Dependencies

No dependencies set.

Reference: admin/orchestrator#43