refactor: move cli sync commit logic from script to gha

Signed-off-by: David Karlsson <[email protected]>

Author: dvdksn

.github/workflows/sync-cli-docs.yml

@@ -51,28 +51,10 @@ jobs:
           path: cli-source
           ref: ${{ steps.get-version.outputs.version }}
           fetch-depth: 0
-      -
-        name: Git config
-        run: |
-          git config user.name "github-actions[bot]"
-          git config user.email "41898282+github-actions[bot]@users.noreply.github.com"
       -
         name: Run sync script
         id: sync
-        run: |
-          set +e
-          ./hack/sync-cli-docs.sh HEAD cli-source
-          EXIT_CODE=$?
-          set -e
-
-          if [ $EXIT_CODE -eq 0 ]; then
-            echo "Changes detected - syncing CLI docs" >> "$GITHUB_STEP_SUMMARY"
-          elif [ $EXIT_CODE -eq 100 ]; then
-            echo "No changes to sync - CLI docs are up to date" >> "$GITHUB_STEP_SUMMARY"
-          else
-            echo "::error::Script failed with exit code $EXIT_CODE"
-            exit $EXIT_CODE
-          fi
+        run: ./hack/sync-cli-docs.sh HEAD cli-source
       -
         name: Create Pull Request
         id: cpr
@@ -84,13 +66,15 @@ jobs:
           delete-branch: true
           add-paths: |
             data/engine-cli/*.yaml
+          commit-message: "cli: sync docs with docker/cli ${{ steps.sync.outputs.git_ref }}"
           title: "cli: sync docs with docker/cli"
           body: |
             ## Summary
 
             Automated sync of CLI documentation from docker/cli repository.
 
             **CLI version**: ${{ steps.get-version.outputs.version }}
+            **Git ref**: ${{ steps.sync.outputs.git_ref }}
           reviewers: "@engine"
       -
         name: Check outputs

hack/sync-cli-docs.sh

@@ -17,21 +17,10 @@ main() {
     (set -e; cd "$worktree_dir"; make -f docker.Makefile yamldocs || { printf "::error::Failed to generate YAML docs!\n"; exit 1; }) || return $?
     cp "$worktree_dir"/docs/yaml/*.yaml ./data/engine-cli/
 
-    if git diff --quiet "./data/engine-cli/*.yaml"; then
-        printf "\e[32m✅ Already up to date\e[0m\n"
-        return 100
-    fi
-
-    echo -e "ℹ️ Changes detected:"
-    git diff --stat "./data/engine-cli/*.yaml" || true
-
     NICE_GIT_REF=$(cd "$worktree_dir" && git describe --always --dirty) || return $?
+    echo "git_ref=$NICE_GIT_REF" >> "${GITHUB_OUTPUT:-/dev/stdout}"
 
-    git add "./data/engine-cli/*.yaml"
-
-    git commit -m "cli: sync docs with docker/cli $NICE_GIT_REF"
-    
-    printf "\e[32m✅ Committed changes\e[0m\n"
+    printf "\e[32m✅ Synced CLI docs from %s\e[0m\n" "$NICE_GIT_REF"
     return 0
 }