opencode-config/docs/decisions/005-pr-25-reviewer-echo-bypass.md
Sergey 0d79fe142b
fix: reviewer agent echo bypass (#25)
* fix(agents): remove echo allow from reviewer (bypass risk)

* docs(handoff): add pr-15 handoff + ADR-005

* docs(handoff): rename handoff/ADR to PR-25 number

* ci: add always-ci workflow for pipeline_status marker

---------

Co-authored-by: opencode-agent <agent@slaid098.dev>
2026-07-23 23:42:41 +03:00

14 lines
No EOL
755 B
Markdown
Raw Permalink Blame History

This file contains ambiguous Unicode characters

This file contains Unicode characters that might be confused with other characters. If you think that this is intentional, you can safely ignore this warning. Use the Escape button to reveal them.

# ADR-005: Remove echo allow from reviewer agent
## Статус
Accepted
## Контекст
reviewer.md имеет edit: deny (read-only контракт). Но "echo *": allow позволял echo "x" > file.txt — bypass write restriction.
## Решение
Удалить "echo *": allow из bash allow-list. Debug вывод через pwd/ls/cat (уже разрешены).
## Альтернативы
- Обновить Rule 10 вместо удаления — отклонено (soft guard в prompt менее надёжен чем deterministic permission guard)
- Оставить echo но запретить > — отклонено (bash permission matching не поддерживает гранулярные операторы)