@trackunit/eslint-plugin-trackunit 0.6.103 → 0.6.105

package/CHANGELOG.md

@@ -1,3 +1,17 @@
+## 0.6.105 (2026-08-13)
+
+### 🧱 Updated Dependencies
+
+- Updated css-classname-utils to 0.0.11
+- Updated shared-utils to 1.16.8
+
+## 0.6.104 (2026-08-13)
+
+### 🧱 Updated Dependencies
+
+- Updated css-classname-utils to 0.0.10
+- Updated shared-utils to 1.16.7
+
 ## 0.6.103 (2026-08-13)
 
 ### 🧱 Updated Dependencies

package/README.md

@@ -95,7 +95,7 @@ All custom rules are enabled automatically through the presets under the `@track
 | Rule                                    | Severity | Auto-fix | Description                                                                                                                            |
 | --------------------------------------- | -------- | -------- | ------------------------------------------------------------------------------------------------------------------------------------- |
 | `@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. |
+| `@trackunit/no-unused-translation-key`  | error    | suggest  | 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.103",
+  "version": "0.6.105",
   "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.9",
+    "@trackunit/css-classname-utils": "0.0.11",
     "@typescript-eslint/eslint-plugin": "8.58.1",
     "@typescript-eslint/utils": "8.58.1",
     "eslint-config-prettier": "^10.1.8",

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

