Skip to content

fix: align sandbox smoke with Ollama#525

Merged
cgfixit merged 1 commit into
mainfrom
codex/cyclaw-optimize-ollama-sandbox
Jul 13, 2026
Merged

fix: align sandbox smoke with Ollama#525
cgfixit merged 1 commit into
mainfrom
codex/cyclaw-optimize-ollama-sandbox

Conversation

@cgfixit

@cgfixit cgfixit commented Jul 13, 2026

Copy link
Copy Markdown
Owner

What

  • point the CyClaw sandbox mock at the current Ollama endpoint (127.0.0.1:11434/v1)
  • use the configured Ollama model id (qwen2.5:7b)
  • temporarily rewrite models.local_llm as well as Grok/Claude during the smoke run
  • update the health contract and sandbox guidance from lm_studio to ollama
  • add regression coverage for the local provider rewrite

Why

main migrated the runtime to Ollama, but the sandbox harness still mocked LM Studio on port 1234 and only rewrote external providers. That meant the smoke test could exercise a real local Ollama instance—or fail for environmental reasons—instead of the deterministic mock it claimed to verify.

Verification

  • python -m pytest tests/test_cyclaw_sandbox_skill.py -q --tb=short — 4 passed
  • ruff check --select E,F,I,B,C4,UP,S ... — clean
  • python -m py_compile ... — clean
  • invariant guard — 27 passed, 0 failed
  • dependency guard — 0 failures, 0 warnings
  • doc-sync guard — 0 drift
  • git diff --check — clean

Risk

Low. The changes are isolated to the repo-local sandbox skill, its mock, documentation references, and focused tests. config.yaml remains restored after each run.

Comment thread .codex/skills/cyclaw-sandbox-test/scripts/mock_lmstudio.py
Comment thread .codex/skills/cyclaw-sandbox-test/scripts/run_sandbox_test.py
@cgfixit cgfixit marked this pull request as ready for review July 13, 2026 21:03
@cgfixit cgfixit merged commit 81c92de into main Jul 13, 2026
33 checks passed
@cgfixit cgfixit deleted the codex/cyclaw-optimize-ollama-sandbox branch July 13, 2026 21:03
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.

3 participants