fix: Transformer による実装を変換処理のチェーンへと変更 #374
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
|
🎉 This PR is included in version 1.22.0 🎉 The release is available on GitHub release Your semantic-release bot 📦🚀 |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Close #372.
Type of Change:
コードの修正
Cause of the Problem (問題の原因)
tuple.tsでは, 反変関手の unzip 処理として(bs: [B1, B2]) => Promise<void>を[(b1: B1) => Promise<void>, (b2: B2) => Promise<void>]へと変換するsplit関数が存在していました. しかしこの実装では, 関数の引数を片方づつ受け取って溜め込み, 両方が揃ったら送り出すという実装になっていました. このために実行タイミングによっては正しく動作しない可能性があり, 再現性がないバグの原因となっていました.Dealing with Problems (問題への対処)
そもそも反変関手の性質上 unzip 処理は不可能であるため, 根本的に複雑過ぎた変換処理を見直すことにしました. メッセージハンドラと同じ形の
Promiseを返すだけの関数を用いて, その関数が拒否された場合はメッセージの変換処理もといメッセージへの応答を中断する方針で実装することにしました.Details of implementation (実施内容)
middleware.tsを追加し, メッセージMをメッセージNへ変換する処理Middleware<M, N>を追加しました。これは単純なPromiseチェーンによって結合され、これまでのボットのフィルタやコマンド向けの変換処理も同様に実装できます.これによって, 「作成されたメッセージへの変換処理」を「更新されたメッセージへの変換処理」へと加工する
liftTuple関数も用意に実装できました. そのためこちらはmiddleware.ts内へと移動しています.