build(codegen): Pipe gen_bindings output through clang-format

Resolves the conflict between BindingCodegenDrift (which compared
committed files to raw generator output) and format-check (which
required clang-formatted source). The generator now formats each
emitted file via clang-format before write/compare, so both checks
agree on the same canonical form. Falls back to raw content if
clang-format is unavailable so stripped CI images can still run
--check mode.

Author: libraz

src/c_api/generated/sheet_counts.cpp

@@ -8,7 +8,6 @@
 // tools/codegen/binding_manifest.yaml. DO NOT EDIT BY HAND.
 // To change behaviour, edit the manifest and re-run the generator.
 
-
 #include <cstddef>
 #include <cstdint>
 #include <string>
@@ -26,8 +25,7 @@ using formulon::c_api::parts::set_binding_error;
 extern "C" fm_status_t fm_workbook_cell_count(const fm_workbook_t* wb, size_t sheet_index, size_t* out_count) {
   clear_last_error();
   if (wb == nullptr || out_count == nullptr) {
-    return set_binding_error(formulon::FormulonErrorCode::kBindingNullPointer,
-                             "fm_workbook_cell_count: NULL argument");
+    return set_binding_error(formulon::FormulonErrorCode::kBindingNullPointer, "fm_workbook_cell_count: NULL argument");
   }
   if (sheet_index >= wb->workbook().sheet_count()) {
     return set_binding_error(
@@ -52,4 +50,3 @@ extern "C" fm_status_t fm_workbook_pivot_count(const fm_workbook_t* wb, size_t s
   *out_count = wb->workbook().sheet(sheet_index).pivot_tables().size();
   return 0;
 }
-

src/c_api/generated/styles_counts.cpp

@@ -8,7 +8,6 @@
 // tools/codegen/binding_manifest.yaml. DO NOT EDIT BY HAND.
 // To change behaviour, edit the manifest and re-run the generator.
 
-
 #include <cstddef>
 #include <cstdint>
 #include <string>
@@ -62,4 +61,3 @@ extern "C" fm_status_t fm_styles_get_font_count(fm_workbook_t* wb, uint32_t* out
   *out_count = static_cast<uint32_t>(wb->workbook().styles().fonts.size());
   return 0;
 }
-

src/c_api/generated/workbook_counts.cpp

@@ -8,7 +8,6 @@
 // tools/codegen/binding_manifest.yaml. DO NOT EDIT BY HAND.
 // To change behaviour, edit the manifest and re-run the generator.
 
-
 #include <cstddef>
 #include <cstdint>
 #include <string>
@@ -50,4 +49,3 @@ extern "C" size_t fm_workbook_table_count(const fm_workbook_t* wb) {
   }
   return wb->workbook().tables().size();
 }
-

src/node_addon/generated/sheet_counts.cc

@@ -7,7 +7,6 @@
 // tools/codegen/binding_manifest.yaml. DO NOT EDIT BY HAND.
 // To change behaviour, edit the manifest and re-run the generator.
 
-
 #include <cstddef>
 #include <cstdint>
 

src/node_addon/generated/styles_counts.cc

@@ -7,7 +7,6 @@
 // tools/codegen/binding_manifest.yaml. DO NOT EDIT BY HAND.
 // To change behaviour, edit the manifest and re-run the generator.
 
-
 #include <cstddef>
 #include <cstdint>
 

src/node_addon/generated/workbook_counts.cc

@@ -7,7 +7,6 @@
 // tools/codegen/binding_manifest.yaml. DO NOT EDIT BY HAND.
 // To change behaviour, edit the manifest and re-run the generator.
 
-
 #include <cstddef>
 #include <cstdint>
 

src/wasm/generated/sheet_counts.cpp

@@ -7,7 +7,6 @@
 // tools/codegen/binding_manifest.yaml. DO NOT EDIT BY HAND.
 // To change behaviour, edit the manifest and re-run the generator.
 
-
 #include <cstddef>
 #include <cstdint>
 

src/wasm/generated/styles_counts.cpp

@@ -7,7 +7,6 @@
 // tools/codegen/binding_manifest.yaml. DO NOT EDIT BY HAND.
 // To change behaviour, edit the manifest and re-run the generator.
 
-
 #include <cstddef>
 #include <cstdint>
 

src/wasm/generated/workbook_counts.cpp

@@ -7,7 +7,6 @@
 // tools/codegen/binding_manifest.yaml. DO NOT EDIT BY HAND.
 // To change behaviour, edit the manifest and re-run the generator.
 
-
 #include <cstddef>
 #include <cstdint>
 

tools/codegen/gen_bindings.py

@@ -26,6 +26,8 @@
 
 import argparse
 import datetime as _dt
+import shutil
+import subprocess
 import sys
 from pathlib import Path
 from typing import Dict, List, Tuple
@@ -431,6 +433,34 @@ def _emit_node_entry(e: dict) -> str:
 # ---------------------------------------------------------------------------
 
 
+def _clang_format(content: str, path: Path) -> str:
+    """Pipe `content` through clang-format using `path`'s filename for style detection.
+
+    Ensures generated files agree with the project-wide format-check (which
+    runs `clang-format --dry-run --Werror` over `src/`). Falls back to the
+    raw content if clang-format is unavailable, so a stripped CI image can
+    still run the generator's --check mode against a previously-formatted
+    snapshot.
+    """
+    cf = shutil.which("clang-format")
+    if cf is None:
+        return content
+    proc = subprocess.run(
+        [cf, f"--assume-filename={path.name}"],
+        input=content,
+        capture_output=True,
+        text=True,
+        check=False,
+    )
+    if proc.returncode != 0:
+        sys.stderr.write(
+            f"clang-format failed on {path.name}: {proc.stderr}\n"
+            "falling back to raw codegen output\n"
+        )
+        return content
+    return proc.stdout
+
+
 def _write_if_changed(path: Path, content: str) -> bool:
     """Write `content` to `path` only if it differs. Returns True on write."""
     path.parent.mkdir(parents=True, exist_ok=True)
@@ -465,6 +495,7 @@ def main(argv: List[str] | None = None) -> int:
     drift = False
     for name, content in sorted(capi_files.items()):
         target = args.out_c_api / name
+        content = _clang_format(content, target)
         if args.check:
             if not target.exists() or target.read_text() != content:
                 sys.stderr.write(f"drift: {target}\n")
@@ -475,6 +506,7 @@ def main(argv: List[str] | None = None) -> int:
                 sys.stdout.write(f"wrote {target}\n")
     for name, content in sorted(embind_files.items()):
         target = args.out_embind / name
+        content = _clang_format(content, target)
         if args.check:
             if not target.exists() or target.read_text() != content:
                 sys.stderr.write(f"drift: {target}\n")
@@ -485,6 +517,7 @@ def main(argv: List[str] | None = None) -> int:
                 sys.stdout.write(f"wrote {target}\n")
     for name, content in sorted(node_files.items()):
         target = args.out_node / name
+        content = _clang_format(content, target)
         if args.check:
             if not target.exists() or target.read_text() != content:
                 sys.stderr.write(f"drift: {target}\n")