Skip to content

Commit c587c47

Browse files
misrasaurabh1claude
andcommitted
fix: skip formatter check for Java projects
- Add Java case in _detect_formatter() that returns empty list - Change default formatter-cmds to empty list instead of black - This fixes "Could not find formatter: black" error for Java projects Java formatter support is not implemented yet, so we skip the check entirely for Java projects. Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
1 parent 512f9d5 commit c587c47

2 files changed

Lines changed: 7 additions & 1 deletion

File tree

codeflash/code_utils/config_parser.py

Lines changed: 3 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -146,7 +146,9 @@ def parse_config_file(
146146
"disable-imports-sorting": False,
147147
"benchmark": False,
148148
}
149-
list_str_keys = {"formatter-cmds": ["black $file"]}
149+
# Note: formatter-cmds defaults to empty list. For Python projects, black is typically
150+
# detected by the project detector. For Java projects, no formatter is supported yet.
151+
list_str_keys = {"formatter-cmds": []}
150152

151153
for key, default_value in str_keys.items():
152154
if key in config:

codeflash/setup/detector.py

Lines changed: 4 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -507,10 +507,14 @@ def _detect_formatter(project_root: Path, language: str) -> tuple[list[str], str
507507
508508
Python: ruff > black
509509
JavaScript: prettier > eslint --fix
510+
Java: not supported yet (returns empty)
510511
511512
"""
512513
if language in ("javascript", "typescript"):
513514
return _detect_js_formatter(project_root)
515+
if language == "java":
516+
# Java formatter support not implemented yet
517+
return [], "not supported for Java"
514518
return _detect_python_formatter(project_root)
515519

516520

0 commit comments

Comments
 (0)