Skip to content

fix(a13): wire release provenance through the variant API - #18

Merged
thetvplus merged 1 commit into
mainfrom
devin/a13-release-assets-20261024
Oct 24, 2026
Merged

thetvplus merged 1 commit into
mainfrom
devin/a13-release-assets-20261024

Conversation

@thetvplus

Copy link
Copy Markdown
Owner

正式模式把 Provider 直接传给 Android SourceSet.assets,AGP 9.4 在配置阶段拒绝,导致两次 Signed Build 尚未产生候选 Artifact 就失败。使用不存在的占位 keystore 和 --dry-run 已准确复现;普通无签名 CI 不会进入该分支。

将来源证明生成器改为带声明输入/输出的任务,通过 AGP Variant API 的 assets.addGeneratedSourceDirectory 接入,自动建立生成到 assets 合并的依赖。保持来源文件字节格式、完整 SHA、版本、包名及所有签名/来源门禁;使用官方支持的接口,不调整构建工具版本或 Provider 限制。

Fast/Full 新增无密钥的正式任务图检查:仅以占位配置执行 --dry-run,确保来源生成先于 assets 合并和打包,并包含 lint。不会读取真实私钥、执行 APK 打包或签名。Signed Build 的 main CI 检查同时要求这一步真实 success。

验证:正式任务图检查通过;实际运行来源生成任务并核对完整 SHA、r13.14.1/142、release 和应用 ID(只用占位配置,无 APK);本地 Fast/Full、944 项 Python(4 既有 skipped)、compileall、CI 可移植性、814 文件编码及 diff-check 通过。Android 运行时代码、版本与证书保持。

这是维护者在 PR #16 后追加的发行流程调整。Base: a9bf3bf。
参考:https://developer.android.com/reference/tools/gradle-api/9.4/com/android/build/api/variant/SourceDirectories.Flat

@chatgpt-codex-connector

chatgpt-codex-connector Bot commented Oct 24, 2026 •

Copy link
Copy Markdown

Codex Review Summary

This comment shows the latest Codex review activity on this pull request.

Review Status Commit Review trigger
📝 Code Review ✅ Completed 2026-10-24T03:33:34.622245Z fcd214f PR opened
ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review" or "@codex security review".

Codex reacts with 👀 while any review is running, comments if it has suggestions, and reacts with 👍 once all reviews finish with no findings.

@thetvplus
thetvplus merged commit b307f7c into main Oct 24, 2026
2 checks passed
@thetvplus
thetvplus deleted the devin/a13-release-assets-20261024 branch October 24, 2026 03:36
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