Winter 2022 build update

.github/workflows/test.yml

@@ -48,22 +48,21 @@ jobs:
         # branch protection rules
         if: github.head_ref != 'changeset-release/main'
         run: yarn release:status
-
-      - name: Buld packages
-        shell: bash
-        run: yarn build       
+     
+      # Fails if project references drift from package.json, fix with reference script
+      - name: Check project reference
+        run: yarn reference:check
 
       - name: Test packages
-        run: yarn test
+        run: yarn workspaces foreach --topological --parallel run test
 
       - name: Lint packages
-        run: yarn lint
+        run: yarn workspaces foreach --topological --parallel run lint
 
       - name: Post coverage
-        uses: vebr/jest-lcov-reporter@v0.2.1
+        uses: ScaCap/code-coverage-assistant@v1 
         # only post on PRs
         if: github.event_name == 'pull_request'
         with:
           github-token: ${{ secrets.GITHUB_TOKEN }}
-          update-comment: true
-          lcov-file: ./coverage/lcov.info
+          monorepo-base-path: "./packages" 

.gitignore

@@ -24,7 +24,7 @@ coverage
 *.tsbuildinfo
 
 # jest
-.cache/jest
+**/.cache/jest
 
 #parcel
 .parcel-cache

conf/README.md

@@ -2,6 +2,14 @@
 
 Configuration files shared by the packages in the monorepo.
 
+## TypeScript
+
+`tsconfig.base.json` is the base TypeScript shared by all tools and packages.
+
 ## eslint
 
 We need to keep `.eslintrc` files out of root or LSPs will create an instance to lint the entire repository. Quickly bogs down.
+
+## jest
+
+`jest.base.js` is imported by all packages.

conf/eslintrc.base.js

@@ -53,5 +53,15 @@ module.exports = {
     ],
     "@typescript-eslint/ban-ts-comment": "off",
   },
-  settings: {},
+  settings: {
+    // import settings along with eslint-import-resolver-typescript allow js imports to be mapped back to ts files
+    "import/parsers": {
+      "@typescript-eslint/parser": [".ts", ".tsx"]
+    },
+    "import/resolver": {
+      "typescript": {
+	"project": ["tsconfig.json"]
+      },
+    }
+  },
 };

conf/jest/jest.base.js

@@ -0,0 +1,37 @@
+const path = require("path");
+
+const tsconfig = path.join(__dirname, "tsconfig.test.json");
+const setupPostMessage = path.join(__dirname, "setupPostMessage.js");
+
+function generate(config) {
+  return Object.assign(
+    {
+      cache: true,
+      cacheDirectory: ".cache/jest",
+      /*
+       * Automatically clear mock calls and instances before every test.
+       * Equivalent to calling jest.clearAllMocks() before each test.
+       * This does not remove any mock implementation that may have been provided.
+       */
+      clearMocks: true,
+      collectCoverage: true,
+      collectCoverageFrom: [ "**/*.{ts,tsx}" ],
+      coverageDirectory: "coverage/",
+      extensionsToTreatAsEsm: [".ts", ".tsx", ".mts"],
+      moduleNameMapper: {
+	'^(\\.{1,2}/.*)\\.js$': '$1',
+	'^uuid$': require.resolve('uuid'),
+      },
+      transform: {
+        "^.+\\.tsx?$": [
+          "ts-jest",
+          { useESM: true, tsconfig, diagnostics: false },
+        ],
+      },
+      setupFiles: [setupPostMessage],
+    },
+    config,
+  );
+}
+
+module.exports = generate

tsconfig.test.json → conf/jest/tsconfig.test.json

