.github/workflows/build_pull_request.yml

@@ -3,18 +3,42 @@ name: Build pull request
 on:
   workflow_dispatch:
     inputs:
+      # Select pull request
       pr-number:
-        description: PR number
+        description: PR number (Without hashtag)
         required: true
+      # Select app flavor
       app-flavor:
         description: App flavor
-        default: release
+        default: 'release'
         type: choice
         options:
           - release
           - debug
           - profile
 
+      # Flutter Configurations,
+      # it's recommended to be set when you have problem regarding with flutter itself
+      # For most part you do not need to change this.
+
+      # Flutter version to use, note that the version had to exist in whether channel
+      # to grab
+      # Try using exact version or particular version on a specific branch instead of "any"
+      flutter-channel:
+        description: Flutter channel
+        default: 'stable'
+        type: choice
+        options:
+          - stable
+          - beta
+          - dev
+          - any
+      flutter-version:
+        description: Flutter version
+        default: '3.29.x'
+
+run-name: "Build pull request ${{ inputs.pr-number }}"
+
 jobs:
   build:
     name: Build
@@ -38,27 +62,76 @@ jobs:
       - name: Set up Flutter
         uses: subosito/flutter-action@v2
         with:
-          channel: stable
+          channel: ${{ inputs.flutter-channel }}
+          flutter-version: ${{ inputs.flutter-version }}
 
       - name: Get dependencies
+        continue-on-error: true
         run: flutter pub get
 
       - name: Generate translations
+        continue-on-error: true
         run: dart run slang
 
       - name: Generate code files
+        continue-on-error: true
         run: dart run build_runner build --delete-conflicting-outputs
 
       - name: Build
+        continue-on-error: true
         id: flutter-build
         run: flutter build apk --${{ inputs.app-flavor }}
         env:
           GITHUB_TOKEN: ${{ secrets.GITHUB_TOKEN }}
 
