Refactor icon generation and usage

Replaces shell scripts with a unified Python script using Jinja2 templates for icon generation. Adds template files and generated icon lists for both Material and Cupertino icons in Dart and Python. Updates Flet core extension to use icon indices instead of codepoints, simplifying icon lookup and usage. Removes legacy generated icon files and updates build scripts to remove '--no-tree-shake-icons' flag.

Author: FeodorFitsner

.appveyor.yml

@@ -213,7 +213,7 @@ for:
 
       - cd client
       - set RELEASE_DIR=build\windows\x64\runner\Release
-      - flutter build windows --no-tree-shake-icons --build-name=%APPVEYOR_BUILD_VERSION%
+      - flutter build windows --build-name=%APPVEYOR_BUILD_VERSION%
 
       - copy "%WINDIR%\system32\msvcp140.dll" %RELEASE_DIR%
       - copy "%WINDIR%\system32\vcruntime140.dll" %RELEASE_DIR%
@@ -277,7 +277,7 @@ for:
     build_script:
       # Flutter macOS client
       - pushd client
-      - flutter build macos --no-tree-shake-icons --build-name=$APPVEYOR_BUILD_VERSION
+      - flutter build macos --build-name=$APPVEYOR_BUILD_VERSION
       - tar -czvf flet-macos.tar.gz -C build/macos/Build/Products/Release Flet.app
       - mkdir -p $flet_sdk_root/packages/flet-desktop/src/flet_desktop/app
       - cp flet-macos.tar.gz $flet_sdk_root/packages/flet-desktop/src/flet_desktop/app
@@ -357,7 +357,7 @@ for:
             patch_flet_desktop_package_name "flet-desktop-light"
             artifact_suffix="-light"
           fi
-      - flutter build linux --no-tree-shake-icons --build-name=$APPVEYOR_BUILD_VERSION
+      - flutter build linux --build-name=$APPVEYOR_BUILD_VERSION
       - mv build/linux/$build_arch/release/bundle build/linux/$build_arch/release/flet
       - tar -czvf flet-linux$artifact_suffix-$ARCH.tar.gz -C build/linux/$build_arch/release flet
       - mkdir -p $flet_sdk_root/packages/flet-desktop/src/flet_desktop/app
@@ -412,7 +412,7 @@ for:
     build_script:
       # Flutter Web client
       - pushd client
-      - flutter build web --wasm --no-tree-shake-icons
+      - flutter build web --wasm
       - cp -R build/web $flet_sdk_root/packages/flet-web/src/flet_web
       # fix on mobile Safari: https://github.com/flutter/flutter/issues/145111#issuecomment-2714599139
       - ls "$(dirname "$(command -v flutter)")/cache/flutter_web_sdk/flutter_js"

.ruff.toml

@@ -0,0 +1 @@
+exclude = ["ci/templates"]

ci/generate_cupertino_icons.sh

@@ -1,21 +1,26 @@
+# /// script
+# dependencies = [
+#   "requests",
+#   "Jinja2",
+# ]
+# ///
+
 import re
-import sys
+from pathlib import Path
 
 import requests
+from jinja2 import Environment, FileSystemLoader
 
 # Regex for parsing icon definitions (handles multi-line IconData)
