enhance(ap): 凍結をリモートに配信する際、Delete PersonではなくUpdate Personとして表現するように

[tamaina]

Resolve #13256

What

• Personにsuspended( https://docs.joinmastodon.org/spec/activitypub/#suspended )フラグを追加
  • ローカルユーザーを凍結/解凍した際はUpdate Actor-Personを送るようにする (いままではDelete/Undo Delete Actor-Person)
• Userテーブルにリモートがそのユーザーを凍結しているのかを記録するisRemoteSuspendedカラムを追加
  • ローカルの凍結と基本的に同様の挙動を示すようにする。ローカルの凍結を解除してもリモートが凍結解除しない限りは凍結されているものとして扱う(Mastodonのspecに準拠)

Why

diff https://github.com/tamaina/misskey/compare/ap-susp-update..remote-suspend

Resolve #13256

ユーザーの凍結をDeleteとして表現するのはいろいろ不都合なので、isRemoteSuspendedフラグで管理する

Additional info (optional)

• Includes and depends on test(backend): e2e/timelines.ts: 非FTT時のテストを追加, 凍結のテストを追加, これにかかる幾つかのバグ修正 #16284
• Includes and depends on enhance: ユーザーを解凍(凍結解除)した際にフォロー関係が元に戻るようにする #16279
• Includes and depends on enhance(ap): ApDeliverManagerService: テストを追加、addAllKnowingSharedInbox Recipeを追加 #16297

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

❌ Patch coverage is 74.30830% with 65 lines in your changes missing coverage. Please review.
✅ Project coverage is 46.24%. Comparing base (76beac4) to head (5e9f2e3).
⚠️ Report is 702 commits behind head on develop.

Files with missing lines	Patch %	Lines
...end/src/core/activitypub/models/ApPersonService.ts	22.22%	14 Missing ⚠️
packages/backend/src/core/AccountUpdateService.ts	75.67%	9 Missing ⚠️
packages/backend/src/core/UserSuspendService.ts	93.05%	5 Missing ⚠️
...ackend/src/server/api/endpoints/users/followers.ts	0.00%	4 Missing ⚠️
packages/backend/src/core/QueueService.ts	0.00%	3 Missing ⚠️
packages/backend/src/core/UserSearchService.ts	25.00%	3 Missing ⚠️
...ages/backend/src/server/web/ClientServerService.ts	0.00%	3 Missing ⚠️
packages/frontend/src/pages/admin-user.vue	0.00%	3 Missing ⚠️
packages/backend/src/core/UserFollowingService.ts	50.00%	2 Missing ⚠️
...d/src/server/api/endpoints/federation/followers.ts	0.00%	2 Missing ⚠️
... and 12 more

Additional details and impacted files

@@             Coverage Diff             @@
##           develop   #16268      +/-   ##
===========================================
+ Coverage    46.03%   46.24%   +0.20%     
===========================================
  Files         1769     1778       +9     
  Lines       182039   182748     +709     
  Branches      5403     5554     +151     
===========================================
+ Hits         83800    84503     +703     
- Misses       98190    98215      +25     
+ Partials        49       30      -19     

☔ 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]

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

差分はこちら

--- base
+++ head
@@ -16434,6 +16434,9 @@
                     "isSuspended": {
                       "type": "boolean"
                     },
+                    "isRemoteSuspended": {
+                      "type": "boolean"
+                    },
                     "isHibernated": {
                       "type": "boolean"
                     },
@@ -16505,6 +16508,7 @@
                     "isModerator",
                     "isSilenced",
                     "isSuspended",
+                    "isRemoteSuspended",
                     "isHibernated",
                     "lastActiveDate",
                     "moderationNote",

Get diff files from Workflow Page

[tamaina]

アカウント削除ではない=follower/followingが減算されないのでテストの書き方を考える必要がある

というか、ローカルユーザーが凍結されてもFFの減算やFF関係の削除処理がされなさそうなのに(この仕様も一考の余地がありそう…)リモートのテストでの検証がそれの検出でっていうのはあまりよろしくない感じはありそう

[tamaina]

@syuilo どう思う？(雑)

[tamaina]

ローカルユーザーが凍結されてもFFの減算やFF関係の削除処理がされなさそう

全然違う、suspend時にunFollowAllが走ってた

[tamaina]

READY FOR REVIEW

[tamaina]

普通にテスト落ちてる?

[tamaina]

と思ったけどmin.node-versionと.node-versionで挙動違うのおかしい

[tamaina]

conflict resolved