Skip to content

Fix Ollama Deps#550

Merged
lbedner merged 1 commit intomainfrom
fix-ollama
Mar 21, 2026
Merged

Fix Ollama Deps#550
lbedner merged 1 commit intomainfrom
fix-ollama

Conversation

@lbedner
Copy link
Copy Markdown
Owner

@lbedner lbedner commented Mar 21, 2026

No description provided.

Copilot AI review requested due to automatic review settings March 21, 2026 01:40
Copy link
Copy Markdown

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

Updates the Aegis project template and docs to better handle optional AI/Ollama integration, while clarifying Overseer positioning in the README.

Changes:

  • Rename template flag usage from include_ai_service to include_ai in UI/environment templates.
  • Make Ollama integration in llm_sync_service.py optional via guarded imports and an early return when Ollama isn’t available.
  • Refresh README copy for Overseer and add a live demo link.

Reviewed changes

Copilot reviewed 4 out of 4 changed files in this pull request and generated 3 comments.

File Description
aegis/templates/copier-aegis-project/{{ project_slug }}/app/services/system/ui.py.jinja Updates AI-related conditional rendering to use include_ai.
aegis/templates/copier-aegis-project/{{ project_slug }}/app/services/ai/etl/llm_sync_service.py Makes Ollama imports optional and adds a guard in sync_ollama.
aegis/templates/copier-aegis-project/{{ project_slug }}/.env.deploy.example.jinja Updates AI service env section conditional to include_ai.
README.md Repositions Overseer messaging and adds a live demo link.

@lbedner lbedner merged commit 73d9c2f into main Mar 21, 2026
7 checks passed
@lbedner lbedner deleted the fix-ollama branch March 21, 2026 10:52
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.

2 participants