@skapxd/eslint-opinionated 0.10.0 → 0.12.0

package/README.md

@@ -548,9 +548,11 @@ de cada regla):
 | `no-default-export` | `allowFilePatterns` (globs, aditivos a los integrados) |
 | `no-emoji` | `allowFilePatterns` (globs) |
 | `no-functions-inside-components` | `allowJsxCallbacks`, `allowArrayMapCallbacks` (ambas `true` por defecto) |
+| `no-nested-if` | `allowFilePatterns` (globs) |
 | `no-promise-chain` | `methods` |
 | `no-tunnel-props` | `allowFilePatterns` (globs), `allowPropPatterns` (regex) |
 | `prefer-abort-signal` | `allowFilePatterns` (globs), `effectNames` (default `["useEffect", "useLayoutEffect"]`) |
+| `result-error-requires-handling` | `allowFilePatterns` (globs) |
 
 Los `allowFilePatterns` de todas las reglas son **globs** (`*` un segmento,
 `**` cualquier profundidad, `{a,b}` alternativas; un patrón sin prefijo
@@ -564,6 +566,7 @@ matchea en cualquier carpeta). Las 7 reglas restantes no tienen opciones: su
 | `skapxd/one-root-function-per-file` | Un archivo, una función top-level semántica. |
 | `skapxd/async-functions-return-result` | Funciones async de dominio deben retornar `Promise<Result<...>>`. **Apagada por defecto; opt-in** (ver motivos en su sección). |
 | `skapxd/result-error-requires-cause` | Un `Result.err` derivado debe preservar `cause: result.error`. |
+| `skapxd/result-error-requires-handling` | Prohíbe descartar en silencio un Result fallido: el error se transforma o se entrega, nunca se ignora. |
 | `skapxd/await-requires-result` | Todo `await` debe resolver en un `Result`: o la función llamada retorna `Promise<Result<...>>` (preferido) o se envuelve en `trySafe`. **Obligatoria en todos los presets tipados.** |
 | `skapxd/no-ad-hoc-ok-result` | Evita contratos `{ ok: ... }` hechos a mano en async exports. |
 | `skapxd/max-hook-size` | Marca hooks grandes o con demasiados `useState`. |
@@ -571,6 +574,7 @@ matchea en cualquier carpeta). Las 7 reglas restantes no tienen opciones: su
 | `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-emoji` | Prohíbe emojis en strings y JSX; cada sistema los renderiza distinto. Usa un icono SVG. |
+| `skapxd/no-nested-if` | Prohíbe `if` anidados: retorno anticipado o `match()`. Menos carga cognitiva y sin puntos ciegos para las demás reglas. |
 | `skapxd/no-tunnel-props` | Ninguna prop viaja más de un nivel: quien la recibe no puede reenviarla a otro componente. Mata el prop drilling. |
 | `skapxd/prefer-abort-signal` | Listeners en efectos se limpian con `AbortController` (`{ signal }` + `abort()`), no con `removeEventListener`. |
 | `skapxd/no-functions-inside-components` | Prohíbe definir funciones dentro de componentes React. |
