[PR #4] [MERGED] fix: support remote Ollama host via OLLAMA_HOST env var #8

New issue

Closed

opened 2026-03-03 12:01:31 +03:00 by kerem · 0 comments

kerem commented

2026-03-03 12:01:31 +03:00

Owner

📋 Pull Request Information

Original PR: https://github.com/ForLoopCodes/contextplus/pull/4
Author: @slegarraga
Created: 3/1/2026
Status: ✅ Merged
Merged: 3/1/2026
Merged by: @ForLoopCodes

Base: main ← Head: fix/remote-ollama-host

📝 Commits (1)

ba1644b fix: support remote Ollama host via OLLAMA_HOST env var

📊 Changes

2 files changed (+2 additions, -2 deletions)

View changed files

📝 src/core/embeddings.ts (+1 -1)
📝 src/tools/semantic-navigate.ts (+1 -1)

📄 Description

Summary

Passes process.env.OLLAMA_HOST to the Ollama client constructor in both src/core/embeddings.ts and src/tools/semantic-navigate.ts.

When OLLAMA_HOST is not set, the value is undefined and the Ollama client defaults to http://localhost:11434, preserving backward compatibility.

Changes

src/core/embeddings.ts: new Ollama() → new Ollama({ host: process.env.OLLAMA_HOST })
src/tools/semantic-navigate.ts: same change

Fixes #3

_{🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.}

## 📋 Pull Request Information **Original PR:** https://github.com/ForLoopCodes/contextplus/pull/4 **Author:** [@slegarraga](https://github.com/slegarraga) **Created:** 3/1/2026 **Status:** ✅ Merged **Merged:** 3/1/2026 **Merged by:** [@ForLoopCodes](https://github.com/ForLoopCodes) **Base:** `main` ← **Head:** `fix/remote-ollama-host` --- ### 📝 Commits (1) - [`ba1644b`](https://github.com/ForLoopCodes/contextplus/commit/ba1644bf9f1b53599b1a5514cce60a17c31cfc0c) fix: support remote Ollama host via OLLAMA_HOST env var ### 📊 Changes **2 files changed** (+2 additions, -2 deletions) <details> <summary>View changed files</summary> 📝 `src/core/embeddings.ts` (+1 -1) 📝 `src/tools/semantic-navigate.ts` (+1 -1) </details> ### 📄 Description ## Summary Passes `process.env.OLLAMA_HOST` to the Ollama client constructor in both `src/core/embeddings.ts` and `src/tools/semantic-navigate.ts`. When `OLLAMA_HOST` is not set, the value is `undefined` and the Ollama client defaults to `http://localhost:11434`, preserving backward compatibility. ## Changes - `src/core/embeddings.ts`: `new Ollama()` → `new Ollama({ host: process.env.OLLAMA_HOST })` - `src/tools/semantic-navigate.ts`: same change Fixes #3 --- <sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>