refactor(backend): meUpdatedイベントに更新後のアカウント情報を含まないように

[kakkokari-gtyih]

What

フロントエンドが/api/iを呼び出すことでアカウント情報を得るようにした。これにより

• Redis PubSubストリーム内を巨大なデータが行き来する回数を減らしパフォーマンスが向上することが考えられる
• 各publish処理が正確なアカウント情報を得るためにUserEntityServiceに依存する状況を解消できる

特に後者は重要で、UserEntityServiceという巨大なサービスへの依存を減らせるのはコードの見通しもよくなる。現にAccountMoveServiceはUserEntityServiceへの依存のせいで、ApPersonServiceと合わせて循環依存を作ってしまっている。これが解消できるのは大きい。

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

[github-actions]

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

[anatawa12]

Redis PubSubストリーム内の問題であれば、streamingのAPI変えるとサードパーティ等影響が大きいからpackages/backend/src/server/api/stream/channels/main.tsでpackする形にするという手もありそうですがMainChannelがUserEntityService二依存するのもやめたほうがいい感じですかね

[codecov]

Codecov Report

Attention: Patch coverage is 6.66667% with 14 lines in your changes missing coverage. Please review.

Project coverage is 57.79%. Comparing base (dd56623) to head (4628a8f).
Report is 1 commits behind head on develop.

Files with missing lines	Patch %	Lines
packages/backend/src/core/UserFollowingService.ts	0.00%	3 Missing ⚠️
packages/backend/src/core/AccountMoveService.ts	0.00%	1 Missing ⚠️
packages/backend/src/core/UserBlockingService.ts	0.00%	1 Missing ⚠️
packages/backend/src/server/ServerService.ts	0.00%	1 Missing ⚠️
...ges/backend/src/server/api/endpoints/i/2fa/done.ts	0.00%	1 Missing ⚠️
...backend/src/server/api/endpoints/i/2fa/key-done.ts	0.00%	1 Missing ⚠️
...nd/src/server/api/endpoints/i/2fa/password-less.ts	0.00%	1 Missing ⚠️
...ckend/src/server/api/endpoints/i/2fa/remove-key.ts	0.00%	1 Missing ⚠️
...ckend/src/server/api/endpoints/i/2fa/unregister.ts	0.00%	1 Missing ⚠️
...ckend/src/server/api/endpoints/i/2fa/update-key.ts	0.00%	1 Missing ⚠️
... and 2 more

Additional details and impacted files

@@             Coverage Diff              @@
##           develop   #15063       +/-   ##
============================================
+ Coverage    39.96%   57.79%   +17.83%     
============================================
  Files         1563      835      -728     
  Lines       197744    93903   -103841     
  Branches      3631     1551     -2080     
============================================
- Hits         79024    54270    -24754     
+ Misses      118115    39609    -78506     
+ Partials       605       24      -581     

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

[anatawa12]

Redis PubSubストリーム内の問題であれば、streamingのAPI変えるとサードパーティ等影響が大きいからpackages/backend/src/server/api/stream/channels/main.tsでpackする形にするという手もありそうですがMainChannelがUserEntityService二依存するのもやめたほうがいい感じですかね

一応これの懸念としてはpackする回数が一人のユーザが複数のコネクションを持っているとpack回数が増える可能性がある点があるとは思います。
逆にUserFollowingService等のpackはコネクション数が0なら無駄だったのが、MainChannelでのpackの場合これは省略される(現行と比べた場合)

EDIT: とかいたけど前半は普通に/api/i呼ぶときとあんまり変わらないわ

[syuilo]

/i の結果って(POSTだとしても)必ず最新であることが保証されてたっけ？

[syuilo]

されてた