@lingual/i18n-check 0.8.13 → 0.8.15

package/dist/{src/bin → bin}/index.js

@@ -14,6 +14,7 @@ const __1 = require("..");
 const errorReporters_1 = require("../errorReporters");
 const flattenTranslations_1 = require("../utils/flattenTranslations");
 const node_path_1 = __importDefault(require("node:path"));
+const findInvalidTranslations_1 = require("../utils/findInvalidTranslations");
 const version = require('../../package.json').version;
 commander_1.program
     .version(version)
@@ -156,7 +157,7 @@ const main = async () => {
         options.checks.includes('invalidKeys')) &&
         targetFiles.length === 0) {
         // Remove missingKeys and invalidKeys from checks since they require multiple files
-        options.checks = options.checks.filter(check => check !== 'missingKeys' && check !== 'invalidKeys');
+        options.checks = options.checks.filter((check) => check !== 'missingKeys' && check !== 'invalidKeys');
         console.log(chalk_1.default.yellow('\nOnly one locale file found. Skipping missingKeys and invalidKeys checks.\n'));
     }
     try {
@@ -193,10 +194,22 @@ const main = async () => {
         else {
             (0, node_process_1.exit)(0);
         }
-        // eslint-disable-next-line @typescript-eslint/no-unused-vars
     }
     catch (e) {
-        console.log(chalk_1.default.red("\nError: Can't validate translations. Check if the format is supported or specify the translation format i.e. -f i18next"));
+        let errorMessage = "\nError: Can't validate translations. Check if the format is supported or specify the translation format i.e. -f i18next";
+        // Use enhanced error message if available
+        if (e instanceof findInvalidTranslations_1.CheckError) {
+            errorMessage = `\n${e.message}`;
+            // Check if the error has location information (from ICU parser)
+            if (e.location) {
+                errorMessage += `\nLocation: Line ${e.location.start.line}, Column ${e.location.start.column}`;
+            }
+            // Check if the error has originalMessage (from ICU parser)
+            if (e.originalMessage != null) {
+                errorMessage += `\nProblematic translation: "${e.originalMessage}"`;
+            }
+        }
+        console.log(chalk_1.default.red(errorMessage));
         (0, node_process_1.exit)(1);
     }
 };

package/dist/{src/utils → utils}/findInvalidTranslations.d.ts

@@ -1,5 +1,16 @@
 import { MessageFormatElement } from '@formatjs/icu-messageformat-parser';
 import { InvalidTranslationEntry, InvalidTranslationsResult, Translation } from '../types';
+type Location = {
+    start: {
+        line: number;
+        column: number;
+    };
+};
+export declare class CheckError extends Error {
+    location?: Location;
+    originalMessage?: string;
+}
 export declare const findInvalidTranslations: (source: Translation, files: Record<string, Translation>) => InvalidTranslationsResult;
 export declare const compareTranslationFiles: (a: Translation, b: Translation) => InvalidTranslationEntry[];
 export declare const hasDiff: (a: MessageFormatElement[], b: MessageFormatElement[]) => boolean;
+export {};

package/dist/{src/utils → utils}/findInvalidTranslations.js

@@ -1,16 +1,46 @@
 "use strict";
 Object.defineProperty(exports, "__esModule", { value: true });
-exports.hasDiff = exports.compareTranslationFiles = exports.findInvalidTranslations = void 0;
+exports.hasDiff = exports.compareTranslationFiles = exports.findInvalidTranslations = exports.CheckError = void 0;
 const icu_messageformat_parser_1 = require("@formatjs/icu-messageformat-parser");