@@ -1,5 +1,5 @@
 {
-  "extends": "./tsconfig.base.json",
+  "extends": "../tsconfig.base.json",
   "compilerOptions": {
     "jsx": "react-jsx",
     "noEmit": true

package.json

@@ -17,13 +17,8 @@
     "packages/*"
   ],
   "scripts": {
-    "clean": "yarn workspaces foreach -p run clean",
-    "build": "yarn clean && yarn workspaces foreach -ipt run build",
-    "lint": "eslint packages/*",
-    "lint:fix": "eslint --fix packages/*",
-    "test": "jest --colors",
-    "test:config": "jest --showConfig",
-    "test:watch": "jest --watch --colors",
+    "references": "yarn workspaces-to-typescript-project-references --includesRoot",
+    "references:check": "yarn workspaces-to-typescript-project-references --includesRoot --check",
     "release:add:empty": "yarn changeset add --empty",
     "release:add": "yarn changeset add",
     "release:status": "yarn changeset status --since origin/main --output",
@@ -32,27 +27,10 @@
     "release": "yarn clean && yarn build && yarn release:publish"
   },
   "devDependencies": {
-    "@babel/preset-env": "^7.19.3",
     "@changesets/changelog-github": "^0.4.7",
-    "@types/babel__preset-env": "^7",
-    "@types/eslint": "^8",
-    "@types/jest": "^29.1.1",
-    "@typescript-eslint/eslint-plugin": "^5.39.0",
-    "@typescript-eslint/parser": "^5.39.0",
-    "babel-jest": "^29.1.2",
-    "eslint": "^8.24.0",
-    "eslint-config-prettier": "^8.5.0",
-    "eslint-plugin-import": "^2.26.0",
-    "eslint-plugin-jest": "^27.1.1",
-    "eslint-plugin-prettier": "^4.2.1",
-    "jest": "^29.1.2",
-    "jest-environment-jsdom": "^29.1.2",
-    "prettier": "^2.7.1",
-    "ts-jest": "^29.0.3",
-    "tslib": "^2.4.0",
-    "typescript": "^4.8.4"
-  },
-  "dependencies": {
-    "@changesets/cli": "^2.25.0"
+    "@changesets/cli": "^2.25.0",
+    "@monorepo-utils/workspaces-to-typescript-project-references": "^2.8.2",
+    "@types/jest": "^29.2.2",
+    "eslint-import-resolver-typescript": "^3.5.2"
   }
 }

packages/client-sdk/.eslintrc.js

@@ -2,7 +2,6 @@ module.exports = {
   extends: ["../../conf/eslintrc.base.js"],
   root: true,
   parserOptions: {
-    project: "tsconfig.lint.json",
     tsconfigRootDir: __dirname,
   },
 };

packages/client-sdk/jest.config.js

@@ -0,0 +1 @@
+module.exports = require('../../conf/jest/jest.base.js')({ testEnvironment: "jsdom" });

packages/client-sdk/package.json

@@ -13,7 +13,10 @@
   "types": "dist/index.d.js",
   "scripts": {
     "clean": "rm -f tsconfig.tsbuildinfo && rm -rf dist",
-    "build": "tsc --build"
+    "build": "tsc --build",
+    "lint": "tsc --build && eslint",
+    "lint:fix": "tsc --build && eslint --fix",
+    "test": "tsc --build && jest"
   },
   "dependencies": {
     "@getmash/jsonrpc-engine": "^1.0.2",
@@ -31,5 +34,27 @@
   },
   "bugs": {
     "url": "https://github.com/getmash/mash-js/issues"
+  },
+  "devDependencies": {
+    "@babel/preset-env": "^7.20.2",
+    "@types/babel__preset-env": "^7",
+    "@types/eslint": "^8",
+    "@types/jest": "^29.2.2",
+    "@types/prettier": "^2",
+    "@typescript-eslint/eslint-plugin": "^5.42.0",
+    "@typescript-eslint/parser": "^5.42.0",
+    "babel-jest": "^29.2.2",
+    "eslint": "^8.26.0",
+    "eslint-config-prettier": "^8.5.0",
+    "eslint-import-resolver-typescript": "^3.5.2",
+    "eslint-plugin-import": "^2.26.0",
+    "eslint-plugin-jest": "^27.1.4",
+    "eslint-plugin-prettier": "^4.2.1",
+    "jest": "^29.2.2",
+    "jest-environment-jsdom": "^29.2.2",
+    "prettier": "^2.7.1",
+    "ts-jest": "^29.0.3",
+    "tslib": "^2.4.1",
+    "typescript": "^4.8.4"
   }
 }

