i18nsmith 0.3.3 → 0.4.0

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 { runAdapterPreflight } from '../utils/adapter-preflight.js';
 import { CliError, withErrorHandling } from '../utils/errors.js';
 
 interface ScanOptions {
@@ -59,6 +60,12 @@ export function registerRename(program: Command): void {
 
         try {
           const config = await loadConfig(options.config);
+
+          // Run preflight checks for write operations
+          if (writeEnabled) {
+            await runAdapterPreflight();
+          }
+
           const renamer = new KeyRenamer(config);
           const summary = await renamer.rename(oldKey, newKey, {
             write: options.write,
@@ -113,6 +120,12 @@ export function registerRename(program: Command): void {
 
         try {
           const config = await loadConfig(options.config);
+
+          // Run preflight checks for write operations
+          if (options.write) {
+            await runAdapterPreflight();
+          }
+
           const mappings = await loadRenameMappings(options.map);
           const renamer = new KeyRenamer(config);
           const summary = await renamer.renameBatch(mappings, { write: options.write, diff: options.diff });

package/src/commands/review.ts

@@ -138,7 +138,7 @@ export function registerReview(program: Command) {
       withErrorHandling(async (options: ReviewCommandOptions) => {
         try {
           const { config, projectRoot, configPath } = await loadConfigWithMeta(options.config);
-          const scanner = new Scanner(config, { workspaceRoot: projectRoot });
+          const scanner = await Scanner.create(config, { workspaceRoot: projectRoot });
           const summary = scanner.scan({ scanCalls: options.scanCalls }) as BucketedScanSummary;
           const buckets = summary.buckets ?? {};
           const needsReview = buckets.needsReview ?? [];

package/src/commands/scan.ts

@@ -75,7 +75,10 @@ export function registerScan(program: Command) {
           if (options.exclude?.length) {
             config.exclude = options.exclude;
           }
-          const scanner = new Scanner(config, { workspaceRoot: projectRoot });
+          // Use factory that registers framework adapters (React/Vue) so
+          // scans include files handled by adapters. Backwards compatible
+          // API: Scanner.create will return a scanner with adapters wired.
+          const scanner = await Scanner.create(config, { workspaceRoot: projectRoot });
           const summary = scanner.scan();
 
           if (options.report) {

package/src/commands/sync.ts

@@ -554,12 +554,15 @@ export function registerSync(program: Command) {
 
         // Handle --auto-rename-suspicious
         if (options.autoRenameSuspicious && summary.suspiciousKeys.length > 0) {
-          await handleAutoRenameSuspicious(
+          const renameDiffs = await handleAutoRenameSuspicious(
             summary,
             options,
             config,
             projectRoot
           );
+          if (renameDiffs && renameDiffs.length > 0) {
+            summary.renameDiffs = renameDiffs;
+          }
         }
 
         // Handle --rewrite-shape
@@ -827,7 +830,7 @@ function printSyncSummary(summary: SyncSummary) {
     } else {
       console.log(
         chalk.gray(
-          "Use --assume key1,key2 to prevent false positives for known runtime-only translation keys."
+          "Use --assume to prevent false positives for known runtime-only translation keys."
         )
       );
     }
@@ -857,7 +860,7 @@ async function handleAutoRenameSuspicious(
   options: SyncCommandOptions,
   config: Awaited<ReturnType<typeof loadConfig>>,
   projectRoot: string
-) {
+): Promise<any[] | undefined> {
   console.log(chalk.blue("\n📝 Auto-rename suspicious keys analysis:"));
 
   // Get existing keys from locale data to check for conflicts
@@ -1008,6 +1011,23 @@ async function handleAutoRenameSuspicious(
       chalk.gray("  Run with --write to apply safe proposals automatically.")
     );
   }
+
+  // Generate diffs for preview mode
+  if (options.diff && report.safeProposals.length > 0) {
+    const mappings = report.safeProposals.map((proposal) => ({
+      from: proposal.originalKey,
+      to: proposal.proposedKey,
+    }));
+
+    const renamer = new KeyRenamer(config, { workspaceRoot: projectRoot });
+    const diffSummary = await renamer.renameBatch(mappings, {
+      write: false,
+      diff: true,
+      allowConflicts: true,
+    });
+
+    return diffSummary.diffs;
+  }
 }
 
 async function handleRewriteShape(

package/src/commands/transform.ts

@@ -1,5 +1,7 @@
 import fs from 'fs/promises';
 import path from 'path';
+import { createRequire } from 'module';
+import { fileURLToPath } from 'url';
 import chalk from 'chalk';
 import type { Command } from 'commander';
 import { loadConfigWithMeta } from '@i18nsmith/core';
@@ -8,6 +10,8 @@ import type { TransformProgress, TransformSummary } from '@i18nsmith/transformer
 import { printLocaleDiffs, writeLocaleDiffPatches } from '../utils/diff-utils.js';
 import { applyPreviewFile, writePreviewFile } from '../utils/preview.js';
 import { CliError, withErrorHandling } from '../utils/errors.js';
+import inquirer from 'inquirer';
+import { detectPackageManager, installDependencies } from '../utils/package-manager.js';
 
 interface TransformOptions {
   config?: string;
@@ -32,6 +36,7 @@ const collectTargetPatterns = (value: string | string[], previous: string[]) =>
   return [...previous, ...tokens];
 };
 
+/* Re-enabled after framework migration stabilization */
 function printTransformSummary(summary: TransformSummary) {
   const counts = summary.candidates.reduce(
     (acc, c) => {
@@ -68,7 +73,8 @@ function printTransformSummary(summary: TransformSummary) {
 
   console.table(preview);
 
-  const pending = summary.candidates.filter((candidate) => candidate.status === 'pending').length;
+  const pending = summary.candidateStats?.pending
+    ?? summary.candidates.filter((candidate) => candidate.status === 'pending').length;
   if (pending > 0) {
     console.log(
       chalk.yellow(
@@ -97,6 +103,13 @@ function printTransformSummary(summary: TransformSummary) {
     console.log(chalk.yellow('Skipped items:'));
     summary.skippedFiles.forEach((item) => console.log(`  • ${item.filePath}: ${item.reason}`));
   }
+
+  if (summary.skippedReasons && Object.keys(summary.skippedReasons).length) {
+    console.log(chalk.yellow('Skipped reasons:'));
+    (Object.entries(summary.skippedReasons) as [string, number][])
+      .sort((a, b) => b[1] - a[1])
+      .forEach(([reason, count]) => console.log(`  • ${reason}: ${count}`));
+  }
 }
 
 function createProgressLogger() {
@@ -194,7 +207,7 @@ export function registerTransform(program: Command) {
     .option('--apply-preview <path>', 'Apply a previously saved transform preview JSON file safely')
     .action(
       withErrorHandling(async (options: TransformOptions) => {
-      if (options.applyPreview) {
+        if (options.applyPreview) {
         await applyPreviewFile('transform', options.applyPreview);
         return;
       }
@@ -225,6 +238,45 @@ export function registerTransform(program: Command) {
           console.log(chalk.gray(`Config found at ${path.relative(cwd, configPath)}`));
           console.log(chalk.gray(`Using project root: ${projectRoot}\n`));
         }
+
+        // Proactively check if Vue files are targeted but the parser is missing
+        const includesVue = config.include?.some(pattern => pattern.includes('.vue')) ?? false;
+        let isVueParserAvailable = false;
+        try {
+          const require = createRequire(import.meta.url);
+          const moduleDir = path.dirname(fileURLToPath(import.meta.url));
+          require.resolve('vue-eslint-parser', { paths: [projectRoot, moduleDir] });
+          isVueParserAvailable = true;
+        } catch {
+          isVueParserAvailable = false;
+        }
+
+        if (includesVue && !isVueParserAvailable) {
+          console.log(chalk.yellow('⚠️  Vue files detected but "vue-eslint-parser" is not installed.'));
+          console.log(chalk.yellow('   Verification of Vue templates might be incomplete or fail.'));
+          
+          if (process.stdout.isTTY) {
+            const { install } = await inquirer.prompt<{ install: boolean }>([
+              {
+                type: 'confirm',
+                name: 'install',
+                message: 'Do you want to install "vue-eslint-parser" (dev dependency) now?',
+                default: true,
+              },
+            ]);
+
+            if (install) {
+              const pm = await detectPackageManager(projectRoot);
+              const cmd = pm === 'npm' ? 'npm install --save-dev vue-eslint-parser' : `${pm} add -D vue-eslint-parser`;
+              console.log(chalk.gray(`> ${cmd}`));
+              // We use installDependencies helper but need to pass dev flag args if not flexible
+              // The helper seems simple: const args = manager === 'npm' ? ['install', ...deps] : ['add', ...deps];
+              // So for dev we need to include -D in deps
+              await installDependencies(pm, ['-D', 'vue-eslint-parser'], projectRoot);
+              console.log(chalk.green('✔ Installed. Continuing...'));
+            }
+          }
+        }
         
         const transformer = new Transformer(config, { workspaceRoot: projectRoot });
         const progressLogger = createProgressLogger();

package/src/commands/translate/csv-handler.ts

@@ -78,6 +78,8 @@ export async function handleCsvExport(options: TranslateCommandOptions): Promise
     const plan = await translationService.buildPlan({
       locales: options.locales,
       force: options.force,
+      // Export should represent what the UI calls "missing" (usually includes empty strings)
+      treatEmptyAsMissing: true,
     });
 
     if (!plan.totalTasks) {
@@ -165,7 +167,6 @@ export async function handleCsvImport(options: TranslateCommandOptions): Promise
     // Parse header
     const headerFields = parseCsvLine(lines[0]);
     const keyIdx = headerFields.indexOf('key');
-    const sourceLocaleIdx = headerFields.indexOf('sourceLocale');
     const sourceValueIdx = headerFields.indexOf('sourceValue');
     const targetLocaleIdx = headerFields.indexOf('targetLocale');
     const translatedValueIdx = headerFields.indexOf('translatedValue');

package/src/e2e.test.ts

@@ -281,13 +281,13 @@ describe('E2E Fixture Tests', () => {
   const renameMap = parseRenameMap(mapContents);
   const helloKey = renameMap['Hello World'];
   expect(helloKey).toBeDefined();
-  const submitKey = renameMap['buttons.submit'];
-  expect(submitKey).toBeDefined();
+  const saveKey = renameMap['Save'];
+  expect(saveKey).toBeDefined();
 
   expect(enLocale).toHaveProperty(helloKey!);
   expect(frLocale).toHaveProperty(helloKey!);
-  expect(enLocale).toHaveProperty(submitKey!);
-  expect(frLocale).toHaveProperty(submitKey!);
+  expect(enLocale).toHaveProperty(saveKey!);
+  expect(frLocale).toHaveProperty(saveKey!);
 
   const sourceFile = await fs.readFile(path.join(fixtureDir, 'src', 'BadKeys.tsx'), 'utf8');
   expect(sourceFile).toContain(`t('${helloKey!}')`);

package/src/fixtures/suspicious-keys/locales/en.json

@@ -1,11 +1,11 @@
 {
-  "Hello World": "Hello World",
-  "Welcome to our app!": "Welcome to our app!",
-  "Click Here": "Click Here",
-  "Save": "Save",
-  "The Quick Brown Fox": "The Quick Brown Fox",
-  "When to use this feature:": "When to use this feature:",
+  "common.badkeys.buttons-submit.3b94a6": "Submit",
+  "common.badkeys.click-here.788c1e": "Click Here",
+  "common.badkeys.hello-world.66bf53": "Hello World",
+  "common.badkeys.save.a495bf": "Save",
+  "common.badkeys.welcome-to-our-app.a53bf0": "Welcome to our app!",
+  "common.title": "Application Title",
   "proper.namespaced.key": "This is a properly namespaced key",
-  "buttons.submit": "Submit",
-  "common.title": "Application Title"
+  "The Quick Brown Fox": "The Quick Brown Fox",
+  "When to use this feature:": "When to use this feature:"
 }

package/src/fixtures/suspicious-keys/locales/fr.json

@@ -1,11 +1,11 @@
 {
-  "Hello World": "Bonjour le monde",
-  "Welcome to our app!": "Bienvenue dans notre application!",
-  "Click Here": "Cliquez ici",
-  "Save": "Sauvegarder",
-  "The Quick Brown Fox": "Le Renard Brun Rapide",
-  "When to use this feature:": "Quand utiliser cette fonctionnalité:",
+  "common.badkeys.buttons-submit.3b94a6": "Soumettre",
+  "common.badkeys.click-here.788c1e": "Cliquez ici",
+  "common.badkeys.hello-world.66bf53": "Bonjour le monde",
+  "common.badkeys.save.a495bf": "Sauvegarder",
+  "common.badkeys.welcome-to-our-app.a53bf0": "Bienvenue dans notre application!",
+  "common.title": "Titre de l'application",
   "proper.namespaced.key": "Ceci est une clé correctement nommée",
-  "buttons.submit": "Soumettre",
-  "common.title": "Titre de l'application"
+  "The Quick Brown Fox": "Le Renard Brun Rapide",
+  "When to use this feature:": "Quand utiliser cette fonctionnalité:"
 }