-      - name: Upload artifacts
+      - name: Prepare comment
+        id: prepare-comment # This should work now?
+        run: |
+          echo "COMMIT_HASH=$(git rev-parse --short HEAD)" >> $GITHUB_ENV
+          if [[ "${{ steps.flutter-build.outcome }}" == "success" ]]; then
+            MESSAGE="✅ Succeeded build on $COMMIT_HASH."
+          else
+            MESSAGE="🚫 Failed build on $COMMIT_HASH."
+          fi
+
+      - name: "Comment on pull request #${{ inputs.pr-number }}"
+        uses: thollander/actions-comment-pull-request@v3
+        with:
+          github-token: ${{ github.token }}
+          pr-number: ${{ inputs.pr-number }}
+          mode: recreate
+          comment-tag: execution
+          message: |
+            ## ⚒️ Build status
+
+            🧪 Workflow triggered by: ${{ github.actor }}
+
+            ${{ steps.prepare-comment.outputs.MESSAGE }}
+
+            Details: [_Job execution **${{ github.run_id }}** / attempt **${{ github.run_attempt }}**_](https://github.com/${{ github.repository }}/actions/runs/${{ github.run_id }})!
+
+            ### ⚙️ Workflow Steps
+
+            | Step                      | Status                                                   |
+            | :------------------------ | :------------------------------------------------------- |
+            | **Get dependencies**      | ${{ steps.get-dependencies.outcome || job.status }}      |
+            | **Generate translations** | ${{ steps.generate-translations.outcome || job.status }} |
+            | **Generate code files**   | ${{ steps.generate-code-files.outcome || job.status }}   |
+            | **Build**                 | ${{ steps.flutter-build.outcome }}                       |
+
+            ### ⚙️ Workflow Configuration
+
+            | Parameter        | Value                                    |
+            | :--------------- | :--------------------------------------- |
+            | App flavor       | ${{ inputs.app-flavor }}                 |
+            | Flutter version  | ${{ inputs.flutter-version }}            |
+            | Flutter channel  | ${{ inputs.flutter-channel }}            |
+
+      - name: Upload Artifact
         if: steps.flutter-build.outcome == 'success'
         uses: actions/upload-artifact@v4
         with:
-          name: revanced-manager-(${{ env.COMMIT_HASH }}
+          name: revanced-manager-(${{ env.COMMIT_HASH }}-${{ inputs.pr-number }}-${{ inputs.app-flavor }}-${{ inputs.flutter-version }})
           path: |
-            build/app/outputs/flutter-apk/app-*.apk
+            build/app/outputs/flutter-apk/app-${{ inputs.app-flavor }}.apk
+            build/app/outputs/flutter-apk/app-${{ inputs.app-flavor }}.apk.sha1

.github/workflows/open_pull_request.yml

@@ -12,8 +12,6 @@ env:
 jobs:
   pull-request:
     name: Open pull request
-    permissions:
-      pull-requests: write
     runs-on: ubuntu-latest
     steps:
       - name: Checkout
@@ -27,3 +25,4 @@ jobs:
           pr_body: |
             This pull request will ${{ env.MESSAGE }}.
           pr_draft: true
+          github_token: ${{ secrets.REPOSITORY_PUSH_ACCESS }}

.github/workflows/release.yml

@@ -6,13 +6,19 @@ on:
     branches:
       - main
       - dev
+    paths:
+      - ".github/workflows/release.yml"
+      - "android/**"
+      - "assets/**"
+      - "lib/**"
+      - "pubspec.yaml"
 
 jobs:
   release:
     name: Release
     permissions:
-      contents: write
       id-token: write
+      contents: write
       attestations: write
     runs-on: ubuntu-latest
     steps:
@@ -22,10 +28,7 @@ jobs:
           fetch-depth: 0
 
       - name: Setup Java
-        uses: actions/setup-java@v4
+        run: echo "JAVA_HOME=$JAVA_HOME_17_X64" >> $GITHUB_ENV
-        with:
-          distribution: 'temurin'
-          java-version: '17'
 
       - name: Setup Node.js
         uses: actions/setup-node@v4
@@ -37,12 +40,6 @@ jobs:
         uses: subosito/flutter-action@v2
         with:
           channel: stable
-          cache: true
-
-      - name: Cache Gradle
-        uses: burrunan/gradle-cache-action@v1
-        with:
-          home-directory: ${{ github.workspace }}/android
 
       - name: Install dependencies
         run: npm ci
@@ -60,17 +57,17 @@ jobs:
         run: |
           echo "${{ secrets.KEYSTORE }}" | base64 --decode > "android/app/keystore.jks"
 
-      - name: Semantic Release
+      - name: Release
-        uses: cycjimmy/semantic-release-action@v4
-        id: semantic
         env:
           GITHUB_TOKEN: ${{ secrets.GITHUB_TOKEN }}
           KEYSTORE_PASSWORD: ${{ secrets.KEYSTORE_PASSWORD }}
           KEYSTORE_ENTRY_ALIAS: ${{ secrets.KEYSTORE_ENTRY_ALIAS }}
           KEYSTORE_ENTRY_PASSWORD: ${{ secrets.KEYSTORE_ENTRY_PASSWORD }}
+        run: |
+          npx semantic-release
           
-      - name: Attest
+      - name: Generate artifact attestation
-        if: steps.semantic.outputs.new_release_published == 'true'
+        if: github.ref == 'refs/heads/main'
-        uses: actions/attest-build-provenance@v2
+        uses: actions/attest-build-provenance@v1
         with:
           subject-path: build/app/outputs/apk/release/revanced-manager-*.apk

.gitignore

@@ -43,7 +43,6 @@ app.*.map.json
 /android/app/release
 
 # Generated files
-android/app/.cxx
 **/*.g.dart
 **/*.locator.dart
 **/*.router.dart

analysis_options.yaml

@@ -15,7 +15,6 @@ analyzer:
     - lib/app/app.router.dart
     - lib/models/patch.g.dart
     - lib/models/patched_application.g.dart
-    - lib/gen/
 
 linter:
   rules:

android/app/proguard-rules.pro

@@ -12,6 +12,3 @@
 -dontwarn com.google.j2objc.annotations.*
 -dontwarn java.awt.**
 -dontwarn javax.**
-
-# Keep this for Share Plus or else you can't share log in Settings
--keep interface android.content.res.XmlResourceParser { *; }

android/app/src/main/kotlin/app/revanced/manager/flutter/MainActivity.kt

@@ -9,6 +9,7 @@ import android.os.Handler
 import android.os.Looper
 import app.revanced.library.ApkUtils
 import app.revanced.library.ApkUtils.applyTo
+import app.revanced.library.installation.installer.LocalInstaller
 import app.revanced.manager.flutter.utils.Aapt
 import app.revanced.manager.flutter.utils.packageInstaller.InstallerReceiver
 import app.revanced.manager.flutter.utils.packageInstaller.UninstallerReceiver
@@ -168,8 +169,7 @@ class MainActivity : FlutterActivity() {
                                             putValue(option.default)
 
                                             option.values?.let { values ->
-                                                put(
+                                                put("values",
-                                                    "values",
                                                     JSONObject().apply {
                                                         values.forEach { (key, value) ->
                                                             putValue(value, key)
@@ -257,19 +257,16 @@ class MainActivity : FlutterActivity() {
 
             // Setup logger
             Logger.getLogger("").apply {
-                handlers.forEach { handler ->
+                handlers.forEach {
-                    handler.close()
+                    it.close()
-                    removeHandler(handler)
+                    removeHandler(it)
                 }
 
                 object : java.util.logging.Handler() {
                     override fun publish(record: LogRecord) {
-                        if (cancel) return
+                        if (record.loggerName?.startsWith("app.revanced") != true || cancel) return
-                        if (
+
-                            record.loggerName?.startsWith("app.revanced") == true ||
+                        updateProgress(-1.0, "", record.message)
-                            // Logger in class brut.util.OS.
-                            record.loggerName == ""
-                        ) updateProgress(-1.0, "", record.message)
                     }
 
                     override fun flush() = Unit

devtools_options.yaml

@@ -1,3 +0,0 @@
-description: This file stores settings for Dart & Flutter DevTools.
-documentation: https://docs.flutter.dev/tools/devtools/extensions#configure-extension-enablement-states
-extensions:

lib/ui/widgets/patchesSelectorView/patch_options_fields.dart

@@ -85,7 +85,7 @@ class _IntAndStringPatchOptionState extends State<IntAndStringPatchOption> {
             crossAxisAlignment: CrossAxisAlignment.start,
             children: [
               TextFieldForPatchOption(
-                value: value.toString(),
+                value: value,
                 patchOption: widget.patchOption,
                 selectedKey: getKey(),
                 onChanged: (value) {

lib/utils/check_for_supported_patch.dart

@@ -1,4 +1,3 @@
-import 'package:flutter/foundation.dart';
 import 'package:revanced_manager/app/app.locator.dart';
 import 'package:revanced_manager/models/patch.dart';
 import 'package:revanced_manager/models/patched_application.dart';
@@ -18,12 +17,12 @@ bool isPatchSupported(Patch patch) {
 bool hasUnsupportedRequiredOption(List<Option> options, Patch patch) {
   final List<String> requiredOptionsType = [];
   final List<String> supportedOptionsType = [
-    'kotlin.String',          
+    'String',
-    'kotlin.Int',          
+    'Boolean',
-    'kotlin.Boolean',     
+    'Int',
-    'kotlin.StringArray', 
+    'StringArray',
-    'kotlin.IntArray',    
+    'IntArray',
-    'kotlin.LongArray',   
+    'LongArray',
   ];
   for (final Option option in options) {
     if (option.required &&
@@ -39,9 +38,6 @@ bool hasUnsupportedRequiredOption(List<Option> options, Patch patch) {
   }
   for (final String optionType in requiredOptionsType) {
     if (!supportedOptionsType.contains(optionType)) {
-      if (kDebugMode) {
-        print('${patch.name} has unsupported required patch option: $requiredOptionsType');
-      }
       return true;
     }
   }

pubspec.yaml

@@ -4,7 +4,7 @@ homepage: https://revanced.app
 
 publish_to: 'none'
 
-version: 1.24.1-dev.4+101800059
+version: 1.24.0+101800055
 
 environment:
   sdk: '>=3.7.0'