i18next-cli 1.67.1 → 1.67.2

package/README.md

@@ -317,7 +317,7 @@ npx i18next-cli lint
 
 - **Hardcoded strings** *(error)* — user-facing text in JSX elements and attributes that isn't wrapped in `t()`/`<Trans>`.
 - **Interpolation parameters** *(error)* — mismatches between `{{placeholders}}` in a translation and the params passed to `t()` (missing or unused). Toggle with `lint.checkInterpolationParams` (default: `true`).
-- **String concatenation** *(warning by default)* — translated strings glued together with `+`, or a sentence split across multiple `<Trans>` components. This breaks in languages that reorder or inflect the pieces; use a single key with placeholders instead. Configure with `lint.checkConcatenation`: `'warn'` / `true` (default) reports it without failing the run, `'error'` makes it fail (exit non-zero, useful for CI), and `'off'` / `false` disables it.
+- **String concatenation** *(warning by default)* — translated strings glued together with `+`, or a sentence split across multiple adjacent translations (`<Trans>` components and/or `{t()}` expressions rendered as siblings). This breaks in languages that reorder or inflect the pieces; use a single key with placeholders instead. Configure with `lint.checkConcatenation`: `'warn'` / `true` (default) reports it without failing the run, `'error'` makes it fail (exit non-zero, useful for CI), and `'off'` / `false` disables it.
 
   ```jsx
   // ⚠️ Flagged — word order can't be translated

package/dist/cjs/cli.js

@@ -37,7 +37,7 @@ const program = new commander.Command();
 program
     .name('i18next-cli')
     .description('A unified, high-performance i18next CLI.')
-    .version('1.67.1'); // This string is replaced with the actual version at build time by rollup
+    .version('1.67.2'); // 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/linter.js

@@ -315,8 +315,10 @@ function lintInterpolationParams(ast, code, config, translationValues) {
  * Flags two patterns:
  *   JS  — a `+` expression where an operand is a t() call:
  *           t('greeting') + ', ' + name
- *   JSX — a sentence split across ≥2 <Trans> components joined by literal text:
- *           <p><Trans>Hello</Trans> and <Trans>World</Trans></p>
+ *   JSX — a sentence split across ≥2 adjacent translation units (a <Trans>
+ *         component or a `{t(...)}` expression) rendered as direct siblings:
+ *           <p><Trans>Hello</Trans> <Trans>World</Trans></p>
+ *           <p><Trans>new</Trans>{t('cat')}</p>
  *
  * The fix in both cases is a single key with placeholders:
  *   t('greeting', { name })  /  <Trans i18nKey="greeting">Hello {{name}}</Trans>
@@ -390,30 +392,33 @@ function lintConcatenation(ast, code, config) {
                 });
             }
         }
-        // JSX: a sentence split across ≥2 <Trans> components joined by literal text.
+        // JSX: a sentence split across ≥2 adjacent translation units rendered as
+        // direct siblings — a <Trans> component or a `{t(...)}` expression. Covers
+        // <Trans>…</Trans><Trans>…</Trans>, <Trans>…</Trans>{t(…)} and {t(…)}{t(…)}.
         if ((node.type === 'JSXElement' || node.type === 'JSXFragment') && Array.isArray(node.children)) {
-            let transCount = 0;
-            let firstTrans = null;
-            let hasLiteralText = false;
+            let unitCount = 0;
+            let firstUnit = null;
             for (const child of node.children) {
                 if (!child || typeof child !== 'object')
                     continue;
+                let isUnit = false;
                 if (child.type === 'JSXElement') {
                     const name = getJsxName(child);
-                    if (name && transComponents.has(name.toLowerCase())) {
-                        transCount++;
-                        if (!firstTrans)
-                            firstTrans = child;
-                    }
+                    isUnit = !!name && transComponents.has(name.toLowerCase());
+                }
+                else if (child.type === 'JSXExpressionContainer') {
+                    isUnit = isTranslationCall(child.expression, config);
                 }
-                else if (child.type === 'JSXText' && child.value.trim() !== '') {
-                    hasLiteralText = true;
+                if (isUnit) {
+                    unitCount++;
+                    if (!firstUnit)
+                        firstUnit = child;
                 }
             }
-            if (transCount >= 2 && hasLiteralText && firstTrans) {
+            if (unitCount >= 2 && firstUnit) {
                 issues.push({
-                    text: 'Avoid splitting a sentence across multiple <Trans> components. Use a single <Trans> with placeholders instead.',
-                    line: lineOf(firstTrans),
+                    text: 'Avoid splitting text across multiple translations (<Trans> or t()). Use a single translation with placeholders instead.',
+                    line: lineOf(firstUnit),
                     type: 'concatenation',
                     severity,
                 });

package/dist/esm/cli.js

@@ -31,7 +31,7 @@ const program = new Command();
 program
     .name('i18next-cli')
     .description('A unified, high-performance i18next CLI.')
-    .version('1.67.1'); // This string is replaced with the actual version at build time by rollup
+    .version('1.67.2'); // 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/linter.js

@@ -313,8 +313,10 @@ function lintInterpolationParams(ast, code, config, translationValues) {
  * Flags two patterns:
  *   JS  — a `+` expression where an operand is a t() call:
  *           t('greeting') + ', ' + name
- *   JSX — a sentence split across ≥2 <Trans> components joined by literal text:
- *           <p><Trans>Hello</Trans> and <Trans>World</Trans></p>
+ *   JSX — a sentence split across ≥2 adjacent translation units (a <Trans>
+ *         component or a `{t(...)}` expression) rendered as direct siblings:
+ *           <p><Trans>Hello</Trans> <Trans>World</Trans></p>
+ *           <p><Trans>new</Trans>{t('cat')}</p>
  *
  * The fix in both cases is a single key with placeholders:
  *   t('greeting', { name })  /  <Trans i18nKey="greeting">Hello {{name}}</Trans>
@@ -388,30 +390,33 @@ function lintConcatenation(ast, code, config) {
                 });
             }
         }
-        // JSX: a sentence split across ≥2 <Trans> components joined by literal text.
+        // JSX: a sentence split across ≥2 adjacent translation units rendered as
+        // direct siblings — a <Trans> component or a `{t(...)}` expression. Covers
+        // <Trans>…</Trans><Trans>…</Trans>, <Trans>…</Trans>{t(…)} and {t(…)}{t(…)}.
         if ((node.type === 'JSXElement' || node.type === 'JSXFragment') && Array.isArray(node.children)) {
-            let transCount = 0;
-            let firstTrans = null;
-            let hasLiteralText = false;
+            let unitCount = 0;
+            let firstUnit = null;
             for (const child of node.children) {
                 if (!child || typeof child !== 'object')
                     continue;
+                let isUnit = false;
                 if (child.type === 'JSXElement') {
                     const name = getJsxName(child);
-                    if (name && transComponents.has(name.toLowerCase())) {
-                        transCount++;
-                        if (!firstTrans)
-                            firstTrans = child;
-                    }
+                    isUnit = !!name && transComponents.has(name.toLowerCase());
+                }
+                else if (child.type === 'JSXExpressionContainer') {
+                    isUnit = isTranslationCall(child.expression, config);
                 }
-                else if (child.type === 'JSXText' && child.value.trim() !== '') {
-                    hasLiteralText = true;
+                if (isUnit) {
+                    unitCount++;
+                    if (!firstUnit)
+                        firstUnit = child;
                 }
             }
-            if (transCount >= 2 && hasLiteralText && firstTrans) {
+            if (unitCount >= 2 && firstUnit) {
                 issues.push({
-                    text: 'Avoid splitting a sentence across multiple <Trans> components. Use a single <Trans> with placeholders instead.',
-                    line: lineOf(firstTrans),
+                    text: 'Avoid splitting text across multiple translations (<Trans> or t()). Use a single translation with placeholders instead.',
+                    line: lineOf(firstUnit),
                     type: 'concatenation',
                     severity,
                 });

package/package.json

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

package/types/linter.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"linter.d.ts","sourceRoot":"","sources":["../src/linter.ts"],"names":[],"mappings":"AAIA,OAAO,EAAE,YAAY,EAAE,MAAM,aAAa,CAAA;AAO1C,OAAO,KAAK,EAAE,oBAAoB,EAAE,MAAM,EAAE,SAAS,EAA6B,MAAM,YAAY,CAAA;AAibpG,KAAK,cAAc,GAAG;IACpB,QAAQ,EAAE;QAAC;YACT,OAAO,EAAE,MAAM,CAAC;SACjB;KAAC,CAAC;IACH,IAAI,EAAE;QAAC;YACL,OAAO,EAAE,OAAO,CAAC;YACjB,OAAO,EAAE,MAAM,CAAC;YAChB,KAAK,EAAE,MAAM,CAAC,MAAM,EAAE,SAAS,EAAE,CAAC,CAAC;SACpC;KAAC,CAAC;IACH,KAAK,EAAE,CAAC,KAAK,EAAE,KAAK,CAAC,CAAC;CACvB,CAAA;AAED,eAAO,MAAM,uBAAuB,EAAE,MAAM,EAAiD,CAAA;AAC7F,eAAO,MAAM,6BAA6B,EAAE,MAAM,EAAqD,CAAA;AAKvG,qBAAa,MAAO,SAAQ,YAAY,CAAC,cAAc,CAAC;IACtD,OAAO,CAAC,MAAM,CAAsB;IACpC,OAAO,CAAC,MAAM,CAAQ;gBAET,MAAM,EAAE,oBAAoB,EAAE,MAAM,GAAE,MAA4B;IAM/E,SAAS,CAAE,KAAK,EAAE,OAAO;IAanB,GAAG;;;;;;;IAuIT,OAAO,CAAC,uBAAuB;YAOjB,qBAAqB;IAWnC,OAAO,CAAC,kBAAkB;IAM1B,OAAO,CAAC,0BAA0B;YAUpB,qBAAqB;YAgBrB,uBAAuB;CAgBtC;AAED;;;;;;;;;;;;;;;;;;;;;;;;;;;;GA4BG;AACH,wBAAsB,SAAS,CAAE,MAAM,EAAE,oBAAoB;;;;;;GAE5D;AAED,wBAAsB,YAAY,CAChC,MAAM,EAAE,oBAAoB,EAC5B,OAAO,GAAE;IAAE,KAAK,CAAC,EAAE,OAAO,CAAC;IAAC,MAAM,CAAC,EAAE,MAAM,CAAA;CAAO,iBA0CnD"}
+{"version":3,"file":"linter.d.ts","sourceRoot":"","sources":["../src/linter.ts"],"names":[],"mappings":"AAIA,OAAO,EAAE,YAAY,EAAE,MAAM,aAAa,CAAA;AAO1C,OAAO,KAAK,EAAE,oBAAoB,EAAE,MAAM,EAAE,SAAS,EAA6B,MAAM,YAAY,CAAA;AAsbpG,KAAK,cAAc,GAAG;IACpB,QAAQ,EAAE;QAAC;YACT,OAAO,EAAE,MAAM,CAAC;SACjB;KAAC,CAAC;IACH,IAAI,EAAE;QAAC;YACL,OAAO,EAAE,OAAO,CAAC;YACjB,OAAO,EAAE,MAAM,CAAC;YAChB,KAAK,EAAE,MAAM,CAAC,MAAM,EAAE,SAAS,EAAE,CAAC,CAAC;SACpC;KAAC,CAAC;IACH,KAAK,EAAE,CAAC,KAAK,EAAE,KAAK,CAAC,CAAC;CACvB,CAAA;AAED,eAAO,MAAM,uBAAuB,EAAE,MAAM,EAAiD,CAAA;AAC7F,eAAO,MAAM,6BAA6B,EAAE,MAAM,EAAqD,CAAA;AAKvG,qBAAa,MAAO,SAAQ,YAAY,CAAC,cAAc,CAAC;IACtD,OAAO,CAAC,MAAM,CAAsB;IACpC,OAAO,CAAC,MAAM,CAAQ;gBAET,MAAM,EAAE,oBAAoB,EAAE,MAAM,GAAE,MAA4B;IAM/E,SAAS,CAAE,KAAK,EAAE,OAAO;IAanB,GAAG;;;;;;;IAuIT,OAAO,CAAC,uBAAuB;YAOjB,qBAAqB;IAWnC,OAAO,CAAC,kBAAkB;IAM1B,OAAO,CAAC,0BAA0B;YAUpB,qBAAqB;YAgBrB,uBAAuB;CAgBtC;AAED;;;;;;;;;;;;;;;;;;;;;;;;;;;;GA4BG;AACH,wBAAsB,SAAS,CAAE,MAAM,EAAE,oBAAoB;;;;;;GAE5D;AAED,wBAAsB,YAAY,CAChC,MAAM,EAAE,oBAAoB,EAC5B,OAAO,GAAE;IAAE,KAAK,CAAC,EAAE,OAAO,CAAC;IAAC,MAAM,CAAC,EAAE,MAAM,CAAA;CAAO,iBA0CnD"}