Skip to content

fix: ollama provider implementation - #282

Open
jdurand wants to merge 1 commit into
Robitx:mainfrom
jdurand:fix/ollama
Open

jdurand wants to merge 1 commit into
Robitx:mainfrom
jdurand:fix/ollama

Conversation

@jdurand

@jdurand jdurand commented Aug 23, 2025 •

Copy link
Copy Markdown

fixes ollama api endpoint and request format to properly work with native ollama api

changes:

  • refactored response parsing logic to skip empty/nil lines for all providers
  • fixed ollama endpoint path to use native api format
  • fixed request payload structure for ollama compatibility
  • added proper message parsing for ollama responses
  • removed unnecessary headers for ollama requests
  • fixed cursor position validation to prevent 'Cursor position outside buffer' errors

@jdurand jdurand changed the title fix ollama provider implementation fix: ollama provider implementation Aug 23, 2025
xianxu referenced this pull request in xianxu/parley.nvim Sep 26, 2026
xianxu referenced this pull request in xianxu/parley.nvim Sep 29, 2026
Tracker-Operation: claim-0902d1826d20
xianxu referenced this pull request in xianxu/parley.nvim Sep 29, 2026
Tracker-Operation: close-33b535642c70
xianxu referenced this pull request in xianxu/parley.nvim Sep 29, 2026
Tracker-Operation: close-967b91a76550
xianxu referenced this pull request in xianxu/parley.nvim Sep 29, 2026
#282: plan: one undo per answer — keep the receipt across no-op refusals
xianxu referenced this pull request in xianxu/parley.nvim Sep 29, 2026
Tracker-Operation: done-55483f0455db
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