整理: MetasStore に CoreManager を内蔵

[tarepan]

内容

概要: MetasStore に CoreManager を内蔵してリファクタリング

MetasStore はキャラクター情報の一部を管理するクラスである。一部とはエンジンに由来する部分である。
キャラクター情報の全部へアクセスするには core が必要であり、MetasStore.load_combined_metas() は CoreAdapter インスタンスを引数に受け取って情報統合しキャラクター情報全部を返している。
昨今リファクタリングが進み、新しく実装された CoreManager を用いたコアアクセスの単純化が可能になった。 MetasStore であれば init で CoreManager を渡し、情報取得時にコアバージョンのみ指定すればよくなる。

このような背景から、MetasStore に CoreManager を内蔵するリファクタリングを提案します。

関連 Issue

part of #1313

[Hiroshiba]

違和感ある依存に感じました！
MetasStoreが真に依存したいのはCoreのメタ情報だけなはずで、CoreManagerを抱えるのは過剰気味な気配を感じました。

コード追ってたのですが、load_combined_metasに渡すのはlist[CoreSpeaker]が良いかもです！（消されてるFIXMEコメントにも書いてありそう）
それにはまず_CoreSpeakerというどう考えてもコア側にあるべき概念をMetasStoreからCoreAdapter.speakers辺りに持っていくとスムーズかもです。

逆にそこまで進んだら割とリファクタリング方針は明確になって、router側でcore.metas.speakersを得てload_combined_metasに渡す感じになる気がします。
まあそうなってくるとload_combined_metasという名称は変かもなのですが。

ちょっと一旦コメントまで･･･！

[tarepan]

👍️
提案頂いた方向性で新たな PR を作成しました（#1352）。
#1352 を確認頂いて、そちらが良さそうであればこの PR は close とします。

[tarepan]

#1352 merge につき本 PR は close とします。