eslint-plugin-turbo 0.0.3 → 0.0.4

package/dist/index.js

@@ -20,7 +20,10 @@ var __copyProps = (to, from, except, desc) => {
   }
   return to;
 };
-var __toESM = (mod, isNodeMode, target) => (target = mod != null ? __create(__getProtoOf(mod)) : {}, __copyProps(isNodeMode || !mod || !mod.__esModule ? __defProp(target, "default", { value: mod, enumerable: true }) : target, mod));
+var __toESM = (mod, isNodeMode, target) => (target = mod != null ? __create(__getProtoOf(mod)) : {}, __copyProps(
+  isNodeMode || !mod || !mod.__esModule ? __defProp(target, "default", { value: mod, enumerable: true }) : target,
+  mod
+));
 var __toCommonJS = (mod) => __copyProps(__defProp({}, "__esModule", { value: true }), mod);
 
 // ../turbo-utils/dist/index.js
@@ -3705,7 +3708,7 @@ function findDependsOnEnvVars({
   dependencies
 }) {
   if (dependencies) {
-    return dependencies.filter((dep) => dep.startsWith("$"));
+    return dependencies.filter((dep) => dep.startsWith("$")).map((envVar) => envVar.slice(1, envVar.length));
   }
   return [];
 }
@@ -3717,17 +3720,26 @@ function getEnvVarDependencies({
   if (!turboJsonContent) {
     return null;
   }
-  const { globalDependencies, pipeline = {} } = turboJsonContent;
-  const allEnvVars = findDependsOnEnvVars({
-    dependencies: globalDependencies
-  });
-  Object.values(pipeline).forEach(({ dependsOn }) => {
+  const {
+    globalDependencies,
+    globalEnv = [],
+    pipeline = {}
+  } = turboJsonContent;
+  const allEnvVars = [
+    ...findDependsOnEnvVars({
+      dependencies: globalDependencies
+    }),
+    ...globalEnv
+  ];
+  Object.values(pipeline).forEach(({ env, dependsOn }) => {
     if (dependsOn) {
       allEnvVars.push(...findDependsOnEnvVars({ dependencies: dependsOn }));
     }
+    if (env) {
+      allEnvVars.push(...env);
+    }
   });
-  const envVarSet = new Set(allEnvVars.map((envVar) => envVar.slice(1, envVar.length)));
-  return envVarSet;
+  return new Set(allEnvVars);
 }
 var getEnvVarDependencies_default = getEnvVarDependencies;
 
@@ -3738,7 +3750,7 @@ var meta = {
     description: "Do not allow the use of `process.env` without including the env key in turbo.json",
     category: "Configuration Issues",
     recommended: true,
-    url: `https://github.com/vercel/turborepo/tree/main/packages/eslint-plugin-turbo/docs/rules/${RULES.noUndeclaredEnvVars}`
+    url: `https://github.com/vercel/turborepo/tree/main/packages/eslint-plugin-turbo/docs/rules/${RULES.noUndeclaredEnvVars}.md`
   },
   schema: [
     {
@@ -3790,10 +3802,16 @@ function create(context) {
       });
     }
   };
+  const isComputed = (node) => {
+    if ("computed" in node.parent) {
+      return node.parent.computed;
+    }
+    return false;
+  };
   return {
     MemberExpression(node) {
       var _a2, _b2;
-      if ("name" in node.object && "name" in node.property) {
+      if ("name" in node.object && "name" in node.property && !isComputed(node)) {
         const objectName = node.object.name;
         const propertyName = node.property.name;
         if (objectName === "process" && propertyName === "env") {

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "eslint-plugin-turbo",
-  "version": "0.0.3",
+  "version": "0.0.4",
   "description": "ESLint plugin for Turborepo",
   "keywords": [
     "turbo",
@@ -15,13 +15,16 @@
     "dist/**"
   ],
   "devDependencies": {
+    "@types/eslint": "^8.4.5",
     "@types/estree": "^1.0.0",
     "@types/jest": "^27.4.0",
     "@types/node": "^16.11.12",
     "jest": "^27.4.3",
     "ts-jest": "^27.1.1",
+    "tsconfig": "0.0.0",
     "tsup": "^6.2.0",
     "turbo-utils": "0.0.0",
+    "turbo-types": "0.0.0",
     "typescript": "^4.7.4"
   },
   "peerDependencies": {
@@ -29,7 +32,9 @@
   },
   "license": "MPL-2.0",
   "scripts": {
+    "release": "pnpm build && pnpm publish",
     "test": "jest",
-    "build": "tsup"
+    "build": "tsup",
+    "check-types": "tsc --noEmit"
   }
 }