@skapxd/eslint-opinionated 0.18.0 → 2.0.0

package/README.md

@@ -350,6 +350,11 @@ tocar contratos. Es la ola que más enseña por repetición:
 - `skapxd/no-nested-if` y `skapxd/no-else` — guard clauses. El refactor más
   formativo que existe para un junior: aplana la lógica o confiesa que la
   función hace demasiado.
+- `skapxd/no-anonymous-condition` — la pareja de las anteriores y **la más
+  cara de todo el catálogo** (cientos de hallazgos en un backend típico):
+  cada condición-cómputo recibe un nombre con criterio. Vale la pena ir por
+  carpetas y SIN prisa — es la que más enseña por hallazgo, y la última de
+  esta ola.
 - `skapxd/one-root-function-per-file` y `skapxd/no-default-export` — el árbol
   de archivos empieza a contar la historia.
 - `skapxd/no-accessors`, `skapxd/max-public-methods` — clases con una
@@ -880,7 +885,7 @@ matchea en cualquier carpeta). Las 7 reglas restantes no tienen opciones: su
 | `skapxd/nest-no-swagger-in-controllers` | Los controllers no se llenan de decoradores de swagger; el plugin introspecciona los DTOs. Preset `nest`. |
 | `skapxd/nest-requires-swagger-plugin` | `nest-cli.json` debe tener el plugin `@nestjs/swagger`: la premisa de las reglas de swagger, verificada. Preset `nest`. |
 | `skapxd/nest-validation-pipe-config` | Todo `new ValidationPipe` configura `transform` y `whitelist`: la premisa de las reglas de DTOs. Preset `nest`. |
-| `skapxd/no-anonymous-condition` | El `if` solo acepta condiciones ya nombradas; todo cómputo (llamada, comparación, `&&`/`||`) se extrae a una `const` con nombre semántico. **Opt-in: no está en ningún preset.** |
+| `skapxd/no-anonymous-condition` | El `if` solo acepta condiciones ya nombradas; todo cómputo (llamada, comparación, `&&`/`||`) se extrae a una `const` con nombre semántico. |
 | `skapxd/no-deep-relative-imports` | Limita la profundidad de los imports relativos (`../`). |
 | `skapxd/no-default-export` | Prohíbe `export default`; el nombre del símbolo es el contrato. Exime configs/stories y, en el preset `next`, los entrypoints del App Router. |
 | `skapxd/no-else` | Prohíbe `else`/`else if`: el else es el estado sin nombre. Retorno anticipado, ternario simple o `match()`. |
