misskey-dev / misskey

🌎 A completely free and open interplanetary microblogging platform 🚀
https://misskey-hub.net/
GNU Affero General Public License v3.0
10.09k stars 1.38k forks source link

ユーザーミュートにメンションが考慮されるように #14995

Open Sayamame-beans opened 4 days ago

Sayamame-beans commented 4 days ago

What

Why

fix: #2690

Additional info (optional)

現在未テストです 多分動きます

Checklist

codecov[bot] commented 4 days ago

Codecov Report

Attention: Patch coverage is 4.54545% with 21 lines in your changes missing coverage. Please review.

Project coverage is 39.96%. Comparing base (c1f19fa) to head (b892686). Report is 4 commits behind head on develop.

Files with missing lines Patch % Lines
packages/backend/src/core/QueryService.ts 5.88% 16 Missing :warning:
packages/backend/src/misc/is-user-related.ts 0.00% 4 Missing :warning:
...backend/src/server/api/endpoints/notes/mentions.ts 0.00% 1 Missing :warning:
Additional details and impacted files ```diff @@ Coverage Diff @@ ## develop #14995 +/- ## =========================================== - Coverage 39.98% 39.96% -0.02% =========================================== Files 1563 1563 Lines 197737 197757 +20 Branches 3644 3630 -14 =========================================== - Hits 79056 79026 -30 - Misses 118077 118126 +49 - Partials 604 605 +1 ```

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.


🚨 Try these New Features:

github-actions[bot] commented 4 days ago

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

Sayamame-beans commented 4 days ago

何か手元でテストしないとヤバそうな雰囲気がするのでdraftに戻します

Sayamame-beans commented 3 days ago

追記: 親投稿や引用元投稿にメンションがあって、その子や引用にはメンションがないようなケースに対しては現状判定出来ていません (isUserRelatedの方にはあるかもしれませんが、generateMutedUserQueryの方にはメンションの情報が無さそうかな、と思っています…実は両方あったりしますかね?) 加えて、自身が付けたメンションによってもノートがミュートされてしまっており、悩ましいところです(#15002 の返答に応じて対応したい気持ちがあります。)

あと、テストがコケている理由がまだ理解しきれていません

Sayamame-beans commented 1 day ago

(何が原因でテストがコケているんでしょうこれ…)

samunohito commented 1 day ago

e2eだけで見るとクリップまわりっぽいですが…

Sayamame-beans commented 1 day ago

調査中(確かにクリップが壊れていました、なぜ…)

Sayamame-beans commented 1 day ago

TLだとFTTが効いてて気付かなかっただけで普通に壊れてました…

Sayamame-beans commented 1 day ago

直しました(すみません)