From cc59bc9347f010dcec52346d0c08d40ed7aace84 Mon Sep 17 00:00:00 2001
From: Nikhil Badyal <59223300+nikhilbadyal@users.noreply.github.com>
Date: Wed, 17 Apr 2024 22:29:58 +0530
Subject: [PATCH] Filter to be patched (#505)

---
 .github/workflows/auto-release.yml | 9 +++------
 .github/workflows/build-apk.yml    | 4 ++++
 check_resource_updates.py          | 9 ++++++---
 3 files changed, 13 insertions(+), 9 deletions(-)

diff --git a/.github/workflows/auto-release.yml b/.github/workflows/auto-release.yml
index 6569de3..bba0863 100644
--- a/.github/workflows/auto-release.yml
+++ b/.github/workflows/auto-release.yml
@@ -50,11 +50,7 @@ jobs:
           GITHUB_TOKEN: ${{ secrets.GITHUB_TOKEN }}
         run: |
           should_build=$(python check_resource_updates.py)
-          if [ "$should_build" = "True" ]; then
-              echo "SHOULD_BUILD=1" >> $GITHUB_OUTPUT
-          else
-              echo "SHOULD_BUILD=0" >> $GITHUB_OUTPUT
-          fi
+          echo "SHOULD_BUILD=$should_build" >> $GITHUB_OUTPUT
     outputs:
       SHOULD_BUILD: ${{ steps.should_build.outputs.SHOULD_BUILD }}
 
@@ -62,7 +58,7 @@ jobs:
     permissions: write-all
     needs: release-check
     uses: ./.github/workflows/build-apk.yml
-    if: ${{ needs.release-check.outputs.SHOULD_BUILD == 1 }}
+    if: ${{ needs.release-check.outputs.SHOULD_BUILD }}
     secrets: inherit
     concurrency:
       group: Auto-Release-${{ github.head_ref || github.run_id }}
@@ -70,3 +66,4 @@ jobs:
     with:
       TELEGRAM_NO_ROOT_UPLOAD: true
       TELEGRAM_ROOT_UPLOAD: true
+      PREFERRED_PATCH_APPS: ${{ needs.release-check.outputs.SHOULD_BUILD }}
diff --git a/.github/workflows/build-apk.yml b/.github/workflows/build-apk.yml
index 30b73de..c16543b 100644
--- a/.github/workflows/build-apk.yml
+++ b/.github/workflows/build-apk.yml
@@ -46,6 +46,10 @@ on:
         description: 'Run the build with tmate debugging enabled.'
         required: false
         default: false
+      PREFERRED_PATCH_APPS:
+        description: "Apps to be patched. Overrides any env set"
+        required: false
+        type: string
   workflow_dispatch:
     inputs:
       GITHUB_UPLOAD:
diff --git a/check_resource_updates.py b/check_resource_updates.py
index 54dc40a..0aeb146 100644
--- a/check_resource_updates.py
+++ b/check_resource_updates.py
@@ -14,6 +14,7 @@ def check_if_build_is_required() -> bool:
     env = Env()
     env.read_env()
     config = RevancedConfig(env)
+    needs_to_repatched = []
     for app_name in env.list("PATCH_APPS", default_build):
         logger.info(f"Checking {app_name}")
         app_obj = get_app(config, app_name)
@@ -39,9 +40,11 @@ def check_if_build_is_required() -> bool:
                 },
             }
             logger.info(f"New build can be triggered caused by {caused_by}")
-            print(True)  # noqa: FBT003,T201
-            return True
-    print(False)  # noqa: FBT003,T201
+            needs_to_repatched.append(app_name)
+    logger.info(f"{needs_to_repatched} are need to repatched.")
+    if len(needs_to_repatched) > 0:
+        print(",".join(needs_to_repatched))  # noqa: T201
+        return True
     return False