+class CheckError extends Error {
+}
+exports.CheckError = CheckError;
+function isLocation(value) {
+    return (typeof value === 'object' &&
+        value != null &&
+        'start' in value &&
+        typeof value.start === 'object' &&
+        value.start != null &&
+        'line' in value.start &&
+        typeof value.start.line === 'number' &&
+        'column' in value.start &&
+        typeof value.start.column === 'number');
+}
 const findInvalidTranslations = (source, files) => {
     const differences = {};
     if (Object.keys(files).length === 0) {
         return differences;
     }
     for (const [lang, file] of Object.entries(files)) {
-        const result = (0, exports.compareTranslationFiles)(source, file);
-        if (result.length > 0) {
-            differences[lang] = result;
+        try {
+            const result = (0, exports.compareTranslationFiles)(source, file);
+            if (result.length > 0) {
+                differences[lang] = result;
+            }
+        }
+        catch (error) {
+            // Re-throw with file context
+            const enhancedError = new CheckError(`Error in translation file "${lang}": ${error instanceof Error ? error.message : String(error)}`);
+            if (error instanceof Error) {
+                if ('location' in error && isLocation(error.location)) {
+                    enhancedError.location = error.location;
+                }
+                if ('originalMessage' in error &&
+                    typeof error.originalMessage === 'string') {
+                    enhancedError.originalMessage = error.originalMessage;
+                }
+            }
+            throw enhancedError;
         }
     }
     return differences;
@@ -32,11 +62,28 @@ const compareTranslationFiles = (a, b) => {
         if (b[key] === undefined) {
             continue;
         }
-        const parsedTranslationA = (0, icu_messageformat_parser_1.parse)(String(a[key]));
-        const parsedTranslationB = (0, icu_messageformat_parser_1.parse)(String(b[key]));
-        if ((0, exports.hasDiff)(parsedTranslationA, parsedTranslationB)) {
-            const msg = getErrorMessage(parsedTranslationA, parsedTranslationB);
-            diffs.push({ key, msg });
+        try {
+            const parsedTranslationA = (0, icu_messageformat_parser_1.parse)(String(a[key]));
+            const parsedTranslationB = (0, icu_messageformat_parser_1.parse)(String(b[key]));
+            if ((0, exports.hasDiff)(parsedTranslationA, parsedTranslationB)) {
+                const msg = getErrorMessage(parsedTranslationA, parsedTranslationB);
+                diffs.push({ key, msg });
+            }
+        }
+        catch (error) {
+            // Re-throw with key context and preserve location/originalMessage
+            const errorMessage = error instanceof Error ? error.message : String(error);
+            const enhancedError = new CheckError(`Failed to parse translation key "${key}": ${errorMessage === 'INVALID_TAG' ? 'Invalid ICU message format tags found in translation content' : errorMessage}`);
+            if (error instanceof Error) {
+                if ('location' in error && isLocation(error.location)) {
+                    enhancedError.location = error.location;
+                }
+                if ('originalMessage' in error &&
+                    typeof error.originalMessage === 'string') {
+                    enhancedError.originalMessage = error.originalMessage;
+                }
+            }
+            throw enhancedError;
         }
     }
     return diffs;

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@lingual/i18n-check",
-  "version": "0.8.13",
+  "version": "0.8.15",
   "description": "i18n translation messages check",
   "license": "MIT",
   "main": "dist/index.js",
@@ -16,7 +16,7 @@
     "lint:fix": "eslint src --fix ",
     "check-format": "prettier --check './{src,translations}/**/*.{js,jsx,ts,tsx,json,html,css}'",
     "test": "vitest run",
-    "test:cli": "tsc && vitest --config vitest.bin.config.ts run src/bin/index.test.ts"
+    "test:cli": "pnpm build && vitest --config vitest.bin.config.ts run src/bin/index.test.ts"
   },
   "files": [
     "dist/",

package/dist/vitest.bin.config.d.ts

@@ -1,2 +0,0 @@
-declare const _default: import("vite", { with: { "resolution-mode": "import" } }).UserConfig;
-export default _default;

package/dist/vitest.bin.config.js

@@ -1,14 +0,0 @@
-"use strict";
-var __importDefault = (this && this.__importDefault) || function (mod) {
-    return (mod && mod.__esModule) ? mod : { "default": mod };
-};
-Object.defineProperty(exports, "__esModule", { value: true });
-// vitest.config.ts
-const config_1 = require("vitest/config");
-const vitest_config_1 = __importDefault(require("./vitest.config"));
-exports.default = (0, config_1.defineConfig)({
-    ...vitest_config_1.default,
-    test: {
-        exclude: ['node_modules', 'dist'],
-    },
-});

package/dist/vitest.config.d.ts

@@ -1,2 +0,0 @@
-declare const _default: import("vite", { with: { "resolution-mode": "import" } }).UserConfig;
-export default _default;

package/dist/vitest.config.js

@@ -1,12 +0,0 @@
-"use strict";
-Object.defineProperty(exports, "__esModule", { value: true });
-// vitest.config.ts
-const config_1 = require("vitest/config");
-exports.default = (0, config_1.defineConfig)({
-    test: {
-        globals: true, // (optional) use Jest-like globals like `describe`, `test`, etc.
-        environment: 'node', // 'node' or 'jsdom' depending on your project
-        // setupFiles: ['./vitest.setup.ts'], // optional setup file
-        exclude: ['src/bin', 'node_modules', 'dist'], // exclude specific test paths
-    },
-});