From e526b38adb5a82266a64ac248928c38250042b7c Mon Sep 17 00:00:00 2001
From: sleynsol <sleyn_sol@protonmail.com>
Date: Mon, 14 Sep 2026 19:18:06 +0200
Subject: [PATCH] fix: review mobile-harness mirror updates before publication

---
 .github/workflows/sync-mobile-harness.yml | 55 ++++++++++++-----------
 README.md                                 | 11 +++++
 2 files changed, 41 insertions(+), 25 deletions(-)

diff --git a/.github/workflows/sync-mobile-harness.yml b/.github/workflows/sync-mobile-harness.yml
index ffcddf8..287f13c 100644
--- a/.github/workflows/sync-mobile-harness.yml
+++ b/.github/workflows/sync-mobile-harness.yml
@@ -6,19 +6,30 @@ on:
   workflow_dispatch:
 
 permissions:
-  contents: write
-  actions: write
+  contents: read
+
+concurrency:
+  group: mobile-harness-sync-proposal
+  cancel-in-progress: false
 
 jobs:
   sync:
     runs-on: ubuntu-latest
     steps:
+      - name: Check automation credential
+        env:
+          AUTOMATION_TOKEN: ${{ secrets.AUTOMATION_PR_TOKEN }}
+        run: test -n "$AUTOMATION_TOKEN"
       - uses: actions/checkout@v4
         with:
           ref: master
+          persist-credentials: false
 
       - name: Clone source
-        run: git clone --depth 1 https://github.com/droidrun/mobile-harness.git /tmp/mobile-harness
+        id: source
+        run: |
+          git clone --depth 1 --branch main https://github.com/droidrun/mobile-harness.git "$RUNNER_TEMP/mobile-harness"
+          echo "sha=$(git -C "$RUNNER_TEMP/mobile-harness" rev-parse HEAD)" >> "$GITHUB_OUTPUT"
 
       - name: Mirror harness files
         run: |
@@ -27,26 +38,20 @@ jobs:
             --exclude '/.github' \
             --exclude '/README.md' \
             --exclude '/assets' \
-            /tmp/mobile-harness/ mobile-harness/
+            "$RUNNER_TEMP/mobile-harness/" mobile-harness/
 
-      - name: Commit and push
-        id: commit
-        run: |
-          git config user.name "github-actions[bot]"
-          git config user.email "41898282+github-actions[bot]@users.noreply.github.com"
-          git add mobile-harness
-          if git diff --cached --quiet; then
-            echo "mobile-harness is up to date"
-            echo "pushed=false" >> "$GITHUB_OUTPUT"
-            exit 0
-          fi
-          git commit -m "chore: sync mobile-harness from droidrun/mobile-harness"
-          git push
-          echo "pushed=true" >> "$GITHUB_OUTPUT"
-
-      # Pushes made with GITHUB_TOKEN do not trigger push workflows.
-      - name: Trigger packaging
-        if: steps.commit.outputs.pushed == 'true'
-        run: gh workflow run package-skills.yml --ref master --repo "$GITHUB_REPOSITORY"
-        env:
-          GH_TOKEN: ${{ github.token }}
+      - name: Propose the mirrored changes for review
+        uses: peter-evans/create-pull-request@5f6978faf089d4d20b00c7766989d076bb2fc7f1 # v8
+        with:
+          token: ${{ secrets.AUTOMATION_PR_TOKEN }}
+          base: master
+          branch: automation/mobile-harness-sync
+          delete-branch: true
+          add-paths: mobile-harness
+          title: 'chore: review mobile-harness sync'
+          commit-message: 'chore: sync mobile-harness ${{ steps.source.outputs.sha }}'
+          body: |
+            Mirror https://github.com/droidrun/mobile-harness/commit/${{ steps.source.outputs.sha }}.
+
+            Review this diff before merging. Packaging and publication run on the
+            resulting push to master, after the PR is merged by a maintainer.
diff --git a/README.md b/README.md
index b32be78..821833b 100644
--- a/README.md
+++ b/README.md
@@ -66,3 +66,14 @@ The `.skill` zip files are built automatically on every push to `master` and att
 │   ├── AGENTS.md
 │   └── ...
 ```
+
+## Reviewing automated updates
+
+The mobile-harness sync opens or updates a PR from
+`automation/mobile-harness-sync` to `master`. The proposal records the upstream
+commit and does not publish changes directly. Review and merge the PR to trigger
+the existing packaging/release workflow. The sync never approves or merges its own changes.
+
+Configure `AUTOMATION_PR_TOKEN` with Contents and Pull requests read/write for
+this repository. The organization policy blocking PR creation with the default
+`GITHUB_TOKEN` remains unchanged.