Fix error message for forbidden error

[wataru86]

What

• publish コマンドで API が Forbidden を返した際のエラーメッセージを修正しました。

before	after

Why

• 記事ファイルに不備がある場合に API が Forbidden を返す可能性があるため。
  • 例
    • private を false から true に変更しようとしたとき
    • タグにスペースが含まれているとき

Refs

• [Qiita CLI] Qiita APIからの403 Forbidden エラーメッセージが不適切 qiita-discussions#561
• https://tc39.es/ecma262/multipage/fundamental-objects.html#sec-installerrorcause

[tomoasleep]

記事ファイルに不備がある場合に API が Forbidden を返す可能性があるため。

• 例
  • private を false から true に変更しようとしたとき
  • タグにスペースが含まれているとき

この例を見る感じ、クライアント側で対処するのではなく、そもそも API が返す status code が適切ではないのでは？ということを思いました。 (400 とか 422 とかを返すべきなのでは？ということを思いました。)

src/lib/error-handler.ts は、名前からして汎用的な処理で、 API のエラーに対して、汎用的なエラーメッセージを返す場所なので、 status code の意味と明らかに異なるメッセージを返すのは避けたいです。

(基本的には API 側で対処してほしいですが、) 一時的にクライアントで対応するにしても、汎用的なエラーメッセージの表示箇所ではなく、 command 単位でエラーメッセージの表示を変更するか (TODO コメント付きで)、クライアント validation などをするかしてください。

[wataru86]

@tomoasleep リマインドです。レビューお願いします 🙏

[tomoasleep]

• API 側の問題はいずれ対応予定
• 先にクライアント側のエラー通知を改善しておきたい

ということで、問題がある API を使用している箇所のみエラーメッセージを変更することにした。LGTM 👍

[tomoasleep]

メモ: xt0rted/block-autosquash-commits-action@v2 でテストがスキップされてるが、 public repo ではスキップは不要だと思う。