Skip to content

fix: ollama connection permissions and add AI language setting#261

Open
M4lmostoso wants to merge 1 commit intoavihaymenahem:mainfrom
M4lmostoso:feature/ollama-fix-and-ai-language
Open

fix: ollama connection permissions and add AI language setting#261
M4lmostoso wants to merge 1 commit intoavihaymenahem:mainfrom
M4lmostoso:feature/ollama-fix-and-ai-language

Conversation

@M4lmostoso
Copy link
Copy Markdown

@M4lmostoso M4lmostoso commented Apr 27, 2026

Summary

Changes

Type of Change

  • Bug fix
  • New feature
  • Enhancement (improving existing feature)
  • Refactor (no behavior change)
  • Documentation
  • CI/Build

Testing

  • Existing tests pass (npm run test)
  • New tests added (if applicable)
  • Manually tested

Screenshots

@M4lmostoso
Copy link
Copy Markdown
Author

Description

This PR fixes connection issues with Ollama on Tauri v2 and introduces a new setting to choose the preferred AI response language.

Key Changes:

  1. HTTP Permissions Fix (Tauri v2): Added explicit scope for http://localhost:11434/* and http://127.0.0.1:11434/* in capabilities/default.json. This resolves the "url not allowed on the configured scope" error when connecting to a local Ollama instance.
  2. AI Language Setting: Added a new dropdown in the AI Settings tab to allow users to select their preferred language (Italian, French, German, etc.).
  3. Multilingual Prompts: Updated aiService.ts to automatically inject language instructions into key AI features such as summaries, replies, and task extraction.
  4. Ollama Provider Improvement: Enhanced the connection test logic in ollamaProvider.ts to be more flexible with different Ollama API response formats.

Testing

  • Verified that "Test Connection" with Ollama now passes correctly.
  • Confirmed that thread summaries are generated in the selected language (e.g., Italian).
  • Verified successful application build on macOS.

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