From 475299346a8019cfadb5726873a4e5733fd91728 Mon Sep 17 00:00:00 2001
From: Marcus Castro <mcaxtr@gmail.com>
Date: Tue, 10 Feb 2026 14:55:53 -0300
Subject: [PATCH] fix: use ok:true wrapper for emojiList consistent with other
 actions

---
 src/agents/tools/slack-actions.ts | 5 ++---
 1 file changed, 2 insertions(+), 3 deletions(-)

diff --git a/src/agents/tools/slack-actions.ts b/src/agents/tools/slack-actions.ts
index 4fc7d410157..97198e3fe7e 100644
--- a/src/agents/tools/slack-actions.ts
+++ b/src/agents/tools/slack-actions.ts
@@ -312,18 +312,17 @@ export async function handleSlackAction(
       throw new Error("Slack emoji list is disabled.");
     }
     const result = readOpts ? await listSlackEmojis(readOpts) : await listSlackEmojis();
-    const ok = result.ok ?? false;
     const limit = readNumberParam(params, "limit", { integer: true });
     if (limit != null && limit > 0 && result.emoji != null) {
       const entries = Object.entries(result.emoji).toSorted(([a], [b]) => a.localeCompare(b));
       if (entries.length > limit) {
         return jsonResult({
-          ok,
+          ok: true,
           emojis: { ...result, emoji: Object.fromEntries(entries.slice(0, limit)) },
         });
       }
     }
-    return jsonResult({ ok, emojis: result });
+    return jsonResult({ ok: true, emojis: result });
   }
 
   throw new Error(`Unknown action: ${action}`);