:@user:がメンションになってた

[ryoha000]

クライアント: traPtitech/traQ_S-UI#1167

[ryoha000]

何も調べずに書いちゃったんですが、Goには否定先読みなさそうですね
https://github.com/traPtitech/traQ/pull/964/files#diff-fa831d5efe640a73ad1f0aca3891024aL136
ここに「最後の文字が:ならreturn」みたいなことをかくのでいいですかね？

[wtks]

tbls.exe追加しちゃったコミット消してもらえませんかね

[sapphi-red]

含まれているだと@ryohaあああああああああ:みたいなパターンも置換されなくなって挙動が不自然になりそう

[sapphi-red]

[@＠]([a-zA-Z0-9_-]{1,32}):にマッチしてたらreturnするってすれば否定先読みと同値になるからそれが良いと思う
正規表現3回実行されるのがパフォーマンス的にどうかは少し気になるけど…

[sapphi-red]

@ryoha000

[Nagarei]

@ryoha:@ryoha: みたいなのもコーナーケースになりそう（前半は可能ならメンションになるべき）

[sapphi-red]

それは@ryoha:ってグループが存在したときに「@ryoha:っていうグループへのメンション+@ryoha:って文字列」になるのか、「@ryohaっていうメンション+:@ryoha:っていう文字列」になるのかが今のところたぶん未定義動作なので切り捨てって感じです(以前のコードの通りであれば、@ryoha:ってグループが存在していてもいなくても@ryoha:っていうメンション+@ryoha:って文字列になるはずです)

[sapphi-red]

@ryoha000 @の前に:が来ないってだけチェックすれば全部問題ない気がしてきた

[ryoha000]

clientもそうしたほうがいいかな？

[sapphi-red]

サーバーの方で書けたらクライアントにも全く同値で構文だけ変えたように実装してほしい:pray:

[ryoha000]

TBLSが何で落ちるかわかりません:pray:

[sapphi-red]

(今)落ちてるのは(tblsじゃなくて)テストだよ

[sapphi-red]

これっぽい

[sapphi-red]

ぼくは問題なさそうだと思います:+1:

/cc @wtks @motoki317