Merge branch 'main' into dallin/unknown-provider-fix

Author: tomasz-stefaniak

.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'

core/context/providers/FileContextProvider.ts

@@ -37,21 +37,6 @@ class FileContextProvider extends BaseContextProvider {
       await extras.ide.getWorkspaceDirs(),
     );
 
-    if (isSecurityConcern(relativePathOrBasename)) {
-      return [
-        {
-          description: last2Parts,
-          content:
-            "Content redacted, this file cannot be viewed for security reasons",
-          name: baseName,
-          uri: {
-            type: "file",
-            value: fileUri,
-          },
-        },
-      ];
-    }
-
     return [
       {
         name: baseName,

core/context/providers/ProblemsContextProvider.ts

@@ -29,14 +29,6 @@ class ProblemsContextProvider extends BaseContextProvider {
           problem.filepath,
           workspaceDirs,
         );
-        if (isSecurityConcern(relativePathOrBasename)) {
-          return {
-            description: "Problems in current file",
-            content:
-              "Content was redacted because the file is detected as a potential security concern",
-            name: `Warnings in ${baseName}`,
-          };
-        }
         const content = await ide.readFile(problem.filepath);
         const lines = content.split("\n");
         const rangeContent = lines

core/indexing/ignore.ts

@@ -67,14 +67,6 @@ export const DEFAULT_SECURITY_IGNORE_DIRS = [
   // Environment and configuration directories
   ".env/",
   "env/",
-  ".venv/",
-  "venv/",
-
-  // IDE directories that may contain cached credentials
-  // For now excluding, reasons for this unclear
-  // ".vscode/",
-  // ".idea/",
-  // ".vs/",
 
   // Cloud provider credential directories
   ".aws/",
@@ -168,6 +160,8 @@ export const ADDITIONAL_INDEXING_IGNORE_FILETYPES = [
   "*.swp",
   "*.jsonl",
   // "*.prompt", // can be incredibly confusing for the LLM to have another set of instructions injected into the prompt
+  // Application specific
+  ".continue/",
 ];
 
 export const ADDITIONAL_INDEXING_IGNORE_DIRS = [
@@ -189,6 +183,13 @@ export const ADDITIONAL_INDEXING_IGNORE_DIRS = [
   ".cache/",
   "gems/",
   "vendor/",
+
+  ".venv/",
+  "venv/",
+
+  ".vscode/",
+  ".idea/",
+  ".vs/",
 ];
 
 // Combined patterns: security + additional

core/indexing/ignore.vitest.ts

@@ -81,18 +81,7 @@ describe("isSecurityConcern", () => {
     it("should detect environment directories as security concerns", () => {
       expect(isSecurityConcern(".env/")).toBe(true);
       expect(isSecurityConcern("env/")).toBe(true);
-      expect(isSecurityConcern(".venv/")).toBe(true);
-      expect(isSecurityConcern("venv/")).toBe(true);
       expect(isSecurityConcern(".env/config")).toBe(true);
-      expect(isSecurityConcern("project/.venv/lib")).toBe(true);
-    });
-
-    it("should detect IDE directories as security concerns", () => {
-      // expect(isSecurityConcern(".vscode/")).toBe(true);
-      // expect(isSecurityConcern(".idea/")).toBe(true);
-      // expect(isSecurityConcern(".vs/")).toBe(true);
-      expect(isSecurityConcern(".vscode/settings.json")).toBe(true);
-      // expect(isSecurityConcern(".idea/workspace.xml")).toBe(true);
     });
 
     it("should detect cloud provider directories as security concerns", () => {