i18nsmith 0.2.1 → 0.3.1

package/src/commands/init.ts

@@ -6,6 +6,7 @@ import chalk from 'chalk';
 import { diagnoseWorkspace, I18nConfig, TranslationConfig, ensureGitignore } from '@i18nsmith/core';
 import { scaffoldTranslationContext, scaffoldI18next } from '../utils/scaffold.js';
 import { hasDependency, readPackageJson } from '../utils/pkg.js';
+import { CliError, withErrorHandling } from '../utils/errors.js';
 
 /**
  * Parse a comma-separated list of glob patterns, respecting brace expansions.
@@ -184,7 +185,8 @@ async function runNonInteractiveInit(commandOptions: InitCommandOptions): Promis
 
     console.log(chalk.blue('\nRun "i18nsmith check" to verify your setup.'));
   } catch (error) {
-    console.error(chalk.red('Failed to write configuration file:'), error);
+    const message = error instanceof Error ? error.message : String(error);
+    throw new CliError(`Failed to write configuration file: ${message}`);
   }
 }
 
@@ -194,14 +196,15 @@ export function registerInit(program: Command) {
     .description('Initialize i18nsmith configuration')
     .option('--merge', 'Merge with existing locales/runtimes when detected', false)
     .option('-y, --yes', 'Skip prompts and use defaults (non-interactive mode)', false)
-    .action(async (commandOptions: InitCommandOptions) => {
-      console.log(chalk.blue('Initializing i18nsmith configuration...'));
-
-      // Non-interactive mode with sensible defaults
-      if (commandOptions.yes) {
-        await runNonInteractiveInit(commandOptions);
-        return;
-      }
+    .action(
+      withErrorHandling(async (commandOptions: InitCommandOptions) => {
+        console.log(chalk.blue('Initializing i18nsmith configuration...'));
+
+        // Non-interactive mode with sensible defaults
+        if (commandOptions.yes) {
+          await runNonInteractiveInit(commandOptions);
+          return;
+        }
 
       const answers = await inquirer.prompt<InitAnswers>([
         {
@@ -380,85 +383,87 @@ export function registerInit(program: Command) {
         seedTargetLocales: answers.seedTargetLocales,
       };
 
-      const workspaceRoot = process.cwd();
-      const mergeDecision = await maybePromptMergeStrategy(config, workspaceRoot, Boolean(commandOptions.merge));
-      if (mergeDecision?.aborted) {
-        console.log(chalk.yellow('Aborting init to avoid overwriting existing i18n assets. Re-run with --merge to bypass.'));
-        return;
-      }
-
-  const configPath = path.join(workspaceRoot, 'i18n.config.json');
-      
-      try {
-        await fs.writeFile(configPath, JSON.stringify(config, null, 2));
-        console.log(chalk.green(`\nConfiguration created at ${configPath}`));
-
-        // Ensure .gitignore has i18nsmith artifacts
-        const gitignoreResult = await ensureGitignore(workspaceRoot);
-        if (gitignoreResult.updated) {
-          console.log(chalk.green(`Updated .gitignore with i18nsmith artifacts`));
+        const workspaceRoot = process.cwd();
+        const mergeDecision = await maybePromptMergeStrategy(config, workspaceRoot, Boolean(commandOptions.merge));
+        if (mergeDecision?.aborted) {
+          console.log(chalk.yellow('Aborting init to avoid overwriting existing i18n assets. Re-run with --merge to bypass.'));
+          return;
         }
 
-        if (answers.scaffoldAdapter && answers.scaffoldAdapterPath) {
-          try {
-            await scaffoldTranslationContext(answers.scaffoldAdapterPath, answers.sourceLanguage, {
-              localesDir: answers.localesDir,
-            });
-            console.log(chalk.green(`Translation context scaffolded at ${answers.scaffoldAdapterPath}`));
-          } catch (error) {
-            console.warn(chalk.yellow(`Skipping adapter scaffold: ${(error as Error).message}`));
+        const configPath = path.join(workspaceRoot, 'i18n.config.json');
+
+        try {
+          await fs.writeFile(configPath, JSON.stringify(config, null, 2));
+          console.log(chalk.green(`\nConfiguration created at ${configPath}`));
+
+          // Ensure .gitignore has i18nsmith artifacts
+          const gitignoreResult = await ensureGitignore(workspaceRoot);
+          if (gitignoreResult.updated) {
+            console.log(chalk.green(`Updated .gitignore with i18nsmith artifacts`));
           }
-        }
 
-        if (
-          answers.adapterPreset === 'react-i18next' &&
-          answers.scaffoldReactRuntime &&
-          answers.reactI18nPath &&
-          answers.reactProviderPath
-        ) {
-          try {
-            await scaffoldI18next(
-              answers.reactI18nPath,
-              answers.reactProviderPath,
-              answers.sourceLanguage,
-              answers.localesDir
-            );
-            console.log(chalk.green('react-i18next runtime scaffolded:'));
-            console.log(chalk.green(`  • ${answers.reactI18nPath}`));
-            console.log(chalk.green(`  • ${answers.reactProviderPath}`));
-            console.log(chalk.blue('\nWrap your app with the provider (e.g. Next.js providers.tsx):'));
-            console.log(
-              chalk.cyan(
-                `import { I18nProvider } from '${answers.reactProviderPath.replace(/\\/g, '/').replace(/\.tsx?$/, '')}';\n<I18nProvider>{children}</I18nProvider>`
-              )
-            );
-          } catch (error) {
-            console.warn(chalk.yellow(`Skipping i18next scaffold: ${(error as Error).message}`));
+          if (answers.scaffoldAdapter && answers.scaffoldAdapterPath) {
+            try {
+              await scaffoldTranslationContext(answers.scaffoldAdapterPath, answers.sourceLanguage, {
+                localesDir: answers.localesDir,
+              });
+              console.log(chalk.green(`Translation context scaffolded at ${answers.scaffoldAdapterPath}`));
+            } catch (error) {
+              console.warn(chalk.yellow(`Skipping adapter scaffold: ${(error as Error).message}`));
+            }
           }
-        }
 
-        if (answers.adapterPreset === 'react-i18next') {
-          const pkg = await readPackageJson();
-          const missingDeps = ['react-i18next', 'i18next'].filter((dep) => !hasDependency(pkg, dep));
-          if (missingDeps.length) {
-            console.log(chalk.yellow('\nDependencies missing for react-i18next adapter:'));
-            missingDeps.forEach((dep) => console.log(chalk.yellow(`  • ${dep}`)));
-            console.log(chalk.blue('Install them with:'));
-            console.log(chalk.cyan('  pnpm add react-i18next i18next'));
+          if (
+            answers.adapterPreset === 'react-i18next' &&
+            answers.scaffoldReactRuntime &&
+            answers.reactI18nPath &&
+            answers.reactProviderPath
+          ) {
+            try {
+              await scaffoldI18next(
+                answers.reactI18nPath,
+                answers.reactProviderPath,
+                answers.sourceLanguage,
+                answers.localesDir
+              );
+              console.log(chalk.green('react-i18next runtime scaffolded:'));
+              console.log(chalk.green(`  • ${answers.reactI18nPath}`));
+              console.log(chalk.green(`  • ${answers.reactProviderPath}`));
+              console.log(chalk.blue('\nWrap your app with the provider (e.g. Next.js providers.tsx):'));
+              console.log(
+                chalk.cyan(
+                  `import { I18nProvider } from '${answers.reactProviderPath.replace(/\\/g, '/').replace(/\.tsx?$/, '')}';\n<I18nProvider>{children}</I18nProvider>`
+                )
+              );
+            } catch (error) {
+              console.warn(chalk.yellow(`Skipping i18next scaffold: ${(error as Error).message}`));
+            }
           }
-        }
 
-        if (mergeDecision?.strategy) {
-          console.log(
-            chalk.blue(
-              `Merge strategy selected: ${mergeDecision.strategy}. Use this when running i18nsmith sync or diagnose to reconcile locales.`
-            )
-          );
+          if (answers.adapterPreset === 'react-i18next') {
+            const pkg = await readPackageJson();
+            const missingDeps = ['react-i18next', 'i18next'].filter((dep) => !hasDependency(pkg, dep));
+            if (missingDeps.length) {
+              console.log(chalk.yellow('\nDependencies missing for react-i18next adapter:'));
+              missingDeps.forEach((dep) => console.log(chalk.yellow(`  • ${dep}`)));
+              console.log(chalk.blue('Install them with:'));
+              console.log(chalk.cyan('  pnpm add react-i18next i18next'));
+            }
+          }
+
+          if (mergeDecision?.strategy) {
+            console.log(
+              chalk.blue(
+                `Merge strategy selected: ${mergeDecision.strategy}. Use this when running i18nsmith sync or diagnose to reconcile locales.`
+              )
+            );
+          }
+        } catch (error) {
+          const message = error instanceof Error ? error.message : String(error);
+          throw new CliError(`Failed to write configuration file: ${message}`);
         }
-      } catch (error) {
-        console.error(chalk.red('Failed to write configuration file:'), error);
-      }
-    });
+      })
+    );
 }
 
 type MergeStrategy = 'keep-source' | 'overwrite' | 'interactive';

package/src/commands/install-hooks.ts

@@ -2,6 +2,7 @@ import fs from 'fs/promises';
 import path from 'path';
 import { Command } from 'commander';
 import { detectPackageManager } from '../utils/package-manager.js';
+import { withErrorHandling } from '../utils/errors.js';
 
 interface InstallHooksOptions {
   yes?: boolean;
@@ -42,25 +43,27 @@ export function registerInstallHooks(program: Command) {
     .option('-y, --yes', 'Skip confirmations')
     .option('--force', 'Force re-install husky even if present')
     .option('--skip', 'Skip husky installation (just create hooks)')
-    .action(async (opts: InstallHooksOptions) => {
-      const cwd = opts.cwd ?? process.cwd();
-  const pm = await detectPackageManager();
-      const hasHusky = await huskyInstalled(cwd);
+    .action(
+      withErrorHandling(async (opts: InstallHooksOptions) => {
+        const cwd = opts.cwd ?? process.cwd();
+        const pm = await detectPackageManager();
+        const hasHusky = await huskyInstalled(cwd);
 
-      if (!hasHusky && !opts.skip) {
-        await ensureHusky(cwd, pm, !!opts.force);
-        console.log('Add "prepare": "husky install" to package.json scripts if missing.');
-      }
+        if (!hasHusky && !opts.skip) {
+          await ensureHusky(cwd, pm, !!opts.force);
+          console.log('Add "prepare": "husky install" to package.json scripts if missing.');
+        }
 
-      const preCommitContent = `#!/bin/sh\n. \"$(dirname "$0")/_/husky.sh\"\n[ -n \"$I18NSMITH_SKIP_HOOKS\" ] && exit 0\nNOCOLOR=1 npx i18nsmith check --fail-on conflicts || exit 1\n`;
-      const prePushContent = `#!/bin/sh\n. \"$(dirname "$0")/_/husky.sh\"\n[ -n \"$I18NSMITH_SKIP_HOOKS\" ] && exit 0\nNOCOLOR=1 npx i18nsmith sync --dry-run --check || exit 1\n`;
+  const preCommitContent = `#!/bin/sh\n. "$(dirname "$0")/_/husky.sh"\n[ -n "$I18NSMITH_SKIP_HOOKS" ] && exit 0\nNOCOLOR=1 npx i18nsmith check --fail-on conflicts || exit 1\n`;
+  const prePushContent = `#!/bin/sh\n. "$(dirname "$0")/_/husky.sh"\n[ -n "$I18NSMITH_SKIP_HOOKS" ] && exit 0\nNOCOLOR=1 npx i18nsmith sync --dry-run --check || exit 1\n`;
 
-      await writeHook(cwd, 'pre-commit', preCommitContent);
-      await writeHook(cwd, 'pre-push', prePushContent);
+        await writeHook(cwd, 'pre-commit', preCommitContent);
+        await writeHook(cwd, 'pre-push', prePushContent);
 
-      console.log('\nHooks added. Set I18NSMITH_SKIP_HOOKS=1 to bypass.');
-      console.log('Prototype complete – future versions will offer interactive selection & monorepo scoping.');
-    });
+        console.log('\nHooks added. Set I18NSMITH_SKIP_HOOKS=1 to bypass.');
+        console.log('Prototype complete – future versions will offer interactive selection & monorepo scoping.');
+      })
+    );
 
   program.addCommand(cmd);
 }

package/src/commands/preflight.ts

@@ -8,8 +8,9 @@ import fs from 'fs/promises';
 import path from 'path';
 import chalk from 'chalk';
 import fg from 'fast-glob';
-import { loadConfigWithMeta, diagnoseWorkspace, I18nConfig } from '@i18nsmith/core';
+import { loadConfigWithMeta, I18nConfig } from '@i18nsmith/core';
 import { hasDependency, readPackageJson } from '../utils/pkg.js';
+import { CliError, withErrorHandling } from '../utils/errors.js';
 
 interface PreflightResult {
   passed: boolean;
@@ -36,18 +37,25 @@ export function registerPreflight(program: Command) {
     .option('-c, --config <path>', 'Path to i18nsmith config file', 'i18n.config.json')
     .option('--fix', 'Attempt to fix issues automatically', false)
     .option('--json', 'Output results as JSON', false)
-    .action(async (options: PreflightOptions) => {
-      const result = await runPreflightChecks(options);
-
-      if (options.json) {
-        console.log(JSON.stringify(result, null, 2));
-        process.exitCode = result.passed ? 0 : 1;
-        return;
-      }
-
-      printPreflightResults(result);
-      process.exitCode = result.passed ? 0 : 1;
-    });
+    .action(
+      withErrorHandling(async (options: PreflightOptions) => {
+        try {
+          const result = await runPreflightChecks(options);
+
+          if (options.json) {
+            console.log(JSON.stringify(result, null, 2));
+            process.exitCode = result.passed ? 0 : 1;
+            return;
+          }
+
+          printPreflightResults(result);
+          process.exitCode = result.passed ? 0 : 1;
+        } catch (error) {
+          const message = error instanceof Error ? error.message : String(error);
+          throw new CliError(`Preflight failed: ${message}`);
+        }
+      })
+    );
 }
 
 async function runPreflightChecks(options: PreflightOptions): Promise<PreflightResult> {

package/src/commands/rename.ts

@@ -4,6 +4,7 @@ import path from 'node:path';
 import { promises as fs } from 'node:fs';
 import { loadConfig, KeyRenamer, type KeyRenameSummary, type KeyRenameBatchSummary, type KeyRenameMapping } from '@i18nsmith/core';
 import { applyPreviewFile, writePreviewFile } from '../utils/preview.js';
+import { CliError, withErrorHandling } from '../utils/errors.js';
 
 interface ScanOptions {
   config: string;
@@ -40,58 +41,60 @@ export function registerRename(program: Command): void {
     .option('--diff', 'Display unified diffs for files that would change', false)
     .option('--preview-output <path>', 'Write preview summary (JSON) to a file (implies dry-run)')
     .option('--apply-preview <path>', 'Apply a previously saved rename preview JSON file safely')
-    .action(async (oldKey: string, newKey: string, options: RenameKeyOptions) => {
-      if (options.applyPreview) {
-        await applyPreviewFile('rename-key', options.applyPreview);
-        return;
-      }
-
-      const previewMode = Boolean(options.previewOutput);
-      const writeEnabled = Boolean(options.write) && !previewMode;
-      if (previewMode && options.write) {
-        console.log(chalk.yellow('Preview requested; ignoring --write and running in dry-run mode.'));
-      }
-      options.write = writeEnabled;
-
-      console.log(chalk.blue(writeEnabled ? 'Renaming translation key...' : 'Planning key rename (dry-run)...'));
-
-      try {
-        const config = await loadConfig(options.config);
-        const renamer = new KeyRenamer(config);
-        const summary = await renamer.rename(oldKey, newKey, {
-          write: options.write,
-          diff: options.diff || previewMode,
-        });
-
-        if (previewMode && options.previewOutput) {
-          const savedPath = await writePreviewFile('rename-key', summary, options.previewOutput);
-          console.log(chalk.green(`Preview written to ${path.relative(process.cwd(), savedPath)}`));
+    .action(
+      withErrorHandling(async (oldKey: string, newKey: string, options: RenameKeyOptions) => {
+        if (options.applyPreview) {
+          await applyPreviewFile('rename-key', options.applyPreview);
           return;
         }
 
-        if (options.report) {
-          const outputPath = path.resolve(process.cwd(), options.report);
-          await fs.mkdir(path.dirname(outputPath), { recursive: true });
-          await fs.writeFile(outputPath, JSON.stringify(summary, null, 2));
-          console.log(chalk.green(`Rename report written to ${outputPath}`));
+        const previewMode = Boolean(options.previewOutput);
+        const writeEnabled = Boolean(options.write) && !previewMode;
+        if (previewMode && options.write) {
+          console.log(chalk.yellow('Preview requested; ignoring --write and running in dry-run mode.'));
         }
+        options.write = writeEnabled;
+
+        console.log(chalk.blue(writeEnabled ? 'Renaming translation key...' : 'Planning key rename (dry-run)...'));
+
+        try {
+          const config = await loadConfig(options.config);
+          const renamer = new KeyRenamer(config);
+          const summary = await renamer.rename(oldKey, newKey, {
+            write: options.write,
+            diff: options.diff || previewMode,
+          });
+
+          if (previewMode && options.previewOutput) {
+            const savedPath = await writePreviewFile('rename-key', summary, options.previewOutput);
+            console.log(chalk.green(`Preview written to ${path.relative(process.cwd(), savedPath)}`));
+            return;
+          }
 
-        if (options.json) {
-          console.log(JSON.stringify(summary, null, 2));
-          return;
-        }
+          if (options.report) {
+            const outputPath = path.resolve(process.cwd(), options.report);
+            await fs.mkdir(path.dirname(outputPath), { recursive: true });
+            await fs.writeFile(outputPath, JSON.stringify(summary, null, 2));
+            console.log(chalk.green(`Rename report written to ${outputPath}`));
+          }
+
+          if (options.json) {
+            console.log(JSON.stringify(summary, null, 2));
+            return;
+          }
 
-        printRenameSummary(summary);
+          printRenameSummary(summary);
 
-        if (!options.write) {
-          console.log(chalk.cyan('\n📋 DRY RUN - No files were modified'));
-          console.log(chalk.yellow('Run again with --write to apply changes.'));
+          if (!options.write) {
+            console.log(chalk.cyan('\n📋 DRY RUN - No files were modified'));
+            console.log(chalk.yellow('Run again with --write to apply changes.'));
+          }
+        } catch (error) {
+          const message = error instanceof Error ? error.message : String(error);
+          throw new CliError(`Rename failed: ${message}`);
         }
-      } catch (error) {
-        console.error(chalk.red('Rename failed:'), (error as Error).message);
-        process.exitCode = 1;
-      }
-    });
+      })
+    );
 
   program
     .command('rename-keys')
@@ -102,49 +105,51 @@ export function registerRename(program: Command): void {
     .option('--report <path>', 'Write JSON summary to a file (for CI or editors)')
     .option('--write', 'Write changes to disk (defaults to dry-run)', false)
     .option('--diff', 'Display unified diffs for files that would change', false)
-    .action(async (options: RenameMapOptions) => {
-      console.log(
-        chalk.blue(options.write ? 'Renaming translation keys from map...' : 'Planning batch rename (dry-run)...')
-      );
-
-      try {
-        const config = await loadConfig(options.config);
-        const mappings = await loadRenameMappings(options.map);
-        const renamer = new KeyRenamer(config);
-        const summary = await renamer.renameBatch(mappings, { write: options.write, diff: options.diff });
-
-        if (options.report) {
-          const outputPath = path.resolve(process.cwd(), options.report);
-          await fs.mkdir(path.dirname(outputPath), { recursive: true });
-          await fs.writeFile(outputPath, JSON.stringify(summary, null, 2));
-          console.log(chalk.green(`Batch rename report written to ${outputPath}`));
-        }
+    .action(
+      withErrorHandling(async (options: RenameMapOptions) => {
+        console.log(
+          chalk.blue(options.write ? 'Renaming translation keys from map...' : 'Planning batch rename (dry-run)...')
+        );
+
+        try {
+          const config = await loadConfig(options.config);
+          const mappings = await loadRenameMappings(options.map);
+          const renamer = new KeyRenamer(config);
+          const summary = await renamer.renameBatch(mappings, { write: options.write, diff: options.diff });
+
+          if (options.report) {
+            const outputPath = path.resolve(process.cwd(), options.report);
+            await fs.mkdir(path.dirname(outputPath), { recursive: true });
+            await fs.writeFile(outputPath, JSON.stringify(summary, null, 2));
+            console.log(chalk.green(`Batch rename report written to ${outputPath}`));
+          }
 
-        if (options.json) {
-          console.log(JSON.stringify(summary, null, 2));
-          return;
-        }
+          if (options.json) {
+            console.log(JSON.stringify(summary, null, 2));
+            return;
+          }
 
-        printRenameBatchSummary(summary);
+          printRenameBatchSummary(summary);
 
-        // Print source file diffs if requested
-        if (options.diff && summary.diffs.length > 0) {
-          console.log(chalk.blue('\nSource file changes:'));
-          for (const diff of summary.diffs) {
-            console.log(chalk.cyan(`\n--- ${diff.relativePath} (${diff.changes} change${diff.changes === 1 ? '' : 's'}) ---`));
-            console.log(diff.diff);
+          // Print source file diffs if requested
+          if (options.diff && summary.diffs.length > 0) {
+            console.log(chalk.blue('\nSource file changes:'));
+            for (const diff of summary.diffs) {
+              console.log(chalk.cyan(`\n--- ${diff.relativePath} (${diff.changes} change${diff.changes === 1 ? '' : 's'}) ---`));
+              console.log(diff.diff);
+            }
           }
-        }
 
-        if (!options.write) {
-          console.log(chalk.cyan('\n📋 DRY RUN - No files were modified'));
-          console.log(chalk.yellow('Run again with --write to apply changes.'));
+          if (!options.write) {
+            console.log(chalk.cyan('\n📋 DRY RUN - No files were modified'));
+            console.log(chalk.yellow('Run again with --write to apply changes.'));
+          }
+        } catch (error) {
+          const message = error instanceof Error ? error.message : String(error);
+          throw new CliError(`Batch rename failed: ${message}`);
         }
-      } catch (error) {
-        console.error(chalk.red('Batch rename failed:'), (error as Error).message);
-        process.exitCode = 1;
-      }
-    });
+      })
+    );
 }
 
 function printRenameSummary(summary: KeyRenameSummary) {