refactor: build chain to rollup (#104)

Author: bkrem

.eslintrc

@@ -1,6 +1,7 @@
 {
   "rules": {
     "@typescript-eslint/ban-ts-ignore": ["off"],
+    "@typescript-eslint/ban-ts-comment": ["off"],
     "@typescript-eslint/camelcase": ["off"],
     "@typescript-eslint/explicit-function-return-type": ["off"],
     "@typescript-eslint/interface-name-prefix": ["off"],
@@ -13,6 +14,8 @@
     "comma-dangle": ["error", "always-multiline"],
     "no-async-promise-executor": ["off"],
     "no-empty-pattern": ["off"],
+    "no-use-before-define": ["off"],
+    "prefer-regex-literals": ["off"],
     "no-undef": ["error"],
     "no-var": ["error"],
     "object-curly-spacing": ["error", "always"],
@@ -33,7 +36,8 @@
     "standard",
     "eslint:recommended",
     "plugin:@typescript-eslint/eslint-recommended",
-    "plugin:@typescript-eslint/recommended"
+    "plugin:@typescript-eslint/recommended",
+    "prettier"
   ],
   "parser": "@typescript-eslint/parser",
   "parserOptions": {

.github/workflows/build.yml

@@ -0,0 +1,71 @@
+on: push
+
+name: CI Checks
+
+jobs:
+  ci:
+    name: CI
+    runs-on: ubuntu-latest
+    strategy:
+      matrix:
+        test-prefix:
+          - misc/logger
+          - misc/encoding
+          - misc/keyvaluestorage
+          - misc/time
+          - misc/heartbeat
+          - misc/history
+          - misc/environment
+          - misc/cacao
+          - misc/identity-keys
+          - misc/events
+          - misc/did-jwt
+          - misc/safe-json
+          - crypto/ecies-25519
+          - crypto/crypto
+          - crypto/randombytes
+          - jsonrpc/types
+          - jsonrpc/provider
+          - jsonrpc/utils
+          - jsonrpc/http-connection
+          - jsonrpc/ws-connection
+          - browser/window-getters
+          - browser/window-metadata
+          - relay/relay-api
+          - relay/relay-auth
+    steps:
+      - uses: actions/checkout@v3
+      - uses: actions/setup-node@v3
+        with:
+          node-version: 16
+
+      - name: Check node_modules cache
+        id: cache-node-modules
+        uses: actions/cache@v3
+        with:
+          path: |
+            node_modules
+            */*/node_modules
+          key: ${{ runner.os }}-nodemodules-${{ hashFiles('**/package-lock.json') }}
+          restore-keys: |
+            ${{ runner.os }}-nodemodules-
+
+      - name: Install NPM Dependencies
+        run: npm install
+
+      - name: Bootstrap packages
+        run: npm run bootstrap
+
+      - name: Prettier
+        run: npm run prettier --prefix=${{ matrix.test-prefix }} --if-present
+
+      - name: Lint
+        run: npm run lint --prefix=${{ matrix.test-prefix }} --if-present
+
+      - name: Build
+        run: npm run build
+
+      - name: Test
+        run: npm run test --prefix=${{ matrix.test-prefix }} --if-present
+        env:
+          TEST_PROJECT_ID: ${{ secrets.TEST_PROJECT_ID }}

.github/workflows/ci.yml

@@ -30,4 +30,3 @@ zip
 .nyc_output
 tsconfig.tsbuildinfo
 
-.vscode

.gitignore

@@ -2,5 +2,6 @@
   "tabWidth": 2,
   "useTabs": false,
   "trailingComma": "all",
-  "printWidth": 100
+  "printWidth": 100,
+  "arrowParens": "always"
 }

.prettierrc

@@ -0,0 +1,18 @@
+{
+  "editor.defaultFormatter": "esbenp.prettier-vscode",
+  "editor.formatOnSave": true,
+  "prettier.configPath": ".prettierrc",
+  "[javascript]": {
+    "editor.defaultFormatter": "esbenp.prettier-vscode"
+  },
+  "[typescript]": {
+    "editor.defaultFormatter": "esbenp.prettier-vscode"
+  },
+  "[jsx]": {
+    "editor.defaultFormatter": "esbenp.prettier-vscode"
+  },
+  "[tsx]": {
+    "editor.defaultFormatter": "esbenp.prettier-vscode"
+  },
+  "eslint.workingDirectories": ["./"]
+}

.vscode/settings.json

@@ -14,9 +14,11 @@
   "files": [
     "dist"
   ],
-  "main": "dist/cjs/index.js",
-  "types": "dist/cjs/index.d.ts",
-  "unpkg": "dist/umd/index.min.js",
+  "main": "dist/index.cjs.js",
+  "module": "dist/index.es.js",
+  "browser": "dist/index.es.js",
+  "unpkg": "dist/index.umd.js",
+  "types": "dist/types/index.d.ts",
   "homepage": "https://github.com/walletconnect/walletconnect-utils",
   "repository": {
     "type": "git",
@@ -27,37 +29,25 @@
   },
   "scripts": {
     "clean": "rm -rf dist",
-    "build:cjs": "tsc -p tsconfig.cjs.json",
-    "build:esm": "tsc -p tsconfig.esm.json",
-    "build:umd": "webpack",
-    "build": "run-s clean build:cjs build:esm build:umd ",
+    "build:pre": "npm run clean",
+    "build:types": "tsc",
+    "build:source": "rollup --config rollup.config.js",
+    "build": "npm run build:pre; npm run build:source; npm run build:types",
     "test": "env TS_NODE_PROJECT=\"tsconfig.cjs.json\" mocha --timeout 5000 --exit -r ts-node/register ./test/**/*.test.ts",
     "lint": "eslint -c '../../.eslintrc' --fix './src/**/*.ts'",
     "npm-publish:latest": "npm publish --access public --tag latest",
     "npm-publish:canary": "npm publish --access public --tag canary",
-    "prepublishOnly": "npm run test && npm run build"
+    "prepublishOnly": "npm run test && npm run build",
+    "prettier": "prettier --config ../../.prettierrc --check {src,test}/**/*.ts",
+    "format": "prettier --config ../../.prettierrc --write {src,test}/**/*.ts"
   },
   "devDependencies": {
     "@types/bn.js": "4.11.6",
     "@types/jest": "^26.0.15",
     "@types/node": "^14.14.7",
-    "husky": "^4.3.0",
     "typescript": "^4.9.5",
     "webpack": "^4.41.6",
     "webpack-cli": "^3.3.11"
   },
-  "husky": {
-    "hooks": {
-      "pre-commit": "run-s lint"
-    }
-  },
-  "prettier": {
-    "printWidth": 80,
-    "semi": true,
-    "singleQuote": false,
-    "trailingComma": "es5"
-  },
-  "dependencies": {
-    "tslib": "1.14.1"
-  }
+  "dependencies": {}
 }

browser/window-getters/package.json

@@ -0,0 +1,4 @@
+import { name, dependencies } from "./package.json";
+import createConfig from "../../rollup.config";
+
+export default createConfig(name, Object.keys(dependencies));

browser/window-getters/rollup.config.js

@@ -1,5 +1,5 @@
 export function getFromWindow<T>(name: string): T | undefined {
-  let res: T | undefined = undefined;
+  let res: T | undefined;
   if (typeof window !== "undefined" && typeof window[name] !== "undefined") {
     res = window[name];
   }