@claude review this PR.

Claude finished @kovtcharov's task —— View job

PR Review

• Read the changed files
• Review code quality and correctness
• Provide feedback

Summary

This PR improves the --fix mode in both util/lint.py and util/lint.ps1 to run only fixable checks (Black + isort) when used alone, making it faster and more focused.

Review Feedback

🟢 Good

1. Clean implementation - The fix-only mode detection is straightforward:

   • lint.py:548: fix_only = args.fix and not specific_checks
   • lint.ps1:413: $FixOnly = $Fix -and -not ($RunPylint -or ...)

2. Consistent behavior - Both Python and PowerShell scripts now behave identically

3. Helpful user feedback - Shows which files were actually fixed and provides clear next steps

4. Progress indicators updated - Shows [1/2] and [2/2] in fix mode vs [1/7], [2/7] in full mode

🟡 Suggestions (Minor)

1. lint.py:76 - The import re inside the function could be moved to the module level for consistency (though it's fine as-is for lazy loading)

2. lint.py:505-514 - The print_fix_summary() function could include counts of files fixed (already tracked in the results):

   def print_fix_summary(results: list[CheckResult]) -> None:
       # Could iterate results to show actual fix counts

3. lint.ps1:62-64 - The $cmd variable is constructed but only used for display. This is fine but could be extracted to a helper for DRY if more commands are added later.

🟢 Edge Cases Handled

• --fix --all runs all checks with fixing enabled ✓
• --fix --black runs only black with fixing enabled ✓
• --fix alone runs only Black + isort ✓

Verdict

Looks good to merge! The changes are well-implemented and achieve the stated goal of making --fix faster when used standalone.