fix: revert `ollama-js` implementation with `fetch`

fmaclen / hollama

A minimal web-UI for talking to Ollama servers

https://hollama.fernando.is

MIT License

316 stars 27 forks source link

fix: revert `ollama-js` implementation with `fetch` #156

Closed fmaclen closed 3 weeks ago

fmaclen commented 3 weeks ago

Replaces ollama-js with fetch() using the old implementation for abortController
We are keeping ollama-js as a source for type definitinos
Replaces custom type definitions OllamaModel, OllamaTagResponse with import type { ListResponse } from 'ollama/browser';

GregoMac1 commented 3 weeks ago

This looks amazing!

I've tested this branch and the only bug that I find is the one we've talked about here.

I think we could merge this into #126, fix that problem and then we're ready to go!

GregoMac1 commented 3 weeks ago

Now that I think about it again, we should find a way to handle the behaviour of the previous isWaitingForResponse. At beforeNavigate, we should be able to detect if a request has been sent to /chat and completion is still empty. In that case, I think it should throw the alert and the user would have to confirm and abort the response, right?