Skip to content

feat: move Kaorios to system paths and upload module test artifacts#7

Merged
Jefino9488 merged 1 commit into
masterfrom
feature/kaorios-system-path-and-ci
Apr 3, 2026
Merged

feat: move Kaorios to system paths and upload module test artifacts#7
Jefino9488 merged 1 commit into
masterfrom
feature/kaorios-system-path-and-ci

Conversation

@Jefino9488
Copy link
Copy Markdown
Member

No description provided.

Copilot AI review requested due to automatic review settings April 3, 2026 04:53
Copy link
Copy Markdown

Copilot AI left a comment

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 adjusts how the Kaorios Toolbox payload is placed inside the generated module (moving from /product-scoped paths to /system-scoped paths) and enhances the CI feature test workflow to retain the produced module ZIPs as downloadable artifacts.

Changes:

  • Update module creation to install Kaorios Toolbox APK and permissions into system/priv-app and system/etc/permissions instead of system/product/....
  • In the feature test workflow, copy each generated module ZIP into the per-test-case results folder with a stable, versioned filename.
  • Upload those per-test-case module ZIPs as GitHub Actions artifacts (with retention configured).

Reviewed changes

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

File Description
scripts/core/module.sh Moves Kaorios Toolbox APK + permission allowlist destinations from system/product/... to system/... paths inside the module output.
.github/workflows/feature-test-suite.yml Captures per-feature generated module ZIPs into test-results/... and uploads them as artifacts for inspection/debugging.

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

@Jefino9488 Jefino9488 merged commit b0a0a25 into master Apr 3, 2026
8 checks passed
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.

2 participants