Skip to content

feat: add first-class local Ollama review - #27

Merged
EmersonBraun merged 2 commits into
mainfrom
codex/ollama-local-recipe
Aug 11, 2026
Merged

feat: add first-class local Ollama review#27
EmersonBraun merged 2 commits into
mainfrom
codex/ollama-local-recipe

Conversation

@EmersonBraun

@EmersonBraun EmersonBraun commented Aug 9, 2026

Copy link
Copy Markdown
Member

What changed

  • adds native Ollama chat and tool-call handling
  • wires the local provider into the CLI
  • documents a bounded local-review recipe and operational safeguards
  • refreshes machine-readable documentation
  • adds adapter and end-to-end CLI tests using a synthetic local endpoint

Why

Teams can run code review with local inference instead of sending repository content to a hosted model provider. The recipe keeps model choice, file limits, concurrency, failure behavior, and network boundaries explicit.

Validation

  • npm run check — 21 tests passed
  • npm pack --dry-run — 43 files
  • git diff --check
  • real local smoke test with Ollama 0.21.0 and qwen3:4b confirmed model loading, stdin ingestion, and successful /api/chat calls without credentials or external source code

The real smoke test also showed that qwen3:4b is slow for the complete seven-lens sequential review. The documented qwen2.5-coder:7b recipe remains the recommended practical baseline; this is a model-performance constraint, not an adapter failure.

This PR does not include an upstream Ollama contribution, release, tag, or package publication.

Comment thread test/ollama-cli.test.mjs Fixed
@EmersonBraun
EmersonBraun force-pushed the codex/ollama-local-recipe branch from 6b819cd to 9e29405 Compare August 11, 2026 02:41
@EmersonBraun
EmersonBraun marked this pull request as ready for review August 11, 2026 11:42
@EmersonBraun
EmersonBraun merged commit 467cfa5 into main Aug 11, 2026
5 checks passed
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.

2 participants