chore(cli): restore upstream's deploymentSpinner field on BuildImageOptions

ConProgramming · cursoragent · ConProgramming · commit 9e00e0a1bcc6 · 2026-05-05T00:32:09.000-04:00
The fork's two-phase deploy work removed the `deploymentSpinner?: any`
field from `BuildImageOptions` (it was never used by buildImage itself,
just declared for external callers). Restoring it brings the type
declaration closer to upstream — the only fork-only deltas in this
section are now `apiClient?` (made optional) and `offlineIndex?` (added).

Co-authored-by: Cursor &lt;cursoragent@cursor.com&gt;
diff --git a/packages/cli-v3/src/deploy/buildImage.ts b/packages/cli-v3/src/deploy/buildImage.ts
@@ -53,6 +53,9 @@ export interface BuildImageOptions {
   buildEnvVars?: Record<string, string | undefined>;
   offlineIndex?: boolean; // When true, skip API-based indexing in the container
   onLog?: (log: string) => void;
+
+  // Optional deployment spinner
+  deploymentSpinner?: any; // Replace 'any' with the actual type if known
 }
 
 export async function buildImage(options: BuildImageOptions): Promise<BuildImageResults> {

Original file line number	Diff line number	Diff line change
`@@ -53,6 +53,9 @@ export interface BuildImageOptions {`
`53`	`53`	`buildEnvVars?: Record<string, string \| undefined>;`
`54`	`54`	`offlineIndex?: boolean; // When true, skip API-based indexing in the container`
`55`	`55`	`onLog?: (log: string) => void;`
	`56`	`+`
	`57`	`+ // Optional deployment spinner`
	`58`	`+ deploymentSpinner?: any; // Replace 'any' with the actual type if known`
`56`	`59`	`}`
`57`	`60`
`58`	`61`	`export async function buildImage(options: BuildImageOptions): Promise<BuildImageResults> {`