@skapxd/eslint-opinionated 0.8.0 → 0.10.0

package/README.md

@@ -549,6 +549,8 @@ de cada regla):
 | `no-emoji` | `allowFilePatterns` (globs) |
 | `no-functions-inside-components` | `allowJsxCallbacks`, `allowArrayMapCallbacks` (ambas `true` por defecto) |
 | `no-promise-chain` | `methods` |
+| `no-tunnel-props` | `allowFilePatterns` (globs), `allowPropPatterns` (regex) |
+| `prefer-abort-signal` | `allowFilePatterns` (globs), `effectNames` (default `["useEffect", "useLayoutEffect"]`) |
 
 Los `allowFilePatterns` de todas las reglas son **globs** (`*` un segmento,
 `**` cualquier profundidad, `{a,b}` alternativas; un patrón sin prefijo
@@ -569,6 +571,8 @@ 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-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. |
 | `skapxd/no-try-catch` | Prohíbe `try/catch`; usa `trySafe` de `@skapxd/result`. |
 | `skapxd/no-promise-chain` | Prohíbe `.then/.catch/.finally`; usa `await` (+ `trySafe`). |
@@ -912,6 +916,58 @@ eximir archivos completos (fixtures, seeds), usa `allowFilePatterns`:
 }]
 ```
 
+### `skapxd/no-tunnel-props`
+
+**Ninguna prop viaja más de un nivel.** El contrato de saltos: quien **crea**
+un valor (estado de un hook, acción de un store, dato calculado) puede pasarlo
+a UN hijo; quien lo **recibe** como prop no puede reenviarlo a otro
+componente. Eso prohíbe exactamente la cadena `abuelo → padre → hijo` — el
+prop drilling — sin tocar el paso legítimo de un nivel.
+
+```tsx
+// ✅ primer salto: el abuelo CREA la acción y la baja un nivel
+const Abuelo = () => {
+  const onSelect = useTranscriptStore((s) => s.select);
+  return <Padre onSelect={onSelect} />;
+};
+
+// ❌ segundo salto: el padre la RECIBE y la reenvía
+const Padre = ({ onSelect }) => <Hijo onSelect={onSelect} />;
+
+// ❌ el rename no lo esconde, y usarla localmente no autoriza el reenvío
+const Padre = ({ onSelect }) => <Hijo handler={onSelect} />;
+
+// ❌ el túnel puro
+const Padre = ({ ...props }) => <Hijo {...props} />;
+```
+
+La detección es local y exacta: si el identifier que pones en una prop de otro
+componente viene de tus **props destructuradas**, no lo creaste tú — es su
+segundo salto.
+
+Las salidas que sugiere el mensaje:
+
+1. **Store global o custom hook**: la acción/estado vive en un store (p. ej.
+   [zustand](https://github.com/pmndrs/zustand)) o un hook, y el componente
+   que la necesita la consume directo — la cadena desaparece:
+
+   ```tsx
+   function Hijo({ entry }: { entry: Entry }) {
+     const select = useTranscriptStore((s) => s.select);
+     return <button onClick={() => select(entry.id)}>…</button>;
+   }
+   ```
+
+2. **Composición**: el padre arma el JSX y el intermedio recibe `children` —
+   el dato viaja dentro del JSX, no por props. (`children` nunca cuenta como
+   túnel: es la alternativa.)
+
+No cuenta como reenvío: usar la prop (`<h2>{title}</h2>`), derivar datos
+(`title={game.title}`), o pasarla a un elemento **nativo** (`value={value}`
+en un `<input>` es la frontera con el DOM). Para wrappers legítimos de un
+design system, exime props por nombre (`allowPropPatterns: ["^className$"]`)
+o archivos completos (`allowFilePatterns`).
+
 ### `skapxd/no-functions-inside-components`
 
 Prohíbe definir funciones **con peso propio** dentro de un componente React
@@ -1031,6 +1087,53 @@ const label = match(status)
   .exhaustive();
 ```
 
+### `skapxd/prefer-abort-signal`
+
+Dentro de un `useEffect`/`useLayoutEffect`, los listeners se limpian con
+`AbortController`, no con `removeEventListener` manual:
+
+```ts
+// ❌ registro y limpieza espejados a mano
+useEffect(() => {
+  const media = window.matchMedia("(prefers-color-scheme: dark)");
+  media.addEventListener("change", onSystemChange);
+  return () => media.removeEventListener("change", onSystemChange);
+}, [settings]);
+
+// ✅ un AbortController por efecto
+useEffect(() => {
+  const controller = new AbortController();
+  const media = window.matchMedia("(prefers-color-scheme: dark)");
+  media.addEventListener("change", onSystemChange, { signal: controller.signal });
+  return () => controller.abort();
+}, [settings]);
+```
+
+Por qué: un solo `abort()` limpia **todos** los listeners del efecto (no hay
+que espejar cada `add` con su `remove`), y elimina el bug clásico de pasar una
+referencia distinta a `removeEventListener` (un `.bind()` o una arrow nueva)
+que deja el listener vivo para siempre.
+
+Reporta dos cosas dentro del callback del efecto (incluidas sus funciones
+anidadas y el cleanup): `addEventListener` sin `signal` en las options, y
+cualquier `removeEventListener`. Fuera de un efecto la regla no opina.
+
+Cuando las options no son un objeto literal, la verificación resuelve en
+capas:
+
+1. **Por scope**: `addEventListener("x", fn, opts)` sigue `opts` hasta su
+   `const opts = {...}` y lo inspecciona — sin necesitar type-checking.
+2. **Por tipo** (con `projectService`): si no hay inicializador visible (un
+   parámetro, un import), pregunta al checker si el **tipo** declara `signal`;
+   si ni el tipo la tiene, es imposible que llegue y se reporta.
+3. Sin inicializador ni tipos: beneficio de la duda.
+
+El boolean de capture (`addEventListener("x", fn, true)`) se reporta siempre:
+no puede traer `signal`.
+
+`effectNames` permite cubrir wrappers propios (`["useEffect",
+"useLayoutEffect", "useIsomorphicEffect"]`).
+
 ### `skapxd/no-jsx-ternary-null`
 
 Cuando renderizas JSX condicional y una rama del ternario es `null`, prefiere la

