not working anymore with Azure OpenAI

[behrica]

The latest version of bosquet does not work with Azure OpenAI any more.
(it did work in the past)

There are various issue with passing "impl", "keys" and so on.

[behrica]

I think we need to do some "cleanup" as well.
I would first suggest to remove the get-api-key function:

bosquet/src/bosquet/openai.clj

Line 14 in 4c3de43

(defn- get-api-key

net.clojars.wkok/openai-clojure already has 2 documented ways to "setup" the API keys.
Either by setting:
OPENAI_API_KEY or
AZURE_OPENAI_API_KEY

or by passing the API keys in "opts" api/create-completion.

I feel that bosquet should not have any proper code to handle the API keys.

It should just make sure, that both can be used by the user of bosquet:

1. Regarding usage of environment vars, it should "do nothing", just document it
2. it should make sure that API keys can be passed inside a template, if the user wants to do so, like this:

(def template
  "
....
{% llm-generate model=text-davinci-003 var-name=play 
   api-key=xxxxx
   }
   ")

[zmedelis]

This definitely needs fixing. Though I am not sure about adding keys to template. Templates are code, they go to git repo, they are being deployed to different servers with possibly different keys. Secrets do not belong in code.

Just as net.clojars.wkok/openai-clojure relies on env vars to get the keys so does Bosquet can do the same. It is up to the user to make sure that all the keys are exported.

[behrica]

"keys in template" is not secure, true.
Thinking about "re-use" of templates,
some other "pieces" should then not be in the template neither.
'model' for example. (this will be different for each user of a template)

[behrica]

solved by PR #15