整理: 合成系の HTTPException をドメインから移動

[tarepan]

内容

概要: 合成系の HTTPException をドメインから移動してリファクタリング

Sing 機能はリリース優先で実装された経緯があり、現在の ENGINE の実装規約に則っていない箇所が複数ある。
HTTPException のドメインミスマッチはその一例であり、現在でもその状況が続いている。
Sing に閉じたエラーであるため、Sing 用エラーを定義しこれを router でキャッチすれば最小実装でドメインから HTTPException を分離できる。

また CancellableEngine はごく初期からの実装であり、HTTPException のドメイン分離規約がない時代の実装が残っている。これもキャンセル可能合成に閉じたエラーであるため、専用エラーと router キャッチで分離が可能である。

このような背景から、合成系の HTTPException をドメインから移動するリファクタリングを提案します。

関連 Issue

無し

[Hiroshiba]

Error忘れはたぶんそうな気がするのでそこだけ！

[tarepan]

@Hiroshiba
全指摘箇所の反映・テストパスを確認しました。Re-review よろしくお願いします。

[Hiroshiba]

TTSEngineという名前が良い感じになったら、良い感じになった名前InputErrorが良さそうなのかなとか思いました！

まあ、VOICEVOX ENGINEでEngineという名前が使われてるので、現TTSEngineはEngineと呼ばないなにか良い名前があると良いのかもですねぇ。。

[Hiroshiba]

LGTM！！