Rel1cx · Rel1cx · Jul 15, 2026 · Jul 15, 2026
diff --git a/.pkgs/configs/tsl.config.base.js b/.pkgs/configs/tsl.config.base.js
@@ -1,5 +1,5 @@
 import { globSync } from "tinyglobby";
-import { defineConfig } from "tsl";
+import { core, defineConfig } from "tsl";
 import { noDuplicateExports, noDuplicateImports, noMultilineTemplateExpressionWithoutAutoDedent, noUnsafeAs, nullish } from "tsl-dx";
 export function buildConfig(cwd) {
     return defineConfig({
@@ -10,23 +10,20 @@ export function buildConfig(cwd) {
             "/build/",
         ],
         rules: [
-            // ...core.all(),
-            // core.strictBooleanExpressions({
-            //   allowAny: false,
-            //   allowNullableBoolean: false,
-            //   allowNullableEnum: false,
-            //   allowNullableNumber: false,
-            //   allowNullableObject: false,
-            //   allowNullableString: false,
-            //   allowNumber: true,
-            //   allowString: false,
-            // }),
-            // core.noConfusingVoidExpression("off"),
-            // core.preferOptionalChain("off"),
-            // core.switchExhaustivenessCheck("off"), // This rule has a issue with `switch (true)` statements
-            // core.switchExhaustivenessCheck({
-            //   considerDefaultExhaustiveForUnions: true,
-            // }),
+            ...core.all(),
+            core.strictBooleanExpressions({
+                allowAny: false,
+                allowNullableBoolean: false,
+                allowNullableEnum: false,
+                allowNullableNumber: false,
+                allowNullableObject: false,
+                allowNullableString: false,
+                allowNumber: true,
+                allowString: false,
+            }),
+            core.noConfusingVoidExpression("off"),
+            core.preferOptionalChain("off"),
+            core.switchExhaustivenessCheck("off"), // This rule has a issue with `switch (true)` statements
             nullish({
                 runtimeLibrary: "@eslint-react/eff",
             }),

diff --git a/.pkgs/configs/tsl.config.base.ts b/.pkgs/configs/tsl.config.base.ts
@@ -1,5 +1,5 @@
 import { globSync } from "tinyglobby";
-import { defineConfig } from "tsl";
+import { core, defineConfig } from "tsl";
 import { noDuplicateExports, noDuplicateImports, noMultilineTemplateExpressionWithoutAutoDedent, noUnsafeAs, nullish } from "tsl-dx";
 
 export function buildConfig(cwd: string) {
@@ -11,23 +11,20 @@ export function buildConfig(cwd: string) {
       "/build/",
     ],
     rules: [
-      // ...core.all(),
-      // core.strictBooleanExpressions({
-      //   allowAny: false,
-      //   allowNullableBoolean: false,
-      //   allowNullableEnum: false,
-      //   allowNullableNumber: false,
-      //   allowNullableObject: false,
-      //   allowNullableString: false,
-      //   allowNumber: true,
-      //   allowString: false,
-      // }),
-      // core.noConfusingVoidExpression("off"),
-      // core.preferOptionalChain("off"),
-      // core.switchExhaustivenessCheck("off"), // This rule has a issue with `switch (true)` statements
-      // core.switchExhaustivenessCheck({
-      //   considerDefaultExhaustiveForUnions: true,
-      // }),
+      ...core.all(),
+      core.strictBooleanExpressions({
+        allowAny: false,
+        allowNullableBoolean: false,
+        allowNullableEnum: false,
+        allowNullableNumber: false,
+        allowNullableObject: false,
+        allowNullableString: false,
+        allowNumber: true,
+        allowString: false,
+      }),
+      core.noConfusingVoidExpression("off"),
+      core.preferOptionalChain("off"),
+      core.switchExhaustivenessCheck("off"), // This rule has a issue with `switch (true)` statements
       nullish({
         runtimeLibrary: "@eslint-react/eff",
       }),

diff --git a/.pkgs/samples/src/booleanPropNaming.ts b/.pkgs/samples/src/booleanPropNaming.ts
@@ -31,7 +31,6 @@ export function booleanPropNaming(options?: BooleanPropNamingOptions): RuleFunct
       "Program:exit"(prog) {
         const comps = query.all(prog);
 
-        // ─── Iterate Components ────────────────────────
         for (const comp of comps) {
           const [propsParam] = comp.node.params;
           if (propsParam == null) continue;
@@ -40,7 +39,6 @@ export function booleanPropNaming(options?: BooleanPropNamingOptions): RuleFunct
           const propsType = chk.getTypeAtLocation(tsNode);
           const declaredProps = propsType.getProperties();
 
-          // ─── Iterate Props ─────────────────────────────
           for (const prop of declaredProps) {
             const propType = chk.getTypeOfSymbolAtLocation(prop, tsNode);
 
@@ -57,7 +55,6 @@ export function booleanPropNaming(options?: BooleanPropNamingOptions): RuleFunct
             if (decl == null) continue;
 
             const declNode = srv.tsNodeToESTreeNodeMap.get(decl);
-            if (declNode == null) continue;
 
             const node = "key" in declNode ? declNode.key : declNode;
 

diff --git a/.pkgs/samples/src/forbidComponentProps.ts b/.pkgs/samples/src/forbidComponentProps.ts
@@ -16,8 +16,7 @@ export function forbidComponentProps(options: ForbidComponentPropsOptions): Rule
       if (propName == null || !forbidden.includes(propName)) return;
 
       // Verify context is JSX opening element
-      const parent = node.parent;
-      if (parent?.type !== "JSXOpeningElement") return;
+      const { parent } = node;
 
       // Extract element name
       const elemName = parent.name.type === "JSXIdentifier" ? parent.name.name : null;

diff --git a/.pkgs/samples/src/forbidDomProps.ts b/.pkgs/samples/src/forbidDomProps.ts
@@ -16,8 +16,7 @@ export function forbidDomProps(options: ForbidDomPropsOptions): RuleFunction {
       if (propName == null || !forbidden.includes(propName)) return;
 
       // Verify context is JSX opening element
-      const parent = node.parent;
-      if (parent?.type !== "JSXOpeningElement") return;
+      const { parent } = node;
 
       // Extract element name
       const elemName = parent.name.type === "JSXIdentifier" ? parent.name.name : null;

diff --git a/.pkgs/samples/src/functionComponentDefinition.ts b/.pkgs/samples/src/functionComponentDefinition.ts
@@ -11,7 +11,6 @@ export function functionComponentDefinition(): RuleFunction {
       visitor,
       {
         "Program:exit"(program) {
-          // ─── Iterate all components ────────────────────
           for (const { node } of query.all(program)) {
             // Guard: must not already be arrow function
             if (node.type === "ArrowFunctionExpression") continue;
@@ -27,24 +26,21 @@ export function functionComponentDefinition(): RuleFunction {
                     if (node.generator) return null;
 
                     const prefix = node.async ? "async " : "";
-                    const typeParams = node.typeParameters ? src.getText(node.typeParameters) : "";
+                    const typeParams = node.typeParameters != null ? src.getText(node.typeParameters) : "";
                     const params = `(${node.params.map((p) => src.getText(p)).join(", ")})`;
-                    const returnType = node.returnType ? src.getText(node.returnType) : "";
+                    const returnType = node.returnType != null ? src.getText(node.returnType) : "";
                     const body = src.getText(node.body);
 
-                    // ─── Case: function declaration ──────────────
-                    if (node.type === "FunctionDeclaration" && node.id) {
+                    if (node.type === "FunctionDeclaration" && node.id != null) {
                       // dprint-ignore
                       return fixer.replaceText(node, `const ${node.id.name} = ${prefix}${typeParams}${params}${returnType} => ${body};`);
                     }
 
-                    // ─── Case: function expression in variable ───
                     if (node.type === "FunctionExpression" && node.parent.type === "VariableDeclarator") {
                       // dprint-ignore
                       return fixer.replaceText(node, `${prefix}${typeParams}${params}${returnType} => ${body}`);
                     }
 
-                    // ─── Case: object method shorthand ───────────
                     if (node.type === "FunctionExpression" && node.parent.type === "Property") {
                       // dprint-ignore
                       return fixer.replaceText(node.parent, `${src.getText(node.parent.key)}: ${prefix}${typeParams}${params}${returnType} => ${body}`);

diff --git a/.pkgs/samples/src/jsxFragments.ts b/.pkgs/samples/src/jsxFragments.ts
@@ -11,8 +11,6 @@ export type JsxsFragmentsOptions = {
 export function jsxFragments(options: JsxsFragmentsOptions = {}): RuleFunction {
   const { mode = "syntax" } = options;
   return (context) => {
-    // ── Helpers ─────────────────────────────────────
-
     function reportSyntaxPreferred(node: TSESTree.JSXOpeningElement, pattern: "React.Fragment" | "Fragment") {
       // Guard: has key prop (legitimate use of standard form)
       const hasAttributes = node.attributes.length > 0;
@@ -22,31 +20,27 @@ export function jsxFragments(options: JsxsFragmentsOptions = {}): RuleFunction {
         node,
         message: `Use shorthand fragment syntax '<>...</>' instead of '<${pattern}>...</${pattern}'.`,
         fix(fixer) {
-          const closing = node.parent?.closingElement;
-          if (!closing) return null;
+          const closing = node.parent.closingElement;
+          if (closing == null) return null;
           return [fixer.replaceText(node, "<>"), fixer.replaceText(closing, "</>")];
         },
       });
     }
 
-    // ── Listeners ────────────────────────────────────
-
     return {
       JSXOpeningElement(node) {
         const name = node.name;
 
-        // ─── Handle <Fragment> (JSXIdentifier) ───────
         if (name.type === "JSXIdentifier" && name.name === "Fragment") {
           if (mode === "syntax") {
             reportSyntaxPreferred(node, "Fragment");
           }
           return;
         }
 
-        // ─── Handle <React.Fragment> (JSXMemberExpression) ─
         if (name.type !== "JSXMemberExpression") return;
         if (name.object.type !== "JSXIdentifier" || name.object.name !== "React") return;
-        if (name.property.type !== "JSXIdentifier" || name.property.name !== "Fragment") return;
+        if (name.property.name !== "Fragment") return;
 
         if (mode === "syntax") {
           reportSyntaxPreferred(node, "React.Fragment");

diff --git a/.pkgs/samples/src/jsxHandlerNames.ts b/.pkgs/samples/src/jsxHandlerNames.ts
@@ -28,9 +28,8 @@ export function jsxHandlerNames(options: JsxHandlerNamesOptions = {}): RuleFunct
       if (!EVENT_HANDLER_REGEX.test(propName)) return;
 
       const value = node.value;
-      if (!value) return;
+      if (value == null) return;
 
-      // ─── Check expression value ────────────────────
       if (value.type === "JSXExpressionContainer") {
         const expression = ast.unwrap(value.expression);
 

diff --git a/.pkgs/samples/src/jsxMaxDepth.ts b/.pkgs/samples/src/jsxMaxDepth.ts
@@ -12,11 +12,9 @@ export function jsxMaxDepth(options: JsxMaxDepthOptions): RuleFunction {
   return (context) => ({
     JSXElement(node) {
       let depth = 0;
-      // eslint-disable-next-line @typescript-eslint/no-explicit-any
-      let parent: any = node.parent;
+      let parent: typeof node.parent | null | undefined = node.parent;
 
-      // ─── Walk up the tree ──────────────────────────
-      while (parent) {
+      while (parent != null) {
         if (parent.type === "JSXElement") {
           depth++;
         }

diff --git a/.pkgs/samples/src/jsxNoBind.ts b/.pkgs/samples/src/jsxNoBind.ts
@@ -11,7 +11,6 @@ export function jsxNoBind(): RuleFunction {
 
       const expr = ast.unwrap(value.expression);
 
-      // ─── Detect forbidden patterns ─────────────────
       switch (true) {
         case expr.type === "ArrowFunctionExpression":
         case expr.type === "FunctionExpression":

diff --git a/.pkgs/samples/src/jsxNoDuplicateProps.ts b/.pkgs/samples/src/jsxNoDuplicateProps.ts
@@ -13,7 +13,6 @@ export function jsxNoDuplicateProps(options: JsxNoDuplicatePropsOptions = {}): R
     JSXOpeningElement(node) {
       const seen = new Map<string, string>();
 
-      // ─── Check each attribute ──────────────────────
       for (const attr of node.attributes) {
         if (attr.type !== "JSXAttribute") continue;
         if (attr.name.type !== "JSXIdentifier") continue;

diff --git a/.pkgs/samples/src/jsxNoLiterals.ts b/.pkgs/samples/src/jsxNoLiterals.ts
@@ -15,16 +15,13 @@ export function jsxNoLiterals(options: JsxNoLiteralsOptions = {}): RuleFunction
   const { noStrings = false, allowedStrings = [], ignoreProps = true } = options;
   const allowedSet = new Set(allowedStrings);
   return (context) => ({
-    // ─── Check literal text children ───────────────
     Literal(node) {
       if (typeof node.value !== "string") return;
       const text = node.value.trim();
       if (text === "" || allowedSet.has(text)) return;
 
       const parent = node.parent;
-      if (!parent) return;
 
-      // ─── Case: prop value ────────────────────────
       if (parent.type === "JSXAttribute") {
         if (!ignoreProps) {
           context.report({
@@ -35,10 +32,8 @@ export function jsxNoLiterals(options: JsxNoLiteralsOptions = {}): RuleFunction
         return;
       }
 
-      // ─── Case: already wrapped ───────────────────
       if (parent.type === "JSXExpressionContainer") return;
 
-      // ─── Case: child of element/fragment ─────────
       if (parent.type === "JSXElement" || parent.type === "JSXFragment") {
         if (noStrings) {
           context.report({
@@ -54,7 +49,6 @@ export function jsxNoLiterals(options: JsxNoLiteralsOptions = {}): RuleFunction
       }
     },
 
-    // ─── Check JSX text nodes ──────────────────────
     JSXText(node) {
       const text = node.value.trim();
       if (text === "" || allowedSet.has(text)) return;

diff --git a/.pkgs/samples/src/jsxPascalCase.ts b/.pkgs/samples/src/jsxPascalCase.ts
@@ -21,7 +21,6 @@ export function jsxPascalCase(options: JsxPascalCaseOptions = {}): RuleFunction
 
       const componentName = name.name;
 
-      // ─── Handle leading underscore ───────────────
       if (componentName.startsWith("_")) {
         if (!allowLeadingUnderscore) {
           context.report({
@@ -37,7 +36,6 @@ export function jsxPascalCase(options: JsxPascalCaseOptions = {}): RuleFunction
       if (firstChar == null) return;
       if (firstChar === firstChar.toLowerCase()) return;
 
-      // ─── Handle all-caps ─────────────────────────
       if (componentName === componentName.toUpperCase()) {
         if (!allowAllCaps) {
           context.report({
@@ -48,7 +46,6 @@ export function jsxPascalCase(options: JsxPascalCaseOptions = {}): RuleFunction
         return;
       }
 
-      // ─── Validate PascalCase ─────────────────────
       if (!pascalCaseRegex.test(componentName)) {
         context.report({
           node: name,

diff --git a/.pkgs/samples/src/jsxPropsNoSpreadMulti.ts b/.pkgs/samples/src/jsxPropsNoSpreadMulti.ts
@@ -6,7 +6,6 @@ export function jsxPropsNoSpreadMulti(): RuleFunction {
     JSXOpeningElement(node) {
       const seen = new Set<string>();
 
-      // ─── Check each spread attribute ───────────────
       for (const attr of node.attributes) {
         if (attr.type !== "JSXSpreadAttribute") continue;
 

diff --git a/.pkgs/samples/src/noAdjacentInlineElements.ts b/.pkgs/samples/src/noAdjacentInlineElements.ts
@@ -41,7 +41,6 @@ export function noAdjacentInlineElements(): RuleFunction {
     JSXElement(node) {
       const children = node.children;
 
-      // ─── Check adjacent pairs ──────────────────────
       for (let i = 0; i < children.length - 1; i++) {
         const current = children[i];
         const next = children[i + 1];

diff --git a/.pkgs/samples/src/noMultiComp.ts b/.pkgs/samples/src/noMultiComp.ts
@@ -9,7 +9,6 @@ export function noMultiComp(): RuleFunction {
       "Program:exit"(program) {
         const components = query.all(program);
 
-        // ─── Report excess components ──────────────────
         for (const { node, name } of components.slice(1)) {
           context.report({
             node,

diff --git a/.pkgs/samples/src/noUnnecessaryUsePrefix.ts b/.pkgs/samples/src/noUnnecessaryUsePrefix.ts
@@ -11,7 +11,9 @@ export function noUnnecessaryUsePrefix(): RuleFunction {
           if (hook.hookCalls.length === 0) {
             context.report({
               node: hook.node,
-              message: `Custom hook "${hook.name}" doesn't call any hooks. A custom hook should use at least one hook, otherwise it's just a regular function.`,
+              message: `Custom hook "${
+                hook.name ?? "unknown"
+              }" doesn't call any hooks. A custom hook should use at least one hook, otherwise it's just a regular function.`,
             });
           }
         }