土曜日隣で直接レビューしていたのでcommitしました。

DMなど、本文以外の投稿オプションで字数制限が変化するケース¶

土曜日、口頭で

PostBoxがDiva::Modelなら、remain_charcount[world, postbox] のような構文で残り文字数を取得できると良さそう

と言いましたが、よく考えると投稿する手段はPostBoxだけじゃないし、World系プラグインがguiに依存することになるのでよくありませんね。

compose spellと全く同じ引数を渡すことにしたら、どのような場合でも常に判断材料が揃うのではないかなと思いました。
残り文字数を知りたい場所では、compose spellを呼ぶはずですから、そのようなルールならわかりやすいと思います。

本題の前に、patch 0001にてURLを含む文字列を入力した際にスタックオーバーフローを起こしてしまう問題がありましたので、patch 0002にて対応しました。
メソッドの移動を行った際に、名前がかぶっていることに気づいていませんでした...

compose spellと全く同じ引数を渡すことにしたら、どのような場合でも常に判断材料が揃うのではないかなと思いました。

とても良いと思います。そのような形で対応しましたので、patch 0003をご確認ください。

対応にあたってPostBox内のcompose呼び出しを探していたところ、remain_charcountの呼び出し側でフッタを差し込むかのフラグの考慮が漏れていることが分かりました。
そのため、双方のオプションを統一できるような形にしています。

topic/1250-remain-charcount ブランチにcommitしました。こっちで試してるぶんには問題ないかなと思います。そのまま当てただけですが、もう一度確認お願いします。

確認しました。問題ありません。

機能トラッカーには レビュー待ち → マージ待ち の状態遷移がないようですので、ステータスはそのままにしておきます。

mergeしました