-ICON_ENTRY_PATTERN = re.compile(
-    r"""
-    ^\s*///.*?"(?P<name>.+?)"(?:\s+\((?P<style>\w+)\))?.*?\n  # doc line
-    ^\s*static\s+const\s+IconData\s+(?P<var_name>\w+)\s*=\s*IconData\s*\(\s*  # var line
-    0x(?P<codepoint>[0-9a-fA-F]+),  # codepoint
-    """,
-    re.MULTILINE | re.VERBOSE,
+ICON_VAR_PATTERN = re.compile(
+    r"""^\s*static const IconData\s+(\w+)\s*=""", re.MULTILINE
 )
 
+file_loader = FileSystemLoader(Path(__file__).parent / "templates")
+templates = Environment(loader=file_loader)
 
-def normalize_enum_name(var_name: str) -> str:
-    return var_name.upper()
+# def normalize_enum_name(var_name: str) -> str:
+#     return var_name.upper()
 
 
 def download_dart_file(url: str) -> str:
@@ -26,45 +31,63 @@ def download_dart_file(url: str) -> str:
 
 
 def parse_dart_icons(dart_content: str, set_id: int):
+    # Extract and sort icon names alphabetically
+    icon_names = sorted(ICON_VAR_PATTERN.findall(dart_content))
+
     icons = []
-    for match in ICON_ENTRY_PATTERN.finditer(dart_content):
-        var_name = match.group("var_name")
-        codepoint = int(match.group("codepoint"), 16)
-        packed_value = (set_id << 24) | codepoint
-        icons.append((normalize_enum_name(var_name), hex(packed_value)))
-    print(f"🔍 Found {len(icons)} icons for set ID {set_id}.")
+    for i, icon_name in enumerate(icon_names):
+        packed_value = (set_id << 16) | i
+        icons.append((icon_name, packed_value))
+
+    print(f"🔍 Found {len(icons)} icons for set ID {set_id} (sorted).")
     return icons
 
 
-def generate_python_enum(icons, output_file: str, class_name: str):
-    with open(output_file, "w", encoding="utf-8") as f:
-        f.write(f"class {class_name}(IconData):\n")
-        for name, code in icons:
-            f.write(f"    {name} = {code}\n")
-    print(f"✅ Enum written to {output_file}")
+def generate_file(icons, template_name, output_file: str):
+    template = templates.get_template(template_name)
+    with open(
+        Path(__file__).parent.joinpath(output_file).resolve(), "w", encoding="utf-8"
+    ) as f:
+        f.write(template.render(icons=icons))
+    print(f"✅ File written to {output_file}")
 
 
 def main():
-    if len(sys.argv) < 5:
-        print(
-            "Usage: python generate_icons.py <https://path/to/Icons.dart> <output-file> <class-name> <set-id>"
-        )
-        sys.exit(1)
-
-    url = sys.argv[1]
-    output_file = sys.argv[2]
-    class_name = sys.argv[3]
-    set_id = int(sys.argv[4])
-
+    # material icons
+    url = "https://raw.githubusercontent.com/flutter/flutter/refs/heads/stable/packages/flutter/lib/src/material/icons.dart"
+    set_id = 1
     dart_content = download_dart_file(url)
-
     icons = parse_dart_icons(dart_content, set_id)
 
-    if not icons:
-        print("⚠️ No icons found. Please check the format.")
-        sys.exit(1)
+    generate_file(
+        icons,
+        "material_icons.dart",
+        "../packages/flet/lib/src/utils/material_icons.dart",
+    )
+
+    generate_file(
+        icons,
+        "material_icons.py",
+        "../sdk/python/packages/flet/src/flet/controls/material/icons.py",
+    )
+
+    # cupertino icons
+    url = "https://raw.githubusercontent.com/flutter/flutter/refs/heads/stable/packages/flutter/lib/src/cupertino/icons.dart"
+    set_id = 2
+    dart_content = download_dart_file(url)
+    icons = parse_dart_icons(dart_content, set_id)
 
-    generate_python_enum(icons, output_file, class_name)
+    generate_file(
+        icons,
+        "cupertino_icons.dart",
+        "../packages/flet/lib/src/utils/cupertino_icons.dart",
+    )
+
+    generate_file(
+        icons,
+        "cupertino_icons.py",
+        "../sdk/python/packages/flet/src/flet/controls/cupertino/cupertino_icons.py",
+    )
 
 
 if __name__ == "__main__":

ci/generate_icons.py

@@ -0,0 +1,7 @@
+import 'package:flutter/cupertino.dart';
+
+List<IconData> cupertinoIcons = [
+  {% for name, code in icons -%}
+  CupertinoIcons.{{ name }},
+  {% endfor -%}
+];

ci/generate_material_icons.sh

@@ -0,0 +1,19 @@
+"""
+Flet Cupertino Icons
+
+To generate/update this file run from the root of the repository:
+
+```
+uv run ci/generate_icons.py
+```
+"""
+
+from flet.controls.icon_data import IconData
+
+__all__ = ["CupertinoIcons"]
+
+
+class CupertinoIcons(IconData):
+    {% for name, code in icons -%}
+    {{ name.upper() }} = {{ "0x%X" % code }}
+    {% endfor -%}

ci/templates/cupertino_icons.dart

@@ -0,0 +1,7 @@
+import 'package:flutter/material.dart';
+
+List<IconData> materialIcons = [
+  {% for name, code in icons -%}
+  Icons.{{ name }},
+  {% endfor -%}
+];

ci/templates/cupertino_icons.py

@@ -0,0 +1,19 @@
+"""
+Flet Material Icons
+
+To generate/update this file run from the root of the repository:
+
+```
+uv run ci/generate_icons.py
+```
+"""
+
+from flet.controls.icon_data import IconData
+
+__all__ = ["Icons"]
+
+
+class Icons(IconData):
+    {% for name, code in icons -%}
+    {{ name.upper() }} = {{ "0x%X" % code }}
+    {% endfor -%}

ci/templates/material_icons.dart

@@ -2,6 +2,9 @@ library flet;
 
 export 'dart:io';
 
+export 'package:flutter/cupertino.dart' show CupertinoIcons;
+export 'package:flutter/material.dart' show Icons;
+
 export 'src/controls/base_controls.dart';
 export 'src/extensions/control.dart';
 export 'src/flet_app.dart';