Change: サークルの送り先アカウント指定方法をaccount_username（Fedibirdと同様）に変更 (#283)

* Change: サークルの送り先アカウント指定方法を`account_username`（Fedibirdと同様）に変更

* Test: テストを追加

* Maybe Fix: Fedibirdで自分限定と認識される問題

app/lib/activitypub/tag_manager.rb

@@ -119,10 +119,7 @@ class ActivityPub::TagManager
         end.compact
       end
     when 'limited'
-      status.mentions.each_with_object([]) do |mention, result|
-        result << uri_for(mention.account)
-        result << followers_uri_for(mention.account) if mention.account.group?
-      end.compact
+      ['kmyblue:Limited'] # to avoid Fedibird personal visibility
     end
   end
 

app/lib/status_reach_finder.rb

@@ -25,6 +25,12 @@ class StatusReachFinder
     (reached_account_inboxes_for_friend + followers_inboxes_for_friend + friend_inboxes).uniq
   end
 
+  def inboxes_for_limited
+    DeliveryFailureTracker.without_unavailable(
+      @status.mentioned_accounts.where.not(domain: nil).pluck(:inbox_url).compact.uniq
+    )
+  end
+
   def all_inboxes
     (inboxes + inboxes_for_misskey + inboxes_for_friend).uniq
   end

app/workers/activitypub/distribution_worker.rb

@@ -7,13 +7,23 @@ class ActivityPub::DistributionWorker < ActivityPub::RawDistributionWorker
     @status  = Status.find(status_id)
     @account = @status.account
 
-    distribute!
+    if @status.limited_visibility?
+      distribute_limited!
+    else
+      distribute!
+    end
   rescue ActiveRecord::RecordNotFound
     true
   end
 
   protected
 
+  def distribute_limited!
+    ActivityPub::DeliveryWorker.push_bulk(inboxes_for_limited, limit: 1_000) do |inbox_url|
+      [payload, @account.id, inbox_url, options]
+    end
+  end
+
   def inboxes
     @inboxes ||= status_reach_finder.inboxes
   end
@@ -26,6 +36,10 @@ class ActivityPub::DistributionWorker < ActivityPub::RawDistributionWorker
     @inboxes_for_friend ||= status_reach_finder.inboxes_for_friend
   end
 
+  def inboxes_for_limited
+    @inboxes_for_limited ||= status_reach_finder.inboxes_for_limited
+  end
+
   def status_reach_finder
     @status_reach_finder ||= StatusReachFinder.new(@status)
   end