整理: _speaker_info() を MetasStore へ移植

[tarepan]

内容

概要: _speaker_info() を MetasStore へ移植してリファクタリング

_speaker_info() は話者情報取得の様々な処理をおこなっている。
これは現在 router 下で定義されているが、本来は router でなくドメインで処理されるべきである。実際、_speaker_info() には移植の FIXME がすでに付与されている。
話者関連ドメインは Metas に集約されており、ここが移植先としてふさわしい。

このような背景から、_speaker_info() を MetasStore へ移植してリファクタリングすることを提案します。

Review 向け情報

root_dir 廃止と仕様変更

本 PR での router root_dir 引数廃止は事実上仕様変更ではない。以下詳細。

MetasStore() は初期化時に root_dir / "speaker_info" を話者ディレクトリとして引数で受け取っている。
一方、移植前の段階では router が引数として root_dir を受け取り、これを _speaker_info() 定義内から参照して root_dir / "speaker_info" パスを生成していた。
形式的には MetasStore の root_dir と router 引数の root_dir を個別に設定できる。しかし実際は MetasStore() も router も generate_app() 内でのみ利用されており、同じ root_dir を利用している。すなわち root_dir は事実上共通である。

本PR では _speaker_info を MetasStore 内に移植する際、_speaker_info() 内の router root_dir 参照を廃止して MetasStore init 側の root_dir 値を使っている。
これは形式的には仕様変更だが、上記の通り、実質的には仕様変更ではない。ゆえに router 引数廃止は無問題である。

関連 Issue

• step1 of refactor: speaker_info ディレクトリを resources/ へ移動 #1339
• part of メモ: Speaker/Metas 系リファクタリング方針・段階・進捗 #1313

[Hiroshiba]

ちょっと相談です！

今までMetasというのが特にドキュメントなどで範囲指定されておらず、キャラの名前とかスタイル名とか程度の文字情報だけ持ってた形でした。
このPRでアイコン等のリソース情報も抱える形になるのですが、その方針が良いか若干迷ってます！

･･･とはいえ名前くらいかもです。Metaというと文字列やobjectという印象があるくらいなので。
イメージとしては真のCharacterInfoになる感じでしょうか。
あるいはCharacterResource･･･？かCharacterMetaResource･･･？

（イメージ湧いてないだけでどれが良さそうか自分でもわかってない感じです）

[tarepan]

このPRでアイコン等のリソース情報も抱える形になるのですが、その方針が良いか

1つのクラスが概略情報と詳細情報（Speaker と SpeakerInfo）を提供するのは自然な形に感じます。

ご指摘の通り、Metas というフワフワ概念名が違和感の原因と感じます。
Metas という語は歴史的経緯 + コア API 名に由来するものなので、ENGINE 内から消えても違和感ありません。
MetasStore の実体は「キャラクター情報の保持と提供」を役割とするマネージャークラスであるため、CharacterManager がより適切な名称と考えます。
そして CharacterManager にリネームされれば本 PR の .speaker_info() 移植は自然に見えます。
（リネームする場合、リネームは後継 PR が適切そうです）

---

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

[Hiroshiba]

LGTM！！

おそらく @sabonerune さんのリソースURLのPRはこちらの変更に追従しないとかもです。
たぶんMetasStore（あとあとはCharacterManager？）にresource_managerを渡す感じになるのかなと！

[Hiroshiba]

うっ･･･　すみません。。_speaker_infoにコンフリクトが発生して、正しく解消できる自信ないです･･･
すみません、おまかせできると･･･ 🙇

[tarepan]

#1318 は変更範囲がデカそうなので、あちらを先に merge するのが丸そうです。
#1318 merge 確認後にコンフリクト解消してメンションします。

[tarepan]

@Hiroshiba
コンフリクト解消しました。マージ可能です。

[Hiroshiba]

LGTM！！

いくつかこちらで変えさせていただきます！