diff --git a/tests/tools/test_file_operations.py b/tests/tools/test_file_operations.py index 225b005cf..a66a81828 100644 --- a/tests/tools/test_file_operations.py +++ b/tests/tools/test_file_operations.py @@ -356,13 +356,6 @@ class TestShellFileOpsHelpers: assert "50|continued" in result assert "51|more" in result - def test_add_line_numbers_padded_env_override(self, file_ops, monkeypatch): - # Legacy fixed-width format available via HERMES_READ_GUTTER=padded. - monkeypatch.setenv("HERMES_READ_GUTTER", "padded") - result = file_ops._add_line_numbers("line one\nline two") - assert " 1|line one" in result - assert " 2|line two" in result - def test_add_line_numbers_truncates_long_lines(self, file_ops): long_line = "x" * (MAX_LINE_LENGTH + 100) result = file_ops._add_line_numbers(long_line) diff --git a/tools/file_operations.py b/tools/file_operations.py index 32d878de0..fd9f793ec 100644 --- a/tools/file_operations.py +++ b/tools/file_operations.py @@ -714,13 +714,9 @@ class ShellFileOperations(FileOperations): line-reference / patch / value-lookup / structure tasks (4/4 both), while dropping line numbers entirely regressed line-referencing (the model hand-counted and was off-by-one, 3/4) — so we keep the - numbers, just not the padding. ``HERMES_READ_GUTTER=padded`` - restores the legacy fixed-width format for anyone who relied on - column alignment. + numbers, just not the padding. """ - import os as _os from tools.tool_output_limits import get_max_line_length - padded = (_os.environ.get("HERMES_READ_GUTTER") or "").lower() == "padded" max_line_length = get_max_line_length() lines = content.split('\n') numbered = [] @@ -728,7 +724,7 @@ class ShellFileOperations(FileOperations): # Truncate long lines if len(line) > max_line_length: line = line[:max_line_length] + "... [truncated]" - numbered.append(f"{i:6d}|{line}" if padded else f"{i}|{line}") + numbered.append(f"{i}|{line}") return '\n'.join(numbered) def _expand_path(self, path: str) -> str: