trigger.pyにコマンドエイリアスがほしい

[quojama]

trigger.py用のgspのtriggerカラムの右側にalias01, alias02のカラムを作って、コマンドごとにエイリアスを2つほど設定できるようにしたい。

@zztkm の読み込み減らすリファクタリングによって、私よくわからなくなってしまいました。

頭の片隅にでも覚えといてください。

[zztkm]

@quojama エイリアスは1個ではだめなんですか？

2つ以上の設定はむしろ覚えづらくさせてしまう気がしました（実装は難しくないので何個でもやれます

[quojama]

いや覚えさせると言うより「予想でコマンド打っても拾えるようにしたい」という感じかな。

今はdynmapのURLを出すコマンドは !!dynmap なんだけど、人によっては !!mcmap と打つ人もいるのではないか。

そういう感じです。

[zztkm]

OKです。

重複する値が出てくる可能性が増えるので以下のような参照順にする予定です。

1. triggerカラム
2. alias01 カラム
3. alias02 カラム

※ カラム内で重複データがあった場合はfind method の仕様に従って、最初に見つかったものを採用します。

[quojama]

わかりました。gsp側で1.2.3で重複は入力できないようにしようと思います

[zztkm]

機能仕様

まとめていく

trigger 検索処理

ソース: https://github.com/pistachiostudio/takohachi/blob/master/cogs/trigger.py#L38

検索順

1. triggerカラム
2. alias01 カラム
3. alias02 カラム

※ カラム内で重複データがあった場合はfind method の仕様に従って、最初に見つかったものを採用します。

Response用の値を取得する処理

ソース: https://github.com/pistachiostudio/takohachi/blob/master/cogs/trigger.py#L59-L65

値はヘッダー名が変更されないければ列の位置が変わっても正しく取得できるように実装しているので特に変更点はない想定(一応検証する