@cursor/july 0.1.49 → 0.1.51

package/src/channels/slack/setup.ts

@@ -1,111 +1,33 @@
 /**
  * Slack setup guide printed by `agent-serve slack setup`.
  *
- * Mirrors the Cursor skill at skills/setup-slack/SKILL.md: connection
- * decision first (Cursor-hosted vs dedicated Socket Mode app), then the
- * dedicated-app phases. {@link SLACK_SETUP_PHASES} stays the dedicated-app
- * checklist — its ids key `.agent-serve/slack/setup-status.json`.
+ * Two products: Cursor-hosted (`cursorAccount`) vs a dedicated Socket
+ * Mode app via `slack create` (dashboard wizard).
  */
 
-export interface SlackSetupPhase {
-  id: string;
-  actor: "agent" | "human";
-  title: string;
-  steps: string[];
-}
+export function formatSlackSetupGuide(): string {
+  return `agent-serve — Slack setup
 
-export const SLACK_SETUP_PHASES: SlackSetupPhase[] = [
-  {
-    id: "generate",
-    actor: "agent",
-    title: "Generate channel + manifests",
-    steps: [
-      "agent-serve slack init --dir . --name My Agent",
-      "Manifests default to Socket Mode (no public Request URL).",
-    ],
-  },
-  {
-    id: "createApps",
-    actor: "human",
-    title: "Create two Slack apps from manifests",
-    steps: [
-      "Open https://api.slack.com/apps -> Create New App from a manifest",
-      "Paste .agent-serve/slack/manifest.dev.json -> create the *dev* app",
-      "Repeat with manifest.prod.json for the *prod* app",
-    ],
-  },
-  {
-    id: "install",
-    actor: "human",
-    title: "Install + mint App-Level Token",
-    steps: [
-      "Install to Workspace; copy Bot User OAuth Token (xoxb-...)",
-      "App-Level Tokens: create token with connections:write; copy xapp-...",
-    ],
-  },
-  {
-    id: "env",
-    actor: "agent",
-    title: "Wire environment",
-    steps: [
-      "Write per-agent tokens into .env.local (see .agent-serve/slack/env.example)",
-      "Multi-agent: <PREFIX>_SLACK_BOT_TOKEN + <PREFIX>_SLACK_APP_TOKEN",
-      "Single agent: SLACK_BOT_TOKEN + SLACK_APP_TOKEN (--no-prefix)",
-      "agent-serve slack doctor --prefix <PREFIX>",
-    ],
-  },
-  {
-    id: "serve",
-    actor: "agent",
-    title: "Serve",
-    steps: [
-      "agent-serve serve --dir . --dev",
-      "Confirm log: [agent-sdk/slack] Socket Mode connected",
-    ],
-  },
-  {
-    id: "smoke",
-    actor: "human",
-    title: "Smoke test",
-    steps: [
-      "Invite the bot to a channel",
-      "@mention it or DM it",
-      "Confirm logs: inbound → session start → reply delivered",
-      "Confirm streaming reply and rotating status",
-    ],
-  },
-];
+Two products. Pick one.
 
-export function formatSlackSetupGuide(): string {
-  const lines: string[] = [
-    "agent-serve — Slack setup",
-    "",
-    "Two connections exist; decide before running the phases below.",
-    "",
-    "  A) Cursor-hosted connection (recommended default, zero Slack admin):",
-    '     slackChannel({ cursorAccount: true, agentName: "MyAgent" })',
-    "     Sign in (agent-serve login / CURSOR_API_KEY), serve, then mention",
-    '     "@Cursor MyAgent ..." in the linked workspace. No app, no tokens.',
-    "  B) Dedicated Slack app (Socket Mode) — the phases below. Choose it",
-    "     when you need your own bot user, channel watching",
-    "     (engagement.channelPosts), or approval buttons / interactivity.",
-    "",
-    "Phases 0-5 cover connection B only.",
-    "Goal: importable dev + prod Slack apps over Socket Mode (no public URL).",
-    "App-Level Token (xapp-...) + Bot Token (xoxb-...) required.",
-    "",
-  ];
-  for (const [index, phase] of SLACK_SETUP_PHASES.entries()) {
-    lines.push(`## Phase ${index} — ${phase.title} (${phase.actor})`);
-    for (const step of phase.steps) {
-      lines.push(`  - ${step}`);
-    }
-    lines.push("");
-  }
-  lines.push("Status file: .agent-serve/slack/setup-status.json");
-  lines.push(
-    "Skill (Cursor): packages/agent-serve/skills/setup-slack/SKILL.md"
-  );
-  lines.push("");
-  return lines.join("\n");
+  A) Cursor-hosted (recommended default, zero Slack admin):
+     slackChannel({ cursorAccount: true, agentName: "MyAgent" })
+     Sign in (agent-serve login / CURSOR_API_KEY), serve, then mention
+     "@Cursor MyAgent ..." in the linked workspace. No app, no tokens.
+
+  B) Dedicated Slack app (Socket Mode):
+     agent-serve slack create --dir .
+     Opens the signed-in Cursor dashboard wizard. Finish Slack consent
+     and the bot name there. The CLI writes tokens into .env.local.
+     Use this when you need your own bot user, channel watching
+     (engagement.channelPosts), or approval buttons.
+
+Do not paste a manifest at api.slack.com. \`slack init\` is removed.
+
+Then:
+  - agent-serve serve --dir . --dev
+  - Invite the bot (Path B) or mention @Cursor <name> (Path A)
+
+Skill: packages/agent-serve/skills/setup-slack/SKILL.md
+`;
 }

