@onrails/eslint-plugin 0.1.1 → 0.1.2

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@onrails/eslint-plugin",
-  "version": "0.1.1",
+  "version": "0.1.2",
   "description": "ESLint rules for @onrails/result boundaries — flags Promise<Result<…>> in public APIs and _unsafeUnwrap*. Replaces unmaintained eslint-plugin-neverthrow.",
   "license": "MIT",
   "author": "Alan R. Soares",

package/src/index.js

@@ -94,11 +94,13 @@ const noUnsafeUnwrap = {
   meta: {
     type: "suggestion",
     docs: {
-      description: "Discourage _unsafeUnwrap / _unsafeUnwrapErr outside tests.",
+      description:
+        "Discourage _unsafeUnwrap / _unsafeUnwrapErr and unwrapOk / unwrapErr / unwrap outside tests. Matching is name-based; same-named functions from other libraries are flagged too.",
     },
     schema: [],
     messages: {
-      unsafe: "Avoid _unsafeUnwrap* — use match(), resolve(), or yieldResult() in tryGen.",
+      unsafe:
+        "Avoid _unsafeUnwrap* and unwrap* outside tests — use match(), resolve(), or yieldResult() in tryGen.",
     },
   },
   create(context) {
@@ -115,6 +117,16 @@ const noUnsafeUnwrap = {
           context.report({ node, messageId: "unsafe" });
         }
       },
+      CallExpression(node) {
+        if (
+          node.callee.type === "Identifier" &&
+          (node.callee.name === "unwrapOk" ||
+            node.callee.name === "unwrapErr" ||
+            node.callee.name === "unwrap")
+        ) {
+          context.report({ node, messageId: "unsafe" });
+        }
+      },
     };
   },
 };