chore: switched to yarn, updates to work with internal monorepo

.github/workflows/default-ci.yml

@@ -2,29 +2,40 @@ name: default-ci-jobs
 on: [push, pull_request]
 jobs:
   lint:
-    runs-on: macos-latest
+    runs-on: ubuntu-latest
     steps:
       - uses: actions/checkout@v2
       - uses: actions/setup-node@v1
         with:
           node-version: 15.x
-      - run: npm ci
-      - run: npm run prettier-check
-      - run: npm run lint
-      - run: npm run bazel:lint
-  unit-tests:
-    runs-on: macos-latest
+
+      - name: Setup node_modules
+        run: yarn
+
+      - name: Lint checks
+        run: |
+          yarn prettier-check
+          yarn lint
+          yarn bazel:lint
+  build_and_test:
+    runs-on: ubuntu-latest
     steps:
       - uses: actions/checkout@v2
       - uses: actions/setup-node@v1
         with:
           node-version: 15.x
-      - run: npm ci
-      - run: npm run test:unit
+      - uses: c-hive/gha-yarn-cache@v1
+      - uses: bazelbuild/setup-bazelisk@v1
+
+      - name: Build all targets
+        run: bazel build //...
+
+      - name: Run all unit tests
+        run: bazel test --test_tag_filters=unit //...
 # TODO: blocked on running integration server through cypress - see issue #15
 #  e2e-tests:
 #    runs-on: macos-latest
 #    steps:
 #      - uses: actions/checkout@v2
 #      - uses: actions/setup-node@v1
-#      - run: npm run test:e2e
+#      - run: yarn test:e2e

DEVELOPER.md

