feat(no-unused-class-name): added support for class directives

Author: marekdedic

src/rules/no-unused-class-name.ts

@@ -3,7 +3,6 @@ import type {
   SourceLocation,
   SvelteAttribute,
   SvelteDirective,
-  SvelteLiteral,
   SvelteShorthandAttribute,
   SvelteSpecialDirective,
   SvelteSpreadAttribute,
@@ -74,20 +73,15 @@ function findClassesInAttribute(
     | SvelteStyleDirective
     | SvelteSpecialDirective,
 ): string[] {
-  // TODO: This only supports direct class attribute - what about shorthands, spread directives etc.
-  if (
-    !("key" in attribute) ||
-    !("name" in attribute.key) ||
-    !("value" in attribute) ||
-    attribute.key.name !== "class"
-  ) {
-    return []
+  if (attribute.type === "SvelteAttribute" && attribute.key.name === "class") {
+    return attribute.value.flatMap((value) =>
+      value.type === "SvelteLiteral" ? value.value.split(" ") : [],
+    )
+  }
+  if (attribute.type === "SvelteDirective" && attribute.kind === "Class") {
+    return [attribute.key.name.name]
   }
-  // TODO: Why multiple values?
-  // TODO: Remove assertions
-  return ((attribute as SvelteAttribute).value[0] as SvelteLiteral).value.split(
-    " ",
-  )
+  return []
 }
 
 /**

tests/fixtures/rules/no-unused-class-name/invalid/class-directive01-errors.yaml

@@ -0,0 +1,8 @@
+- message: Unused class "first".
+  line: 1
+  column: 1
+  suggestions: null
+- message: Unused class "second".
+  line: 3
+  column: 1
+  suggestions: null

tests/fixtures/rules/no-unused-class-name/invalid/class-directive01-input.svelte

@@ -0,0 +1,3 @@
+<div class:first={true}>Hello</div>
+
+<span class:second={false}>World!</span>