Merge pull request #7296 from continuedev/nate/more-warnings-fixes

Nate/more-warnings-fixes

Author: sestinj

.github/actions/run-jetbrains-tests/action.yml

@@ -93,6 +93,13 @@ runs:
         path: binary/node_modules
         key: ${{ runner.os }}-binary-node-modules-${{ hashFiles('binary/package-lock.json') }}
 
+    - name: Install binary dependencies
+      if: steps.binary-cache.outputs.cache-hit != 'true'
+      shell: bash
+      run: |
+        cd binary
+        npm ci
+
     - name: Build the binaries
       shell: bash
       run: |

.github/workflows/cli-pr-checks.yml

@@ -36,7 +36,7 @@ jobs:
       - name: Install dependencies
         run: |
           cd extensions/cli
-          npm ci
+          npm ci --include=optional
 
       - name: Run linting
         run: |
@@ -58,8 +58,14 @@ jobs:
         uses: actions/setup-node@v4
         with:
           node-version: ${{ matrix.node-version }}
-          cache: "npm"
-          cache-dependency-path: extensions/cli/package-lock.json
+
+      - name: Cache CLI node_modules
+        uses: actions/cache@v4
+        with:
+          path: extensions/cli/node_modules
+          key: ${{ runner.os }}-node${{ matrix.node-version }}-cli-modules-${{ hashFiles('extensions/cli/package-lock.json') }}
+          restore-keys: |
+            ${{ runner.os }}-node${{ matrix.node-version }}-cli-modules-
 
       - name: Setup packages
         uses: ./.github/actions/setup-packages
@@ -72,13 +78,18 @@ jobs:
       - name: Install dependencies
         run: |
           cd extensions/cli
-          npm ci
+          npm ci --include=optional
 
       - name: Build
         run: |
           cd extensions/cli
           npm run build
 
+      - name: Run smoke tests
+        run: |
+          cd extensions/cli
+          npm run test:smoke
+
       # e2e tests are failing on Windows specifically - likely due to stdout flush issues
       - name: Run tests
         if: matrix.os != 'windows-latest'