@@ -20,7 +20,7 @@ or
 npm install -g @bazel/bazelisk
 ```
 
-`Bazel` will invoke `NPM` and manage all dependencies.
+`Bazel` will invoke `Yarn` and manage all dependencies.
 
 ## `bazel` vs `ibazel`
 

WORKSPACE

@@ -15,20 +15,20 @@ http_archive(
 # Check the rules_nodejs version and download npm dependencies
 # Note: bazel (version 2 and after) will check the .bazelversion file so we don't need to
 # assert on that.
-load("@build_bazel_rules_nodejs//:index.bzl", "check_rules_nodejs_version", "node_repositories", "npm_install")
+load("@build_bazel_rules_nodejs//:index.bzl", "check_rules_nodejs_version", "node_repositories", "yarn_install")
 
 check_rules_nodejs_version(minimum_version_string = "2.2.0")
 
 # Setup the Node.js toolchain
 node_repositories(
-    node_version = "15.0.0",
+    node_version = "15.0.1",
     package_json = ["//:package.json"],
 )
 
-npm_install(
+yarn_install(
     name = "npm",
     package_json = "//:package.json",
-    package_lock_json = "//:package-lock.json",
+    yarn_lock = "//:yarn.lock",
 )
 
 load("@npm//@bazel/cypress:index.bzl", "cypress_repository")

client/BUILD.bazel

@@ -1,4 +1,5 @@
 load("@build_bazel_rules_nodejs//:index.bzl", "generated_file_test", "js_library", "pkg_npm")
+load("@npm//@bazel/esbuild:index.bzl", "esbuild")
 load("@npm//@bazel/rollup:index.bzl", "rollup_bundle")
 load("@npm//@bazel/terser:index.bzl", "terser_minified")
 load("@npm//@bazel/typescript:index.bzl", "ts_project")
@@ -31,6 +32,20 @@ ts_project(
     # TODO: https://bazelbuild.github.io/rules_nodejs/TypeScript.html#ts_project-supports_workers
 )
 
+esbuild(
+    name = "client_cjs",
+    entry_point = "index.ts",
+    platform = "node",
+    tool = select({
+        "@bazel_tools//src/conditions:darwin": "@esbuild_darwin//:bin/esbuild",
+        "@bazel_tools//src/conditions:linux_x86_64": "@esbuild_linux//:bin/esbuild",
+        "@bazel_tools//src/conditions:windows": "@esbuild_windows//:esbuild.exe",
+    }),
+    deps = [
+        ":client",
+    ],
+)
+
 filegroup(
     name = "project.d.ts",
     srcs = [":project"],
@@ -114,10 +129,12 @@ api_extractor(
         "--verbose",
         "--local",
         "--config",
-        "client/api-extractor.json",
+        "client/api-extractor-$(TARGET_CPU).json",
     ],
     data = [
         "api-extractor.json",
+        "api-extractor-darwin.json",
+        "api-extractor-k8.json",
         ":project",
         "//:package.json",
         "//:tsconfig.base.json",
@@ -148,12 +165,11 @@ pkg_npm(
     substitutions = {"//internal/": "//"},
     deps = [
         ":LICENSE",
+        ":client_cjs",
         ":qwik",
         ":qwik.d.ts",
         ":qwik.min",
         ":qwikloader",
         ":qwikloader.min",
-        "//client/cjs:package.json",
-        "//client/cjs:qwik",
     ],
 )

client/api-extractor-darwin.json

@@ -0,0 +1,6 @@
+{
+  "$schema": "https://developer.microsoft.com/json-schemas/api-extractor/v7/api-extractor.schema.json",
+
+  "extends": "./api-extractor.json",
+  "projectFolder": "../bazel-out/darwin-fastbuild/bin"
+}

client/api-extractor-k8.json

@@ -0,0 +1,6 @@
+{
+  "$schema": "https://developer.microsoft.com/json-schemas/api-extractor/v7/api-extractor.schema.json",
+
+  "extends": "./api-extractor.json",
+  "projectFolder": "../bazel-out/k8-fastbuild/bin"
+}

client/api-extractor.json

@@ -32,7 +32,7 @@
    * SUPPORTED TOKENS: <lookup>
    * DEFAULT VALUE: "<lookup>"
    */
-  "projectFolder": "..",
+  "projectFolder": "../bazel-out/darwin-fastbuild/bin",
 
   /**
    * (REQUIRED) Specifies the .d.ts file to be used as the starting point for analysis.  API Extractor
@@ -45,7 +45,7 @@
    *
    * SUPPORTED TOKENS: <projectFolder>, <packageName>, <unscopedPackageName>
    */
-  "mainEntryPointFilePath": "<projectFolder>/bazel-out/darwin-fastbuild/bin/client/index.d.ts",
+  "mainEntryPointFilePath": "<projectFolder>/client/index.d.ts",
 
   /**
    * A list of NPM package names whose exports should be treated as part of this package.
@@ -89,7 +89,7 @@
      * DEFAULT VALUE: no overrideTsconfig section
      */
     "overrideTsconfig": {
-      "include": ["./bazel-out/darwin-fastbuild/bin/client/index.d.ts"]
+      "include": [".//client/index.d.ts"]
     }
     /**
      * This option causes the compiler to be invoked with the --skipLibCheck option. This option is not recommended
@@ -135,7 +135,7 @@
      * SUPPORTED TOKENS: <projectFolder>, <packageName>, <unscopedPackageName>
      * DEFAULT VALUE: "<projectFolder>/etc/"
      */
-    "reportFolder": "<projectFolder>/bazel-out/darwin-fastbuild/bin/client",
+    "reportFolder": "<projectFolder>/client",
 
     /**
      * Specifies the folder where the temporary report file is written.  The file name portion is determined by
@@ -195,7 +195,7 @@
      * SUPPORTED TOKENS: <projectFolder>, <packageName>, <unscopedPackageName>
      * DEFAULT VALUE: "<projectFolder>/dist/<unscopedPackageName>.d.ts"
      */
-    "untrimmedFilePath": "<projectFolder>/bazel-out/darwin-fastbuild/bin/client/<unscopedPackageName>.d.ts"
+    "untrimmedFilePath": "<projectFolder>/client/<unscopedPackageName>.d.ts"
 
     /**
      * Specifies the output path for a .d.ts rollup file to be generated with trimming for a "beta" release.