gtx-cli 2.1.17 → 2.1.18

package/CHANGELOG.md

@@ -1,5 +1,11 @@
 # gtx-cli
 
+## 2.1.18
+
+### Patch Changes
+
+- [#637](https://github.com/generaltranslation/gt/pull/637) [`9c40a3c`](https://github.com/generaltranslation/gt/commit/9c40a3c729bf690381959679078c11c9c29bcdf2) Thanks [@fernando-aviles](https://github.com/fernando-aviles)! - Skipping over empty files when sending for translation
+
 ## 2.1.17
 
 ### Patch Changes

package/dist/formats/files/translate.js

@@ -35,30 +35,46 @@ export async function aggregateFiles(settings) {
         else {
             dataFormat = 'JSX';
         }
-        const jsonFiles = filePaths.json.map((filePath) => {
+        const jsonFiles = filePaths.json
+            .map((filePath) => {
             const content = readFile(filePath);
-            const parsedJson = parseJson(content, filePath, settings.options || {}, settings.defaultLocale);
             const relativePath = getRelative(filePath);
+            const parsedJson = parseJson(content, filePath, settings.options || {}, settings.defaultLocale);
             return {
                 content: parsedJson,
                 fileName: relativePath,
                 fileFormat: 'JSON',
                 dataFormat,
             };
+        })
+            .filter((file) => {
+            if (!file || typeof file.content !== 'string' || !file.content.trim()) {
+                logWarning(`Skipping ${file?.fileName ?? 'unknown'}: JSON file is empty`);
+                return false;
+            }
+            return true;
         });
         allFiles.push(...jsonFiles);
     }
     // Process YAML files
     if (filePaths.yaml) {
-        const yamlFiles = filePaths.yaml.map((filePath) => {
+        const yamlFiles = filePaths.yaml
+            .map((filePath) => {
             const content = readFile(filePath);
-            const { content: parsedYaml, fileFormat } = parseYaml(content, filePath, settings.options || {});
             const relativePath = getRelative(filePath);
+            const { content: parsedYaml, fileFormat } = parseYaml(content, filePath, settings.options || {});
             return {
                 content: parsedYaml,
                 fileName: relativePath,
                 fileFormat,
             };
+        })
+            .filter((file) => {
+            if (!file || typeof file.content !== 'string' || !file.content.trim()) {
+                logWarning(`Skipping ${file?.fileName ?? 'unknown'}: YAML file is empty`);
+                return false;
+            }
+            return true;
         });
         allFiles.push(...yamlFiles);
     }
@@ -73,8 +89,7 @@ export async function aggregateFiles(settings) {
                 if (fileType === 'mdx') {
                     const validation = isValidMdx(content, filePath);
                     if (!validation.isValid) {
-                        const errorMsg = validation.error ? `: ${validation.error}` : '';
-                        logWarning(`Skipping ${relativePath}: MDX file is not AST parsable${errorMsg}`);
+                        logWarning(`Skipping ${relativePath}: MDX file is not AST parsable${validation.error ? `: ${validation.error}` : ''}`);
                         return null;
                     }
                 }
@@ -85,7 +100,15 @@ export async function aggregateFiles(settings) {
                     fileFormat: fileType.toUpperCase(),
                 };
             })
-                .filter((file) => file !== null);
+                .filter((file) => {
+                if (!file ||
+                    typeof file.content !== 'string' ||
+                    !file.content.trim()) {
+                    logWarning(`Skipping ${file?.fileName ?? 'unknown'}: File is empty after sanitization`);
+                    return false;
+                }
+                return true;
+            });
             allFiles.push(...files);
         }
     }

package/dist/utils/localizeStaticImports.js

@@ -8,17 +8,6 @@ import remarkMdx from 'remark-mdx';
 import remarkFrontmatter from 'remark-frontmatter';
 import { visit } from 'unist-util-visit';
 const { isMatch } = micromatch;
-/**
- * Checks if a file exists at the given path
- */
-function fileExists(filePath) {
-    try {
-        return fs.existsSync(filePath);
-    }
-    catch {
-        return false;
-    }
-}
 /**
  * Localizes static imports in content files.
  * Currently only supported for md and mdx files. (/docs/ -> /[locale]/docs/)
@@ -55,6 +44,10 @@ export default async function localizeStaticImports(settings) {
         }
         if (defaultLocaleFiles.length > 0) {
             const defaultPromise = Promise.all(defaultLocaleFiles.map(async (filePath) => {
+                // Check if file exists before processing
+                if (!fs.existsSync(filePath)) {
+                    return;
+                }
                 // Get file content
                 const fileContent = await fs.promises.readFile(filePath, 'utf8');
                 // Localize the file using default locale
@@ -72,6 +65,10 @@ export default async function localizeStaticImports(settings) {
         const targetFiles = Object.values(filesMap).filter((path) => path.endsWith('.md') || path.endsWith('.mdx'));
         // Replace the placeholder path with the target path
         await Promise.all(targetFiles.map(async (filePath) => {
+            // Check if file exists before processing
+            if (!fs.existsSync(filePath)) {
+                return;
+            }
             // Get file content
             const fileContent = await fs.promises.readFile(filePath, 'utf8');
             // Localize the file
@@ -211,8 +208,7 @@ function transformImportPath(fullPath, patternHead, targetLocale, defaultLocale,
             const currentDir = path.dirname(currentFilePath);
             resolvedPath = path.resolve(currentDir, newPath);
         }
-        const pathExists = fileExists(resolvedPath);
-        if (!pathExists) {
+        if (!fs.existsSync(resolvedPath)) {
             return null;
         }
     }

package/dist/utils/localizeStaticUrls.js

@@ -49,6 +49,10 @@ export default async function localizeStaticUrls(settings, targetLocales) {
         }
         if (defaultLocaleFiles.length > 0) {
             const defaultPromise = Promise.all(defaultLocaleFiles.map(async (filePath) => {
+                // Check if file exists before processing
+                if (!fs.existsSync(filePath)) {
+                    return;
+                }
                 // Get file content
                 const fileContent = await fs.promises.readFile(filePath, 'utf8');
                 // Localize the file using default locale

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "gtx-cli",
-  "version": "2.1.17",
+  "version": "2.1.18",
   "main": "dist/index.js",
   "bin": "dist/main.js",
   "files": [