improving sync when no pr is necessary

Author: DoomsCoder

.github/workflows/sync_translations.yml

@@ -9,7 +9,10 @@ concurrency:
 
 # We use a machine account PAT from secrets so workflows are triggered
 # the default token is not needed and should be fully restricted
-permissions: {}
+permissions:
+  contents: write
+  pull-requests: write
+  checks: write
 
 jobs:
   sync_translations:
@@ -19,7 +22,7 @@ jobs:
     steps:
       - uses: actions/checkout@v6
         with:
-          token: ${{ secrets.MACHINE_ACCOUNT_PAT }}
+          token: ${{ secrets.GITHUB_TOKEN }}
           ref: 'main'
           fetch-depth: 0
 
@@ -38,7 +41,7 @@ jobs:
 
       - name: Get second to latest run time
         env:
-          GH_TOKEN: ${{ secrets.MACHINE_ACCOUNT_PAT }}
+          GH_TOKEN: ${{ secrets.GITHUB_TOKEN }}
         run: |
           OUTPUT=$(gh run list --workflow sync_translations.yml --json updatedAt | jq -r '.[1].updatedAt')
           echo "LATEST_RUN=$OUTPUT" >> $GITHUB_ENV
@@ -50,7 +53,7 @@ jobs:
           script: |
             const lastRun = new Date(process.env.LATEST_RUN);
             const secondsPassed = Math.floor((Date.now() - lastRun.getTime()) / 1000);
-            const timeLeft = 1800 - secondsPassed > 0 ? 1800 - secondsPassed: 0;
+            const timeLeft = 10 - secondsPassed > 0 ? 10 - secondsPassed: 0;
             console.log('Seconds to sleep for: ' + timeLeft);
             core.setOutput('time_left', timeLeft);
 
@@ -69,49 +72,74 @@ jobs:
           yarn build
 
       - name: Push translation sources to crowdin
-        run: |
-          cd ./tools/localization
-          yarn start upload
+#        run: |
+#          cd ./tools/localization
+#          yarn start upload
+        run : echo "Skipping update for testing"
 
       - name: Pull translation updates from crowdin
-        run: |
-          cd ./tools/localization
-          yarn start download
+#        run: |
+#          cd ./tools/localization
+#          yarn start download
+        run : echo "Skipping update for testing"
 
       - name: Extract downloaded ankidroid.zip file
-        run: |
-          cd ./tools/localization
-          yarn start extract
+#        run: |
+#          cd ./tools/localization
+#          yarn start extract
+        run : echo "Skipping update for testing"
 
       - name: Update translation to AnkiDroid res
-        run: |
-          cd ./tools/localization
-          yarn start update
+#        run: |
+#          cd ./tools/localization
+#          yarn start update
+        run : echo "Skipping update for testing"
 
-      - name: Commit changes
-        run: |
-          git add docs/marketing/localized_description AnkiDroid/src/main/res
-          git commit -am 'Updated strings from Crowdin'
-          git push --set-upstream origin +i18n_sync
-          echo "The results of the sync are on the i18n_sync branch, PR them from there for merge."
-          echo "https://github.com/ankidroid/Anki-Android/compare/i18n_sync?expand=1"
-
-      - name: Check if there are strings changes
+      - name: Check for changes and handle Neutral result
         id: tr_check
-        run: |
-          git checkout i18n_sync
-          COMMITS_COUNT=`git rev-list --count HEAD ^main`
-          if [ "$COMMITS_COUNT" -gt 0 ]; then
-            echo "HAS_CHANGES=true" >> $GITHUB_OUTPUT
-          else
-            echo "HAS_CHANGES=false" >> $GITHUB_OUTPUT
-          fi
+        if: always()
+        uses: actions/github-script@v7
+        with:
+          script: |
+            const { execSync } = require('child_process');
+            
+            try {
+              execSync('git add docs/marketing/localized_description AnkiDroid/src/main/res');
+            } catch (e) {
+              console.log("No files found to step, continuing...")
+            }
+            
+            const status = execSync('git diff --cached --name-only').toString().trim();
+            
+            if(!status) {
+              console.log("No changes detected. Setting status to neutral.");
+              
+              await github.rest.checks.create({
+                owner: context.repo.owner,
+                repo: context.repo.repo,
+                name: 'Sync Translations with Crowdin',
+                head_sha: context.sha,
+                status: 'completed',
+                conclusion: 'neutral',
+                output: {
+                  title: 'Translation Sync',
+                  summary: 'No translation changes were found on Crowdin to sync'
+                  }
+                });
+              
+                core.setOutput('HAS_CHANGES', 'false');
+            } else {
+              core.setOutput('HAS_CHANGES', 'true');
+              execSync("git commit -m 'Updated strings from Crowdin'");
+              execSync("git push --set-upstream origin +i18n_sync");
+              console.log("Changes pushed to i18n_sync branch");
+            }
 
       - name: Create PR for strings changes if needed
-        if: ${{ steps.tr_check.outputs.HAS_CHANGES }}
+        if: steps.tr_check.outputs.HAS_CHANGES == 'true'
         uses: actions/github-script@v9
         with:
-          github-token: ${{ secrets.MACHINE_ACCOUNT_PAT }}
+          github-token: ${{ secrets.GITHUB_TOKEN }}
           script: |
             const now = new Date();
             // Date format used: YYYY/MM/DD HH:MM , UTC time(ex: 2023/01/13 08:38)