@@ -1199,8 +1204,9 @@ con las mismas cinco formas de guard.
 > }
 > ```
 >
-> (`skapxd/await-requires-try-safe` es el nombre anterior; sigue funcionando
-> como alias deprecado y se eliminará en una versión futura.)
+> (`skapxd/await-requires-try-safe` fue el nombre anterior; el alias se
+> eliminó en la v1.0.0 — si tu config lo menciona, renómbralo a
+> `skapxd/await-requires-result`: mismo comportamiento, mismas opciones.)
 
 Hay dos caminos válidos, y la regla recomienda el primero:
 
@@ -1700,18 +1706,15 @@ Lo que **sí dispara**: llamadas, comparaciones (`a.length <= b.max`,
 (`if (total % 2)`). La extracción directa a `const` conserva el narrowing
 (TS 4.4+, aliased conditions).
 
-**Opt-in deliberado: no está en ningún preset.** La calibración contra 4
-proyectos reales (2026-06-12) midió 473/95/308 hallazgos en tres backends
-NestJS en producción y 44 en un front pequeño — señal genuina en la muestra
-revisada, pero un orden de magnitud más invasiva que cualquier regla de las
-bases. Actívala por proyecto (o por carpeta, estilo ola 3 del playbook de
-adopción):
-
-```js
-rules: {
-  "skapxd/no-anonymous-condition": "error",
-}
-```
+**Está en las reglas base** — y es la más invasiva del catálogo: la
+calibración contra 4 proyectos reales (2026-06-12) midió 473/95/308
+hallazgos en tres backends NestJS en producción y 44 en un front pequeño
+(señal genuina en la muestra revisada a mano). En un proyecto existente,
+trátala con el playbook de adopción: entra apagada en la lista de
+pendientes y se enciende por carpetas, nombrando con criterio — el valor de
+la regla son los nombres, y un nombre autogenerado la traiciona. Este mismo
+repo la tiene en su lista de pendientes (245 condiciones heredadas) — la
+regla nació subiendo la vara que su propio código aún está alcanzando.
 
 ### `skapxd/no-deep-relative-imports`
 

package/dist/astro/index.d.mts

@@ -14,6 +14,7 @@ declare function createAstroConfigs(pluginReference: unknown): ({
         "skapxd/max-public-methods": string;
         "skapxd/no-accessors": string;
         "skapxd/no-ad-hoc-ok-result": string;
+        "skapxd/no-anonymous-condition": string;
         "skapxd/no-deep-relative-imports": string;
         "skapxd/no-default-export": string;
         "skapxd/no-else": string;
@@ -40,6 +41,7 @@ declare function createAstroConfigs(pluginReference: unknown): ({
         "skapxd/max-public-methods": string;
         "skapxd/no-accessors": string;
         "skapxd/no-ad-hoc-ok-result": string;
+        "skapxd/no-anonymous-condition": string;
         "skapxd/no-deep-relative-imports": string;
         "skapxd/no-default-export": string;
         "skapxd/no-else": string;

package/dist/astro/index.d.ts

@@ -14,6 +14,7 @@ declare function createAstroConfigs(pluginReference: unknown): ({
         "skapxd/max-public-methods": string;
         "skapxd/no-accessors": string;
         "skapxd/no-ad-hoc-ok-result": string;
+        "skapxd/no-anonymous-condition": string;
         "skapxd/no-deep-relative-imports": string;
         "skapxd/no-default-export": string;
         "skapxd/no-else": string;
@@ -40,6 +41,7 @@ declare function createAstroConfigs(pluginReference: unknown): ({
         "skapxd/max-public-methods": string;
         "skapxd/no-accessors": string;
         "skapxd/no-ad-hoc-ok-result": string;
+        "skapxd/no-anonymous-condition": string;
         "skapxd/no-deep-relative-imports": string;
         "skapxd/no-default-export": string;
         "skapxd/no-else": string;

package/dist/astro/index.js

@@ -40,6 +40,10 @@ var baseRules = {
   "skapxd/max-public-methods": "error",
   "skapxd/no-accessors": "error",
   "skapxd/no-ad-hoc-ok-result": "error",
+  // En las bases por decisión del dueño (issue #2): con poca adopción
+  // todavía, es el momento de subir la vara; los legacy la apagan en su
+  // lista de pendientes (playbook de adopción del README).
+  "skapxd/no-anonymous-condition": "error",
   "skapxd/no-deep-relative-imports": "error",
   "skapxd/no-default-export": "error",
   "skapxd/no-else": "error",

package/dist/astro/index.js.map

@@ -1 +1 @@
-{"version":3,"sources":["../../src/astro/index.ts","../../src/shared/configs/base-rules.ts","../../src/shared/configs/create-base-language-options.ts","../../src/shared/configs/create-typed-language-options.ts","../../src/astro/configs.ts"],"sourcesContent":["export { createAstroConfigs } from \"./configs\";\n","export const baseRules = {\n  \"skapxd/class-properties-require-readonly\": \"error\",\n  \"skapxd/max-public-methods\": \"error\",\n  \"skapxd/no-accessors\": \"error\",\n  \"skapxd/no-ad-hoc-ok-result\": \"error\",\n  \"skapxd/no-deep-relative-imports\": \"error\",\n  \"skapxd/no-default-export\": \"error\",\n  \"skapxd/no-else\": \"error\",\n  \"skapxd/no-emoji\": \"error\",\n  \"skapxd/no-nested-if\": \"error\",\n  \"skapxd/no-runtime-state-guard\": \"error\",\n  \"skapxd/no-promise-chain\": \"error\",\n  \"skapxd/no-try-catch\": \"error\",\n  \"skapxd/one-root-function-per-file\": \"error\",\n  \"skapxd/prefer-tagged-union-state\": \"error\",\n  \"skapxd/requires-strict-tsconfig\": \"error\",\n  \"skapxd/prefer-ts-pattern\": \"error\",\n  \"skapxd/result-error-requires-cause\": \"error\",\n  \"skapxd/result-error-requires-handling\": \"error\",\n};\n","import tseslint from \"typescript-eslint\";\n\n// Variante sin type-checking: solo el parser, para presets que aplican a TS\n// pero no necesitan projectService (base, package, next/base, next/react).\nexport function createBaseLanguageOptions() {\n  return {\n    parser: tseslint.parser,\n  };\n}\n","import tseslint from \"typescript-eslint\";\n\nexport function createTypedLanguageOptions() {\n  return {\n    // Sin el parser explícito, un consumidor que use solo estos presets\n    // obtiene \"Parsing error\" en cada archivo TS (espree no parsea TS).\n    parser: tseslint.parser,\n    parserOptions: {\n      projectService: true,\n    },\n  };\n}\n","import {\n  baseRules,\n  createBaseLanguageOptions,\n  createTypedLanguageOptions,\n} from \"#/shared/configs\";\n\nexport function createAstroConfigs(pluginReference: unknown) {\n  const baseLanguageOptions = createBaseLanguageOptions();\n  const typedLanguageOptions = createTypedLanguageOptions();\n\n  return [\n    {\n      files: [\"src/**/*.{ts,tsx}\"],\n      languageOptions: baseLanguageOptions,\n      name: \"skapxd/astro/base\",\n      plugins: { skapxd: pluginReference },\n      rules: baseRules,\n    },\n    // Los .astro no llevan parser propio: lo aporta eslint-plugin-astro,\n    // que el consumidor debe tener configurado.\n    {\n      files: [\"src/**/*.astro\"],\n      name: \"skapxd/astro/astro-files\",\n      plugins: { skapxd: pluginReference },\n      rules: baseRules,\n    },\n    {\n      files: [\"src/**/*.{ts,tsx}\"],\n      languageOptions: typedLanguageOptions,\n      name: \"skapxd/astro/typescript\",\n      plugins: { skapxd: pluginReference },\n      rules: {\n        \"skapxd/await-requires-result\": \"error\",\n        \"skapxd/result-error-requires-cause\": \"error\",\n        \"skapxd/result-error-requires-handling\": \"error\",\n      },\n    },\n  ];\n}\n"],"mappings":";;;;;;;;;;;;;;;;;;;;;;;;;;;;;;AAAA;AAAA;AAAA;AAAA;AAAA;;;ACAO,IAAM,YAAY;AAAA,EACvB,4CAA4C;AAAA,EAC5C,6BAA6B;AAAA,EAC7B,uBAAuB;AAAA,EACvB,8BAA8B;AAAA,EAC9B,mCAAmC;AAAA,EACnC,4BAA4B;AAAA,EAC5B,kBAAkB;AAAA,EAClB,mBAAmB;AAAA,EACnB,uBAAuB;AAAA,EACvB,iCAAiC;AAAA,EACjC,2BAA2B;AAAA,EAC3B,uBAAuB;AAAA,EACvB,qCAAqC;AAAA,EACrC,oCAAoC;AAAA,EACpC,mCAAmC;AAAA,EACnC,4BAA4B;AAAA,EAC5B,sCAAsC;AAAA,EACtC,yCAAyC;AAC3C;;;ACnBA,+BAAqB;AAId,SAAS,4BAA4B;AAC1C,SAAO;AAAA,IACL,QAAQ,yBAAAA,QAAS;AAAA,EACnB;AACF;;;ACRA,IAAAC,4BAAqB;AAEd,SAAS,6BAA6B;AAC3C,SAAO;AAAA;AAAA;AAAA,IAGL,QAAQ,0BAAAC,QAAS;AAAA,IACjB,eAAe;AAAA,MACb,gBAAgB;AAAA,IAClB;AAAA,EACF;AACF;;;ACLO,SAAS,mBAAmB,iBAA0B;AAC3D,QAAM,sBAAsB,0BAA0B;AACtD,QAAM,uBAAuB,2BAA2B;AAExD,SAAO;AAAA,IACL;AAAA,MACE,OAAO,CAAC,mBAAmB;AAAA,MAC3B,iBAAiB;AAAA,MACjB,MAAM;AAAA,MACN,SAAS,EAAE,QAAQ,gBAAgB;AAAA,MACnC,OAAO;AAAA,IACT;AAAA;AAAA;AAAA,IAGA;AAAA,MACE,OAAO,CAAC,gBAAgB;AAAA,MACxB,MAAM;AAAA,MACN,SAAS,EAAE,QAAQ,gBAAgB;AAAA,MACnC,OAAO;AAAA,IACT;AAAA,IACA;AAAA,MACE,OAAO,CAAC,mBAAmB;AAAA,MAC3B,iBAAiB;AAAA,MACjB,MAAM;AAAA,MACN,SAAS,EAAE,QAAQ,gBAAgB;AAAA,MACnC,OAAO;AAAA,QACL,gCAAgC;AAAA,QAChC,sCAAsC;AAAA,QACtC,yCAAyC;AAAA,MAC3C;AAAA,IACF;AAAA,EACF;AACF;","names":["tseslint","import_typescript_eslint","tseslint"]}
+{"version":3,"sources":["../../src/astro/index.ts","../../src/shared/configs/base-rules.ts","../../src/shared/configs/create-base-language-options.ts","../../src/shared/configs/create-typed-language-options.ts","../../src/astro/configs.ts"],"sourcesContent":["export { createAstroConfigs } from \"./configs\";\n","export const baseRules = {\n  \"skapxd/class-properties-require-readonly\": \"error\",\n  \"skapxd/max-public-methods\": \"error\",\n  \"skapxd/no-accessors\": \"error\",\n  \"skapxd/no-ad-hoc-ok-result\": \"error\",\n  // En las bases por decisión del dueño (issue #2): con poca adopción\n  // todavía, es el momento de subir la vara; los legacy la apagan en su\n  // lista de pendientes (playbook de adopción del README).\n  \"skapxd/no-anonymous-condition\": \"error\",\n  \"skapxd/no-deep-relative-imports\": \"error\",\n  \"skapxd/no-default-export\": \"error\",\n  \"skapxd/no-else\": \"error\",\n  \"skapxd/no-emoji\": \"error\",\n  \"skapxd/no-nested-if\": \"error\",\n  \"skapxd/no-runtime-state-guard\": \"error\",\n  \"skapxd/no-promise-chain\": \"error\",\n  \"skapxd/no-try-catch\": \"error\",\n  \"skapxd/one-root-function-per-file\": \"error\",\n  \"skapxd/prefer-tagged-union-state\": \"error\",\n  \"skapxd/requires-strict-tsconfig\": \"error\",\n  \"skapxd/prefer-ts-pattern\": \"error\",\n  \"skapxd/result-error-requires-cause\": \"error\",\n  \"skapxd/result-error-requires-handling\": \"error\",\n};\n","import tseslint from \"typescript-eslint\";\n\n// Variante sin type-checking: solo el parser, para presets que aplican a TS\n// pero no necesitan projectService (base, package, next/base, next/react).\nexport function createBaseLanguageOptions() {\n  return {\n    parser: tseslint.parser,\n  };\n}\n","import tseslint from \"typescript-eslint\";\n\nexport function createTypedLanguageOptions() {\n  return {\n    // Sin el parser explícito, un consumidor que use solo estos presets\n    // obtiene \"Parsing error\" en cada archivo TS (espree no parsea TS).\n    parser: tseslint.parser,\n    parserOptions: {\n      projectService: true,\n    },\n  };\n}\n","import {\n  baseRules,\n  createBaseLanguageOptions,\n  createTypedLanguageOptions,\n} from \"#/shared/configs\";\n\nexport function createAstroConfigs(pluginReference: unknown) {\n  const baseLanguageOptions = createBaseLanguageOptions();\n  const typedLanguageOptions = createTypedLanguageOptions();\n\n  return [\n    {\n      files: [\"src/**/*.{ts,tsx}\"],\n      languageOptions: baseLanguageOptions,\n      name: \"skapxd/astro/base\",\n      plugins: { skapxd: pluginReference },\n      rules: baseRules,\n    },\n    // Los .astro no llevan parser propio: lo aporta eslint-plugin-astro,\n    // que el consumidor debe tener configurado.\n    {\n      files: [\"src/**/*.astro\"],\n      name: \"skapxd/astro/astro-files\",\n      plugins: { skapxd: pluginReference },\n      rules: baseRules,\n    },\n    {\n      files: [\"src/**/*.{ts,tsx}\"],\n      languageOptions: typedLanguageOptions,\n      name: \"skapxd/astro/typescript\",\n      plugins: { skapxd: pluginReference },\n      rules: {\n        \"skapxd/await-requires-result\": \"error\",\n        \"skapxd/result-error-requires-cause\": \"error\",\n        \"skapxd/result-error-requires-handling\": \"error\",\n      },\n    },\n  ];\n}\n"],"mappings":";;;;;;;;;;;;;;;;;;;;;;;;;;;;;;AAAA;AAAA;AAAA;AAAA;AAAA;;;ACAO,IAAM,YAAY;AAAA,EACvB,4CAA4C;AAAA,EAC5C,6BAA6B;AAAA,EAC7B,uBAAuB;AAAA,EACvB,8BAA8B;AAAA;AAAA;AAAA;AAAA,EAI9B,iCAAiC;AAAA,EACjC,mCAAmC;AAAA,EACnC,4BAA4B;AAAA,EAC5B,kBAAkB;AAAA,EAClB,mBAAmB;AAAA,EACnB,uBAAuB;AAAA,EACvB,iCAAiC;AAAA,EACjC,2BAA2B;AAAA,EAC3B,uBAAuB;AAAA,EACvB,qCAAqC;AAAA,EACrC,oCAAoC;AAAA,EACpC,mCAAmC;AAAA,EACnC,4BAA4B;AAAA,EAC5B,sCAAsC;AAAA,EACtC,yCAAyC;AAC3C;;;ACvBA,+BAAqB;AAId,SAAS,4BAA4B;AAC1C,SAAO;AAAA,IACL,QAAQ,yBAAAA,QAAS;AAAA,EACnB;AACF;;;ACRA,IAAAC,4BAAqB;AAEd,SAAS,6BAA6B;AAC3C,SAAO;AAAA;AAAA;AAAA,IAGL,QAAQ,0BAAAC,QAAS;AAAA,IACjB,eAAe;AAAA,MACb,gBAAgB;AAAA,IAClB;AAAA,EACF;AACF;;;ACLO,SAAS,mBAAmB,iBAA0B;AAC3D,QAAM,sBAAsB,0BAA0B;AACtD,QAAM,uBAAuB,2BAA2B;AAExD,SAAO;AAAA,IACL;AAAA,MACE,OAAO,CAAC,mBAAmB;AAAA,MAC3B,iBAAiB;AAAA,MACjB,MAAM;AAAA,MACN,SAAS,EAAE,QAAQ,gBAAgB;AAAA,MACnC,OAAO;AAAA,IACT;AAAA;AAAA;AAAA,IAGA;AAAA,MACE,OAAO,CAAC,gBAAgB;AAAA,MACxB,MAAM;AAAA,MACN,SAAS,EAAE,QAAQ,gBAAgB;AAAA,MACnC,OAAO;AAAA,IACT;AAAA,IACA;AAAA,MACE,OAAO,CAAC,mBAAmB;AAAA,MAC3B,iBAAiB;AAAA,MACjB,MAAM;AAAA,MACN,SAAS,EAAE,QAAQ,gBAAgB;AAAA,MACnC,OAAO;AAAA,QACL,gCAAgC;AAAA,QAChC,sCAAsC;AAAA,QACtC,yCAAyC;AAAA,MAC3C;AAAA,IACF;AAAA,EACF;AACF;","names":["tseslint","import_typescript_eslint","tseslint"]}

package/dist/astro/index.mjs

@@ -1,7 +1,7 @@
 import {
   createAstroConfigs
-} from "../chunk-GSQWHSHV.mjs";
-import "../chunk-QDQUU6QK.mjs";
+} from "../chunk-3LQ4KQP5.mjs";
+import "../chunk-TFVTEI2T.mjs";
 export {
   createAstroConfigs
 };

package/dist/{chunk-3GEFHNU7.mjs → chunk-33N6EMC2.mjs}

@@ -2,7 +2,7 @@ import {
   baseRules,
   createTypedLanguageOptions,
   typeDrivenRules
-} from "./chunk-QDQUU6QK.mjs";
+} from "./chunk-TFVTEI2T.mjs";
 
 // src/constants/nest-entrypoint-file-patterns.ts
 var nestEntrypointFilePatterns = [
@@ -130,4 +130,4 @@ function createNestConfigs(pluginReference) {
 export {
   createNestConfigs
 };
-//# sourceMappingURL=chunk-3GEFHNU7.mjs.map
+//# sourceMappingURL=chunk-33N6EMC2.mjs.map

package/dist/{chunk-GSQWHSHV.mjs → chunk-3LQ4KQP5.mjs}

@@ -2,7 +2,7 @@ import {
   baseRules,
   createBaseLanguageOptions,
   createTypedLanguageOptions
-} from "./chunk-QDQUU6QK.mjs";
+} from "./chunk-TFVTEI2T.mjs";
 
 // src/astro/configs.ts
 function createAstroConfigs(pluginReference) {
@@ -41,4 +41,4 @@ function createAstroConfigs(pluginReference) {
 export {
   createAstroConfigs
 };
-//# sourceMappingURL=chunk-GSQWHSHV.mjs.map
+//# sourceMappingURL=chunk-3LQ4KQP5.mjs.map

package/dist/{chunk-54EOEKYS.mjs → chunk-FTSYXBE4.mjs}

@@ -6,7 +6,7 @@ import {
   baseRules,
   createBaseLanguageOptions,
   createTypedLanguageOptions
-} from "./chunk-QDQUU6QK.mjs";
+} from "./chunk-TFVTEI2T.mjs";
 
 // src/next/configs.ts
 var nextDefaultExportFileGlob = `{${[
@@ -71,4 +71,4 @@ function createNextConfigs(pluginReference) {
 export {
   createNextConfigs
 };
-//# sourceMappingURL=chunk-54EOEKYS.mjs.map
+//# sourceMappingURL=chunk-FTSYXBE4.mjs.map

package/dist/{chunk-77A4SFGD.mjs → chunk-S2MDPOWE.mjs}

@@ -4216,15 +4216,6 @@ var rules = {
   "async-functions-return-result": asyncFunctionsReturnResult,
   "no-ad-hoc-ok-result": noAdHocOkResult,
   "await-requires-result": awaitRequiresResult,
-  // Alias deprecado del nombre anterior; se elimina en una versión futura.
-  "await-requires-try-safe": {
-    ...awaitRequiresResult,
-    meta: {
-      ...awaitRequiresResult.meta,
-      deprecated: true,
-      replacedBy: ["skapxd/await-requires-result"]
-    }
-  },
   "result-error-requires-cause": resultErrorRequiresCause,
   "result-error-requires-handling": resultErrorRequiresHandling,
   "max-hook-size": maxHookSize,
@@ -4268,4 +4259,4 @@ var rules = {
 export {
   rules
 };
-//# sourceMappingURL=chunk-77A4SFGD.mjs.map
+//# sourceMappingURL=chunk-S2MDPOWE.mjs.map