@timmo001/oxlint-rules 0.6.1 → 0.6.2

package/README.md

@@ -138,8 +138,8 @@ 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 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
+  `executor.run("gh", ...)`, or an argument array starting with `"gh"` passed
+  to a spawn, exec, run or command function, 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,14 +3004,20 @@ 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 launchesGh(args) {
-  const [command, argv] = args;
+var launcherName = /spawn|exec|run|command/i;
+function calleeName(callee) {
+  if (callee.type === "Identifier")
+    return callee.name;
+  return staticMemberName(callee);
+}
+function launchesGh(node) {
+  const [command, argv] = node.arguments;
   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";
+  return first?.type === "Literal" && first.value === "gh" && launcherName.test(calleeName(node.callee) ?? "");
 }
 var noRawGhRule = defineRule24({
   meta: {
@@ -3041,7 +3047,7 @@ var noRawGhRule = defineRule24({
             return;
           }
         }
-        if (launchesGh(node.arguments))
+        if (launchesGh(node))
           context.report({ node, messageId: "spawnGh" });
       }
     };

package/dist/configs/recommended-effect.js

@@ -55,14 +55,20 @@ 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 launchesGh(args) {
-  const [command, argv] = args;
+var launcherName = /spawn|exec|run|command/i;
+function calleeName(callee) {
+  if (callee.type === "Identifier")
+    return callee.name;
+  return staticMemberName(callee);
+}
+function launchesGh(node) {
+  const [command, argv] = node.arguments;
   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";
+  return first?.type === "Literal" && first.value === "gh" && launcherName.test(calleeName(node.callee) ?? "");
 }
 var noRawGhRule = defineRule({
   meta: {
@@ -92,7 +98,7 @@ var noRawGhRule = defineRule({
             return;
           }
         }
-        if (launchesGh(node.arguments))
+        if (launchesGh(node))
           context.report({ node, messageId: "spawnGh" });
       }
     };

package/dist/effect/index.js

@@ -50,14 +50,20 @@ 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 launchesGh(args) {
-  const [command, argv] = args;
+var launcherName = /spawn|exec|run|command/i;
+function calleeName(callee) {
+  if (callee.type === "Identifier")
+    return callee.name;
+  return staticMemberName(callee);
+}
+function launchesGh(node) {
+  const [command, argv] = node.arguments;
   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";
+  return first?.type === "Literal" && first.value === "gh" && launcherName.test(calleeName(node.callee) ?? "");
 }
 var noRawGhRule = defineRule({
   meta: {
@@ -87,7 +93,7 @@ var noRawGhRule = defineRule({
             return;
           }
         }
-        if (launchesGh(node.arguments))
+        if (launchesGh(node))
           context.report({ node, messageId: "spawnGh" });
       }
     };

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@timmo001/oxlint-rules",
-  "version": "0.6.1",
+  "version": "0.6.2",
   "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,12 +61,23 @@ function isGhBinding(
   );
 }
 
+/** Callee names that take an argv array to launch, such as `Bun.spawn` or `execFileSync`. */
+const launcherName = /spawn|exec|run|command/i;
+
+function calleeName(callee: ESTree.Expression): string | null {
+  if (callee.type === "Identifier") return callee.name;
+
+  return staticMemberName(callee);
+}
+
 /**
- * Whether call arguments launch gh: `"gh"` followed by its argv, as in
- * `run("gh", args)`, or an argv array starting with `"gh"`.
+ * Whether a call launches gh: `"gh"` followed by its argv, as in
+ * `run("gh", args)`, or an argv array starting with `"gh"` passed to a
+ * spawn, exec, run or command function. Recording argv, as in
+ * `calls.push(["gh", ...args])`, is not launching it.
  */
-function launchesGh(args: ESTree.CallExpression["arguments"]): boolean {
-  const [command, argv] = args;
+function launchesGh(node: ESTree.CallExpression): boolean {
+  const [command, argv] = node.arguments;
 
   if (command?.type === "Literal")
     return command.value === "gh" && argv !== undefined;
@@ -74,7 +85,11 @@ function launchesGh(args: ESTree.CallExpression["arguments"]): boolean {
   if (command?.type !== "ArrayExpression") return false;
   const first = command.elements[0];
 
-  return first?.type === "Literal" && first.value === "gh";
+  return (
+    first?.type === "Literal" &&
+    first.value === "gh" &&
+    launcherName.test(calleeName(node.callee) ?? "")
+  );
 }
 
 /** Route gh through effect-gh's typed operations instead of raw argv or direct spawns. */
@@ -125,8 +140,7 @@ export const noRawGhRule = defineRule({
           }
         }
 
-        if (launchesGh(node.arguments))
-          context.report({ node, messageId: "spawnGh" });
+        if (launchesGh(node)) context.report({ node, messageId: "spawnGh" });
       },
     };
   },