chore(config): remove scout agent #117

Merged

slaid098 merged 2 commits from chore/config/remove-scout-agent into main 2026-08-21 00:12:00 +03:00

Conversation 1 Commits 2 Files changed 4 +8 -15

slaid098 commented 2026-08-21 00:04:37 +03:00

Owner

Copy link

Что сделано

• Удалён блок agent.scout из .opencode/opencode.json.
• Из permission.task у general, reviewer, vision убрано "scout": "allow" — осталось {"*": "deny", "explore": "allow", "vision": "allow"}.
• Обновлены тесты tests/test_permissions.py: кортежи агентов без "scout" в test_agents_have_models, test_agent_models_exact, test_agents_have_reasoning_effort_max; expected-словарь в test_general_reviewer_task_permissions.
• Убраны упоминания scout из промпта agents/vision.md и примера в skills/configure-opencode/SKILL.md.
• grep -ri scout по репо — 0 совпадений.

Почему

scout — конфиг-агент без промпт-файла, бесполезен, пользователь просит удалить.

Watch out

Модели агентов в этом PR не меняются (смена reviewer на kimi-k3 — отдельный PR #116).

Pending

—

Closes #115

## Что сделано - Удалён блок `agent.scout` из `.opencode/opencode.json`. - Из `permission.task` у `general`, `reviewer`, `vision` убрано `"scout": "allow"` — осталось `{"*": "deny", "explore": "allow", "vision": "allow"}`. - Обновлены тесты `tests/test_permissions.py`: кортежи агентов без `"scout"` в `test_agents_have_models`, `test_agent_models_exact`, `test_agents_have_reasoning_effort_max`; expected-словарь в `test_general_reviewer_task_permissions`. - Убраны упоминания scout из промпта `agents/vision.md` и примера в `skills/configure-opencode/SKILL.md`. - `grep -ri scout` по репо — 0 совпадений. ## Почему scout — конфиг-агент без промпт-файла, бесполезен, пользователь просит удалить. ## Watch out Модели агентов в этом PR не меняются (смена reviewer на kimi-k3 — отдельный PR #116). ## Pending — Closes #115

slaid098 added 1 commit 2026-08-21 00:04:37 +03:00

chore(config): remove scout agent from opencode.json e217e3e50f

slaid098 commented 2026-08-21 00:08:02 +03:00

Author

Owner

Copy link

Code Review Summary

PR полностью удаляет агента scout: блок agent.scout в opencode.json, allow-правила permission.task у general/reviewer/vision, упоминания в vision.md и configure-opencode/SKILL.md, плюс синхронно обновлены тесты. Удаление чистое и полное — репо-скан подтверждает 0 остаточных ссылок, JSON валиден, тесты проходят, CI green.

Positives

• Полнота удаления: все 4 источника упоминаний обновлены; grep -i scout по репо (включая .opencode/) — 0 совпадений, заявление в PR body подтверждено.
• Paired update в одном PR: тесты (test_agents_have_models, test_agent_models_exact, test_agents_have_reasoning_effort_max, test_general_reviewer_task_permissions) обновлены вместе с конфигом — окно сломанного main закрыто.
• Документация синхронизирована: пример в configure-opencode/SKILL.md и промпт vision.md соответствуют новому allow-списку ({"*": "deny", "explore": "allow", "vision": "allow"}).
• Cross-file проверка чистая: check-permissions.py (glob *.md — scout файла не имел), pipeline-status.py (нет ссылок на scout), test_agent_frontmatter.py (проверяет 3 существующих файла) — readers не затронуты, missing paired update отсутствует.
• PR hygiene: title в conventional-формате, body с 4 осмысленными heading'ами, Closes #115.

Suggestions (info, not blocking)

• PR body, Watch out [info] Можно добавить напоминание, что после merge нужен рестарт opencode (агенты грузятся при старте) — стандартная процедура из configure-opencode skill, не блокирует.
• mergeable: false в Forgejo API [info] Флаг в API-ответе может быть устаревшим; diff против origin/main применяется чисто, конфликтов нет — при сомнении перепроверить перед merge.

Verdict: APPROVE

## Code Review Summary PR полностью удаляет агента scout: блок `agent.scout` в `opencode.json`, allow-правила `permission.task` у general/reviewer/vision, упоминания в `vision.md` и `configure-opencode/SKILL.md`, плюс синхронно обновлены тесты. Удаление чистое и полное — репо-скан подтверждает 0 остаточных ссылок, JSON валиден, тесты проходят, CI green. ### Positives - **Полнота удаления:** все 4 источника упоминаний обновлены; `grep -i scout` по репо (включая `.opencode/`) — 0 совпадений, заявление в PR body подтверждено. - **Paired update в одном PR:** тесты (`test_agents_have_models`, `test_agent_models_exact`, `test_agents_have_reasoning_effort_max`, `test_general_reviewer_task_permissions`) обновлены вместе с конфигом — окно сломанного main закрыто. - **Документация синхронизирована:** пример в `configure-opencode/SKILL.md` и промпт `vision.md` соответствуют новому allow-списку (`{"*": "deny", "explore": "allow", "vision": "allow"}`). - **Cross-file проверка чистая:** `check-permissions.py` (glob `*.md` — scout файла не имел), `pipeline-status.py` (нет ссылок на scout), `test_agent_frontmatter.py` (проверяет 3 существующих файла) — readers не затронуты, missing paired update отсутствует. - **PR hygiene:** title в conventional-формате, body с 4 осмысленными heading'ами, `Closes #115`. ### Suggestions (info, not blocking) - **PR body, Watch out** [info] Можно добавить напоминание, что после merge нужен рестарт opencode (агенты грузятся при старте) — стандартная процедура из configure-opencode skill, не блокирует. - **`mergeable: false` в Forgejo API** [info] Флаг в API-ответе может быть устаревшим; diff против `origin/main` применяется чисто, конфликтов нет — при сомнении перепроверить перед merge. ### Verdict: APPROVE

slaid098 force-pushed chore/config/remove-scout-agent from e217e3e50f to 2803e401bd 2026-08-21 00:09:45 +03:00 Compare

slaid098 merged commit 793727290e into main 2026-08-21 00:12:00 +03:00

slaid098 deleted branch chore/config/remove-scout-agent 2026-08-21 00:12:00 +03:00

slaid098 referenced this pull request from a commit 2026-08-21 00:12:01 +03:00

chore(config): remove scout agent (#117)

Sign in to join this conversation.

Reviewers

No reviewers

Labels

Clear labels   

bug

Something is not working

  

chore

  

config

  

enhancement

  

refactor

  

security

No labels

bug

chore

config

enhancement

refactor

security

Milestone

Clear milestone

No items

No milestone

Projects

Clear projects

No items

No project

Assignees

Clear assignees

No assignees

1 participant

Notifications

Subscribe

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

slaid098/opencode-config!117

No description provided.