format_generated_code was called without the language parameter in
process_review(), defaulting to "python". This created temp files with
.py extension when formatting JS/TS code, causing prettier to fail.
Trace IDs: 11e9745d, 1578f081, 7e8abab2 (73 affected logs total)
Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
Fixed two test failures in omni-java:
1. test_formatter_cmds_non_existent:
- Default formatter-cmds changed from ["black $file"] to [] (commit c587c475)
- Updated test expectation to match new default
- Formatter detection now handled by project detector
- Empty list prevents "Could not find formatter: black" errors for Java projects
2. test_float_values_slightly_different:
- Python comparator now uses math.isclose(rel_tol=1e-9) for numeric comparison (commit 98a5a438)
- Updated test to expect equivalent=True for values within epsilon tolerance
- Added test_float_values_significantly_different to verify detection of actual differences
- Test added before epsilon-based comparison was implemented, causing mismatch
Both tests now pass and accurately reflect current codebase behavior.
Test results: 2 fixed tests passing
Co-Authored-By: Claude Sonnet 4.5 <noreply@anthropic.com>
* formatting failure msg
* fix
* make formatter optional in the optimizer only to not break other logic & fix unit tests
* small fix
* fix lint issue
---------
Co-authored-by: saga4 <saga4@codeflashs-MacBook-Air.local>
Co-authored-by: ali <mohammed18200118@gmail.com>
* check large diffs with black, and skipp formatting in such case (after optimizing)
* new line
* better log messages
* remove unnecessary check
* new line
* remove unused comment
* the max lines for formatting changes to 100
* refactoring
* refactoring and improvements
* added black as dev dependency
* made some refactor changes that codeflash suggested
* remove unused function
* formatting & using internal black dep
* fix black import issue
* handle formatting files with no formatting issues
* use user pre-defined formatting commands, instead of using black
* make sure format_code recieves file path as path type not as str
* formatting and linting
* typo
* revert lock file changes
* remove comment
* pass helper functions source code to the formatter for diff checking
* more unit tests
* enhancements
* Update formatter.py
add a todo comment
* Update formatter.py
Fix ruff lint
---------
Co-authored-by: Sarthak Agarwal <sarthak.saga@gmail.com>