@cursor/july 0.1.105 → 0.1.108

package/src/internal/reminder-schedule.ts

@@ -23,8 +23,14 @@ export interface ResolvedSchedule {
   nextFireAtMs: number;
 }
 
-/** Resolve create-input schedule fields into a concrete next-fire plan. */
-export function resolveSchedule(input: ReminderCreateInput): ResolvedSchedule {
+/**
+ * Resolve create-input schedule fields into a concrete next-fire plan.
+ * Pass `nowMs` for deterministic CP / test resolution; CP owns this call.
+ */
+export function resolveSchedule(
+  input: Pick<ReminderCreateInput, "every" | "cron" | "delay" | "at">,
+  nowMs: number = Date.now()
+): ResolvedSchedule {
   const shapes = [
     input.every !== undefined,
     input.cron !== undefined,
@@ -37,7 +43,6 @@ export function resolveSchedule(input: ReminderCreateInput): ResolvedSchedule {
     );
   }
 
-  const now = Date.now();
   if (input.every !== undefined) {
     const intervalMs = parseDuration(input.every);
     if (intervalMs <= 0) {
@@ -49,7 +54,7 @@ export function resolveSchedule(input: ReminderCreateInput): ResolvedSchedule {
       kind: "every",
       value: String(input.every),
       intervalMs,
-      nextFireAtMs: now + intervalMs,
+      nextFireAtMs: nowMs + intervalMs,
     };
   }
   if (input.cron !== undefined) {
@@ -57,7 +62,7 @@ export function resolveSchedule(input: ReminderCreateInput): ResolvedSchedule {
     return {
       kind: "cron",
       value: input.cron,
-      nextFireAtMs: nextCronFireMs(input.cron, now),
+      nextFireAtMs: nextCronFireMs(input.cron, nowMs),
     };
   }
   if (input.delay !== undefined) {
@@ -65,7 +70,7 @@ export function resolveSchedule(input: ReminderCreateInput): ResolvedSchedule {
     return {
       kind: "delay",
       value: String(input.delay),
-      nextFireAtMs: now + delayMs,
+      nextFireAtMs: nowMs + delayMs,
     };
   }
   const atMs =

package/src/internal/reminder-store.ts

@@ -2,10 +2,10 @@
  * Reminder document store used by the local control plane
  * (`<stateRoot>/control-plane/reminders/`).
  *
- * Handlers (`run` / function `until`) are not serializable — they live only
- * in the in-memory {@link ReminderRunner} registry. The control plane holds
- * schedule, prompt, target, and status so prompt-form reminders survive
- * restart.
+ * Anonymous `run` / function `until` handlers are not serializable — they live
+ * only in the in-memory {@link ReminderRunner} registry. Named `handler` +
+ * `args` and prompt-form reminders survive restart (handlers resolve from
+ * `agent/reminders/` on every fire).
  */
 
 import {
@@ -18,6 +18,7 @@ import {
 } from "node:fs/promises";
 import { join } from "node:path";
 import type {
+  JsonObject,
   ReminderBusyPolicy,
   ReminderScheduleKind,
   ReminderStatus,
@@ -31,6 +32,12 @@ export type {
   ReminderStatus,
 } from "../types.js";
 
+/**
+ * Internal record status. `firing` is CP claim state and must not appear on
+ * public {@link ReminderStatus} / {@link ReminderInfo}.
+ */
+export type ReminderRecordStatus = ReminderStatus | "firing";
+
 /** Serializable reminder record (no function bodies). */
 export interface ReminderRecord {
   version: typeof REMINDER_SNAPSHOT_VERSION;
@@ -46,8 +53,15 @@ export interface ReminderRecord {
   /** Raw schedule value: every/delay string, cron expression, or ISO `at`. */
   scheduleValue: string;
   intervalMs?: number;
-  /** Prompt form — mutually exclusive with in-memory `run`. */
+  /** Prompt form — mutually exclusive with `handler` and in-memory `run`. */
   prompt?: string;
+  /**
+   * Named handler stem (`agent/reminders/<name>.ts`). Mutually exclusive with
+   * `prompt` and in-memory `run`.
+   */
+  handler?: string;
+  /** Serializable args for {@link handler}. */
+  args?: JsonObject;
   /**
    * True when create() registered a `run` handler. Handlers are not
    * serializable; after restart, armed records with this flag and no live
@@ -59,9 +73,14 @@ export interface ReminderRecord {
   /** Natural-language until (agent-facing); function until is in-memory only. */
   untilPrompt?: string;
   busyPolicy: ReminderBusyPolicy;
-  status: ReminderStatus;
+  status: ReminderRecordStatus;
   disarmReason?: string;
   fireCount: number;
+  /**
+   * CAS generation. Cancel/rekey and successful outcome apply bump this so
+   * in-flight outcomes cannot resurrect a superseded reminder.
+   */
+  generation?: number;
   nextFireAtMs: number;
   createdAt: string;
   updatedAt: string;
@@ -120,7 +139,8 @@ export class ReminderStore {
 
   async listArmed(): Promise<ReminderRecord[]> {
     const all = await this.list();
-    return all.filter((r) => r.status === "armed");
+    // Treat in-flight claims as still armed for restart / listing purposes.
+    return all.filter((r) => r.status === "armed" || r.status === "firing");
   }
 
   async get(id: string): Promise<ReminderRecord | undefined> {

package/src/internal/request-error-status.ts

@@ -7,12 +7,18 @@ import {
   hasGuestNetworkErrno,
   isQuietGuestConnectStall,
 } from "./guest-network.js";
+import {
+  HOSTED_GUEST_RETRY_TERMINAL,
+  HOSTED_GUEST_RETRY_TRANSIENT,
+  type HostedGuestRetryDisposition,
+} from "./hosted-delivery-protocol.js";
 import { SessionAuthError, SessionBusyError } from "./session-engine.js";
 
 export interface MappedRequestError {
   status: 400 | 403 | 409 | 503;
   code: string;
   message: string;
+  retry: HostedGuestRetryDisposition;
 }
 
 /**
@@ -23,23 +29,39 @@ export function mapThrownRequestError(
   error: unknown
 ): MappedRequestError | undefined {
   if (error instanceof SessionBusyError) {
-    return { status: 409, code: "session_busy", message: error.message };
+    return {
+      status: 409,
+      code: "session_busy",
+      message: error.message,
+      retry: HOSTED_GUEST_RETRY_TRANSIENT,
+    };
   }
   if (error instanceof V2EngineError) {
     return {
       status: 409,
       code: "v2_identity_conflict",
       message: "Control-plane session identity conflicts with existing state.",
+      retry: HOSTED_GUEST_RETRY_TERMINAL,
     };
   }
   if (error instanceof SessionAuthError) {
-    return { status: 403, code: "forbidden", message: error.message };
+    return {
+      status: 403,
+      code: "forbidden",
+      message: error.message,
+      retry: HOSTED_GUEST_RETRY_TERMINAL,
+    };
   }
   if (
     error instanceof InvalidAsOfError ||
     error instanceof VacuousTimeArgsError
   ) {
-    return { status: 400, code: "invalid_request", message: error.message };
+    return {
+      status: 400,
+      code: "invalid_request",
+      message: error.message,
+      retry: HOSTED_GUEST_RETRY_TERMINAL,
+    };
   }
   if (
     error instanceof GuestNetworkUnavailableError ||
@@ -52,6 +74,7 @@ export function mapThrownRequestError(
         error instanceof Error
           ? error.message
           : "Guest network is not ready for outbound requests.",
+      retry: HOSTED_GUEST_RETRY_TRANSIENT,
     };
   }
   if (isQuietGuestConnectStall(error)) {
@@ -62,6 +85,7 @@ export function mapThrownRequestError(
         error instanceof Error
           ? error.message
           : "Guest Connect stalled without a network errno.",
+      retry: HOSTED_GUEST_RETRY_TRANSIENT,
     };
   }
   return undefined;

package/src/internal/resolve-prod-target.ts

@@ -16,6 +16,7 @@ import {
   validateDeploymentSlug,
 } from "./deploy-client.js";
 import { CLI_COMMAND_NAME as CLI } from "./distribution.js";
+import { rejectV2ForLegacyCommand } from "./legacy-command-guard.js";
 import { sleep as defaultSleep } from "./sleep.js";
 import { stderrPalette } from "./terminal-style.js";
 
@@ -38,6 +39,8 @@ export interface ResolveProdTargetOptions {
   apiKey?: string;
   fetchImpl?: typeof fetch;
   err?: (text: string) => void;
+  /** Skip when the caller already ran {@link rejectV2ForLegacyCommand}. */
+  skipLegacyArchitectureCheck?: boolean;
   /** Test seam for reachability-retry backoff. */
   sleep?: (ms: number) => Promise<void>;
 }
@@ -95,6 +98,13 @@ export async function resolveProdTarget(
       `Could not resolve Cursor credentials/team for the hosted deployment (see above).`
     );
   }
+  if (options.skipLegacyArchitectureCheck !== true) {
+    await rejectV2ForLegacyCommand({
+      client: auth.client,
+      teamId: auth.teamId,
+      slug,
+    });
+  }
 
   // A hibernated engine can briefly omit engineAccess while getPod fails
   // during wake. Retry a few times before surfacing "not reachable yet".
@@ -198,6 +208,11 @@ export async function resolveProdAliasPlaygroundUrl(
       `Could not resolve Cursor credentials/team for the hosted deployment (see above).`
     );
   }
+  await rejectV2ForLegacyCommand({
+    client: auth.client,
+    teamId: auth.teamId,
+    slug,
+  });
 
   let detail: DeploymentDetail;
   try {

package/src/internal/scm/pr-url.ts

@@ -1,10 +1,21 @@
 /**
  * Provider-neutral PR URL parse. GitHub.com plus the Origin page shapes
  * humans paste: cursor.com/codebase, review.cursor.com/pr, origin.cursor.com.
+ *
+ * {@link parseScmPrUrl} is the wider parse the review seams use: it also
+ * recognizes GitLab merge requests (`/-/merge_requests/<iid>` on any host),
+ * Bitbucket Cloud / Data Center pull requests, and GitHub Enterprise Server
+ * (`/<owner>/<repo>/pull/<n>` on a host that is none of the above). The
+ * narrow {@link parsePrUrl} keeps its github.com + Origin contract for the
+ * free-text callers (Slack `pr-ref`) where "any host with /pull/N" would be
+ * too eager.
  */
 
 export type PrProvider = "github" | "origin";
 
+/** Every SCM the review seams can address. */
+export type ScmPrProvider = PrProvider | "gitlab" | "bitbucket";
+
 export type ParsedPrUrl = {
   provider: PrProvider;
   owner: string;
@@ -13,6 +24,27 @@ export type ParsedPrUrl = {
   url: string;
 };
 
+export type ParsedScmPrUrl = {
+  provider: ScmPrProvider;
+  /**
+   * Namespace half of the repo: GitHub owner, GitLab group path (nested
+   * groups joined with `/`), Bitbucket workspace / project key (`~user`
+   * for Data Center personal repos).
+   */
+  owner: string;
+  repo: string;
+  number: number;
+  /** Canonical PR page URL. */
+  url: string;
+  /**
+   * Hostname the PR lives on (lowercase). `github.com` for dotcom, the
+   * GHES hostname for GitHub Enterprise, `gitlab.com` or the self-managed
+   * host for GitLab, `bitbucket.org` or the Data Center host for Bitbucket.
+   * Origin PRs report the Origin page host.
+   */
+  host: string;
+};
+
 const ORIGIN_PAGE = /^origin(-[a-z0-9]+)?\.cursor\.com$/i;
 
 function positiveInt(raw: string | undefined): number | undefined {
@@ -27,19 +59,28 @@ function positiveInt(raw: string | undefined): number | undefined {
 }
 
 function parsed(
-  provider: PrProvider,
+  provider: ScmPrProvider,
+  host: string,
   owner: string | undefined,
   repo: string | undefined,
-  number: number | undefined
-): ParsedPrUrl | undefined {
-  if (owner === undefined || repo === undefined || number === undefined) {
+  number: number | undefined,
+  url?: string
+): ParsedScmPrUrl | undefined {
+  if (
+    owner === undefined ||
+    owner === "" ||
+    repo === undefined ||
+    repo === "" ||
+    number === undefined
+  ) {
     return undefined;
   }
-  const url =
-    provider === "origin"
+  const canonical =
+    url ??
+    (provider === "origin"
       ? `https://cursor.com/codebase/${owner}/${repo}/pull/${number}`
-      : `https://github.com/${owner}/${repo}/pull/${number}`;
-  return { provider, owner, repo, number, url };
+      : `https://${host}/${owner}/${repo}/pull/${number}`);
+  return { provider, owner, repo, number, url: canonical, host };
 }
 
 /** Strip Slack mrkdwn wrappers and trailing punctuation before `URL`. */
@@ -50,27 +91,49 @@ function normalizeRaw(raw: string): string {
     .replace(/[>\].,;]+$/, "");
 }
 
+function isGithubDotCom(host: string): boolean {
+  return host === "github.com" || host === "www.github.com";
+}
+
+/** github.com and Origin only — the contract free-text PR references use. */
 export function parsePrUrl(raw: string): ParsedPrUrl | undefined {
+  const wide = parseScmPrUrl(raw);
+  if (wide === undefined) {
+    return undefined;
+  }
+  if (wide.provider === "origin") {
+    return { ...wide, provider: "origin" };
+  }
+  if (wide.provider === "github" && isGithubDotCom(wide.host)) {
+    return { ...wide, provider: "github" };
+  }
+  return undefined;
+}
+
+/**
+ * Every provider shape, including GitHub Enterprise Server. The GitLab and
+ * Bitbucket path shapes are unambiguous on any host; a plain
+ * `/<owner>/<repo>/pull/<n>` on a host that is not github.com, Origin,
+ * GitLab, or Bitbucket is taken as GHES (the only remaining forge with that
+ * route). Callers that must not accept arbitrary hosts use
+ * {@link parsePrUrl}.
+ */
+export function parseScmPrUrl(raw: string): ParsedScmPrUrl | undefined {
   let url: URL;
   try {
     url = new URL(normalizeRaw(raw));
   } catch {
     return undefined;
   }
+  if (url.protocol !== "https:" && url.protocol !== "http:") {
+    return undefined;
+  }
 
   const parts = url.pathname.split("/").filter(Boolean);
   const host = url.hostname.toLowerCase();
 
-  if (
-    (host === "github.com" || host === "www.github.com") &&
-    parts.length >= 4 &&
-    parts[2] === "pull"
-  ) {
-    return parsed("github", parts[0], parts[1], positiveInt(parts[3]));
-  }
-
   if (ORIGIN_PAGE.test(host) && parts.length >= 4 && parts[2] === "github") {
-    return parsed("origin", parts[0], parts[1], positiveInt(parts[3]));
+    return parsed("origin", host, parts[0], parts[1], positiveInt(parts[3]));
   }
 
   if (
@@ -79,17 +142,101 @@ export function parsePrUrl(raw: string): ParsedPrUrl | undefined {
     parts.length >= 5 &&
     parts[3] === "pull"
   ) {
-    return parsed("origin", parts[1], parts[2], positiveInt(parts[4]));
+    return parsed("origin", host, parts[1], parts[2], positiveInt(parts[4]));
   }
 
   if (host === "review.cursor.com" && parts[0] === "pr") {
     if (parts.length >= 5 && parts[3] === "github") {
-      return parsed("origin", parts[1], parts[2], positiveInt(parts[4]));
+      return parsed("origin", host, parts[1], parts[2], positiveInt(parts[4]));
     }
     if (parts.length >= 4) {
-      return parsed("origin", parts[1], parts[2], positiveInt(parts[3]));
+      return parsed("origin", host, parts[1], parts[2], positiveInt(parts[3]));
     }
   }
 
+  // GitLab MR page: /<group>/[<subgroup>/…]<project>/-/merge_requests/<iid>
+  const separator = parts.indexOf("-");
+  if (
+    separator >= 2 &&
+    parts[separator + 1] === "merge_requests" &&
+    parts.length >= separator + 3
+  ) {
+    const owner = parts.slice(0, separator - 1).join("/");
+    const repo = parts[separator - 1];
+    const number = positiveInt(parts[separator + 2]);
+    return parsed(
+      "gitlab",
+      host,
+      owner,
+      repo,
+      number,
+      number === undefined
+        ? undefined
+        : `https://${host}/${owner}/${repo}/-/merge_requests/${number}`
+    );
+  }
+
+  // Bitbucket Cloud: bitbucket.org/<workspace>/<slug>/pull-requests/<id>
+  if (
+    (host === "bitbucket.org" || host === "www.bitbucket.org") &&
+    parts.length >= 4 &&
+    parts[2] === "pull-requests"
+  ) {
+    const number = positiveInt(parts[3]);
+    return parsed(
+      "bitbucket",
+      "bitbucket.org",
+      parts[0],
+      parts[1],
+      number,
+      number === undefined
+        ? undefined
+        : `https://bitbucket.org/${parts[0]}/${parts[1]}/pull-requests/${number}`
+    );
+  }
+
+  // Bitbucket Data Center: [/<context>]/projects/<KEY>/repos/<slug>/pull-requests/<id>
+  // (personal repos: /users/<name>/repos/<slug>/…, full name `~name/slug`).
+  const dc = parts.findIndex(
+    (part, index) =>
+      (part === "projects" || part === "users") &&
+      parts[index + 2] === "repos" &&
+      parts[index + 4] === "pull-requests"
+  );
+  if (dc !== -1 && parts.length >= dc + 6) {
+    const number = positiveInt(parts[dc + 5]);
+    const owner = parts[dc] === "users" ? `~${parts[dc + 1]}` : parts[dc + 1];
+    const slug = parts[dc + 3];
+    const context = parts.slice(0, dc).join("/");
+    return parsed(
+      "bitbucket",
+      host,
+      owner,
+      slug,
+      number,
+      number === undefined
+        ? undefined
+        : `${url.origin}/${context === "" ? "" : `${context}/`}${parts.slice(dc, dc + 4).join("/")}/pull-requests/${number}`
+    );
+  }
+
+  // GitHub (dotcom or Enterprise Server): /<owner>/<repo>/pull/<n>. Checked
+  // last so the forge-specific shapes above win on shared hosts.
+  if (
+    parts.length >= 4 &&
+    parts[2] === "pull" &&
+    !ORIGIN_PAGE.test(host) &&
+    !/(^|\.)cursor\.com$/.test(host)
+  ) {
+    const githubHost = isGithubDotCom(host) ? "github.com" : host;
+    return parsed(
+      "github",
+      githubHost,
+      parts[0],
+      parts[1],
+      positiveInt(parts[3])
+    );
+  }
+
   return undefined;
 }

package/src/internal/sdk-runner.ts

@@ -518,7 +518,7 @@ export class CursorSdkRunner implements AgentRunner {
     const resolved = resolveApiKeySync({ apiKey: this.options.apiKey });
     if (resolved === undefined) {
       throw new Error(
-        `The Cursor SDK runner requires a Cursor credential: run \`${CLI} login\`, set CURSOR_API_KEY or CURSOR_API_KEY_FILE, or pass serve({ apiKey }).`
+        `The Cursor SDK runner requires a Cursor credential: run \`${CLI} login\`, set CURSOR_API_KEY, CURSOR_SERVICE_ACCOUNT_KEY, or CURSOR_API_KEY_FILE, or pass serve({ apiKey }).`
       );
     }
     return resolved.apiKey;
@@ -805,6 +805,7 @@ function toTurnOutcome(result: RunResult): TurnOutcome {
     ...(result.status === "error"
       ? { errorMessage: runErrorMessage(result) }
       : {}),
+    errorCode: result.status === "error" ? result.error?.code : undefined,
     ...(usage === undefined ? {} : { usage }),
   };
 }