c-3lab / dim

📦 dim: Manage the open data in your project like a package manager.
MIT License
126 stars 21 forks source link

Feature/show openai error message #113

Closed minheibis closed 1 year ago

minheibis commented 1 year ago

issue #112 への対応

libs/openai_client.ts 実装自体はこちらの変更で、openai のエラーの際のレスポンスを表示するようにしています。

tests/libs/actions.generate.test.ts に"handles error when OpenAI API returns an error"というテストを追加したのですが、テストの書き方がこれで良かったのかわからないので、見ていただけるとありがたいです。

codecov[bot] commented 1 year ago

Codecov Report

Merging #113 (c044cdf) into main (33da6a0) will not change coverage. The diff coverage is 100.00%.

@@            Coverage Diff            @@
##              main      #113   +/-   ##
=========================================
  Coverage   100.00%   100.00%           
=========================================
  Files           20        20           
  Lines         1392      1429   +37     
  Branches       146       147    +1     
=========================================
+ Hits          1392      1429   +37     
Files Changed Coverage Δ
deps.ts 100.00% <100.00%> (ø)
libs/openai_client.ts 100.00% <100.00%> (ø)
tests/helper.ts 100.00% <100.00%> (ø)
minheibis commented 1 year ago

エラーの場合、以下のように表示されます

deno run -A dim.ts generate -t "tests/test_data/valid_csv.csv" "Python code that converts this csv data to json"
⠦ Generate a code ...
error message by ky client: 
Request failed with status code 401 Unauthorized

error response by openai:
 {
  "error": {
    "message": "Incorrect API key provided: aaa. You can find your API key at https://platform.openai.com/account/api-keys.",
    "type": "invalid_request_error",
    "param": null,
    "code": "invalid_api_key"
  }
}
ryo-ma commented 1 year ago

@minheibis ありがとうございます。大方問題なさそうです。少し細かい点で指摘させていただきましたのでご確認ください。

minheibis commented 1 year ago

@ryo-ma ご確認ありがとうございます、エラーメッセージの説明について、大文字から始まるように修正しました。

ryo-ma commented 1 year ago

@minheibis ありがとうございます。問題なさそうなのでマージしていきます。 またソーシャルハックデーでも一緒に開発しましょう。お待ちしております。

minheibis commented 1 year ago

早速マージありがとうございます! はい、ぜひよろしくお願いします!