スレッドミュートにおいて、RN/引用/リアクションの通知もミュートするように | スレッドミュートのE2Eテスト修正

[Sayamame-beans]

What

スレッドミュートはこれまで、対象スレッドでの返信通知とメンション通知にのみ作用していましたが、リノート通知と引用通知、リアクション通知に対しても作用するようにしました。

なお、以下のケースでは通知がミュートされません。(ミュートを行っているユーザーをAとします)

• ミュート対象ノートを、当該スレッドの外にあるAへの返信/メンション付きノートにおいて引用する

これは、ミュートした意図と関係ない会話の中で引用した場合にミュートされてしまうことが無いようにするためです。
逆に言えば、そのようにすることでスレッドミュートを迂回して通知を発生させられてしまうということですが、それが悪質なユーザーによるものであれば、ユーザーのブロック/ミュート、及び通知受け取り範囲の調整が可能であるため問題ないと判断しました。

Why

fix: #13965

Additional info (optional)

• ユーザー側のWebhookにおいて、ユーザーミュートは作用していないという情報を耳にしました。
  スレッドミュートはWebhookにも適用される処理に(以前から)なっているのですが、これは適用しないべきなのでしょうか?
• ユーザーの「投稿を通知」において、返信も通知できるように #15269 で新規投稿通知が返信を通知するようになった場合、スレッドミュートはそこにも適用する必要が生じます。
  (現状は返信が含まれないため、スレッドミュートを適用する必要のある箇所がありません。)
• 本仕様変更に関するテストの追加~が出来ていません 追加しました。これで良いか若干自信が無いですが…

Checklist

• Read the contribution guide
• Test working in a local environment
• (If needed) Add story of storybook
• (If needed) Update CHANGELOG.md
• (If possible) Add tests

[Sayamame-beans]

タイトルを変え忘れたので修正しました…

[github-actions]

このPRによるapi.jsonの差分
差分はありません。
Get diff files from Workflow Page

[codecov]

Codecov Report

❌ Patch coverage is 11.42857% with 31 lines in your changes missing coverage. Please review.
✅ Project coverage is 62.41%. Comparing base (1391269) to head (d898d79).

Files with missing lines	Patch %	Lines
packages/backend/src/core/NoteCreateService.ts	0.00%	18 Missing ⚠️
packages/backend/src/core/ReactionService.ts	23.52%	13 Missing ⚠️

Additional details and impacted files

@@             Coverage Diff             @@
##           develop   #15271      +/-   ##
===========================================
- Coverage    62.42%   62.41%   -0.02%     
===========================================
  Files         1162     1162              
  Lines       116557   116581      +24     
  Branches      9079     9072       -7     
===========================================
- Hits         72764    72760       -4     
- Misses       41601    41612      +11     
- Partials      2192     2209      +17     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:

• ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
• 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

[Sayamame-beans]

conflictを解消

[Sayamame-beans]

test更新したんですが、何故failedしたのか謎…? (failした箇所と同じ操作をローカル環境で行う分には正常に動いているように見える)
テストの組み方ミスしてるかも…?

[Sayamame-beans]

test通りました

[Sayamame-beans]

fixed ci failure

[Sayamame-beans]

コンフリクト解消

[syuilo]

うーんRNやリアクションの通知はスレッドとはそんなに関係ないから通知してもいい気がするわね

[syuilo]

日本人で多くのメンションを含む長大なスレッドを作る人はほとんど見たことがないので、実際にこの機能を使うであろう海外ユーザーに意見を求めたい気がするわね

[syuilo]

(実装するにしても、DBに毎回ほとんどのケースで無駄になるクエリが発生するのは非効率だから別の実装方法を考えたい)

[Sayamame-beans]

スレッドミュートとは別の機能として分離したいということでしょうか

[anatawa12]

日本人で多くのメンションを含む長大なスレッドを作る人はほとんど見たことがない

まぁまぁあると思う

[Sayamame-beans]

コンフリクト解消

[Sayamame-beans]

conflict resolved

[github-actions]

Backend memory usage comparison

Before GC

Metric	base (MB)	head (MB)	Diff (MB)	Diff (%)
VmRSS	306.00 MB	303.67 MB	-2.32 MB	-0.76%
VmHWM	306.00 MB	303.67 MB	-2.32 MB	-0.76%
VmSize	23104.60 MB	23102.17 MB	-2.43 MB	-0.01%
VmData	1368.87 MB	1366.15 MB	-2.71 MB	-0.19%

After GC

Metric	base (MB)	head (MB)	Diff (MB)	Diff (%)
VmRSS	306.01 MB	303.68 MB	-2.32 MB	-0.76%
VmHWM	306.01 MB	303.68 MB	-2.32 MB	-0.76%
VmSize	23104.60 MB	23102.17 MB	-2.43 MB	-0.01%
VmData	1368.87 MB	1366.15 MB	-2.71 MB	-0.19%

After Request

Metric	base (MB)	head (MB)	Diff (MB)	Diff (%)
VmRSS	306.21 MB	304.00 MB	-2.20 MB	-0.72%
VmHWM	306.21 MB	304.00 MB	-2.20 MB	-0.72%
VmSize	23104.60 MB	23102.17 MB	-2.43 MB	-0.01%
VmData	1368.87 MB	1366.15 MB	-2.71 MB	-0.19%

See workflow logs for details