gtx-cli 2.6.12 → 2.6.13

package/CHANGELOG.md

@@ -1,5 +1,14 @@
 # gtx-cli
 
+## 2.6.13
+
+### Patch Changes
+
+- [#995](https://github.com/generaltranslation/gt/pull/995) [`b33e1fd`](https://github.com/generaltranslation/gt/commit/b33e1fd3073b23c2cc5db900619fb4c8d4a64c1f) Thanks [@fernando-aviles](https://github.com/fernando-aviles)! - Improve warning display for CLI
+
+- Updated dependencies [[`4a66903`](https://github.com/generaltranslation/gt/commit/4a669031f74a0b20783709752ab7fc0ab40869df)]:
+  - generaltranslation@8.1.9
+
 ## 2.6.12
 
 ### Patch Changes

package/dist/api/downloadFileBatch.js

@@ -8,6 +8,7 @@ import { mergeJson } from '../formats/json/mergeJson.js';
 import mergeYaml from '../formats/yaml/mergeYaml.js';
 import { getDownloadedVersions, saveDownloadedVersions, ensureNestedObject, } from '../fs/config/downloadedVersions.js';
 import { recordDownloaded } from '../state/recentDownloads.js';
+import { recordWarning } from '../state/translateWarnings.js';
 import stringify from 'fast-json-stable-stringify';
 /**
  * Downloads multiple translation files in a single batch request
@@ -56,6 +57,7 @@ export async function downloadFileBatch(fileTracker, files, options, forceDownlo
                 const fileProperties = fileTracker.completed.get(fileKey);
                 if (!outputPath || !fileProperties) {
                     logger.warn(`No input/output path found for file: ${fileKey}`);
+                    recordWarning('failed_download', fileKey, 'No input/output path found');
                     result.failed.push(requestedFile);
                     continue;
                 }
@@ -139,6 +141,7 @@ export async function downloadFileBatch(fileTracker, files, options, forceDownlo
             }
             catch (error) {
                 logger.error(`Error saving file ${fileKey}: ` + error);
+                recordWarning('failed_download', fileKey, `Error saving file: ${error}`);
                 result.failed.push(requestedFile);
             }
         }

package/dist/cli/base.js

@@ -20,6 +20,8 @@ import { handleStage } from './commands/stage.js';
 import { handleSetupProject } from './commands/setupProject.js';
 import { handleDownload, handleTranslate, postProcessTranslations, } from './commands/translate.js';
 import { getDownloaded, clearDownloaded } from '../state/recentDownloads.js';
+import { clearWarnings } from '../state/translateWarnings.js';
+import { displayTranslateSummary } from '../console/displayTranslateSummary.js';
 import updateConfig from '../fs/config/updateConfig.js';
 import { createLoadTranslationsFile } from '../fs/createLoadTranslationsFile.js';
 import { saveLocalEdits } from '../api/saveLocalEdits.js';
@@ -130,6 +132,8 @@ export class BaseCLI {
             await postProcessTranslations(settings, include);
         }
         clearDownloaded();
+        displayTranslateSummary();
+        clearWarnings();
     }
     setupUploadCommand() {
         attachTranslateFlags(this.program

package/dist/console/displayTranslateSummary.d.ts

@@ -0,0 +1 @@
+export declare function displayTranslateSummary(): void;

package/dist/console/displayTranslateSummary.js

@@ -0,0 +1,42 @@
+import chalk from 'chalk';
+import { logger } from './logger.js';
+import { getWarnings, hasWarnings, } from '../state/translateWarnings.js';
+const CATEGORY_LABELS = {
+    skipped_file: 'Files skipped',
+    failed_move: 'File moves failed',
+    failed_translation: 'Translations failed',
+    failed_download: 'Downloads failed',
+};
+const CATEGORY_ORDER = [
+    'skipped_file',
+    'failed_move',
+    'failed_translation',
+    'failed_download',
+];
+export function displayTranslateSummary() {
+    if (!hasWarnings())
+        return;
+    const warnings = getWarnings();
+    // Group by category
+    const grouped = new Map();
+    for (const w of warnings) {
+        let list = grouped.get(w.category);
+        if (!list) {
+            list = [];
+            grouped.set(w.category, list);
+        }
+        list.push({ fileName: w.fileName, reason: w.reason });
+    }
+    const lines = [chalk.yellow('⚠  Warnings:'), ''];
+    for (const category of CATEGORY_ORDER) {
+        const items = grouped.get(category);
+        if (!items)
+            continue;
+        lines.push(`    ${CATEGORY_LABELS[category]} (${items.length}):`);
+        for (const item of items) {
+            lines.push(`      - ${item.fileName}: ${item.reason}`);
+        }
+        lines.push('');
+    }
+    logger.warn(lines.join('\n'));
+}

package/dist/formats/files/translate.js

@@ -1,4 +1,5 @@
 import { logger } from '../../console/logger.js';
+import { recordWarning } from '../../state/translateWarnings.js';
 import { getRelative, readFile } from '../../fs/findFilepath.js';
 import { SUPPORTED_FILE_EXTENSIONS } from './supportedFiles.js';
 import sanitizeFileContent from '../../utils/sanitizeFileContent.js';
@@ -50,6 +51,7 @@ export async function aggregateFiles(settings) {
                 }
                 catch (e) {
                     logger.warn(`Skipping ${relativePath}: JSON file is not parsable`);
+                    recordWarning('skipped_file', relativePath, 'JSON file is not parsable');
                     return null;
                 }
             }
@@ -69,6 +71,7 @@ export async function aggregateFiles(settings) {
                 return false;
             if (typeof file.content !== 'string' || !file.content.trim()) {
                 logger.warn(`Skipping ${file.fileName}: JSON file is empty`);
+                recordWarning('skipped_file', file.fileName, 'JSON file is empty');
                 return false;
             }
             return true;
@@ -88,6 +91,7 @@ export async function aggregateFiles(settings) {
                 }
                 catch (e) {
                     logger.warn(`Skipping ${relativePath}: YAML file is not parsable`);
+                    recordWarning('skipped_file', relativePath, 'YAML file is not parsable');
                     return null;
                 }
             }
@@ -104,6 +108,7 @@ export async function aggregateFiles(settings) {
             .filter((file) => {
             if (!file || typeof file.content !== 'string' || !file.content.trim()) {
                 logger.warn(`Skipping ${file?.fileName ?? 'unknown'}: YAML file is empty`);
+                recordWarning('skipped_file', file?.fileName ?? 'unknown', 'YAML file is empty');
                 return false;
             }
             return true;
@@ -123,6 +128,7 @@ export async function aggregateFiles(settings) {
                         const validation = isValidMdx(content, filePath);
                         if (!validation.isValid) {
                             logger.warn(`Skipping ${relativePath}: MDX file is not AST parsable${validation.error ? `: ${validation.error}` : ''}`);
+                            recordWarning('skipped_file', relativePath, `MDX file is not AST parsable${validation.error ? `: ${validation.error}` : ''}`);
                             return null;
                         }
                     }
@@ -152,6 +158,7 @@ export async function aggregateFiles(settings) {
                     typeof file.content !== 'string' ||
                     !file.content.trim()) {
                     logger.warn(`Skipping ${file?.fileName ?? 'unknown'}: File is empty after sanitization`);
+                    recordWarning('skipped_file', file?.fileName ?? 'unknown', 'File is empty after sanitization');
                     return false;
                 }
                 return true;

package/dist/generated/version.d.ts

@@ -1 +1 @@
-export declare const PACKAGE_VERSION = "2.6.12";
+export declare const PACKAGE_VERSION = "2.6.13";

package/dist/generated/version.js

@@ -1,2 +1,2 @@
 // This file is auto-generated. Do not edit manually.
-export const PACKAGE_VERSION = '2.6.12';
+export const PACKAGE_VERSION = '2.6.13';

package/dist/state/translateWarnings.d.ts

@@ -0,0 +1,10 @@
+export type WarningCategory = 'skipped_file' | 'failed_move' | 'failed_translation' | 'failed_download';
+export type TranslateWarning = {
+    category: WarningCategory;
+    fileName: string;
+    reason: string;
+};
+export declare function recordWarning(category: WarningCategory, fileName: string, reason: string): void;
+export declare function getWarnings(): TranslateWarning[];
+export declare function hasWarnings(): boolean;
+export declare function clearWarnings(): void;

package/dist/state/translateWarnings.js

@@ -0,0 +1,13 @@
+const warnings = [];
+export function recordWarning(category, fileName, reason) {
+    warnings.push({ category, fileName, reason });
+}
+export function getWarnings() {
+    return warnings;
+}
+export function hasWarnings() {
+    return warnings.length > 0;
+}
+export function clearWarnings() {
+    warnings.length = 0;
+}

package/dist/workflow/DownloadStep.js

@@ -2,6 +2,7 @@ import chalk from 'chalk';
 import { WorkflowStep } from './Workflow.js';
 import { logger } from '../console/logger.js';
 import { downloadFileBatch, } from '../api/downloadFileBatch.js';
+import { recordWarning } from '../state/translateWarnings.js';
 export class DownloadTranslationsStep extends WorkflowStep {
     gt;
     settings;
@@ -74,6 +75,9 @@ export class DownloadTranslationsStep extends WorkflowStep {
                 this.spinner?.stop(chalk.green(`Downloaded ${batchResult.successful.length} files${batchResult.skipped.length > 0 ? `, skipped ${batchResult.skipped.length} files` : ''}`));
                 if (batchResult.failed.length > 0) {
                     logger.warn(`Failed to download ${batchResult.failed.length} files: ${batchResult.failed.map((f) => f.inputPath).join('\n')}`);
+                    for (const f of batchResult.failed) {
+                        recordWarning('failed_download', f.inputPath, `Failed to download for locale ${f.locale}`);
+                    }
                 }
             }
             else {

package/dist/workflow/UploadSourcesStep.js

@@ -1,5 +1,6 @@
 import { WorkflowStep } from './Workflow.js';
 import { logger } from '../console/logger.js';
+import { recordWarning } from '../state/translateWarnings.js';
 import chalk from 'chalk';
 export class UploadSourcesStep extends WorkflowStep {
     gt;
@@ -72,6 +73,12 @@ export class UploadSourcesStep extends WorkflowStep {
             const failed = moveResult.summary.failed;
             if (failed > 0) {
                 logger.warn(`Failed to migrate ${failed} moved file${failed !== 1 ? 's' : ''}`);
+                for (const r of moveResult.results) {
+                    if (!r.success) {
+                        const move = moves.find((m) => m.newFileId === r.newFileId);
+                        recordWarning('failed_move', move?.newFileName ?? r.newFileId, r.error ?? 'Unknown error');
+                    }
+                }
             }
         }
         // Build a map of branch:fileId:versionId to fileData

package/dist/workflow/download.js

@@ -5,6 +5,7 @@ import { PollTranslationJobsStep } from './PollJobsStep.js';
 import { DownloadTranslationsStep } from './DownloadStep.js';
 import { logErrorAndExit } from '../console/logging.js';
 import { logger } from '../console/logger.js';
+import { recordWarning } from '../state/translateWarnings.js';
 import { BranchStep } from './BranchStep.js';
 import chalk from 'chalk';
 /**
@@ -70,6 +71,9 @@ export async function downloadTranslations(fileVersionData, jobData, branchData,
             logger.error(`${chalk.red(`${pollResult.fileTracker.failed.size} file(s) failed to translate:`)}\n${Array.from(pollResult.fileTracker.failed.entries())
                 .map(([key, value]) => `- ${value.fileName}`)
                 .join('\n')}`);
+            for (const [, value] of pollResult.fileTracker.failed) {
+                recordWarning('failed_translation', value.fileName, `Failed to translate for locale ${value.locale}`);
+            }
         }
         if (!pollResult.success) {
             return false;

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "gtx-cli",
-  "version": "2.6.12",
+  "version": "2.6.13",
   "main": "dist/index.js",
   "bin": "dist/main.js",
   "files": [
@@ -110,7 +110,7 @@
     "unified": "^11.0.5",
     "unist-util-visit": "^5.0.0",
     "yaml": "^2.8.0",
-    "generaltranslation": "8.1.8"
+    "generaltranslation": "8.1.9"
   },
   "devDependencies": {
     "@babel/types": "^7.28.4",