fix: make things work

* Updates jest and eslint config to adhere to local package aliases.
* Formally adopts NPM workspaces.
* Deletes node_modules in the `clean` Makefile targets

Author: adamstankiewicz

common/extendESLintConfig.js

@@ -0,0 +1,21 @@
+const path = require('path');
+
+const extendESLintConfig = (config) => {
+  const importNoUnresolved = config.rules['import/no-unresolved'];
+  if (importNoUnresolved ) {
+    const originalIgnore = importNoUnresolved[1].ignore;
+    if (!originalIgnore.includes('@edx/frontend-enterprise-*')) {
+      importNoUnresolved[1].ignore = [...originalIgnore, '@edx/frontend-enterprise-*'];
+    }
+  }
+  config.settings = {
+    ...config.settings,
+    'import/resolver': {
+      alias: [
+        ['@edx/frontend-enterprise-utils', '../utils/src'],
+      ],
+    },
+  };
+};
+
+module.exports =  extendESLintConfig;

common/extendJestConfig.js

@@ -3,7 +3,7 @@ const extendJestConfig = (config) => {
   config.modulePathIgnorePatterns = config.modulePathIgnorePatterns ? [...config.modulePathIgnorePatterns, ...modulePathsToIgnore] : modulePathsToIgnore;
   config.moduleNameMapper = {
     ...config.moduleNameMapper,
-    "@edx/frontend-enterprise-(.*)": "<rootDir>/../$1/src",
+    '@edx/frontend-enterprise-(.*)': '<rootDir>/../$1/src',
   };
 };
 

package-lock.json

@@ -12,14 +12,14 @@
   ],
   "scripts": {
     "setup": "npm install && npm run bootstrap",
-    "bootstrap": "lerna exec -- npm install",
-    "dev": "lerna run dev --parallel",
-    "clean": "rm -rf ./node_modules && lerna run clean && lerna clean --yes",
-    "build": "lerna run build --parallel",
-    "test": "lerna run test --parallel",
-    "lint": "lerna run lint --parallel",
-    "lint:fix": "lerna run lint:fix --include-dependencies --include-dependents --since origin/master --parallel",
-    "changed": "lerna changed"
+    "bootstrap": "npx lerna exec -- npm install",
+    "dev": "npx lerna run dev --parallel",
+    "clean": "rm -rf ./node_modules && npx lerna run clean",
+    "build": "npx lerna run build --parallel",
+    "test": "npx lerna run test --parallel",
+    "lint": "npx lerna run lint --parallel",
+    "lint:fix": "npx lerna run lint:fix --include-dependencies --include-dependents --since origin/master --parallel",
+    "changed": "npx lerna changed"
   },
   "devDependencies": {
     "@commitlint/config-conventional": "13.2.0",

package.json

@@ -0,0 +1,7 @@
+const { createConfig } = require('@edx/frontend-build');
+const extendESLintConfig = require('../../common/extendESLintConfig');
+
+const config = createConfig('eslint');
+extendESLintConfig(config);
+
+module.exports = config;

packages/catalog-search/.eslintrc.js

@@ -1,6 +1,6 @@
 build:
 	make clean
-	./node_modules/.bin/fedx-scripts babel src --out-dir dist --source-maps --ignore **/*.test.jsx,**/*.test.js,**/setupTest.js --copy-files
+	../../node_modules/.bin/fedx-scripts babel src --out-dir dist --source-maps --ignore **/*.test.jsx,**/*.test.js,**/setupTest.js --copy-files
 	@# --copy-files will bring in everything else that wasn't processed by babel. Remove what we don't want.
 	@find dist -name '*.test.js*' -delete
 	cp ./package.json ./dist/package.json
@@ -9,4 +9,4 @@ build:
 	cp ./CHANGELOG.md ./dist/CHANGELOG.md
 
 clean:
-	rm -rf ./dist
+	rm -rf ./dist ./node_modules