Refactor CSS class scanning and hover tests

- Removed the CSS class scanning logic from the project service to streamline functionality.
- Deleted outdated hover tests for custom classes to improve test clarity and maintainability.
- Enhanced error handling and logging during project directory scanning for CSS files.

Author: felix-droese

packages/tailwindcss-language-server/src/projects.ts

@@ -24,7 +24,6 @@ import { URI } from 'vscode-uri'
 import { showError, showWarning, SilentError } from './util/error'
 import * as path from 'node:path'
 import * as fs from 'node:fs'
-import * as fsPromises from 'node:fs/promises'
 import findUp from 'find-up'
 import picomatch from 'picomatch'
 import { resolveFrom, setPnpApi } from './util/resolveFrom'
@@ -57,7 +56,6 @@ import { doCodeActions } from '@tailwindcss/language-service/src/codeActions/cod
 import { getDocumentColors } from '@tailwindcss/language-service/src/documentColorProvider'
 import { getDocumentLinks } from '@tailwindcss/language-service/src/documentLinksProvider'
 import { debounce } from 'debounce'
-import { scanCssFilesForCustomClasses } from '@tailwindcss/language-service/src/util/css-class-scanner'
 import { getModuleDependencies } from './util/getModuleDependencies'
 import assert from 'node:assert'
 // import postcssLoadConfig from 'postcss-load-config'
@@ -1071,40 +1069,6 @@ export async function createProjectService(
     }
     state.variants = getVariants(state)
 
-    // Scan CSS files for custom classes (v3 projects)
-    const cssFiles = Array.from(state.dependencies ?? []).filter((dep) => dep.endsWith('.css'))
-
-    // Also scan the main CSS file if it's a CSS config
-    if (state.isCssConfig && state.configPath) {
-      cssFiles.push(state.configPath)
-    }
-
-    // Also scan CSS files in the project directory for custom classes
-    try {
-      const projectDir = path.dirname(state.configPath)
-      const projectFiles = await fsPromises.readdir(projectDir)
-      const projectCssFiles = projectFiles
-        .filter((file) => file.endsWith('.css'))
-        .map((file) => path.join(projectDir, file))
-
-      // Add project CSS files that aren't already in dependencies
-      for (const cssFile of projectCssFiles) {
-        if (!cssFiles.includes(cssFile)) {
-          cssFiles.push(cssFile)
-        }
-      }
-    } catch (error) {
-      console.error('Error scanning project directory for CSS files:', error)
-    }
-
-    if (cssFiles.length > 0) {
-      try {
-        await scanCssFilesForCustomClasses(state, cssFiles)
-      } catch (error) {
-        console.error('Error scanning CSS files for custom classes:', error)
-      }
-    }
-
     let screens = dlv(state.config, 'theme.screens', dlv(state.config, 'screens', {}))
     state.screens = isObject(screens) ? Object.keys(screens) : []
 
@@ -1186,20 +1150,7 @@ export async function createProjectService(
       state.variants = getVariants(state)
       state.blocklist = Array.from(designSystem.invalidCandidates ?? [])
 
-      // Scan CSS files for custom classes
       let deps = designSystem.dependencies()
-      const cssFiles = Array.from(deps).filter((dep) => dep.endsWith('.css'))
-
-      // Also scan the main CSS file
-      cssFiles.push(state.configPath)
-
-      if (cssFiles.length > 0) {
-        try {
-          await scanCssFilesForCustomClasses(state, cssFiles)
-        } catch (error) {
-          console.error('Error scanning CSS files for custom classes:', error)
-        }
-      }
 
       for (let dep of deps) {
         dependencies.add(dep)

packages/tailwindcss-language-server/tests/hover/hover.test.js

@@ -554,84 +554,6 @@ withFixture('v4/path-mappings', (c) => {
   })
 })
 
-withFixture('custom-classes', (c) => {
-  async function testHover(
-    name,
-    { text, lang, position, exact = false, expected, expectedRange, settings },
-  ) {
-    test(name, async ({ expect }) => {
-      let textDocument = await c.openDocument({ text, lang, settings })
-      let res = await c.sendRequest('textDocument/hover', {
-        textDocument,
-        position,
-      })
-
-      if (!exact && expected) {
-        expected = {
-          contents: {
-            language: 'css',
-            value: expected,
-          },
-          range: expectedRange,
-        }
-      }
-
-      expect(res).toEqual(expected)
-    })
-  }
-
-  testHover('custom button class', {
-    text: '<div class="custom-button">',
-    position: { line: 0, character: 13 },
-    expected:
-      '.custom-button {\n' +
-      '  background-color: #1a9dd9;\n' +
-      '  color: white;\n' +
-      '  padding: 0.5rem 1rem;\n' +
-      '  border-radius: 0.25rem;\n' +
-      '  font-weight: 600;\n' +
-      '}',
-    expectedRange: {
-      start: { line: 0, character: 12 },
-      end: { line: 0, character: 25 },
-    },
-  })
-
-  testHover('typography h3 class', {
-    text: '<div class="typography-h3">',
-    position: { line: 0, character: 13 },
-    expected:
-      '.typography-h3 {\n' +
-      '  font-family: Montserrat;\n' +
-      '  font-size: 48px;\n' +
-      '  font-style: normal;\n' +
-      '  font-weight: 700;\n' +
-      '  line-height: 116.7%;\n' +
-      '}',
-    expectedRange: {
-      start: { line: 0, character: 12 },
-      end: { line: 0, character: 25 },
-    },
-  })
-
-  testHover('card class', {
-    text: '<div class="card">',
-    position: { line: 0, character: 13 },
-    expected:
-      '.card {\n' +
-      '  background-color: white;\n' +
-      '  border: 1px solid #e5e7eb;\n' +
-      '  border-radius: 0.5rem;\n' +
-      '  padding: 1.5rem;\n' +
-      '  box-shadow: 0 1px 3px 0 rgba(0, 0, 0, 0.1);\n' +
-      '}',
-    expectedRange: {
-      start: { line: 0, character: 12 },
-      end: { line: 0, character: 16 },
-    },
-  })
-})
-
 defineTest({
   name: 'Can hover showing theme values used in var(…) and theme(…) functions',
   fs: {