🪟 🔧 Switch to pnpm for package managing

.github/actions/cache-build-artifacts/action.yml

@@ -31,6 +31,15 @@ runs:
         restore-keys: |
           ${{ inputs.cache-key }}-npm-${{ runner.os }}-
 
+    - name: pnpm Caching
+      uses: actions/cache@v3
+      with:
+        path: |
+          ~/.pnpm-store
+        key: ${{ inputs.cache-key }}-pnpm-${{ runner.os }}-${{ hashFiles('**/pnpm-lock.yaml') }}
+        restore-keys: |
+          ${{ inputs.cache-key }}-pnpm-${{ runner.os }}-
+
     # this intentionally does not use restore-keys so we don't mess with gradle caching
     - name: Gradle and Python Caching
       uses: actions/cache@v3

airbyte-webapp/.eslintrc.js

@@ -9,7 +9,7 @@ module.exports = {
     "plugin:jsx-a11y/recommended",
     "plugin:@airbyte/recommended",
   ],
-  plugins: ["react", "@typescript-eslint", "prettier", "unused-imports", "css-modules", "jsx-a11y", "@airbyte"],
+  plugins: ["@typescript-eslint", "prettier", "unused-imports", "css-modules", "jsx-a11y", "@airbyte"],
   parserOptions: {
     ecmaVersion: 2020,
     sourceType: "module",

airbyte-webapp/.npmrc

@@ -1 +1,2 @@
 engine-strict=true
+enable-pre-post-scripts=true

airbyte-webapp/build.gradle

@@ -1,28 +1,46 @@
 plugins {
     id "base"
-    id "com.github.node-gradle.node" version "3.3.0"
+    id "com.github.node-gradle.node" version "3.4.0"
 }
 
 // Use the node version that's defined in the .nvmrc file
 def nodeVersion = new File("${projectDir}/.nvmrc").text.trim();
 
+// Read pnpm version to use from package.json engines.pnpm entry
+def pnpmVer = new groovy.json.JsonSlurper().parse(new File("${projectDir}/package.json")).engines.pnpm.trim();
+
 // This array should contain a path to all configs that are common to most build tasks and
 // might affect them (i.e. if any of those files change we want to rerun most tasks)
 def commonConfigs = [
     '.env',
     'package.json',
-    'package-lock.json',
+    'pnpm-lock.yaml',
     'tsconfig.json',
     '.prettierrc.js'
 ]
 
 node {
     download = true
     version = nodeVersion
-    npmInstallCommand = 'ci'
+    pnpmVersion = pnpmVer
+}
+
+task validateLockFiles {
+    description "Validate only a pnpm-lock.yaml lock file exists"
+    doLast {
+        assert file("pnpm-lock.yaml").exists()
+        assert !file("package-lock.json").exists()
+        assert !file("yarn.lock").exists()
+    }
 }
 
-npm_run_build {
+pnpmInstall.dependsOn validateLockFiles
+
+task pnpmBuild(type: PnpmTask) {
+    dependsOn pnpmInstall
+
+    args = ['build']
+
     inputs.files commonConfigs
     inputs.file '.eslintrc.js'
     inputs.dir 'public'
@@ -40,7 +58,7 @@ task test(type: NpmTask) {
 }
 
 task licenseCheck(type: NpmTask) {
-    dependsOn npmInstall
+    dependsOn pnpmInstall
 
     args = ['run', 'license-check']
     inputs.files commonConfigs
@@ -52,7 +70,7 @@ task licenseCheck(type: NpmTask) {
 }
 
 task validateLinks(type: NpmTask) {
-    dependsOn npmInstall
+    dependsOn pnpmInstall
 
     args = ['run', 'validate-links']
 
@@ -62,7 +80,7 @@ task validateLinks(type: NpmTask) {
 }
 
 task buildStorybook(type: NpmTask) {
-    dependsOn npmInstall
+    dependsOn pnpmInstall
     args = ['run', 'build:storybook']
 
     inputs.files commonConfigs
@@ -78,7 +96,7 @@ task buildStorybook(type: NpmTask) {
 }
 
 task copyBuildOutput(type: Copy) {
-    dependsOn copyDocker, npm_run_build
+    dependsOn copyDocker, pnpmBuild
 
     from "${project.projectDir}/build/app"
     into 'build/docker/bin/build'