@@ -689,10 +693,58 @@ if (!result.ok) {
 }
 ```
 
+Reconoce todas las formas del guard de Result fallido — `!result.ok`,
+`result.ok === false`, `result.ok !== true`, `Result.isErr(result)` y
+`if (result.error)` — y dentro del guard exige el `cause` en todo
+`Result.err(...)`. Un `Result.err()` **sin argumentos** también se reporta:
+descartar el error por completo es el peor caso, no una exención. Y un `cause`
+con otro valor (`cause: new Error(...)`) no cuenta: tiene que ser literalmente
+el `result.error` del guard.
+
 Esta regla es type-aware. Usa TypeScript parser services para confirmar que el
 valor del guard y `Result.err` vienen de `@skapxd/result`. Por eso funciona con
 aliases, re-exports y tipos inferidos, sin depender solo del nombre importado en
-el archivo.
+el archivo. Su punto ciego histórico —el `Result.err` escondido en un `if`
+anidado— lo elimina `skapxd/no-nested-if` de raíz.
+
+### `skapxd/result-error-requires-handling`
+
+La hermana de la anterior cierra la última puerta de evasión: el **descarte
+silencioso**. Detectar el fallo y botarlo sin tocarlo es legal para
+`result-error-requires-cause` (no hay transformación que vigilar), pero deja
+morir información valiosa sin que nadie lo decidiera conscientemente:
+
+```ts
+const result = await copyTextToClipboard(text);
+if (!result.ok) return;            // ❌ el error muere aquí, en silencio
+```
+
+El contrato: dentro de un guard de Result fallido, `result.error` (o el
+result completo) debe **fluir a alguna parte**. Dos salidas:
+
+```ts
+// 1. Transformarlo (y result-error-requires-cause vigila el cause)
+if (!result.ok) {
+  return Result.err({ cause: result.error, message: "...", type: "COPY_FAILED" });
+}
+
+// 2. Entregárselo a alguien: telemetría, estado de error, log de dominio
+if (!result.ok) {
+  trackClipboardFailure(result.error);
+  return;
+}
+
+// (propagar el result completo también vale: `if (!result.ok) return result;`)
+```
+
+**No hay tercera salida.** `void result.error` no cuenta como manejo, y
+manejar sin tocar el error (`setFailed(true)`) tampoco — el detalle se perdió
+igual. Esto es deliberado: si darle seguimiento a un error es crítico o no,
+no puede depender de la interpretación de quien escribe; el camino por
+defecto nunca es ignorarlo.
+
+Type-aware como su hermana: solo aplica a Results reales de `@skapxd/result`,
+con las mismas cinco formas de guard.
 
 ### `skapxd/await-requires-result`
 
@@ -848,6 +900,35 @@ Revisa imports estáticos (`import`), re-exports (`export ... from`) e imports
 dinámicos (`import(...)`). El remedio habitual es un alias de ruta (`@/...`) o
 acercar el módulo a quien lo usa.
 
+### `skapxd/no-nested-if`
+
+Prohíbe un `if` dentro de otro `if` (en la misma función). Cada nivel de
+anidación suma carga cognitiva para quien lee — y además crea puntos ciegos
+para las demás reglas: un `Result.err` dentro de un if anidado quedaba fuera
+del alcance de `result-error-requires-cause`. Esta regla elimina la categoría
+completa de evasión en vez de parchear cada caso.
+
+```ts
+// ❌ anidado: el lector mantiene dos condiciones en la cabeza
+if (!response.ok) {
+  if (shouldReport) {
+    return Result.err({ cause: response.error, message: "...", type: "X" });
+  }
+}
+
+// ✅ retorno anticipado: una condición a la vez, camino feliz sin sangría
+if (!response.ok && shouldReport) {
+  return Result.err({ cause: response.error, message: "...", type: "X" });
+}
+
+// ✅ o match() si son variantes de un mismo valor
+```
+
+No cuenta como anidación: la cadena `else if` (es secuencia, no anidación), y
+una función definida dentro del `if` (unidad cognitiva aparte). El propio
+código de este plugin se aplanó con retorno anticipado al activar la regla —
+cinco casos, todos quedaron más legibles.
+
 ### `skapxd/no-default-export`
 
 Prohíbe `export default` (incluida la forma `export { x as default }`). Con

package/dist/astro/index.d.mts

@@ -14,11 +14,13 @@ declare function createAstroConfigs(pluginReference: unknown): ({
         "skapxd/no-deep-relative-imports": string;
         "skapxd/no-default-export": string;
         "skapxd/no-emoji": string;
+        "skapxd/no-nested-if": string;
         "skapxd/no-promise-chain": string;
         "skapxd/no-try-catch": string;
         "skapxd/one-root-function-per-file": string;
         "skapxd/prefer-ts-pattern": string;
         "skapxd/result-error-requires-cause": string;
+        "skapxd/result-error-requires-handling": string;
     };
 } | {
     files: string[];
@@ -31,11 +33,13 @@ declare function createAstroConfigs(pluginReference: unknown): ({
         "skapxd/no-deep-relative-imports": string;
         "skapxd/no-default-export": string;
         "skapxd/no-emoji": string;
+        "skapxd/no-nested-if": string;
         "skapxd/no-promise-chain": string;
         "skapxd/no-try-catch": string;
         "skapxd/one-root-function-per-file": string;
         "skapxd/prefer-ts-pattern": string;
         "skapxd/result-error-requires-cause": string;
+        "skapxd/result-error-requires-handling": string;
     };
     languageOptions?: undefined;
 } | {
@@ -53,6 +57,7 @@ declare function createAstroConfigs(pluginReference: unknown): ({
     rules: {
         "skapxd/await-requires-result": string;
         "skapxd/result-error-requires-cause": string;
+        "skapxd/result-error-requires-handling": string;
     };
 })[];
 

package/dist/astro/index.d.ts

@@ -14,11 +14,13 @@ declare function createAstroConfigs(pluginReference: unknown): ({
         "skapxd/no-deep-relative-imports": string;
         "skapxd/no-default-export": string;
         "skapxd/no-emoji": string;
+        "skapxd/no-nested-if": string;
         "skapxd/no-promise-chain": string;
         "skapxd/no-try-catch": string;
         "skapxd/one-root-function-per-file": string;
         "skapxd/prefer-ts-pattern": string;
         "skapxd/result-error-requires-cause": string;
+        "skapxd/result-error-requires-handling": string;
     };
 } | {
     files: string[];
@@ -31,11 +33,13 @@ declare function createAstroConfigs(pluginReference: unknown): ({
         "skapxd/no-deep-relative-imports": string;
         "skapxd/no-default-export": string;
         "skapxd/no-emoji": string;
+        "skapxd/no-nested-if": string;
         "skapxd/no-promise-chain": string;
         "skapxd/no-try-catch": string;
         "skapxd/one-root-function-per-file": string;
         "skapxd/prefer-ts-pattern": string;
         "skapxd/result-error-requires-cause": string;
+        "skapxd/result-error-requires-handling": string;
     };
     languageOptions?: undefined;
 } | {
@@ -53,6 +57,7 @@ declare function createAstroConfigs(pluginReference: unknown): ({
     rules: {
         "skapxd/await-requires-result": string;
         "skapxd/result-error-requires-cause": string;
+        "skapxd/result-error-requires-handling": string;
     };
 })[];
 

package/dist/astro/index.js

@@ -40,11 +40,13 @@ var baseRules = {
   "skapxd/no-deep-relative-imports": "error",
   "skapxd/no-default-export": "error",
   "skapxd/no-emoji": "error",
+  "skapxd/no-nested-if": "error",
   "skapxd/no-promise-chain": "error",
   "skapxd/no-try-catch": "error",
   "skapxd/one-root-function-per-file": "error",
   "skapxd/prefer-ts-pattern": "error",
-  "skapxd/result-error-requires-cause": "error"
+  "skapxd/result-error-requires-cause": "error",
+  "skapxd/result-error-requires-handling": "error"
 };
 
 // src/shared/configs/create-base-language-options.ts
@@ -95,7 +97,8 @@ function createAstroConfigs(pluginReference) {
       plugins: { skapxd: pluginReference },
       rules: {
         "skapxd/await-requires-result": "error",
-        "skapxd/result-error-requires-cause": "error"
+        "skapxd/result-error-requires-cause": "error",
+        "skapxd/result-error-requires-handling": "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/no-ad-hoc-ok-result\": \"error\",\n  \"skapxd/no-deep-relative-imports\": \"error\",\n  \"skapxd/no-default-export\": \"error\",\n  \"skapxd/no-emoji\": \"error\",\n  \"skapxd/no-promise-chain\": \"error\",\n  \"skapxd/no-try-catch\": \"error\",\n  \"skapxd/one-root-function-per-file\": \"error\",\n  \"skapxd/prefer-ts-pattern\": \"error\",\n  \"skapxd/result-error-requires-cause\": \"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      },\n    },\n  ];\n}\n"],"mappings":";;;;;;;;;;;;;;;;;;;;;;;;;;;;;;AAAA;AAAA;AAAA;AAAA;AAAA;;;ACAO,IAAM,YAAY;AAAA,EACvB,8BAA8B;AAAA,EAC9B,mCAAmC;AAAA,EACnC,4BAA4B;AAAA,EAC5B,mBAAmB;AAAA,EACnB,2BAA2B;AAAA,EAC3B,uBAAuB;AAAA,EACvB,qCAAqC;AAAA,EACrC,4BAA4B;AAAA,EAC5B,sCAAsC;AACxC;;;ACVA,+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,MACxC;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/no-ad-hoc-ok-result\": \"error\",\n  \"skapxd/no-deep-relative-imports\": \"error\",\n  \"skapxd/no-default-export\": \"error\",\n  \"skapxd/no-emoji\": \"error\",\n  \"skapxd/no-nested-if\": \"error\",\n  \"skapxd/no-promise-chain\": \"error\",\n  \"skapxd/no-try-catch\": \"error\",\n  \"skapxd/one-root-function-per-file\": \"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,8BAA8B;AAAA,EAC9B,mCAAmC;AAAA,EACnC,4BAA4B;AAAA,EAC5B,mBAAmB;AAAA,EACnB,uBAAuB;AAAA,EACvB,2BAA2B;AAAA,EAC3B,uBAAuB;AAAA,EACvB,qCAAqC;AAAA,EACrC,4BAA4B;AAAA,EAC5B,sCAAsC;AAAA,EACtC,yCAAyC;AAC3C;;;ACZA,+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-7VZBQ6FD.mjs";
-import "../chunk-GEVX3BTI.mjs";
+} from "../chunk-JA2ZO3KQ.mjs";
+import "../chunk-QADXO5IL.mjs";
 export {
   createAstroConfigs
 };

package/dist/{chunk-3WLCAPUA.mjs → chunk-DJXM5PMG.mjs}

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

package/dist/{chunk-7VZBQ6FD.mjs → chunk-JA2ZO3KQ.mjs}

@@ -2,7 +2,7 @@ import {
   baseRules,
   createBaseLanguageOptions,
   createTypedLanguageOptions
-} from "./chunk-GEVX3BTI.mjs";
+} from "./chunk-QADXO5IL.mjs";
 
 // src/astro/configs.ts
 function createAstroConfigs(pluginReference) {
@@ -31,7 +31,8 @@ function createAstroConfigs(pluginReference) {
       plugins: { skapxd: pluginReference },
       rules: {
         "skapxd/await-requires-result": "error",
-        "skapxd/result-error-requires-cause": "error"
+        "skapxd/result-error-requires-cause": "error",
+        "skapxd/result-error-requires-handling": "error"
       }
     }
   ];
@@ -40,4 +41,4 @@ function createAstroConfigs(pluginReference) {
 export {
   createAstroConfigs
 };
-//# sourceMappingURL=chunk-7VZBQ6FD.mjs.map
+//# sourceMappingURL=chunk-JA2ZO3KQ.mjs.map

package/dist/{chunk-7VZBQ6FD.mjs.map → chunk-JA2ZO3KQ.mjs.map}

@@ -1 +1 @@
-{"version":3,"sources":["../src/astro/configs.ts"],"sourcesContent":["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      },\n    },\n  ];\n}\n"],"mappings":";;;;;;;AAMO,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,MACxC;AAAA,IACF;AAAA,EACF;AACF;","names":[]}
+{"version":3,"sources":["../src/astro/configs.ts"],"sourcesContent":["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":";;;;;;;AAMO,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":[]}

package/dist/{chunk-WO7EUISC.mjs → chunk-MSOA2V5B.mjs}

@@ -644,14 +644,16 @@ var asyncFunctionsReturnResult = {
         return;
       }
       const returnType = node.returnType?.typeAnnotation;
+      const missingReturnTypeIsReportable = options.checkMissingReturnType || containsCallNamed(node.body, options.checkMissingReturnTypeWhenCallNames);
+      if (!returnType && missingReturnTypeIsReportable) {
+        context.report({
+          data: { name: functionName },
+          messageId: "missingReturnType",
+          node: reportNode
+        });
+        return;
+      }
       if (!returnType) {
-        if (options.checkMissingReturnType || containsCallNamed(node.body, options.checkMissingReturnTypeWhenCallNames)) {
-          context.report({
-            data: { name: functionName },
-            messageId: "missingReturnType",
-            node: reportNode
-          });
-        }
         return;
       }
       if (isSkapxdResultReturnType(returnType)) {
@@ -813,14 +815,13 @@ function getAwaitScopeName(node) {
 function getEnclosingTrySafeCall(node, trySafeCallNames) {
   let currentNode = node.parent;
   while (currentNode) {
-    if (isFunctionNode(currentNode)) {
-      const parent = currentNode.parent;
-      if (parent?.type === "CallExpression" && parent.arguments.includes(currentNode) && isCalleeNamed(parent.callee, trySafeCallNames)) {
-        return parent;
-      }
-      return null;
+    if (!isFunctionNode(currentNode)) {
+      currentNode = currentNode.parent;
+      continue;
     }
-    currentNode = currentNode.parent;
+    const parent = currentNode.parent;
+    const isTrySafeArgument = parent?.type === "CallExpression" && parent.arguments.includes(currentNode) && isCalleeNamed(parent.callee, trySafeCallNames);
+    return isTrySafeArgument ? parent : null;
   }
   return null;
 }
@@ -954,6 +955,18 @@ var awaitRequiresResult = {
   }
 };
 
+// src/utils/get-error-member-object.ts
+function getErrorMemberObject(node) {
+  const unwrappedNode = unwrapExpression(node);
+  if (unwrappedNode.type !== "MemberExpression" || unwrappedNode.object.type !== "Identifier" || !isMemberPropertyNamed(unwrappedNode, "error")) {
+    return null;
+  }
+  return {
+    name: unwrappedNode.object.name,
+    node: unwrappedNode.object
+  };
+}
+
 // src/utils/get-ok-member-object.ts
 function getOkMemberObject(node) {
   const unwrappedNode = unwrapExpression(node);
@@ -1002,6 +1015,9 @@ function getResultCheckArgument(node, methodName) {
 // src/utils/get-failed-result-guard.ts
 function getFailedResultGuard(node) {
   const unwrappedNode = unwrapExpression(node);
+  if (unwrappedNode.type === "MemberExpression") {
+    return getErrorMemberObject(unwrappedNode);
+  }
   if (unwrappedNode.type === "UnaryExpression" && unwrappedNode.operator === "!") {
     return getOkMemberObject(unwrappedNode.argument) ?? getResultCheckArgument(unwrappedNode.argument, "isOk");
   }
@@ -1119,10 +1135,7 @@ var resultErrorRequiresCause = {
           if (!isSkapxdResultErrCall(resultErrCall, typeContext)) {
             continue;
           }
-          if (resultErrCall.arguments.length === 0) {
-            continue;
-          }
-          if (resultErrPreservesCause(resultErrCall.arguments[0], resultGuard.name)) {
+          if (resultErrCall.arguments.length > 0 && resultErrPreservesCause(resultErrCall.arguments[0], resultGuard.name)) {
             continue;
           }
           context.report({
@@ -1138,6 +1151,88 @@ var resultErrorRequiresCause = {
   }
 };
 
+// src/utils/collect-identifiers-named.ts
+function collectIdentifiersNamed(node, name, results = []) {
+  if (node?.type === "Identifier" && node.name === name) {
+    results.push(node);
+  }
+  for (const child of getNodeChildren(node)) {
+    collectIdentifiersNamed(child, name, results);
+  }
+  return results;
+}
+
+// src/utils/get-result-error-requires-handling-options.ts
+function getResultErrorRequiresHandlingOptions(options = {}) {
+  return {
+    allowFilePatterns: options.allowFilePatterns ?? []
+  };
+}
+
+// src/utils/is-consumed-result-reference.ts
+function isConsumedResultReference(identifier) {
+  const member = identifier.parent?.type === "MemberExpression" && identifier.parent.object === identifier ? identifier.parent : null;
+  const reference = member ?? identifier;
+  const parent = reference.parent;
+  if (parent?.type === "UnaryExpression" && parent.operator === "void") {
+    return false;
+  }
+  return parent?.type !== "ExpressionStatement";
+}
+
+// src/rules/result-error-requires-handling.ts
+var resultErrorRequiresHandling = {
+  meta: {
+    type: "problem",
+    docs: {
+      description: "Prohibe descartar en silencio un Result fallido: el error se transforma o se entrega, nunca se ignora."
+    },
+    messages: {
+      unhandledResultError: "El guard detecta que `{{name}}` fallo, pero `{{name}}.error` muere aqui sin seguimiento. Transformalo (`Result.err({ cause: {{name}}.error, ... })`), entregaselo a alguien (telemetria, estado de error, log de dominio) o propaga el result completo. Si darle seguimiento es critico o no, no es una interpretacion: todo error fluye a alguna parte."
+    },
+    schema: [
+      {
+        additionalProperties: false,
+        properties: {
+          allowFilePatterns: {
+            items: { type: "string" },
+            type: "array"
+          }
+        },
+        type: "object"
+      }
+    ]
+  },
+  create(context) {
+    const options = getResultErrorRequiresHandlingOptions(context.options[0]);
+    const filename = context.filename ?? context.getFilename();
+    const typeContext = getTypeContext(context);
+    if (matchesAnyGlob(filename, options.allowFilePatterns)) {
+      return {};
+    }
+    return {
+      IfStatement(node) {
+        const resultGuard = getFailedResultGuard(node.test);
+        if (!typeContext || !resultGuard || !isSkapxdResultExpression(resultGuard.node, typeContext)) {
+          return;
+        }
+        const references = collectIdentifiersNamed(
+          node.consequent,
+          resultGuard.name
+        );
+        if (references.some(isConsumedResultReference)) {
+          return;
+        }
+        context.report({
+          data: { name: resultGuard.name },
+          messageId: "unhandledResultError",
+          node: node.test
+        });
+      }
+    };
+  }
+};
+
 // src/utils/count-own-use-state-calls-in-node.ts
 function countOwnUseStateCallsInNode(node) {
   if (!isAstNode(node)) {
@@ -1448,17 +1543,6 @@ var noEmoji = {
   }
 };
 
-// src/utils/collect-identifiers-named.ts
-function collectIdentifiersNamed(node, name, results = []) {
-  if (node?.type === "Identifier" && node.name === name) {
-    results.push(node);
-  }
-  for (const child of getNodeChildren(node)) {
-    collectIdentifiersNamed(child, name, results);
-  }
-  return results;
-}
-
 // src/utils/get-no-tunnel-props-options.ts
 function getNoTunnelPropsOptions(options = {}) {
   return {
@@ -1740,14 +1824,9 @@ function hasAbortSignalOption(callExpression, sourceCode, typeContext) {
   if (options.type === "ObjectExpression") {
     return objectExpressionHasSignal(options);
   }
-  if (options.type === "Identifier") {
-    const initializer = getVariableInitializer(
-      options,
-      sourceCode.getScope(options)
-    );
-    if (initializer?.type === "ObjectExpression") {
-      return objectExpressionHasSignal(initializer);
-    }
+  const initializer = options.type === "Identifier" ? getVariableInitializer(options, sourceCode.getScope(options)) : null;
+  if (initializer?.type === "ObjectExpression") {
+    return objectExpressionHasSignal(initializer);
   }
   if (typeContext) {
     const type = typeContext.services.getTypeAtLocation(options);
@@ -1760,11 +1839,9 @@ function hasAbortSignalOption(callExpression, sourceCode, typeContext) {
 function isInsideEffectCallback(node, effectNames) {
   let current = node.parent;
   while (current) {
-    if (isFunctionNode(current)) {
-      const call = current.parent;
-      if (call?.type === "CallExpression" && call.arguments[0] === current && isCalleeNamed(call.callee, effectNames)) {
-        return true;
-      }
+    const call = isFunctionNode(current) ? current.parent : null;
+    if (call?.type === "CallExpression" && call.arguments[0] === current && isCalleeNamed(call.callee, effectNames)) {
+      return true;
     }
     current = current.parent;
   }
@@ -1903,6 +1980,67 @@ var noJsxTernaryNull = {
   }
 };
 
+// src/utils/get-no-nested-if-options.ts
+function getNoNestedIfOptions(options = {}) {
+  return {
+    allowFilePatterns: options.allowFilePatterns ?? []
+  };
+}
+
+// src/utils/is-nested-if-statement.ts
+function isNestedIfStatement(node) {
+  if (node.parent?.type === "IfStatement" && node.parent.alternate === node) {
+    return false;
+  }
+  let current = node.parent;
+  while (current && !isFunctionNode(current)) {
+    if (current.type === "IfStatement") {
+      return true;
+    }
+    current = current.parent;
+  }
+  return false;
+}
+
+// src/rules/no-nested-if.ts
+var noNestedIf = {
+  meta: {
+    type: "suggestion",
+    docs: {
+      description: "Prohibe if anidados; usa retorno anticipado (guard clauses) o match() de ts-pattern."
+    },
+    messages: {
+      noNestedIf: "No anides un if dentro de otro: cada nivel suma carga cognitiva y crea puntos ciegos para las demas reglas. Aplana con retorno anticipado (`if (!x) return ...;` y sigue el camino feliz) o decide con `match()` de ts-pattern si son variantes de un mismo valor."
+    },
+    schema: [
+      {
+        additionalProperties: false,
+        properties: {
+          allowFilePatterns: {
+            items: { type: "string" },
+            type: "array"
+          }
+        },
+        type: "object"
+      }
+    ]
+  },
+  create(context) {
+    const options = getNoNestedIfOptions(context.options[0]);
+    const filename = context.filename ?? context.getFilename();
+    if (matchesAnyGlob(filename, options.allowFilePatterns)) {
+      return {};
+    }
+    return {
+      IfStatement(node) {
+        if (isNestedIfStatement(node)) {
+          context.report({ messageId: "noNestedIf", node });
+        }
+      }
+    };
+  }
+};
+
 // src/utils/is-promise-type.ts
 function isPromiseType(type, typeContext) {
   return typeContext.checker.getPromisedTypeOfPromise(type) !== void 0;
@@ -1945,11 +2083,11 @@ var noPromiseChain = {
         if (!method) {
           return;
         }
-        if (typeContext) {
-          const type = typeContext.services.getTypeAtLocation(callee.object);
-          if (!isPromiseType(type, typeContext)) {
-            return;
-          }
+        if (typeContext && !isPromiseType(
+          typeContext.services.getTypeAtLocation(callee.object),
+          typeContext
+        )) {
+          return;
         }
         context.report({
           data: { method },
@@ -1978,6 +2116,7 @@ var rules = {
     }
   },
   "result-error-requires-cause": resultErrorRequiresCause,
+  "result-error-requires-handling": resultErrorRequiresHandling,
   "max-hook-size": maxHookSize,
   "no-deep-relative-imports": noDeepRelativeImports,
   "no-default-export": noDefaultExport,
@@ -1988,10 +2127,11 @@ var rules = {
   "prefer-abort-signal": preferAbortSignal,
   "prefer-ts-pattern": preferTsPattern,
   "no-jsx-ternary-null": noJsxTernaryNull,
+  "no-nested-if": noNestedIf,
   "no-promise-chain": noPromiseChain
 };
 
 export {
   rules
 };
-//# sourceMappingURL=chunk-WO7EUISC.mjs.map
+//# sourceMappingURL=chunk-MSOA2V5B.mjs.map