fix: path anonymization should better deal with paths containing %

Signed-off-by: Fred Bricon <fbricon@gmail.com>

Author: fbricon

src/common/utils/telemetryUtils.ts

@@ -6,11 +6,12 @@
 
 // Regex patterns for path sanitization
 const NODE_MODULES_REGEX = /[\\\/]?(node_modules|node_modules\.asar)[\\\/]/;
-const FILE_REGEX_PATTERN = /(file:\/\/)?([a-zA-Z]:(\\\\|\\|\/)|(\\\\|\\|\/))?([\w-\._]+(\\\\|\\|\/))+[\w-\._]+/g;
+const FILE_REGEX_PATTERN = /(file:\/\/?)?([a-zA-Z]:(\\\\|\\|\/)|(\\\\|\\|\/))?([\w-\._%]+(\\\\|\\|\/))+[\w-\._%]+/g;
 
 // Default cleanup patterns to preserve common system/library paths
 const DEFAULT_CLEANUP_PATTERNS: RegExp[] = [
     /java\.\S*/,          // Java standard library and related packages (any non-whitespace chars)
+    /\/jdt\.ls-java-project\S*/, // Eclipse JDT Language Server Java project paths
 ];
 
 /**

src/tests/utils/telemetryUtils.test.ts

@@ -166,6 +166,18 @@ suite('Test anonymizeFilePaths', () => {
         assert.strictEqual(result, input);
     });
 
+    test('should preserve JDT Language Server Java project paths with default patterns', () => {
+        const input = 'Error at /jdt.ls-java-project/src/main/java/com/example/MyClass.java:45';
+        const result = anonymizeFilePaths(input);
+        assert.strictEqual(result, input);
+    });
+
+    test('should anonymize file:/ URLs even with JDT workspace paths', () => {
+        const input = 'Failed to read package name from file:/Users/someguy/Library/Application%20Support/Code/User/workspaceStorage/4f4553a7f3f79b5e8qa7c889b39fe96b/redhat.java/jdt_ws/Hello.java/';
+        const result = anonymizeFilePaths(input);
+        assert.strictEqual(result, 'Failed to read package name from <REDACTED: user-file-path>/');
+    });
+
     test('should anonymize user paths but preserve Java paths with default patterns', () => {
         const input = 'Error at /Users/john/project/src/main.ts:45 and java.base/java.lang.String.<init>(String.java:123)';
         const result = anonymizeFilePaths(input);