create-quiver 0.15.3 → 0.16.0

package/src/create-quiver/commands/plan.js

@@ -1,4 +1,5 @@
 const { relativePosixPath } = require('../lib/paths');
+const { formatStatus, translatorForHuman } = require('../lib/i18n/read-only-format');
 const { filterSlicesForExecution, resolveProjectState } = require('../lib/project-state-resolver');
 const { topoSort } = require('../lib/slice-graph');
 
@@ -147,13 +148,21 @@ function collectPlan(repoRoot, options = {}) {
 }
 
 function formatHumanPlan(report, options = {}) {
+  const translator = translatorForHuman(options);
   const unicode = Boolean(options.unicode) || /UTF-8/i.test(process.env.LANG || '');
-  const title = options.onlyReady ? 'Ready slices' : (report.include_completed ? 'Quiver plan (including completed)' : 'Quiver plan');
+  const title = options.onlyReady
+    ? translator.t('plan.title.ready')
+    : (report.include_completed ? translator.t('plan.title.including_completed') : translator.t('plan.title'));
   const pathSeparator = unicode ? ' → ' : ' -> ';
-  const lines = [title, `Total hours: ${report.total_hours}`, `Critical path: ${report.critical_path.length > 0 ? report.critical_path.join(pathSeparator) : '-'}`, ''];
+  const lines = [
+    title,
+    `${translator.t('plan.total_hours')}: ${report.total_hours}`,
+    `${translator.t('plan.critical_path')}: ${report.critical_path.length > 0 ? report.critical_path.join(pathSeparator) : '-'}`,
+    '',
+  ];
 
   if (report.plan.length === 0) {
-    lines.push(report.include_completed ? 'No slices found for the selected filters.' : 'No pending slices found.');
+    lines.push(report.include_completed ? translator.t('plan.empty.filtered') : translator.t('plan.empty.pending'));
     return `${lines.join('\n')}\n`;
   }
 
@@ -163,21 +172,35 @@ function formatHumanPlan(report, options = {}) {
     item.ref,
     item.title,
     `${item.hours}`,
-    item.status,
+    formatStatus(item.status, translator),
   ]);
 
+  const headers = [
+    translator.t('plan.header.index'),
+    translator.t('plan.header.ticket'),
+    translator.t('plan.header.spec_slice'),
+    translator.t('plan.header.title'),
+    translator.t('plan.header.hours'),
+    translator.t('plan.header.status'),
+  ];
   const widths = rows[0].map((_, colIndex) => Math.max(
     rows.reduce((max, row) => Math.max(max, row[colIndex].length), 0),
-    ['[N]', 'TICKET', 'SPEC/SLICE', 'TITLE', 'HOURS', 'STATUS'][colIndex].length,
+    headers[colIndex].length,
   ));
 
-  lines.push(['[N]', 'TICKET', 'SPEC/SLICE', 'TITLE', 'HOURS', 'STATUS'].map((label, index) => label.padEnd(widths[index])).join('  '));
+  lines.push(headers.map((label, index) => label.padEnd(widths[index])).join('  '));
   lines.push(widths.map((width) => '-'.repeat(width)).join('  '));
 
   for (const row of rows) {
     lines.push(row.map((value, index) => value.padEnd(widths[index])).join('  '));
   }
 
+  const missingEstimateCount = report.plan.filter((item) => Number(item.hours) <= 0).length;
+  if (missingEstimateCount > 0) {
+    lines.push('');
+    lines.push(translator.t('plan.missing_estimates_note', { count: missingEstimateCount }));
+  }
+
   return `${lines.join('\n')}\n`;
 }
 

package/src/create-quiver/commands/spec.js

@@ -5,6 +5,7 @@ const { checkHandoff } = require('../lib/handoff');
 const { openEditor } = require('../lib/cli/editor');
 const { selectOption } = require('../lib/cli/selectors');
 const { createUx } = require('../lib/cli/ux');
+const { createTranslator } = require('../lib/i18n/catalog');
 const { parseJsonWithComments } = require('../lib/json');
 const { assertPathInsideRoot, validateProjectRelativePaths } = require('../lib/paths');
 const { buildApprovalCandidateReport, formatReviewSummary } = require('../lib/ai/approval-candidates');
@@ -23,18 +24,22 @@ function toRelativePosix(root, filePath) {
   return path.relative(root, filePath).split(path.sep).join('/');
 }
 