@@ -41,7 +41,7 @@ export declare const configs: {
                     "no-template-strings-in-classname-prop": import("@typescript-eslint/utils/ts-eslint").RuleModule<"templateStringInClassName", [], unknown, import("@typescript-eslint/utils/ts-eslint").RuleListener> & {
                         name: string;
                     };
-                    "no-unused-translation-key": import("@typescript-eslint/utils/ts-eslint").RuleModule<"unusedKey", [], unknown, import("@typescript-eslint/utils/ts-eslint").RuleListener> & {
+                    "no-unused-translation-key": import("@typescript-eslint/utils/ts-eslint").RuleModule<"unusedKey" | "removeUnusedKey", [], unknown, import("@typescript-eslint/utils/ts-eslint").RuleListener> & {
                         name: string;
                     };
                     "require-classname-alternatives": import("@typescript-eslint/utils/ts-eslint").RuleModule<"bannedClassAutoFix" | "bannedClassSuggest" | "suggestReplacement", [({
@@ -245,7 +245,7 @@ export declare const configs: {
                     "no-template-strings-in-classname-prop": import("@typescript-eslint/utils/ts-eslint").RuleModule<"templateStringInClassName", [], unknown, import("@typescript-eslint/utils/ts-eslint").RuleListener> & {
                         name: string;
                     };
-                    "no-unused-translation-key": import("@typescript-eslint/utils/ts-eslint").RuleModule<"unusedKey", [], unknown, import("@typescript-eslint/utils/ts-eslint").RuleListener> & {
+                    "no-unused-translation-key": import("@typescript-eslint/utils/ts-eslint").RuleModule<"unusedKey" | "removeUnusedKey", [], unknown, import("@typescript-eslint/utils/ts-eslint").RuleListener> & {
                         name: string;
                     };
                     "require-classname-alternatives": import("@typescript-eslint/utils/ts-eslint").RuleModule<"bannedClassAutoFix" | "bannedClassSuggest" | "suggestReplacement", [({
@@ -365,7 +365,7 @@ export declare const configs: {
                     "no-template-strings-in-classname-prop": import("@typescript-eslint/utils/ts-eslint").RuleModule<"templateStringInClassName", [], unknown, import("@typescript-eslint/utils/ts-eslint").RuleListener> & {
                         name: string;
                     };
-                    "no-unused-translation-key": import("@typescript-eslint/utils/ts-eslint").RuleModule<"unusedKey", [], unknown, import("@typescript-eslint/utils/ts-eslint").RuleListener> & {
+                    "no-unused-translation-key": import("@typescript-eslint/utils/ts-eslint").RuleModule<"unusedKey" | "removeUnusedKey", [], unknown, import("@typescript-eslint/utils/ts-eslint").RuleListener> & {
                         name: string;
                     };
                     "require-classname-alternatives": import("@typescript-eslint/utils/ts-eslint").RuleModule<"bannedClassAutoFix" | "bannedClassSuggest" | "suggestReplacement", [({
@@ -501,7 +501,7 @@ export declare const configs: {
                     "no-template-strings-in-classname-prop": import("@typescript-eslint/utils/ts-eslint").RuleModule<"templateStringInClassName", [], unknown, import("@typescript-eslint/utils/ts-eslint").RuleListener> & {
                         name: string;
                     };
-                    "no-unused-translation-key": import("@typescript-eslint/utils/ts-eslint").RuleModule<"unusedKey", [], unknown, import("@typescript-eslint/utils/ts-eslint").RuleListener> & {
+                    "no-unused-translation-key": import("@typescript-eslint/utils/ts-eslint").RuleModule<"unusedKey" | "removeUnusedKey", [], unknown, import("@typescript-eslint/utils/ts-eslint").RuleListener> & {
                         name: string;
                     };
                     "require-classname-alternatives": import("@typescript-eslint/utils/ts-eslint").RuleModule<"bannedClassAutoFix" | "bannedClassSuggest" | "suggestReplacement", [({

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

@@ -59,7 +59,7 @@ export declare const localRulesPlugin: {
         "no-template-strings-in-classname-prop": import("@typescript-eslint/utils/ts-eslint").RuleModule<"templateStringInClassName", [], unknown, import("@typescript-eslint/utils/ts-eslint").RuleListener> & {
             name: string;
         };
-        "no-unused-translation-key": import("@typescript-eslint/utils/ts-eslint").RuleModule<"unusedKey", [], unknown, import("@typescript-eslint/utils/ts-eslint").RuleListener> & {
+        "no-unused-translation-key": import("@typescript-eslint/utils/ts-eslint").RuleModule<"unusedKey" | "removeUnusedKey", [], unknown, import("@typescript-eslint/utils/ts-eslint").RuleListener> & {
             name: string;
         };
         "require-classname-alternatives": import("@typescript-eslint/utils/ts-eslint").RuleModule<"bannedClassAutoFix" | "bannedClassSuggest" | "suggestReplacement", [({

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

@@ -41,7 +41,7 @@ export declare const base: (import("eslint").Linter.FlatConfig<import("eslint").
                 "no-template-strings-in-classname-prop": import("@typescript-eslint/utils/ts-eslint").RuleModule<"templateStringInClassName", [], unknown, import("@typescript-eslint/utils/ts-eslint").RuleListener> & {
                     name: string;
                 };
-                "no-unused-translation-key": import("@typescript-eslint/utils/ts-eslint").RuleModule<"unusedKey", [], unknown, import("@typescript-eslint/utils/ts-eslint").RuleListener> & {
+                "no-unused-translation-key": import("@typescript-eslint/utils/ts-eslint").RuleModule<"unusedKey" | "removeUnusedKey", [], unknown, import("@typescript-eslint/utils/ts-eslint").RuleListener> & {
                     name: string;
                 };
                 "require-classname-alternatives": import("@typescript-eslint/utils/ts-eslint").RuleModule<"bannedClassAutoFix" | "bannedClassSuggest" | "suggestReplacement", [({
@@ -245,7 +245,7 @@ export declare const base: (import("eslint").Linter.FlatConfig<import("eslint").
                 "no-template-strings-in-classname-prop": import("@typescript-eslint/utils/ts-eslint").RuleModule<"templateStringInClassName", [], unknown, import("@typescript-eslint/utils/ts-eslint").RuleListener> & {
                     name: string;
                 };
-                "no-unused-translation-key": import("@typescript-eslint/utils/ts-eslint").RuleModule<"unusedKey", [], unknown, import("@typescript-eslint/utils/ts-eslint").RuleListener> & {
+                "no-unused-translation-key": import("@typescript-eslint/utils/ts-eslint").RuleModule<"unusedKey" | "removeUnusedKey", [], unknown, import("@typescript-eslint/utils/ts-eslint").RuleListener> & {
                     name: string;
                 };
                 "require-classname-alternatives": import("@typescript-eslint/utils/ts-eslint").RuleModule<"bannedClassAutoFix" | "bannedClassSuggest" | "suggestReplacement", [({
@@ -365,7 +365,7 @@ export declare const base: (import("eslint").Linter.FlatConfig<import("eslint").
                 "no-template-strings-in-classname-prop": import("@typescript-eslint/utils/ts-eslint").RuleModule<"templateStringInClassName", [], unknown, import("@typescript-eslint/utils/ts-eslint").RuleListener> & {
                     name: string;
                 };
-                "no-unused-translation-key": import("@typescript-eslint/utils/ts-eslint").RuleModule<"unusedKey", [], unknown, import("@typescript-eslint/utils/ts-eslint").RuleListener> & {
+                "no-unused-translation-key": import("@typescript-eslint/utils/ts-eslint").RuleModule<"unusedKey" | "removeUnusedKey", [], unknown, import("@typescript-eslint/utils/ts-eslint").RuleListener> & {
                     name: string;
                 };
                 "require-classname-alternatives": import("@typescript-eslint/utils/ts-eslint").RuleModule<"bannedClassAutoFix" | "bannedClassSuggest" | "suggestReplacement", [({

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

@@ -135,7 +135,7 @@ exports.base = [
             "@trackunit": plugins_1.localRulesPlugin,
         },
         rules: {
-            "@trackunit/no-unused-translation-key": "warn",
+            "@trackunit/no-unused-translation-key": "error",
         },
     },
     jest_overrides_1.baseJestOverrides,

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

@@ -40,7 +40,7 @@ export declare const reactPreset: (import("eslint").Linter.Config<import("eslint
                 "no-template-strings-in-classname-prop": import("@typescript-eslint/utils/ts-eslint").RuleModule<"templateStringInClassName", [], unknown, import("@typescript-eslint/utils/ts-eslint").RuleListener> & {
                     name: string;
                 };
-                "no-unused-translation-key": import("@typescript-eslint/utils/ts-eslint").RuleModule<"unusedKey", [], unknown, import("@typescript-eslint/utils/ts-eslint").RuleListener> & {
+                "no-unused-translation-key": import("@typescript-eslint/utils/ts-eslint").RuleModule<"unusedKey" | "removeUnusedKey", [], unknown, import("@typescript-eslint/utils/ts-eslint").RuleListener> & {
                     name: string;
                 };
                 "require-classname-alternatives": import("@typescript-eslint/utils/ts-eslint").RuleModule<"bannedClassAutoFix" | "bannedClassSuggest" | "suggestReplacement", [({

package/src/lib/rules/no-unused-translation-key/no-unused-translation-key.d.ts

@@ -12,13 +12,16 @@
  * prefix, so keys behind one cannot be reported until the call site uses a
  * static key — what the companion `no-dynamic-translation-key` rule asks for.
  *
- * Matching is a heuristic over source text: this rule is `warn`-only and has no
- * fixer, so a key it reports is a candidate for review rather than for deletion.
+ * Matching is a heuristic over source text: this rule has no autofix, so a key
+ * it reports is still a candidate for review before applying the removal
+ * suggestion.
  *
  * Runs only on `**\/locales/en/translation.json` — the English file is the
  * source of truth; other languages are managed by Crowdin.
  */
 import { ESLintUtils } from "@typescript-eslint/utils";
-export declare const noUnusedTranslationKey: ESLintUtils.RuleModule<"unusedKey", [], unknown, ESLintUtils.RuleListener> & {
+type MessageIds = "unusedKey" | "removeUnusedKey";
+export declare const noUnusedTranslationKey: ESLintUtils.RuleModule<MessageIds, [], unknown, ESLintUtils.RuleListener> & {
     name: string;
 };
+export {};

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

@@ -16,8 +16,9 @@ const tslib_1 = require("tslib");
  * prefix, so keys behind one cannot be reported until the call site uses a
  * static key — what the companion `no-dynamic-translation-key` rule asks for.
  *
- * Matching is a heuristic over source text: this rule is `warn`-only and has no
- * fixer, so a key it reports is a candidate for review rather than for deletion.
+ * Matching is a heuristic over source text: this rule has no autofix, so a key
+ * it reports is still a candidate for review before applying the removal
+ * suggestion.
  *
  * Runs only on `**\/locales/en/translation.json` — the English file is the
  * source of truth; other languages are managed by Crowdin.
@@ -28,67 +29,78 @@ const file_utils_1 = require("../../utils/file-utils");
 const used_strings_1 = require("./used-strings");
 const createRule = utils_1.ESLintUtils.RuleCreator(name => `https://github.com/trackunit/manager/blob/main/libs/eslint/plugin-trackunit/src/lib/rules/${name}/${name}.ts`);
 const TRANSLATION_FILE_SUFFIX = "/locales/en/translation.json";
-const parseTranslationObject = (raw) => {
-    let parsed;
-    try {
-        parsed = JSON.parse(raw);
-    }
-    catch {
-        return null;
+const getKeyText = (key) => (key.type === "JSONIdentifier" ? key.name : String(key.value));
+/**
+ * The source range to drop so the remaining object stays valid JSON: a property
+ * with a following sibling reaches up to that sibling (taking its own separating
+ * comma and indentation with it), a trailing property reaches back to the end of
+ * the preceding one, and a sole property leaves an empty object behind.
+ */
+const getRemovalRange = (property) => {
+    const siblings = property.parent.properties;
+    const position = siblings.indexOf(property);
+    const nextSibling = siblings[position + 1];
+    const previousSibling = siblings[position - 1];
+    if (nextSibling) {
+        return [property.range[0], nextSibling.range[0]];
     }
-    if (typeof parsed !== "object" || parsed === null || Array.isArray(parsed)) {
-        return null;
+    if (previousSibling) {
+        return [previousSibling.range[1], property.range[1]];
     }
-    return { ...parsed };
+    return [property.parent.range[0] + 1, property.parent.range[1] - 1];
 };
 exports.noUnusedTranslationKey = createRule({
     name: "no-unused-translation-key",
     meta: {
         type: "suggestion",
+        hasSuggestions: true,
         docs: {
             description: "Report translation keys in a library's locales/en/translation.json that are never referenced in that library's source.",
         },
         messages: {
             unusedKey: 'Translation key "{{key}}" is not referenced anywhere in this library and can be removed.',
+            removeUnusedKey: 'Remove translation key "{{key}}".',
         },
         schema: [],
     },
     defaultOptions: [],
     create(context) {
+        const filePath = context.filename.replace(/\\/g, "/");
+        if (!filePath.endsWith(TRANSLATION_FILE_SUFFIX)) {
+            return {};
+        }
+        const projectJsonPath = (0, file_utils_1.findNearestFile)(context.filename, "project.json");
+        if (!projectJsonPath) {
+            return {};
+        }
+        // Scanning the whole library is expensive, so only pay for it once a key is actually inspected.
+        let usedStrings;
+        const getUsed = () => {
+            usedStrings ?? (usedStrings = (0, used_strings_1.getUsedTranslationStrings)(path.dirname(projectJsonPath)));
+            return usedStrings;
+        };
         return {
-            Program: () => {
-                const filePath = context.filename.replace(/\\/g, "/");
-                if (!filePath.endsWith(TRANSLATION_FILE_SUFFIX)) {
+            JSONProperty: (property) => {
+                // Translation files are flat; anything nested belongs to a value, not to a key.
+                if (property.parent.parent.type !== "JSONExpressionStatement") {
                     return;
                 }
-                const projectJsonPath = (0, file_utils_1.findNearestFile)(context.filename, "project.json");
-                if (!projectJsonPath) {
+                const key = getKeyText(property.key);
+                if ((0, used_strings_1.isTranslationKeyUsed)(getUsed(), key)) {
                     return;
                 }
-                const raw = context.sourceCode.getText();
-                const translations = parseTranslationObject(raw);
-                if (!translations) {
-                    return;
-                }
-                const used = (0, used_strings_1.getUsedTranslationStrings)(path.dirname(projectJsonPath));
-                const lines = raw.split("\n");
-                for (const key of Object.keys(translations)) {
-                    if ((0, used_strings_1.isTranslationKeyUsed)(used, key)) {
-                        continue;
-                    }
-                    const keyToken = `"${key}"`;
-                    const lineIndex = lines.findIndex(text => text.includes(`${keyToken}:`));
-                    const line = lineIndex >= 0 ? lineIndex + 1 : 1;
-                    const startColumn = lineIndex >= 0 ? Math.max(lines[lineIndex]?.indexOf(keyToken) ?? 0, 0) : 0;
-                    context.report({
-                        messageId: "unusedKey",
-                        data: { key },
-                        loc: {
-                            start: { line, column: startColumn },
-                            end: { line, column: startColumn + keyToken.length },
+                context.report({
+                    messageId: "unusedKey",
+                    data: { key },
+                    loc: property.key.loc,
+                    suggest: [
+                        {
+                            messageId: "removeUnusedKey",
+                            data: { key },
+                            fix: fixer => fixer.removeRange(getRemovalRange(property)),
                         },
-                    });
-                }
+                    ],
+                });
             },
         };
     },

package/src/lib/rules-map.d.ts

@@ -37,7 +37,7 @@ export declare const rulesMap: {
     "no-template-strings-in-classname-prop": ESLintUtils.RuleModule<"templateStringInClassName", [], unknown, ESLintUtils.RuleListener> & {
         name: string;
     };
-    "no-unused-translation-key": ESLintUtils.RuleModule<"unusedKey", [], unknown, ESLintUtils.RuleListener> & {
+    "no-unused-translation-key": ESLintUtils.RuleModule<"unusedKey" | "removeUnusedKey", [], unknown, ESLintUtils.RuleListener> & {
         name: string;
     };
     "require-classname-alternatives": ESLintUtils.RuleModule<"bannedClassAutoFix" | "bannedClassSuggest" | "suggestReplacement", [({