js-style-kit 0.5.3 → 0.6.1

package/dist/index.js

@@ -17,6 +17,7 @@ var configNames = {
   nextjs: "nextjs",
   perfectionist: "perfectionist",
   preferArrowFunction: "prefer-arrow-function",
+  query: "@tanstack/query",
   react: "react",
   reactRefresh: "react-refresh",
   storybook: "storybook:stories",
@@ -28,6 +29,7 @@ var configNames = {
   unicorn: "unicorn"
 };
 var pluginPrefixMap = /* @__PURE__ */ new Map([
+  ["@tanstack/query", configNames.query],
   ["@typescript-eslint", configNames.typescript],
   ["import", configNames.import],
   ["import-x", configNames.import],
@@ -827,6 +829,32 @@ var processCustomRules = (customRules) => {
   }, {});
 };
 
+// src/eslint/query/config.ts
+import queryPlugin from "@tanstack/eslint-plugin-query";
+
+// src/eslint/query/rules.ts
+var queryRules = {
+  "@tanstack/query/exhaustive-deps": "warn",
+  "@tanstack/query/infinite-query-property-order": "warn",
+  "@tanstack/query/mutation-property-order": "warn",
+  "@tanstack/query/no-rest-destructuring": "warn",
+  "@tanstack/query/no-unstable-deps": "warn",
+  "@tanstack/query/no-void-query-fn": "warn",
+  "@tanstack/query/stable-query-client": "warn"
+};
+
+// src/eslint/query/config.ts
+var queryConfig = (customRules) => ({
+  name: configNames.query,
+  plugins: {
+    "@tanstack/query": queryPlugin
+  },
+  rules: {
+    ...queryRules,
+    ...customRules ?? {}
+  }
+});
+
 // src/eslint/react-refresh/config.ts
 import reactRefresh from "eslint-plugin-react-refresh";
 
@@ -863,7 +891,11 @@ import pluginReactHooks from "eslint-plugin-react-hooks";
 import globals from "globals";
 
 // src/eslint/react/rules.ts
-var reactRules = (functionStyle, typescript) => {
+var reactRules = ({
+  functionStyle,
+  reactCompiler,
+  typescript
+}) => {
   const functionStyleMap = {
     arrow: "arrow-function",
     declaration: "function-declaration",
@@ -874,9 +906,39 @@ var reactRules = (functionStyle, typescript) => {
      * Disabled in favor of TypeScript for type checking
      */
     ...typescript ? {} : { "react/prop-types": "warn" },
+    /**
+     * Core React Hooks rules
+     */
     "react-hooks/exhaustive-deps": "warn",
-    "react-hooks/react-compiler": "warn",
     "react-hooks/rules-of-hooks": "warn",
+    /**
+     * React compiler rules
+     */
+    ...reactCompiler ? {
+      /**
+       * Does not seem to be working, and overlaps with static-components
+       */
+      // "react-hooks/component-hook-factories": "warn",
+      /**
+       * seems unecessary unless you're rolling your own React setup, users can always enable
+       */
+      // "react-hooks/config": "warn",
+      "react-hooks/error-boundaries": "warn",
+      "react-hooks/globals": "warn",
+      "react-hooks/immutability": "warn",
+      "react-hooks/incompatible-library": "warn",
+      "react-hooks/preserve-manual-memoization": "warn",
+      "react-hooks/purity": "warn",
+      "react-hooks/refs": "warn",
+      "react-hooks/set-state-in-effect": "warn",
+      "react-hooks/set-state-in-render": "warn",
+      /**
+       * overlaps with react/no-unstable-nested-components
+       */
+      // "react-hooks/static-components": "warn",
+      "react-hooks/unsupported-syntax": "warn",
+      "react-hooks/use-memo": "warn"
+    } : {},
     /**
      * Require an explicit type when using button elements.
      *
@@ -897,12 +959,6 @@ var reactRules = (functionStyle, typescript) => {
         unnamedComponents: functionStyle === "arrow" ? "arrow-function" : "function-expression"
       }
     ],
-    /**
-     * Require destructuring and symmetric naming of `useState` hook value and setter variables.
-     *
-     * 🚫 Not fixable - https://github.com/jsx-eslint/eslint-plugin-react/blob/master/docs/rules/hook-use-state.md
-     */
-    "react/hook-use-state": "warn",
     /**
      * Require consistent boolean attributes notation in JSX.
      *
@@ -993,7 +1049,12 @@ var reactRules = (functionStyle, typescript) => {
 };
 
 // src/eslint/react/config.ts
-var reactEslintConfig = (functionStyle, typescript, customRules) => {
+var reactEslintConfig = ({
+  customRules,
+  functionStyle,
+  reactCompiler,
+  typescript
+}) => {
   return {
     languageOptions: {
       globals: {
@@ -1011,7 +1072,7 @@ var reactEslintConfig = (functionStyle, typescript, customRules) => {
       "react-hooks": pluginReactHooks
     },
     rules: {
-      ...reactRules(functionStyle, typescript),
+      ...reactRules({ functionStyle, reactCompiler, typescript }),
       ...customRules ?? {}
     },
     settings: {
@@ -1036,7 +1097,7 @@ var storybookConfig = (customRules) => [
     },
     rules: {
       // Default Storybook rules
-      "import/no-anonymous-default-export": "off",
+      "import-x/no-anonymous-default-export": "off",
       "react-hooks/rules-of-hooks": "off",
       "storybook/await-interactions": "warn",
       "storybook/context-in-play-function": "warn",
@@ -1496,6 +1557,7 @@ var eslintConfig = ({
   ignores = [],
   importPlugin = true,
   jsdoc: jsdoc2 = { requireJsdoc: false },
+  query = false,
   react: react2 = false,
   rules: rules2,
   sorting = true,
@@ -1541,10 +1603,11 @@ var eslintConfig = ({
     );
   }
   if (react2) {
+    const reactOptions = isObject(react2) ? react2 : {};
     const shouldUseReactRefresh = (
       // Explicit setting takes precedence
-      isObject(react2) && react2.reactRefresh === true || // Framework-based default (vite/none use reactRefresh by default)
-      isObject(react2) && (react2.framework === "vite" || react2.framework === "none") && react2.reactRefresh !== false
+      reactOptions.reactRefresh === true || // Framework-based default (vite/none use reactRefresh by default)
+      (reactOptions.framework === "vite" || reactOptions.framework === "none") && reactOptions.reactRefresh !== false
     );
     if (shouldUseReactRefresh) {
       configs.push(
@@ -1552,16 +1615,20 @@ var eslintConfig = ({
       );
     }
     configs.push(
-      reactEslintConfig(
+      reactEslintConfig({
+        customRules: categorizedRules[configNames.react],
         functionStyle,
-        Boolean(typescript),
-        categorizedRules[configNames.react]
-      )
+        reactCompiler: reactOptions.reactCompiler ?? true,
+        typescript: Boolean(typescript)
+      })
     );
     if (usingNextjs) {
       configs.push(nextjsConfig(categorizedRules[configNames.nextjs]));
     }
   }
+  if (query) {
+    configs.push(queryConfig(categorizedRules[configNames.query]));
+  }
   if (testing !== false) {
     const mergedTestingConfig = isObject(testing) ? { ...defaultTestingConfig, ...testing } : defaultTestingConfig;
     const { filenamePattern, files, formattingRules, framework, itOrTest } = mergedTestingConfig;
@@ -1605,75 +1672,6 @@ var eslintConfig = ({
   return configs;
 };
 
-// src/prettier/patch-tailwind-plugin.ts
-import * as fs from "fs";
-import * as path from "path";
-var DECLARE_BLOCK = `declare module 'prettier' {
-    interface RequiredOptions extends PluginOptions {
-    }
-    interface ParserOptions extends PluginOptions {
-    }
-}`;
-var patchTailwindPlugin = () => {
-  try {
-    console.info(
-      "js-style-kit: Patching prettier-plugin-tailwindcss declaration file..."
-    );
-    let pluginPath;
-    const rootPath = path.join(
-      "node_modules",
-      "prettier-plugin-tailwindcss",
-      "dist",
-      "index.d.ts"
-    );
-    const packagePath = path.join(
-      process.cwd(),
-      "node_modules",
-      "prettier-plugin-tailwindcss",
-      "dist",
-      "index.d.ts"
-    );
-    if (fs.existsSync(rootPath)) {
-      pluginPath = rootPath;
-    } else if (fs.existsSync(packagePath)) {
-      pluginPath = packagePath;
-    } else {
-      console.warn(
-        "js-style-kit: prettier-plugin-tailwindcss not found, skipping patch"
-      );
-      return false;
-    }
-    console.info(`js-style-kit: Found plugin at ${pluginPath}`);
-    const backupPath = `${pluginPath}.backup`;
-    if (!fs.existsSync(backupPath)) {
-      fs.writeFileSync(backupPath, fs.readFileSync(pluginPath, "utf8"));
-      console.info("js-style-kit: Created backup of original declaration file");
-    }
-    const content = fs.readFileSync(pluginPath, "utf8");
-    if (!content.includes(DECLARE_BLOCK)) {
-      console.info(
-        "js-style-kit: No patching needed, declaration block not found or already patched"
-      );
-      return true;
-    }
-    const patchedContent = content.replace(
-      DECLARE_BLOCK,
-      "// Removed unsafe declare module statement"
-    );
-    fs.writeFileSync(pluginPath, patchedContent);
-    console.info(
-      "js-style-kit: Successfully patched prettier-plugin-tailwindcss declaration file"
-    );
-    return true;
-  } catch (error) {
-    console.error(
-      "js-style-kit: Error patching prettier-plugin-tailwindcss:",
-      error
-    );
-    return false;
-  }
-};
-
 // src/prettier/index.ts
 var prettierConfig = (options = {}) => {
   const {
@@ -1724,7 +1722,6 @@ var prettierConfig = (options = {}) => {
     } else {
       config.tailwindFunctions = defaultTailwindFunctions;
     }
-    patchTailwindPlugin();
   }
   config.plugins = plugins;
   return config;