jackMort / jackMort/ChatGPT.nvim

Make it compatible with deepinfra

Open
#438 0 comments 0 reactions 0 assignees View on GitHub
enhancement good first issue
Dominant language
Lua
Stars
4k
Forks
317
PR merge metrics
No merged PRs in 30d

Description

I am trying to use thsi plugin with deepinfra which offers llama3b-70b on the cheap and a openAI compatible api, how ever its api endpoints has "openai" in the middle like this

https://api.deepinfra.com/v1/openai/chat/completions

And current code hard-coded to add `/v1/chat/completions` path to api base address, and this makes it not able to make api calls correctly to deepinfra api endpoint.

Would it be possible to make "v1" part of api base url? I know this is a breaking change, but it does not make a lot of sense to hard-code an api version in request paths, right?

Contributor guide

No contributing guide indexed for this repository

Research direction

Start by locating the request code that hard-codes `/v1/chat/completions` onto the API base address. Check how the base URL is configured and verify that allowing the versioned path in the base address works with the DeepInfra endpoint without breaking existing API calls.

Written by the indexing model from the issue text.

Assessment

Tech stack
lua
Domain
api
Issue type
Feature
Difficulty
3/5
Estimated time
1-2 days
Activity status
Stale
Clarity
Mostly clear
Newbie friendliness
45/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.