package/src/channels/slack/types.ts

@@ -421,7 +421,7 @@ export interface SlackEngagementOptions {
   /**
    * Opt-in channel watch (see {@link SlackChannelPostsOptions}). Requires
    * the Slack app to subscribe to `message.channels` / `message.groups`
-   * (`slack init --channel-posts`, or add the events to an existing app).
+   * (`slack create --channel-posts`).
    */
   channelPosts?: SlackChannelPostsOptions;
   /**
@@ -650,9 +650,8 @@ export interface SlackManifestOptions {
   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.
+   * provisions these apps through its manager app. Pass `false` when the
+   * caller is not a manager user token.
    */
   managedByCursor?: boolean;
 }

package/src/internal/cli-ax.ts

@@ -20,7 +20,7 @@ import type {
 import { serve } from "../index.js";
 import type { AgentServeHandle } from "../types.js";
 import { lookupSessionContinuation, runChat } from "./chat-client.js";
-import { cmdLogin } from "./cli-cursor.js";
+import { ensureCursorLogin } from "./cli-cursor.js";
 import { resolveDeployAuth } from "./cli-deploy.js";
 import { resolveApiKey } from "./cursor/credentials.js";
 import { CLI_COMMAND_NAME, PACKAGE_NAME } from "./distribution.js";
@@ -392,8 +392,8 @@ export async function cmdInit(options: AxCliOptions): Promise<number> {
 
   let loginExit = 0;
   if (!signedIn) {
-    process.stdout.write("Signing you in to Cursor\n\n");
-    loginExit = await cmdLogin({});
+    // init always offers the browser flow (the --json path above skipped it).
+    loginExit = await ensureCursorLogin({ interactive: true });
     signedIn = loginExit === 0 && (await resolveApiKey()) !== undefined;
     process.stdout.write("\n");
   }

package/src/internal/cli-cursor.ts

@@ -145,6 +145,40 @@ export async function cmdLogin(options: CursorCliOptions): Promise<number> {
   return 0;
 }
 
+/**
+ * Gate for commands that need a Cursor credential before they can do useful
+ * work (`serve`, `dev`, interactive `init`). Already-signed-in hosts
+ * (`--api-key` / `CURSOR_API_KEY` / stored login) return immediately.
+ * Otherwise an interactive host runs {@link cmdLogin}; a non-interactive
+ * one (CI, Docker) fails fast with {@link SIGN_IN_HINT} instead of polling
+ * a browser that will never open.
+ */
+export async function ensureCursorLogin(
+  options: {
+    apiKey?: string;
+    /**
+     * Run the browser login flow when unsigned. Defaults to stderr being a
+     * TTY.
+     */
+    interactive?: boolean;
+    /** Test seam. */
+    login?: (options: CursorCliOptions) => Promise<number>;
+  } = {}
+): Promise<number> {
+  const resolved = await resolveApiKey({ apiKey: options.apiKey });
+  if (resolved !== undefined) {
+    return 0;
+  }
+  const interactive = options.interactive ?? process.stderr.isTTY === true;
+  if (!interactive) {
+    process.stderr.write(`Not signed in to Cursor. ${SIGN_IN_HINT}\n`);
+    return 1;
+  }
+  process.stdout.write("Signing you in to Cursor\n\n");
+  const login = options.login ?? cmdLogin;
+  return login({});
+}
+
 export async function cmdLogout(): Promise<number> {
   const stored = await readStoredCredentials();
   const removed = await clearStoredCredentials();