@cursor/july 0.1.108 → 0.1.111

package/src/internal/cli-docs.ts

@@ -42,8 +42,8 @@ export interface DocsServerHandle {
 }
 
 export async function cmdDocs(options: DocsCliOptions = {}): Promise<number> {
-  const stdout = options.stdoutWrite ?? ((c) => void process.stdout.write(c));
-  const stderr = options.stderrWrite ?? ((c) => void process.stderr.write(c));
+  const stdout = options.stdoutWrite ?? (c => void process.stdout.write(c));
+  const stderr = options.stderrWrite ?? (c => void process.stderr.write(c));
 
   await (options.ensureBundle ?? ensureDocsSiteBundle)();
   if (!(options.siteExists ?? docsSiteExists)()) {
@@ -58,7 +58,7 @@ export async function cmdDocs(options: DocsCliOptions = {}): Promise<number> {
     handle = await startDocsServer({
       port: options.port ?? 0,
       host: options.host ?? "127.0.0.1",
-      renderFile: options.renderFile ?? ((path) => renderDocsFile(path)),
+      renderFile: options.renderFile ?? (path => renderDocsFile(path)),
       createServerImpl: options.createServerImpl ?? createServer,
     });
   } catch (error) {
@@ -88,7 +88,7 @@ export async function cmdDocs(options: DocsCliOptions = {}): Promise<number> {
   }
 
   try {
-    await new Promise<void>((resolve) => {
+    await new Promise<void>(resolve => {
       if (controller.signal.aborted) {
         resolve();
         return;
@@ -132,7 +132,7 @@ export async function startDocsServer(args: {
     port: address.port,
     close: () =>
       new Promise<void>((resolve, reject) => {
-        server.close((err) => (err !== undefined ? reject(err) : resolve()));
+        server.close(err => (err !== undefined ? reject(err) : resolve()));
       }),
   };
 }

package/src/internal/cli-github.ts

@@ -272,10 +272,10 @@ async function cmdGithubReplay(options: GithubCliOptions): Promise<number> {
   const port = options.port ?? DEFAULT_PORT;
   let candidates = channels;
   if (options.slug !== undefined) {
-    candidates = candidates.filter((c) => c.slug === options.slug);
+    candidates = candidates.filter(c => c.slug === options.slug);
   }
   if (options.channel !== undefined) {
-    candidates = candidates.filter((c) => c.channelId === options.channel);
+    candidates = candidates.filter(c => c.channelId === options.channel);
   }
   if (options.url !== undefined && candidates.length > 1) {
     process.stderr.write(
@@ -291,7 +291,7 @@ async function cmdGithubReplay(options: GithubCliOptions): Promise<number> {
   }
   const targets: GitHubForwardTarget[] =
     options.url === undefined
-      ? candidates.map((c) => ({
+      ? candidates.map(c => ({
           slug: c.slug,
           channelId: c.channelId,
           url: channelForwardUrl({
@@ -345,7 +345,7 @@ async function cmdGithubReplay(options: GithubCliOptions): Promise<number> {
   }
 
   // Best-effort: mirror a real CI check (app slug / conclusion) for CI events.
-  const wantsCi = eventsToReplay.some((event) =>
+  const wantsCi = eventsToReplay.some(event =>
     ["check_run", "check_suite", "workflow_run", "status"].includes(event)
   );
   const check = wantsCi
@@ -378,7 +378,7 @@ async function cmdGithubReplay(options: GithubCliOptions): Promise<number> {
   const secret = options.secret ?? process.env.GITHUB_WEBHOOK_SECRET;
   process.stderr.write(
     `[agent-sdk] replaying ${ref.label} (${synthesized
-      .map((s) => s.eventName)
+      .map(s => s.eventName)
       .join(
         ", "
       )}) → ${targets.length} channel${targets.length === 1 ? "" : "s"}\n`
@@ -420,10 +420,7 @@ async function cmdGithubReplay(options: GithubCliOptions): Promise<number> {
       event.eventName,
       targets
     );
-    if (
-      deliveries.length === 0 ||
-      deliveries.some((delivery) => !delivery.ok)
-    ) {
+    if (deliveries.length === 0 || deliveries.some(delivery => !delivery.ok)) {
       anyRejected = true;
     }
     report.push({ ...eventReport(event), deliveries });
@@ -431,7 +428,7 @@ async function cmdGithubReplay(options: GithubCliOptions): Promise<number> {
       const summary =
         deliveries
           .map(
-            (delivery) =>
+            delivery =>
               `${delivery.label} ${delivery.status || "ERR"}${delivery.ok ? "" : " (rejected)"}`
           )
           .join(", ") || "(no matching channel)";
@@ -467,12 +464,12 @@ async function deliverReplayEvent(
   targets: readonly GitHubForwardTarget[]
 ): Promise<ReplayDelivery[]> {
   const recipients = targets.filter(
-    (target) => target.events.length === 0 || target.events.includes(eventName)
+    target => target.events.length === 0 || target.events.includes(eventName)
   );
   const payload = new Uint8Array(body);
   // Bounded by the channel count (a handful) — see fanoutDelivery.
   return Promise.all(
-    recipients.map(async (target) => {
+    recipients.map(async target => {
       try {
         const response = await fetch(target.url, {
           method: "POST",
@@ -531,7 +528,7 @@ async function fetchReplayCheck(
       }>;
     }>(`repos/${ref.owner}/${ref.repo}/commits/${sha}/check-runs?per_page=100`);
     const runs = data.check_runs ?? [];
-    const chosen = runs.find((run) => run.conclusion === "failure") ?? runs[0];
+    const chosen = runs.find(run => run.conclusion === "failure") ?? runs[0];
     if (chosen === undefined) {
       return undefined;
     }
@@ -575,7 +572,7 @@ async function cmdGithubEvents(options: GithubCliOptions): Promise<number> {
     process.stdout.write(
       `${JSON.stringify(
         {
-          channels: targets.map((t) => ({
+          channels: targets.map(t => ({
             slug: t.slug,
             channelId: t.channelId,
             url: urlFor(t),
@@ -702,7 +699,7 @@ function reportDoctor(
   json: boolean | undefined
 ): number {
   // Warnings (e.g. an env-token override) are advisory and do not fail doctor.
-  const ok = checks.every((check) => check.ok || check.warn === true);
+  const ok = checks.every(check => check.ok || check.warn === true);
   if (json === true) {
     process.stdout.write(`${JSON.stringify({ ok, checks }, null, 2)}\n`);
   } else {
@@ -809,7 +806,7 @@ async function tryExec(options: {
 }
 
 function spawnGh(args: string[]): Promise<number> {
-  return new Promise((resolve) => {
+  return new Promise(resolve => {
     const child = spawn("gh", args, { stdio: "inherit" });
     const forward = (signal: NodeJS.Signals): void => {
       if (!child.killed) {
@@ -818,7 +815,7 @@ function spawnGh(args: string[]): Promise<number> {
     };
     process.on("SIGINT", forward);
     process.on("SIGTERM", forward);
-    child.on("error", (error) => {
+    child.on("error", error => {
       process.stderr.write(`failed to run gh: ${error.message}\n`);
       resolve(1);
     });
@@ -835,7 +832,7 @@ function parseEventList(value: string | undefined): string[] | undefined {
   }
   const events = value
     .split(/[\s,]+/)
-    .map((event) => event.trim())
+    .map(event => event.trim())
     .filter(Boolean);
   return events.length === 0 ? undefined : events;
 }
@@ -843,6 +840,6 @@ function parseEventList(value: string | undefined): string[] | undefined {
 /** Render the gh argv for logging with the signing secret masked. */
 function redact(args: string[]): string {
   return args
-    .map((arg) => (arg.startsWith("--secret=") ? "--secret=***" : arg))
+    .map(arg => (arg.startsWith("--secret=") ? "--secret=***" : arg))
     .join(" ");
 }

package/src/internal/cli-gitlab.ts

@@ -134,10 +134,10 @@ async function resolveTargets(
   const port = options.port ?? DEFAULT_PORT;
   let candidates: GitLabChannelTarget[] = channels;
   if (options.slug !== undefined) {
-    candidates = candidates.filter((c) => c.slug === options.slug);
+    candidates = candidates.filter(c => c.slug === options.slug);
   }
   if (options.channel !== undefined) {
-    candidates = candidates.filter((c) => c.channelId === options.channel);
+    candidates = candidates.filter(c => c.channelId === options.channel);
   }
   if (options.url !== undefined && candidates.length > 1) {
     process.stderr.write(
@@ -161,7 +161,7 @@ async function resolveTargets(
       },
     ];
   }
-  return candidates.map((c) => ({
+  return candidates.map(c => ({
     slug: c.slug,
     channelId: c.channelId,
     url: channelForwardUrl({
@@ -192,7 +192,7 @@ async function cmdGitlabReplay(options: GitlabCliOptions): Promise<number> {
   }
 
   const requested = parseEventList(options.events);
-  const declared = [...new Set(targets.flatMap((target) => target.events))];
+  const declared = [...new Set(targets.flatMap(target => target.events))];
   const { events, unsupported } = selectGitLabReplayEvents(requested, declared);
   if (unsupported.length > 0) {
     process.stderr.write(
@@ -236,7 +236,7 @@ async function cmdGitlabReplay(options: GitlabCliOptions): Promise<number> {
         method: "GET",
         path: `/projects/${projectSegment}/merge_requests/${ref.iid}/pipelines`,
       })
-      .then((list) => list[0])
+      .then(list => list[0])
       .catch(() => undefined);
   }
 
@@ -303,7 +303,7 @@ async function cmdGitlabReplay(options: GitlabCliOptions): Promise<number> {
       event.eventName,
       targets
     );
-    if (deliveries.length === 0 || deliveries.some((d) => !d.ok)) {
+    if (deliveries.length === 0 || deliveries.some(d => !d.ok)) {
       anyRejected = true;
     }
     report.push({ ...eventReport(event), deliveries });
@@ -311,7 +311,7 @@ async function cmdGitlabReplay(options: GitlabCliOptions): Promise<number> {
       const summary =
         deliveries
           .map(
-            (d) => `${d.label} ${d.status || "ERR"}${d.ok ? "" : " (rejected)"}`
+            d => `${d.label} ${d.status || "ERR"}${d.ok ? "" : " (rejected)"}`
           )
           .join(", ") || "(no matching channel)";
       process.stderr.write(`[agent-sdk]   ${label} → ${summary}\n`);
@@ -344,10 +344,10 @@ async function deliverReplayEvent(
   targets: readonly ReplayTarget[]
 ): Promise<ReplayDelivery[]> {
   const recipients = targets.filter(
-    (target) => target.events.length === 0 || target.events.includes(eventName)
+    target => target.events.length === 0 || target.events.includes(eventName)
   );
   return Promise.all(
-    recipients.map(async (target) => {
+    recipients.map(async target => {
       const label =
         target.slug === ""
           ? target.channelId
@@ -398,7 +398,7 @@ async function cmdGitlabEvents(options: GitlabCliOptions): Promise<number> {
   const channels = findGitLabChannelTargets(await resolveAgentMounts(dir));
   const host = options.host ?? DEFAULT_HOST;
   const port = options.port ?? DEFAULT_PORT;
-  const rows = channels.map((c) => ({
+  const rows = channels.map(c => ({
     slug: c.slug,
     channelId: c.channelId,
     url: channelForwardUrl({
@@ -479,7 +479,7 @@ function parseEventList(value: string | undefined): string[] | undefined {
   }
   const events = value
     .split(/[\s,]+/)
-    .map((event) => event.trim())
+    .map(event => event.trim())
     .filter(Boolean);
   return events.length === 0 ? undefined : events;
 }

package/src/internal/cli-manifest.ts

@@ -40,7 +40,7 @@ export async function cmdManifest(
   for (const diagnostic of project.diagnostics) {
     err(`${diagnostic.severity}: ${diagnostic.message}\n`);
   }
-  if (project.diagnostics.some((d) => d.severity === "error")) {
+  if (project.diagnostics.some(d => d.severity === "error")) {
     return 1;
   }
 

package/src/internal/cli-mcp-oauth.ts

@@ -69,18 +69,16 @@ function describe(error: unknown): string {
 export async function cmdMcpOauth(
   options: McpOauthCliOptions
 ): Promise<number> {
-  const err = options.err ?? ((text) => process.stderr.write(text));
-  const out = options.out ?? ((text) => process.stdout.write(text));
+  const err = options.err ?? (text => process.stderr.write(text));
+  const out = options.out ?? (text => process.stdout.write(text));
   const project = await loadAgentProject(options.dir);
   const authored = [
     ...project.agent.connections,
     ...(project.agent.hostConnections ?? []),
   ];
-  const connection = authored.find(
-    (entry) => entry.name === options.connection
-  );
+  const connection = authored.find(entry => entry.name === options.connection);
   if (connection === undefined) {
-    const known = authored.map((entry) => entry.name).join(", ") || "(none)";
+    const known = authored.map(entry => entry.name).join(", ") || "(none)";
     err(
       `${LOG_PREFIX} Unknown MCP connection "${options.connection}". Authored: ${known}.\n`
     );
@@ -159,7 +157,7 @@ export async function authorizeHostMcpOauth(
   const provider = await createHostMcpOAuthProvider({
     identifier: connection,
     resourceUrl: args.url,
-    onRedirect: (url) => {
+    onRedirect: url => {
       authorizationUrl = url;
     },
   });

package/src/internal/cli-mcp.ts

@@ -180,7 +180,7 @@ export async function cmdMcpServe(options: McpCliOptions): Promise<number> {
     emitFailure(stdout, line, outcome.message, err);
   };
 
-  await new Promise<void>((done) => {
+  await new Promise<void>(done => {
     let buffer = "";
     const enqueueLine = (raw: string): void => {
       const line = raw.trim();
@@ -325,7 +325,7 @@ function createProdTargetSource(
       }
       if (inFlight === undefined) {
         inFlight = resolveFresh()
-          .then((target) => {
+          .then(target => {
             cached = target;
             return target;
           })
@@ -786,7 +786,7 @@ function resolveServerName(name: string | undefined, fallback: string): string {
 
 /** Last path segment of the agent base URL (the mount slug), if any. */
 function deriveNameFromUrl(baseUrl: string): string | undefined {
-  const segments = new URL(baseUrl).pathname.split("/").filter((s) => s !== "");
+  const segments = new URL(baseUrl).pathname.split("/").filter(s => s !== "");
   const last = segments.at(-1);
   if (last === undefined || last === "") {
     return undefined;

package/src/internal/cli-skills.ts

@@ -32,8 +32,8 @@ export interface InstallSkillsCliOptions {
 export async function cmdInstallSkills(
   options: InstallSkillsCliOptions = {}
 ): Promise<number> {
-  const out = options.out ?? ((text) => process.stdout.write(text));
-  const err = options.err ?? ((text) => process.stderr.write(text));
+  const out = options.out ?? (text => process.stdout.write(text));
+  const err = options.err ?? (text => process.stderr.write(text));
   const plan = await planSkillInstall({
     packageRoot: options.packageRoot,
     skillsHome: options.skillsHome,

package/src/internal/cli-slack.ts

@@ -650,7 +650,7 @@ export async function cmdSlackCreate(
   ctx.out(
     `Slack is connected for ${target.slug}` +
       (poll.appId === undefined ? "" : ` (app ${poll.appId})`) +
-      `.\n  wrote ${envEntries.map((entry) => entry.key).join(" + ")} to ${envFile}\n\n`
+      `.\n  wrote ${envEntries.map(entry => entry.key).join(" + ")} to ${envFile}\n\n`
   );
 
   const doctor = options.doctorImpl ?? runSlackDoctor;
@@ -682,11 +682,11 @@ async function resolveProvisionedApp(
   } catch (error) {
     return renderSlackApiError(error, ctx.err);
   }
-  const sameSlug = status.apps.filter((app) => app.slug === target.slug);
-  let candidates = sameSlug.filter((app) => app.env === target.env);
+  const sameSlug = status.apps.filter(app => app.slug === target.slug);
+  let candidates = sameSlug.filter(app => app.env === target.env);
   if (options.slackTeam !== undefined) {
     candidates = candidates.filter(
-      (app) => app.slackTeamId === options.slackTeam
+      app => app.slackTeamId === options.slackTeam
     );
   }
   if (candidates.length === 0) {
@@ -827,7 +827,7 @@ function formatAppList(
 ): string {
   return apps
     .map(
-      (app) =>
+      app =>
         `  - ${app.slackAppId} (${app.env}) in workspace ${app.slackTeamId} — ${app.installStatus}\n`
     )
     .join("");

package/src/internal/cloud-turn-cost.ts

@@ -107,7 +107,7 @@ export function applyCloudUsageToEvents(
     }
   } else if (
     billed.runs.length === unpricedIndexes.length &&
-    billed.runs.every((run) => usageTokenSum(run.usage) > 0)
+    billed.runs.every(run => usageTokenSum(run.usage) > 0)
   ) {
     for (let i = 0; i < unpricedIndexes.length; i++) {
       const index = unpricedIndexes[i];

package/src/internal/continuation-channel.ts

@@ -222,7 +222,8 @@ export function channelDeclaredEventTypes(
     meta?.gitlabWebhookEvents ??
     meta?.bitbucketWebhookEvents ??
     meta?.deploymentEventTypes ??
-    meta?.factoryEventTypes
+    meta?.factoryEventTypes ??
+    meta?.slackEventTypes
   );
 }
 

package/src/internal/continuation-identity.ts

@@ -18,6 +18,7 @@
  */
 
 import { randomBytes } from "node:crypto";
+import { isSlackImChannel, slackThreadKey } from "./slack-thread-key.js";
 
 export type ConversationType =
   | "pr"
@@ -320,7 +321,7 @@ function membershipOrNone(
   none: ContinuationSelector
 ): ContinuationSelector {
   const allowed = new Set(builtinForType(type)?.events ?? []);
-  if ((eventTypes ?? []).some((event) => !allowed.has(event))) {
+  if ((eventTypes ?? []).some(event => !allowed.has(event))) {
     return none;
   }
   return typed;
@@ -330,10 +331,10 @@ function githubV2Default(eventTypes?: readonly string[]): ContinuationSelector {
   const events = eventTypes ?? [];
   const issueEvents = new Set(builtinForType("issue")?.events ?? []);
   const prEvents = new Set(builtinForType("pr")?.events ?? []);
-  if (events.length > 0 && events.every((event) => issueEvents.has(event))) {
+  if (events.length > 0 && events.every(event => issueEvents.has(event))) {
     return "github.issue/v2";
   }
-  if (events.some((event) => !prEvents.has(event))) {
+  if (events.some(event => !prEvents.has(event))) {
     return "github.none/v2";
   }
   return "github.pr/v2";
@@ -378,7 +379,7 @@ export function omittedV2MembershipDowngrade(
     return undefined;
   }
   const allowed = new Set(typed.events);
-  const unexpected = events.filter((event) => !allowed.has(event));
+  const unexpected = events.filter(event => !allowed.has(event));
   if (unexpected.length === 0) {
     return undefined;
   }
@@ -435,7 +436,7 @@ function newEvtKey(): string {
 function builtinForType(
   type: ConversationType
 ): ContinuationBuiltin | undefined {
-  return BUILTINS.find((builtin) => builtin.type === type);
+  return BUILTINS.find(builtin => builtin.type === type);
 }
 
 function formatConversationKey(
@@ -462,7 +463,7 @@ function formatConversationKey(
     case "bitbucket-pr":
       return `bitbucket-pr:${fields.fullName}#${fields.number}`;
     case "thread":
-      return `${fields.channelId}:${fields.threadTs}`;
+      return slackThreadKey(fields.channelId ?? "", fields.threadTs ?? "");
     case "deployment":
       return `deployments:${fields.deploymentId}`;
     case "change-monitor-checkpoint":
@@ -576,7 +577,7 @@ export function handshakeContinuation(input: {
   }
   const eventTypes = input.eventTypes ?? [];
   const allowed = new Set(builtin.events);
-  const unexpected = eventTypes.filter((event) => !allowed.has(event));
+  const unexpected = eventTypes.filter(event => !allowed.has(event));
   if (unexpected.length > 0) {
     return {
       ok: false,
@@ -787,18 +788,29 @@ function extractSlack(payload: unknown): string | undefined {
   if (record === undefined) {
     return undefined;
   }
+  // Events-API deliveries are the full event_callback body, so the
+  // identity fields live under `event`. Top-level fields still win for
+  // pre-flattened payloads (Socket Mode local dev, tests). Keep in
+  // lockstep with the control-plane twin (SlackThreadExtractor).
+  const event = asRecord(record.event);
   const channelId =
     nonEmpty(record.channelId) ??
     nonEmpty(record.channel_id) ??
-    nonEmpty(record.channel);
+    nonEmpty(record.channel) ??
+    nonEmpty(event?.channel);
   const threadTs =
     nonEmpty(record.threadTs) ??
     nonEmpty(record.thread_ts) ??
-    nonEmpty(record.ts);
-  if (channelId === undefined || threadTs === undefined) {
+    nonEmpty(event?.thread_ts) ??
+    nonEmpty(record.ts) ??
+    nonEmpty(event?.ts);
+  if (channelId === undefined) {
     return undefined;
   }
-  return `cont:${formatConversationKey("thread", { channelId, threadTs })}`;
+  if (!isSlackImChannel(channelId) && threadTs === undefined) {
+    return undefined;
+  }
+  return `cont:${slackThreadKey(channelId, threadTs ?? "")}`;
 }
 
 function extractOrigin(payload: unknown): string | undefined {

package/src/internal/conversation-mirror.ts

@@ -109,6 +109,27 @@ export function createConversationMirror(
     }
     mirrored.set(sdkAgentId, blobIds);
   };
+  /**
+   * Content of the last manifest each conversation **successfully** wrote
+   * (minus `persistedAtMs`, which changes on every call by construction).
+   * A persist whose agent document and blob list are unchanged skips the
+   * manifest rewrite — `persist` runs on every turn boundary, but most
+   * boundaries between checkpoints change nothing durable. Same bound and
+   * eviction as `mirrored`; a miss only costs a redundant rewrite.
+   */
+  const lastManifestContent = new Map<string, string>();
+  const rememberManifest = (sdkAgentId: string, content: string): void => {
+    if (
+      !lastManifestContent.has(sdkAgentId) &&
+      lastManifestContent.size >= MAX_TRACKED_CONVERSATIONS
+    ) {
+      const oldest = lastManifestContent.keys().next();
+      if (!oldest.done) {
+        lastManifestContent.delete(oldest.value);
+      }
+    }
+    lastManifestContent.set(sdkAgentId, content);
+  };
   /**
    * Per-conversation serialization so a persist cannot interleave with a
    * hydrate (or another persist) for the same conversation. Entries are
@@ -193,14 +214,26 @@ export function createConversationMirror(
           // The referenced run is pod-local bookkeeping and is not mirrored;
           // a hydrated store must not point at a run row it does not hold.
           agent: { ...document, activeRunId: null },
-          blobIds: blobIds.filter((blobId) => uploaded.has(blobId)),
+          blobIds: blobIds.filter(blobId => uploaded.has(blobId)),
           persistedAtMs: Date.now(),
         };
+        // Skip the rewrite when nothing durable changed. `persistedAtMs`
+        // is excluded from the comparison — it differs on every call by
+        // construction and carries no restore semantics.
+        const content = JSON.stringify({
+          agent: manifest.agent,
+          blobIds: manifest.blobIds,
+        });
+        if (lastManifestContent.get(sdkAgentId) === content) {
+          return;
+        }
+        lastManifestContent.delete(sdkAgentId);
         // Manifest last: it only ever references blobs already uploaded.
         await sink.put(
           agentSdkAgentStoreKeys.conversationManifest(agentName, sdkAgentId),
           new TextEncoder().encode(JSON.stringify(manifest))
         );
+        rememberManifest(sdkAgentId, content);
       });
     },
 
@@ -240,7 +273,7 @@ export function createConversationMirror(
         const bodies = await mapBounded(
           manifest.blobIds,
           HYDRATE_DOWNLOAD_CONCURRENCY,
-          async (blobId) => ({
+          async blobId => ({
             blobId,
             body: await sink.get(
               agentSdkAgentStoreKeys.conversationBlob(
@@ -251,7 +284,7 @@ export function createConversationMirror(
             ),
           })
         );
-        const missing = bodies.filter((entry) => entry.body === undefined);
+        const missing = bodies.filter(entry => entry.body === undefined);
         if (missing.length > 0) {
           // The manifest lists exactly what a previous persist uploaded, so
           // any definitive absence is store loss — and the checkpoint graph