Revert some changes not needed

Author: JPedersen

eslint.config.ts

@@ -1,62 +1,147 @@
-import js from "@eslint/js";
-import tseslint from "@typescript-eslint/eslint-plugin";
-import tsparser from "@typescript-eslint/parser";
-import react from "eslint-plugin-react";
-import reactHooks from "eslint-plugin-react-hooks";
-import importPlugin from "eslint-plugin-import";
-import type { Linter } from "eslint";
+import js from '@eslint/js';
+import tseslint from '@typescript-eslint/eslint-plugin';
+import tsparser from '@typescript-eslint/parser';
+// @ts-ignore - Plugin doesn't have proper ESLint 9 types yet
+import importPlugin from 'eslint-plugin-import';
+// @ts-ignore - Plugin doesn't have proper ESLint 9 types yet
+import react from 'eslint-plugin-react';
+// @ts-ignore - Plugin doesn't have proper ESLint 9 types yet
+import reactHooks from 'eslint-plugin-react-hooks';
+import prettierConfig from 'eslint-config-prettier';
 
 export default [
+  // Base recommended configurations
   js.configs.recommended,
+  
+  // TypeScript configuration for all TypeScript files
   {
-    files: ["**/*.{js,jsx,ts,tsx}"],
+    files: ['src/**/*.{ts,tsx}'],
     languageOptions: {
       parser: tsparser,
       parserOptions: {
-        ecmaVersion: "latest",
-        sourceType: "module",
+        ecmaVersion: 'latest',
+        sourceType: 'module',
+        project: './tsconfig.json',
         ecmaFeatures: {
           jsx: true,
         },
       },
       globals: {
-        // Node.js globals
-        process: "readonly",
-        console: "readonly",
-        // TypeScript/React globals
-        React: "readonly",
-        JSX: "readonly",
+        // Only keep React global for JSX without explicit React imports
+        React: 'writable',
       },
     },
     plugins: {
-      "@typescript-eslint": tseslint as any,
-      react: react as any,
-      "react-hooks": reactHooks as any,
-      import: importPlugin as any,
+      '@typescript-eslint': tseslint,
     },
     rules: {
+      // Disable core ESLint rules that are covered by TypeScript
+      'no-unused-vars': 'off',
+      'no-undef': 'off', // TypeScript handles this
+      
+      // Apply TypeScript ESLint recommended rules
       ...tseslint.configs.recommended.rules,
-      ...react.configs.recommended.rules,
-      ...reactHooks.configs.recommended.rules,
-      "react/react-in-jsx-scope": "off",
-      "react/prop-types": "off",
-      "@typescript-eslint/no-unused-vars": ["error", { argsIgnorePattern: "^_" }],
-      "@typescript-eslint/explicit-function-return-type": "off",
-      "@typescript-eslint/explicit-module-boundary-types": "off",
-      "@typescript-eslint/no-explicit-any": "warn",
-      // Allow regex escape characters
-      "no-useless-escape": "off",
+      // Override specific rules for our project needs
+      '@typescript-eslint/ban-ts-comment': 'off',
+      '@typescript-eslint/no-explicit-any': 'warn',
     },
-    settings: {
-      react: {
-        version: "detect",
+  },
+
+  // Node.js CLI configuration
+  {
+    files: ['src/cli/**/*.{ts,tsx}'],
+    languageOptions: {
+      parser: tsparser,
+      parserOptions: {
+        ecmaVersion: 'latest',
+        sourceType: 'module',
+        project: './tsconfig.json',
       },
-      "import/resolver": {
-        typescript: {},
+      globals: {
+        // Only the Node.js globals actually used in CLI files
+        process: 'readonly',
+        console: 'readonly',
       },
     },
+    plugins: {
+      '@typescript-eslint': tseslint,
+    },
+    rules: {
+      'no-unused-vars': 'off',
+      'no-undef': 'off',
+      ...tseslint.configs.recommended.rules,
+      '@typescript-eslint/ban-ts-comment': 'off',
+      '@typescript-eslint/no-explicit-any': 'warn',
+    },
   },
+  
+  // React configuration
   {
-    ignores: ["lib/**", "node_modules/**", "coverage/**"],
+    files: ['src/**/*.{js,jsx,ts,tsx}'],
+    languageOptions: {
+      parser: tsparser,
+      parserOptions: {
+        ecmaVersion: 'latest',
+        sourceType: 'module',
+        project: './tsconfig.json',
+        ecmaFeatures: {
+          jsx: true,
+        },
+      },
+      globals: {
+        React: 'writable',
+      },
+    },
+    plugins: {
+      react,
+      'react-hooks': reactHooks,
+      import: importPlugin,
+    },
+    settings: {
+      react: {
+        version: 'detect',
+      },
+      'import/resolver': {
+        typescript: {
+          project: './tsconfig.json',
+        },
+        node: {
+          extensions: ['.js', '.jsx', '.ts', '.tsx'],
+        },
+      },
+    },
+    rules: {
+      // Apply React recommended rules
+      ...react.configs.recommended.rules,
+      ...react.configs['jsx-runtime'].rules,
+      
+      // Apply React Hooks recommended rules
+      ...reactHooks.configs.recommended.rules,
+      
+      // Apply Import recommended rules with custom order
+      'import/no-unresolved': 'off', // TypeScript handles this
+      'import/order': [
+        'error',
+        {
+          groups: ['builtin', 'external', 'internal'],
+          pathGroups: [
+            {
+              pattern: 'react',
+              group: 'external',
+              position: 'before',
+            },
+          ],
+          pathGroupsExcludedImportTypes: ['react'],
+          'newlines-between': 'always',
+          alphabetize: {
+            order: 'asc',
+            caseInsensitive: true,
+          },
+        },
+      ],
+    },
   },
-];
+  
+  // Prettier configuration (disables conflicting rules)
+  prettierConfig,
+];

src/cli/utils.ts

@@ -24,7 +24,7 @@ export const extractStyles = (path: string) => {
       : fs.readFileSync(path).toString();
 
   return fileContent.match(
-    /([a-zA-Z_]*)(?:[.]{1})([a-zA-Z_]+[\w\-_]*)(?:[\s\.\,\{\>#\:]{0})/gim,
+    /([a-zA-Z_]*)(?:[.]{1})([a-zA-Z_]+[\w\-_]*)(?:[\s.,{>#:]{0})/gim,
   );
 };