Fix operator precedence in test-unit parser and nested node removal

Author: cosgroveb

swesmith/bug_gen/procedural/ruby/base.py

@@ -45,8 +45,11 @@ def walk(n):
     def replace_node(code: str, node, replacement: str) -> str:
         """Replace a tree-sitter node's text via byte offsets."""
         code_bytes = code.encode("utf8")
-        new_bytes = (code_bytes[:node.start_byte]
-                     + replacement.encode("utf8") + code_bytes[node.end_byte:])
+        new_bytes = (
+            code_bytes[: node.start_byte]
+            + replacement.encode("utf8")
+            + code_bytes[node.end_byte :]
+        )
         return new_bytes.decode("utf8")
 
     def _remove_matching_nodes(
@@ -64,6 +67,7 @@ def _remove_matching_nodes(
         def collect(n):
             if n.type in node_types and self.flip():
                 removals.append(n)
+                return  # skip children to avoid stale byte offsets on nested removals
             for child in n.children:
                 collect(child)
 
@@ -74,7 +78,9 @@ def collect(n):
 
         source_bytes = code_entity.src_code.encode("utf8")
         for node in sorted(removals, key=lambda x: x.start_byte, reverse=True):
-            source_bytes = source_bytes[:node.start_byte] + source_bytes[node.end_byte:]
+            source_bytes = (
+                source_bytes[: node.start_byte] + source_bytes[node.end_byte :]
+            )
 
         modified_code = source_bytes.decode("utf8")
 

swesmith/profiles/ruby.py

@@ -73,10 +73,12 @@ def parse_log_ruby_test(log: str) -> dict[str, str]:
             current_class = stripped[:-1]
             continue
         # test-unit verbose: "  test_name:   .: (0.001234)"
-        if current_class and ".: (" in stripped or "F: (" in stripped or "E: (" in stripped:
+        if current_class and (
+            ".: (" in stripped or "F: (" in stripped or "E: (" in stripped
+        ):
             match = re.match(r"^(\S+):\s+([.FE]):\s+\(", stripped)
             if match:
-                test_name = f"{current_class}#{ match.group(1)}"
+                test_name = f"{current_class}#{match.group(1)}"
                 status_char = match.group(2)
                 if status_char == ".":
                     test_status_map[test_name] = TestStatus.PASSED.value