@cursor/july 0.1.20 → 0.1.22

package/src/channels/slack/block-actions.test.ts

@@ -0,0 +1,452 @@
+import { describe, expect, it } from "vitest";
+import type { ChannelSession, SendMessageFn } from "../../types.js";
+import { blockActionAuth } from "./auth.js";
+import {
+  mrkdwnSections,
+  SLACK_SECTION_MRKDWN_MAX,
+  SLACK_TABLE_MAX_COLUMNS,
+  SLACK_TABLE_MAX_ROWS,
+  slackLinkCell,
+  slackRawTextCell,
+  slackTableBlock,
+} from "./blocks.js";
+import { createSlackLogger } from "./index.js";
+import {
+  dispatchBlockAction,
+  parseSlackBlockAction,
+  resolveBlockActionCard,
+} from "./interactive.js";
+import type { SlackBlockAction } from "./types.js";
+
+const testAuth = {
+  authenticator: "slack",
+  principalId: "slack:T1:C9:5.0",
+  principalType: "user" as const,
+  attributes: { channel_id: "C9", thread_ts: "5.0", user_id: "U7" },
+};
+
+function mockSession(id = "ses_click"): ChannelSession {
+  return {
+    id,
+    continuationToken: null,
+    isNew: true,
+    info: {
+      id,
+      channelId: "slack",
+      mode: "chat",
+      auth: testAuth,
+    },
+    getEventStream: async () => new ReadableStream(),
+    waitForCompletion: async () => ({ status: "finished" }),
+    setContinuationToken: async () => undefined,
+  };
+}
+
+function createSendRecorder(): {
+  send: SendMessageFn;
+  calls: Array<{ message: string; options: Parameters<SendMessageFn>[1] }>;
+} {
+  const calls: Array<{
+    message: string;
+    options: Parameters<SendMessageFn>[1];
+  }> = [];
+  const send: SendMessageFn = async (message, sendOptions) => {
+    calls.push({ message, options: sendOptions });
+    return mockSession();
+  };
+  return { send, calls };
+}
+
+function testHost() {
+  return {
+    mcp: {
+      names: () => [] as string[],
+      listTools: async () => [],
+      callTool: async () => ({ content: [] }),
+    },
+    github: {
+      getOctokit: async () => {
+        throw new Error("github client not configured in test");
+      },
+    },
+    slack: {
+      getClient: async () => {
+        throw new Error("slack client not configured in test");
+      },
+    },
+    kv: {
+      get: async () => undefined,
+      put: async () => {},
+      delete: async () => {},
+    },
+  };
+}
+
+function blockActionsPayload(
+  overrides: { actionId?: string; value?: string; threadTs?: string } = {}
+): Record<string, unknown> {
+  return {
+    type: "block_actions",
+    team: { id: "T1", domain: "acme" },
+    user: { id: "U7", username: "netto", team_id: "T1" },
+    channel: { id: "C9", name: "deploys" },
+    container: { type: "message", message_ts: "5.0", channel_id: "C9" },
+    message: {
+      ts: "5.0",
+      text: "the ask",
+      ...(overrides.threadTs === undefined
+        ? {}
+        : { thread_ts: overrides.threadTs }),
+      blocks: [{ type: "section" }, { type: "actions", block_id: "b1" }],
+    },
+    actions: [
+      {
+        type: "button",
+        action_id: overrides.actionId ?? "my_button",
+        block_id: "b1",
+        value: overrides.value ?? '{"pr":"https://example.test/pr/1"}',
+        text: { type: "plain_text", text: "Deploy" },
+        action_ts: "6.123456",
+      },
+    ],
+  };
+}
+
+describe("parseSlackBlockAction", () => {
+  it("parses a button click with label, clicker, and message blocks", () => {
+    const action = parseSlackBlockAction(blockActionsPayload());
+    expect(action).toMatchObject({
+      actionId: "my_button",
+      blockId: "b1",
+      value: '{"pr":"https://example.test/pr/1"}',
+      label: "Deploy",
+      channelId: "C9",
+      // Top-level message: the click stays in that message's thread.
+      threadTs: "5.0",
+      messageTs: "5.0",
+      userId: "U7",
+      userName: "netto",
+      teamId: "T1",
+      actionTs: "6.123456",
+    });
+    expect(action?.messageBlocks).toHaveLength(2);
+  });
+
+  it("prefers the message thread_ts for replies", () => {
+    const action = parseSlackBlockAction(
+      blockActionsPayload({ threadTs: "1.0" })
+    );
+    expect(action?.threadTs).toBe("1.0");
+    expect(action?.messageTs).toBe("5.0");
+  });
+
+  it("defaults a missing button value to the empty string", () => {
+    const payload = blockActionsPayload();
+    delete (
+      (payload.actions as Array<Record<string, unknown>>)[0] as Record<
+        string,
+        unknown
+      >
+    ).value;
+    expect(parseSlackBlockAction(payload)?.value).toBe("");
+  });
+
+  it("returns null for non-block_actions payloads", () => {
+    expect(parseSlackBlockAction({ type: "view_submission" })).toBeNull();
+    expect(parseSlackBlockAction(null)).toBeNull();
+    expect(
+      parseSlackBlockAction({ type: "block_actions", actions: [] })
+    ).toBeNull();
+  });
+});
+
+describe("dispatchBlockAction", () => {
+  it("dispatches the handler decision on the click's thread", async () => {
+    const { send, calls } = createSendRecorder();
+    const seen: SlackBlockAction[] = [];
+    const handled = await dispatchBlockAction(blockActionsPayload(), {
+      handler: (_ctx, action) => {
+        seen.push(action);
+        return { auth: testAuth, message: "U7 clicked Deploy — ship it." };
+      },
+      send,
+      credentials: undefined,
+      host: testHost(),
+      log: createSlackLogger(() => undefined),
+    });
+    expect(handled).toBe(true);
+    expect(seen).toHaveLength(1);
+    expect(seen[0]?.actionId).toBe("my_button");
+    expect(calls).toHaveLength(1);
+    expect(calls[0]?.options?.continuationToken).toBe("C9:5.0");
+    expect(calls[0]?.message).toContain("<slack_context>");
+    expect(calls[0]?.message).toContain("U7 clicked Deploy — ship it.");
+    const state = calls[0]?.options?.state as {
+      channelId?: string;
+      threadTs?: string;
+      triggeringUserId?: string;
+    };
+    expect(state.channelId).toBe("C9");
+    expect(state.threadTs).toBe("5.0");
+    expect(state.triggeringUserId).toBe("U7");
+  });
+
+  it("skips clicks without an action_ts (would alias dedupe keys)", async () => {
+    const { send, calls } = createSendRecorder();
+    const payload = blockActionsPayload();
+    delete (
+      (payload.actions as Array<Record<string, unknown>>)[0] as Record<
+        string,
+        unknown
+      >
+    ).action_ts;
+    const handled = await dispatchBlockAction(payload, {
+      handler: () => ({ auth: testAuth }),
+      send,
+      credentials: undefined,
+      host: testHost(),
+      log: createSlackLogger(() => undefined),
+    });
+    expect(handled).toBe(true);
+    expect(calls).toHaveLength(0);
+  });
+
+  it("skips dispatch when the handler returns null", async () => {
+    const { send, calls } = createSendRecorder();
+    const handled = await dispatchBlockAction(blockActionsPayload(), {
+      handler: () => null,
+      send,
+      credentials: undefined,
+      host: testHost(),
+      log: createSlackLogger(() => undefined),
+    });
+    expect(handled).toBe(true);
+    expect(calls).toHaveLength(0);
+  });
+
+  it("returns false for payloads that are not block actions", async () => {
+    const { send, calls } = createSendRecorder();
+    const handled = await dispatchBlockAction(
+      { type: "view_submission" },
+      {
+        handler: () => ({ auth: testAuth }),
+        send,
+        credentials: undefined,
+        host: testHost(),
+        log: createSlackLogger(() => undefined),
+      }
+    );
+    expect(handled).toBe(false);
+    expect(calls).toHaveLength(0);
+  });
+});
+
+describe("resolveBlockActionCard", () => {
+  function parsedAction(): SlackBlockAction {
+    const action = parseSlackBlockAction(blockActionsPayload());
+    if (action === null) {
+      throw new Error("fixture must parse");
+    }
+    return action;
+  }
+
+  it("swaps only the clicked actions row for the note", async () => {
+    const calls: Array<{ method: string; args: Record<string, unknown> }> = [];
+    const action = parsedAction();
+    action.messageBlocks = [
+      ...(action.messageBlocks ?? []),
+      { type: "actions", block_id: "someone_elses_row" },
+    ];
+    const ok = await resolveBlockActionCard({
+      botToken: {
+        kind: "api-transport",
+        label: "test",
+        async call(method: string, args: Record<string, unknown>) {
+          calls.push({ method, args });
+          return { ok: true };
+        },
+      },
+      action,
+      note: "resolved by <@U7>",
+    });
+    expect(ok).toBe(true);
+    expect(calls).toHaveLength(1);
+    expect(calls[0]?.method).toBe("chat.update");
+    expect(calls[0]?.args.ts).toBe("5.0");
+    const updated = JSON.stringify(calls[0]?.args.blocks);
+    expect(updated).not.toContain('"block_id":"b1"');
+    expect(updated).toContain("someone_elses_row");
+    expect(updated).toContain("resolved by <@U7>");
+  });
+
+  it("returns false when the payload carried no message blocks", async () => {
+    const action = parsedAction();
+    action.messageBlocks = undefined;
+    const ok = await resolveBlockActionCard({
+      botToken: {
+        kind: "api-transport",
+        label: "test",
+        async call() {
+          throw new Error("must not call Slack");
+        },
+      },
+      action,
+      note: "resolved",
+    });
+    expect(ok).toBe(false);
+  });
+});
+
+describe("blockActionAuth", () => {
+  it("builds the click's thread-scoped principal with the clicker", () => {
+    const action = parseSlackBlockAction(blockActionsPayload());
+    const auth = blockActionAuth(action as SlackBlockAction);
+    expect(auth.principalId).toBe("slack:T1:C9:5.0");
+    expect(auth.attributes?.user_id).toBe("U7");
+    expect(auth.attributes?.user_name).toBe("netto");
+  });
+});
+
+describe("mrkdwnSections", () => {
+  it("packs lines into capped sections preserving order", () => {
+    const lines = Array.from(
+      { length: 200 },
+      (_, i) => `• item-${i}-${"x".repeat(20)}`
+    );
+    const sections = mrkdwnSections(["Header:", ...lines]) as Array<{
+      type: string;
+      text: { type: string; text: string };
+    }>;
+    expect(sections.length).toBeGreaterThan(1);
+    for (const section of sections) {
+      expect(section.type).toBe("section");
+      expect(section.text.text.length).toBeLessThanOrEqual(
+        SLACK_SECTION_MRKDWN_MAX
+      );
+    }
+    const joined = sections.map((section) => section.text.text).join("\n");
+    expect(joined.startsWith("Header:")).toBe(true);
+    for (const line of lines) {
+      expect(joined).toContain(line);
+    }
+  });
+
+  it("keeps a short list in one section", () => {
+    expect(mrkdwnSections(["Header:", "• a", "• b"])).toHaveLength(1);
+  });
+
+  it("hard-splits a single oversized line at the cap", () => {
+    const sections = mrkdwnSections([
+      "x".repeat(SLACK_SECTION_MRKDWN_MAX + 500),
+    ]) as Array<{ text: { text: string } }>;
+    expect(sections).toHaveLength(2);
+    expect(sections[0]?.text.text.length).toBe(SLACK_SECTION_MRKDWN_MAX);
+    expect(sections[1]?.text.text.length).toBe(500);
+  });
+});
+
+describe("slackTableBlock", () => {
+  it("builds a native table with header + rich_text link cells", () => {
+    const block = slackTableBlock({
+      headers: ["Service", "SHA", "Anyship", "Logs"],
+      columnSettings: [
+        { is_wrapped: true },
+        { align: "left" },
+        { align: "left" },
+        { align: "left" },
+      ],
+      rows: [
+        [
+          slackRawTextCell("portal-website"),
+          slackLinkCell({
+            url: "https://github.com/anysphere/everysphere/commit/abc",
+            text: "abcdef0",
+          }),
+          slackLinkCell({
+            url: "https://anytool.corp.anysphere.co/anyship/deployments?deployment=d1",
+            text: "open",
+          }),
+          slackRawTextCell("—"),
+        ],
+      ],
+    });
+    expect(block).toEqual({
+      type: "table",
+      column_settings: [
+        { is_wrapped: true },
+        { align: "left" },
+        { align: "left" },
+        { align: "left" },
+      ],
+      rows: [
+        [
+          { type: "raw_text", text: "Service" },
+          { type: "raw_text", text: "SHA" },
+          { type: "raw_text", text: "Anyship" },
+          { type: "raw_text", text: "Logs" },
+        ],
+        [
+          { type: "raw_text", text: "portal-website" },
+          {
+            type: "rich_text",
+            elements: [
+              {
+                type: "rich_text_section",
+                elements: [
+                  {
+                    type: "link",
+                    url: "https://github.com/anysphere/everysphere/commit/abc",
+                    text: "abcdef0",
+                  },
+                ],
+              },
+            ],
+          },
+          {
+            type: "rich_text",
+            elements: [
+              {
+                type: "rich_text_section",
+                elements: [
+                  {
+                    type: "link",
+                    url: "https://anytool.corp.anysphere.co/anyship/deployments?deployment=d1",
+                    text: "open",
+                  },
+                ],
+              },
+            ],
+          },
+          { type: "raw_text", text: "—" },
+        ],
+      ],
+    });
+  });
+
+  it("returns undefined for empty headers", () => {
+    expect(slackTableBlock({ headers: [], rows: [] })).toBeUndefined();
+  });
+
+  it("rejects oversized or jagged tables", () => {
+    expect(() =>
+      slackTableBlock({
+        headers: Array.from({ length: SLACK_TABLE_MAX_COLUMNS + 1 }, () => "c"),
+        rows: [],
+      })
+    ).toThrow(/columns exceeds/);
+    expect(() =>
+      slackTableBlock({
+        headers: ["A"],
+        rows: Array.from({ length: SLACK_TABLE_MAX_ROWS }, () => [
+          slackRawTextCell("x"),
+        ]),
+      })
+    ).toThrow(/rows exceeds/);
+    expect(() =>
+      slackTableBlock({
+        headers: ["A", "B"],
+        rows: [[slackRawTextCell("only-one")]],
+      })
+    ).toThrow(/expected 2/);
+  });
+});