packages/client-sdk/tsconfig.json

@@ -1,10 +1,9 @@
 {
-  "extends": "../../tsconfig.base.json",
+  "extends": "../../conf/tsconfig.base.json",
   "compilerOptions": {
     "rootDir": "./src",
-    "outDir": "./dist",
+    "outDir": "./dist"
   },
-  "include": ["./src/**/*.ts"],
-  "exclude": ["./src/**/*.test.*"],
+  "include": ["./src/**/*"],
   "references": [{ "path": "../jsonrpc-engine" }, { "path": "../post-message" }]
 }

packages/jsonrpc-engine/.eslintrc.js

@@ -2,7 +2,6 @@ module.exports = {
   extends: ["../../conf/eslintrc.base.js"],
   root: true,
   parserOptions: {
-    project: "tsconfig.lint.json",
     tsconfigRootDir: __dirname,
   },
 };

packages/jsonrpc-engine/package.json

@@ -1,23 +1,47 @@
 {
   "name": "@getmash/jsonrpc-engine",
   "version": "1.0.2",
-  "description": "JsonRPC wrapper over  window.postMessage to allow a structured method to call functions through window.postMessage",
+  "description": "JsonRPC wrapper over window.postMessage to allow a structured method to call functions through window.postMessage",
   "keywords": [
     "post message",
     "jsonrpc"
   ],
   "main": "dist/index.js",
+  "type": "module",
   "types": "dist/index.d.ts",
   "scripts": {
     "clean": "rm -f tsconfig.tsbuildinfo && rm -rf dist",
-    "build": "tsc --build"
+    "build": "tsc --build",
+    "lint": "tsc --build && eslint",
+    "lint:fix": "tsc --build && eslint --fix",
+    "test": "tsc --build && jest"
   },
   "dependencies": {
     "@getmash/post-message": "^1.0.2",
     "uuid": "^9.0.0"
   },
   "devDependencies": {
-    "@types/uuid": "^8.3.4"
+    "@babel/preset-env": "^7.20.2",
+    "@types/babel__preset-env": "^7",
+    "@types/eslint": "^8",
+    "@types/jest": "^29.2.2",
+    "@types/prettier": "^2",
+    "@types/uuid": "^8.3.4",
+    "@typescript-eslint/eslint-plugin": "^5.42.0",
+    "@typescript-eslint/parser": "^5.42.0",
+    "babel-jest": "^29.2.2",
+    "eslint": "^8.26.0",
+    "eslint-config-prettier": "^8.5.0",
+    "eslint-import-resolver-typescript": "^3.5.2",
+    "eslint-plugin-import": "^2.26.0",
+    "eslint-plugin-jest": "^27.1.4",
+    "eslint-plugin-prettier": "^4.2.1",
+    "jest": "^29.2.2",
+    "jest-environment-jsdom": "^29.2.2",
+    "prettier": "^2.7.1",
+    "ts-jest": "^29.0.3",
+    "tslib": "^2.4.1",
+    "typescript": "^4.8.4"
   },
   "files": [
     "dist"

packages/jsonrpc-engine/tsconfig.json

@@ -1,10 +1,9 @@
 {
-  "extends": "../../tsconfig.base.json",
+  "extends": "../../conf/tsconfig.base.json",
   "compilerOptions": {
     "rootDir": "./src",
-    "outDir": "./dist",
+    "outDir": "./dist"
   },
-  "include": ["./src/**/*.ts"],
-  "exclude": ["./src/**/*.test.*"],
+  "include": ["./src/**/*"],
   "references": [{ "path": "../post-message" }]
 }

packages/post-message/.eslintrc.js → packages/post-message/.eslintrc.cjs

@@ -2,7 +2,6 @@ module.exports = {
   extends: ["../../conf/eslintrc.base.js"],
   root: true,
   parserOptions: {
-    project: "tsconfig.lint.json",
     tsconfigRootDir: __dirname,
   },
 };

packages/post-message/jest.config.cjs

@@ -0,0 +1 @@
+module.exports = require('../../conf/jest/jest.base.js')({ testEnvironment: "jsdom" });