Skip to content

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

Draft
EmersonBraun wants to merge 1 commit into
mainfrom
codex/ollama-local-recipe
Draft

feat: add first-class local Ollama review#27
EmersonBraun wants to merge 1 commit into
mainfrom
codex/ollama-local-recipe

Conversation

@EmersonBraun

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 — 20 tests passed
  • npm pack --dry-run — 43 files
  • git diff --check

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

Comment thread test/ollama-cli.test.mjs
tool_calls: [{ function: { name: 'submit_findings', arguments: { findings: [] } } }],
},
done: false,
})}\n${JSON.stringify({ done: true, prompt_eval_count: 8, eval_count: 2 })}\n`)
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