improve binary file filtering
This commit is contained in:
parent
94d655ce91
commit
9c76745bcc
|
|
@ -39,7 +39,7 @@ _global_memory: Dict[
|
||||||
bool,
|
bool,
|
||||||
str,
|
str,
|
||||||
List[str],
|
List[str],
|
||||||
List[WorkLogEntry]
|
List[WorkLogEntry],
|
||||||
],
|
],
|
||||||
] = {
|
] = {
|
||||||
"research_notes": [],
|
"research_notes": [],
|
||||||
|
|
@ -462,7 +462,8 @@ def emit_related_files(files: List[str]) -> str:
|
||||||
|
|
||||||
# Return summary message
|
# Return summary message
|
||||||
if binary_files:
|
if binary_files:
|
||||||
return f"Files noted. {len(binary_files)} binary files were skipped."
|
binary_files_list = ", ".join(f"'{file}'" for file in binary_files)
|
||||||
|
return f"Files noted. Binary files skipped: {binary_files_list}"
|
||||||
else:
|
else:
|
||||||
return "Files noted."
|
return "Files noted."
|
||||||
|
|
||||||
|
|
|
||||||
|
|
@ -750,7 +750,9 @@ def test_emit_related_files_binary_filtering(reset_memory, tmp_path, monkeypatch
|
||||||
|
|
||||||
# Verify the result message mentions skipped binary files
|
# Verify the result message mentions skipped binary files
|
||||||
assert "Files noted." in result
|
assert "Files noted." in result
|
||||||
assert "2 binary files were skipped" in result
|
assert "Binary files skipped:" in result
|
||||||
|
assert f"'{binary_file1}'" in result
|
||||||
|
assert f"'{binary_file2}'" in result
|
||||||
|
|
||||||
# Verify only text files were added to related_files
|
# Verify only text files were added to related_files
|
||||||
assert len(_global_memory["related_files"]) == 2
|
assert len(_global_memory["related_files"]) == 2
|
||||||
|
|
|
||||||
Loading…
Reference in New Issue