@credal/actions 0.2.111 → 0.2.112

package/dist/actions/providers/slackUser/searchSlack.d.ts

@@ -4,13 +4,14 @@ export interface SlackSearchMessage {
     channelId: string;
     ts: string;
     text?: string;
-    userId?: string;
+    userEmail?: string;
     permalink?: string;
     /** If thread: full thread (root first). If not thread: small context window around the hit. */
     context?: Array<{
         ts: string;
         text?: string;
-        userId?: string;
+        userEmail?: string;
+        userName?: string;
     }>;
 }
 declare const searchSlack: slackUserSearchSlackFunction;

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

@@ -12,6 +12,34 @@ import { MISSING_AUTH_TOKEN } from "../../util/missingAuthConstants.js";
 import pLimit from "p-limit";
 const HIT_ENRICH_POOL = 10;
 const limitHit = pLimit(HIT_ENRICH_POOL);
+class SlackUserCache {
+    constructor(client) {
+        this.client = client;
+        this.cache = new Map();
+        this.client = client;
+    }
+    get(id) {
+        return __awaiter(this, void 0, void 0, function* () {
+            var _a, _b, _c, _d, _e, _f, _g, _h;
+            const result = this.cache.get(id);
+            if (result)
+                return result;
+            const res = yield this.client.users.info({ user: id });
+            const u = {
+                name: (_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 : "",
+                email: (_h = (_g = (_f = res.user) === null || _f === void 0 ? void 0 : _f.profile) === null || _g === void 0 ? void 0 : _g.email) !== null && _h !== void 0 ? _h : "",
+            };
+            if (res.user && id && res.user.name) {
+                this.cache.set(id, u);
+                return u;
+            }
+            return undefined;
+        });
+    }
+    set(id, { email, name }) {
+        this.cache.set(id, { email, name });
+    }
+}
 /* ===================== Helpers ===================== */
 function normalizeChannelOperand(ch) {
     const s = ch.trim();
@@ -33,7 +61,7 @@ function timeFilter(range) {
             return "";
     }
 }
-function lookupUserIdsByEmail(client, emails) {
+function lookupUserIdsByEmail(client, emails, slackUserCache) {
     return __awaiter(this, void 0, void 0, function* () {
         const ids = [];
         const tasks = emails.map((raw) => __awaiter(this, void 0, void 0, function* () {
@@ -43,6 +71,9 @@ function lookupUserIdsByEmail(client, emails) {
                 return null;
             const res = yield client.users.lookupByEmail({ email });
             const id = (_a = res.user) === null || _a === void 0 ? void 0 : _a.id;
+            if (res.user && id && res.user.name) {
+                slackUserCache.set(id, { email, name: res.user.name });
+            }
             if (id)
                 return id;
             return null;
@@ -132,10 +163,11 @@ const searchSlack = (_a) => __awaiter(void 0, [_a], void 0, function* ({ params,
         throw new Error(MISSING_AUTH_TOKEN);
     }
     const client = new WebClient(authParams.authToken);
+    const slackUserCache = new SlackUserCache(client);
     const { emails, channel, topic, timeRange, limit } = params;
     const parts = [];
     if (emails === null || emails === void 0 ? void 0 : emails.length) {
-        const userIds = yield lookupUserIdsByEmail(client, emails);
+        const userIds = yield lookupUserIdsByEmail(client, emails, slackUserCache);
         const { user_id: myUserId } = yield client.auth.test();
         if (!myUserId)
             throw new Error("Failed to get my user ID.");
@@ -164,17 +196,20 @@ const searchSlack = (_a) => __awaiter(void 0, [_a], void 0, function* ({ params,
     const count = Math.max(1, Math.min(100, limit));
     const searchRes = yield client.search.messages({ query, count, highlight: true });
     const matches = (_c = (_b = searchRes.messages) === null || _b === void 0 ? void 0 : _b.matches) !== null && _c !== void 0 ? _c : [];
-    const hits = matches.slice(0, limit).map(m => {
-        var _a, _b;
-        return ({
+    const hitsPromises = matches.slice(0, limit).map((m) => __awaiter(void 0, void 0, void 0, function* () {
+        var _a, _b, _c, _d;
+        const user = m.user ? yield slackUserCache.get(m.user) : undefined;
+        return {
             channelId: ((_a = m.channel) === null || _a === void 0 ? void 0 : _a.id) || ((_b = m.channel) === null || _b === void 0 ? void 0 : _b.name) || "",
             ts: m.ts,
             text: m.text,
-            userId: m.user,
-        });
-    });
+            userEmail: (_c = user === null || user === void 0 ? void 0 : user.email) !== null && _c !== void 0 ? _c : undefined,
+            userName: (_d = user === null || user === void 0 ? void 0 : user.name) !== null && _d !== void 0 ? _d : undefined,
+        };
+    }));
+    const hits = yield Promise.all(hitsPromises);
     const tasks = hits.map(h => limitHit(() => __awaiter(void 0, void 0, void 0, function* () {
-        var _a, _b, _c, _d;
+        var _a, _b, _c, _d, _e, _f;
         if (!h.ts)
             return null;
         try {
@@ -186,12 +221,26 @@ const searchSlack = (_a) => __awaiter(void 0, [_a], void 0, function* ({ params,
                     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 }));
+                const contextPromises = thread
+                    .filter(t => t.ts)
+                    .map((t) => __awaiter(void 0, void 0, void 0, function* () {
+                    var _a, _b;
+                    const user = t.user ? yield slackUserCache.get(t.user) : undefined;
+                    return {
+                        ts: t.ts,
+                        text: t.text,
+                        userEmail: (_a = user === null || user === void 0 ? void 0 : user.email) !== null && _a !== void 0 ? _a : undefined,
+                        userName: (_b = user === null || user === void 0 ? void 0 : user.name) !== null && _b !== void 0 ? _b : undefined,
+                    };
+                }));
+                const context = yield Promise.all(contextPromises);
+                const user = anchor.user ? yield slackUserCache.get(anchor.user) : undefined;
                 return {
                     channelId: h.channelId,
                     ts: rootTs,
                     text: (_a = anchor.text) !== null && _a !== void 0 ? _a : h.text,
-                    userId: (_b = anchor.user) !== null && _b !== void 0 ? _b : h.userId,
+                    userEmail: (_b = user === null || user === void 0 ? void 0 : user.email) !== null && _b !== void 0 ? _b : h.userEmail,
+                    userName: (_c = user === null || user === void 0 ? void 0 : user.name) !== null && _c !== void 0 ? _c : h.userName,
                     context,
                     permalink,
                 };
@@ -202,24 +251,39 @@ const searchSlack = (_a) => __awaiter(void 0, [_a], void 0, function* ({ params,
                     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 }));
+                const contextPromises = ctx
+                    .filter(t => t.ts)
+                    .map((t) => __awaiter(void 0, void 0, void 0, function* () {
+                    var _a, _b;
+                    const user = t.user ? yield slackUserCache.get(t.user) : undefined;
+                    return {
+                        ts: t.ts,
+                        text: t.text,
+                        userEmail: (_a = user === null || user === void 0 ? void 0 : user.email) !== null && _a !== void 0 ? _a : undefined,
+                        userName: (_b = user === null || user === void 0 ? void 0 : user.name) !== null && _b !== void 0 ? _b : undefined,
+                    };
+                }));
+                const context = yield Promise.all(contextPromises);
+                const user = (anchor === null || anchor === void 0 ? void 0 : anchor.user) ? yield slackUserCache.get(anchor.user) : undefined;
                 return {
                     channelId: h.channelId,
                     ts: 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,
+                    text: (_d = anchor === null || anchor === void 0 ? void 0 : anchor.text) !== null && _d !== void 0 ? _d : h.text,
+                    userEmail: (_e = user === null || user === void 0 ? void 0 : user.email) !== null && _e !== void 0 ? _e : h.userEmail,
+                    userName: (_f = user === null || user === void 0 ? void 0 : user.name) !== null && _f !== void 0 ? _f : h.userName,
                     context,
                     permalink,
                 };
             }
         }
-        catch (_e) {
+        catch (_g) {
             // fallback minimal object; still in parallel
             return {
                 channelId: h.channelId,
                 ts: h.ts,
                 text: h.text,
-                userId: h.userId,
+                userEmail: h.userEmail,
+                userName: h.userName,
                 permalink: yield getPermalink(client, h.channelId, h.ts),
             };
         }

package/package.json

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