Skip to content

Conversation

@tetsuya-ki
Copy link
Contributor

@tetsuya-ki tetsuya-ki commented Dec 14, 2025

What

Not Plannedとありますが、需要があるかもしれないのでプルリクエストを出しておきます。
キャッシュからミュート・ブロックした情報を取得し、集計し直す処理となっています。
DBには負荷をかけませんが、多少処理が重たい可能性があります。

Why

リアクションやノートがが多すぎてミュートしたユーザーや過去ブロックしたユーザーのリアクションを表示しないため。

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

fix: #13456 (追記させていただきました)

@dosubot dosubot bot added the size:L This PR changes 100-499 lines, ignoring generated files. label Dec 14, 2025
@github-actions github-actions bot added packages/frontend Client side specific issue/PR packages/backend Server side specific issue/PR labels Dec 14, 2025
@codecov
Copy link

codecov bot commented Dec 14, 2025

Codecov Report

❌ Patch coverage is 5.29801% with 143 lines in your changes missing coverage. Please review.
✅ Project coverage is 62.79%. Comparing base (78435dc) to head (05156ca).
⚠️ Report is 10 commits behind head on develop.

Files with missing lines Patch % Lines
...ges/backend/src/core/entities/NoteEntityService.ts 2.63% 74 Missing ⚠️
...end/src/core/entities/NoteReactionEntityService.ts 9.83% 55 Missing ⚠️
...kages/frontend/src/composables/use-note-capture.ts 0.00% 7 Missing and 6 partials ⚠️
...rc/server/api/endpoints/notes/show-partial-bulk.ts 0.00% 1 Missing ⚠️
Additional details and impacted files
@@             Coverage Diff              @@
##           develop   #16988       +/-   ##
============================================
+ Coverage    13.97%   62.79%   +48.82%     
============================================
  Files          237     1150      +913     
  Lines        11273   115365   +104092     
  Branches      3728     7914     +4186     
============================================
+ Hits          1575    72448    +70873     
- Misses        7578    40769    +33191     
- Partials      2120     2148       +28     

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

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@github-actions
Copy link
Contributor

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

@github-actions
Copy link
Contributor

Backend Memory Usage Comparison

Metric base head Diff
RSS 351.53 MB 356.53 MB 5.00 MB (1.42%)

See workflow logs for details

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

packages/backend Server side specific issue/PR packages/frontend Client side specific issue/PR size:L This PR changes 100-499 lines, ignoring generated files.

Projects

Development

Successfully merging this pull request may close these issues.

自分がブロックした人のリアクションをレスポンスから除外して欲しい

1 participant