Remove Prettier calls from Python scripts - formatting handled by pre-commit hook

Author: vikahaze

scripts/normalize_book_sets.py

@@ -13,7 +13,6 @@
 import json
 import os
 import argparse
-import subprocess
 from pathlib import Path
 
 
@@ -300,22 +299,10 @@ def normalize_book_sets(
             print("[DRY RUN] Would write changes to file (use --write to apply)")
         else:
             try:
-                # Write JSON file (prettier will format it)
+                # Write JSON file (prettier will format it via pre-commit hook)
                 with open(book_sets_file, "w", encoding="utf-8") as f:
                     json.dump(data, f, ensure_ascii=False)
 
-                # Format with prettier
-                try:
-                    subprocess.run(
-                        ["npx", "prettier", "--write", book_sets_file],
-                        check=True,
-                        capture_output=True,
-                    )
-                except subprocess.CalledProcessError as e:
-                    print(f"Warning: Prettier formatting failed: {e}")
-                except FileNotFoundError:
-                    print("Warning: npx/prettier not found, skipping formatting")
-
                 print("\n" + "=" * 70)
                 print(f"✓ Successfully updated '{book_sets_file}'")
             except Exception as e:

scripts/normalize_json.py

@@ -10,7 +10,6 @@
 
 import json
 import argparse
-import subprocess
 from collections import OrderedDict
 
 
@@ -33,20 +32,10 @@ def sort_json_by_numeric_keys(input_file, output_file):
         sorted_items = sorted(data.items(), key=lambda item: int(item[0]))
         sorted_data = OrderedDict(sorted_items)
 
-        # Write JSON (Prettier will format it)
+        # Write JSON (Prettier will format it via pre-commit hook)
         with open(output_file, "w", encoding="utf-8") as f:
             json.dump(sorted_data, f, ensure_ascii=False)
 
-        # Format with prettier
-        try:
-            subprocess.run(
-                ["npx", "prettier", "--write", output_file],
-                check=True,
-                capture_output=True,
-            )
-        except (subprocess.CalledProcessError, FileNotFoundError):
-            pass  # Silently fail if prettier is not available
-
         print(
             f"Successfully sorted '{input_file}' and saved the result to '{output_file}'."
         )

scripts/sort_book_sets_by_difficulty.py

@@ -17,7 +17,6 @@
 import os
 import sys
 import argparse
-import subprocess
 from pathlib import Path
 
 # Paths
@@ -184,22 +183,9 @@ def main():
         print(f"Dry run complete. Would update {updated_count} set(s).")
     else:
         if updated_count > 0:
-            # Save updated book-sets.json
+            # Save updated book-sets.json (Prettier will format it via pre-commit hook)
             with open(BOOK_SETS_PATH, "w", encoding="utf-8") as f:
                 json.dump(book_sets, f, ensure_ascii=False)
-            
-            # Format with prettier
-            try:
-                subprocess.run(
-                    ["npx", "prettier", "--write", str(BOOK_SETS_PATH)],
-                    check=True,
-                    capture_output=True,
-                )
-            except subprocess.CalledProcessError as e:
-                print(f"Warning: Prettier formatting failed: {e}")
-            except FileNotFoundError:
-                print("Warning: npx/prettier not found, skipping formatting")
-            
             print(f"✓ Updated {updated_count} book set(s) in {BOOK_SETS_PATH}")
         else:
             print("No changes needed. All sets are already sorted.")

scripts/update_problems_from_csv.py

@@ -156,20 +156,9 @@ def save_json(data):
         )
         sorted_data = OrderedDict(sorted_items)
 
+        # Write JSON (Prettier will format it via pre-commit hook)
         with open(JSON_PATH, "w", encoding="utf-8") as f:
             json.dump(sorted_data, f, ensure_ascii=False)
-        
-        # Format with prettier
-        import subprocess
-        try:
-            subprocess.run(
-                ["npx", "prettier", "--write", str(JSON_PATH)],
-                check=True,
-                capture_output=True,
-            )
-        except (subprocess.CalledProcessError, FileNotFoundError):
-            pass  # Silently fail if prettier is not available
-        
         return True
     except Exception as e:
         print(f"Error saving JSON: {e}")