Use webpack-env typings instead of node typings in rush-sdk

Author: iclanton

common/config/rush/pnpm-lock.yaml

@@ -27,7 +27,7 @@
     "@rushstack/heft": "workspace:*",
     "@rushstack/heft-node-rig": "workspace:*",
     "@types/heft-jest": "1.0.1",
-    "@types/node": "12.20.24",
-    "@types/semver": "7.3.5"
+    "@types/semver": "7.3.5",
+    "@types/webpack-env": "1.13.0"
   }
 }

libraries/rush-sdk/package.json

@@ -31,10 +31,10 @@ declare const global: NodeJS.Global &
     ___rush___rushLibModuleFromInstallAndRunRush?: RushLibModuleType;
   };
 
-// eslint-disable-next-line @typescript-eslint/naming-convention,@typescript-eslint/no-explicit-any
-declare const __non_webpack_require__: ((moduleName: string) => any) | undefined;
 function _require<TResult>(moduleName: string): TResult {
   if (typeof __non_webpack_require__ === 'function') {
+    // If this library has been webpacked, we need to call the real `require` function
+    // that doesn't get turned into a __webpack_require__ statement
     return __non_webpack_require__(moduleName);
   } else {
     return require(moduleName);

libraries/rush-sdk/src/index.ts

@@ -1,6 +1,9 @@
 {
   "extends": "./node_modules/@rushstack/heft-node-rig/profiles/default/tsconfig-base.json",
   "compilerOptions": {
-    "types": ["heft-jest", "node"]
+    "types": [
+      "heft-jest",
+      "webpack-env" // Use webpack-env here instead of node so we have __non_webpack_require__
+    ]
   }
 }