Skip to content

fix(issue-manager): default OLLAMA_HOST_URL to localhost (#8)#21

Merged
canoo merged 1 commit intocanoo:mainfrom
mvanhorn:osc/8-localhost-default
Apr 23, 2026
Merged

fix(issue-manager): default OLLAMA_HOST_URL to localhost (#8)#21
canoo merged 1 commit intocanoo:mainfrom
mvanhorn:osc/8-localhost-default

Conversation

@mvanhorn
Copy link
Copy Markdown
Contributor

Closes #8.

Changes tools/utilities/issue-manager.sh:

  • Line 29 default: http://192.168.1.101:11434http://localhost:11434
  • Line 16 header comment updated to match

Users with a LAN Ollama instance can still set OLLAMA_HOST_URL as an env var.

The script previously defaulted to http://192.168.1.101:11434, which
exposed an internal LAN topology and diverged from every other script
in the repo that targets localhost. Default to http://localhost:11434
so the script runs out-of-the-box for contributors without the LAN
instance, and update the env-var comment in the header to match.
@canoo canoo self-requested a review April 23, 2026 08:37
@canoo canoo merged commit 6687021 into canoo:main Apr 23, 2026
3 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.

Replace hardcoded LAN IP with localhost in issue-manager.sh

2 participants