Skip to content

Conversation

@tai-cha
Copy link
Contributor

@tai-cha tai-cha commented Nov 26, 2025

What

  • Signed Getを受け取った場合にユーザーを判定してフォロワー限定や指名ノートを返せるように
  • Announceの解決にActorに対するローカルのフォロワーでSigned Getすることで解決を図る
  • quote系やreplyに関しても同様に解決

Why

Resolve #15521

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

@tai-cha
Copy link
Contributor Author

tai-cha commented Nov 26, 2025

大枠としてはできているけど、ブロックの考慮とか細かいところがまだ

@github-actions
Copy link
Contributor

github-actions bot commented Nov 26, 2025

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

@codecov
Copy link

codecov bot commented Nov 26, 2025

Codecov Report

❌ Patch coverage is 14.67890% with 93 lines in your changes missing coverage. Please review.
✅ Project coverage is 45.51%. Comparing base (c4f53ab) to head (ed9b9d9).
⚠️ Report is 36 commits behind head on develop.

Files with missing lines Patch % Lines
...ges/backend/src/server/ActivityPubServerService.ts 10.20% 44 Missing ⚠️
...ges/backend/src/core/activitypub/ApInboxService.ts 6.25% 30 Missing ⚠️
...ckend/src/core/activitypub/models/ApNoteService.ts 8.33% 11 Missing ⚠️
packages/backend/src/core/UserFollowingService.ts 37.50% 5 Missing ⚠️
.../backend/src/core/activitypub/ApResolverService.ts 71.42% 2 Missing ⚠️
...ckages/backend/src/server/api/endpoints/ap/show.ts 0.00% 1 Missing ⚠️
Additional details and impacted files
@@             Coverage Diff              @@
##           develop   #16871       +/-   ##
============================================
+ Coverage    17.20%   45.51%   +28.30%     
============================================
  Files          884     1793      +909     
  Lines        82923   186839   +103916     
  Branches      1366     5582     +4216     
============================================
+ Hits         14266    85032    +70766     
- Misses       68650   101778    +33128     
- Partials         7       29       +22     

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

@tai-cha
Copy link
Contributor Author

tai-cha commented Nov 26, 2025

未知のフォロワー限定投稿に対し、リプライを追加しても、親投稿、リプライともに連合されない

#15521 内のこれに対する対処がまだ

@tai-cha
Copy link
Contributor Author

tai-cha commented Nov 26, 2025

サーバーブロックも考慮できるようになるからしてもいいかも
#15993

@tai-cha tai-cha changed the title fix: フォロワー限定投稿の本人投稿を解決できるように(signed getされた際にfollowers, specifiedのノートも取得可能に) fix: フォロワー限定投稿の本人リプライ、リノートを解決できるように(signed getされた際にfollowers, specifiedのノートも取得可能に) Nov 27, 2025
@tai-cha
Copy link
Contributor Author

tai-cha commented Nov 27, 2025

未知のフォロワー限定投稿に対し、リプライを追加しても、親投稿、リプライともに連合されない

これには対処した

@tai-cha
Copy link
Contributor Author

tai-cha commented Nov 27, 2025

投稿者本人がフォロワー限定投稿を引用しても、それが未知のもの(当該サーバーとの連合開始以前の投稿等)である場合は、引用したノート本体のみ連合され、どのノートが引用されているかは分からず(引用でないように見える)、引用されたノートの連合もされない

これに対するテストも必要そう

@tai-cha
Copy link
Contributor Author

tai-cha commented Nov 27, 2025

投稿者本人がフォロワー限定投稿を引用しても、それが未知のもの(当該サーバーとの連合開始以前の投稿等)である場合は、引用したノート本体のみ連合され、どのノートが引用されているかは分からず(引用でないように見える)、引用されたノートの連合もされない

従来の挙動を壊さずできた気がする

@tai-cha
Copy link
Contributor Author

tai-cha commented Nov 27, 2025

投稿者本人がフォロワー限定投稿を引用しても、それが未知のもの(当該サーバーとの連合開始以前の投稿等)である場合は、引用したノート本体のみ連合され、どのノートが引用されているかは分からず(引用でないように見える)、引用されたノートの連合もされない

これもできた&テスト動いた

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

Labels

packages/backend:test packages/backend Server side specific issue/PR

Projects

1 participant