fixes to find_in_file and CI report format error

Author: dsarno

.github/workflows/claude-nl-suite.yml

@@ -816,10 +816,10 @@ jobs:
 
           def id_from_filename(p: Path):
             n = p.name
-            m = re.match(r'NL(\d+)_results\.xml$', n, re.I)
+            m = re.match(r'NL-?(\d+)_results\.xml$', n, re.I)
             if m:
               return f"NL-{int(m.group(1))}"
-            m = re.match(r'T([A-J])_results\.xml$', n, re.I)
+            m = re.match(r'T-?([A-J])_results\.xml$', n, re.I)
             if m:
               return f"T-{m.group(1).upper()}"
             return None
@@ -863,10 +863,10 @@ jobs:
           seen = set()
           def id_from_filename(p: Path):
             n = p.name
-            m = re.match(r'NL(\d+)_results\.xml$', n, re.I)
+            m = re.match(r'NL-?(\d+)_results\.xml$', n, re.I)
             if m:
               return f"NL-{int(m.group(1))}"
-            m = re.match(r'T([A-J])_results\.xml$', n, re.I)
+            m = re.match(r'T-?([A-J])_results\.xml$', n, re.I)
             if m:
               return f"T-{m.group(1).upper()}"
             return None

Server/src/services/tools/find_in_file.py

@@ -73,6 +73,7 @@ async def find_in_file(
     max_results: Annotated[int, "Cap results to avoid huge payloads"] = 200,
     ignore_case: Annotated[bool | str | None, "Case insensitive search"] = True,
 ) -> dict[str, Any]:
+    # project_root is currently unused but kept for interface consistency
     unity_instance = get_unity_instance_from_context(ctx)
     await ctx.info(
         f"Processing find_in_file: {uri} (unity_instance={unity_instance or 'default'})")
@@ -100,7 +101,7 @@ async def find_in_file(
         try:
             contents = base64.b64decode(data.get("encodedContents", "").encode(
                 "utf-8")).decode("utf-8", "replace")
-        except Exception:
+        except (ValueError, TypeError, base64.binascii.Error):
             contents = contents or ""
 
     if contents is None:
@@ -120,18 +121,6 @@ async def find_in_file(
     except re.error as e:
         return {"success": False, "message": f"Invalid regex pattern: {e}"}
 
-    matches = []
-    lines = contents.splitlines()
-    
-    # Helper to map index to line number
-    def get_line_number(index, content_lines):
-        # This is a bit slow for large files if we do it for every match, 
-        # but robust. 
-        # Better: iterate matches and count newlines?
-        # Or just search line by line?
-        # Searching line by line is safer for line-based results, but regex might span lines.
-        pass
-
     # If the regex is not multiline specific (doesn't contain \n literal match logic), 
     # we could iterate lines. But users might use multiline regexes.
     # Let's search the whole content and map back to lines.