slaid098/opencode-config
Watch
1
0
Fork
You've already forked opencode-config
 0
Code Issues Pull requests Projects Releases Packages Wiki Activity Actions

chore(config): switch general and reviewer to glm-5.2 #120

Merged
slaid098 merged 1 commit from chore/config/glm52-general-reviewer into main 2026-08-21 02:40:39 +03:00
Conversation 1 Commits 1 Files changed 2 +6 -5
slaid098 commented 2026-08-21 02:37:19 +03:00
Owner
Copy link

Что сделано

  • .opencode/opencode.json: agent.general.model — ollama-cloud/deepseek-v4-flash → ollama-cloud/glm-5.2; agent.reviewer.model — ollama-cloud/kimi-k3 → ollama-cloud/glm-5.2. reasoningEffort: "max" сохранён у обоих.
  • tests/test_permissions.py, test_agent_models_exact: flash-кортеж теперь (explore, memory-syncer), ожидания general = ollama-cloud/glm-5.2, reviewer = ollama-cloud/glm-5.2.
  • grep -ri kimi .opencode/ — 0 совпадений.

Почему

Kimi K3 оказалась слишком тяжёлой (ресурсоёмкая) — откат. По решению пользователя практические задачи (исполнение + ревью) идут на GLM-5.2 (субъективно умнее flash, паритет по бенчмаркам 52 vs 53, но больший вес — осознанный трейд-офф), flash остаётся на лёгких операциях (explore, memory-syncer). Планирование — выбор модели из UI, не хардкодится.

Watch out

После merge нужен рестарт opencode-контейнера. GLM-5.2 дороже flash (~$0.44 vs $0.11 за задачу по Intelligence Index) — это осознанный выбор в пользу качества на execution/review. План на будущее: перевод на GLM-5.3, когда появится у провайдера (зафиксировано в памяти).

Pending

—

Closes #119

## Что сделано - `.opencode/opencode.json`: `agent.general.model` — `ollama-cloud/deepseek-v4-flash` → `ollama-cloud/glm-5.2`; `agent.reviewer.model` — `ollama-cloud/kimi-k3` → `ollama-cloud/glm-5.2`. `reasoningEffort: "max"` сохранён у обоих. - `tests/test_permissions.py`, `test_agent_models_exact`: flash-кортеж теперь `(explore, memory-syncer)`, ожидания `general = ollama-cloud/glm-5.2`, `reviewer = ollama-cloud/glm-5.2`. - `grep -ri kimi .opencode/` — 0 совпадений. ## Почему Kimi K3 оказалась слишком тяжёлой (ресурсоёмкая) — откат. По решению пользователя практические задачи (исполнение + ревью) идут на GLM-5.2 (субъективно умнее flash, паритет по бенчмаркам 52 vs 53, но больший вес — осознанный трейд-офф), flash остаётся на лёгких операциях (explore, memory-syncer). Планирование — выбор модели из UI, не хардкодится. ## Watch out После merge нужен рестарт opencode-контейнера. GLM-5.2 дороже flash (~$0.44 vs $0.11 за задачу по Intelligence Index) — это осознанный выбор в пользу качества на execution/review. План на будущее: перевод на GLM-5.3, когда появится у провайдера (зафиксировано в памяти). ## Pending — Closes #119
chore(config): switch general and reviewer to glm-5.2
All checks were successful
CI (always) / bootstrap (pull_request) Successful in 5s
Details
CI / bootstrap (pull_request) Successful in 8s
Details
Permission Security Check / check (pull_request) Successful in 9s
Details
CI / lint (pull_request) Successful in 35s
Details
CI / typecheck (pull_request) Successful in 36s
Details
CI / complexity (pull_request) Successful in 36s
Details
CI / test (3.13) (pull_request) Successful in 1m47s
Details
98d0d6fe0b
slaid098 commented 2026-08-21 02:40:06 +03:00
Author
Owner
Copy link

Code Review Summary

