fix: use JSON parsing for ripgrep results in workspace removal

  The processSearchResults function was still using legacy vimgrep format
  parsing while the actual ripgrep command uses --json output. This caused
  the "Remove from Workspace" feature to fail completely.

  - Replace vimgrep parsing with parseRipgrepResults from core/tlog-search
  - Update test cases to use JSON format matching ripgrep --json output
  - Remove unused RIPGREP_LINE_INDEX_OFFSET constant

Author: kubrickcode

src/src/remover.test.ts

@@ -6,11 +6,11 @@ import {
 
 describe("Remover Functions", () => {
   describe("processSearchResults", () => {
-    test("processes ripgrep results correctly", () => {
+    test("processes ripgrep JSON results correctly", () => {
       const searchResults = [
-        '/path/to/file1.ts:10:5:console.log("[TLOG] test1");',
-        '/path/to/file2.ts:20:3:console.log("[TLOG] test2");',
-        '/path/to/file1.ts:30:8:console.log("[TLOG] test3");',
+        '{"type":"match","data":{"path":{"text":"/path/to/file1.ts"},"lines":{"text":"console.log(\\"[TLOG] test1\\");"},"line_number":10,"submatches":[{"start":5}]}}',
+        '{"type":"match","data":{"path":{"text":"/path/to/file2.ts"},"lines":{"text":"console.log(\\"[TLOG] test2\\");"},"line_number":20,"submatches":[{"start":3}]}}',
+        '{"type":"match","data":{"path":{"text":"/path/to/file1.ts"},"lines":{"text":"console.log(\\"[TLOG] test3\\");"},"line_number":30,"submatches":[{"start":8}]}}',
       ];
 
       const result = processSearchResults(searchResults);
@@ -30,12 +30,12 @@ describe("Remover Functions", () => {
       });
     });
 
-    test("ignores malformed lines", () => {
+    test("ignores malformed JSON lines", () => {
       const searchResults = [
-        '/path/to/file1.ts:10:5:console.log("[TLOG] test");',
-        "malformed-line-without-colon",
-        '/path/to/file2.ts:not-a-number:3:console.log("[TLOG] test");',
-        '/path/to/file3.ts:15:2:console.log("[TLOG] test");',
+        '{"type":"match","data":{"path":{"text":"/path/to/file1.ts"},"lines":{"text":"console.log(\\"[TLOG] test\\");"},"line_number":10,"submatches":[{"start":5}]}}',
+        "malformed-json-line",
+        '{"type":"begin"}',
+        '{"type":"match","data":{"path":{"text":"/path/to/file3.ts"},"lines":{"text":"console.log(\\"[TLOG] test\\");"},"line_number":15,"submatches":[{"start":2}]}}',
       ];
 
       const result = processSearchResults(searchResults);
@@ -45,14 +45,17 @@ describe("Remover Functions", () => {
       expect(result[1].filePath).toBe("/path/to/file3.ts");
     });
 
-    test("ignores lines with extra colons in path", () => {
+    test("ignores non-match JSON types", () => {
       const searchResults = [
-        '/path/with:colons/file.ts:10:5:console.log("[TLOG] test");',
+        '{"type":"begin"}',
+        '{"type":"match","data":{"path":{"text":"/path/to/file1.ts"},"lines":{"text":"console.log(\\"[TLOG] test\\");"},"line_number":10,"submatches":[{"start":5}]}}',
+        '{"type":"end"}',
       ];
 
       const result = processSearchResults(searchResults);
 
-      expect(result).toHaveLength(0);
+      expect(result).toHaveLength(1);
+      expect(result[0].filePath).toBe("/path/to/file1.ts");
     });
   });
 

src/src/remover.ts

@@ -6,7 +6,6 @@ import {
   CONFIRMATION_YES,
   CONFIRMATION_NO,
 } from "./core/tlog-patterns";
-const RIPGREP_LINE_INDEX_OFFSET = 1;
 
 type RemovalScope = "current" | "workspace";
 
@@ -189,23 +188,13 @@ const deleteLinesFromSearchResults = async (
 export const processSearchResults = (
   searchResults: string[]
 ): Array<{ filePath: string; lineNumber: number }> => {
-  const processedResults: Array<{ filePath: string; lineNumber: number }> = [];
-
-  searchResults.forEach((line) => {
-    const parts = line.split(":");
-    if (parts.length < 2) return;
-
-    const filePath = parts[0];
-    const lineNumberStr = parts[1];
-    const lineNumber = parseInt(lineNumberStr, 10);
-
-    if (isNaN(lineNumber)) return;
-
-    const zeroBasedLineNumber = lineNumber - RIPGREP_LINE_INDEX_OFFSET;
-    processedResults.push({ filePath, lineNumber: zeroBasedLineNumber });
-  });
-
-  return processedResults;
+  const stdout = searchResults.join('\n');
+  const parsedResults = parseRipgrepResults(stdout);
+  
+  return parsedResults.map(result => ({
+    filePath: result.filePath,
+    lineNumber: result.line
+  }));
 };
 
 export const createFileLineMap = (
@@ -228,7 +217,12 @@ export const createFileLineMap = (
 };
 
 const parseSearchResults = (results: string[]): Map<string, number[]> => {
-  const processedResults = processSearchResults(results);
+  const stdout = results.join('\n');
+  const parsedResults = parseRipgrepResults(stdout);
+  const processedResults = parsedResults.map(result => ({
+    filePath: result.filePath,
+    lineNumber: result.line
+  }));
   return createFileLineMap(processedResults);
 };