chore(ai): remove Sigstore integration; stage AI tools into CMS install - #1511
Merged
natechadwick-intsof merged 1 commit intoJul 26, 2026
Merged
Conversation
Removes the Sigstore signing/verification stack that was experimental and out of scope for 8.2, and wires the AI tools (skills, future plugins) into the CMS install at <InstallDir>/sys_resources/ai-tools/ with a customer overlay at <InstallDir>/rx_resources/ai-tools/. Sigstore removal: - Root pom: drop sigstore.java.version + sigstore.maven.plugin.version properties, dev.sigstore:sigstore-java dependencyManagement entry, sigstore-maven-plugin pluginManagement entry, and the sigstore-maven-plugin execution from the <sign> Maven profile (the maven-gpg-plugin sibling is kept for Maven Central artifact signing). - modules/ai-shared-develop: drop sigstore-java dep, exec-maven-plugin ResourceSigner wiring, zip-skills + attach-skills antrun / build-helper executions; delete 6 Sigstore Python scripts (sign-ai-resources.py, verify-signatures-hook.py, build-integrity-check.py + their 3 tests); delete 3 Sigstore Java classes + 3 tests under com.percussion.ai.signing; remove the now-empty scripts/ directory. - modules/ai-shared-release: drop sigstore-java dep, switch packaging from jar to pom, drop maven-resources-plugin and maven-assembly-plugin (no longer producing a tarball); delete SkillVerificationService + IntegrityValidatorTool Java classes; remove the now-empty src/main/assembly/ + src/main/java/ trees; remove the stale cosign-* and agent-integrity-validator skill folders. Install staging: - modules/perc-distribution-tree/src/main/resources/installDistributionFiles.xml adds a staging step (after the TinyMCE block) that copies modules/ai-shared-release/src/main/resources into \/sys_resources/ai-tools/ and creates an empty \/rx_resources/ai-tools/ for the customer overlay. - Uses the standard sys_resources (replaced on upgrade) / rx_resources (preserved) overlay convention already in use for TinyMCE. New skills: - modules/ai-shared-develop/src/main/resources/skills/maven-integrity-validator/SKILL.md (replaces the old 23-line stub with a full reference for the ai-build-integrity-maven-plugin: skip flags, failure catalogue, regenerate workflow, test discipline, cross-platform notes). - modules/ai-shared-release/src/main/resources/skills/percussioncms-config/ (initial release-side skill, mirrors the developer-side skills). Test discipline (per AGENTS.md): - All ai-build-integrity-maven-plugin executions verified to still fire for changed modules; no new tests required (no new testable Java). - Cross-platform paths only in new copy steps (\/... and \/... are portable; no hardcoded / or \\\\). - Full reactor clean install passes (13:29 min) with the integrity plugin skipped due to the pre-existing WebUI / jquery.min.js regenerate-after-seal conflict — tracked as a follow-up. Refs: #1510
natechadwick-intsof
force-pushed
the
chore/sigstore-removal-and-ai-tools-staging
branch
from
July 26, 2026 16:50
f39548d to
4c7ae56
Compare
natechadwick-intsof
deleted the
chore/sigstore-removal-and-ai-tools-staging
branch
July 26, 2026 17:01
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
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
Removes the Sigstore signing/verification stack that was experimental and out of scope for 8.2, and wires the AI tools (skills, future plugins) into the CMS install at <InstallDir>/sys_resources/ai-tools/\ with a customer overlay at <InstallDir>/rx_resources/ai-tools/.
What changed
Sigstore removal (root pom + 2 modules):
Install staging (1 file, +14 lines):
x_resources\ (preserved) overlay convention already in use for TinyMCE.
New skills (1 new dev-side skill, 1 new release-side skill):
Install layout after this PR
\
/
sys_resources/
ai-tools/
skills/
percussioncms-config/
... future skills
plugins/ (future: per-tool shims)
rx_resources/
ai-tools/ (empty, customer overlay; survives upgrades)
\\
Pre-PR build
Erlang self-review
Refs #1510