opencode-config/docs/handoff/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

16 lines
No EOL
705 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.

# PR: Fix reviewer agent echo bypass
## Что сделано
- Удалён `"echo *": allow` из .opencode/agents/reviewer.md bash allow-list (PR #25, closes #15)
- echo больше не может писать файлы (echo "x" > file.txt)
- Rule 10 становится accurate ("echo не в allow-list")
## Почему
Security: echo * + edit: deny = bypass risk. echo с перенаправлением пишет файл, обходя read-only контракт ревьюера.
## Pending
- Нет
## Watch out
- reviewer всё ещё имеет pwd, ls, cat для debug вывода
- Удаление echo не влияет на функциональность ревьюера