Update idf-build.yml

Author: baba-dev

.github/workflows/idf-build.yml

@@ -7,39 +7,42 @@ on:
 jobs:
   build:
     runs-on: ubuntu-latest
+    permissions:
+      contents: read
+      actions: read
     steps:
-      - name: Checkout repository
+      - name: Checkout (no LFS, include submodules)
         uses: actions/checkout@v4
         with:
-          lfs: false            # if you track any assets with Git LFS
+          lfs: false
+          submodules: recursive
 
-      - name: Set up Python (for fetch script)
+      - name: Set up Python (for helper scripts)
         uses: actions/setup-python@v5
         with:
-          python-version: '3.11'
+          python-version: '3.x'
 
-      - name: Install Python deps (if any)
-        run: |
-          if [ -f requirements.txt ]; then pip install -r requirements.txt; fi
-
-      - name: Fetch external repos (matches your local step)
+      - name: Fetch external repos
         run: python ./fetch_repos.py
-        # runs in repo root, same as your local flow
 
-      - name: Build with ESP-IDF 5.4.2 for ESP32-P4
+      - name: Build with ESP-IDF 5.4.2 (esp32p4)
         uses: espressif/esp-idf-ci-action@v1
         with:
           esp_idf_version: v5.4.2
           target: esp32p4
           path: platforms/tab5
           command: idf.py build
-        # The CI action already sets up the IDF environment. :contentReference[oaicite:2]{index=2}
 
-      - name: Upload firmware artifacts
+      # Optional: show the auto-generated flash args for reference in logs
+      - name: Show flash args
+        if: always()
+        run: |
+          cat platforms/tab5/build/flash_args || true
+
+      - name: Upload build artifacts (entire build dir)
         uses: actions/upload-artifact@v4
         with:
-          name: tab5-firmware-${{ github.sha }}
+          name: tab5-build-${{ github.run_id }}
           path: |
-            platforms/tab5/build/**/*.bin
-            platforms/tab5/build/**/*.elf
-            platforms/tab5/build/**/*.map
+            platforms/tab5/build/**
+          retention-days: 7