Update ignoredDirectoriesAndFiles to avoid false positives

Author: jguoamz

packages/core/src/codewhispererChat/constants.ts

@@ -59,111 +59,13 @@ export const defaultStreamingResponseTimeoutInMs = 180_000
 export const ignoredDirectoriesAndFiles = [
     // Dependency directories
     'node_modules',
-    '.venv',
-    'venv',
-    'bower_components',
-    'jspm_packages',
     // Build outputs
     'dist',
     'build',
     'out',
-    'target',
-    '.gradle',
-    '.pytest_cache',
-    '.tox',
-    '__snapshots__',
-    // Compiled files
-    '*.class',
-    '*.o',
-    '*.a',
-    '*.so',
-    '*.pyc',
-    '__pycache__',
-    '*.exe',
-    '*.dll',
-    // Package files
-    '*.jar',
-    '*.gem',
-    '*.vsix',
-    '*.zip',
-    '*.tar.gz',
-    '*.rar',
-    // IDE and editor files
+    // IDE and editor directories
     '.idea/',
     '.vscode/',
-    '*.sublime-*',
-    '*.swp',
-    '*.swo',
-    '.project',
-    '.classpath',
-    '*.iml',
-    // Log files
-    '*.log',
-    'logs/',
-    'npm-debug.log*',
-    // Coverage and test reports
-    'coverage/',
-    '.nyc_output/',
-    'test-results/',
-    '.test-reports/',
-    // Cache directories
-    '.cache/',
-    '.sass-cache/',
-    '.eslintcache',
-    '.parcel-cache',
-    // Environment and local configuration
-    '.env',
-    '.env.local',
-    '*.env.*',
-    '*.local.json',
-    '*.local.yml',
-    'config.local.*',
-    '.npmrc',
-    '.yarnrc',
-    '.dockerignore',
     // OS specific files
     '.DS_Store',
-    'Thumbs.db',
-    'desktop.ini',
-    // Temporary files
-    'tmp/',
-    'temp/',
-    '*.tmp',
-    '*.bak',
-    '*.bk',
-    // Generated documentation
-    'docs/_build/',
-    'site/',
-    'public/',
-    // Database files
-    '*.sqlite',
-    '*.db',
-    // Secrets and credentials
-    '*.pem',
-    '*.key',
-    'id_rsa',
-    'id_dsa',
-    '*.pfx',
-    '*.p12',
-    'credentials.json',
-    '*_credentials.*',
-    'aws-credentials.*',
-    'secrets.*',
-    // Version Control Directories
-    '.git/',
-    '.svn/',
-    '.hg/',
-    '.bzr/',
-    // Generated Code
-    '*.generated.*',
-    '*.auto.*',
-    '*.g.*',
-    // Cloud Provider Specific
-    '.terraform/',
-    '.serverless/',
-    'cdk.out/',
-    '.aws-sam/',
-    '.amplify/',
-    // Mobile Development
-    'Pods/',
 ]

packages/core/src/codewhispererChat/tools/tool_index.json

@@ -83,7 +83,7 @@
     },
     "listDirectory": {
         "name": "listDirectory",
-        "description": "List the contents of a directory and its subdirectories, it will filter out build and artifacts directories and files such as `node_modules/`, `build/`, `.git/` or `*.log`.\n * Use this tool for discovery, before using more targeted tools like fsRead.\n *Useful to try to understand the file structure before diving deeper into specific files.\n *Can be used to explore the codebase.\n *Results clearly distinguish between files, directories or symlinks with [F], [D] and [L] prefixes.",
+        "description": "List the contents of a directory and its subdirectories, it will filter out build outputs such as `build/`, `out/` and `dist` and dependency directory such as `node_modules/` and IDE directories such as `.idea/`, `.vscode/`.\n * Use this tool for discovery, before using more targeted tools like fsRead.\n *Useful to try to understand the file structure before diving deeper into specific files.\n *Can be used to explore the codebase.\n *Results clearly distinguish between files, directories or symlinks with [F], [D] and [L] prefixes.",
         "inputSchema": {
             "type": "object",
             "properties": {

packages/core/src/testInteg/shared/utilities/workspaceUtils.test.ts

@@ -595,7 +595,6 @@ describe('workspaceUtils', () => {
     describe('shouldIgnoreDirAndFile', function () {
         it('handles exact matches', function () {
             assert.strictEqual(shouldIgnoreDirAndFile('node_modules', vscode.FileType.Directory), true)
-            assert.strictEqual(shouldIgnoreDirAndFile('.env', vscode.FileType.File), true)
             assert.strictEqual(shouldIgnoreDirAndFile('random_file.txt', vscode.FileType.File), false)
         })
 
@@ -604,24 +603,6 @@ describe('workspaceUtils', () => {
             assert.strictEqual(shouldIgnoreDirAndFile('logs', vscode.FileType.Directory), true)
             assert.strictEqual(shouldIgnoreDirAndFile('.idea', vscode.FileType.File), false)
         })
-
-        it('handles wildcard patterns at the start', function () {
-            assert.strictEqual(shouldIgnoreDirAndFile('example.class', vscode.FileType.File), true)
-            assert.strictEqual(shouldIgnoreDirAndFile('test.log', vscode.FileType.File), true)
-            assert.strictEqual(shouldIgnoreDirAndFile('class.example', vscode.FileType.File), false)
-        })
-
-        it('handles wildcard patterns at the end', function () {
-            assert.strictEqual(shouldIgnoreDirAndFile('npm-debug.log123', vscode.FileType.File), true)
-            assert.strictEqual(shouldIgnoreDirAndFile('npm-debug.txt', vscode.FileType.File), false)
-        })
-
-        it('handles complex wildcard patterns', function () {
-            assert.strictEqual(shouldIgnoreDirAndFile('test.env.local', vscode.FileType.File), true)
-            assert.strictEqual(shouldIgnoreDirAndFile('my_credentials.json', vscode.FileType.File), true)
-            assert.strictEqual(shouldIgnoreDirAndFile('config.local.json', vscode.FileType.File), true)
-            assert.strictEqual(shouldIgnoreDirAndFile('random.txt', vscode.FileType.File), false)
-        })
     })
 
     describe('findStringInDirectory', function () {