fix: move ide dirs and venv dirs to indexing not security ignores

Author: RomneyDa

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/",
@@ -191,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", () => {