Revert back to yarn, but keep the artifact renaming

Author: Newton Der

.github/workflows/build.yml

@@ -76,18 +76,23 @@ jobs:
         uses: actions/setup-node@v4
         with:
           node-version: 20
-          # Use npm for caching, not yarn
-          cache: 'npm'
-          cache-dependency-path: '**/package-lock.json'
 
-      # Step 4: Apply patches from the 'patches' directory if it exists.
+      # Step 4: Cache Yarn dependencies to speed up subsequent builds.
+      - name: Cache Yarn dependencies
+        uses: actions/cache@v4
+        with:
+          path: |
+            vscode/node_modules
+          key: ${{ runner.os }}-node20-${{ hashFiles('vscode/package.json', 'vscode/yarn.lock') }}
+
+      # Step 5: Apply patches from the 'patches' directory if it exists.
       - name: Apply patches (if any)
         run: |
           if [ -d patches ] && [ "$(ls -A patches)" ]; then
             quilt push -a || true
           fi
 
-      # Step 5: Generate a version string for this specific build.
+      # Step 6: Generate a version string for this specific build.
       # It's based on the commit SHA to create a unique identifier.
       - name: Set Development Version
         id: version
@@ -97,32 +102,31 @@ jobs:
           echo "VERSION=$VERSION" >> $GITHUB_ENV
           echo "Generated version for this build: $VERSION"
 
-      # Step 6: The main build process for vscode, now using npm.
+      # Step 7: The main build process for vscode.
       - name: Build vscode
         run: |
           cd vscode
           export DISABLE_V8_COMPILE_CACHE=1
           export UV_THREADPOOL_SIZE=4
+          npm i -g node-gyp
+          yarn install --network-concurrency 1
           
-          # Install dependencies using npm
-          npm install
-          
-          # The logic for temporarily removing and re-adding ripgrep remains
+          # Remove and re-add ripgrep
           VSCODE_RIPGREP_VERSION=$(jq -r '.dependencies."@vscode/ripgrep"' package.json)
           mv package.json package.json.orig
           jq 'del(.dependencies."@vscode/ripgrep")' package.json.orig > package.json
           
           # Re-run install to remove ripgrep
-          npm install
+          yarn install
           
-          # Add ripgrep back using npm
-          npm install --ignore-scripts "@vscode/ripgrep@${VSCODE_RIPGREP_VERSION}"
+          # Add ripgrep back
+          yarn add --ignore-scripts "@vscode/ripgrep@${VSCODE_RIPGREP_VERSION}"
           
           ARCH_ALIAS=linux-x64
-          # Run the gulp build task using npx
-          npx gulp vscode-reh-web-${ARCH_ALIAS}-min
+          # Run the gulp build task
+          yarn gulp vscode-reh-web-${ARCH_ALIAS}-min
 
-      # Step 7: Find the exact path of the original build output directory.
+      # Step 8: Find the exact path of the original build output directory.
       - name: Find build output
         id: find_output
         run: |
@@ -134,7 +138,7 @@ jobs:
           echo "Build output found at: $BUILD_PATH"
           echo "build_path=$BUILD_PATH" >> $GITHUB_OUTPUT
       
-      # Step 8: Rename the build output directory to sagemaker-code-editor
+      # Step 9: Rename the build output directory to sagemaker-code-editor
       - name: Rename build output directory
         id: rename_output
         run: |
@@ -144,7 +148,7 @@ jobs:
           echo "Renamed build output directory to: $PARENT_DIR/sagemaker-code-editor"
           echo "build_path=$PARENT_DIR/sagemaker-code-editor" >> $GITHUB_OUTPUT
       
-      # Step 9: Create a compressed tarball of the renamed build output.
+      # Step 10: Create a compressed tarball of the renamed build output.
       - name: Create tarball archive
         run: |
           TARBALL="sagemaker-code-editor-${{ env.VERSION }}.tar.gz"
@@ -154,11 +158,11 @@ jobs:
           echo "Creating '$TARBALL' from '$BUILD_DIR_NAME' in '$PARENT_DIR'"
           tar czf $TARBALL -C "$PARENT_DIR" "$BUILD_DIR_NAME"
       
-      # Step 10: Upload the tarball as a build artifact.
+      # Step 11: Upload the tarball as a build artifact.
       - name: Upload build artifact
         uses: actions/upload-artifact@v4
         with:
-          name: npm-package
+          name: yarn-package
           path: sagemaker-code-editor-${{ env.VERSION }}.tar.gz
   # Run end-to-end tests after the build is complete
   run-e2e-tests: