Don't break when generating rules produces an error (#954)

* Use `node:util` to format LSP console messages

* Don’t break intellisense when generating rules throws an error for a single utility

Author: thecrypticace

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

@@ -19,26 +19,14 @@ import { Utils, URI } from 'vscode-uri'
 import { getLanguageModelCache } from './languageModelCache'
 import { Stylesheet } from 'vscode-css-languageservice'
 import dlv from 'dlv'
+import { interceptLogs } from '../util/logs'
 
 let connection = createConnection(ProposedFeatures.all)
 
-console.log = connection.console.log.bind(connection.console)
-console.error = connection.console.error.bind(connection.console)
-
-function formatError(message: string, err: any): string {
-  if (err instanceof Error) {
-    let error = <Error>err
-    return `${message}: ${error.message}\n${error.stack}`
-  } else if (typeof err === 'string') {
-    return `${message}: ${err}`
-  } else if (err) {
-    return `${message}: ${err.toString()}`
-  }
-  return message
-}
+interceptLogs(console, connection)
 
 process.on('unhandledRejection', (e: any) => {
-  connection.console.error(formatError(`Unhandled exception`, e))
+  console.error("Unhandled exception", e)
 })
 
 let documents: TextDocuments<TextDocument> = new TextDocuments(TextDocument)

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

@@ -1,9 +1,9 @@
 import './lib/env'
 import { createConnection } from 'vscode-languageserver/node'
-import { formatError } from './util/error'
 // @ts-ignore
 import preflight from 'tailwindcss/lib/css/preflight.css'
 import { TW } from './tw'
+import { interceptLogs } from './util/logs'
 
 // @ts-ignore
 // new Function(…) is used to work around issues with esbuild
@@ -25,11 +25,10 @@ new Function(
 const connection =
   process.argv.length <= 2 ? createConnection(process.stdin, process.stdout) : createConnection()
 
-console.log = connection.console.log.bind(connection.console)
-console.error = connection.console.error.bind(connection.console)
+interceptLogs(console, connection)
 
 process.on('unhandledRejection', (e: any) => {
-  connection.console.error(formatError(`Unhandled exception`, e))
+  console.error(`Unhandled rejection`, e)
 })
 
 const tw = new TW(connection)

packages/tailwindcss-language-server/src/util/logs.ts

@@ -0,0 +1,14 @@
+import type { Connection } from 'vscode-languageserver'
+import { format } from 'node:util'
+
+function formatForLogging(params: any[]): string {
+  return params.map((item) => format(item)).join(' ')
+}
+
+export function interceptLogs(console: Console, connection: Connection) {
+  console.debug = (...params: any[]) => connection.console.info(formatForLogging(params))
+  console.error = (...params: any[]) => connection.console.error(formatForLogging(params))
+  console.warn = (...params: any[]) => connection.console.warn(formatForLogging(params))
+  console.info = (...params: any[]) => connection.console.info(formatForLogging(params))
+  console.log = (...params: any[]) => connection.console.log(formatForLogging(params))
+}

packages/tailwindcss-language-service/src/util/color.ts

@@ -167,7 +167,16 @@ export function getColor(state: State, className: string): culori.Color | Keywor
       }
     }
 
-    let { root, rules } = jit.generateRules(state, [className])
+    let result!: ReturnType<typeof jit.generateRules>
+    try {
+      result = jit.generateRules(state, [className])
+    } catch (err) {
+      console.error(`Error generating rules for className: ${className}`)
+      console.error(err)
+      return null
+    }
+
+    let { root, rules } = result
     if (rules.length === 0) return null
 
     let decls: Record<string, string | string[]> = {}