@buenojs/bueno 0.8.4 → 0.8.6

package/src/cli/commands/migration.ts

@@ -4,25 +4,31 @@
  * Manage database migrations
  */
 
-import { defineCommand } from './index';
-import { getOption, hasFlag, type ParsedArgs } from '../core/args';
-import { cliConsole, colors, printTable } from '../core/console';
-import { spinner } from '../core/spinner';
+import { type ParsedArgs, getOption, hasFlag } from "../core/args";
+import { cliConsole, colors, printTable } from "../core/console";
+import { spinner } from "../core/spinner";
+import { CLIError, CLIErrorType } from "../index";
 import {
 	fileExists,
-	readFile,
-	writeFile,
-	listFiles,
 	getProjectRoot,
 	isBuenoProject,
 	joinPaths,
-} from '../utils/fs';
-import { CLIError, CLIErrorType } from '../index';
+	listFiles,
+	readFile,
+	writeFile,
+} from "../utils/fs";
+import { defineCommand } from "./index";
 
 /**
  * Migration actions
  */
-type MigrationAction = 'create' | 'up' | 'down' | 'reset' | 'refresh' | 'status';
+type MigrationAction =
+	| "create"
+	| "up"
+	| "down"
+	| "reset"
+	| "refresh"
+	| "status";
 
 /**
  * Generate migration ID
@@ -30,11 +36,11 @@ type MigrationAction = 'create' | 'up' | 'down' | 'reset' | 'refresh' | 'status'
 function generateMigrationId(): string {
 	const now = new Date();
 	const year = now.getFullYear();
-	const month = String(now.getMonth() + 1).padStart(2, '0');
-	const day = String(now.getDate()).padStart(2, '0');
-	const hour = String(now.getHours()).padStart(2, '0');
-	const minute = String(now.getMinutes()).padStart(2, '0');
-	const second = String(now.getSeconds()).padStart(2, '0');
+	const month = String(now.getMonth() + 1).padStart(2, "0");
+	const day = String(now.getDate()).padStart(2, "0");
+	const hour = String(now.getHours()).padStart(2, "0");
+	const minute = String(now.getMinutes()).padStart(2, "0");
+	const second = String(now.getSeconds()).padStart(2, "0");
 	return `${year}${month}${day}${hour}${minute}${second}`;
 }
 
@@ -44,17 +50,14 @@ function generateMigrationId(): string {
 async function getMigrationsDir(): Promise<string> {
 	const projectRoot = await getProjectRoot();
 	if (!projectRoot) {
-		throw new CLIError(
-			'Not in a project directory',
-			CLIErrorType.NOT_FOUND,
-		);
+		throw new CLIError("Not in a project directory", CLIErrorType.NOT_FOUND);
 	}
 
 	// Check common locations
 	const possibleDirs = [
-		joinPaths(projectRoot, 'server', 'database', 'migrations'),
-		joinPaths(projectRoot, 'database', 'migrations'),
-		joinPaths(projectRoot, 'migrations'),
+		joinPaths(projectRoot, "server", "database", "migrations"),
+		joinPaths(projectRoot, "database", "migrations"),
+		joinPaths(projectRoot, "migrations"),
 	];
 
 	for (const dir of possibleDirs) {
@@ -64,14 +67,14 @@ async function getMigrationsDir(): Promise<string> {
 	}
 
 	// Default to server/database/migrations
-	return possibleDirs[0] ?? '';
+	return possibleDirs[0] ?? "";
 }
 
 /**
  * Get migration files
  */
 async function getMigrationFiles(dir: string): Promise<string[]> {
-	if (!await fileExists(dir)) {
+	if (!(await fileExists(dir))) {
 		return [];
 	}
 
@@ -100,11 +103,11 @@ function parseMigrationFile(filename: string): { id: string; name: string } {
 async function createMigration(name: string, dryRun: boolean): Promise<string> {
 	const migrationsDir = await getMigrationsDir();
 	const id = generateMigrationId();
-	const kebabName = name.toLowerCase().replace(/\s+/g, '-');
+	const kebabName = name.toLowerCase().replace(/\s+/g, "-");
 	const fileName = `${id}_${kebabName}.ts`;
 	const filePath = joinPaths(migrationsDir, fileName);
 
-	const template = `import { createMigration, type MigrationRunner } from '@buenojs/bueno';
+	const template = `import { createMigration, type MigrationRunner } from '@buenojs/bueno/migrations';
 
 export default createMigration('${id}', '${kebabName}')
   .up(async (db: MigrationRunner) => {
@@ -126,10 +129,10 @@ export default createMigration('${id}', '${kebabName}')
 `;
 
 	if (dryRun) {
-		cliConsole.log(`\n${colors.bold('File:')} ${filePath}`);
-		cliConsole.log(colors.bold('Content:'));
+		cliConsole.log(`\n${colors.bold("File:")} ${filePath}`);
+		cliConsole.log(colors.bold("Content:"));
 		cliConsole.log(template);
-		cliConsole.log('');
+		cliConsole.log("");
 		return filePath;
 	}
 
@@ -145,19 +148,19 @@ async function showStatus(): Promise<void> {
 	const files = await getMigrationFiles(migrationsDir);
 
 	if (files.length === 0) {
-		cliConsole.info('No migrations found');
+		cliConsole.info("No migrations found");
 		return;
 	}
 
-	cliConsole.header('Migration Status');
+	cliConsole.header("Migration Status");
 
 	const rows = files.map((file) => {
-		const info = parseMigrationFile(file.split('/').pop() ?? '');
-		return [info.id, info.name, colors.yellow('Pending')];
+		const info = parseMigrationFile(file.split("/").pop() ?? "");
+		return [info.id, info.name, colors.yellow("Pending")];
 	});
 
-	printTable(['ID', 'Name', 'Status'], rows);
-	cliConsole.log('');
+	printTable(["ID", "Name", "Status"], rows);
+	cliConsole.log("");
 	cliConsole.log(`Total: ${files.length} migration(s)`);
 }
 
@@ -169,45 +172,52 @@ async function handleMigration(args: ParsedArgs): Promise<void> {
 	const action = args.positionals[0] as MigrationAction | undefined;
 	if (!action) {
 		throw new CLIError(
-			'Action is required. Usage: bueno migration <action>',
+			"Action is required. Usage: bueno migration <action>",
 			CLIErrorType.INVALID_ARGS,
 		);
 	}
 
-	const validActions: MigrationAction[] = ['create', 'up', 'down', 'reset', 'refresh', 'status'];
+	const validActions: MigrationAction[] = [
+		"create",
+		"up",
+		"down",
+		"reset",
+		"refresh",
+		"status",
+	];
 	if (!validActions.includes(action)) {
 		throw new CLIError(
-			`Unknown action: ${action}. Valid actions: ${validActions.join(', ')}`,
+			`Unknown action: ${action}. Valid actions: ${validActions.join(", ")}`,
 			CLIErrorType.INVALID_ARGS,
 		);
 	}
 
 	// Get options
-	const dryRun = hasFlag(args, 'dry-run');
-	const steps = getOption(args, 'steps', {
-		name: 'steps',
-		alias: 'n',
-		type: 'number',
+	const dryRun = hasFlag(args, "dry-run");
+	const steps = getOption(args, "steps", {
+		name: "steps",
+		alias: "n",
+		type: "number",
 		default: 1,
-		description: '',
+		description: "",
 	});
 
 	// Check if in a Bueno project (except for create with dry-run)
-	if (action !== 'create' || !dryRun) {
+	if (action !== "create" || !dryRun) {
 		if (!(await isBuenoProject())) {
 			throw new CLIError(
-				'Not in a Bueno project directory. Run this command from a Bueno project.',
+				"Not in a Bueno project directory. Run this command from a Bueno project.",
 				CLIErrorType.NOT_FOUND,
 			);
 		}
 	}
 
 	switch (action) {
-		case 'create': {
+		case "create": {
 			const name = args.positionals[1];
 			if (!name) {
 				throw new CLIError(
-					'Migration name is required. Usage: bueno migration create <name>',
+					"Migration name is required. Usage: bueno migration create <name>",
 					CLIErrorType.INVALID_ARGS,
 				);
 			}
@@ -216,7 +226,7 @@ async function handleMigration(args: ParsedArgs): Promise<void> {
 			try {
 				const filePath = await createMigration(name, dryRun);
 				if (dryRun) {
-					s.info('Dry run complete');
+					s.info("Dry run complete");
 				} else {
 					s.success(`Created ${colors.green(filePath)}`);
 				}
@@ -227,83 +237,91 @@ async function handleMigration(args: ParsedArgs): Promise<void> {
 			break;
 		}
 
-		case 'up': {
-			cliConsole.info('Running pending migrations...');
-			cliConsole.log('');
+		case "up": {
+			cliConsole.info("Running pending migrations...");
+			cliConsole.log("");
 			cliConsole.warn(
-				'Migration execution requires database connection. Use the MigrationRunner in your application code.',
+				"Migration execution requires database connection. Use the MigrationRunner in your application code.",
 			);
-			cliConsole.log('');
-			cliConsole.log('Example:');
-			cliConsole.log(colors.cyan(`
-		import { createMigrationRunner, loadMigrations } from '@buenojs/bueno';
+			cliConsole.log("");
+			cliConsole.log("Example:");
+			cliConsole.log(
+				colors.cyan(`
+		import { createMigrationRunner, loadMigrations } from '@buenojs/bueno/migrations';
 import { db } from './database';
 
 const runner = createMigrationRunner(db);
 const migrations = await loadMigrations('./database/migrations');
 await runner.migrate(migrations);
-`));
+`),
+			);
 			break;
 		}
 
-		case 'down': {
+		case "down": {
 			cliConsole.info(`Rolling back ${steps} migration(s)...`);
-			cliConsole.log('');
+			cliConsole.log("");
 			cliConsole.warn(
-				'Migration rollback requires database connection. Use the MigrationRunner in your application code.',
+				"Migration rollback requires database connection. Use the MigrationRunner in your application code.",
 			);
-			cliConsole.log('');
-			cliConsole.log('Example:');
-			cliConsole.log(colors.cyan(`
-		import { createMigrationRunner, loadMigrations } from '@buenojs/bueno';
+			cliConsole.log("");
+			cliConsole.log("Example:");
+			cliConsole.log(
+				colors.cyan(`
+		import { createMigrationRunner, loadMigrations } from '@buenojs/bueno/migrations';
 import { db } from './database';
 
 const runner = createMigrationRunner(db);
 const migrations = await loadMigrations('./database/migrations');
 await runner.rollback(migrations, ${steps});
-`));
+`),
+			);
 			break;
 		}
 
-		case 'reset': {
-			cliConsole.info('Rolling back all migrations...');
-			cliConsole.log('');
+		case "reset": {
+			cliConsole.info("Rolling back all migrations...");
+			cliConsole.log("");
 			cliConsole.warn(
-				'Migration reset requires database connection. Use the MigrationRunner in your application code.',
+				"Migration reset requires database connection. Use the MigrationRunner in your application code.",
 			);
-			cliConsole.log('');
-			cliConsole.log('Example:');
-			cliConsole.log(colors.cyan(`
-		import { createMigrationRunner, loadMigrations } from '@buenojs/bueno';
+			cliConsole.log("");
+			cliConsole.log("Example:");
+			cliConsole.log(
+				colors.cyan(`
+		import { createMigrationRunner, loadMigrations } from '@buenojs/bueno/migrations';
 import { db } from './database';
 
 const runner = createMigrationRunner(db);
 const migrations = await loadMigrations('./database/migrations');
 await runner.reset(migrations);
-`));
+`),
+			);
 			break;
 		}
 
-		case 'refresh': {
-			cliConsole.info('Refreshing all migrations...');
-			cliConsole.log('');
+		case "refresh": {
+			cliConsole.info("Refreshing all migrations...");
+			cliConsole.log("");
 			cliConsole.warn(
-				'Migration refresh requires database connection. Use the MigrationRunner in your application code.',
+				"Migration refresh requires database connection. Use the MigrationRunner in your application code.",
 			);
-			cliConsole.log('');
-			cliConsole.log('Example:');
-			cliConsole.log(colors.cyan(`
-		import { createMigrationRunner, loadMigrations } from '@buenojs/bueno';
+			cliConsole.log("");
+			cliConsole.log("Example:");
+			cliConsole.log(
+				colors.cyan(`
+		import { createMigrationRunner, loadMigrations } from '@buenojs/bueno/migrations';
 import { db } from './database';
 
 const runner = createMigrationRunner(db);
 const migrations = await loadMigrations('./database/migrations');
 await runner.refresh(migrations);
-`));
+`),
+			);
 			break;
 		}
 
-		case 'status': {
+		case "status": {
 			await showStatus();
 			break;
 		}
@@ -313,43 +331,44 @@ await runner.refresh(migrations);
 // Register the command
 defineCommand(
 	{
-		name: 'migration',
-		description: 'Manage database migrations',
+		name: "migration",
+		description: "Manage database migrations",
 		positionals: [
 			{
-				name: 'action',
+				name: "action",
 				required: true,
-				description: 'Action to perform (create, up, down, reset, refresh, status)',
+				description:
+					"Action to perform (create, up, down, reset, refresh, status)",
 			},
 			{
-				name: 'name',
+				name: "name",
 				required: false,
-				description: 'Migration name (required for create action)',
+				description: "Migration name (required for create action)",
 			},
 		],
 		options: [
 			{
-				name: 'steps',
-				alias: 'n',
-				type: 'number',
+				name: "steps",
+				alias: "n",
+				type: "number",
 				default: 1,
-				description: 'Number of migrations to rollback',
+				description: "Number of migrations to rollback",
 			},
 			{
-				name: 'dry-run',
-				type: 'boolean',
+				name: "dry-run",
+				type: "boolean",
 				default: false,
-				description: 'Show what would happen without executing',
+				description: "Show what would happen without executing",
 			},
 		],
 		examples: [
-			'bueno migration create add-users-table',
-			'bueno migration up',
-			'bueno migration down --steps 3',
-			'bueno migration reset',
-			'bueno migration refresh',
-			'bueno migration status',
+			"bueno migration create add-users-table",
+			"bueno migration up",
+			"bueno migration down --steps 3",
+			"bueno migration reset",
+			"bueno migration refresh",
+			"bueno migration status",
 		],
 	},
 	handleMigration,
-);
+);