fix: noteを取得する系のAPIでデフォルトでリプライが含まれないようになる破壊的変更を修正

[anatawa12]

What

eb740e2 で行われたデフォルト動作の破壊的変更を戻します

Why

Additional info (optional)

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

[codecov]

Codecov Report

All modified lines are covered by tests ✅

Comparison is base (6840434) 79.00% compared to head (b991651) 78.94%.

Additional details and impacted files

@@             Coverage Diff             @@
##           develop   #11950      +/-   ##
===========================================
- Coverage    79.00%   78.94%   -0.07%     
===========================================
  Files          928      929       +1     
  Lines        99089    99194     +105     
  Branches      7913     7883      -30     
===========================================
+ Hits         78289    78308      +19     
- Misses       20800    20886      +86     

Files	Coverage Δ
...c/server/api/endpoints/notes/user-list-timeline.ts	50.31% <100.00%> (ø)
...es/backend/src/server/api/endpoints/users/notes.ts	82.55% <100.00%> (-1.35%)	⬇️

... and 19 files with indirect coverage changes

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

[syuilo]

デフォルトでリプライは含まれない方が自然なのと、#11946で一部そもそもwithRepliesが消える(リプライが含まれるタイムラインは取得できなくなる)ので現状のままで良いかも

[anatawa12]

デフォルトでリプライは含まれない方が自然

少なくともリストについてはどちらも含まれたほうが自然だと思います

[anatawa12]

他人へのリプライも表示されてるべきだとは思いますが、とくに自分へのリプライは表示されてるほうが自然だと思います。

[syuilo]

とくに自分へのリプライは表示されてるほうが自然

それはwithReplies関係なく表示される仕様ですね
されないとしたらバグ

[syuilo]

#11946 で直ってるかも

[anatawa12]

API内部の条件式としてreplyId is NULLなので表示されないように実装されてるように思えます

[syuilo]

リプライを含むかどうかはAPIのパラメータではなくフォローもしくはリストのメンバーごとに設定できるようになりました 🙏🏻

[anatawa12]

👍 良さそうです