@agentuity/cli 0.0.42 → 0.0.44

package/src/tui.ts

@@ -4,9 +4,12 @@
  * Provides semantic helpers for console output with automatic icons and colors.
  * Uses Bun's built-in color support and ANSI escape codes.
  */
+import { stringWidth } from 'bun';
 import enquirer from 'enquirer';
-import type { OrganizationList } from '@agentuity/server';
+import { type OrganizationList, projectList } from '@agentuity/server';
 import type { ColorScheme } from './terminal';
+import { type APIClient as APIClientType } from './api';
+import * as readline from 'readline';
 
 // Icons
 const ICONS = {
@@ -18,8 +21,13 @@ const ICONS = {
 	bullet: '•',
 } as const;
 
-function shouldUseColors(): boolean {
-	return !process.env.NO_COLOR && process.env.TERM !== 'dumb' && !!process.stdout.isTTY;
+export function shouldUseColors(): boolean {
+	return (
+		!process.env.NO_COLOR &&
+		!process.env.CI &&
+		process.env.TERM !== 'dumb' &&
+		!!process.stdout.isTTY
+	);
 }
 
 // Color definitions (light/dark adaptive) using Bun.color
@@ -71,10 +79,15 @@ function getColors() {
 	} as const;
 }
 
-let currentColorScheme: ColorScheme = 'dark';
+let currentColorScheme: ColorScheme = process.env.CI ? 'light' : 'dark';
 
 export function setColorScheme(scheme: ColorScheme): void {
 	currentColorScheme = scheme;
+	process.env.COLOR_SCHEME = scheme;
+}
+
+export function isDarkMode(): boolean {
+	return currentColorScheme === 'dark';
 }
 
 function getColor(colorKey: keyof ReturnType<typeof getColors>): string {
@@ -141,6 +154,15 @@ export function muted(text: string): string {
 	return `${color}${text}${reset}`;
 }
 
+/**
+ * Format text in warn color
+ */
+export function warn(text: string): string {
+	const color = getColor('warning');
+	const reset = getColor('reset');
+	return `${color}${text}${reset}`;
+}
+
 /**
  * Format text in bold
  */
@@ -153,13 +175,13 @@ export function bold(text: string): string {
 /**
  * Format text as a link (blue and underlined)
  */
-export function link(url: string): string {
+export function link(url: string, title?: string): string {
 	const color = getColor('link');
 	const reset = getColor('reset');
 
-	// Check if terminal supports hyperlinks (OSC 8)
-	if (supportsHyperlinks()) {
-		return `\x1b]8;;${url}\x07${color}${url}${reset}\x1b]8;;\x07`;
+	// Check if terminal supports hyperlinks (OSC 8) and colors are enabled
+	if (shouldUseColors() && supportsHyperlinks()) {
+		return `\x1b]8;;${url}\x07${color}${title ?? url}${reset}\x1b]8;;\x07`;
 	}
 
 	return `${color}${url}${reset}`;
@@ -207,24 +229,40 @@ export function newline(): void {
 	console.log('');
 }
 
+/**
+ * Get the display width of a string, handling ANSI codes and OSC 8 hyperlinks
+ *
+ * Note: Bun.stringWidth() counts OSC 8 hyperlink escape sequences in the width,
+ * which causes incorrect alignment. We strip OSC 8 codes first, then use Bun.stringWidth()
+ * to handle regular ANSI codes and unicode characters correctly.
+ */
+function getDisplayWidth(str: string): number {
+	// Remove OSC-8 hyperlink sequences using Unicode escapes (\u001b = ESC, \u0007 = BEL) to satisfy linter
+	// eslint-disable-next-line no-control-regex
+	const withoutOSC8 = str.replace(/\u001b\]8;;[^\u0007]*\u0007/g, '');
+	return Bun.stringWidth(withoutOSC8);
+}
+
 /**
  * Pad a string to a specific length on the right
  */
 export function padRight(str: string, length: number, pad = ' '): string {
-	if (str.length >= length) {
+	const displayWidth = getDisplayWidth(str);
+	if (displayWidth >= length) {
 		return str;
 	}
-	return str + pad.repeat(length - str.length);
+	return str + pad.repeat(length - displayWidth);
 }
 
 /**
  * Pad a string to a specific length on the left
  */
 export function padLeft(str: string, length: number, pad = ' '): string {
-	if (str.length >= length) {
+	const displayWidth = getDisplayWidth(str);
+	if (displayWidth >= length) {
 		return str;
 	}
-	return pad.repeat(length - str.length) + str;
+	return pad.repeat(length - displayWidth) + str;
 }
 
 interface BannerOptions {
@@ -244,11 +282,8 @@ interface BannerOptions {
  * Responsive to terminal width - adapts to narrow terminals
  */
 export function banner(title: string, body: string, options?: BannerOptions): void {
-	// Get terminal width, default to 80 if not available, minimum 40
-	const termWidth = process.stdout.columns || 80;
-	const minWidth = options?.minWidth ?? 40;
-	const maxWidth = Math.max(minWidth, Math.min(termWidth - 2, 80)); // Between 40 and 80, with 2 char margin
-	const padding = options?.padding ?? 4;
+	// Get terminal width, default to 120 if not available
+	const termWidth = process.stdout.columns || 120;
 
 	const border = {
 		topLeft: '╭',
@@ -259,15 +294,27 @@ export function banner(title: string, body: string, options?: BannerOptions): vo
 		vertical: '│',
 	};
 
-	// Split body into lines and wrap if needed
-	const bodyLines = wrapText(body, maxWidth - padding); // -4 for padding and borders
-
-	// Calculate width based on content
+	// Calculate content width first (before wrapping)
 	const titleWidth = getDisplayWidth(title);
-	const maxBodyWidth = Math.max(...bodyLines.map((line) => getDisplayWidth(line)));
-	const contentWidth = Math.max(minWidth, Math.max(titleWidth, maxBodyWidth) + padding);
-	const boxWidth = Math.min(contentWidth, maxWidth); // +N for padding
-	const innerWidth = boxWidth - padding;
+	const bodyLines = body.split('\n');
+	const maxBodyWidth = Math.max(0, ...bodyLines.map((line) => getDisplayWidth(line)));
+	const requiredContentWidth = Math.max(titleWidth, maxBodyWidth);
+
+	// Box width = content + borders (2) + side spaces (2)
+	const boxWidth = Math.min(requiredContentWidth + 4, termWidth);
+
+	// If required content width exceeds terminal width, skip box and print plain text
+	if (requiredContentWidth + 4 > termWidth) {
+		console.log('\n' + bold(title));
+		console.log(body + '\n');
+		return;
+	}
+
+	// Inner width is box width minus borders (2) and side spaces (2)
+	const innerWidth = boxWidth - 4;
+
+	// Wrap text to fit box width
+	const wrappedBodyLines = wrapText(body, innerWidth);
 
 	// Colors
 	const borderColor = getColor('muted');
@@ -293,10 +340,7 @@ export function banner(title: string, body: string, options?: BannerOptions): vo
 	const titleDisplayWidth = getDisplayWidth(title);
 	if (options?.centerTitle === true || options?.centerTitle === undefined) {
 		const titlePadding = Math.max(0, Math.floor((innerWidth - titleDisplayWidth) / 2));
-		const titleRightPadding = Math.max(
-			0,
-			Math.max(0, innerWidth - titlePadding - titleDisplayWidth) - padding
-		);
+		const titleRightPadding = Math.max(0, innerWidth - titlePadding - titleDisplayWidth);
 		const titleLine =
 			' '.repeat(titlePadding) +
 			`${titleColor}${bold(title)}${reset}` +
@@ -305,7 +349,7 @@ export function banner(title: string, body: string, options?: BannerOptions): vo
 			`${borderColor}${border.vertical} ${reset}${titleLine}${borderColor} ${border.vertical}${reset}`
 		);
 	} else {
-		const titleRightPadding = Math.max(0, Math.max(0, innerWidth - titleDisplayWidth) - padding);
+		const titleRightPadding = Math.max(0, innerWidth - titleDisplayWidth);
 		const titleLine = `${titleColor}${bold(title)}${reset}` + ' '.repeat(titleRightPadding);
 		lines.push(
 			`${borderColor}${border.vertical} ${reset}${titleLine}${borderColor} ${border.vertical}${reset}`
@@ -320,9 +364,9 @@ export function banner(title: string, body: string, options?: BannerOptions): vo
 	}
 
 	// Body lines
-	for (const line of bodyLines) {
+	for (const line of wrappedBodyLines) {
 		const lineWidth = getDisplayWidth(line);
-		const linePadding = Math.max(0, Math.max(0, innerWidth - lineWidth) - padding);
+		const linePadding = Math.max(0, innerWidth - lineWidth);
 		lines.push(
 			`${borderColor}${border.vertical} ${reset}${line}${' '.repeat(linePadding)}${borderColor} ${border.vertical}${reset}`
 		);
@@ -464,7 +508,7 @@ export function showSignupBenefits(): void {
 	];
 
 	console.log('');
-	lines.forEach((line) => console.log(CYAN + line + RESET));
+	lines.map((line) => console.log(CYAN + line + RESET));
 	console.log('');
 }
 
@@ -472,24 +516,32 @@ export function showSignupBenefits(): void {
  * Display a message when unauthenticated to let the user know certain capabilities are disabled
  */
 export function showLoggedOutMessage(): void {
-	const CYAN = Bun.color('yellow', 'ansi-16m');
+	const YELLOW = Bun.color('yellow', 'ansi-16m');
 	const TEXT =
 		currentColorScheme === 'dark' ? Bun.color('white', 'ansi') : Bun.color('black', 'ansi');
 	const RESET = '\x1b[0m';
 
+	const signupTitle = 'Sign up / Login';
+	const showInline = supportsHyperlinks();
+	const signupURL = 'https://app.agentuity.com/sign-up';
+	const signupLink = showInline
+		? link(signupURL, signupTitle)
+		: ' '.repeat(stringWidth(signupTitle));
+	const showNewLine = showInline ? '' : `║ ${RESET}${link(signupURL)}${YELLOW}            ║`;
+
 	const lines = [
 		'╔══════════════════════════════════════════════╗',
 		`║ ⨺ Unauthenticated (local mode)               ║`,
 		'║                                              ║',
-		`║ ${TEXT}Certain capabilities such as the AI services${CYAN} ║`,
-		`║ ${TEXT}and devmode remote are unavailable when${CYAN}      ║`,
-		`║ ${TEXT}unauthenticated.${CYAN}                             ║`,
+		`║ ${TEXT}Certain capabilities such as the AI services${YELLOW} ║`,
+		`║ ${TEXT}and devmode remote are unavailable when${YELLOW}      ║`,
+		`║ ${TEXT}unauthenticated.${YELLOW} ${signupLink}${YELLOW}             ║`,
+		showNewLine,
 		'╚══════════════════════════════════════════════╝',
 	];
 
 	console.log('');
-	lines.forEach((line) => console.log(CYAN + line + RESET));
-	console.log('');
+	lines.filter(Boolean).map((line) => console.log(YELLOW + line + RESET));
 }
 
 /**
@@ -544,20 +596,6 @@ export async function copyToClipboard(text: string): Promise<boolean> {
 	}
 }
 
-/**
- * Get the display width of a string, handling ANSI codes and OSC 8 hyperlinks
- *
- * Note: Bun.stringWidth() counts OSC 8 hyperlink escape sequences in the width,
- * which causes incorrect alignment. We strip OSC 8 codes first, then use Bun.stringWidth()
- * to handle regular ANSI codes and unicode characters correctly.
- */
-function getDisplayWidth(str: string): number {
-	// Strip OSC 8 hyperlink sequences: \x1b]8;;URL\x07...\x1b]8;;\x07
-	// eslint-disable-next-line no-control-regex
-	const withoutOSC8 = str.replace(/\x1b\]8;;[^\x07]*\x07/g, '');
-	return Bun.stringWidth(withoutOSC8);
-}
-
 /**
  * Extract ANSI codes from the beginning of a string
  */
@@ -667,6 +705,11 @@ export interface SimpleSpinnerOptions<T> {
 	type?: 'simple';
 	message: string;
 	callback: (() => Promise<T>) | Promise<T>;
+	/**
+	 * If true, clear the spinner output on success (no icon, no message)
+	 * Defaults to false
+	 */
+	clearOnSuccess?: boolean;
 }
 
 /**
@@ -676,6 +719,11 @@ export interface ProgressSpinnerOptions<T> {
 	type: 'progress';
 	message: string;
 	callback: (progress: SpinnerProgressCallback) => Promise<T>;
+	/**
+	 * If true, clear the spinner output on success (no icon, no message)
+	 * Defaults to false
+	 */
+	clearOnSuccess?: boolean;
 }
 
 /**
@@ -726,7 +774,7 @@ export async function spinner<T>(
 	const reset = getColor('reset');
 
 	// If no TTY, just execute the callback without animation
-	if (!process.stdout.isTTY) {
+	if (!process.stderr.isTTY) {
 		try {
 			const result =
 				options.type === 'progress'
@@ -735,6 +783,12 @@ export async function spinner<T>(
 						? await options.callback()
 						: await options.callback;
 
+			// If clearOnSuccess is true, don't show success message
+			if (!options.clearOnSuccess) {
+				const successColor = getColor('success');
+				console.error(`${successColor}${ICONS.success} ${message}${reset}`);
+			}
+
 			return result;
 		} catch (err) {
 			const errorColor = getColor('error');
@@ -757,7 +811,7 @@ export async function spinner<T>(
 	let currentProgress: number | undefined;
 
 	// Hide cursor
-	process.stdout.write('\x1B[?25l');
+	process.stderr.write('\x1B[?25l');
 
 	// Start animation
 	const interval = setInterval(() => {
@@ -772,7 +826,7 @@ export async function spinner<T>(
 				: '';
 
 		// Clear line and render
-		process.stdout.write('\r\x1B[K' + `${frame} ${message}${progressIndicator}`);
+		process.stderr.write('\r\x1B[K' + `${frame} ${message}${progressIndicator}`);
 		frameIndex++;
 	}, 120);
 
@@ -792,27 +846,30 @@ export async function spinner<T>(
 
 		// Clear interval and line
 		clearInterval(interval);
-		process.stdout.write('\r\x1B[K');
+		process.stderr.write('\r\x1B[K');
 
-		// Show success
-		const successColor = getColor('success');
-		console.log(`${successColor}${ICONS.success} ${message}${reset}`);
+		// If clearOnSuccess is false, show success message
+		if (!options.clearOnSuccess) {
+			// Show success
+			const successColor = getColor('success');
+			console.error(`${successColor}${ICONS.success} ${message}${reset}`);
+		}
 
 		// Show cursor
-		process.stdout.write('\x1B[?25h');
+		process.stderr.write('\x1B[?25h');
 
 		return result;
 	} catch (err) {
 		// Clear interval and line
 		clearInterval(interval);
-		process.stdout.write('\r\x1B[K');
+		process.stderr.write('\r\x1B[K');
 
 		// Show error
 		const errorColor = getColor('error');
 		console.error(`${errorColor}${ICONS.error} ${message}${reset}`);
 
 		// Show cursor
-		process.stdout.write('\x1B[?25h');
+		process.stderr.write('\x1B[?25h');
 
 		throw err;
 	}
@@ -847,12 +904,10 @@ export interface CommandRunnerOptions {
 	 * If true or undefined, will truncate each line of output
 	 */
 	truncate?: boolean;
-
 	/**
 	 * If undefined, will show up to 3 last lines of output while running. Customize the number with this property.
 	 */
 	maxLinesOutput?: number;
-
 	/**
 	 * If undefined, will show up to 10 last lines on failure. Customize the number with this property.
 	 */
@@ -1076,6 +1131,33 @@ export async function runCommand(options: CommandRunnerOptions): Promise<number>
 	}
 }
 
+/**
+ * Prompt user for text input
+ * Returns the input string
+ */
+export async function prompt(message: string): Promise<string> {
+	process.stdout.write(message);
+
+	// Check if we're in a TTY environment
+	if (!process.stdin.isTTY) {
+		console.log('');
+		return '';
+	}
+
+	// Use readline for full line input
+	const rl = readline.createInterface({
+		input: process.stdin,
+		output: process.stdout,
+	});
+
+	return new Promise((resolve) => {
+		rl.question('', (answer: string) => {
+			rl.close();
+			resolve(answer);
+		});
+	});
+}
+
 export async function selectOrganization(
 	orgs: OrganizationList,
 	initial?: string
@@ -1087,17 +1169,75 @@ export async function selectOrganization(
 		);
 	}
 
-	if (orgs.length === 1) {
-		return orgs[0].id;
+	if (process.env.AGENTUITY_CLOUD_ORG_ID) {
+		const org = orgs.find((o) => o.id === process.env.AGENTUITY_CLOUD_ORG_ID);
+		if (org) {
+			return org.id;
+		}
+	}
+
+	if (!process.stdin.isTTY) {
+		if (orgs.length === 1) {
+			return orgs[0].id;
+		}
+		if (initial) {
+			return initial;
+		}
+		fatal(
+			'Organization selection required but cannot prompt in non-interactive environment. Set AGENTUITY_CLOUD_ORG_ID or provide a default organization using --org-id'
+		);
 	}
 
 	const response = await enquirer.prompt<{ action: string }>({
 		type: 'select',
 		name: 'action',
 		message: 'Select an organization',
-		initial,
+		initial: initial || (orgs.length === 1 ? orgs[0].id : undefined),
 		choices: orgs.map((o) => ({ message: o.name, name: o.id })),
 	});
 
 	return response.action;
 }
+
+/**
+ * show a project list picker
+ *
+ * @param apiClient
+ * @param showDeployment
+ * @returns
+ */
+export async function showProjectList(
+	apiClient: APIClientType,
+	showDeploymentId = false
+): Promise<string> {
+	const projects = await spinner({
+		message: 'Fetching projects',
+		clearOnSuccess: true,
+		callback: () => {
+			return projectList(apiClient, showDeploymentId);
+		},
+	});
+
+	if (projects.length === 0) {
+		return '';
+	}
+
+	// TODO: might want to sort by the last org_id we used
+	if (projects) {
+		projects.sort((a, b) => {
+			return a.name.localeCompare(b.name);
+		});
+	}
+
+	const response = await enquirer.prompt<{ id: string }>({
+		type: 'select',
+		name: 'id',
+		message: 'Select a project:',
+		choices: projects.map((p) => ({
+			name: p.id,
+			message: `${p.name.padEnd(25, ' ')} ${muted(p.id)} ${showDeploymentId ? muted(p.latestDeploymentId ?? 'no deployment') : ''}`,
+		})),
+	});
+
+	return response.id;
+}