Атомарный paired-update: переключение general и reviewer на ollama-cloud/glm-5.2 в opencode.json синхронно с обновлением test_agent_models_exact в tests/test_permissions.py. Окна сломанного main нет — writer (config) и reader (test) меняются одним PR. CI зелёный.

Positives

  • Paired update в одном PR — opencode.json:417,436 + test_permissions.py:410-419 меняются вместе, тест отражает новую реальность (flash-кортеж теперь (explore, memory-syncer), glm-5.2-кортеж (general, reviewer)).
  • Docstring теста обновлён (test_permissions.py:413) — описание соответствует новым ожиданиям, не осталось вводящего в заблуждение "kimi-k3 for reviewer".
  • reasoningEffort: "max" сохранён у обоих агентов — поведенческий контракт не деградировал.
  • Cross-file sweep чист: grep -rE 'kimi-k3|deepseek-v4-flash|glm-5.2|minimax-m3' по .md/.json/.py/.ts — все оставшиеся ссылки согласованы (vision.md:85 про minimax-m3 не затронут; docs/decisions/083 и handoff — исторические ADR, содержат «glm-5.2» как контекст, не как конфиг).
  • PR body: 4 секции заполнены осмысленно, Closes #119 есть, Watch out честно фиксирует трейд-офф по стоимости и необходимость рестарта контейнера.
  • Заголовок conventional: chore(config): switch general and reviewer to glm-5.2 — в лимите.

Suggestions (info, not blocking)

  • tests/test_permissions.py:416-418 [style] Можно было бы параметризовать через pytest.mark.parametrize для единообразия, но текущий loop по кортежу читается не хуже — не блокирует.
  • opencode.json:413,454 [info] explore и memory-syncer остаются на flash — это соответствует описанию в PR body («лёгкие операции»), контракт не нарушен.

Verdict: APPROVE

## Code Review Summary Атомарный paired-update: переключение `general` и `reviewer` на `ollama-cloud/glm-5.2` в `opencode.json` синхронно с обновлением `test_agent_models_exact` в `tests/test_permissions.py`. Окна сломанного main нет — writer (config) и reader (test) меняются одним PR. CI зелёный. ### Positives - **Paired update в одном PR** — `opencode.json:417,436` + `test_permissions.py:410-419` меняются вместе, тест отражает новую реальность (flash-кортеж теперь `(explore, memory-syncer)`, glm-5.2-кортеж `(general, reviewer)`). - **Docstring теста обновлён** (`test_permissions.py:413`) — описание соответствует новым ожиданиям, не осталось вводящего в заблуждение "kimi-k3 for reviewer". - **`reasoningEffort: "max"` сохранён** у обоих агентов — поведенческий контракт не деградировал. - **Cross-file sweep чист**: `grep -rE 'kimi-k3|deepseek-v4-flash|glm-5.2|minimax-m3'` по `.md/.json/.py/.ts` — все оставшиеся ссылки согласованы (vision.md:85 про minimax-m3 не затронут; docs/decisions/083 и handoff — исторические ADR, содержат «glm-5.2» как контекст, не как конфиг). - **PR body**: 4 секции заполнены осмысленно, `Closes #119` есть, Watch out честно фиксирует трейд-офф по стоимости и необходимость рестарта контейнера. - **Заголовок conventional**: `chore(config): switch general and reviewer to glm-5.2` — в лимите. ### Suggestions (info, not blocking) - **tests/test_permissions.py:416-418** [style] Можно было бы параметризовать через `pytest.mark.parametrize` для единообразия, но текущий loop по кортежу читается не хуже — не блокирует. - **opencode.json:413,454** [info] `explore` и `memory-syncer` остаются на flash — это соответствует описанию в PR body («лёгкие операции»), контракт не нарушен. ### Verdict: APPROVE
slaid098 deleted branch chore/config/glm52-general-reviewer 2026-08-21 02:40:40 +03:00
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
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!120
No description provided.