Improve extension_api_parser and check it configs in the CI

Author: touilleMan

.github/workflows/build.yml

@@ -65,6 +65,8 @@ jobs:
           python --version
           pip install -U pip
           pip install -r requirements.in
+      - name: 'Sanity check on extension_api_parser'
+        run: python -m scripts.extension_api_parser
       - name: 'Setup project'
         run: python .github/scripts/meson_setup_or_dump_log.py build/
       - name: 'Build project'

scripts/extension_api_parser/__main__.py

@@ -2,10 +2,29 @@
 Helper to debug the parser
 """
 
+import os
 import sys
 from pathlib import Path
 
 from . import parse_extension_api_json, BuildConfig
 
 
-parse_extension_api_json(Path(sys.argv[1]), BuildConfig.DOUBLE_64)
+try:
+    extension_api_path = Path(sys.argv[1])
+except IndexError:
+    extension_api_path = (
+        (Path(__file__).parent / "../../godot_headers/extension_api.json")
+        .resolve()
+        .relative_to(os.getcwd())
+    )
+
+
+try:
+    build_configs = [BuildConfig(sys.argv[2])]
+except IndexError:
+    build_configs = BuildConfig
+
+
+for build_config in build_configs:
+    print(f"Checking {extension_api_path} with config {build_config.value}")
+    parse_extension_api_json(extension_api_path, build_config)

scripts/extension_api_parser/api.py

@@ -211,14 +211,18 @@ def merge_builtins_size_info(api_json: dict, build_config: BuildConfig) -> None:
     for item in api_json["builtin_classes"]:
         name = item["name"]
         item["size"] = builtin_class_sizes[name]
+        # TODO: correct me once https://github.com/godotengine/godot/pull/64365 is merged
         for member in builtin_class_member_offsets.get(name, ()):
             for item_member in item["members"]:
                 if item_member["name"] == member["member"]:
                     item_member["offset"] = member["offset"]
                     # Float builtin in extension_api.json is always 64bits long,
                     # however builtins made of floating point number can be made of
                     # 32bits (C float) or 64bits (C double)
-                    if item_member["type"] == "float":
+                    # But Color is a special case: it is always made of 32bits floats !
+                    if name == "Color":
+                        item_member["type"] = "meta:float"
+                    elif item_member["type"] == "float":
                         if build_config in (BuildConfig.FLOAT_32, BuildConfig.FLOAT_64):
                             item_member["type"] = "meta:float"
                         else: