なんか盛大にバグっていたので直しました。

ところで、本来は UserServiceRetriever#selectby をidnameでも動くようにオーバーライドしていやるのが正しい気がしますが、それをやると Service#user_initialize のとこで post.scan が no_mainthread に引っかかってこけました。というか、Userがnew_ifnecessaryのたびにfindbyidを呼んでるのがなんかおかしい気がします。

こちらでも確認したので、developに放り込んでおきました。
Retriever#new_ifnecessary は、全てのRetriever::Modelに問い合わせて見つからなかった場合は、新しいインスタンスを作るメソッドで、Retriever上のメモリキャッシュから読み出していますが、Userがオーバライドしている方ではそうなっていませんね。この変更が行われた 5b2463b30a26ae7050001b058e8a1cb1021a1311 を見てもこの振る舞いは正しいと思えませんでしたが、MessageではオーバライドしていないのにUserでだけオーバライドしているのには何らかの意図がある可能性もあるので、調査してみます。