Skip to content

chore(harness-omp): pin advisor model role in both presets#25

Merged
todie merged 2 commits into
mainfrom
chore/omp-advisor-role-pin
Jul 18, 2026
Merged

chore(harness-omp): pin advisor model role in both presets#25
todie merged 2 commits into
mainfrom
chore/omp-advisor-role-pin

Conversation

@todie

@todie todie commented Jul 18, 2026

Copy link
Copy Markdown
Member

Guardrail from the 2026-07-11..14 omp advisor 402 storm: the advisor model role resolved to a credit-dead paid provider (openrouter/openai/gpt-5.5) and burned one failed call per primary-agent turn (~1600 total) — omp's advisor has no circuit-breaker on provider failure.

  • config.local.yml: advisor: ollama/qwen3.5-9b-mtp (in-family smol)
  • config.gateway-slim.yml: advisor: llm/zai/GLM-5.2:low
  • preset.test.ts: strip the :level thinking suffix omp selectors carry before the declared-model lookup

Live ~/.omp/agent/config.yml already pinned (llm/neuralwatt/glm-5.2:low); engram #12404 records the incident.

https://claude.ai/code/session_016rDA5nEbELSsx6ojnyZdDG


View with Codesmith Autofix with Codesmith
Need help on this PR? Tag /codesmith with what you need. Autofix is disabled.

After the 2026-07-11..14 advisor 402 storm (advisor role resolved to a
credit-dead paid provider; one failed call per primary turn, no
circuit-breaker in omp), the advisor role is always pinned explicitly and
in-family: ollama smol for the local preset, gateway GLM-5.2:low for
gateway-slim. Preset test now strips the ':level' thinking suffix omp
selectors legitimately carry.

Claude-Session: https://claude.ai/code/session_016rDA5nEbELSsx6ojnyZdDG
@todie
todie force-pushed the chore/omp-advisor-role-pin branch from 58b13bd to fd670e0 Compare July 18, 2026 06:22
…(estate-orient)

The estate-orient skill landed in packages/skills without the preset
allowlists being updated — the sync test has been red on main since.
Unblocks CI for this PR; not related to the advisor pin.

Claude-Session: https://claude.ai/code/session_016rDA5nEbELSsx6ojnyZdDG
@todie
todie merged commit cb98058 into main Jul 18, 2026
2 checks passed
@todie
todie deleted the chore/omp-advisor-role-pin branch July 18, 2026 06:25
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant