i18next-cli 1.59.0 → 1.59.1

package/dist/cjs/cli.js

@@ -32,7 +32,7 @@ const program = new commander.Command();
 program
     .name('i18next-cli')
     .description('A unified, high-performance i18next CLI.')
-    .version('1.59.0'); // This string is replaced with the actual version at build time by rollup
+    .version('1.59.1'); // This string is replaced with the actual version at build time by rollup
 // new: global config override option
 program.option('-c, --config <path>', 'Path to i18next-cli config file (overrides detection)');
 program

package/dist/cjs/utils/file-utils.js

@@ -7,6 +7,24 @@ var config = require('../config.js');
 var json5Parser = require('@croct/json5-parser');
 var yaml = require('yaml');
 
+/**
+ * Thrown when an existing translation file in a structured data format
+ * (JSON/JSON5/YAML) exists on disk but cannot be parsed. Callers should treat
+ * this as fatal rather than as an empty/missing file, since overwriting an
+ * unparseable file would silently destroy its contents (e.g. a merge conflict
+ * marker accidentally committed into an en.json).
+ */
+class ParseTranslationFileError extends Error {
+    filePath;
+    cause;
+    constructor(filePath, cause) {
+        const detail = cause instanceof Error ? cause.message : String(cause);
+        super(`Could not parse translation file ${filePath}: ${detail}`);
+        this.filePath = filePath;
+        this.cause = cause;
+        this.name = 'ParseTranslationFileError';
+    }
+}
 /**
  * Resolve an output template (string or function) into an actual path string.
  *
@@ -58,23 +76,48 @@ async function loadTranslationFile(filePath) {
     catch {
         return null; // File doesn't exist
     }
-    try {
-        const ext = node_path.extname(fullPath).toLowerCase();
-        if (ext === '.json5') {
-            const content = await promises.readFile(fullPath, 'utf-8');
+    const ext = node_path.extname(fullPath).toLowerCase();
+    // Structured data formats (JSON/JSON5/YAML): the file exists (it passed the
+    // access() check above) but could not be parsed. Treating this as `null` is
+    // dangerous: callers coalesce it to an empty object and may overwrite the
+    // existing file, silently destroying its contents (e.g. when a merge conflict
+    // marker breaks an en.json that a watcher then re-extracts). Fail loudly so
+    // the caller can stop instead of overwriting good data with nothing.
+    if (ext === '.json5') {
+        const content = await promises.readFile(fullPath, 'utf-8');
+        try {
             // Parse as a JSON5 object node
             const node = json5Parser.JsonParser.parse(content, json5Parser.JsonObjectNode);
             return node.toJSON();
         }
-        else if (ext === '.yaml' || ext === '.yml') {
-            const content = await promises.readFile(fullPath, 'utf-8');
+        catch (error) {
+            throw new ParseTranslationFileError(filePath, error);
+        }
+    }
+    else if (ext === '.yaml' || ext === '.yml') {
+        const content = await promises.readFile(fullPath, 'utf-8');
+        try {
             return yaml.parse(content);
         }
-        else if (ext === '.json') {
-            const content = await promises.readFile(fullPath, 'utf-8');
+        catch (error) {
+            throw new ParseTranslationFileError(filePath, error);
+        }
+    }
+    else if (ext === '.json') {
+        const content = await promises.readFile(fullPath, 'utf-8');
+        try {
             return JSON.parse(content);
         }
-        else if (ext === '.ts' || ext === '.js') {
+        catch (error) {
+            throw new ParseTranslationFileError(filePath, error);
+        }
+    }
+    else if (ext === '.ts' || ext === '.js') {
+        // .ts/.js resource files are loaded via jiti, which can fail for reasons
+        // unrelated to corruption (e.g. a transitive import or a side-effectful
+        // module). This path has been deliberately lenient since #59, so keep
+        // degrading to `null` here rather than aborting the whole command.
+        try {
             // Load TypeScript path aliases for proper module resolution
             const aliases = await config.getTsConfigAliases();
             const jiti$1 = jiti.createJiti(process.cwd(), {
@@ -84,12 +127,12 @@ async function loadTranslationFile(filePath) {
             const module = await jiti$1.import(fullPath, { default: true });
             return module;
         }
-        return null; // Unsupported file type
-    }
-    catch (error) {
-        console.warn(`Could not parse translation file ${filePath}:`, error);
-        return null;
+        catch (error) {
+            console.warn(`Could not parse translation file ${filePath}:`, error);
+            return null;
+        }
     }
+    return null; // Unsupported file type
 }
 // Helper to load raw JSON5 content for preservation
 async function loadRawJson5Content(filePath) {
@@ -156,6 +199,7 @@ function inferFormatFromPath(filePath, defaultFormat = 'json') {
     return defaultFormat || 'json';
 }
 
+exports.ParseTranslationFileError = ParseTranslationFileError;
 exports.getOutputPath = getOutputPath;
 exports.inferFormatFromPath = inferFormatFromPath;
 exports.loadRawJson5Content = loadRawJson5Content;

package/dist/esm/cli.js

@@ -30,7 +30,7 @@ const program = new Command();
 program
     .name('i18next-cli')
     .description('A unified, high-performance i18next CLI.')
-    .version('1.59.0'); // This string is replaced with the actual version at build time by rollup
+    .version('1.59.1'); // This string is replaced with the actual version at build time by rollup
 // new: global config override option
 program.option('-c, --config <path>', 'Path to i18next-cli config file (overrides detection)');
 program

package/dist/esm/utils/file-utils.js

@@ -5,6 +5,24 @@ import { getTsConfigAliases } from '../config.js';
 import { JsonParser, JsonObjectNode } from '@croct/json5-parser';
 import yaml from 'yaml';
 
+/**
+ * Thrown when an existing translation file in a structured data format
+ * (JSON/JSON5/YAML) exists on disk but cannot be parsed. Callers should treat
+ * this as fatal rather than as an empty/missing file, since overwriting an
+ * unparseable file would silently destroy its contents (e.g. a merge conflict
+ * marker accidentally committed into an en.json).
+ */
+class ParseTranslationFileError extends Error {
+    filePath;
+    cause;
+    constructor(filePath, cause) {
+        const detail = cause instanceof Error ? cause.message : String(cause);
+        super(`Could not parse translation file ${filePath}: ${detail}`);
+        this.filePath = filePath;
+        this.cause = cause;
+        this.name = 'ParseTranslationFileError';
+    }
+}
 /**
  * Resolve an output template (string or function) into an actual path string.
  *
@@ -56,23 +74,48 @@ async function loadTranslationFile(filePath) {
     catch {
         return null; // File doesn't exist
     }
-    try {
-        const ext = extname(fullPath).toLowerCase();
-        if (ext === '.json5') {
-            const content = await readFile(fullPath, 'utf-8');
+    const ext = extname(fullPath).toLowerCase();
+    // Structured data formats (JSON/JSON5/YAML): the file exists (it passed the
+    // access() check above) but could not be parsed. Treating this as `null` is
+    // dangerous: callers coalesce it to an empty object and may overwrite the
+    // existing file, silently destroying its contents (e.g. when a merge conflict
+    // marker breaks an en.json that a watcher then re-extracts). Fail loudly so
+    // the caller can stop instead of overwriting good data with nothing.
+    if (ext === '.json5') {
+        const content = await readFile(fullPath, 'utf-8');
+        try {
             // Parse as a JSON5 object node
             const node = JsonParser.parse(content, JsonObjectNode);
             return node.toJSON();
         }
-        else if (ext === '.yaml' || ext === '.yml') {
-            const content = await readFile(fullPath, 'utf-8');
+        catch (error) {
+            throw new ParseTranslationFileError(filePath, error);
+        }
+    }
+    else if (ext === '.yaml' || ext === '.yml') {
+        const content = await readFile(fullPath, 'utf-8');
+        try {
             return yaml.parse(content);
         }
-        else if (ext === '.json') {
-            const content = await readFile(fullPath, 'utf-8');
+        catch (error) {
+            throw new ParseTranslationFileError(filePath, error);
+        }
+    }
+    else if (ext === '.json') {
+        const content = await readFile(fullPath, 'utf-8');
+        try {
             return JSON.parse(content);
         }
-        else if (ext === '.ts' || ext === '.js') {
+        catch (error) {
+            throw new ParseTranslationFileError(filePath, error);
+        }
+    }
+    else if (ext === '.ts' || ext === '.js') {
+        // .ts/.js resource files are loaded via jiti, which can fail for reasons
+        // unrelated to corruption (e.g. a transitive import or a side-effectful
+        // module). This path has been deliberately lenient since #59, so keep
+        // degrading to `null` here rather than aborting the whole command.
+        try {
             // Load TypeScript path aliases for proper module resolution
             const aliases = await getTsConfigAliases();
             const jiti = createJiti(process.cwd(), {
@@ -82,12 +125,12 @@ async function loadTranslationFile(filePath) {
             const module = await jiti.import(fullPath, { default: true });
             return module;
         }
-        return null; // Unsupported file type
-    }
-    catch (error) {
-        console.warn(`Could not parse translation file ${filePath}:`, error);
-        return null;
+        catch (error) {
+            console.warn(`Could not parse translation file ${filePath}:`, error);
+            return null;
+        }
     }
+    return null; // Unsupported file type
 }
 // Helper to load raw JSON5 content for preservation
 async function loadRawJson5Content(filePath) {
@@ -154,4 +197,4 @@ function inferFormatFromPath(filePath, defaultFormat = 'json') {
     return defaultFormat || 'json';
 }
 
-export { getOutputPath, inferFormatFromPath, loadRawJson5Content, loadTranslationFile, serializeTranslationFile };
+export { ParseTranslationFileError, getOutputPath, inferFormatFromPath, loadRawJson5Content, loadTranslationFile, serializeTranslationFile };

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "i18next-cli",
-  "version": "1.59.0",
+  "version": "1.59.1",
   "description": "A unified, high-performance i18next CLI.",
   "type": "module",
   "bin": {

package/types/utils/file-utils.d.ts

@@ -1,4 +1,16 @@
 import type { I18nextToolkitConfig } from '../types.js';
+/**
+ * Thrown when an existing translation file in a structured data format
+ * (JSON/JSON5/YAML) exists on disk but cannot be parsed. Callers should treat
+ * this as fatal rather than as an empty/missing file, since overwriting an
+ * unparseable file would silently destroy its contents (e.g. a merge conflict
+ * marker accidentally committed into an en.json).
+ */
+export declare class ParseTranslationFileError extends Error {
+    readonly filePath: string;
+    readonly cause?: unknown | undefined;
+    constructor(filePath: string, cause?: unknown | undefined);
+}
 /**
  * Ensures that the directory for a given file path exists.
  * Creates all necessary parent directories recursively if they don't exist.

package/types/utils/file-utils.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"file-utils.d.ts","sourceRoot":"","sources":["../../src/utils/file-utils.ts"],"names":[],"mappings":"AAGA,OAAO,KAAK,EAAE,oBAAoB,EAAE,MAAM,aAAa,CAAA;AAKvD;;;;;;;;;;;GAWG;AACH,wBAAsB,qBAAqB,CAAE,QAAQ,EAAE,MAAM,GAAG,OAAO,CAAC,IAAI,CAAC,CAG5E;AAED;;;;;;;;;;;GAWG;AACH,wBAAsB,aAAa,CAAE,QAAQ,EAAE,MAAM,GAAG,OAAO,CAAC,MAAM,CAAC,CAEtE;AAED;;;;;;;;;;;GAWG;AACH,wBAAsB,cAAc,CAAE,QAAQ,EAAE,MAAM,EAAE,IAAI,EAAE,MAAM,GAAG,OAAO,CAAC,IAAI,CAAC,CAEnF;AAED;;;;;;;;GAQG;AACH,wBAAgB,aAAa,CAC3B,cAAc,EAAE,MAAM,GAAG,CAAC,CAAC,QAAQ,EAAE,MAAM,EAAE,SAAS,CAAC,EAAE,MAAM,KAAK,MAAM,CAAC,GAAG,SAAS,EACvF,QAAQ,EAAE,MAAM,EAChB,SAAS,CAAC,EAAE,MAAM,GACjB,MAAM,CA8BR;AAED;;;;GAIG;AACH,wBAAsB,mBAAmB,CAAE,QAAQ,EAAE,MAAM,GAAG,OAAO,CAAC,MAAM,CAAC,MAAM,EAAE,GAAG,CAAC,GAAG,IAAI,CAAC,CAwChG;AAGD,wBAAsB,mBAAmB,CAAE,QAAQ,EAAE,MAAM,GAAG,OAAO,CAAC,MAAM,GAAG,IAAI,CAAC,CAQnF;AAED;;;GAGG;AACH,wBAAgB,wBAAwB,CACtC,IAAI,EAAE,MAAM,CAAC,MAAM,EAAE,GAAG,CAAC,EACzB,MAAM,GAAE,oBAAoB,CAAC,SAAS,CAAC,CAAC,cAAc,CAAU,EAChE,WAAW,GAAE,MAAM,GAAG,MAAU,EAChC,UAAU,CAAC,EAAE,MAAM,GAClB,MAAM,CA6BR;AAED;;;;;GAKG;AACH,wBAAgB,mBAAmB,CACjC,QAAQ,EAAE,MAAM,EAChB,aAAa,GAAE,oBAAoB,CAAC,SAAS,CAAC,CAAC,cAAc,CAAU,GACtE,WAAW,CAAC,oBAAoB,CAAC,SAAS,CAAC,CAAC,cAAc,CAAC,CAAC,CAO9D"}
+{"version":3,"file":"file-utils.d.ts","sourceRoot":"","sources":["../../src/utils/file-utils.ts"],"names":[],"mappings":"AAGA,OAAO,KAAK,EAAE,oBAAoB,EAAE,MAAM,aAAa,CAAA;AAKvD;;;;;;GAMG;AACH,qBAAa,yBAA0B,SAAQ,KAAK;aACrB,QAAQ,EAAE,MAAM;aAAkB,KAAK,CAAC,EAAE,OAAO;gBAAjD,QAAQ,EAAE,MAAM,EAAkB,KAAK,CAAC,EAAE,OAAO,YAAA;CAK/E;AAED;;;;;;;;;;;GAWG;AACH,wBAAsB,qBAAqB,CAAE,QAAQ,EAAE,MAAM,GAAG,OAAO,CAAC,IAAI,CAAC,CAG5E;AAED;;;;;;;;;;;GAWG;AACH,wBAAsB,aAAa,CAAE,QAAQ,EAAE,MAAM,GAAG,OAAO,CAAC,MAAM,CAAC,CAEtE;AAED;;;;;;;;;;;GAWG;AACH,wBAAsB,cAAc,CAAE,QAAQ,EAAE,MAAM,EAAE,IAAI,EAAE,MAAM,GAAG,OAAO,CAAC,IAAI,CAAC,CAEnF;AAED;;;;;;;;GAQG;AACH,wBAAgB,aAAa,CAC3B,cAAc,EAAE,MAAM,GAAG,CAAC,CAAC,QAAQ,EAAE,MAAM,EAAE,SAAS,CAAC,EAAE,MAAM,KAAK,MAAM,CAAC,GAAG,SAAS,EACvF,QAAQ,EAAE,MAAM,EAChB,SAAS,CAAC,EAAE,MAAM,GACjB,MAAM,CA8BR;AAED;;;;GAIG;AACH,wBAAsB,mBAAmB,CAAE,QAAQ,EAAE,MAAM,GAAG,OAAO,CAAC,MAAM,CAAC,MAAM,EAAE,GAAG,CAAC,GAAG,IAAI,CAAC,CA8DhG;AAGD,wBAAsB,mBAAmB,CAAE,QAAQ,EAAE,MAAM,GAAG,OAAO,CAAC,MAAM,GAAG,IAAI,CAAC,CAQnF;AAED;;;GAGG;AACH,wBAAgB,wBAAwB,CACtC,IAAI,EAAE,MAAM,CAAC,MAAM,EAAE,GAAG,CAAC,EACzB,MAAM,GAAE,oBAAoB,CAAC,SAAS,CAAC,CAAC,cAAc,CAAU,EAChE,WAAW,GAAE,MAAM,GAAG,MAAU,EAChC,UAAU,CAAC,EAAE,MAAM,GAClB,MAAM,CA6BR;AAED;;;;;GAKG;AACH,wBAAgB,mBAAmB,CACjC,QAAQ,EAAE,MAAM,EAChB,aAAa,GAAE,oBAAoB,CAAC,SAAS,CAAC,CAAC,cAAc,CAAU,GACtE,WAAW,CAAC,oBAAoB,CAAC,SAAS,CAAC,CAAC,cAAc,CAAC,CAAC,CAO9D"}