fix: 複数idを指定するusers/showが関係ないユーザを返すことがある問題を修正

[anatawa12]

What

• 存在しないユーザー(削除済み等で発生し得る?)のIDをusers/showのuserIdsに入れてリクエストすると、関係ないユーザーの情報が返ってくるAPIのバグがあることを発見
  • 関係ないユーザーが宛先に含まれてしまう問題を引き起こす可能性があります

#13717 (comment) の修正です。

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

[anatawa12]

nullable arrayにするかは悩みましたがAPIの仕様がsetではなくarrayっぽいので複数のユーザをまとめてアクセスするのを最適化するのに使えるように設計されてたのでnullable arrayにしました

[codecov]

Codecov Report

Attention: Patch coverage is 57.14286% with 3 lines in your changes are missing coverage. Please review.

Project coverage is 65.15%. Comparing base (f6df940) to head (af6cc7f).
Report is 1 commits behind head on develop.

❗ Current head af6cc7f differs from pull request most recent head a35dc9f

Please upload reports for the commit a35dc9f to get more accurate results.

Files	Patch %	Lines
...ges/backend/src/server/api/endpoints/users/show.ts	0.00%	3 Missing ⚠️

Additional details and impacted files

@@             Coverage Diff             @@
##           develop   #13765      +/-   ##
===========================================
- Coverage    66.46%   65.15%   -1.31%     
===========================================
  Files          988      985       -3     
  Lines       116719   112117    -4602     
  Branches      4511     4422      -89     
===========================================
- Hits         77572    73054    -4518     
+ Misses       39116    39032      -84     
  Partials        31       31              

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

[github-actions]

このPRによるapi.jsonの差分

差分はこちら

Get diff files from Workflow Page

[syuilo]

nullが入るとあらゆるところで判定が必要になるからnon-nullで良いと思った

[anatawa12]

削除されてる等のときに別の処理したいことないですかね

[anatawa12]

まぁidでマッチするようにすればいいからレスポンスのさぃず変えるようにしますか

[anatawa12]

non-nullにしました

[syuilo]

関数そのまま渡すと型は同じまま引数変わった時にバグるわね

[anatawa12]

このファイル内で同様の処理が数多くあったのでそれに揃えたのですがそれらも同時にu => pushVisibleUserに治すということですかね

[syuilo]

ついでにやっちゃってもらえると助かります🙏

[anatawa12]

ごめんなさい。修正点あるの忘れてました。修正しました。マージターゲット修正したほうがいいですか

[anatawa12]

conflict resolved

[syuilo]

🙏