Skip to content

Improve module size loading and harden build workflow - #236

Closed
weibolin-259 wants to merge 2 commits into
LyraVoid:mainfrom
weibolin-259:upstream-ci-module-cache
Closed

weibolin-259 wants to merge 2 commits into
LyraVoid:mainfrom
weibolin-259:upstream-ci-module-cache

Conversation

@weibolin-259

Copy link
Copy Markdown

Summary

  • Persist module size cache and batch size measurements to reduce module-list overhead.
  • Pass GitHub event and secret data through environment variables and safely escape Telegram captions.

Validation

  • succeeds locally on the latest upstream base.
  • Unit test task passes; some modules currently have no unit test sources.

The upstream build-environment dependency updates from the fork were not included because the current upstream already contains equivalent changes.

- Persist id to (bytes, versionCode) size cache into SharedPreferences
  so cold start hits the cache immediately, removing the transient 0 KB
  flash on the module list before du finishes
- Invalidate cache per module when versionCode changes, fixing stale
  sizes shown after a module update
- Replace per-module for-loop du (one fork/exec per module) with a
  single busybox du -sb invocation for all module dirs
- Skip size prefetch entirely when the show-more-module-info setting is
  disabled; measure on demand when the toggle is turned on
- Add mutex to prevent overlapping du runs from rapid refresh or toggle
Commit messages containing quotes, backticks or dollar-expansions broke
the bash scripts where head_commit.message and similar fields were
interpolated directly into run blocks, failing the prepare job.

- prepare: pass inputs and event fields through env, not string
  interpolation
- upload: encode commit message, author and other caption fields with
  jq so Telegram captions survive arbitrary characters
- keystore and Telegram secrets likewise moved to env vars
@weibolin-259

Copy link
Copy Markdown
Author

Closing this PR per request; these changes are intended for the fork's own commit history, not for upstream.

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.

1 participant