chore: refine slack reactions and status

Author: LIU9293

package.json

@@ -1,6 +1,6 @@
 {
   "name": "ode",
-  "version": "0.0.12",
+  "version": "0.0.13",
   "description": "Ode - OpenCode chat controller for Slack",
   "module": "packages/core/index.ts",
   "type": "module",

packages/agents/shared.ts

@@ -42,7 +42,8 @@ export function buildSlackSystemPrompt(slack?: SlackContext): string {
     lines.push("- Payload: {\"action\":\"post_message\",\"channelId\":\"...\",\"threadId\":\"...\",\"messageId\":\"...\",\"text\":\"...\"}");
   }
   lines.push("- Supported actions: post_message, add_reaction, get_thread_messages, ask_user, get_user_info, upload_file.");
-  lines.push("- Required fields: channelId; threadId for thread actions; messageId for reactions; userId for get_user_info.");
+  lines.push("- Required fields: channelId; threadId for thread actions; messageId + emoji for reactions; userId for get_user_info.");
+  lines.push("- add_reaction schema: { action: \"add_reaction\", channelId: string, messageId: string, emoji: \"thumbsup\" | \"eyes\" | \"ok_hand\" }");
   lines.push("- You can use any tool available via bash, curl");
   lines.push("");
   lines.push("IMPORTANT: Your text output is automatically posted to Slack.");

packages/ims/slack/api.ts

@@ -39,13 +39,24 @@ function requireString(value: unknown, label: string): string {
   return value;
 }
 
+const ALLOWED_REACTIONS = new Set(["thumbsup", "eyes", "ok_hand"]);
+const REACTION_ALIASES: Record<string, string> = {
+  thumbup: "thumbsup",
+  ok: "ok_hand",
+};
+
 function normalizeEmojiName(emoji: string): string {
   const trimmed = emoji.trim();
   if (!trimmed) {
     throw new Error("emoji is required");
   }
   const stripped = trimmed.replace(/^:+|:+$/g, "").replace(/:/g, "");
-  return stripped || trimmed;
+  const normalized = stripped || trimmed;
+  const alias = REACTION_ALIASES[normalized] ?? normalized;
+  if (!ALLOWED_REACTIONS.has(alias)) {
+    throw new Error("emoji must be one of: thumbsup, eyes, ok_hand");
+  }
+  return alias;
 }
 
 function normalizeOptionLabel(option: unknown): string {

packages/ims/slack/client.ts

@@ -19,9 +19,7 @@ import { markdownToSlack, splitForSlack, truncateForSlack } from "./formatter";
 import {
   markThreadActive,
   isThreadActive,
-  getChannelSettings,
   getOpenCodeSession,
-  setOpenCodeSession,
   getGitHubAuthRecordForUser,
   getGitHubUserConfigDir,
   getPendingRestartMessages,
@@ -145,6 +143,26 @@ function getOdeSlackApiUrl(): string | undefined {
   return getSlackActionApiUrl();
 }
 
+async function buildSlackContext(
+  cwd: string,
+  channelId: string,
+  threadId: string,
+  userId: string,
+  threadHistory?: string | null
+): Promise<OpenCodeMessageContext> {
+  return {
+    threadHistory: threadHistory || undefined,
+    slack: {
+      channelId,
+      threadId,
+      userId,
+      threadHistory: threadHistory || undefined,
+      hasCustomSlackTool: await hasOdeSlackTool(cwd),
+      odeSlackApiUrl: getOdeSlackApiUrl(),
+    },
+  };
+}
+
 function buildGitEnvironmentForUser(userId: string): Record<string, string> {
   const env: Record<string, string> = {};
   const authRecord = getGitHubAuthRecordForUser(userId);
@@ -668,6 +686,13 @@ function buildToolDetails(tool: SessionMessageState["tools"][number], workingPat
   return title ? trimToolPath(title, workingPath) : "";
 }
 
+const TOOL_DETAIL_LIMIT = 30;
+
+function truncateToolDetail(detail: string): string {
+  if (detail.length <= TOOL_DETAIL_LIMIT) return detail;
+  return `${detail.slice(0, TOOL_DETAIL_LIMIT)}...`;
+}
+
 function buildToolLines(state: SessionMessageState, workingPath: string): string[] {
   const tools = state.tools || [];
   if (tools.length === 0) return [];
@@ -680,7 +705,8 @@ function buildToolLines(state: SessionMessageState, workingPath: string): string
   const lines = [header];
   for (const tool of items) {
     const details = buildToolDetails(tool, workingPath);
-    const suffix = details ? ` — ${details}` : "";
+    const truncated = details ? truncateToolDetail(details) : "";
+    const suffix = truncated ? ` — ${truncated}` : "";
     lines.push(`${getToolIcon(tool.status)} ${tool.name}${suffix}`);
   }
 
@@ -1410,17 +1436,13 @@ async function handleUserMessageInternal(
     ? await fetchThreadHistory(client, channelId, threadId, messageId)
     : null;
 
-  const messageContext: OpenCodeMessageContext = {
-    threadHistory: threadHistory || undefined,
-    slack: {
-      channelId,
-      threadId,
-      userId: threadOwnerUserId,
-      threadHistory: threadHistory || undefined,
-      hasCustomSlackTool: await hasOdeSlackTool(cwd),
-      odeSlackApiUrl: getOdeSlackApiUrl(),
-    },
-  };
+  const messageContext = await buildSlackContext(
+    cwd,
+    channelId,
+    threadId,
+    threadOwnerUserId,
+    threadHistory
+  );
 
   const onTodosUpdated = async (todos: TrackedTodo[]) => {
     await upsertPlanMessage(session, channelId, threadId, todos);
@@ -1721,15 +1743,12 @@ export async function handleButtonSelection(
 
   try {
     // Build context - the selection is the user's response
-    const messageContext = {
-      slack: {
-        channelId,
-        threadId,
-        userId: threadOwnerUserId,
-        hasCustomSlackTool: await hasOdeSlackTool(cwd),
-        odeSlackApiUrl: getOdeSlackApiUrl(),
-      },
-    };
+    const messageContext = await buildSlackContext(
+      cwd,
+      channelId,
+      threadId,
+      threadOwnerUserId
+    );
 
     // Send to OpenCode - the selection as the user's message
     const responses = await sendOpenCodeMessage(
@@ -1896,9 +1915,6 @@ export function setupMessageHandlers(): void {
         return;
       }
     }
-    if (!localMode) {
-    }
-
     if (localMode && !channelServerUrl) {
       await say({
         text: "OpenCode server URL missing for this channel. Set it in ~/.config/ode/ode.json.",

packages/utils/session-inspector.ts

@@ -77,8 +77,11 @@ export function buildSessionMessageState(
 
     if (type === "session.updated") {
       const title = eventData?.properties?.info?.title;
-      if (typeof title === "string" && title.trim()) {
-        state.sessionTitle = title.trim();
+      if (typeof title === "string") {
+        const trimmedTitle = title.trim();
+        if (trimmedTitle && !trimmedTitle.startsWith("New session")) {
+          state.sessionTitle = trimmedTitle;
+        }
       }
     }