Skip to content

ADFA-5395 | Adopt build provenance in AI plugins and release pipeline - #88

Draft
jatezzz wants to merge 2 commits into
mainfrom
feat/ADFA-5395-adopt-plugin-provenance
Draft

ADFA-5395 | Adopt build provenance in AI plugins and release pipeline#88
jatezzz wants to merge 2 commits into
mainfrom
feat/ADFA-5395-adopt-plugin-provenance

Conversation

@jatezzz

@jatezzz jatezzz commented Sep 7, 2026

Copy link
Copy Markdown
Contributor

Description

Adopted build provenance placeholders across the AI plugins, documented the provenance contract, and integrated automated enforcement in the release pipeline.

  • AI Plugins: Added ${pluginVcsRevision} and ${pluginBuildTimestamp} to AndroidManifest.xml for ai-core, ai-agent-gemini, ai-agent-local, ai-agent-mcp, ai-agent-openai, and get-ai-models.
  • Versions: Bumped versions for ai-agent-local, ai-core, and get-ai-models.
  • CI/Scripts: Updated scripts/update-libs.sh to export a 12-character PLUGIN_LIBS_REVISION.
  • Provenance Verification: Added a verify_provenance step in scripts/update-libs.sh to fail the build if cgp-build.properties is missing or invalid.
  • NDK Installer Fix: Untracked ndk-cmake.tar.xz in ndk-installer-plugin and added it to .gitignore to prevent persistent +dirty builds.
  • NDK Assembly Guard: Updated the ndk-installer-plugin build file to explicitly fail if the NDK asset is missing prior to assembly.
  • Documentation: Added a comprehensive "Build provenance" section to CLAUDE.md.
  • Code Review: Updated the plugin-review skill rubric to check for missing or +dirty provenance in artifact submissions.

Details

Screen_Recording_20260907_160809_Code.on.the.Go.mp4
image

Ticket

ADFA-5395

Observation

As noted in the new documentation inside CLAUDE.md, the libs_revision effectively maps to the commit before the "chore: update libs" commit. Ensure the update-libs action has refreshed libs/ before building these plugins, or the manifest merger will fail outright.

@claude claude Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Claude Code Review

This repository is configured for manual code reviews. Comment @claude review for a one-time review, or @claude review always to subscribe this PR to a review on every future push.

Tip: disable this comment in your organization's Code Review settings.

@jatezzz
jatezzz requested review from a team, Daniel-ADFA and itsaky-adfa September 7, 2026 21:13
@jatezzz
jatezzz force-pushed the feat/ADFA-5395-adopt-plugin-provenance branch from 74b3cbd to d982781 Compare September 7, 2026 21:23
Wire ${pluginVcsRevision}/${pluginBuildTimestamp} into the six AI manifests,
bump versions where history moved past them, and export PLUGIN_LIBS_REVISION
from update-libs.sh. Needs the libs/ refresh first, or the merger fails.
@jatezzz
jatezzz force-pushed the feat/ADFA-5395-adopt-plugin-provenance branch from d982781 to 7d4caf4 Compare September 8, 2026 15:54
@hal-eisen-adfa
hal-eisen-adfa marked this pull request as draft September 8, 2026 23:04
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