Skip to content

manager: add support for opening and flashing zip files directly#2967

Merged
tiann merged 2 commits into
tiann:mainfrom
sakana164:zip
Nov 22, 2025
Merged

manager: add support for opening and flashing zip files directly#2967
tiann merged 2 commits into
tiann:mainfrom
sakana164:zip

Conversation

@sakana164

@sakana164 sakana164 commented Nov 20, 2025

Copy link
Copy Markdown
Contributor

Signed-off-by: sakana164 [76257039+sakana164@users.noreply.github.com]

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull Request Overview

This PR adds support for opening ZIP module files directly with the KernelSU manager application by registering an intent filter and implementing automatic navigation to the flash screen when a ZIP file is opened.

  • Registers an intent filter in AndroidManifest.xml to handle ZIP file VIEW intents with content scheme
  • Implements automatic navigation to FlashScreen in MainActivity when a ZIP file intent is detected

Reviewed Changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated 2 comments.

File Description
manager/app/src/main/AndroidManifest.xml Adds intent filter to MainActivity for handling ZIP files with VIEW action, content scheme, and application/zip MIME type
manager/app/src/main/java/me/weishu/kernelsu/ui/MainActivity.kt Adds LaunchedEffect to detect ZIP file intents and navigate to FlashScreen with the file URI when isManager is true

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread manager/app/src/main/java/me/weishu/kernelsu/ui/MainActivity.kt Outdated
Comment thread manager/app/src/main/java/me/weishu/kernelsu/ui/MainActivity.kt Outdated

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull Request Overview

Copilot reviewed 2 out of 2 changed files in this pull request and generated 2 comments.


💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread manager/app/src/main/AndroidManifest.xml Outdated
Comment thread manager/app/src/main/java/me/weishu/kernelsu/ui/MainActivity.kt Outdated

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull Request Overview

Copilot reviewed 2 out of 2 changed files in this pull request and generated 4 comments.


💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread manager/app/src/main/java/me/weishu/kernelsu/ui/MainActivity.kt
Comment thread manager/app/src/main/java/me/weishu/kernelsu/ui/MainActivity.kt
Comment thread manager/app/src/main/java/me/weishu/kernelsu/ui/MainActivity.kt

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Copilot encountered an error and was unable to review this pull request. You can try again by re-requesting a review.

@tiann tiann merged commit 4584217 into tiann:main Nov 22, 2025
15 checks passed
@sakana164 sakana164 deleted the zip branch November 22, 2025 09:20
u9521 added a commit to u9521/KernelSU that referenced this pull request Jan 4, 2026
…nn#2967)

Co-authored-by: sakana164 <76257039+sakana164@users.noreply.github.com>
Signed-off-by: u9521 <63995396+u9521@users.noreply.github.com>
u9521 added a commit to u9521/KernelSU that referenced this pull request Jan 5, 2026
…nn#2967)

Co-authored-by: sakana164 <76257039+sakana164@users.noreply.github.com>
Signed-off-by: u9521 <63995396+u9521@users.noreply.github.com>
u9521 added a commit to u9521/KernelSU that referenced this pull request Jan 13, 2026
…nn#2967)

Co-authored-by: sakana164 <76257039+sakana164@users.noreply.github.com>
Signed-off-by: u9521 <63995396+u9521@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants