@skapxd/eslint-opinionated 0.2.0 → 0.4.0

package/dist/index.js

@@ -38,8 +38,8 @@ module.exports = __toCommonJS(index_exports);
 
 // src/shared/configs/base-rules.ts
 var baseRules = {
-  "skapxd/no-ad-hoc-ok-result": "warn",
-  "skapxd/no-deep-relative-imports": "warn",
+  "skapxd/no-ad-hoc-ok-result": "error",
+  "skapxd/no-deep-relative-imports": "error",
   "skapxd/no-promise-chain": "error",
   "skapxd/no-try-catch": "error",
   "skapxd/one-root-function-per-file": "error",
@@ -100,15 +100,16 @@ function createSharedConfigs(pluginReference) {
       plugins: { skapxd: pluginReference },
       rules: {
         ...baseRules,
-        // En el front no se obliga a retornar Result, pero toda llamada
-        // asíncrona debe ir en trySafe — salvo que lo awaiteado ya retorne
-        // Result/Promise<Result> (exención type-aware de la regla).
-        "skapxd/await-requires-try-safe": "error",
+        // En el front no se obliga a retornar Result, pero todo await debe
+        // resolver en uno: o la función llamada ya retorna Promise<Result>
+        // (camino preferido: errores modelados en el dominio) o se envuelve
+        // en trySafe en el sitio.
+        "skapxd/await-requires-result": "error",
         "skapxd/jsx-return-name-pascal-case": "error",
         "skapxd/no-functions-inside-components": "error",
         "skapxd/no-jsx-ternary-null": "error",
         "skapxd/max-hook-size": [
-          "warn",
+          "error",
           {
             maxLines: 120,
             maxUseState: 1
@@ -212,7 +213,7 @@ function createNextConfigs(pluginReference) {
         "skapxd/no-functions-inside-components": "error",
         "skapxd/no-jsx-ternary-null": "error",
         "skapxd/max-hook-size": [
-          "warn",
+          "error",
           {
             maxLines: 120,
             maxUseState: 1
@@ -1032,8 +1033,8 @@ var noAdHocOkResult = {
   }
 };
 
-// src/utils/get-await-requires-try-safe-options.ts
-function getAwaitRequiresTrySafeOptions(options = {}) {
+// src/utils/get-await-requires-result-options.ts
+function getAwaitRequiresResultOptions(options = {}) {
   return {
     allowFilePatterns: options.allowFilePatterns ?? [],
     trySafeCallNames: options.trySafeCallNames ?? ["trySafe"]
@@ -1120,15 +1121,15 @@ function isTrySafeCall(node, trySafeCallNames) {
   return node?.type === "CallExpression" && isCalleeNamed(node.callee, trySafeCallNames);
 }
 
-// src/rules/await-requires-try-safe.ts
-var awaitRequiresTrySafe = {
+// src/rules/await-requires-result.ts
+var awaitRequiresResult = {
   meta: {
     type: "problem",
     docs: {
-      description: "Exige que los await esten protegidos por trySafe."
+      description: "Exige que todo await resuelva en un Result: una funcion que retorne Promise<Result<...>> o trySafe en el sitio."
     },
     messages: {
-      unprotectedAwait: "El await dentro de `{{name}}` no esta protegido por trySafe. Envuelve la operacion asi: `{{suggestion}}`."
+      awaitWithoutResult: "El await dentro de `{{name}}` no resuelve en un Result. Mejor opcion: extrae la operacion a una funcion que retorne Promise<Result<...>> y modela ahi los errores de dominio (el trySafe vive dentro de esa funcion). Alternativa: envuelvela aqui mismo: `{{suggestion}}`."
     },
     schema: [
       {
@@ -1148,7 +1149,7 @@ var awaitRequiresTrySafe = {
     ]
   },
   create(context) {
-    const options = getAwaitRequiresTrySafeOptions(context.options[0]);
+    const options = getAwaitRequiresResultOptions(context.options[0]);
     const filename = context.filename ?? context.getFilename();
     const sourceCode = context.sourceCode ?? context.getSourceCode();
     const typeContext = getTypeContext(context);
@@ -1183,7 +1184,7 @@ var awaitRequiresTrySafe = {
             name: getAwaitScopeName(node),
             suggestion: getTrySafeAwaitSuggestion(node.argument, sourceCode)
           },
-          messageId: "unprotectedAwait",
+          messageId: "awaitWithoutResult",
           node
         });
       }
@@ -1758,7 +1759,16 @@ var rules = {
   "jsx-return-name-pascal-case": jsxReturnNamePascalCase,
   "async-functions-return-result": asyncFunctionsReturnResult,
   "no-ad-hoc-ok-result": noAdHocOkResult,
-  "await-requires-try-safe": awaitRequiresTrySafe,
+  "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,
   "max-hook-size": maxHookSize,
   "no-deep-relative-imports": noDeepRelativeImports,
@@ -1777,7 +1787,7 @@ var plugin = {
   configs: {},
   meta: {
     name: "@skapxd/eslint-opinionated",
-    version: "0.2.0"
+    version: "0.4.0"
   },
   rules
 };