@credal/actions 0.2.147 → 0.2.149

package/dist/actions/providers/slackUser/searchSlack.js

@@ -22,24 +22,37 @@ class SlackUserCache {
     constructor(client) {
         this.client = client;
         this.cache = new Map();
+        this.pendingRequests = new Map();
         this.client = client;
     }
     get(id) {
         return __awaiter(this, void 0, void 0, function* () {
-            var _a, _b, _c, _d, _e, _f, _g, _h, _j, _k;
             const cached = this.cache.get(id);
             if (cached)
                 return cached;
-            const res = yield this.client.users.info({ user: id });
-            const u = {
-                name: (_g = (_e = (_c = (_b = (_a = res.user) === null || _a === void 0 ? void 0 : _a.profile) === null || _b === void 0 ? void 0 : _b.display_name) !== null && _c !== void 0 ? _c : (_d = res.user) === null || _d === void 0 ? void 0 : _d.real_name) !== null && _e !== void 0 ? _e : (_f = res.user) === null || _f === void 0 ? void 0 : _f.name) !== null && _g !== void 0 ? _g : "",
-                email: (_k = (_j = (_h = res.user) === null || _h === void 0 ? void 0 : _h.profile) === null || _j === void 0 ? void 0 : _j.email) !== null && _k !== void 0 ? _k : "",
-            };
-            if (res.user && id) {
-                this.cache.set(id, u);
-                return u;
-            }
-            return undefined;
+            const pending = this.pendingRequests.get(id);
+            if (pending)
+                return pending;
+            const promise = (() => __awaiter(this, void 0, void 0, function* () {
+                var _a, _b, _c, _d, _e, _f, _g, _h, _j, _k;
+                try {
+                    const res = yield this.client.users.info({ user: id });
+                    const u = {
+                        name: (_g = (_e = (_c = (_b = (_a = res.user) === null || _a === void 0 ? void 0 : _a.profile) === null || _b === void 0 ? void 0 : _b.display_name) !== null && _c !== void 0 ? _c : (_d = res.user) === null || _d === void 0 ? void 0 : _d.real_name) !== null && _e !== void 0 ? _e : (_f = res.user) === null || _f === void 0 ? void 0 : _f.name) !== null && _g !== void 0 ? _g : "",
+                        email: (_k = (_j = (_h = res.user) === null || _h === void 0 ? void 0 : _h.profile) === null || _j === void 0 ? void 0 : _j.email) !== null && _k !== void 0 ? _k : "",
+                    };
+                    if (res.user && id) {
+                        this.cache.set(id, u);
+                        return u;
+                    }
+                    return undefined;
+                }
+                finally {
+                    this.pendingRequests.delete(id);
+                }
+            }))();
+            this.pendingRequests.set(id, promise);
+            return promise;
         });
     }
     set(id, { email, name }) {
@@ -238,44 +251,58 @@ const searchSlack = (_a) => __awaiter(void 0, [_a], void 0, function* ({ params,
     const targetIds = (emails === null || emails === void 0 ? void 0 : emails.length) ? yield lookupUserIdsByEmail(client, emails, cache) : [];
     const filteredTargetIds = targetIds.filter(id => id !== myUserId);
     const allMatches = [];
-    // --- Scoped DM/MPIM searches ---
+    // --- Parallel search execution for better performance ---
+    const searchPromises = [];
+    // Scoped DM/MPIM searches
     if (filteredTargetIds.length === 1) {
-        allMatches.push(...(yield searchScoped({ client, scope: `<@${filteredTargetIds[0]}>`, topic, timeRange, limit })));
+        searchPromises.push(searchScoped({ client, scope: `<@${filteredTargetIds[0]}>`, topic, timeRange, limit }));
     }
     else if (filteredTargetIds.length >= 2) {
-        const mpimName = yield tryGetMPIMName(client, filteredTargetIds);
-        if (mpimName) {
-            allMatches.push(...(yield searchScoped({ client, scope: mpimName, topic, timeRange, limit })));
-        }
-        for (const id of filteredTargetIds) {
-            allMatches.push(...(yield searchScoped({ client, scope: `<@${id}>`, topic, timeRange, limit })));
-        }
+        // Run MPIM lookup and individual DM searches in parallel
+        const searchMPIM = () => __awaiter(void 0, void 0, void 0, function* () {
+            const mpimName = yield tryGetMPIMName(client, filteredTargetIds);
+            return mpimName ? searchScoped({ client, scope: mpimName, topic, timeRange, limit }) : [];
+        });
+        searchPromises.push(searchMPIM());
+        // Add individual DM searches
+        searchPromises.push(...filteredTargetIds.map(id => searchScoped({ client, scope: `<@${id}>`, topic, timeRange, limit })));
     }
     else if (channel) {
-        allMatches.push(...(yield searchScoped({ client, scope: normalizeChannelOperand(channel), topic, timeRange, limit })));
+        searchPromises.push(searchScoped({ client, scope: normalizeChannelOperand(channel), topic, timeRange, limit }));
     }
-    // --- Topic-wide search ---
-    const topicMatches = topic ? yield searchByTopic({ client, topic, timeRange, limit }) : [];
-    allMatches.push(...topicMatches);
+    // Topic-wide search in parallel
+    searchPromises.push(...(topic ? [searchByTopic({ client, topic, timeRange, limit })] : []));
+    // Execute all searches in parallel
+    const searchResults = yield Promise.all(searchPromises);
+    searchResults.forEach(matches => allMatches.push(...matches));
     // --- Expand hits with context + filter overlap ---
+    // Create a channel info cache to avoid redundant API calls
+    const channelInfoCache = new Map();
     const expanded = yield Promise.all(allMatches.map(m => limitHit(() => __awaiter(void 0, void 0, void 0, function* () {
-        var _a, _b, _c, _d, _e, _f;
+        var _a, _b, _c, _d, _e, _f, _g, _h;
         if (!m.ts || !((_a = m.channel) === null || _a === void 0 ? void 0 : _a.id))
             return null;
         const anchor = yield fetchOneMessage(client, m.channel.id, m.ts);
         const rootTs = (anchor === null || anchor === void 0 ? void 0 : anchor.thread_ts) || m.ts;
         let members = [];
-        // Check convo type (DM, MPIM, channel)
-        const convoInfo = yield client.conversations.info({ channel: m.channel.id });
-        const isIm = (_b = convoInfo.channel) === null || _b === void 0 ? void 0 : _b.is_im;
-        const isMpim = (_c = convoInfo.channel) === null || _c === void 0 ? void 0 : _c.is_mpim;
+        // Check convo type (DM, MPIM, channel) with caching
+        let channelInfo = channelInfoCache.get(m.channel.id);
+        if (!channelInfo) {
+            const convoInfo = yield client.conversations.info({ channel: m.channel.id });
+            channelInfo = {
+                isIm: (_c = (_b = convoInfo.channel) === null || _b === void 0 ? void 0 : _b.is_im) !== null && _c !== void 0 ? _c : false,
+                isMpim: (_e = (_d = convoInfo.channel) === null || _d === void 0 ? void 0 : _d.is_mpim) !== null && _e !== void 0 ? _e : false,
+            };
+            channelInfoCache.set(m.channel.id, channelInfo);
+        }
+        const { isIm, isMpim } = channelInfo;
         const [contextMsgs, permalink] = (anchor === null || anchor === void 0 ? void 0 : anchor.thread_ts)
             ? [yield fetchThread(client, m.channel.id, rootTs), yield getPermalink(client, m.channel.id, rootTs)]
             : [yield fetchContextWindow(client, m.channel.id, m.ts), yield getPermalink(client, m.channel.id, m.ts)];
         let passesFilter = false;
         if (isIm || isMpim) {
             // DM/MPIM: use members, not authorship
-            const membersRes = (_d = (yield client.conversations.members({ channel: m.channel.id })).members) !== null && _d !== void 0 ? _d : [];
+            const membersRes = (_f = (yield client.conversations.members({ channel: m.channel.id })).members) !== null && _f !== void 0 ? _f : [];
             members = yield Promise.all(membersRes.map((uid) => __awaiter(void 0, void 0, void 0, function* () {
                 const u = yield cache.get(uid);
                 return { userId: uid, userEmail: u === null || u === void 0 ? void 0 : u.email, userName: u === null || u === void 0 ? void 0 : u.name };
@@ -302,8 +329,8 @@ const searchSlack = (_a) => __awaiter(void 0, [_a], void 0, function* ({ params,
             channelId: m.channel.id,
             ts: rootTs,
             text: (anchor === null || anchor === void 0 ? void 0 : anchor.text) ? yield expandSlackEntities(client, cache, anchor.text) : undefined,
-            userEmail: (anchor === null || anchor === void 0 ? void 0 : anchor.user) ? (_e = (yield cache.get(anchor.user))) === null || _e === void 0 ? void 0 : _e.email : undefined,
-            userName: (anchor === null || anchor === void 0 ? void 0 : anchor.user) ? (_f = (yield cache.get(anchor.user))) === null || _f === void 0 ? void 0 : _f.name : undefined,
+            userEmail: (anchor === null || anchor === void 0 ? void 0 : anchor.user) ? (_g = (yield cache.get(anchor.user))) === null || _g === void 0 ? void 0 : _g.email : undefined,
+            userName: (anchor === null || anchor === void 0 ? void 0 : anchor.user) ? (_h = (yield cache.get(anchor.user))) === null || _h === void 0 ? void 0 : _h.name : undefined,
             context,
             permalink,
             members,

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@credal/actions",
-  "version": "0.2.147",
+  "version": "0.2.149",
   "type": "module",
   "description": "AI Actions by Credal AI",
   "sideEffects": false,