Skip to content

Conversation

@shravan097
Copy link
Contributor

Summary

Modified Ollama provider to disable tools in chat mode, enabling simple conversation-only interactions with local Ollama models.

Motivation

I needed a better UI for local Ollama chat without tools, preferring Goose's interface over the CLI for simple conversations while maintaining data privacy.

Screenshots

Screen.Recording.2025-07-08.at.4.43.04.PM.mov

Testing

manually tested . see screenshots above. change seems minimal for automated tests. happy to add them if needed.

@shravan097 shravan097 marked this pull request as ready for review July 9, 2025 00:13
@michaelneale michaelneale self-assigned this Jul 9, 2025
Copy link
Collaborator

@michaelneale michaelneale left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I like this - good idea, can merge when build passes

@shravan097
Copy link
Contributor Author

@michaelneale CI timed out due to network issue. would you be able to retrigger the workflow ?

@shravan097
Copy link
Contributor Author

shravan097 commented Jul 10, 2025

@michaelneale thank you for re-triggering the workflow. the build failed again and error does not make sense for my changes. build & test passes locally. tagging @Kvadratni
@Kvadratni i noticed your recent PR also had similar build error collect2: fatal error: ld terminated with signal 9 [Killed] compilation terminated. I am also facing similar build error for this PR.

@Kvadratni were you able to resolve this by retrying or there were more steps ?

@michaelneale
Copy link
Collaborator

@shravan097 sorry about this, if you update to main and push again, I think build should be better now, using too many resources (running out of CPU or space at times) - nearly there!

@shravan097 shravan097 force-pushed the allow-chat-only-ollama-models branch from 16cd738 to 65deb97 Compare July 10, 2025 06:02
@shravan097
Copy link
Contributor Author

@michaelneale no worries at all. i just rebased and pushed the changes. 🤞 hope this works. thank you again for reviewing this.

@lily-de lily-de merged commit 4ba991e into block:main Jul 11, 2025
7 checks passed
zanesq added a commit that referenced this pull request Jul 11, 2025
* 'main' of github.com:block/goose:
  feat: Allow Ollama for non-tool models for chat only (#3308)
michaelneale added a commit to jackjackbits/goose that referenced this pull request Jul 14, 2025
* main: (51 commits)
  docs: reflecting benefits of CLI providers (block#3399)
  feat: fetch openrouter supported models in `goose configure` (block#3347)
  Add the ability to configure rustyline to use a different edit mode (e.g. vi) (block#2769)
  docs: update CLI provider guide (block#3397)
  Streamable HTTP CLI flag (block#3394)
  docs: Show both remote options for extensions in CLI (block#3392)
  docs: fix YouTube Transcript MCP package manager (block#3390)
  docs: simplify alby mcp (block#3379)
  docs: add max turns (block#3372)
  feat(cli): add cost estimation per provider for Goose CLI (block#3330)
  feat: Allow Ollama for non-tool models for chat only (block#3308)
  [cli] Add --provider and --model CLI options to run command (block#3295)
  Docs: Lead/worker model in Goose Desktop (block#3342)
  revert: refactor: abstract keyring logic to better enable DI (block#3358)
  Drop temporal-service binary (block#3340)
  docs: add fuzzy search (block#3357)
  Fix name of GPT-4.1 System Prompt (block#3348) (block#3351)
  docs: add goose-mobile (block#3315)
  refactor: abstract keyring logic to better enable DI (block#3262)
  fix: correct tool use for anthropic (block#3311)
  ...
atarantino pushed a commit to atarantino/goose that referenced this pull request Jul 14, 2025
s-soroosh pushed a commit to s-soroosh/goose that referenced this pull request Jul 18, 2025
kwsantiago pushed a commit to kwsantiago/goose that referenced this pull request Jul 19, 2025
cbruyndoncx pushed a commit to cbruyndoncx/goose that referenced this pull request Jul 20, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants