fix material icons automation (#4761)

Agent-Logs-Url: https://github.com/codenameone/CodenameOne/sessions/09ed0e92-f319-4f2e-8fbc-ff44ed2c1a9e

Co-authored-by: copilot-swe-agent[bot] <198982749+Copilot@users.noreply.github.com>
Co-authored-by: shai-almog <67850168+shai-almog@users.noreply.github.com>

Author: Copilot

.github/scripts/update_material_icons.py

@@ -22,7 +22,10 @@
 )
 MATERIAL_CSS_URL = "https://fonts.googleapis.com/icon?family=Material+Icons"
 
-CONSTANT_DOC = "    /// Material design icon font character code see\\n    /// https://www.material.io/resources/icons/ for full list\\n"
+CONSTANT_DOC = (
+    "    /// Material design icon font character code see\n"
+    "    /// https://www.material.io/resources/icons/ for full list\n"
+)
 CONSTANT_RE = re.compile(r"^\s*public static final char MATERIAL_")
 SENTINEL = "    private static Font materialDesignFont;"
 
@@ -67,6 +70,10 @@ def generate_constants_block(entries: list[tuple[str, str]]) -> str:
     return "\n".join(lines) + "\n"
 
 
+def is_material_constant_doc_line(line: str) -> bool:
+    return line.lstrip().startswith("///")
+
+
 def update_fontimage(constants_block: str, check_only: bool) -> bool:
     source = FONT_IMAGE_PATH.read_text(encoding="utf-8")
     lines = source.splitlines(keepends=True)
@@ -76,6 +83,8 @@ def update_fontimage(constants_block: str, check_only: bool) -> bool:
     for i, line in enumerate(lines):
         if start_idx is None and CONSTANT_RE.match(line):
             start_idx = i
+            while start_idx > 0 and is_material_constant_doc_line(lines[start_idx - 1]):
+                start_idx -= 1
         if line.strip("\n") == SENTINEL:
             end_idx = i
             break

.github/workflows/material-icons-update.yml

@@ -2,6 +2,7 @@ name: Material Icons Update
 
 on:
   schedule:
+    # Weekly on Mondays at 06:00 UTC.
     - cron: '0 6 * * 1'
   workflow_dispatch:
   pull_request:
@@ -10,6 +11,7 @@ on:
       - '.github/scripts/update_material_icons.py'
 
 permissions:
+  actions: write
   contents: write
   pull-requests: write
 
@@ -32,6 +34,7 @@ jobs:
 
       - name: Create pull request
         if: github.event_name != 'pull_request'
+        id: create-pull-request
         uses: peter-evans/create-pull-request@v7
         with:
           commit-message: "Update Material Icons font and FontImage constants"
@@ -49,6 +52,18 @@ jobs:
             dependencies
             automation
 
+      - name: Trigger PR CI for generated branch
+        if: github.event_name != 'pull_request' && steps.create-pull-request.outputs.pull-request-number != ''
+        uses: actions/github-script@v7
+        with:
+          script: |
+            await github.rest.actions.createWorkflowDispatch({
+              owner: context.repo.owner,
+              repo: context.repo.repo,
+              workflow_id: 'pr.yml',
+              ref: 'automation/material-icons-update'
+            });
+
       - name: Build update preview for PR validation
         if: github.event_name == 'pull_request'
         run: |

.github/workflows/pr.yml

@@ -1,6 +1,7 @@
 name: PR CI
 
 on:
+  workflow_dispatch:
   pull_request:
     branches:
       - master