-function readInputText(repoRoot, inputPath) {
+function readInputText(repoRoot, inputPath, options = {}) {
+  const translator = createTranslator(options.language);
   const resolved = path.resolve(repoRoot, inputPath || '');
   if (!inputPath || !fs.existsSync(resolved)) {
-    throw new Error(formatError(`missing reviewed and approved plan input: ${inputPath || '<default>'}`));
+    throw new Error(formatError(translator.t('spec.error.missing_approved_plan_input', {
+      path: inputPath || '<default>',
+    })));
   }
   return fs.readFileSync(resolved, 'utf8');
 }
 
-function resolveSpecDir(repoRoot, specInput) {
+function resolveSpecDir(repoRoot, specInput, options = {}) {
+  const translator = createTranslator(options.language);
   const value = String(specInput || '').trim();
   if (!value) {
-    throw new Error(formatError('missing spec directory. Use: npx create-quiver spec validate specs/<spec-slug>'));
+    throw new Error(formatError(translator.t('spec.error.missing_directory_validate')));
   }
 
   const candidates = [
@@ -50,7 +55,7 @@ function resolveSpecDir(repoRoot, specInput) {
     }
   }
 
-  throw new Error(formatError(`spec directory not found: ${value}`));
+  throw new Error(formatError(translator.t('spec.error.directory_not_found', { path: value })));
 }
 
 function findSliceJsonFiles(specDir) {
@@ -141,7 +146,7 @@ function missingGitFields(git) {
 
 function buildSpecValidationReport(repoRoot, specInput, options = {}) {
   const strict = options.strict === true;
-  const specDir = resolveSpecDir(repoRoot, specInput);
+  const specDir = resolveSpecDir(repoRoot, specInput, options);
   const relativeSpecDir = toRelativePosix(repoRoot, specDir);
   const specSlug = path.basename(specDir);
   const errors = [];
@@ -270,45 +275,49 @@ function buildSpecValidationReport(repoRoot, specInput, options = {}) {
   };
 }
 
-function formatSpecValidationReport(report) {
+function formatSpecValidationReport(report, options = {}) {
+  const translator = createTranslator(options.language);
   const lines = [
-    'Quiver spec validation',
-    `Spec: ${report.specDir}`,
-    `Slices: ${report.slices}`,
-    `Strict: ${report.strict ? 'yes' : 'no'}`,
+    translator.t('spec_validate.title'),
+    translator.t('spec_validate.spec', { path: report.specDir }),
+    translator.t('spec_validate.slices', { count: report.slices }),
+    translator.t('spec_validate.strict', { value: report.strict ? translator.t('common.yes') : translator.t('common.no') }),
   ];
 
   if (report.checked.length > 0) {
-    lines.push('Checked files:');
+    lines.push(translator.t('spec_validate.checked_files'));
     for (const file of report.checked) {
       lines.push(`- ${file}`);
     }
   }
 
   if (report.warnings.length > 0) {
-    lines.push('Warnings:');
+    lines.push(translator.t('spec_validate.warnings'));
     for (const warning of report.warnings) {
       lines.push(`- ${warning}`);
     }
   }
 
   if (report.errors.length > 0) {
-    lines.push('Errors:');
+    lines.push(translator.t('spec_validate.errors'));
     for (const error of report.errors) {
       lines.push(`- ${error}`);
     }
   }
 
-  lines.push(report.ok ? 'PASS: spec validation passed.' : 'FAIL: spec validation failed.');
+  lines.push(report.ok ? translator.t('spec_validate.result.pass') : translator.t('spec_validate.result.fail'));
   return `${lines.join('\n')}\n`;
 }
 
 function runValidateSpec(repoRoot, specInput, options = {}) {
   const report = buildSpecValidationReport(repoRoot, specInput, options);
-  process.stdout.write(formatSpecValidationReport(report));
+  process.stdout.write(formatSpecValidationReport(report, options));
 
   if (!report.ok) {
-    const error = new Error(formatError(`spec validate failed for ${report.specDir}`));
+    const translator = createTranslator(options.language);
+    const error = new Error(formatError(translator.t('spec.error.validate_failed', {
+      path: report.specDir,
+    })));
     error.code = 'SPEC_VALIDATE_FAILED';
     error.details = report;
     throw error;
@@ -320,7 +329,7 @@ function runValidateSpec(repoRoot, specInput, options = {}) {
 function buildSpecCreatePreview(repoRoot, options = {}) {
   const resolved = resolveReviewedTechnicalPlanInput(repoRoot, options.input || undefined);
   const inputPath = resolved.inputPath;
-  const inputText = readInputText(repoRoot, inputPath);
+  const inputText = readInputText(repoRoot, inputPath, options);
   const manifest = buildSpecGenerationManifest({
     inputPath,
     inputText,
@@ -331,7 +340,9 @@ function buildSpecCreatePreview(repoRoot, options = {}) {
   const relativeSpecDir = toRelativePosix(repoRoot, preview.specDir);
 
   if (fs.existsSync(preview.specDir)) {
-    throw new Error(formatError(`spec directory already exists: ${relativeSpecDir}`));
+    throw new Error(formatError(createTranslator(options.language).t('spec.error.directory_already_exists', {
+      path: relativeSpecDir,
+    })));
   }
 
   return {
@@ -342,9 +353,10 @@ function buildSpecCreatePreview(repoRoot, options = {}) {
   };
 }
 
-function formatNextCommands(specDir) {
+function formatNextCommands(specDir, options = {}) {
+  const translator = createTranslator(options.language);
   return [
-    'Next safe commands:',
+    translator.t('spec_create.next_safe_commands'),
     `- npx create-quiver spec start ${specDir}`,
     `- npx create-quiver spec status ${specDir}`,
     '- npx create-quiver next',
@@ -352,14 +364,15 @@ function formatNextCommands(specDir) {
   ];
 }
 
-function formatSpecCreateDryRun(preview) {
+function formatSpecCreateDryRun(preview, options = {}) {
+  const translator = createTranslator(options.language);
   const lines = [
-    'Quiver spec create dry-run',
-    `Spec slug: ${preview.manifest.slug}`,
-    `Title: ${preview.manifest.title}`,
-    `Input file: ${preview.inputPath}`,
-    `Target: ${preview.relativeSpecDir}`,
-    `Planned files: ${preview.preview.files.length}`,
+    translator.t('spec_create.title.dry_run'),
+    translator.t('spec_create.label.slug', { slug: preview.manifest.slug }),
+    translator.t('spec_create.label.title', { title: preview.manifest.title }),
+    translator.t('spec_create.label.input_file', { path: preview.inputPath }),
+    translator.t('spec_create.label.target', { path: preview.relativeSpecDir }),
+    translator.t('spec_create.label.planned_files', { count: preview.preview.files.length }),
   ];
 
   for (const file of preview.preview.files) {
@@ -367,28 +380,29 @@ function formatSpecCreateDryRun(preview) {
   }
 
   if (preview.reviewRequested) {
-    lines.push('Review requested: yes (dry-run preview only; no editor opened and no files written).');
+    lines.push(translator.t('spec_create.review_requested.dry_run'));
   }
 
-  lines.push(...formatNextCommands(preview.relativeSpecDir));
-  lines.push('No files will be written in dry-run mode.');
+  lines.push(...formatNextCommands(preview.relativeSpecDir, options));
+  lines.push(translator.t('spec_create.no_files_written_dry_run'));
   return `${lines.join('\n')}\n`;
 }
 
-function formatSpecCreateResult(result, repoRoot) {
+function formatSpecCreateResult(result, repoRoot, options = {}) {
+  const translator = createTranslator(options.language);
   const relativeSpecDir = toRelativePosix(repoRoot, result.specDir);
   const lines = [
-    'Quiver spec created',
-    `Spec slug: ${result.manifest.slug}`,
-    `Target: ${relativeSpecDir}`,
-    `Files written: ${result.files.length}`,
+    translator.t('spec_create.title.created'),
+    translator.t('spec_create.label.slug', { slug: result.manifest.slug }),
+    translator.t('spec_create.label.target', { path: relativeSpecDir }),
+    translator.t('spec_create.label.files_written', { count: result.files.length }),
   ];
 
   for (const filePath of result.files) {
     lines.push(`- ${toRelativePosix(repoRoot, filePath)}`);
   }
 
-  lines.push(...formatNextCommands(relativeSpecDir));
+  lines.push(...formatNextCommands(relativeSpecDir, options));
   return `${lines.join('\n')}\n`;
 }
 
@@ -397,24 +411,27 @@ function createSpecReviewFile(preview, options = {}) {
   const reviewDir = options.reviewDir || fs.mkdtempSync(path.join(os.tmpdir(), 'quiver-spec-create-review-'));
   const reviewPath = path.join(reviewDir, 'spec-create-preview.md');
   fs.mkdirSync(reviewDir, { recursive: true });
-  fs.writeFileSync(reviewPath, formatSpecCreateDryRun({ ...preview, reviewRequested: false }));
+  fs.writeFileSync(reviewPath, formatSpecCreateDryRun({ ...preview, reviewRequested: false }, options));
   return reviewPath;
 }
 
-const SPEC_METHODOLOGY_OPTIONS = Object.freeze([
-  {
-    label: 'WDD + SDD',
-    value: 'wdd-sdd',
-    hint: 'Workflow Driven Development + Spec Driven Development',
-    default: true,
-  },
-]);
+function specMethodologyOptions(translator) {
+  return [
+    {
+      label: 'WDD + SDD',
+      value: 'wdd-sdd',
+      hint: translator.t('spec_create.methodology.wdd_sdd.hint'),
+      default: true,
+    },
+  ];
+}
 
 async function resolveInteractiveSpecCreateOptions(repoRoot, preview, options = {}) {
   if (options.interactive !== true) {
     return options;
   }
 
+  const translator = createTranslator(options.language);
   const ux = options.ux || createUx({
     interactive: true,
     promptConfirm: options.promptConfirm,
@@ -425,7 +442,7 @@ async function resolveInteractiveSpecCreateOptions(repoRoot, preview, options =
   });
 
   if (!ux.mode.usePrompts) {
-    throw new Error(formatError('spec create --interactive requires an interactive TTY. Use --input, --spec, --review, and --dry-run for non-interactive automation.'));
+    throw new Error(formatError(translator.t('spec_create.error.interactive_tty_required')));
   }
 
   const selectorOptions = {
@@ -443,20 +460,20 @@ async function resolveInteractiveSpecCreateOptions(repoRoot, preview, options =
   const technicalPlanCandidates = buildApprovalCandidateReport(repoRoot, 'technical-plan');
   const approvedVersion = technicalPlanCandidates.approved?.version
     ? `v${technicalPlanCandidates.approved.version}`
-    : 'approved';
+    : translator.t('status.approved');
   const reviewSummary = formatReviewSummary(technicalPlanCandidates.review);
-  const selectedMethodology = await selectOption('¿Qué metodología aplica esta spec?', SPEC_METHODOLOGY_OPTIONS, {
+  const selectedMethodology = await selectOption(translator.t('spec_create.prompt.methodology'), specMethodologyOptions(translator), {
     ...selectorOptions,
     defaultValue: 'wdd-sdd',
     flag: '--methodology',
     name: 'methodology',
     value: options.methodology || '',
   });
-  const selectedInput = await selectOption('¿Qué plan aprobado querés usar?', [
+  const selectedInput = await selectOption(translator.t('spec_create.prompt.input'), [
     {
       label: preview.inputPath,
       value: preview.inputPath,
-      hint: ['Plan técnico revisado y aprobado', approvedVersion, reviewSummary].filter(Boolean).join(', '),
+      hint: [translator.t('spec_create.input.approved_plan_hint'), approvedVersion, reviewSummary].filter(Boolean).join(', '),
       default: true,
     },
   ], {
@@ -466,17 +483,17 @@ async function resolveInteractiveSpecCreateOptions(repoRoot, preview, options =
     name: 'approved technical plan input',
     value: options.input || '',
   });
-  const selectedReview = await selectOption('¿Cómo querés revisar antes de escribir?', [
+  const selectedReview = await selectOption(translator.t('spec_create.prompt.review'), [
     {
-      label: 'Crear despues de confirmar',
+      label: translator.t('spec_create.review.direct.label'),
       value: 'direct',
-      hint: 'Muestra resumen y pide aprobacion final',
+      hint: translator.t('spec_create.review.direct.hint'),
       default: options.review !== true,
     },
     {
-      label: 'Abrir preview de review',
+      label: translator.t('spec_create.review.editor.label'),
       value: 'review',
-      hint: 'Usa $VISUAL o $EDITOR antes de escribir',
+      hint: translator.t('spec_create.review.editor.hint'),
       default: options.review === true,
     },
   ], {
@@ -487,14 +504,14 @@ async function resolveInteractiveSpecCreateOptions(repoRoot, preview, options =
   });
 
   ux.summary([
-    { label: 'Spec', value: preview.manifest.slug },
-    { label: 'Metodologia', value: selectedMethodology.label },
-    { label: 'Input', value: selectedInput.value },
-    { label: 'Plan tecnico', value: ['aprobado', approvedVersion, reviewSummary].filter(Boolean).join(', ') },
-    { label: 'Target', value: preview.relativeSpecDir },
-    { label: 'Archivos planificados', value: preview.preview.files.length },
-    { label: 'Revision', value: selectedReview.value === 'review' ? 'abrir preview' : 'confirmacion directa' },
-  ], { title: 'Spec create' });
+    { label: translator.t('spec_create.summary.spec'), value: preview.manifest.slug },
+    { label: translator.t('spec_create.summary.methodology'), value: selectedMethodology.label },
+    { label: translator.t('spec_create.summary.input'), value: selectedInput.value },
+    { label: translator.t('spec_create.summary.technical_plan'), value: [translator.t('status.approved'), approvedVersion, reviewSummary].filter(Boolean).join(', ') },
+    { label: translator.t('spec_create.summary.target'), value: preview.relativeSpecDir },
+    { label: translator.t('spec_create.summary.planned_files'), value: preview.preview.files.length },
+    { label: translator.t('spec_create.summary.review'), value: selectedReview.value === 'review' ? translator.t('spec_create.summary.review_editor') : translator.t('spec_create.summary.review_direct') },
+  ], { title: translator.t('spec_create.summary.title') });
 
   return {
     ...options,
@@ -520,7 +537,7 @@ async function confirmSpecCreate(message, options = {}) {
   });
   const confirmed = await ux.promptConfirm(message, { initialValue: false });
   if (!confirmed) {
-    throw new Error(formatError('spec create interactive approval declined. No files were written.'));
+    throw new Error(formatError(createTranslator(options.language).t('spec_create.error.approval_declined')));
   }
 }
 
@@ -534,7 +551,9 @@ async function reviewSpecCreatePreview(repoRoot, preview, options = {}) {
   const canOpenEditor = hasEditorRunner || options.stdinIsTTY === true || (options.stdinIsTTY !== false && Boolean(process.stdin.isTTY));
 
   if (!canOpenEditor) {
-    throw new Error(formatError(`spec create --review requires an interactive terminal or an injected editor runner.\nReview artifact: ${reviewPath}`));
+    throw new Error(formatError(createTranslator(options.language).t('spec_create.error.review_tty_required', {
+      path: reviewPath,
+    })));
   }
 
   const editorResult = hasEditorRunner
@@ -542,7 +561,11 @@ async function reviewSpecCreatePreview(repoRoot, preview, options = {}) {
     : openEditor(reviewPath, { cwd: repoRoot, env: options.env || process.env });
 
   if (!editorResult || editorResult.ok !== true) {
-    throw new Error(formatError(`${editorResult?.reason || 'spec create review was canceled.'}\nReview artifact: ${reviewPath}`));
+    const translator = createTranslator(options.language);
+    throw new Error(formatError(translator.t('spec_create.error.review_canceled', {
+      path: reviewPath,
+      reason: editorResult?.reason || translator.t('spec_create.error.review_canceled.default_reason'),
+    })));
   }
 
   return reviewPath;
@@ -551,9 +574,10 @@ async function reviewSpecCreatePreview(repoRoot, preview, options = {}) {
 async function runCreateSpec(repoRoot, options = {}) {
   const preview = buildSpecCreatePreview(repoRoot, options);
   const resolvedOptions = await resolveInteractiveSpecCreateOptions(repoRoot, preview, options);
+  const languageOptions = { language: resolvedOptions.language };
 
   if (resolvedOptions.dryRun) {
-    process.stdout.write(formatSpecCreateDryRun({ ...preview, reviewRequested: resolvedOptions.review === true }));
+    process.stdout.write(formatSpecCreateDryRun({ ...preview, reviewRequested: resolvedOptions.review === true }, languageOptions));
     return {
       task: 'spec-create',
       dryRun: true,
@@ -564,13 +588,13 @@ async function runCreateSpec(repoRoot, options = {}) {
   }
 
   const reviewPath = await reviewSpecCreatePreview(repoRoot, preview, resolvedOptions);
-  await confirmSpecCreate(`Create spec '${preview.manifest.slug}'?`, resolvedOptions);
+  await confirmSpecCreate(createTranslator(resolvedOptions.language).t('spec_create.confirm_create', { slug: preview.manifest.slug }), resolvedOptions);
 
   const result = generateSpecArtifacts(repoRoot, {
     input: resolvedOptions.input || preview.inputPath,
     specSlug: resolvedOptions.specSlug,
   });
-  process.stdout.write(formatSpecCreateResult(result, repoRoot));
+  process.stdout.write(formatSpecCreateResult(result, repoRoot, languageOptions));
 
   return {
     task: 'spec-create',