i18nsmith 0.2.1 → 0.3.0

package/src/commands/review.ts

@@ -4,6 +4,7 @@ import chalk from 'chalk';
 import inquirer from 'inquirer';
 import type { Command } from 'commander';
 import { loadConfigWithMeta, Scanner, type ScanCandidate, type ScanSummary } from '@i18nsmith/core';
+import { CliError, withErrorHandling } from '../utils/errors.js';
 
 interface ReviewCommandOptions {
   config?: string;
@@ -133,94 +134,96 @@ export function registerReview(program: Command) {
     .option('--json', 'Print raw bucket data as JSON', false)
     .option('--limit <n>', 'Limit the number of items per session (default: 20)', (value) => parseInt(value, 10))
     .option('--scan-calls', 'Include translation call arguments', false)
-    .action(async (options: ReviewCommandOptions) => {
-      try {
-        const { config, projectRoot, configPath } = await loadConfigWithMeta(options.config);
-        const scanner = new Scanner(config, { workspaceRoot: projectRoot });
-  const summary = scanner.scan({ scanCalls: options.scanCalls }) as BucketedScanSummary;
-  const buckets = summary.buckets ?? {};
-  const needsReview = buckets.needsReview ?? [];
-  const skipped = buckets.skipped ?? [];
-
-        if (options.json) {
-          console.log(JSON.stringify({ needsReview, skipped }, null, 2));
-          return;
-        }
-
-        if (!needsReview.length) {
-          console.log(chalk.green('No borderline candidates detected.'));
-          const reasons = summarizeSkipReasons(skipped);
-          if (reasons.length) {
-            console.log(chalk.gray('Most common skip reasons:'));
-            reasons.forEach((line) => console.log(chalk.gray(`  • ${line}`)));
+    .action(
+      withErrorHandling(async (options: ReviewCommandOptions) => {
+        try {
+          const { config, projectRoot, configPath } = await loadConfigWithMeta(options.config);
+          const scanner = new Scanner(config, { workspaceRoot: projectRoot });
+          const summary = scanner.scan({ scanCalls: options.scanCalls }) as BucketedScanSummary;
+          const buckets = summary.buckets ?? {};
+          const needsReview = buckets.needsReview ?? [];
+          const skipped = buckets.skipped ?? [];
+
+          if (options.json) {
+            console.log(JSON.stringify({ needsReview, skipped }, null, 2));
+            return;
           }
-          return;
-        }
 
-        if (!process.stdout.isTTY || process.env.CI === 'true') {
-          console.log(chalk.red('Interactive review requires a TTY. Use --json for non-interactive output.'));
-          process.exitCode = 1;
-          return;
-        }
+          if (!needsReview.length) {
+            console.log(chalk.green('No borderline candidates detected.'));
+            const reasons = summarizeSkipReasons(skipped);
+            if (reasons.length) {
+              console.log(chalk.gray('Most common skip reasons:'));
+              reasons.forEach((line) => console.log(chalk.gray(`  • ${line}`)));
+            }
+            return;
+          }
 
-        const limit = normalizeLimit(options.limit);
-        const queue = needsReview.slice(0, limit);
-        console.log(
-          chalk.blue(
-            `Reviewing ${queue.length} of ${needsReview.length} candidate${needsReview.length === 1 ? '' : 's'} (limit=${limit}).`
-          )
-        );
-
-        const allowPatterns: string[] = [];
-        const denyPatterns: string[] = [];
-
-        for (const candidate of queue) {
-          printCandidate(candidate);
-          const action = await promptAction();
-          if (action === 'stop') {
-            break;
+          if (!process.stdout.isTTY || process.env.CI === 'true') {
+            console.log(chalk.red('Interactive review requires a TTY. Use --json for non-interactive output.'));
+            process.exitCode = 1;
+            return;
           }
-          if (action === 'skip') {
-            continue;
+
+          const limit = normalizeLimit(options.limit);
+          const queue = needsReview.slice(0, limit);
+          console.log(
+            chalk.blue(
+              `Reviewing ${queue.length} of ${needsReview.length} candidate${needsReview.length === 1 ? '' : 's'} (limit=${limit}).`
+            )
+          );
+
+          const allowPatterns: string[] = [];
+          const denyPatterns: string[] = [];
+
+          for (const candidate of queue) {
+            printCandidate(candidate);
+            const action = await promptAction();
+            if (action === 'stop') {
+              break;
+            }
+            if (action === 'skip') {
+              continue;
+            }
+            const pattern = literalToRegexPattern(candidate.text);
+            if (action === 'allow') {
+              allowPatterns.push(pattern);
+              console.log(chalk.green(`  → Queued ${pattern} for allowPatterns`));
+            } else if (action === 'deny') {
+              denyPatterns.push(pattern);
+              console.log(chalk.yellow(`  → Queued ${pattern} for denyPatterns`));
+            }
           }
-          const pattern = literalToRegexPattern(candidate.text);
-          if (action === 'allow') {
-            allowPatterns.push(pattern);
-            console.log(chalk.green(`  → Queued ${pattern} for allowPatterns`));
-          } else if (action === 'deny') {
-            denyPatterns.push(pattern);
-            console.log(chalk.yellow(`  → Queued ${pattern} for denyPatterns`));
+
+          if (!allowPatterns.length && !denyPatterns.length) {
+            console.log(chalk.gray('No config changes requested.'));
+            return;
           }
-        }
 
-        if (!allowPatterns.length && !denyPatterns.length) {
-          console.log(chalk.gray('No config changes requested.'));
-          return;
-        }
+          const { allowAdded, denyAdded, wrote } = await writeExtractionOverrides(
+            configPath,
+            allowPatterns,
+            denyPatterns
+          );
 
-        const { allowAdded, denyAdded, wrote } = await writeExtractionOverrides(
-          configPath,
-          allowPatterns,
-          denyPatterns
-        );
+          if (!wrote) {
+            console.log(chalk.gray('Patterns already existed; config unchanged.'));
+            return;
+          }
 
-        if (!wrote) {
-          console.log(chalk.gray('Patterns already existed; config unchanged.'));
-          return;
+          console.log(
+            chalk.green(
+              `Updated ${relativize(configPath)} (${allowAdded} allow, ${denyAdded} deny pattern${
+                allowAdded + denyAdded === 1 ? '' : 's'
+              } added).`
+            )
+          );
+        } catch (error) {
+          const message = error instanceof Error ? error.message : String(error);
+          throw new CliError(`Review failed: ${message}`);
         }
-
-        console.log(
-          chalk.green(
-            `Updated ${relativize(configPath)} (${allowAdded} allow, ${denyAdded} deny pattern${
-              allowAdded + denyAdded === 1 ? '' : 's'
-            } added).`
-          )
-        );
-      } catch (error) {
-        console.error(chalk.red('Review failed:'), (error as Error).message);
-        process.exitCode = 1;
-      }
-    });
+      })
+    );
 }
 
 function relativize(filePath: string): string {

package/src/commands/scaffold-adapter.ts

@@ -2,10 +2,11 @@ import { Command } from 'commander';
 import inquirer from 'inquirer';
 import chalk from 'chalk';
 import { diagnoseWorkspace, loadConfig } from '@i18nsmith/core';
-import { scaffoldTranslationContext, scaffoldI18next, ScaffoldResult } from '../utils/scaffold.js';
+import { scaffoldTranslationContext, scaffoldI18next } from '../utils/scaffold.js';
 import { readPackageJson, hasDependency } from '../utils/pkg.js';
 import { detectPackageManager, installDependencies } from '../utils/package-manager.js';
 import { maybeInjectProvider } from '../utils/provider-injector.js';
+import { CliError, withErrorHandling } from '../utils/errors.js';
 
 interface ScaffoldCommandOptions {
   type?: 'custom' | 'react-i18next';
@@ -44,7 +45,8 @@ export function registerScaffoldAdapter(program: Command) {
     .option('--install-deps', 'Automatically install adapter dependencies when missing', false)
     .option('--dry-run', 'Preview provider injection changes without modifying files', false)
     .option('--no-skip-if-detected', 'Force scaffolding even if existing adapters/providers are detected')
-    .action(async (options: ScaffoldCommandOptions) => {
+    .action(
+      withErrorHandling(async (options: ScaffoldCommandOptions) => {
       console.log(chalk.blue('Scaffolding translation resources...'));
 
       const answers = await inquirer.prompt<ScaffoldAnswers>([
@@ -221,9 +223,11 @@ export function registerScaffoldAdapter(program: Command) {
           }
         }
       } catch (error) {
-        console.error(chalk.red('Failed to scaffold adapter:'), (error as Error).message);
+        const message = error instanceof Error ? error.message : String(error);
+        throw new CliError(`Failed to scaffold adapter: ${message}`);
       }
-    });
+    })
+    );
 }
 
 async function detectExistingRuntime(): Promise<string | null> {

package/src/commands/scan.ts

@@ -4,6 +4,7 @@ import chalk from 'chalk';
 import type { Command } from 'commander';
 import { loadConfigWithMeta, Scanner } from '@i18nsmith/core';
 import type { ScanCandidate } from '@i18nsmith/core';
+import { CliError, withErrorHandling } from '../utils/errors.js';
 
 interface ScanOptions {
   config?: string;
@@ -54,72 +55,74 @@ export function registerScan(program: Command) {
     .option('--list-files', 'List the files that were scanned', false)
     .option('--include <patterns...>', 'Override include globs from config (comma or space separated)', collectTargetPatterns, [])
     .option('--exclude <patterns...>', 'Override exclude globs from config (comma or space separated)', collectTargetPatterns, [])
-    .action(async (options: ScanOptions) => {
-      console.log(chalk.blue('Starting scan...'));
+    .action(
+      withErrorHandling(async (options: ScanOptions) => {
+        console.log(chalk.blue('Starting scan...'));
 
-      try {
-        const { config, projectRoot, configPath } = await loadConfigWithMeta(options.config);
-        
-        // Inform user if config was found in a parent directory
-        const cwd = process.cwd();
-        if (projectRoot !== cwd) {
-          console.log(chalk.gray(`Config found at ${path.relative(cwd, configPath)}`));
-          console.log(chalk.gray(`Using project root: ${projectRoot}\n`));
-        }
-        
-        if (options.include?.length) {
-          config.include = options.include;
-        }
-        if (options.exclude?.length) {
-          config.exclude = options.exclude;
-        }
-        const scanner = new Scanner(config, { workspaceRoot: projectRoot });
-        const summary = scanner.scan();
+        try {
+          const { config, projectRoot, configPath } = await loadConfigWithMeta(options.config);
 
-        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(`Scan report written to ${outputPath}`));
-        }
+          // Inform user if config was found in a parent directory
+          const cwd = process.cwd();
+          if (projectRoot !== cwd) {
+            console.log(chalk.gray(`Config found at ${path.relative(cwd, configPath)}`));
+            console.log(chalk.gray(`Using project root: ${projectRoot}\n`));
+          }
 
-        if (options.json) {
-          console.log(JSON.stringify(summary, null, 2));
-          return;
-        }
+          if (options.include?.length) {
+            config.include = options.include;
+          }
+          if (options.exclude?.length) {
+            config.exclude = options.exclude;
+          }
+          const scanner = new Scanner(config, { workspaceRoot: projectRoot });
+          const summary = scanner.scan();
 
-        console.log(
-          chalk.green(
-            `Scanned ${summary.filesScanned} file${summary.filesScanned === 1 ? '' : 's'} and found ${summary.candidates.length} candidate${summary.candidates.length === 1 ? '' : 's'}.`
-          )
-        );
+          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(`Scan report written to ${outputPath}`));
+          }
 
-        if (summary.candidates.length === 0) {
-          console.log(chalk.yellow('No translatable strings found.'));
-          return;
-        }
+          if (options.json) {
+            console.log(JSON.stringify(summary, null, 2));
+            return;
+          }
+
+          console.log(
+            chalk.green(
+              `Scanned ${summary.filesScanned} file${summary.filesScanned === 1 ? '' : 's'} and found ${summary.candidates.length} candidate${summary.candidates.length === 1 ? '' : 's'}.`
+            )
+          );
+
+          if (summary.candidates.length === 0) {
+            console.log(chalk.yellow('No translatable strings found.'));
+            return;
+          }
 
-        printCandidateTable(summary.candidates);
+          printCandidateTable(summary.candidates);
 
-        if (options.listFiles) {
-          if (summary.filesExamined.length === 0) {
-            console.log(chalk.yellow('No files matched the configured include/exclude patterns.'));
-          } else {
-            console.log(chalk.blue(`Files scanned (${summary.filesExamined.length}):`));
-            const preview = summary.filesExamined.slice(0, 200);
-            preview.forEach((file) => console.log(`  • ${file}`));
-            if (summary.filesExamined.length > preview.length) {
-              console.log(
-                chalk.gray(
-                  `  ...and ${summary.filesExamined.length - preview.length} more. Use --target to narrow the list.`
-                )
-              );
+          if (options.listFiles) {
+            if (summary.filesExamined.length === 0) {
+              console.log(chalk.yellow('No files matched the configured include/exclude patterns.'));
+            } else {
+              console.log(chalk.blue(`Files scanned (${summary.filesExamined.length}):`));
+              const preview = summary.filesExamined.slice(0, 200);
+              preview.forEach((file) => console.log(`  • ${file}`));
+              if (summary.filesExamined.length > preview.length) {
+                console.log(
+                  chalk.gray(
+                    `  ...and ${summary.filesExamined.length - preview.length} more. Use --target to narrow the list.`
+                  )
+                );
+              }
             }
           }
+        } catch (error) {
+          const message = error instanceof Error ? error.message : String(error);
+          throw new CliError(`Scan failed: ${message}`);
         }
-      } catch (error) {
-        console.error(chalk.red('Scan failed:'), (error as Error).message);
-        process.exitCode = 1;
-      }
-    });
+      })
+    );
 }