@cursor/july 0.1.97 → 0.1.100

package/src/channels/slack/api.ts

@@ -16,7 +16,7 @@ import {
 } from "./constants.js";
 import { tryResolveAppToken, tryResolveBotToken } from "./credentials.js";
 import { defaultSlackLog, type SlackLog } from "./log.js";
-import type { StreamTaskChunk } from "./stream.js";
+import type { StreamChunk } from "./stream.js";
 import type {
   SlackApiTransport,
   SlackBinding,
@@ -194,10 +194,18 @@ export function truncateTitle(title: string, max?: number): string {
 // ---------------------------------------------------------------------------
 
 /**
- * Show or clear the Working chip. Agent UX ignores empty
- * `assistant.threads.setStatus`; `agents.sessions.setStatus` `active` is
- * what actually clears it. The assistant call stays for loading_messages
- * and older workspaces.
+ * Show or clear the status chip under the thread.
+ *
+ * Status text goes through `assistant.threads.setStatus` alone. The agent
+ * session is only written when clearing: Agent UX ignores an empty assistant
+ * status, so `agents.sessions.setStatus` `active` is what actually drops the
+ * chip. That same write must not accompany status text — Slack applies it
+ * asynchronously and it blanks the chip when it lands after the text, which
+ * reads as flicker between updates. A `processing` write would be worse:
+ * Agent UX paints its own generic "is working…" over the assistant text
+ * while the session is `processing`. An open `chat.startStream` also flips
+ * the session to `processing` on Slack's side, so status text only shows on
+ * non-streaming turns. The empty assistant call stays for older workspaces.
  */
 export async function setThreadStatus(args: {
   botToken: SlackBotToken | undefined;
@@ -210,21 +218,12 @@ export async function setThreadStatus(args: {
     return { ok: false, error: "missing_channel_or_thread" };
   }
   const status = truncateTypingStatus(args.status);
-  const sessionResult = await callSlackApiSoft({
-    botToken: args.botToken,
-    operation: "agents.sessions.setStatus",
-    body: {
-      channel_id: args.channelId,
-      thread_ts: args.threadTs,
-      status: status.length > 0 ? "processing" : "active",
-    },
-  });
-
   const body: Record<string, unknown> = {
     channel_id: args.channelId,
     thread_ts: args.threadTs,
     status,
   };
+
   if (status.length > 0) {
     const loadingRaw =
       args.loadingMessages !== undefined && args.loadingMessages.length > 0
@@ -237,7 +236,22 @@ export async function setThreadStatus(args: {
     if (loading_messages.length > 0) {
       body.loading_messages = loading_messages;
     }
+    return callSlackApiSoft({
+      botToken: args.botToken,
+      operation: "assistant.threads.setStatus",
+      body,
+    });
   }
+
+  const sessionResult = await callSlackApiSoft({
+    botToken: args.botToken,
+    operation: "agents.sessions.setStatus",
+    body: {
+      channel_id: args.channelId,
+      thread_ts: args.threadTs,
+      status: "active",
+    },
+  });
   const assistantResult = await callSlackApiSoft({
     botToken: args.botToken,
     operation: "assistant.threads.setStatus",
@@ -300,7 +314,7 @@ export async function startChatStream(args: {
   channel: string;
   thread_ts?: string;
   markdown_text?: string;
-  chunks?: StreamTaskChunk[];
+  chunks?: StreamChunk[];
   recipient_user_id?: string;
   recipient_team_id?: string;
 }): Promise<SoftSlackResult<{ ts: string; channel: string }>> {
@@ -346,7 +360,7 @@ export async function appendChatStream(args: {
   channel: string;
   ts: string;
   markdown_text?: string;
-  chunks?: StreamTaskChunk[];
+  chunks?: StreamChunk[];
 }): Promise<SoftSlackResult> {
   const body: Record<string, unknown> = {
     channel: args.channel,
@@ -370,6 +384,7 @@ export async function stopChatStream(args: {
   channel: string;
   ts: string;
   markdown_text?: string;
+  chunks?: StreamChunk[];
   blocks?: unknown[];
 }): Promise<SoftSlackResult> {
   const body: Record<string, unknown> = {
@@ -379,6 +394,9 @@ export async function stopChatStream(args: {
   if (args.markdown_text !== undefined) {
     body.markdown_text = args.markdown_text;
   }
+  if (args.chunks !== undefined && args.chunks.length > 0) {
+    body.chunks = args.chunks;
+  }
   if (args.blocks !== undefined && args.blocks.length > 0) {
     body.blocks = args.blocks;
   }

package/src/channels/slack/constants.ts

@@ -13,6 +13,12 @@ export const SLACK_LOADING_MESSAGE_MAX_LEN = 50;
 /** Slack `assistant.threads.setStatus` accepts at most 5 `loading_messages`. */
 export const SLACK_LOADING_MESSAGES_MAX = 5;
 export const SLACK_TITLE_MAX_LEN = 80;
+/**
+ * `markdown_text` (and so a streamed message, cards included) tops out near
+ * 12k characters; past it Slack answers `msg_too_long`. Plain `text` allows
+ * about 40k.
+ */
+export const SLACK_MARKDOWN_TEXT_MAX_LEN = 12_000;
 
 /** Rotating assistant status lines (Slack `loading_messages`). */
 export const SLACK_DEFAULT_LOADING_MESSAGES = [