@agentuity/cli 0.0.53 → 0.0.54

package/src/cmd/profile/create.ts

@@ -59,7 +59,7 @@ export const createCommand = createSubcommand({
 			return logger.fatal(
 				`Profile "${name}" already exists at ${existing.filename}`,
 				ErrorCode.RESOURCE_ALREADY_EXISTS
-			);
+			) as never;
 		}
 
 		await ensureConfigDir();
@@ -103,10 +103,10 @@ export const createCommand = createSubcommand({
 		} catch (error) {
 			const message = error instanceof Error ? error.message : String(error);
 			const stack = error instanceof Error ? error.stack : undefined;
-			logger.fatal(
+			return logger.fatal(
 				`Failed to create profile: ${message}${stack ? `\n${stack}` : ''}`,
 				ErrorCode.INTERNAL_ERROR
-			);
+			) as never;
 		}
 	},
 });

package/src/cmd/profile/delete.ts

@@ -43,7 +43,7 @@ export const deleteCommand = createSubcommand({
 		const profile = profiles.find((p) => p.name === name);
 
 		if (!profile) {
-			return logger.fatal(`Profile "${name}" not found`, ErrorCode.RESOURCE_NOT_FOUND);
+			return logger.fatal(`Profile "${name}" not found`, ErrorCode.RESOURCE_NOT_FOUND) as never;
 		}
 
 		// Ask for confirmation unless --confirm flag is passed
@@ -66,7 +66,10 @@ export const deleteCommand = createSubcommand({
 
 			return { success: true, name };
 		} catch (error) {
-			logger.fatal(`Failed to delete profile: ${error}`);
+			return logger.fatal(
+				`Failed to delete profile: ${error}`,
+				ErrorCode.FILE_WRITE_ERROR
+			) as never;
 		}
 	},
 });

package/src/cmd/profile/list.ts

@@ -1,9 +1,18 @@
+import { z } from 'zod';
 import { createSubcommand } from '../../types';
 import { fetchProfiles } from '../../config';
 import { basename, dirname } from 'node:path';
 import * as tui from '../../tui';
 import { getCommand } from '../../command-prefix';
 
