@credal/actions 0.2.110 → 0.2.111

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

@@ -9,6 +9,9 @@ var __awaiter = (this && this.__awaiter) || function (thisArg, _arguments, P, ge
 };
 import { WebClient } from "@slack/web-api";
 import { MISSING_AUTH_TOKEN } from "../../util/missingAuthConstants.js";
+import pLimit from "p-limit";
+const HIT_ENRICH_POOL = 10;
+const limitHit = pLimit(HIT_ENRICH_POOL);
 /* ===================== Helpers ===================== */
 function normalizeChannelOperand(ch) {
     const s = ch.trim();
@@ -32,17 +35,22 @@ function timeFilter(range) {
 }
 function lookupUserIdsByEmail(client, emails) {
     return __awaiter(this, void 0, void 0, function* () {
-        var _a;
         const ids = [];
-        for (const raw of emails) {
+        const tasks = emails.map((raw) => __awaiter(this, void 0, void 0, function* () {
+            var _a;
             const email = raw.trim();
             if (!email)
-                continue;
+                return null;
             const res = yield client.users.lookupByEmail({ email });
             const id = (_a = res.user) === null || _a === void 0 ? void 0 : _a.id;
             if (id)
-                ids.push(id);
-        }
+                return id;
+            return null;
+        }));
+        const settled = yield Promise.allSettled(tasks);
+        for (const r of settled)
+            if (r.status === "fulfilled" && r.value)
+                ids.push(r.value);
         return ids;
     });
 }
@@ -119,7 +127,7 @@ function fetchContextWindow(client, channel, ts) {
 }
 /* ===================== Main Export ===================== */
 const searchSlack = (_a) => __awaiter(void 0, [_a], void 0, function* ({ params, authParams, }) {
-    var _b, _c, _d, _e, _f, _g;
+    var _b, _c;
     if (!authParams.authToken) {
         throw new Error(MISSING_AUTH_TOKEN);
     }
@@ -128,13 +136,17 @@ const searchSlack = (_a) => __awaiter(void 0, [_a], void 0, function* ({ params,
     const parts = [];
     if (emails === null || emails === void 0 ? void 0 : emails.length) {
         const userIds = yield lookupUserIdsByEmail(client, emails);
-        if (userIds.length === 0)
+        const { user_id: myUserId } = yield client.auth.test();
+        if (!myUserId)
+            throw new Error("Failed to get my user ID.");
+        const userIdsWithoutMe = userIds.filter(id => id !== myUserId);
+        if (userIdsWithoutMe.length === 0)
             throw new Error("No users resolved from emails.");
-        if (userIds.length == 1) {
-            parts.push(`in:<@${userIds[0]}>`);
+        if (userIdsWithoutMe.length == 1) {
+            parts.push(`in:<@${userIdsWithoutMe[0]}>`);
         }
         else {
-            const convoName = yield getMPIMName(client, userIds);
+            const convoName = yield getMPIMName(client, userIdsWithoutMe);
             parts.push(`in:${convoName}`);
         }
     }
@@ -161,58 +173,62 @@ const searchSlack = (_a) => __awaiter(void 0, [_a], void 0, function* ({ params,
             userId: m.user,
         });
     });
-    const results = [];
-    for (const h of hits) {
+    const tasks = hits.map(h => limitHit(() => __awaiter(void 0, void 0, void 0, function* () {
+        var _a, _b, _c, _d;
         if (!h.ts)
-            continue;
+            return null;
         try {
             const anchor = yield fetchOneMessage(client, h.channelId, h.ts);
             const rootTs = (anchor === null || anchor === void 0 ? void 0 : anchor.thread_ts) || h.ts;
             if (anchor === null || anchor === void 0 ? void 0 : anchor.thread_ts) {
-                const thread = yield fetchThread(client, h.channelId, rootTs);
-                const normalizedThreads = [];
-                for (const t of thread) {
-                    if (!t.ts)
-                        continue;
-                    normalizedThreads.push({ ts: t.ts, text: t.text, userId: t.user });
-                }
-                results.push({
+                // thread: fetch thread + permalink concurrently
+                const [thread, permalink] = yield Promise.all([
+                    fetchThread(client, h.channelId, rootTs),
+                    getPermalink(client, h.channelId, rootTs),
+                ]);
+                const context = thread.filter(t => t.ts).map(t => ({ ts: t.ts, text: t.text, userId: t.user }));
+                return {
                     channelId: h.channelId,
                     ts: rootTs,
-                    text: (_d = anchor.text) !== null && _d !== void 0 ? _d : h.text,
-                    userId: (_e = anchor.user) !== null && _e !== void 0 ? _e : h.userId,
-                    context: normalizedThreads,
-                    permalink: yield getPermalink(client, h.channelId, rootTs),
-                });
+                    text: (_a = anchor.text) !== null && _a !== void 0 ? _a : h.text,
+                    userId: (_b = anchor.user) !== null && _b !== void 0 ? _b : h.userId,
+                    context,
+                    permalink,
+                };
             }
             else {
-                const ctx = yield fetchContextWindow(client, h.channelId, h.ts);
-                const normalizedThreads = [];
-                for (const t of ctx) {
-                    if (!t.ts)
-                        continue;
-                    normalizedThreads.push({ ts: t.ts, text: t.text, userId: t.user });
-                }
-                results.push({
+                // not a thread: fetch context window + permalink concurrently
+                const [ctx, permalink] = yield Promise.all([
+                    fetchContextWindow(client, h.channelId, h.ts),
+                    getPermalink(client, h.channelId, h.ts),
+                ]);
+                const context = ctx.filter(t => t.ts).map(t => ({ ts: t.ts, text: t.text, userId: t.user }));
+                return {
                     channelId: h.channelId,
                     ts: h.ts,
-                    text: (_f = anchor === null || anchor === void 0 ? void 0 : anchor.text) !== null && _f !== void 0 ? _f : h.text,
-                    userId: (_g = anchor === null || anchor === void 0 ? void 0 : anchor.user) !== null && _g !== void 0 ? _g : h.userId,
-                    context: normalizedThreads,
-                    permalink: yield getPermalink(client, h.channelId, h.ts),
-                });
+                    text: (_c = anchor === null || anchor === void 0 ? void 0 : anchor.text) !== null && _c !== void 0 ? _c : h.text,
+                    userId: (_d = anchor === null || anchor === void 0 ? void 0 : anchor.user) !== null && _d !== void 0 ? _d : h.userId,
+                    context,
+                    permalink,
+                };
             }
         }
-        catch (_h) {
-            results.push({
+        catch (_e) {
+            // fallback minimal object; still in parallel
+            return {
                 channelId: h.channelId,
                 ts: h.ts,
                 text: h.text,
                 userId: h.userId,
                 permalink: yield getPermalink(client, h.channelId, h.ts),
-            });
+            };
         }
-    }
+    })));
+    const settled = yield Promise.allSettled(tasks);
+    const results = [];
+    for (const r of settled)
+        if (r.status === "fulfilled" && r.value)
+            results.push(r.value);
     results.sort((a, b) => Number(b.ts) - Number(a.ts));
     return { query, results };
 });

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@credal/actions",
-  "version": "0.2.110",
+  "version": "0.2.111",
   "type": "module",
   "description": "AI Actions by Credal AI",
   "sideEffects": false,
@@ -66,6 +66,7 @@
     "mammoth": "^1.4.27",
     "mongodb": "^6.13.1",
     "node-forge": "^1.3.1",
+    "p-limit": "^7.1.1",
     "pdf2json": "^3.1.6",
     "resend": "^4.7.0",
     "snowflake-sdk": "^2.0.2",