package/dist/astro/index.mjs

@@ -1,7 +1,7 @@
 import {
   createAstroConfigs
-} from "../chunk-MSH2BTXH.mjs";
-import "../chunk-Z6CU2N4C.mjs";
+} from "../chunk-7VZBQ6FD.mjs";
+import "../chunk-GEVX3BTI.mjs";
 export {
   createAstroConfigs
 };

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

@@ -6,7 +6,7 @@ import {
   baseRules,
   createBaseLanguageOptions,
   createTypedLanguageOptions
-} from "./chunk-Z6CU2N4C.mjs";
+} from "./chunk-GEVX3BTI.mjs";
 
 // src/next/configs.ts
 var nextDefaultExportFileGlob = `{${[
@@ -53,6 +53,8 @@ function createNextConfigs(pluginReference) {
       rules: {
         "skapxd/jsx-return-name-pascal-case": "error",
         "skapxd/no-functions-inside-components": "error",
+        "skapxd/no-tunnel-props": "error",
+        "skapxd/prefer-abort-signal": "error",
         "skapxd/no-jsx-ternary-null": "error",
         "skapxd/max-hook-size": [
           "error",
@@ -69,4 +71,4 @@ function createNextConfigs(pluginReference) {
 export {
   createNextConfigs
 };
-//# sourceMappingURL=chunk-2IXJXIW5.mjs.map
+//# sourceMappingURL=chunk-3WLCAPUA.mjs.map

package/dist/{chunk-2IXJXIW5.mjs.map → chunk-3WLCAPUA.mjs.map}

@@ -1 +1 @@
-{"version":3,"sources":["../src/next/configs.ts"],"sourcesContent":["import { nextAppMetadataFileStems } from \"#/constants/next-app-metadata-file-stems\";\nimport { nextAppRouteSegmentFileStems } from \"#/constants/next-app-route-segment-file-stems\";\nimport {\n  baseRules,\n  createBaseLanguageOptions,\n  createTypedLanguageOptions,\n} from \"#/shared/configs\";\n\n// Entrypoints donde Next exige `export default` (page, layout, sitemap, ...):\n// la regla no-default-export los exime automáticamente en este preset.\nconst nextDefaultExportFileGlob = `{${[\n  ...nextAppRouteSegmentFileStems,\n  ...nextAppMetadataFileStems,\n].join(\",\")}}.{js,jsx,ts,tsx}`;\n\nexport function createNextConfigs(pluginReference: unknown) {\n  const baseLanguageOptions = createBaseLanguageOptions();\n  const typedLanguageOptions = createTypedLanguageOptions();\n\n  return [\n    {\n      languageOptions: baseLanguageOptions,\n      name: \"skapxd/next/base\",\n      plugins: { skapxd: pluginReference },\n      rules: {\n        ...baseRules,\n        \"skapxd/no-default-export\": [\n          \"error\",\n          {\n            allowFilePatterns: [nextDefaultExportFileGlob],\n          },\n        ],\n      },\n    },\n    {\n      files: [\"src/app/api/**/*.{ts,tsx}\", \"src/server/**/*.{ts,tsx}\"],\n      languageOptions: typedLanguageOptions,\n      name: \"skapxd/next/server\",\n      plugins: { skapxd: pluginReference },\n      rules: {\n        ...baseRules,\n        // Obligatoria: todo await resuelve en Result. A diferencia de\n        // async-functions-return-result (apagada por defecto), no necesita\n        // excepciones para los entrypoints de Next: envolver un await en\n        // trySafe es compatible con cualquier firma que imponga el framework.\n        \"skapxd/await-requires-result\": \"error\",\n      },\n    },\n    {\n      files: [\"**/*.tsx\"],\n      languageOptions: baseLanguageOptions,\n      name: \"skapxd/next/react\",\n      plugins: { skapxd: pluginReference },\n      rules: {\n        \"skapxd/jsx-return-name-pascal-case\": \"error\",\n        \"skapxd/no-functions-inside-components\": \"error\",\n        \"skapxd/no-jsx-ternary-null\": \"error\",\n        \"skapxd/max-hook-size\": [\n          \"error\",\n          {\n            maxLines: 120,\n            maxUseState: 1,\n          },\n        ],\n      },\n    },\n  ];\n}\n"],"mappings":";;;;;;;;;;;AAUA,IAAM,4BAA4B,IAAI;AAAA,EACpC,GAAG;AAAA,EACH,GAAG;AACL,EAAE,KAAK,GAAG,CAAC;AAEJ,SAAS,kBAAkB,iBAA0B;AAC1D,QAAM,sBAAsB,0BAA0B;AACtD,QAAM,uBAAuB,2BAA2B;AAExD,SAAO;AAAA,IACL;AAAA,MACE,iBAAiB;AAAA,MACjB,MAAM;AAAA,MACN,SAAS,EAAE,QAAQ,gBAAgB;AAAA,MACnC,OAAO;AAAA,QACL,GAAG;AAAA,QACH,4BAA4B;AAAA,UAC1B;AAAA,UACA;AAAA,YACE,mBAAmB,CAAC,yBAAyB;AAAA,UAC/C;AAAA,QACF;AAAA,MACF;AAAA,IACF;AAAA,IACA;AAAA,MACE,OAAO,CAAC,6BAA6B,0BAA0B;AAAA,MAC/D,iBAAiB;AAAA,MACjB,MAAM;AAAA,MACN,SAAS,EAAE,QAAQ,gBAAgB;AAAA,MACnC,OAAO;AAAA,QACL,GAAG;AAAA;AAAA;AAAA;AAAA;AAAA,QAKH,gCAAgC;AAAA,MAClC;AAAA,IACF;AAAA,IACA;AAAA,MACE,OAAO,CAAC,UAAU;AAAA,MAClB,iBAAiB;AAAA,MACjB,MAAM;AAAA,MACN,SAAS,EAAE,QAAQ,gBAAgB;AAAA,MACnC,OAAO;AAAA,QACL,sCAAsC;AAAA,QACtC,yCAAyC;AAAA,QACzC,8BAA8B;AAAA,QAC9B,wBAAwB;AAAA,UACtB;AAAA,UACA;AAAA,YACE,UAAU;AAAA,YACV,aAAa;AAAA,UACf;AAAA,QACF;AAAA,MACF;AAAA,IACF;AAAA,EACF;AACF;","names":[]}
+{"version":3,"sources":["../src/next/configs.ts"],"sourcesContent":["import { nextAppMetadataFileStems } from \"#/constants/next-app-metadata-file-stems\";\nimport { nextAppRouteSegmentFileStems } from \"#/constants/next-app-route-segment-file-stems\";\nimport {\n  baseRules,\n  createBaseLanguageOptions,\n  createTypedLanguageOptions,\n} from \"#/shared/configs\";\n\n// Entrypoints donde Next exige `export default` (page, layout, sitemap, ...):\n// la regla no-default-export los exime automáticamente en este preset.\nconst nextDefaultExportFileGlob = `{${[\n  ...nextAppRouteSegmentFileStems,\n  ...nextAppMetadataFileStems,\n].join(\",\")}}.{js,jsx,ts,tsx}`;\n\nexport function createNextConfigs(pluginReference: unknown) {\n  const baseLanguageOptions = createBaseLanguageOptions();\n  const typedLanguageOptions = createTypedLanguageOptions();\n\n  return [\n    {\n      languageOptions: baseLanguageOptions,\n      name: \"skapxd/next/base\",\n      plugins: { skapxd: pluginReference },\n      rules: {\n        ...baseRules,\n        \"skapxd/no-default-export\": [\n          \"error\",\n          {\n            allowFilePatterns: [nextDefaultExportFileGlob],\n          },\n        ],\n      },\n    },\n    {\n      files: [\"src/app/api/**/*.{ts,tsx}\", \"src/server/**/*.{ts,tsx}\"],\n      languageOptions: typedLanguageOptions,\n      name: \"skapxd/next/server\",\n      plugins: { skapxd: pluginReference },\n      rules: {\n        ...baseRules,\n        // Obligatoria: todo await resuelve en Result. A diferencia de\n        // async-functions-return-result (apagada por defecto), no necesita\n        // excepciones para los entrypoints de Next: envolver un await en\n        // trySafe es compatible con cualquier firma que imponga el framework.\n        \"skapxd/await-requires-result\": \"error\",\n      },\n    },\n    {\n      files: [\"**/*.tsx\"],\n      languageOptions: baseLanguageOptions,\n      name: \"skapxd/next/react\",\n      plugins: { skapxd: pluginReference },\n      rules: {\n        \"skapxd/jsx-return-name-pascal-case\": \"error\",\n        \"skapxd/no-functions-inside-components\": \"error\",\n        \"skapxd/no-tunnel-props\": \"error\",\n        \"skapxd/prefer-abort-signal\": \"error\",\n        \"skapxd/no-jsx-ternary-null\": \"error\",\n        \"skapxd/max-hook-size\": [\n          \"error\",\n          {\n            maxLines: 120,\n            maxUseState: 1,\n          },\n        ],\n      },\n    },\n  ];\n}\n"],"mappings":";;;;;;;;;;;AAUA,IAAM,4BAA4B,IAAI;AAAA,EACpC,GAAG;AAAA,EACH,GAAG;AACL,EAAE,KAAK,GAAG,CAAC;AAEJ,SAAS,kBAAkB,iBAA0B;AAC1D,QAAM,sBAAsB,0BAA0B;AACtD,QAAM,uBAAuB,2BAA2B;AAExD,SAAO;AAAA,IACL;AAAA,MACE,iBAAiB;AAAA,MACjB,MAAM;AAAA,MACN,SAAS,EAAE,QAAQ,gBAAgB;AAAA,MACnC,OAAO;AAAA,QACL,GAAG;AAAA,QACH,4BAA4B;AAAA,UAC1B;AAAA,UACA;AAAA,YACE,mBAAmB,CAAC,yBAAyB;AAAA,UAC/C;AAAA,QACF;AAAA,MACF;AAAA,IACF;AAAA,IACA;AAAA,MACE,OAAO,CAAC,6BAA6B,0BAA0B;AAAA,MAC/D,iBAAiB;AAAA,MACjB,MAAM;AAAA,MACN,SAAS,EAAE,QAAQ,gBAAgB;AAAA,MACnC,OAAO;AAAA,QACL,GAAG;AAAA;AAAA;AAAA;AAAA;AAAA,QAKH,gCAAgC;AAAA,MAClC;AAAA,IACF;AAAA,IACA;AAAA,MACE,OAAO,CAAC,UAAU;AAAA,MAClB,iBAAiB;AAAA,MACjB,MAAM;AAAA,MACN,SAAS,EAAE,QAAQ,gBAAgB;AAAA,MACnC,OAAO;AAAA,QACL,sCAAsC;AAAA,QACtC,yCAAyC;AAAA,QACzC,0BAA0B;AAAA,QAC1B,8BAA8B;AAAA,QAC9B,8BAA8B;AAAA,QAC9B,wBAAwB;AAAA,UACtB;AAAA,UACA;AAAA,YACE,UAAU;AAAA,YACV,aAAa;AAAA,UACf;AAAA,QACF;AAAA,MACF;AAAA,IACF;AAAA,EACF;AACF;","names":[]}

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

@@ -2,7 +2,7 @@ import {
   baseRules,
   createBaseLanguageOptions,
   createTypedLanguageOptions
-} from "./chunk-Z6CU2N4C.mjs";
+} from "./chunk-GEVX3BTI.mjs";
 
 // src/astro/configs.ts
 function createAstroConfigs(pluginReference) {
@@ -40,4 +40,4 @@ function createAstroConfigs(pluginReference) {
 export {
   createAstroConfigs
 };
-//# sourceMappingURL=chunk-MSH2BTXH.mjs.map
+//# sourceMappingURL=chunk-7VZBQ6FD.mjs.map

package/dist/{chunk-Z6CU2N4C.mjs → chunk-GEVX3BTI.mjs}

@@ -70,7 +70,14 @@ function createSharedConfigs(pluginReference) {
         // en trySafe en el sitio.
         "skapxd/await-requires-result": "error",
         "skapxd/jsx-return-name-pascal-case": "error",
+        // Anti prop-drilling: ninguna prop viaja más de un nivel. Quien la
+        // crea puede pasarla a UN hijo; quien la recibe no la reenvía —
+        // estado y acciones a un store global o custom hook.
         "skapxd/no-functions-inside-components": "error",
+        "skapxd/no-tunnel-props": "error",
+        // Listeners en efectos: un AbortController por efecto, cleanup con
+        // un solo abort() en vez de removeEventListener por listener.
+        "skapxd/prefer-abort-signal": "error",
         "skapxd/no-jsx-ternary-null": "error",
         "skapxd/max-hook-size": [
           "error",
@@ -107,4 +114,4 @@ export {
   createSharedConfigs,
   strictConfig
 };
-//# sourceMappingURL=chunk-Z6CU2N4C.mjs.map
+//# sourceMappingURL=chunk-GEVX3BTI.mjs.map

package/dist/{chunk-Z6CU2N4C.mjs.map → chunk-GEVX3BTI.mjs.map}

@@ -1 +1 @@
-{"version":3,"sources":["../src/shared/configs/create-typed-language-options.ts","../src/shared/configs/base-rules.ts","../src/shared/configs/create-base-language-options.ts","../src/shared/configs/create-shared-configs.ts","../src/shared/configs/strict-config.ts"],"sourcesContent":["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","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 { baseRules } from \"./base-rules\";\nimport { createBaseLanguageOptions } from \"./create-base-language-options\";\nimport { createTypedLanguageOptions } from \"./create-typed-language-options\";\n\nexport function createSharedConfigs(pluginReference: unknown) {\n  const baseLanguageOptions = createBaseLanguageOptions();\n  const typedLanguageOptions = createTypedLanguageOptions();\n\n  return {\n    backend: {\n      languageOptions: typedLanguageOptions,\n      name: \"skapxd/shared/backend\",\n      plugins: { skapxd: pluginReference },\n      rules: {\n        ...baseRules,\n        // La regla obligatoria del sistema de errores es await-requires-result\n        // (todo await resuelve en Result). async-functions-return-result queda\n        // apagada por defecto: exigir la firma por decreto choca con los bordes\n        // del framework y bloquea la adopción incremental; la presión sobre los\n        // awaits produce el mismo estado final. Ver \"¿Por qué está apagada por\n        // defecto?\" en el README.\n        \"skapxd/await-requires-result\": \"error\",\n      },\n    },\n    base: {\n      languageOptions: baseLanguageOptions,\n      name: \"skapxd/shared/base\",\n      plugins: { skapxd: pluginReference },\n      rules: baseRules,\n    },\n    frontend: {\n      languageOptions: typedLanguageOptions,\n      name: \"skapxd/shared/frontend\",\n      plugins: { skapxd: pluginReference },\n      rules: {\n        ...baseRules,\n        // En el front no se obliga a retornar Result, pero todo await debe\n        // resolver en uno: o la función llamada ya retorna Promise<Result>\n        // (camino preferido: errores modelados en el dominio) o se envuelve\n        // en trySafe en el sitio.\n        \"skapxd/await-requires-result\": \"error\",\n        \"skapxd/jsx-return-name-pascal-case\": \"error\",\n        \"skapxd/no-functions-inside-components\": \"error\",\n        \"skapxd/no-jsx-ternary-null\": \"error\",\n        \"skapxd/max-hook-size\": [\n          \"error\",\n          {\n            maxLines: 120,\n            maxUseState: 1,\n          },\n        ],\n      },\n    },\n    package: {\n      languageOptions: baseLanguageOptions,\n      name: \"skapxd/shared/package\",\n      plugins: { skapxd: pluginReference },\n      rules: {\n        \"skapxd/one-root-function-per-file\": \"error\",\n      },\n    },\n  };\n}\n","// Config endurecida: ignora TODOS los comentarios `eslint-disable` (y demás\n// directivas inline) en los archivos que cubre. Así ni una persona ni un agente\n// pueden saltarse una regla con `// eslint-disable-next-line`.\nexport const strictConfig = {\n  linterOptions: {\n    noInlineConfig: true,\n  },\n  name: \"skapxd/strict\",\n};\n"],"mappings":";AAAA,OAAO,cAAc;AAEd,SAAS,6BAA6B;AAC3C,SAAO;AAAA;AAAA;AAAA,IAGL,QAAQ,SAAS;AAAA,IACjB,eAAe;AAAA,MACb,gBAAgB;AAAA,IAClB;AAAA,EACF;AACF;;;ACXO,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,OAAOA,eAAc;AAId,SAAS,4BAA4B;AAC1C,SAAO;AAAA,IACL,QAAQA,UAAS;AAAA,EACnB;AACF;;;ACJO,SAAS,oBAAoB,iBAA0B;AAC5D,QAAM,sBAAsB,0BAA0B;AACtD,QAAM,uBAAuB,2BAA2B;AAExD,SAAO;AAAA,IACL,SAAS;AAAA,MACP,iBAAiB;AAAA,MACjB,MAAM;AAAA,MACN,SAAS,EAAE,QAAQ,gBAAgB;AAAA,MACnC,OAAO;AAAA,QACL,GAAG;AAAA;AAAA;AAAA;AAAA;AAAA;AAAA;AAAA,QAOH,gCAAgC;AAAA,MAClC;AAAA,IACF;AAAA,IACA,MAAM;AAAA,MACJ,iBAAiB;AAAA,MACjB,MAAM;AAAA,MACN,SAAS,EAAE,QAAQ,gBAAgB;AAAA,MACnC,OAAO;AAAA,IACT;AAAA,IACA,UAAU;AAAA,MACR,iBAAiB;AAAA,MACjB,MAAM;AAAA,MACN,SAAS,EAAE,QAAQ,gBAAgB;AAAA,MACnC,OAAO;AAAA,QACL,GAAG;AAAA;AAAA;AAAA;AAAA;AAAA,QAKH,gCAAgC;AAAA,QAChC,sCAAsC;AAAA,QACtC,yCAAyC;AAAA,QACzC,8BAA8B;AAAA,QAC9B,wBAAwB;AAAA,UACtB;AAAA,UACA;AAAA,YACE,UAAU;AAAA,YACV,aAAa;AAAA,UACf;AAAA,QACF;AAAA,MACF;AAAA,IACF;AAAA,IACA,SAAS;AAAA,MACP,iBAAiB;AAAA,MACjB,MAAM;AAAA,MACN,SAAS,EAAE,QAAQ,gBAAgB;AAAA,MACnC,OAAO;AAAA,QACL,qCAAqC;AAAA,MACvC;AAAA,IACF;AAAA,EACF;AACF;;;AC3DO,IAAM,eAAe;AAAA,EAC1B,eAAe;AAAA,IACb,gBAAgB;AAAA,EAClB;AAAA,EACA,MAAM;AACR;","names":["tseslint"]}
+{"version":3,"sources":["../src/shared/configs/create-typed-language-options.ts","../src/shared/configs/base-rules.ts","../src/shared/configs/create-base-language-options.ts","../src/shared/configs/create-shared-configs.ts","../src/shared/configs/strict-config.ts"],"sourcesContent":["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","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 { baseRules } from \"./base-rules\";\nimport { createBaseLanguageOptions } from \"./create-base-language-options\";\nimport { createTypedLanguageOptions } from \"./create-typed-language-options\";\n\nexport function createSharedConfigs(pluginReference: unknown) {\n  const baseLanguageOptions = createBaseLanguageOptions();\n  const typedLanguageOptions = createTypedLanguageOptions();\n\n  return {\n    backend: {\n      languageOptions: typedLanguageOptions,\n      name: \"skapxd/shared/backend\",\n      plugins: { skapxd: pluginReference },\n      rules: {\n        ...baseRules,\n        // La regla obligatoria del sistema de errores es await-requires-result\n        // (todo await resuelve en Result). async-functions-return-result queda\n        // apagada por defecto: exigir la firma por decreto choca con los bordes\n        // del framework y bloquea la adopción incremental; la presión sobre los\n        // awaits produce el mismo estado final. Ver \"¿Por qué está apagada por\n        // defecto?\" en el README.\n        \"skapxd/await-requires-result\": \"error\",\n      },\n    },\n    base: {\n      languageOptions: baseLanguageOptions,\n      name: \"skapxd/shared/base\",\n      plugins: { skapxd: pluginReference },\n      rules: baseRules,\n    },\n    frontend: {\n      languageOptions: typedLanguageOptions,\n      name: \"skapxd/shared/frontend\",\n      plugins: { skapxd: pluginReference },\n      rules: {\n        ...baseRules,\n        // En el front no se obliga a retornar Result, pero todo await debe\n        // resolver en uno: o la función llamada ya retorna Promise<Result>\n        // (camino preferido: errores modelados en el dominio) o se envuelve\n        // en trySafe en el sitio.\n        \"skapxd/await-requires-result\": \"error\",\n        \"skapxd/jsx-return-name-pascal-case\": \"error\",\n        // Anti prop-drilling: ninguna prop viaja más de un nivel. Quien la\n        // crea puede pasarla a UN hijo; quien la recibe no la reenvía —\n        // estado y acciones a un store global o custom hook.\n        \"skapxd/no-functions-inside-components\": \"error\",\n        \"skapxd/no-tunnel-props\": \"error\",\n        // Listeners en efectos: un AbortController por efecto, cleanup con\n        // un solo abort() en vez de removeEventListener por listener.\n        \"skapxd/prefer-abort-signal\": \"error\",\n        \"skapxd/no-jsx-ternary-null\": \"error\",\n        \"skapxd/max-hook-size\": [\n          \"error\",\n          {\n            maxLines: 120,\n            maxUseState: 1,\n          },\n        ],\n      },\n    },\n    package: {\n      languageOptions: baseLanguageOptions,\n      name: \"skapxd/shared/package\",\n      plugins: { skapxd: pluginReference },\n      rules: {\n        \"skapxd/one-root-function-per-file\": \"error\",\n      },\n    },\n  };\n}\n","// Config endurecida: ignora TODOS los comentarios `eslint-disable` (y demás\n// directivas inline) en los archivos que cubre. Así ni una persona ni un agente\n// pueden saltarse una regla con `// eslint-disable-next-line`.\nexport const strictConfig = {\n  linterOptions: {\n    noInlineConfig: true,\n  },\n  name: \"skapxd/strict\",\n};\n"],"mappings":";AAAA,OAAO,cAAc;AAEd,SAAS,6BAA6B;AAC3C,SAAO;AAAA;AAAA;AAAA,IAGL,QAAQ,SAAS;AAAA,IACjB,eAAe;AAAA,MACb,gBAAgB;AAAA,IAClB;AAAA,EACF;AACF;;;ACXO,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,OAAOA,eAAc;AAId,SAAS,4BAA4B;AAC1C,SAAO;AAAA,IACL,QAAQA,UAAS;AAAA,EACnB;AACF;;;ACJO,SAAS,oBAAoB,iBAA0B;AAC5D,QAAM,sBAAsB,0BAA0B;AACtD,QAAM,uBAAuB,2BAA2B;AAExD,SAAO;AAAA,IACL,SAAS;AAAA,MACP,iBAAiB;AAAA,MACjB,MAAM;AAAA,MACN,SAAS,EAAE,QAAQ,gBAAgB;AAAA,MACnC,OAAO;AAAA,QACL,GAAG;AAAA;AAAA;AAAA;AAAA;AAAA;AAAA;AAAA,QAOH,gCAAgC;AAAA,MAClC;AAAA,IACF;AAAA,IACA,MAAM;AAAA,MACJ,iBAAiB;AAAA,MACjB,MAAM;AAAA,MACN,SAAS,EAAE,QAAQ,gBAAgB;AAAA,MACnC,OAAO;AAAA,IACT;AAAA,IACA,UAAU;AAAA,MACR,iBAAiB;AAAA,MACjB,MAAM;AAAA,MACN,SAAS,EAAE,QAAQ,gBAAgB;AAAA,MACnC,OAAO;AAAA,QACL,GAAG;AAAA;AAAA;AAAA;AAAA;AAAA,QAKH,gCAAgC;AAAA,QAChC,sCAAsC;AAAA;AAAA;AAAA;AAAA,QAItC,yCAAyC;AAAA,QACzC,0BAA0B;AAAA;AAAA;AAAA,QAG1B,8BAA8B;AAAA,QAC9B,8BAA8B;AAAA,QAC9B,wBAAwB;AAAA,UACtB;AAAA,UACA;AAAA,YACE,UAAU;AAAA,YACV,aAAa;AAAA,UACf;AAAA,QACF;AAAA,MACF;AAAA,IACF;AAAA,IACA,SAAS;AAAA,MACP,iBAAiB;AAAA,MACjB,MAAM;AAAA,MACN,SAAS,EAAE,QAAQ,gBAAgB;AAAA,MACnC,OAAO;AAAA,QACL,qCAAqC;AAAA,MACvC;AAAA,IACF;AAAA,EACF;AACF;;;AClEO,IAAM,eAAe;AAAA,EAC1B,eAAe;AAAA,IACb,gBAAgB;AAAA,EAClB;AAAA,EACA,MAAM;AACR;","names":["tseslint"]}

package/dist/{chunk-47EZPLJX.mjs → chunk-WO7EUISC.mjs}

@@ -1448,6 +1448,144 @@ 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 {
+    allowFilePatterns: options.allowFilePatterns ?? [],
+    // Regex de nombres de prop que sí pueden reenviarse (p. ej.
+    // ["^className$", "^style$"] en wrappers de un design system).
+    allowPropPatterns: options.allowPropPatterns ?? []
+  };
+}
+
+// src/utils/get-object-pattern-prop-names.ts
+function getObjectPatternPropNames(pattern) {
+  const result = { propNames: [], restName: null };
+  if (pattern?.type !== "ObjectPattern") {
+    return result;
+  }
+  for (const property of pattern.properties) {
+    if (property.type === "RestElement" && property.argument.type === "Identifier") {
+      result.restName = property.argument.name;
+      continue;
+    }
+    if (property.type === "Property" && property.key.type === "Identifier" && property.value.type === "Identifier" && property.key.name === property.value.name) {
+      result.propNames.push(property.key.name);
+    }
+  }
+  return result;
+}
+
+// src/utils/is-pascal-case-jsx-element.ts
+function isPascalCaseJsxElement(openingElement) {
+  return openingElement?.type === "JSXOpeningElement" && openingElement.name?.type === "JSXIdentifier" && isPascalCaseName(openingElement.name.name);
+}
+
+// src/utils/is-forwarded-prop-reference.ts
+function isForwardedPropReference(identifier) {
+  const container = identifier.parent;
+  if (container?.type !== "JSXExpressionContainer") {
+    return false;
+  }
+  const attribute = container.parent;
+  if (attribute?.type !== "JSXAttribute") {
+    return false;
+  }
+  return isPascalCaseJsxElement(attribute.parent);
+}
+
+// src/rules/no-tunnel-props.ts
+var noTunnelProps = {
+  meta: {
+    type: "problem",
+    docs: {
+      description: "Ninguna prop viaja mas de un nivel: quien la recibe no puede reenviarla a otro componente."
+    },
+    messages: {
+      forwardedProp: "La prop `{{prop}}` que `{{component}}` recibe se reenvia a otro componente: ya va por su segundo salto (abuelo -> padre -> hijo). Quien CREA un valor puede pasarlo UN nivel; quien lo recibe no lo reenvia. Mueve el estado/accion a un store global (p. ej. zustand) o a un custom hook y consumelo donde se usa, o deja que el padre componga el JSX (`children`).",
+      spreadTunnel: "`{{component}}` reenvia TODAS sus props con `{...{{name}}}` a otro componente: es un tunel puro. Mueve el estado a un store global (p. ej. zustand) o a un custom hook, o deja que el padre componga el JSX (`children`)."
+    },
+    schema: [
+      {
+        additionalProperties: false,
+        properties: {
+          allowFilePatterns: {
+            items: { type: "string" },
+            type: "array"
+          },
+          allowPropPatterns: {
+            items: { type: "string" },
+            type: "array"
+          }
+        },
+        type: "object"
+      }
+    ]
+  },
+  create(context) {
+    const options = getNoTunnelPropsOptions(context.options[0]);
+    const filename = context.filename ?? context.getFilename();
+    if (matchesAnyGlob(filename, options.allowFilePatterns)) {
+      return {};
+    }
+    function reportSpreadTunnel(node, componentName, restName) {
+      const spreads = collectIdentifiersNamed(node.body, restName).filter(
+        (identifier) => identifier.parent?.type === "JSXSpreadAttribute" && isPascalCaseJsxElement(identifier.parent.parent)
+      );
+      if (spreads.length > 0) {
+        context.report({
+          data: { component: componentName, name: restName },
+          messageId: "spreadTunnel",
+          node: spreads[0].parent
+        });
+      }
+    }
+    function reportForwardedProps(node, componentName, propNames) {
+      for (const propName of propNames) {
+        if (matchesAnyPattern(propName, options.allowPropPatterns)) {
+          continue;
+        }
+        for (const usage of collectIdentifiersNamed(node.body, propName)) {
+          if (isForwardedPropReference(usage)) {
+            context.report({
+              data: { component: componentName, prop: propName },
+              messageId: "forwardedProp",
+              node: usage.parent.parent
+            });
+          }
+        }
+      }
+    }
+    function reportIfTunnelComponent(node) {
+      const componentName = getFunctionName(node);
+      if (!isPascalCaseName(componentName)) {
+        return;
+      }
+      const { propNames, restName } = getObjectPatternPropNames(node.params[0]);
+      if (restName) {
+        reportSpreadTunnel(node, componentName, restName);
+      }
+      reportForwardedProps(node, componentName, propNames);
+    }
+    return {
+      ArrowFunctionExpression: reportIfTunnelComponent,
+      FunctionDeclaration: reportIfTunnelComponent,
+      FunctionExpression: reportIfTunnelComponent
+    };
+  }
+};
+
 // src/utils/get-no-functions-inside-components-options.ts
 function getNoFunctionsInsideComponentsOptions(options = {}) {
   return {
@@ -1559,6 +1697,141 @@ var noTryCatch = {
   }
 };
 
+// src/utils/get-prefer-abort-signal-options.ts
+function getPreferAbortSignalOptions(options = {}) {
+  return {
+    allowFilePatterns: options.allowFilePatterns ?? [],
+    effectNames: options.effectNames ?? ["useEffect", "useLayoutEffect"]
+  };
+}
+
+// src/utils/get-variable-initializer.ts
+function getVariableInitializer(identifier, scope) {
+  let current = scope;
+  while (current) {
+    const variable = current.variables.find(
+      (candidate) => candidate.name === identifier.name
+    );
+    if (variable) {
+      const definition = variable.defs[0];
+      return definition?.node?.type === "VariableDeclarator" ? definition.node.init : null;
+    }
+    current = current.upper;
+  }
+  return null;
+}
+
+// src/utils/object-expression-has-signal.ts
+function objectExpressionHasSignal(objectExpression) {
+  return objectExpression.properties.some(
+    (property) => property.type === "SpreadElement" || isPropertyKeyNamed(property, "signal")
+  );
+}
+
+// src/utils/has-abort-signal-option.ts
+function hasAbortSignalOption(callExpression, sourceCode, typeContext) {
+  const options = callExpression.arguments[2];
+  if (!options) {
+    return false;
+  }
+  if (options.type === "Literal") {
+    return false;
+  }
+  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);
+    }
+  }
+  if (typeContext) {
+    const type = typeContext.services.getTypeAtLocation(options);
+    return Boolean(typeContext.checker.getPropertyOfType(type, "signal"));
+  }
+  return true;
+}
+
+// src/utils/is-inside-effect-callback.ts
+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;
+      }
+    }
+    current = current.parent;
+  }
+  return false;
+}
+
+// src/rules/prefer-abort-signal.ts
+var preferAbortSignal = {
+  meta: {
+    type: "suggestion",
+    docs: {
+      description: "En efectos de React, los listeners se limpian con AbortController, no con removeEventListener."
+    },
+    messages: {
+      addWithoutSignal: "Este addEventListener vive en un efecto sin `signal`. Crea `const controller = new AbortController()`, pasa `{ signal: controller.signal }` como tercer argumento y limpia con `return () => controller.abort()`: un solo abort cubre todos los listeners del efecto.",
+      removeInsteadOfAbort: "No quites listeners a mano en el cleanup: registra con `{ signal: controller.signal }` y reemplaza este removeEventListener por `return () => controller.abort()`. Evita el bug clasico de pasar una referencia distinta al remover."
+    },
+    schema: [
+      {
+        additionalProperties: false,
+        properties: {
+          allowFilePatterns: {
+            items: { type: "string" },
+            type: "array"
+          },
+          effectNames: {
+            items: { type: "string" },
+            type: "array"
+          }
+        },
+        type: "object"
+      }
+    ]
+  },
+  create(context) {
+    const options = getPreferAbortSignalOptions(context.options[0]);
+    const filename = context.filename ?? context.getFilename();
+    const sourceCode = context.sourceCode ?? context.getSourceCode();
+    const typeContext = getTypeContext(context);
+    if (matchesAnyGlob(filename, options.allowFilePatterns)) {
+      return {};
+    }
+    return {
+      CallExpression(node) {
+        if (node.callee?.type !== "MemberExpression") {
+          return;
+        }
+        const isAdd = isMemberPropertyNamed(node.callee, "addEventListener");
+        const isRemove = isMemberPropertyNamed(node.callee, "removeEventListener");
+        if (!isAdd && !isRemove) {
+          return;
+        }
+        if (!isInsideEffectCallback(node, options.effectNames)) {
+          return;
+        }
+        if (isRemove) {
+          context.report({ messageId: "removeInsteadOfAbort", node });
+          return;
+        }
+        if (!hasAbortSignalOption(node, sourceCode, typeContext)) {
+          context.report({ messageId: "addWithoutSignal", node });
+        }
+      }
+    };
+  }
+};
+
 // src/rules/prefer-ts-pattern.ts
 var preferTsPattern = {
   meta: {
@@ -1709,8 +1982,10 @@ var rules = {
   "no-deep-relative-imports": noDeepRelativeImports,
   "no-default-export": noDefaultExport,
   "no-emoji": noEmoji,
+  "no-tunnel-props": noTunnelProps,
   "no-functions-inside-components": noFunctionsInsideComponents,
   "no-try-catch": noTryCatch,
+  "prefer-abort-signal": preferAbortSignal,
   "prefer-ts-pattern": preferTsPattern,
   "no-jsx-ternary-null": noJsxTernaryNull,
   "no-promise-chain": noPromiseChain
@@ -1719,4 +1994,4 @@ var rules = {
 export {
   rules
 };
-//# sourceMappingURL=chunk-47EZPLJX.mjs.map
+//# sourceMappingURL=chunk-WO7EUISC.mjs.map