@atproto/bsky 0.0.29 → 0.0.30

package/src/services/feed/index.ts

@@ -139,6 +139,7 @@ export class FeedService {
   async getPostInfos(
     postUris: string[],
     viewer: string | null,
+    includeSoftDeleted?: boolean,
   ): Promise<PostInfoMap> {
     if (postUris.length < 1) return {}
     const db = this.db.db
@@ -149,8 +150,12 @@ export class FeedService {
       .innerJoin('actor', 'actor.did', 'post.creator')
       .innerJoin('record', 'record.uri', 'post.uri')
       .leftJoin('post_agg', 'post_agg.uri', 'post.uri')
-      .where(notSoftDeletedClause(ref('actor'))) // Ensures post reply parent/roots get omitted from views when taken down
-      .where(notSoftDeletedClause(ref('record')))
+      .if(!includeSoftDeleted, (qb) =>
+        qb.where(notSoftDeletedClause(ref('actor'))),
+      ) // Ensures post reply parent/roots get omitted from views when taken down
+      .if(!includeSoftDeleted, (qb) =>
+        qb.where(notSoftDeletedClause(ref('record'))),
+      )
       .select([
         'post.uri as uri',
         'post.cid as cid',
@@ -249,12 +254,13 @@ export class FeedService {
       dids: Set<string>
       uris: Set<string>
       viewer: string | null
+      includeSoftDeleted?: boolean
     },
     depth = 0,
   ): Promise<FeedHydrationState> {
     const { viewer, dids, uris } = refs
     const [posts, threadgates, labels, bam] = await Promise.all([
-      this.getPostInfos(Array.from(uris), viewer),
+      this.getPostInfos(Array.from(uris), viewer, refs.includeSoftDeleted),
       this.threadgatesByPostUri(Array.from(uris)),
       this.services.label.getLabelsForSubjects([...uris, ...dids]),
       this.services.graph.getBlockAndMuteState(
@@ -266,7 +272,7 @@ export class FeedService {
     const [profileState, blocks, lists] = await Promise.all([
       this.services.actor.views.profileHydration(
         Array.from(dids),
-        { viewer },
+        { viewer, includeSoftDeleted: refs.includeSoftDeleted },
         { bam, labels },
       ),
       this.blocksForPosts(posts, bam),

package/src/services/feed/views.ts

@@ -208,7 +208,9 @@ export class FeedViews {
     const post = posts[uri]
     const gate = threadgates[uri]
     const author = actors[post?.creator]
-    if (!post || !author) return undefined
+    if (!post || !author) {
+      return undefined
+    }
     const postLabels = labels[uri] ?? []
     const postSelfLabels = getSelfLabels({
       uri: post.uri,

package/tests/views/author-feed.test.ts

@@ -138,7 +138,7 @@ describe('pds author feed views', () => {
     )
   })
 
-  it('blocked by actor takedown.', async () => {
+  it('non-admins blocked by actor takedown.', async () => {
     const { data: preBlock } = await agent.api.app.bsky.feed.getAuthorFeed(
       { actor: alice },
       { headers: await network.serviceHeaders(carol) },
@@ -163,11 +163,18 @@ describe('pds author feed views', () => {
       },
     )
 
-    const attempt = agent.api.app.bsky.feed.getAuthorFeed(
+    const attemptAsUser = agent.api.app.bsky.feed.getAuthorFeed(
       { actor: alice },
       { headers: await network.serviceHeaders(carol) },
     )
-    await expect(attempt).rejects.toThrow('Profile not found')
+    await expect(attemptAsUser).rejects.toThrow('Profile not found')
+
+    const attemptAsAdmin = await agent.api.app.bsky.feed.getAuthorFeed(
+      { actor: alice },
+      { headers: network.bsky.adminAuthHeaders() },
+    )
+    expect(attemptAsAdmin.data.feed.length).toBeGreaterThan(0)
+    expect(attemptAsAdmin.data.feed.length).toEqual(preBlock.feed.length)
 
     // Cleanup
     await agent.api.com.atproto.admin.updateSubjectStatus(
@@ -215,13 +222,26 @@ describe('pds author feed views', () => {
       },
     )
 
-    const { data: postBlock } = await agent.api.app.bsky.feed.getAuthorFeed(
-      { actor: alice },
-      { headers: await network.serviceHeaders(carol) },
+    const [{ data: postBlockAsUser }, { data: postBlockAsAdmin }] =
+      await Promise.all([
+        agent.api.app.bsky.feed.getAuthorFeed(
+          { actor: alice },
+          { headers: await network.serviceHeaders(carol) },
+        ),
+        agent.api.app.bsky.feed.getAuthorFeed(
+          { actor: alice },
+          { headers: network.bsky.adminAuthHeaders() },
+        ),
+      ])
+
+    expect(postBlockAsUser.feed.length).toEqual(preBlock.feed.length - 1)
+    expect(postBlockAsUser.feed.map((item) => item.post.uri)).not.toContain(
+      post.uri,
+    )
+    expect(postBlockAsAdmin.feed.length).toEqual(preBlock.feed.length)
+    expect(postBlockAsAdmin.feed.map((item) => item.post.uri)).toContain(
+      post.uri,
     )
-
-    expect(postBlock.feed.length).toEqual(preBlock.feed.length - 1)
-    expect(postBlock.feed.map((item) => item.post.uri)).not.toContain(post.uri)
 
     // Cleanup
     await agent.api.com.atproto.admin.updateSubjectStatus(