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

package/CHANGELOG.md

@@ -1,3 +1,26 @@
+## 0.6.102 (2026-08-13)
+
+### 🚀 Features
+
+- **react-components:** export non-hook localStorage accessors ([31ad79ac14d](https://github.com/Trackunit/manager/commit/31ad79ac14d))
+
+### 🧱 Updated Dependencies
+
+- Updated css-classname-utils to 0.0.8
+- Updated shared-utils to 1.16.5
+
+### ❤️ Thank You
+
+- Cursor @cursoragent
+- Mikkel Thorbjørn Andersen
+
+## 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.102",
   "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.8",
     "@typescript-eslint/eslint-plugin": "8.58.1",
     "@typescript-eslint/utils": "8.58.1",
     "eslint-config-prettier": "^10.1.8",

package/src/lib/config/fragments/no-direct-storage-access.js

@@ -43,6 +43,7 @@ const storageHookImplementationFiles = [
     "libs/react/components/src/hooks/localStorage/useLocalStorageReducer.ts",
     "libs/react/components/src/hooks/localStorage/useSessionStorage.ts",
     "libs/react/components/src/hooks/localStorage/useSessionStorageReducer.ts",
+    "libs/shared/utils/src/localStorage/localStorageAccessors.ts",
 ];
 exports.noDirectStoragePropertyExceptionOverrides = {
     basePath: (0, utils_1.findMonorepoRoot)(),

package/src/lib/config/index.d.ts

@@ -179,6 +179,7 @@ export declare const configs: {
             "no-console": string;
             "no-var": string;
             "prefer-const": string;
+            "prefer-template": string;
             "no-template-curly-in-string": string;
             eqeqeq: string[];
             "no-undef-init": string;

package/src/lib/config/presets/base.d.ts

@@ -179,6 +179,7 @@ export declare const base: (import("eslint").Linter.FlatConfig<import("eslint").
         "no-console": string;
         "no-var": string;
         "prefer-const": string;
+        "prefer-template": string;
         "no-template-curly-in-string": string;
         eqeqeq: string[];
         "no-undef-init": string;

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

@@ -93,6 +93,7 @@ exports.base = [
             "no-console": "error",
             "no-var": "error",
             "prefer-const": "error",
+            "prefer-template": "error",
             "no-template-curly-in-string": "error",
             eqeqeq: ["error", "always"],
             "no-undef-init": "error",
@@ -107,7 +108,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" });
                 }
             },
         };

package/src/lib/utils/import-utils.js

@@ -86,14 +86,14 @@ const addImportSpecifiers = (options) => {
         const lastImport = sourceCode.ast.body.find((node) => node.type === utils_1.AST_NODE_TYPES.ImportDeclaration);
         if (lastImport) {
             // Add after existing imports with leading newline
-            const importStatement = "\n" + buildImportStatement(sourceCode, packageName, specifiers, lastImport);
+            const importStatement = `\n${buildImportStatement(sourceCode, packageName, specifiers, lastImport)}`;
             fixes.push(fixer.insertTextAfter(lastImport, importStatement));
         }
         else {
             // No imports exist - add at the beginning with trailing newlines
             const firstNode = sourceCode.ast.body[0];
             if (firstNode) {
-                const importStatement = buildImportStatement(sourceCode, packageName, specifiers) + "\n\n";
+                const importStatement = `${buildImportStatement(sourceCode, packageName, specifiers)}\n\n`;
                 fixes.push(fixer.insertTextBefore(firstNode, importStatement));
             }
         }