ci: check pr changes before running

Author: zyrouge

.github/workflows/code-analysis.yml

@@ -17,10 +17,12 @@ on:
 
 permissions: {}
 
+env:
+    RUN_LINT: "yes"
+
 jobs:
     build:
         runs-on: ubuntu-latest
-        if: github.event_name != 'pull_request_review' || github.event.review.state == 'approved'
 
         steps:
             - uses: actions/checkout@v4
@@ -39,7 +41,14 @@ jobs:
             - name: 🚧 Do prerequisites
               run: npm ci
 
+            - name: 🔎 Check pull request changes
+              if: github.event_name == 'pull_request_target'
+              run: |
+                  git fetch origin "${{ github.base_ref }}" --depth=1
+                  echo "RUN_LINT=$(npm run git:diff-files-yn 'origin/${{ github.base_ref }}' 'app/' 'i18n/' 'gradle/')" >> $GITHUB_ENV
+
             - name: 🚨 Analyze code
+              if: env.RUN_LINT == 'no'
               run: |
                   npm run prebuild
                   chmod +x ./gradlew

.github/workflows/i18n-summary.yml

@@ -17,6 +17,9 @@ permissions:
     contents: write
     pull-requests: write
 
+env:
+    PR_COMMENT: "no"
+
 jobs:
     i18n-summary:
         runs-on: ubuntu-latest
@@ -44,8 +47,14 @@ jobs:
                   checkout-orphan: true
                   force: true
 
-            - name: 💬 Comment summary
+            - name: 🔎 Check pull request changes
               if: github.event_name == 'pull_request_target'
+              run: |
+                  git fetch origin "${{ github.base_ref }}" --depth=1
+                  echo "PR_COMMENT=$(npm run git:diff-files-yn 'origin/${{ github.base_ref }}' '.phrasey/' 'i18n/')" >> $GITHUB_ENV
+
+            - name: 💬 Comment summary
+              if: github.event_name == 'pull_request_target' && env.PR_COMMENT == 'yes'
               uses: actions/github-script@v7
               with:
                   script: |

cli/git/diff-files-yn.ts

@@ -0,0 +1,16 @@
+import { Git } from "../helpers/git";
+
+const main = async () => {
+    const [branch, ...pathPrefix] = process.argv.slice(2);
+    if (!branch) throw new Error("Missing argument: tag");
+    if (!pathPrefix.length) throw new Error("Missing argument: pathPrefix");
+    const files = await Git.diffNames(branch);
+    const affected = files.some((x) => affectsPathPrefix(pathPrefix, x));
+    console.log(!affected ? "yes" : "no");
+};
+
+main();
+
+function affectsPathPrefix(pathPrefix: string[], file: string) {
+    return pathPrefix.some((x) => file.startsWith(x));
+}

cli/helpers/git.ts

@@ -37,8 +37,8 @@ export class Git {
         return proc.status === 0;
     }
 
-    static async diffNames(tag: string) {
-        const proc = await Git.spawn(["diff", "--name-only", tag, "."]);
+    static async diffNames(branch: string) {
+        const proc = await Git.spawn(["diff", "--name-only", branch, "."]);
         if (proc.status !== 0) {
             throw new Error(`Unable to get diff (${proc.stderr})`);
         }

package.json

@@ -13,6 +13,7 @@
         "git:tag-exists": "tsx ./cli/git/tag-exists.ts",
         "git:tag-exists-yn": "tsx ./cli/git/tag-exists-yn.ts",
         "git:latest-tag": "tsx ./cli/git/latest-tag.ts",
+        "git:diff-files-yn": "tsx ./cli/git/diff-files-yn.ts",
         "android:move-outputs": "tsx ./cli/android/move-outputs.ts",
         "prebuild": "npm run i18n:build",
         "postbuild": "npm run android:move-outputs",