Add "Search Only" to OpenAI model options and make OpenAI API key input optional
#1
by
shinichi-a
- opened
In this pull request, the following changes have been made:
Added "Search Only" to the options for the OpenAI model. This allows users to perform only a search without using the OpenAI model.
Made the input of the OpenAI API key optional. This allows the search to be executed even if the OpenAI API key is not entered.
These changes allow users to perform only a search without using the OpenAI model and to execute a search even if the OpenAI API key is not entered, improving user convenience.
Please review and let me know your thoughts.
PRありがとうございます。とりわけ困っていないので、merge する予定はないのですが、（直して欲しいという意味ではなく）いくつかコメント、フィードバックいたしますね。
- 「OpenAI API Key」は任意で、初期状態の入力していない状態では通常の検索のみという挙動です。そのため、このPRの実装が入らずとも、あまり困らないのかなと思っています。
- 察すると、envのOpenAI key に値を入れている環境で、OpenAI での生成をしたくない、ということに思われますが、もしそのような挙動をしたい場合、オプションで「検索のみ」といった項目を用意して、その場合OpenAI関連のフォーム項目を全てdisabledにするようなUIが望ましく感じます。
- OPENAI_MODEL_NAMES に OPENAI_MODEL_NAMES 以外の文字列を入れているため、本来の目的と異なる定数となってしまっています。
- 私が書いたコメントの削除や私のフォーマットのスタイル変更(blackではないフォーマッタを使っている?)など、本筋とは関係ない変更点が多いので、一般的にはmergeされにくいPull Request なのかな、と思います。私もよっぽどのメリットがないと、mergeしないであろう変更点が多く含まれています。