@timmo001/oxlint-rules 0.6.0 → 0.6.1

package/README.md

@@ -137,8 +137,9 @@ they only run when the consumer enables `typeAware`.
 
 - `no-raw-gh`: reports the raw `execute`, `json` and `stream` methods called on
   a `yield* Gh` binding from `@timmo001/effect-gh`, `Api.raw` from that package,
-  and any call whose first argument is `"gh"` or an array starting with `"gh"`,
-  such as `executor.run("gh", ...)` or `Bun.spawn(["gh", ...])`. Use the
+  and calls that launch gh: `"gh"` followed by its arguments, such as
+  `executor.run("gh", ...)`, or an argument array starting with `"gh"`, such
+  as `Bun.spawn(["gh", ...])`. Use the
   package's typed operations, and add one to effect-gh when a command has none.
   effect-gh's own relative imports are not reported.
 - `no-try-catch-in-effect-generators`: diagnoses synchronous `try/catch` owned

package/dist/cli.js

@@ -3004,12 +3004,13 @@ function isYieldGh(sourceCode, node) {
 function isGhBinding(sourceCode, identifier) {
   return resolveVariable(sourceCode, identifier)?.defs.some((definition) => definition.type === "Variable" && definition.node.type === "VariableDeclarator" && isYieldGh(sourceCode, definition.node.init)) ?? false;
 }
-function isGhArgument(node) {
-  if (node?.type === "Literal")
-    return node.value === "gh";
-  if (node?.type !== "ArrayExpression")
+function launchesGh(args) {
+  const [command, argv] = args;
+  if (command?.type === "Literal")
+    return command.value === "gh" && argv !== undefined;
+  if (command?.type !== "ArrayExpression")
     return false;
-  const first = node.elements[0];
+  const first = command.elements[0];
   return first?.type === "Literal" && first.value === "gh";
 }
 var noRawGhRule = defineRule24({
@@ -3040,7 +3041,7 @@ var noRawGhRule = defineRule24({
             return;
           }
         }
-        if (isGhArgument(node.arguments[0]))
+        if (launchesGh(node.arguments))
           context.report({ node, messageId: "spawnGh" });
       }
     };

package/dist/configs/recommended-effect.js

@@ -55,12 +55,13 @@ function isYieldGh(sourceCode, node) {
 function isGhBinding(sourceCode, identifier) {
   return resolveVariable(sourceCode, identifier)?.defs.some((definition) => definition.type === "Variable" && definition.node.type === "VariableDeclarator" && isYieldGh(sourceCode, definition.node.init)) ?? false;
 }
-function isGhArgument(node) {
-  if (node?.type === "Literal")
-    return node.value === "gh";
-  if (node?.type !== "ArrayExpression")
+function launchesGh(args) {
+  const [command, argv] = args;
+  if (command?.type === "Literal")
+    return command.value === "gh" && argv !== undefined;
+  if (command?.type !== "ArrayExpression")
     return false;
-  const first = node.elements[0];
+  const first = command.elements[0];
   return first?.type === "Literal" && first.value === "gh";
 }
 var noRawGhRule = defineRule({
@@ -91,7 +92,7 @@ var noRawGhRule = defineRule({
             return;
           }
         }
-        if (isGhArgument(node.arguments[0]))
+        if (launchesGh(node.arguments))
           context.report({ node, messageId: "spawnGh" });
       }
     };

package/dist/effect/index.js

@@ -50,12 +50,13 @@ function isYieldGh(sourceCode, node) {
 function isGhBinding(sourceCode, identifier) {
   return resolveVariable(sourceCode, identifier)?.defs.some((definition) => definition.type === "Variable" && definition.node.type === "VariableDeclarator" && isYieldGh(sourceCode, definition.node.init)) ?? false;
 }
-function isGhArgument(node) {
-  if (node?.type === "Literal")
-    return node.value === "gh";
-  if (node?.type !== "ArrayExpression")
+function launchesGh(args) {
+  const [command, argv] = args;
+  if (command?.type === "Literal")
+    return command.value === "gh" && argv !== undefined;
+  if (command?.type !== "ArrayExpression")
     return false;
-  const first = node.elements[0];
+  const first = command.elements[0];
   return first?.type === "Literal" && first.value === "gh";
 }
 var noRawGhRule = defineRule({
@@ -86,7 +87,7 @@ var noRawGhRule = defineRule({
             return;
           }
         }
-        if (isGhArgument(node.arguments[0]))
+        if (launchesGh(node.arguments))
           context.report({ node, messageId: "spawnGh" });
       }
     };

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@timmo001/oxlint-rules",
-  "version": "0.6.0",
+  "version": "0.6.1",
   "description": "Shared Oxlint plugins and configs with optional Effect rules.",
   "type": "module",
   "license": "Apache-2.0",

package/src/effect/rules/no-raw-gh.ts

@@ -61,11 +61,18 @@ function isGhBinding(
   );
 }
 
-function isGhArgument(node: ESTree.Node | undefined): boolean {
-  if (node?.type === "Literal") return node.value === "gh";
+/**
+ * Whether call arguments launch gh: `"gh"` followed by its argv, as in
+ * `run("gh", args)`, or an argv array starting with `"gh"`.
+ */
+function launchesGh(args: ESTree.CallExpression["arguments"]): boolean {
+  const [command, argv] = args;
 
-  if (node?.type !== "ArrayExpression") return false;
-  const first = node.elements[0];
+  if (command?.type === "Literal")
+    return command.value === "gh" && argv !== undefined;
+
+  if (command?.type !== "ArrayExpression") return false;
+  const first = command.elements[0];
 
   return first?.type === "Literal" && first.value === "gh";
 }
@@ -118,7 +125,7 @@ export const noRawGhRule = defineRule({
           }
         }
 
-        if (isGhArgument(node.arguments[0]))
+        if (launchesGh(node.arguments))
           context.report({ node, messageId: "spawnGh" });
       },
     };