@mkaradeniz/eslint-config 5.9.0 → 5.11.0

package/noLet.mjs

@@ -0,0 +1,26 @@
+import { baseRules } from './rules/baseRules.mjs';
+
+const noLetRestriction = {
+  message:
+    'Prefer const + functional patterns (map/filter/reduce/ternary). Extract to a util function if needed. Only disable for: try/catch assignment, iterators/generators, for-await-of accumulation, or closures requiring mutable state (debounce, event handlers).',
+  selector: 'VariableDeclaration[kind="let"]',
+};
+
+/**
+ * Bans `let` declarations via `no-restricted-syntax`.
+ *
+ * Accepts an optional source rules object to merge with. Defaults to `baseRules`.
+ * Pass `reactRules` when composing with `reactConfig` or `nextConfig` to preserve their extra selectors.
+ *
+ * @example
+ * import { createNoLetConfig } from '@mkaradeniz/eslint-config/noLet.mjs';
+ * import { reactRules } from '@mkaradeniz/eslint-config/rules/reactRules.mjs';
+ * export default [...nextConfig, ...createNoLetConfig(reactRules)];
+ */
+export const createNoLetConfig = (sourceRules = baseRules) => [
+  {
+    rules: {
+      'no-restricted-syntax': ['warn', ...sourceRules.rules['no-restricted-syntax'].slice(1), noLetRestriction],
+    },
+  },
+];

package/package.json

@@ -1,11 +1,12 @@
 {
   "name": "@mkaradeniz/eslint-config",
-  "version": "5.9.0",
+  "version": "5.11.0",
   "private": false,
   "files": [
     "base.mjs",
     "web.mjs",
     "next.mjs",
+    "noLet.mjs",
     "react.mjs",
     "shadcn.mjs",
     "tanstackQuery.mjs",

package/rules/baseRules.mjs

@@ -98,6 +98,26 @@ export const baseRules = {
         message: 'Use !isNotNullOrUndefined() instead of === undefined.',
         selector: 'BinaryExpression[operator="==="][right.type="Identifier"][right.name="undefined"]',
       },
+      {
+        message:
+          "Don't compare to true/false directly. For optional booleans use isNotNullOrUndefined(x) && x or isNotNullOrUndefined(x) && !x.",
+        selector: 'BinaryExpression[operator="==="][right.type="Literal"][right.raw="true"]',
+      },
+      {
+        message:
+          "Don't compare to true/false directly. For optional booleans use isNotNullOrUndefined(x) && x or isNotNullOrUndefined(x) && !x.",
+        selector: 'BinaryExpression[operator="==="][right.type="Literal"][right.raw="false"]',
+      },
+      {
+        message:
+          "Don't compare to true/false directly. For optional booleans use isNotNullOrUndefined(x) && x or isNotNullOrUndefined(x) && !x.",
+        selector: 'BinaryExpression[operator="!=="][right.type="Literal"][right.raw="true"]',
+      },
+      {
+        message:
+          "Don't compare to true/false directly. For optional booleans use isNotNullOrUndefined(x) && x or isNotNullOrUndefined(x) && !x.",
+        selector: 'BinaryExpression[operator="!=="][right.type="Literal"][right.raw="false"]',
+      },
       {
         message: 'Exported functions with >1 parameter must use a single object parameter.',
         selector: 'ExportNamedDeclaration > VariableDeclaration > VariableDeclarator > ArrowFunctionExpression[params.length>1]',
@@ -114,11 +134,6 @@ export const baseRules = {
         message: 'Use envConfigClient or envConfigServer instead of process.env.',
         selector: 'MemberExpression[object.name="process"][property.name="env"]',
       },
-      {
-        message:
-          'Prefer const + functional patterns (map/filter/reduce/ternary). Extract to a util function if needed. Only disable for: try/catch assignment, iterators/generators, for-await-of accumulation, or closures requiring mutable state (debounce, event handlers).',
-        selector: 'VariableDeclaration[kind="let"]',
-      },
     ],
     'no-return-assign': ['warn'],
     'no-script-url': ['warn'],