i18next-cli 1.56.0 → 1.56.2

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.56.0'); // This string is replaced with the actual version at build time by rollup
+    .version('1.56.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

@@ -755,14 +755,22 @@ function findHardcodedStrings(ast, code, config) {
         }
         if (node.type === 'StringLiteral') {
             const parent = currentAncestors[currentAncestors.length - 2];
+            const grandparent = currentAncestors[currentAncestors.length - 3];
             // Determine whether this attribute is inside any ignored element (handles nested Trans etc.)
             const insideIgnored = isWithinIgnoredElement(currentAncestors);
-            if (parent?.type === 'JSXAttribute' && !insideIgnored) {
-                const rawAttrName = extractAttrName(parent.name);
+            // A StringLiteral can be an attribute value in two forms:
+            //   <tag attr="value" />       → parent is JSXAttribute
+            //   <tag attr={"value"} />     → parent is JSXExpressionContainer, grandparent is JSXAttribute
+            const attrNode = parent?.type === 'JSXAttribute'
+                ? parent
+                : (parent?.type === 'JSXExpressionContainer' && grandparent?.type === 'JSXAttribute' ? grandparent : null);
+            if (attrNode && !insideIgnored) {
+                const rawAttrName = extractAttrName(attrNode.name);
                 const attrNameLower = rawAttrName ? String(rawAttrName).toLowerCase() : null;
                 // Check tag-level acceptance if acceptedTagsSet provided: attributes should only be considered
-                // when the nearest enclosing element is accepted.
-                const parentElement = currentAncestors.slice(0, -2).reverse().find(a => a && typeof a === 'object' && (a.type === 'JSXElement' || a.type === 'JSXOpeningElement' || a.type === 'JSXSelfClosingElement'));
+                // when the nearest enclosing element is accepted. Use the ancestors above the attrNode.
+                const attrNodeIdx = currentAncestors.indexOf(attrNode);
+                const parentElement = currentAncestors.slice(0, attrNodeIdx).reverse().find(a => a && typeof a === 'object' && (a.type === 'JSXElement' || a.type === 'JSXOpeningElement' || a.type === 'JSXSelfClosingElement'));
                 if (acceptedTagsSet && parentElement) {
                     const parentName = extractJSXName(parentElement);
                     if (!parentName || !acceptedTagsSet.has(String(parentName).toLowerCase())) {
@@ -786,6 +794,21 @@ function findHardcodedStrings(ast, code, config) {
                     }
                 }
             }
+            // Hardcoded string inside a JSX expression container used as element child:
+            //   <tag>{"hello"}</tag>   → parent is JSXExpressionContainer, grandparent is JSXElement/JSXFragment
+            // Apply the same filters used for JSXText so this behaves like raw text.
+            const isJsxChildExpression = parent?.type === 'JSXExpressionContainer' &&
+                (grandparent?.type === 'JSXElement' || grandparent?.type === 'JSXFragment');
+            if (isJsxChildExpression && !insideIgnored) {
+                // Respect attribute-only mode: when acceptedAttributes is set without acceptedTags,
+                // only attribute strings are linted — skip JSX child text.
+                if (!(acceptedAttributesSet && !acceptedTagsSet)) {
+                    const text = node.value.trim();
+                    if (text && text.length > 1 && text !== '...' && !isUrlOrPath(text) && isNaN(Number(text)) && !text.startsWith('{{')) {
+                        nodesToLint.push(node);
+                    }
+                }
+            }
         }
         // Recurse into children
         for (const key of Object.keys(node)) {

package/dist/cjs/status.js

@@ -10,6 +10,7 @@ require('glob');
 var nestedObject = require('./utils/nested-object.js');
 var fileUtils = require('./utils/file-utils.js');
 var pluralRules = require('./utils/plural-rules.js');
+var nesting = require('./utils/nesting.js');
 var contextVariants = require('./utils/context-variants.js');
 var funnelMsgTracker = require('./utils/funnel-msg-tracker.js');
 require('./extractor/parsers/jsx-parser.js');
@@ -107,13 +108,23 @@ async function generateStatusReport(config) {
         keysByNs,
         locales: new Map(),
     };
-    // Discover context variants that live in the primary translation file but
-    // are not directly extracted as keys (see issue #243). When source code uses
-    // a dynamic context value like `t('exportType', { context: type })`, the
-    // extractor can only tag the base key as "accepting context"; the actual
-    // context values (`_gas`, `_water`, ...) are only visible in the primary
-    // translation file. Without this scan, status never checks those variants
-    // for translation gaps in secondary locales.
+    // Build per-namespace "virtual" key lists for translation entries that the
+    // AST-based extractor cannot see on its own. Both inputs come from the
+    // primary translation file:
+    //
+    //  1. Context variants of an accepting-context key (see issue #243).
+    //     `t('exportType', { context: dynamic })` only registers the base key;
+    //     the concrete `exportType_gas` / `exportType_water` variants live in
+    //     the primary file.
+    //
+    //  2. Keys reachable only via `$t(...)` nested references from inside an
+    //     existing translation value (see follow-up to issue #241).
+    //     `"girlsAndBoys": "... $t(boys, {\"count\": x}) ..."` doesn't appear
+    //     in source code, yet the referenced keys (`boys`, plus per-locale
+    //     plural forms) must be checked in every secondary locale.
+    //
+    // Both scans need the primary translation file per namespace, so the load
+    // is shared.
     const keysAcceptingContext = new Set();
     for (const keys of keysByNs.values()) {
         for (const k of keys) {
@@ -122,14 +133,61 @@ async function generateStatusReport(config) {
         }
     }
     const contextVariantsByNs = new Map();
-    if (keysAcceptingContext.size > 0) {
-        const primaryMerged = mergeNamespaces
-            ? ((await fileUtils.loadTranslationFile(node_path.resolve(process.cwd(), fileUtils.getOutputPath(config.extract.output, primaryLanguage, (defaultNS === false ? 'translation' : (defaultNS || 'translation')))))) || {})
-            : null;
-        for (const ns of keysByNs.keys()) {
-            const primaryNsTranslations = mergeNamespaces
-                ? (primaryMerged?.[ns] ?? primaryMerged ?? {})
-                : ((await fileUtils.loadTranslationFile(node_path.resolve(process.cwd(), fileUtils.getOutputPath(config.extract.output, primaryLanguage, ns)))) || {});
+    const nestedReferenceKeysByNs = new Map();
+    const primaryMergedForScan = mergeNamespaces
+        ? ((await fileUtils.loadTranslationFile(node_path.resolve(process.cwd(), fileUtils.getOutputPath(config.extract.output, primaryLanguage, (defaultNS === false ? 'translation' : (defaultNS || 'translation')))))) || {})
+        : null;
+    const collectNestedRefsFromValue = (value, refNs, bucket, seen) => {
+        if (typeof value === 'string') {
+            if (seen.has(value))
+                return;
+            seen.add(value);
+            const refs = nesting.parseNestedReferences(value, {
+                nestingPrefix: config.extract.nestingPrefix,
+                nestingSuffix: config.extract.nestingSuffix,
+                nestingOptionsSeparator: config.extract.nestingOptionsSeparator,
+                nsSeparator: config.extract.nsSeparator,
+                defaultNS: config.extract.defaultNS
+            });
+            for (const ref of refs) {
+                // References with an explicit namespace that differs from the current
+                // bucket are ignored — they belong to another namespace's scan.
+                const normalizedRefNs = ref.ns === undefined || ref.ns === null
+                    ? (config.extract.defaultNS ?? 'translation')
+                    : ref.ns;
+                if (normalizedRefNs !== refNs)
+                    continue;
+                if (ref.context !== undefined) {
+                    const ctxKey = `${ref.key}${contextSeparator}${ref.context}`;
+                    if (ref.hasCount) {
+                        // Treat `key_ctx` as a base plural key; the per-locale loop
+                        // expands it into the correct CLDR forms for each target locale.
+                        bucket.push({ key: ctxKey, hasCount: true });
+                    }
+                    else {
+                        bucket.push({ key: ref.key });
+                        bucket.push({ key: ctxKey });
+                    }
+                }
+                else if (ref.hasCount) {
+                    bucket.push({ key: ref.key, hasCount: true });
+                }
+                else {
+                    bucket.push({ key: ref.key });
+                }
+            }
+        }
+        else if (value && typeof value === 'object' && !Array.isArray(value)) {
+            for (const v of Object.values(value)) {
+                collectNestedRefsFromValue(v, refNs, bucket, seen);
+            }
+        }
+    };
+    for (const ns of keysByNs.keys()) {
+        const primaryNsTranslations = mergeNamespaces
+            ? (primaryMergedForScan?.[ns] ?? primaryMergedForScan ?? {})
+            : ((await fileUtils.loadTranslationFile(node_path.resolve(process.cwd(), fileUtils.getOutputPath(config.extract.output, primaryLanguage, ns)))) || {});
+        if (keysAcceptingContext.size > 0) {
             const primaryKeys = nestedObject.getNestedKeys(primaryNsTranslations, keySeparator ?? '.');
             const variants = [];
             for (const primaryKey of primaryKeys) {
@@ -140,6 +198,10 @@ async function generateStatusReport(config) {
             if (variants.length > 0)
                 contextVariantsByNs.set(ns, variants);
         }
+        const nestedRefKeys = [];
+        collectNestedRefsFromValue(primaryNsTranslations, ns, nestedRefKeys, new Set());
+        if (nestedRefKeys.length > 0)
+            nestedReferenceKeysByNs.set(ns, nestedRefKeys);
     }
     for (const locale of secondaryLanguages) {
         let totalTranslatedForLocale = 0;
@@ -209,7 +271,14 @@ async function generateStatusReport(config) {
                 return primaryState;
             };
             const processedKeys = new Set();
-            for (const { key: baseKey, hasCount, isOrdinal, isExpandedPlural } of keysInNs) {
+            // Combine AST-extracted keys with nested-reference keys discovered in
+            // the primary translation file (see follow-up on issue #241). Both go
+            // through the same plural-expansion logic; processedKeys dedupes.
+            const nestedRefKeys = nestedReferenceKeysByNs.get(ns) || [];
+            const combinedKeysInNs = nestedRefKeys.length > 0
+                ? [...keysInNs, ...nestedRefKeys]
+                : keysInNs;
+            for (const { key: baseKey, hasCount, isOrdinal, isExpandedPlural } of combinedKeysInNs) {
                 if (hasCount) {
                     if (isExpandedPlural) {
                         // This is an already-expanded plural variant key (e.g., key_one, key_other)

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.56.0'); // This string is replaced with the actual version at build time by rollup
+    .version('1.56.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

@@ -753,14 +753,22 @@ function findHardcodedStrings(ast, code, config) {
         }
         if (node.type === 'StringLiteral') {
             const parent = currentAncestors[currentAncestors.length - 2];
+            const grandparent = currentAncestors[currentAncestors.length - 3];
             // Determine whether this attribute is inside any ignored element (handles nested Trans etc.)
             const insideIgnored = isWithinIgnoredElement(currentAncestors);
-            if (parent?.type === 'JSXAttribute' && !insideIgnored) {
-                const rawAttrName = extractAttrName(parent.name);
+            // A StringLiteral can be an attribute value in two forms:
+            //   <tag attr="value" />       → parent is JSXAttribute
+            //   <tag attr={"value"} />     → parent is JSXExpressionContainer, grandparent is JSXAttribute
+            const attrNode = parent?.type === 'JSXAttribute'
+                ? parent
+                : (parent?.type === 'JSXExpressionContainer' && grandparent?.type === 'JSXAttribute' ? grandparent : null);
+            if (attrNode && !insideIgnored) {
+                const rawAttrName = extractAttrName(attrNode.name);
                 const attrNameLower = rawAttrName ? String(rawAttrName).toLowerCase() : null;
                 // Check tag-level acceptance if acceptedTagsSet provided: attributes should only be considered
-                // when the nearest enclosing element is accepted.
-                const parentElement = currentAncestors.slice(0, -2).reverse().find(a => a && typeof a === 'object' && (a.type === 'JSXElement' || a.type === 'JSXOpeningElement' || a.type === 'JSXSelfClosingElement'));
+                // when the nearest enclosing element is accepted. Use the ancestors above the attrNode.
+                const attrNodeIdx = currentAncestors.indexOf(attrNode);
+                const parentElement = currentAncestors.slice(0, attrNodeIdx).reverse().find(a => a && typeof a === 'object' && (a.type === 'JSXElement' || a.type === 'JSXOpeningElement' || a.type === 'JSXSelfClosingElement'));
                 if (acceptedTagsSet && parentElement) {
                     const parentName = extractJSXName(parentElement);
                     if (!parentName || !acceptedTagsSet.has(String(parentName).toLowerCase())) {
@@ -784,6 +792,21 @@ function findHardcodedStrings(ast, code, config) {
                     }
                 }
             }
+            // Hardcoded string inside a JSX expression container used as element child:
+            //   <tag>{"hello"}</tag>   → parent is JSXExpressionContainer, grandparent is JSXElement/JSXFragment
+            // Apply the same filters used for JSXText so this behaves like raw text.
+            const isJsxChildExpression = parent?.type === 'JSXExpressionContainer' &&
+                (grandparent?.type === 'JSXElement' || grandparent?.type === 'JSXFragment');
+            if (isJsxChildExpression && !insideIgnored) {
+                // Respect attribute-only mode: when acceptedAttributes is set without acceptedTags,
+                // only attribute strings are linted — skip JSX child text.
+                if (!(acceptedAttributesSet && !acceptedTagsSet)) {
+                    const text = node.value.trim();
+                    if (text && text.length > 1 && text !== '...' && !isUrlOrPath(text) && isNaN(Number(text)) && !text.startsWith('{{')) {
+                        nodesToLint.push(node);
+                    }
+                }
+            }
         }
         // Recurse into children
         for (const key of Object.keys(node)) {

package/dist/esm/status.js

@@ -8,6 +8,7 @@ import 'glob';
 import { getNestedKeys, getNestedValue } from './utils/nested-object.js';
 import { loadTranslationFile, getOutputPath } from './utils/file-utils.js';
 import { safePluralRules } from './utils/plural-rules.js';
+import { parseNestedReferences } from './utils/nesting.js';
 import { isContextVariantOfAcceptingKey } from './utils/context-variants.js';
 import { shouldShowFunnel, recordFunnelShown } from './utils/funnel-msg-tracker.js';
 import './extractor/parsers/jsx-parser.js';
@@ -105,13 +106,23 @@ async function generateStatusReport(config) {
         keysByNs,
         locales: new Map(),
     };
-    // Discover context variants that live in the primary translation file but
-    // are not directly extracted as keys (see issue #243). When source code uses
-    // a dynamic context value like `t('exportType', { context: type })`, the
-    // extractor can only tag the base key as "accepting context"; the actual
-    // context values (`_gas`, `_water`, ...) are only visible in the primary
-    // translation file. Without this scan, status never checks those variants
-    // for translation gaps in secondary locales.
+    // Build per-namespace "virtual" key lists for translation entries that the
+    // AST-based extractor cannot see on its own. Both inputs come from the
+    // primary translation file:
+    //
+    //  1. Context variants of an accepting-context key (see issue #243).
+    //     `t('exportType', { context: dynamic })` only registers the base key;
+    //     the concrete `exportType_gas` / `exportType_water` variants live in
+    //     the primary file.
+    //
+    //  2. Keys reachable only via `$t(...)` nested references from inside an
+    //     existing translation value (see follow-up to issue #241).
+    //     `"girlsAndBoys": "... $t(boys, {\"count\": x}) ..."` doesn't appear
+    //     in source code, yet the referenced keys (`boys`, plus per-locale
+    //     plural forms) must be checked in every secondary locale.
+    //
+    // Both scans need the primary translation file per namespace, so the load
+    // is shared.
     const keysAcceptingContext = new Set();
     for (const keys of keysByNs.values()) {
         for (const k of keys) {
@@ -120,14 +131,61 @@ async function generateStatusReport(config) {
         }
     }
     const contextVariantsByNs = new Map();
-    if (keysAcceptingContext.size > 0) {
-        const primaryMerged = mergeNamespaces
-            ? ((await loadTranslationFile(resolve(process.cwd(), getOutputPath(config.extract.output, primaryLanguage, (defaultNS === false ? 'translation' : (defaultNS || 'translation')))))) || {})
-            : null;
-        for (const ns of keysByNs.keys()) {
-            const primaryNsTranslations = mergeNamespaces
-                ? (primaryMerged?.[ns] ?? primaryMerged ?? {})
-                : ((await loadTranslationFile(resolve(process.cwd(), getOutputPath(config.extract.output, primaryLanguage, ns)))) || {});
+    const nestedReferenceKeysByNs = new Map();
+    const primaryMergedForScan = mergeNamespaces
+        ? ((await loadTranslationFile(resolve(process.cwd(), getOutputPath(config.extract.output, primaryLanguage, (defaultNS === false ? 'translation' : (defaultNS || 'translation')))))) || {})
+        : null;
+    const collectNestedRefsFromValue = (value, refNs, bucket, seen) => {
+        if (typeof value === 'string') {
+            if (seen.has(value))
+                return;
+            seen.add(value);
+            const refs = parseNestedReferences(value, {
+                nestingPrefix: config.extract.nestingPrefix,
+                nestingSuffix: config.extract.nestingSuffix,
+                nestingOptionsSeparator: config.extract.nestingOptionsSeparator,
+                nsSeparator: config.extract.nsSeparator,
+                defaultNS: config.extract.defaultNS
+            });
+            for (const ref of refs) {
+                // References with an explicit namespace that differs from the current
+                // bucket are ignored — they belong to another namespace's scan.
+                const normalizedRefNs = ref.ns === undefined || ref.ns === null
+                    ? (config.extract.defaultNS ?? 'translation')
+                    : ref.ns;
+                if (normalizedRefNs !== refNs)
+                    continue;
+                if (ref.context !== undefined) {
+                    const ctxKey = `${ref.key}${contextSeparator}${ref.context}`;
+                    if (ref.hasCount) {
+                        // Treat `key_ctx` as a base plural key; the per-locale loop
+                        // expands it into the correct CLDR forms for each target locale.
+                        bucket.push({ key: ctxKey, hasCount: true });
+                    }
+                    else {
+                        bucket.push({ key: ref.key });
+                        bucket.push({ key: ctxKey });
+                    }
+                }
+                else if (ref.hasCount) {
+                    bucket.push({ key: ref.key, hasCount: true });
+                }
+                else {
+                    bucket.push({ key: ref.key });
+                }
+            }
+        }
+        else if (value && typeof value === 'object' && !Array.isArray(value)) {
+            for (const v of Object.values(value)) {
+                collectNestedRefsFromValue(v, refNs, bucket, seen);
+            }
+        }
+    };
+    for (const ns of keysByNs.keys()) {
+        const primaryNsTranslations = mergeNamespaces
+            ? (primaryMergedForScan?.[ns] ?? primaryMergedForScan ?? {})
+            : ((await loadTranslationFile(resolve(process.cwd(), getOutputPath(config.extract.output, primaryLanguage, ns)))) || {});
+        if (keysAcceptingContext.size > 0) {
             const primaryKeys = getNestedKeys(primaryNsTranslations, keySeparator ?? '.');
             const variants = [];
             for (const primaryKey of primaryKeys) {
@@ -138,6 +196,10 @@ async function generateStatusReport(config) {
             if (variants.length > 0)
                 contextVariantsByNs.set(ns, variants);
         }
+        const nestedRefKeys = [];
+        collectNestedRefsFromValue(primaryNsTranslations, ns, nestedRefKeys, new Set());
+        if (nestedRefKeys.length > 0)
+            nestedReferenceKeysByNs.set(ns, nestedRefKeys);
     }
     for (const locale of secondaryLanguages) {
         let totalTranslatedForLocale = 0;
@@ -207,7 +269,14 @@ async function generateStatusReport(config) {
                 return primaryState;
             };
             const processedKeys = new Set();
-            for (const { key: baseKey, hasCount, isOrdinal, isExpandedPlural } of keysInNs) {
+            // Combine AST-extracted keys with nested-reference keys discovered in
+            // the primary translation file (see follow-up on issue #241). Both go
+            // through the same plural-expansion logic; processedKeys dedupes.
+            const nestedRefKeys = nestedReferenceKeysByNs.get(ns) || [];
+            const combinedKeysInNs = nestedRefKeys.length > 0
+                ? [...keysInNs, ...nestedRefKeys]
+                : keysInNs;
+            for (const { key: baseKey, hasCount, isOrdinal, isExpandedPlural } of combinedKeysInNs) {
                 if (hasCount) {
                     if (isExpandedPlural) {
                         // This is an already-expanded plural variant key (e.g., key_one, key_other)

package/package.json

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

package/types/status.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"status.d.ts","sourceRoot":"","sources":["../src/status.ts"],"names":[],"mappings":"AAKA,OAAO,KAAK,EAAE,oBAAoB,EAAgB,MAAM,YAAY,CAAA;AAMpE;;GAEG;AACH,UAAU,aAAa;IACrB,0EAA0E;IAC1E,MAAM,CAAC,EAAE,MAAM,CAAC;IAChB,wCAAwC;IACxC,SAAS,CAAC,EAAE,MAAM,CAAC;IACnB,uEAAuE;IACvE,cAAc,CAAC,EAAE,OAAO,CAAC;CAC1B;AAqDD;;;;;;;;;;;;;;;;;;GAkBG;AACH,wBAAsB,SAAS,CAAE,MAAM,EAAE,oBAAoB,EAAE,OAAO,GAAE,aAAkB,iBAuBzF"}
+{"version":3,"file":"status.d.ts","sourceRoot":"","sources":["../src/status.ts"],"names":[],"mappings":"AAKA,OAAO,KAAK,EAAE,oBAAoB,EAAgB,MAAM,YAAY,CAAA;AAOpE;;GAEG;AACH,UAAU,aAAa;IACrB,0EAA0E;IAC1E,MAAM,CAAC,EAAE,MAAM,CAAC;IAChB,wCAAwC;IACxC,SAAS,CAAC,EAAE,MAAM,CAAC;IACnB,uEAAuE;IACvE,cAAc,CAAC,EAAE,OAAO,CAAC;CAC1B;AAqDD;;;;;;;;;;;;;;;;;;GAkBG;AACH,wBAAsB,SAAS,CAAE,MAAM,EAAE,oBAAoB,EAAE,OAAO,GAAE,aAAkB,iBAuBzF"}