Skip to content

Support Ollama#60

Closed
conmale289 wants to merge 1 commit intocalesthio:masterfrom
conmale289:feature/support-ollama
Closed

Support Ollama#60
conmale289 wants to merge 1 commit intocalesthio:masterfrom
conmale289:feature/support-ollama

Conversation

@conmale289
Copy link
Copy Markdown

Summary

Describe what changed.

Why

Explain the problem being solved.

Scope

  • Focused bug fix
  • Small UX improvement
  • New source
  • Dashboard change
  • Docs/config change

Validation

List the commands, checks, or manual validation you performed.

Screenshots

If the dashboard or any visible output changed, add screenshots.

Config and Docs

  • No new environment variables
  • .env.example updated if needed
  • README.md updated if behavior changed

Source Additions

If this PR adds a new source, explain:

  • why the source improves signal quality
  • whether it requires an API key
  • how it degrades when the key is missing
  • what changed in apis/briefing.mjs and dashboard/inject.mjs

Checklist

  • This PR stays within one bugfix or one feature family
  • I kept unrelated changes out of the diff
  • I considered security for any mixed-source content rendering
  • I tested the changed path locally

@conmale289 conmale289 requested a review from calesthio as a code owner March 20, 2026 16:21
@The-R4V3N
Copy link
Copy Markdown
Collaborator

Hey @conmale289, thanks for working on this!
Just a heads up — I already have an open PR for
Ollama support in #38, which was submitted a
few days earlier and is linked to issue #30.

A few differences worth noting:

I also noticed this PR includes unrelated
localization changes (CRUCIX_LANG,
CRUCIX_TRANSLATE) in .env.example — those might
be better suited for a separate PR to keep the
scope focused.

Not trying to step on your toes at all — just
want to avoid duplicate effort! Maybe we can
coordinate or you could review #38 and share
any feedback?

@calesthio
Copy link
Copy Markdown
Owner

I haven't been able to get to this yet because work has been busy, but I definitely plan to review it over the weekend.

@conmale289
Copy link
Copy Markdown
Author

Thank you so much. I will close this pr

@conmale289 conmale289 closed this Mar 22, 2026
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.

3 participants