整理: Character 利用範囲を拡大

[tarepan]

内容

内部構造体 Character の利用範囲を拡大するリファクタリングを提案します。

• load_combined_metas() 戻り値
• filter_speakers_and_styles() 引数

を Character 化しました。
Speaker との界面がエンジン内部に未だ存在するため、キャストのためのユーティリティを追加しました。

関連 Issue

part of #1313

[Hiroshiba]

ほぼLGTMです！

ちょっとややこしくなってきた印象を受けました！

API用と内部用とで名称をちょっとだけ変えて扱いと対応させる方針は破綻してくる気がします。
（Manifest側の見てそう思いました）
話者情報関連はSpeakerとCharacterですが、初めて見たコミッターにとってだいぶややこしいだろうなと気づきました。

やってみないとわからなかった課題感に気づけてきたので、ちょっとissue作って解決策考える場を用意しても良いかも･･･？
ほんと良い前例があれば真似たいですね･･･。

[tarepan]

ちょっとややこしくなってきた印象 ... SpeakerとCharacterですが、初めて見たコミッターにとってだいぶややこしい

現在は内部の Speaker → Character 移行中であるため、かなり分かりづらいです。完全移行後には（移行前と比べて）かなり良くなるので、PR & レビューをガシガシ進めていければ/頂ければと思います。

API用と内部用とで名称をちょっとだけ変えて扱いと対応させる方針は破綻 ...
やってみないとわからなかった課題感に気づけてきたので、ちょっとissue作って解決策考える場を用意しても良いかも

👍️
課題の整理をする良いタイミングに感じます。
コード書いてる側だと理解できてしまうので、レビュアー側で感じた課題を @Hiroshiba さんが issue 化して頂ければと思います。

---

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

[Hiroshiba]

LGTM！！

こちらもSpeakerとCharacterの範囲がわかるとこまで一気に実装があると変更がわかりやすいかもです。
（可能なら移動だけ先or後に）