+const ProfileListResponseSchema = z.array(
+	z.object({
+		name: z.string().describe('Profile name'),
+		filename: z.string().describe('Profile file path'),
+		selected: z.boolean().describe('Whether this profile is currently selected'),
+	})
+);
+
 export const listCommand = createSubcommand({
 	name: 'list',
 	description: 'List all available profiles',
@@ -11,21 +20,32 @@ export const listCommand = createSubcommand({
 	idempotent: true,
 	aliases: ['ls'],
 	examples: [getCommand('profile list'), getCommand('profile ls')],
+	schema: {
+		response: ProfileListResponseSchema,
+	},
 
-	async handler() {
+	async handler(ctx) {
+		const { options } = ctx;
 		const profiles = await fetchProfiles();
 
-		if (profiles.length === 0) {
-			tui.info('No profiles found');
-			return;
+		if (!options.json) {
+			if (profiles.length === 0) {
+				tui.info('No profiles found');
+			} else {
+				console.log('Available profiles:');
+				for (const profile of profiles) {
+					const marker = profile.selected ? '•' : ' ';
+					const name = tui.padRight(profile.name, 15, ' ');
+					const path = `${basename(dirname(profile.filename))}/${basename(profile.filename)}`;
+					console.log(`${marker} ${name} ${tui.muted(path)}`);
+				}
+			}
 		}
 
-		console.log('Available profiles:');
-		for (const profile of profiles) {
-			const marker = profile.selected ? '•' : ' ';
-			const name = tui.padRight(profile.name, 15, ' ');
-			const path = `${basename(dirname(profile.filename))}/${basename(profile.filename)}`;
-			console.log(`${marker} ${name} ${tui.muted(path)}`);
-		}
+		return profiles.map((p) => ({
+			name: p.name,
+			filename: p.filename,
+			selected: p.selected,
+		}));
 	},
 });

package/src/cmd/profile/show.ts

@@ -19,9 +19,6 @@ export const showCommand = createSubcommand({
 		getCommand('profile show staging --json'),
 	],
 	schema: {
-		options: z.object({
-			json: z.boolean().optional().describe('Show the JSON config'),
-		}),
 		args: z
 			.object({
 				name: z.string().optional().describe('Profile name to show (optional)'),
@@ -32,7 +29,7 @@ export const showCommand = createSubcommand({
 	idempotent: true,
 
 	async handler(ctx) {
-		const { logger, args, opts } = ctx;
+		const { logger, args, options } = ctx;
 
 		try {
 			let current = false;
@@ -47,19 +44,25 @@ export const showCommand = createSubcommand({
 			const profile = profiles.find((p) => p.name === name);
 
 			if (!profile) {
-				return logger.fatal(`Profile "${name}" not found`, ErrorCode.RESOURCE_NOT_FOUND);
+				return logger.fatal(
+					`Profile "${name}" not found`,
+					ErrorCode.RESOURCE_NOT_FOUND
+				) as never;
 			}
 
 			const profilePath = profile.filename;
 			current = profile.selected;
 
-			tui.info(`Profile: ${profilePath}`);
-
 			const content = await loadConfig(current ? undefined : profilePath);
+			if (!content) {
+				return logger.fatal(
+					`Failed to load profile configuration`,
+					ErrorCode.INTERNAL_ERROR
+				) as never;
+			}
 
-			if (opts?.json) {
-				console.log(JSON.stringify(content, null, 2));
-			} else {
+			if (!options.json) {
+				tui.info(`Profile: ${profilePath}`);
 				tui.newline();
 				const textContent = await readFile(profilePath, 'utf-8');
 				console.log(textContent);
@@ -68,9 +71,9 @@ export const showCommand = createSubcommand({
 			return content;
 		} catch (error) {
 			if (error instanceof Error) {
-				logger.fatal(`Failed to show profile: ${error.message}`);
+				return logger.fatal(`Failed to show profile: ${error.message}`) as never;
 			} else {
-				logger.fatal('Failed to show profile');
+				return logger.fatal('Failed to show profile') as never;
 			}
 		}
 	},

package/src/cmd/project/delete.ts

@@ -59,7 +59,7 @@ export const deleteSubcommand = createSubcommand({
 
 			if (projects.length === 0) {
 				tui.info('No projects found to delete');
-				return;
+				return { success: false, projectIds: [], count: 0 };
 			}
 
 			// Sort projects by name

package/src/cmd/version/index.ts

@@ -24,7 +24,11 @@ export const command = createCommand({
 			return version;
 		} catch (error) {
 			const logger = createLogger();
-			logger.fatal('Failed to retrieve version: %s', error, ErrorCode.INTERNAL_ERROR);
+			return logger.fatal(
+				'Failed to retrieve version: %s',
+				error,
+				ErrorCode.INTERNAL_ERROR
+			) as never;
 		}
 	},
 });

package/src/tui.ts

@@ -831,7 +831,9 @@ export async function spinner<T>(
 						: await options.callback;
 
 			// If clearOnSuccess is true, don't show success message
-			if (!options.clearOnSuccess) {
+			// Also skip success message in JSON mode
+			const isJsonMode = outputOptions?.json === true;
+			if (!options.clearOnSuccess && !isJsonMode) {
 				const successColor = getColor('success');
 				console.error(`${successColor}${ICONS.success} ${message}${reset}`);
 			}

package/src/types.ts

@@ -252,6 +252,7 @@ export function createSubcommand<
 	O extends Optional | undefined = undefined,
 	A extends z.ZodType | undefined = undefined,
 	Op extends z.ZodType | undefined = undefined,
+	Res extends z.ZodType | undefined = undefined,
 >(definition: {
 	name: string;
 	description: string;
@@ -267,12 +268,22 @@ export function createSubcommand<
 	tags?: string[];
 	schema?: A extends z.ZodType
 		? Op extends z.ZodType
-			? { args: A; options: Op; response?: z.ZodType }
-			: { args: A; response?: z.ZodType }
+			? Res extends z.ZodType
+				? { args: A; options: Op; response: Res }
+				: { args: A; options: Op; response?: z.ZodType }
+			: Res extends z.ZodType
+				? { args: A; response: Res }
+				: { args: A; response?: z.ZodType }
 		: Op extends z.ZodType
-			? { options: Op; response?: z.ZodType }
-			: { response?: z.ZodType };
-	handler(ctx: CommandContext<R, O, A, Op>): unknown | Promise<unknown>;
+			? Res extends z.ZodType
+				? { options: Op; response: Res }
+				: { options: Op; response?: z.ZodType }
+			: Res extends z.ZodType
+				? { response: Res }
+				: { response?: z.ZodType };
+	handler(
+		ctx: CommandContext<R, O, A, Op>
+	): Res extends z.ZodType ? z.infer<Res> | Promise<z.infer<Res>> : unknown | Promise<unknown>;
 }): SubcommandDefinition {
 	return definition as unknown as SubcommandDefinition;
 }
@@ -282,6 +293,7 @@ export function createCommand<
 	O extends Optional | undefined = undefined,
 	A extends z.ZodType | undefined = undefined,
 	Op extends z.ZodType | undefined = undefined,
+	Res extends z.ZodType | undefined = undefined,
 >(definition: {
 	name: string;
 	description: string;
@@ -297,12 +309,22 @@ export function createCommand<
 	tags?: string[];
 	schema?: A extends z.ZodType
 		? Op extends z.ZodType
-			? { args: A; options: Op; response?: z.ZodType }
-			: { args: A; response?: z.ZodType }
+			? Res extends z.ZodType
+				? { args: A; options: Op; response: Res }
+				: { args: A; options: Op; response?: z.ZodType }
+			: Res extends z.ZodType
+				? { args: A; response: Res }
+				: { args: A; response?: z.ZodType }
 		: Op extends z.ZodType
-			? { options: Op; response?: z.ZodType }
-			: { response?: z.ZodType };
-	handler?(ctx: CommandContext<R, O, A, Op>): unknown | Promise<unknown>;
+			? Res extends z.ZodType
+				? { options: Op; response: Res }
+				: { options: Op; response?: z.ZodType }
+			: Res extends z.ZodType
+				? { response: Res }
+				: { response?: z.ZodType };
+	handler?(
+		ctx: CommandContext<R, O, A, Op>
+	): Res extends z.ZodType ? z.infer<Res> | Promise<z.infer<Res>> : unknown | Promise<unknown>;
 	subcommands?: SubcommandDefinition[];
 }): CommandDefinition {
 	return definition as unknown as CommandDefinition;