@lightupai/polaris 0.0.38 → 0.0.39

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@lightupai/polaris",
-  "version": "0.0.38",
+  "version": "0.0.39",
   "type": "module",
   "bin": {
     "polaris": "bin/polaris",

package/src/service/server.ts

@@ -502,23 +502,45 @@ export async function startServer(opts: {
           }
         }
 
-        // Generate short aliases from first name, disambiguate collisions
+        // Generate short aliases: prefer display_name, fall back to name, then handle
+        // Filter out slackbot
+        const filtered = team.filter((m) => m.slack_handle !== "slackbot");
+        team.length = 0;
+        team.push(...filtered);
+
+        function deriveAlias(m: typeof team[0]): string {
+          // Best source: display name (what the person chose)
+          const displayName = m.slack_display?.trim();
+          if (displayName) {
+            const first = displayName.split(/\s+/)[0]?.toLowerCase().replace(/[^a-z]/g, "") || "";
+            if (first.length >= 2) return first;
+          }
+          // Next: real name, but skip short/initial-only first names
+          const parts = m.name.split(/\s+/);
+          for (const part of parts) {
+            const clean = part.toLowerCase().replace(/[^a-z]/g, "");
+            if (clean.length >= 2) return clean;
+          }
+          // Fall back to slack handle
+          return m.slack_handle || "";
+        }
+
         const aliasCounts = new Map<string, number>();
         for (const m of team) {
-          const firstName = m.name.split(/\s+/)[0]?.toLowerCase().replace(/[^a-z]/g, "") || "";
-          if (firstName) aliasCounts.set(firstName, (aliasCounts.get(firstName) ?? 0) + 1);
+          const alias = deriveAlias(m);
+          if (alias) aliasCounts.set(alias, (aliasCounts.get(alias) ?? 0) + 1);
         }
         for (const m of team) {
-          const parts = m.name.split(/\s+/);
-          const firstName = parts[0]?.toLowerCase().replace(/[^a-z]/g, "") || "";
-          if (!firstName) {
+          const alias = deriveAlias(m);
+          if (!alias) {
             (m as Record<string, unknown>).alias = null;
-          } else if ((aliasCounts.get(firstName) ?? 0) > 1 && parts.length > 1) {
-            // Collision — append first letter of last name
-            const lastInitial = parts[parts.length - 1]?.[0]?.toLowerCase() ?? "";
-            (m as Record<string, unknown>).alias = `${firstName}${lastInitial}`;
+          } else if ((aliasCounts.get(alias) ?? 0) > 1) {
+            // Collision — append first letter of last name or use handle
+            const parts = m.name.split(/\s+/);
+            const lastInitial = parts.length > 1 ? parts[parts.length - 1]?.[0]?.toLowerCase() ?? "" : "";
+            (m as Record<string, unknown>).alias = lastInitial ? `${alias}${lastInitial}` : m.slack_handle || alias;
           } else {
-            (m as Record<string, unknown>).alias = firstName;
+            (m as Record<string, unknown>).alias = alias;
           }
         }