From 7fe1f8b39fbe43060c5e2e3fc8d1f64be6087c3c Mon Sep 17 00:00:00 2001 From: Hazelnoot Date: Tue, 16 Sep 2025 11:58:11 -0400 Subject: [PATCH] use CacheService.getUsers instead of promiseMap+CacheService.getOptionalUserById --- packages/backend/src/core/ReactionService.ts | 9 +++++---- 1 file changed, 5 insertions(+), 4 deletions(-) diff --git a/packages/backend/src/core/ReactionService.ts b/packages/backend/src/core/ReactionService.ts index 6af04dcba5..6d69b079ce 100644 --- a/packages/backend/src/core/ReactionService.ts +++ b/packages/backend/src/core/ReactionService.ts @@ -34,7 +34,6 @@ import { CacheService } from '@/core/CacheService.js'; import { NoteVisibilityService } from '@/core/NoteVisibilityService.js'; import { TimeService } from '@/global/TimeService.js'; import { CollapsedQueueService } from '@/core/CollapsedQueueService.js'; -import { promiseMap } from '@/misc/promise-map.js'; import type { DataSource } from 'typeorm'; const FALLBACK = '\u2764'; @@ -299,9 +298,11 @@ export class ReactionService implements OnModuleInit { if (['public', 'home', 'followers'].includes(note.visibility)) { dm.addFollowersRecipe(); } else if (note.visibility === 'specified') { - const visibleUsers = await promiseMap(note.visibleUserIds, async id => await this.cacheService.findOptionalUserById(id), { limit: 2 }); - for (const u of visibleUsers.filter(u => u && isRemoteUser(u))) { - dm.addDirectRecipe(u as MiRemoteUser); + const visibleUsers = await this.cacheService.getUsers(note.visibleUserIds); + for (const u of visibleUsers.values()) { + if (isRemoteUser(u)) { + dm.addDirectRecipe(u as MiRemoteUser); + } } }