@cursor/july 0.1.16 → 0.1.18

package/src/channels/slack/thread-context.ts

@@ -8,6 +8,7 @@
 import { callSlackApiSoft } from "./api.js";
 import { slackTextToMarkdown } from "./inbound.js";
 import { defaultSlackLog, type SlackLog } from "./log.js";
+import { slackApiMessageBody } from "./message-body.js";
 import type { SlackBotToken, SoftSlackResult } from "./types.js";
 
 /** Cap replies fetched from Slack (plus the parent). */
@@ -95,75 +96,6 @@ function asString(value: unknown): string | undefined {
   return typeof value === "string" ? value : undefined;
 }
 
-/** Pull readable text out of Slack attachment / Block Kit payloads. */
-export function slackApiMessageBody(message: JsonRecord): string {
-  const parts: string[] = [];
-  const text = asString(message.text);
-  if (text !== undefined && text.trim() !== "") {
-    parts.push(text);
-  }
-
-  const attachments = Array.isArray(message.attachments)
-    ? message.attachments
-    : [];
-  for (const attachment of attachments) {
-    const rec = asRecord(attachment);
-    if (rec === null) {
-      continue;
-    }
-    for (const key of [
-      "pretext",
-      "title",
-      "text",
-      "fallback",
-      "footer",
-    ] as const) {
-      const value = asString(rec[key]);
-      if (value !== undefined && value.trim() !== "") {
-        parts.push(value);
-      }
-    }
-    const titleLink = asString(rec.title_link);
-    if (titleLink !== undefined && titleLink.trim() !== "") {
-      parts.push(titleLink);
-    }
-  }
-
-  const blocks = Array.isArray(message.blocks) ? message.blocks : [];
-  for (const block of blocks) {
-    const rec = asRecord(block);
-    if (rec === null) {
-      continue;
-    }
-    const blockText = asRecord(rec.text);
-    const blockTextValue = asString(blockText?.text);
-    if (blockTextValue !== undefined && blockTextValue.trim() !== "") {
-      parts.push(blockTextValue);
-    }
-    const fields = Array.isArray(rec.fields) ? rec.fields : [];
-    for (const field of fields) {
-      const fieldRec = asRecord(field);
-      const fieldText = asString(fieldRec?.text);
-      if (fieldText !== undefined && fieldText.trim() !== "") {
-        parts.push(fieldText);
-      }
-    }
-  }
-
-  // Dedupe identical chunks (Datadog often repeats text/fallback across fields).
-  const seen = new Set<string>();
-  const deduped: string[] = [];
-  for (const part of parts) {
-    const trimmed = part.trim();
-    if (trimmed === "" || seen.has(trimmed)) {
-      continue;
-    }
-    seen.add(trimmed);
-    deduped.push(trimmed);
-  }
-  return deduped.join("\n");
-}
-
 function formatThreadEntry(
   message: JsonRecord,
   role: "parent" | "reply"

package/src/channels/slack/types.ts

@@ -89,8 +89,16 @@ export interface SlackAuthor {
 }
 
 export interface SlackMessage {
+  /** Raw top-level `text` exactly as Slack sent it. Often empty on bot posts. */
   text: string;
-  /** Text roughly suitable for the model (mention markers cleaned). */
+  /**
+   * Raw mrkdwn body: top-level `text` plus attachment / Block Kit content.
+   * Alert feeds (Datadog, PagerDuty) leave `text` empty and put the whole
+   * notification in attachments, so read `body` for message content. Unlike
+   * {@link markdown} this keeps `<url|label>` links intact.
+   */
+  body: string;
+  /** Body roughly suitable for the model (mention markers cleaned). */
   markdown: string;
   ts: string;
   threadTs: string;
@@ -175,6 +183,83 @@ export type SlackMessageHandler = (
   | undefined
   | Promise<SlackDispatchDecision | null | undefined>;
 
+/**
+ * Host probes into a Slack thread's continuation. Optional on dispatch
+ * deps; the serve host wires them at channel start.
+ */
+export interface SlackContinuationProbes {
+  /**
+   * When set, full-thread hydration runs only for cold-start turns (no
+   * existing session). Warm turns load a `<slack_thread_delta>` instead.
+   */
+  hasContinuationSession?: (continuationToken: string) => Promise<boolean>;
+  /** True when a turn is already running for this Slack continuation. */
+  isContinuationBusy?: (continuationToken: string) => Promise<boolean>;
+  /**
+   * Warm-delta watermark from session channel state (`lastBotMessageTs`).
+   * When absent, delta falls back to the last matching bot post in-thread.
+   */
+  getContinuationLastBotMessageTs?: (
+    continuationToken: string
+  ) => Promise<string | undefined>;
+  /** Cancel the active turn + clear coalesced nudges for this continuation. */
+  interruptContinuation?: (continuationToken: string) => Promise<boolean>;
+}
+
+/**
+ * A Block Kit action click (Socket Mode `block_actions` payload) that the
+ * framework does not own — the built-in tool-approval Approve/Deny actions
+ * are routed to `resolveApproval` before author handlers see anything.
+ */
+export interface SlackBlockAction {
+  /** `action_id` of the clicked element. */
+  actionId: string;
+  /** `block_id` of the actions row holding the element (Slack always sets one). */
+  blockId: string;
+  /**
+   * Element `value` payload; empty string when absent (button-style
+   * elements only — select/overflow `selected_option`s are not read).
+   */
+  value: string;
+  /** Plain-text label of the clicked element, when present. */
+  label?: string;
+  channelId: string;
+  /**
+   * Thread of the message carrying the element (the message's own ts when
+   * it is a top-level post — clicks on a thread root stay in its thread).
+   */
+  threadTs: string;
+  /** The message carrying the clicked element. */
+  messageTs: string;
+  /** That message's blocks, for `chat.update` resolution cards. */
+  messageBlocks?: unknown[];
+  /** Slack user who clicked. */
+  userId: string;
+  userName?: string;
+  teamId?: string;
+  /**
+   * Slack `action_ts` — unique per click; block-action dispatch dedupes on
+   * it. Empty when the payload omitted it (malformed; never dispatched).
+   */
+  actionTs: string;
+}
+
+/**
+ * Author handler for {@link SlackBlockAction} clicks. Return a
+ * {@link SlackDispatchDecision} to dispatch a turn on the click's thread —
+ * the click is dispatched like an inbound message from the clicking user
+ * (thread-scoped continuation, `<slack_context>` naming the clicker) — or
+ * `null` to ignore the click.
+ */
+export type SlackBlockActionHandler = (
+  ctx: SlackBinding,
+  action: SlackBlockAction
+) =>
+  | SlackDispatchDecision
+  | null
+  | undefined
+  | Promise<SlackDispatchDecision | null | undefined>;
+
 /**
  * Opt-in channel watch: dispatch on posts in explicitly allowlisted
  * channels without requiring an @mention. There is deliberately no
@@ -371,6 +456,18 @@ export interface SlackChannelOptions {
    * only — rejected on `cursorAccount: true`.
    */
   interactivity?: boolean;
+  /**
+   * Handler for Block Kit actions the framework does not own (the built-in
+   * tool-approval Approve/Deny clicks are routed to `resolveApproval`
+   * first). Return a {@link SlackDispatchDecision} — typically with a
+   * thread-scoped auth via `buildSlackAuthContext` so replies in the same
+   * thread continue the session — to dispatch a turn on the click's
+   * thread, or `null` to ignore the click. Setting this implies
+   * interactive routing; the Slack app manifest still needs
+   * `interactivity` enabled (`buildSlackManifest({ interactivity: true })`).
+   * Socket Mode only — rejected on `cursorAccount: true`.
+   */
+  onBlockAction?: SlackBlockActionHandler;
   /**
    * Suggested prompts shown when an assistant thread starts (and optionally
    * after a turn completes).
@@ -416,6 +513,13 @@ export interface SlackManifestOptions {
   channelPosts?: boolean;
   /** Seed suggested prompts into the manifest `assistant_view`. */
   suggestedPrompts?: Array<{ title: string; message: string }>;
+  /**
+   * Emit `settings.managed_app_settings`. Defaults to `true` — Cursor
+   * provisions these apps through its manager app. Pass `false` for a
+   * manifest a human pastes into api.slack.com: only a manager user token
+   * may set that block.
+   */
+  managedByCursor?: boolean;
 }
 
 /** Soft-fail result for optional Slack API features. */

package/src/connections.ts

@@ -30,8 +30,10 @@
  * to be signed in (`agent-serve login` or `CURSOR_API_KEY`):
  *
  * ```ts
- * // agent/mcp-connections/cursor.ts — every connected server
+ * // agent/mcp-connections/cursor.ts — every connected server (no allowlist)
  * export default defineConnection({ cursorAccount: true });
+ * // equivalent explicit form:
+ * export default defineConnection({ cursorAccount: true, servers: "*" });
  *
  * // or restrict to specific connectors by name
  * export default defineConnection({ cursorAccount: true, servers: ["Linear"] });
@@ -86,6 +88,7 @@ export type {
   HostMcpRegistry,
   HostMcpToolInfo,
 } from "./types.js";
+export { normalizeCursorAccountServers } from "./types.js";
 
 export function defineConnection(
   config: ConnectionConfig