ci(android): reduce APK guardrail runtime#394
Merged
Conversation
- skip mobile-full Android builds on PRs unless explicitly labeled - add bundled model artifact guardrail for release-bearing paths - document runtime model delivery and APK size investigation policy - keep lean Android size checks and security workflows enabled Reduces routine Actions time while protecting release bundle size.
Contributor
🚀 PR Quick Check Summary
|
Avoid sort-to-head broken pipe failures under bash pipefail when limiting APK report entries. Keeps the largest-entry APK report while allowing lean Android size checks to complete.
Contributor
🚀 PR Quick Check Summary
|
|
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
This PR reduces routine GitHub Actions runtime while preserving required release and security guardrails for issue #253.
Goal: prevent accidental APK/model bloat and avoid spending PR build time on the known-heavy full Android variant unless it is explicitly requested.
Core outcome:
mobile-fullAndroid APK builds are skipped on PRs by default and run when the PR has thefull-android-buildlabel.Changes
Code
scripts/check-bundled-model-artifacts.shand regression coverage.scripts/check-apk-size.shto include largest APK entries in reports.CI
ci.yml, includingbuild_full_androidcontrol.Risks
full-android-build; push-to-main still runs the full matrix.Rollback plan:
mobile-fullskip gate fromci.yml.Testing
scripts/test-check-bundled-model-artifacts.shscripts/test-check-apk-size.shscripts/test-check-module-sizes.shscripts/check-bundled-model-artifacts.shruby -e 'require "yaml"; ARGV.each { |f| YAML.load_file(f); puts "OK #{f}" }' .github/workflows/ci.yml .github/workflows/pr-checks.yml .github/workflows/iptv_sanity.yml .github/workflows/smoke-tests.ymlbash -n scripts/check-bundled-model-artifacts.sh scripts/test-check-bundled-model-artifacts.sh scripts/check-apk-size.sh scripts/test-check-apk-size.shgit diff --checkCloses #253 only after the full APK footprint is brought under the production target; this PR is a guardrail/runtime reduction step under that issue.