fix(rule_fast): retain body literals when comments/URLs contain parens

The skeleton regex added in 6fb595a used [^(]*$ to exclude computed
assignments, but it also dropped innocent literals like
`limit = 42  # note (v2)` and URL strings containing parentheses.
Tighten the pattern to literal numbers/strings only and keep trailing
comments.

Also fix _compress_command dropping the exit-code tail on 7-9 line logs
when len(lines) <= 9 left tail empty so remainder lines were discarded.

Co-authored-by: Daniel <DJLougen@users.noreply.github.com>

Author: cursoragent

hive/rule_fast/__init__.py

@@ -175,7 +175,10 @@ def _classify(role: str, content_type: str, content: str = "") -> str:
     r"^\s*(?:async\s+def|def|class|pub fn|fn|function|export)\b"
     r"|^[A-Za-z_]\w*\s*=\s"
     r"|^(?:import|from)\s"
-    r"|^\s+[A-Z_a-z]\w*\s*(?::[^=]+)?=\s*(?:-?\d|[\"'])[^(]*$"
+    # Literal body assignments only — allow trailing comments and quoted
+    # strings that contain parens; still reject computed rhs (`payload.get(...)`).
+    r"|^\s+[A-Z_a-z]\w*\s*(?::[^=]+)?=\s*"
+    r"(?:-?\d+\s*(?:#.*)?|\"[^\"]*\"|'[^']*')\s*$"
 )
 
 
@@ -229,8 +232,14 @@ def _compress_command(content: str) -> str:
     """
     lines = content.splitlines()
     head = lines[:6]
-    tail = lines[-3:] if len(lines) > 9 else []
-    middle = lines[6 : len(lines) - len(tail)]
+    # Keep the tail (exit code) for short logs too; the >9 guard only avoids
+    # head/tail overlap on longer output.
+    if len(lines) > 9:
+        tail = lines[-3:]
+        middle = lines[6 : len(lines) - len(tail)]
+    else:
+        tail = lines[6:]
+        middle = []
     errors = [line for line in middle if _RE_ERRORISH.search(line)][:_MAX_ERROR_LINES]
     parts = head + (["..."] if middle else []) + errors + (["..."] if tail else []) + tail
     return f"[cmd] ({len(content)} chars)\n" + "\n".join(parts)

tests/test_stack.py

@@ -10,6 +10,8 @@
     Label,
     Message,
     RuleFastHoneyComb,
+    _compact_file,
+    _compress_command,
     _compress_test_output,
     _infer_content_type,
 )
@@ -54,6 +56,40 @@ def test_compress_test_output_collapses_lines():
     assert "12 passed" not in out  # we didn't claim that
 
 
+def test_compact_file_keeps_literal_assignments_with_paren_comments():
+    """Body literals must survive comments/URLs containing parentheses."""
+    content = "\n".join(
+        [
+            "def handler(payload, *, timeout=30):",
+            '    """Handler."""',
+            "    limit = 4321  # see RFC (section 3)",
+            '    url = "http://registry.internal/path(v2)/push"',
+            "    x = payload.get('retry')",
+            "    return limit",
+        ]
+    )
+    out = _compact_file(content)
+    assert "limit = 4321" in out
+    assert 'url = "http://registry.internal/path(v2)/push"' in out
+    assert "payload.get" not in out
+
+
+def test_compress_command_keeps_exit_code_on_short_logs():
+    short = "\n".join(
+        [
+            "$ make deploy",
+            "step 1 ok",
+            "step 2 ok",
+            "step 3 ok",
+            "step 4 ok",
+            "step 5 ok",
+            "step 6 ok",
+            "exit 1",
+        ]
+    )
+    assert "exit 1" in _compress_command(short)
+
+
 def test_route_falls_back_when_no_policy():
     stack = HiveStack()
     decision = stack.route({"goal": "x", "state": {}, "available_tools": []})