@trackunit/eslint-plugin-trackunit 0.6.101-alpha-d0f54ff2967.0 → 0.6.101

package/CHANGELOG.md

@@ -1,3 +1,10 @@
+## 0.6.101 (2026-08-12)
+
+### 🧱 Updated Dependencies
+
+- Updated css-classname-utils to 0.0.7
+- Updated shared-utils to 1.16.4
+
 ## 0.6.100 (2026-08-11)
 
 ### 🩹 Fixes

package/README.md

@@ -94,7 +94,7 @@ All custom rules are enabled automatically through the presets under the `@track
 
 | Rule                                    | Severity | Auto-fix | Description                                                                                                                            |
 | --------------------------------------- | -------- | -------- | ------------------------------------------------------------------------------------------------------------------------------------- |
-| `@trackunit/no-dynamic-translation-key` | warn     | —        | Discourages building i18next keys from template strings (`t(`a.${x}`)`). Map values to explicit keys with a `switch`/`Record`.        |
+| `@trackunit/no-dynamic-translation-key` | error    | —        | Discourages building i18next keys from template strings (`t(`a.${x}`)`). Map values to explicit keys with a `switch`/`Record`.        |
 | `@trackunit/no-unused-translation-key`  | warn     | —        | Reports keys in a library's `locales/en/translation.json` that are never referenced in that library's source. |
 
 ## Utilities

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@trackunit/eslint-plugin-trackunit",
-  "version": "0.6.101-alpha-d0f54ff2967.0",
+  "version": "0.6.101",
   "license": "SEE LICENSE IN LICENSE.txt",
   "repository": "https://github.com/Trackunit/manager",
   "engines": {
@@ -8,7 +8,7 @@
   },
   "dependencies": {
     "@nx/eslint-plugin": "23.1.0",
-    "@trackunit/css-classname-utils": "0.0.7-alpha-d0f54ff2967.0",
+    "@trackunit/css-classname-utils": "0.0.7",
     "@typescript-eslint/eslint-plugin": "8.58.1",
     "@typescript-eslint/utils": "8.58.1",
     "eslint-config-prettier": "^10.1.8",

package/src/lib/config/presets/base.js

@@ -107,7 +107,7 @@ exports.base = [
             "no-restricted-globals": no_direct_storage_access_1.noDirectStorageGlobalsRule,
             "@trackunit/no-internal-barrel-files": "error",
             "@trackunit/no-typescript-assertion": "warn",
-            "@trackunit/no-dynamic-translation-key": "warn",
+            "@trackunit/no-dynamic-translation-key": "error",
             ...import_rules_1.importRules,
             ...typescript_rules_1.typescriptRules,
         },

package/src/lib/rules/no-dynamic-translation-key/no-dynamic-translation-key.js

@@ -25,6 +25,26 @@ const translation_key_types_1 = require("../../utils/translation-key-types");
 const createRule = utils_1.ESLintUtils.RuleCreator(name => `https://github.com/trackunit/manager/blob/main/libs/eslint/plugin-trackunit/src/lib/rules/${name}/${name}.ts`);
 /** A template literal that actually interpolates a value, e.g. `foo.${bar}` (not a plain `foo`). */
 const isInterpolatedTemplate = (node) => node.type === utils_1.AST_NODE_TYPES.TemplateLiteral && node.expressions.length > 0;
+/**
+ * Strips `as`/`satisfies`/non-null/`<T>`-assertion wrappers to reach the underlying expression.
+ *
+ * `` t(`a.${x}` as never) `` parses as a `TSAsExpression` whose `.expression` is the real
+ * `TemplateLiteral`, so without this the wrapper silently hides the dynamic key from the rule.
+ * That is an escape hatch we must not offer: casting a widened interpolation is exactly what this
+ * rule exists to forbid, and the sibling `no-unused-translation-key` already unwraps the same
+ * wrappers (see its `expand-dynamic-key.ts#unwrapToTemplateExpression`), so leaving them here means
+ * one rule sees the key and the other does not.
+ */
+const unwrapTypeWrappers = (node) => {
+    let current = node;
+    while (current.type === utils_1.AST_NODE_TYPES.TSAsExpression ||
+        current.type === utils_1.AST_NODE_TYPES.TSSatisfiesExpression ||
+        current.type === utils_1.AST_NODE_TYPES.TSNonNullExpression ||
+        current.type === utils_1.AST_NODE_TYPES.TSTypeAssertion) {
+        current = current.expression;
+    }
+    return current;
+};
 /**
  * Whether a template literal's interpolations should be flagged: true unless every
  * interpolated expression's TypeScript type is a finite string-literal union (in which
@@ -75,8 +95,12 @@ exports.noDynamicTranslationKey = createRule({
                     return;
                 }
                 const [firstArg] = node.arguments;
-                if (firstArg && isInterpolatedTemplate(firstArg) && hasWidenedInterpolation(context, firstArg)) {
-                    context.report({ node: firstArg, messageId: "dynamicKey" });
+                if (!firstArg) {
+                    return;
+                }
+                const key = unwrapTypeWrappers(firstArg);
+                if (isInterpolatedTemplate(key) && hasWidenedInterpolation(context, key)) {
+                    context.report({ node: key, messageId: "dynamicKey" });
                 }
             },
             JSXAttribute(node) {
@@ -84,10 +108,12 @@ exports.noDynamicTranslationKey = createRule({
                     return;
                 }
                 const { value } = node;
-                if (value?.type === utils_1.AST_NODE_TYPES.JSXExpressionContainer &&
-                    isInterpolatedTemplate(value.expression) &&
-                    hasWidenedInterpolation(context, value.expression)) {
-                    context.report({ node: value.expression, messageId: "dynamicKey" });
+                if (value?.type !== utils_1.AST_NODE_TYPES.JSXExpressionContainer) {
+                    return;
+                }
+                const key = unwrapTypeWrappers(value.expression);
+                if (isInterpolatedTemplate(key) && hasWidenedInterpolation(context, key)) {
+                    context.report({ node: key, messageId: "dynamicKey" });
                 }
             },
         };