gtx-cli 2.4.13 → 2.4.15

package/CHANGELOG.md

@@ -1,5 +1,18 @@
 # gtx-cli
 
+## 2.4.15
+
+### Patch Changes
+
+- [#792](https://github.com/generaltranslation/gt/pull/792) [`b6d6869`](https://github.com/generaltranslation/gt/commit/b6d686917316f6ed44130a54509459a7f9ee35fa) Thanks [@fernando-aviles](https://github.com/fernando-aviles)! - Skipping over JSON and YAML files with failing parse to avoid crashing
+
+## 2.4.14
+
+### Patch Changes
+
+- Updated dependencies [[`3da05a1`](https://github.com/generaltranslation/gt/commit/3da05a12a37a62ace3c7e321aa2fed5a4af52ad9)]:
+  - generaltranslation@7.9.1
+
 ## 2.4.13
 
 ### Patch Changes

package/dist/formats/files/translate.js

@@ -4,6 +4,7 @@ import { SUPPORTED_FILE_EXTENSIONS } from './supportedFiles.js';
 import sanitizeFileContent from '../../utils/sanitizeFileContent.js';
 import { parseJson } from '../json/parseJson.js';
 import parseYaml from '../yaml/parseYaml.js';
+import YAML from 'yaml';
 import { determineLibrary } from '../../fs/determineFramework.js';
 import { isValidMdx } from '../../utils/validateMdx.js';
 export const SUPPORTED_DATA_FORMATS = ['JSX', 'ICU', 'I18NEXT'];
@@ -39,6 +40,14 @@ export async function aggregateFiles(settings) {
             .map((filePath) => {
             const content = readFile(filePath);
             const relativePath = getRelative(filePath);
+            // Pre-validate JSON parseability
+            try {
+                JSON.parse(content);
+            }
+            catch (e) {
+                logWarning(`Skipping ${relativePath}: JSON file is not parsable`);
+                return null;
+            }
             const parsedJson = parseJson(content, filePath, settings.options || {}, settings.defaultLocale);
             return {
                 content: parsedJson,
@@ -48,8 +57,10 @@ export async function aggregateFiles(settings) {
             };
         })
             .filter((file) => {
-            if (!file || typeof file.content !== 'string' || !file.content.trim()) {
-                logWarning(`Skipping ${file?.fileName ?? 'unknown'}: JSON file is empty`);
+            if (!file)
+                return false;
+            if (typeof file.content !== 'string' || !file.content.trim()) {
+                logWarning(`Skipping ${file.fileName}: JSON file is empty`);
                 return false;
             }
             return true;
@@ -62,6 +73,14 @@ export async function aggregateFiles(settings) {
             .map((filePath) => {
             const content = readFile(filePath);
             const relativePath = getRelative(filePath);
+            // Pre-validate YAML parseability
+            try {
+                YAML.parse(content);
+            }
+            catch (e) {
+                logWarning(`Skipping ${relativePath}: YAML file is not parsable`);
+                return null;
+            }
             const { content: parsedYaml, fileFormat } = parseYaml(content, filePath, settings.options || {});
             return {
                 content: parsedYaml,
@@ -70,8 +89,10 @@ export async function aggregateFiles(settings) {
             };
         })
             .filter((file) => {
-            if (!file || typeof file.content !== 'string' || !file.content.trim()) {
-                logWarning(`Skipping ${file?.fileName ?? 'unknown'}: YAML file is empty`);
+            if (!file)
+                return false;
+            if (typeof file.content !== 'string' || !file.content.trim()) {
+                logWarning(`Skipping ${file.fileName}: YAML file is empty`);
                 return false;
             }
             return true;

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "gtx-cli",
-  "version": "2.4.13",
+  "version": "2.4.15",
   "main": "dist/index.js",
   "bin": "dist/main.js",
   "files": [
@@ -94,7 +94,7 @@
     "unified": "^11.0.5",
     "unist-util-visit": "^5.0.0",
     "yaml": "^2.8.0",
-    "generaltranslation": "7.9.0"
+    "generaltranslation": "7.9.1"
   },
   "devDependencies": {
     "@babel/types": "^7.28.4",