feat(config): switch reviewer model to kimi-k3 #118

Merged

slaid098 merged 2 commits from feat/config/reviewer-kimi-k3 into main 2026-08-21 00:22:32 +03:00

Conversation 1 Commits 2 Files changed 2 +6 -6

slaid098 commented 2026-08-21 00:15:31 +03:00

Owner

Copy link

Что сделано

• .opencode/opencode.json: agent.reviewer.model — ollama-cloud/deepseek-v4-pro → ollama-cloud/kimi-k3. reasoningEffort: "max" сохранён (kimi-k3 поддерживает effort).
• tests/test_permissions.py: test_agent_models_exact ожидает ollama-cloud/kimi-k3 для reviewer.

Почему

Пользователь хочет, чтобы код-ревью выполняла модель Kimi K3. Провайдер ollama-cloud (Ollama Cloud, ollama.com/v1) уже настроен (API-ключ в auth.json), модель kimi-k3 подтверждена live-списком /api/tags сервера и встроена в реестр opencode (reasoning c effort, tool_call, structured_output, контекст 1M) — декларация в provider.models не нужна, как у minimax-m3 (vision) и прежнего deepseek-v4-pro.

Watch out

После merge нужен рестарт opencode-контейнера, чтобы reviewer подхватил новую модель. При первом ревью проверь, что запросы к kimi-k3 проходят (если opencode не зарезолвит модель из реестра — добавить декларацию в provider.ollama-cloud.models по образцу deepseek-v4-flash:0731, это отдельный follow-up).

Pending

—

Closes #116

## Что сделано - `.opencode/opencode.json`: `agent.reviewer.model` — `ollama-cloud/deepseek-v4-pro` → `ollama-cloud/kimi-k3`. `reasoningEffort: "max"` сохранён (kimi-k3 поддерживает effort). - `tests/test_permissions.py`: `test_agent_models_exact` ожидает `ollama-cloud/kimi-k3` для reviewer. ## Почему Пользователь хочет, чтобы код-ревью выполняла модель Kimi K3. Провайдер `ollama-cloud` (Ollama Cloud, `ollama.com/v1`) уже настроен (API-ключ в auth.json), модель `kimi-k3` подтверждена live-списком `/api/tags` сервера и встроена в реестр opencode (reasoning c effort, tool_call, structured_output, контекст 1M) — декларация в `provider.models` не нужна, как у `minimax-m3` (vision) и прежнего `deepseek-v4-pro`. ## Watch out После merge нужен рестарт opencode-контейнера, чтобы reviewer подхватил новую модель. При первом ревью проверь, что запросы к kimi-k3 проходят (если opencode не зарезолвит модель из реестра — добавить декларацию в `provider.ollama-cloud.models` по образцу `deepseek-v4-flash:0731`, это отдельный follow-up). ## Pending — Closes #116

slaid098 added 1 commit 2026-08-21 00:15:31 +03:00

feat(config): switch reviewer model to kimi-k3 cd96977175

slaid098 added 1 commit 2026-08-21 00:19:07 +03:00

test(permissions): fix docstrings after scout removal dc170320bb

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

Author

Owner

Copy link

Code Review Summary

Смена модели reviewer-агента на ollama-cloud/kimi-k3 с синхронным обновлением теста test_agent_models_exact и docstring'ов (6→5 субагентов — соответствует фактическому набору explore/general/reviewer/memory-syncer/vision). Изменение минимальное, консистентное, paired update writer↔reader выполнен в одном PR.

Positives

• Paired update: opencode.json:436 (writer) и tests/test_permissions.py:417 (reader) обновлены совместно — окно сломанного main закрыто.
• Консистентность с паттерном: kimi-k3 не декларирована в provider.ollama-cloud.models — как minimax-m3 и прежний deepseek-v4-pro (встроены в реестр opencode). reasoningEffort: "max" сохранён, тест test_agents_have_reasoning_effort_max проходит.
• Docstring-фикс: «6 subagents» → «5 subagents» исправляет устаревшее описание (тест итерирует 5 имён).
• PR hygiene: conventional title, body с 4 heading'ами, Closes #116, Watch out описывает рестарт контейнера и fallback-план.
• Проверки: JSON валиден, pytest tests/test_permissions.py — 30 passed, CI green.

Suggestions (info, not blocking)

• .opencode/opencode.json:65 [info] Декларация deepseek/deepseek-v4-pro в провайдере deepseek больше нигде не используется (reviewer перешёл на ollama-cloud ещё раньше). Если это каталог для ручного выбора в UI — оставить; если нет — кандидат на удаление отдельным PR.
• tests/test_permissions.py:413 [style] Docstring test_agent_models_exact стал длиннее 79 символов (PEP8 E501) — можно перенести на две строки, но не блокирует.

Verdict: APPROVE

## Code Review Summary Смена модели reviewer-агента на `ollama-cloud/kimi-k3` с синхронным обновлением теста `test_agent_models_exact` и docstring'ов (6→5 субагентов — соответствует фактическому набору explore/general/reviewer/memory-syncer/vision). Изменение минимальное, консистентное, paired update writer↔reader выполнен в одном PR. ### Positives - **Paired update**: `opencode.json:436` (writer) и `tests/test_permissions.py:417` (reader) обновлены совместно — окно сломанного main закрыто. - **Консистентность с паттерном**: `kimi-k3` не декларирована в `provider.ollama-cloud.models` — как `minimax-m3` и прежний `deepseek-v4-pro` (встроены в реестр opencode). `reasoningEffort: "max"` сохранён, тест `test_agents_have_reasoning_effort_max` проходит. - **Docstring-фикс**: «6 subagents» → «5 subagents» исправляет устаревшее описание (тест итерирует 5 имён). - **PR hygiene**: conventional title, body с 4 heading'ами, Closes #116, Watch out описывает рестарт контейнера и fallback-план. - **Проверки**: JSON валиден, `pytest tests/test_permissions.py` — 30 passed, CI green. ### Suggestions (info, not blocking) - **.opencode/opencode.json:65** [info] Декларация `deepseek/deepseek-v4-pro` в провайдере `deepseek` больше нигде не используется (reviewer перешёл на ollama-cloud ещё раньше). Если это каталог для ручного выбора в UI — оставить; если нет — кандидат на удаление отдельным PR. - **tests/test_permissions.py:413** [style] Docstring `test_agent_models_exact` стал длиннее 79 символов (PEP8 E501) — можно перенести на две строки, но не блокирует. ### Verdict: APPROVE

slaid098 merged commit d37abd92b1 into main 2026-08-21 00:22:32 +03:00

slaid098 deleted branch feat/config/reviewer-kimi-k3 2026-08-21 00:22:32 +03:00

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

feat(config): switch reviewer model to kimi-k3 (#118)

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!118

No description provided.