From f8fd4f31f6031b4e68ea2cfd51b10c6a83b8a932 Mon Sep 17 00:00:00 2001
From: ChinYikMing <yikming2222@gmail.com>
Date: Sat, 19 Jul 2025 00:25:23 +0800
Subject: [PATCH] CI: Push wasm commit on changes only
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

When rerunning the deploy wasm CI, previously successful jobs may fail
due to a reproducible error:
nothing to commit, working tree clean
Error: Process completed with exit code 1.

This occurs because the successful deployment has already committed the
changes. To prevent this, the workflow should check whether there are
any staged changes before attempting to commit. If there’s nothing to
commit, skip the push step. Otherwise, proceed with committing and
pushing the new changes.
---
 .github/workflows/deploy-wasm.yml | 24 ++++++++++++++++--------
 1 file changed, 16 insertions(+), 8 deletions(-)

diff --git a/.github/workflows/deploy-wasm.yml b/.github/workflows/deploy-wasm.yml
index 88920af4..1a228f68 100644
--- a/.github/workflows/deploy-wasm.yml
+++ b/.github/workflows/deploy-wasm.yml
@@ -92,6 +92,7 @@ jobs:
             mv /tmp/rv32emu-system-demo/rv32emu.wasm ./system
             mv /tmp/rv32emu-system-demo/rv32emu.worker.js ./system
       - name: Commit files
+        id: commit
         if: ${{ steps.changed-files.outputs.any_modified == 'true' ||
             github.event_name == 'workflow_dispatch' ||
             (github.event_name == 'repository_dispatch' && github.event.action == 'deploy_system_wasm') }}
@@ -99,11 +100,14 @@ jobs:
             git config --local user.email "github-actions[bot]@users.noreply.github.com"
             git config --local user.name "github-actions[bot]"
             git add system/
-            git commit -m "Add changes to system emulation"
+            if git diff --cached --quiet; then
+              echo "committed=false" >> $GITHUB_OUTPUT
+            else
+              git commit -m "Add changes to system emulation"
+              echo "committed=true" >> $GITHUB_OUTPUT
+            fi
       - name: Push changes
-        if: ${{ steps.changed-files.outputs.any_modified == 'true' ||
-            github.event_name == 'workflow_dispatch' ||
-            (github.event_name == 'repository_dispatch' && github.event.action == 'deploy_system_wasm') }}
+        if: steps.commit.outputs.committed == 'true'
         uses: ad-m/github-push-action@master
         with:
           repository: sysprog21/rv32emu-demo
@@ -184,6 +188,7 @@ jobs:
             mv /tmp/rv32emu-demo/rv32emu.wasm .
             mv /tmp/rv32emu-demo/rv32emu.worker.js .
       - name: Commit files
+        id: commit
         if: ${{ steps.changed-files.outputs.any_modified == 'true' ||
             github.event_name == 'workflow_dispatch' ||
             (github.event_name == 'repository_dispatch' && github.event.action == 'deploy_user_wasm') }}
@@ -191,11 +196,14 @@ jobs:
             git config --local user.email "github-actions[bot]@users.noreply.github.com"
             git config --local user.name "github-actions[bot]"
             git add --all
-            git commit -m "Add changes to user emulation"
+            if git diff --cached --quiet; then
+              echo "committed=false" >> $GITHUB_OUTPUT
+            else
+              git commit -m "Add changes to user emulation"
+              echo "committed=true" >> $GITHUB_OUTPUT
+            fi
       - name: Push changes
-        if: ${{ steps.changed-files.outputs.any_modified == 'true' ||
-            github.event_name == 'workflow_dispatch' ||
-            (github.event_name == 'repository_dispatch' && github.event.action == 'deploy_user_wasm') }}
+        if: steps.commit.outputs.committed == 'true'
         uses: ad-m/github-push-action@master
         with:
           repository: sysprog21/rv32emu-demo