fix(backend): RBTの修正

kakkokari-gtyih commented 2 weeks ago

What

リアクションの絵文字の変換を必ず行うように（convertLegacyReactionsではレガシ絵文字の変換以外も行われているため、その旨を示すコメントも追記）
- 絵文字が分裂する問題を修正できるかも
metaが更新されてリアクションバッファリングが有効 → 無効になったら即bakeするように
- リアクションバッファリングを無効化したらbakeされるまでに行われたリアクションがbakeされない問題を修正
- globalEventのmetaUpdatedの型が変わり、beforeとafter両方を取るようになった
- 重いかもしれないのでこっちは入れなくてもいいかも（もしくは単純にqueueの処理をmetaの値で弾かないように変えるだけでもいいかも。それだと反映は最大1日程度遅れるけど…）

Why

Additional info (optional)

Checklist

[x] 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[bot] commented 2 weeks ago

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

codecov[bot] commented 2 weeks ago

Codecov Report

Attention: Patch coverage is 68.49315% with 23 lines in your changes missing coverage. Please review.

Project coverage is 39.56%. Comparing base (ca967e8) to head (411ab35). Report is 2 commits behind head on develop.

Files with missing lines	Patch %	Lines
...ages/backend/src/core/ReactionsBufferingService.ts	68.00%	16 Missing :warning:
...ges/backend/src/core/entities/NoteEntityService.ts	0.00%	3 Missing :warning:
packages/backend/src/GlobalModule.ts	0.00%	2 Missing :warning:
packages/backend/src/core/MetaService.ts	0.00%	2 Missing :warning:

Additional details and impacted files

```diff @@ Coverage Diff @@ ## develop #14621 +/- ## ========================================== Coverage 39.55% 39.56% ========================================== Files 1542 1542 Lines 192719 192752 +33 Branches 2472 3537 +1065 ========================================== + Hits 76237 76267 +30 + Misses 115917 115888 -29 - Partials 565 597 +32 ```

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

syuilo commented 2 weeks ago

e2eが失敗しとる？

kakkokari-gtyih commented 2 weeks ago

metaUpdateが2回発火されているかもしれない

kakkokari-gtyih commented 2 weeks ago

e2eが失敗しとる？

謎

kakkokari-gtyih commented 2 weeks ago

metaUpdateが2回発火されているかもしれない

これ大丈夫？

syuilo commented 2 weeks ago

metaUpdateが2回発火されているかもしれない

これ大丈夫？

PR前からということ？

kakkokari-gtyih commented 2 weeks ago

metaUpdateが2回発火されているかもしれない

これ大丈夫？

PR前からということ？

PR前から起こっているかどうかは今確認中

syuilo commented 2 weeks ago

いずれにしろ2回発火はよろしくない

kakkokari-gtyih commented 2 weeks ago

developでも2回ログが来てる

kakkokari-gtyih commented 2 weeks ago

e2eは明らかな実装ミスを見つけて直したのでなおったと思う

syuilo commented 2 weeks ago

🙏🏻

kakkokari-gtyih commented 2 weeks ago

いずれにしろ2回発火はよろしくない

→ https://github.com/misskey-dev/misskey/issues/14622

misskey-dev / misskey