hayabusa
hayabusa copied to clipboard
Separate the contents of src/detections/rule/mod.rs into modules
src/detections/rule/mod.rsの中に、DetectionNodeやRuleNodeが混じって入っていて、カオスになってきている気がする。 mod構文とそれらが入り混じっていて、読みにくく、管理しにくい問題がある。
分けようと考えたけど、うまくモジュール構成を決められない。
@kazuminn ありがとうございます!コードの整理も大事ですね。 一旦モジュール構成を考えてみて、 @fukusuket と @hitenkoku に見て貰うのはどうですか? 何か悩んでいることがあれば、ここで共有して下さい。
@kazuminn ミーティングで話したところ、良い感じに分離できるのであれば、是非分離した方が良さそうなので、お願いしたいですが、他のissue(新機能等)と比べると優先度が低いという結論でした。と言いながら、気になって先にやっておきたいのであれば、全然良いです。 基本的にお任せします!
@fukusuket @hitenkoku 相談ですが、
src/detection/rule/utiles.rs
を作っちゃって、mod xxx以外をそこに突っ込むのは、どうでしょう? DetectionNodeとRuleNodeとaggresultとテストなどをそこに移動する感じです。
@kazuminn 良いと思います! あとAggresultをcount.rsあたりに移動できるのであれば、ファイル名もutiles.rsからrulenode.rsとかにできそうと思いました。(見当違いでしたらすみません!)
@fukusuket 私もrulenode.rsにする案で問題ないと思います。。