Skip to content

chore: trigger axi npm package build on CLI release (CLI-1696) - #7118

Merged
nick-y-snyk merged 2 commits into
mainfrom
chore/CLI-1696_trigger_axi_publish
Aug 12, 2026
Merged

chore: trigger axi npm package build on CLI release (CLI-1696)#7118
nick-y-snyk merged 2 commits into
mainfrom
chore/CLI-1696_trigger_axi_publish

Conversation

@nick-y-snyk

@nick-y-snyk nick-y-snyk commented Aug 12, 2026

Copy link
Copy Markdown
Contributor

What does this PR do?

Adds axi as a 4th target in trigger-distribution-channels, alongside snyk-images, scoop-snyk, and homebrew-tap. Reuses the existing trigger_repository_event helper — fires a repository_dispatch with event_type: cli_release and client_payload: {version, release_channel} at github.com/snyk/axi/dispatches.

axi's build-on-cli-release.yml already listens for cli_release and reads client_payload.version/release_channel so rc/preview builds don't clobber the npm latest tag.

CLI-1696: create and publish the @snyk/axi npm package, triggered by CLI releases.

Where should the reviewer start?

trigger_repository_event in release-scripts/upload-artifacts.sh — the new call is a straight copy of the existing snyk-images/scoop-snyk/homebrew-tap entries, just a different repo + event type.

How should this be manually tested?

shellcheck clean. Simulated the JSON payload construction and confirmed it parses into the exact version/release_channel keys axi's workflow reads.

Live end-to-end validated: @snyk/axi@0.1.0 has been bootstrap-published (npm requires an initial token-based publish before Trusted Publishing can be configured). axi's OIDC trusted-publishing workflow itself was separately dry-run tested end-to-end on a disposable package (@nick-snyk/axi-test) before landing, including the actual npm publish step succeeding via GitHub Actions OIDC. Trusted Publisher config for @snyk/axi is pending setup by someone with npm org access; this PR's dispatch call works independently of that.

What's the product update that needs to be communicated to CLI users?

None — internal release plumbing, no user-facing change.

Risk assessment

Low. trigger_repository_event failures are caught and logged, non-blocking to the rest of the release (per its existing docstring); a broken/missing axi dispatch does not fail the CLI release itself.

Supersedes #7117 (opened from a fork before I had direct write access to this repo).

Adds axi as a 4th target alongside snyk-images/scoop-snyk/homebrew-tap
in trigger-distribution-channels, reusing the existing
trigger_repository_event helper (repository_dispatch, event_type
cli_release, payload {version, release_channel}). axi's
build-on-cli-release.yml already listens for this event.

CLI-1696
@snyk-io

snyk-io Bot commented Aug 12, 2026

Copy link
Copy Markdown

Snyk checks have passed. No issues have been found so far.

Status Scan Engine Critical High Medium Low Total (0)
Open Source Security 0 0 0 0 0 issues
Licenses 0 0 0 0 0 issues
Code Security 0 0 0 0 0 issues

💻 Catch issues earlier using the plugins for VS Code, JetBrains IDEs, Visual Studio, and Eclipse.

@nick-y-snyk
nick-y-snyk enabled auto-merge August 12, 2026 14:30
@nick-y-snyk
nick-y-snyk disabled auto-merge August 12, 2026 14:33
@snyk-pr-review-bot

Copy link
Copy Markdown

PR Reviewer Guide 🔍

🧪 No relevant tests
🔒 No security concerns identified
⚡ No major issues detected
📚 Repository Context Analyzed

This review considered 9 relevant code sections from 8 files (average relevance: 0.66)

🤖 Repository instructions applied (from AGENTS.md)

@github-actions

Copy link
Copy Markdown
Contributor
Warnings
⚠️ There are multiple commits on your branch, please squash them locally before merging!

Generated by 🚫 dangerJS against 3a18886

@nick-y-snyk
nick-y-snyk enabled auto-merge August 12, 2026 15:54
@nick-y-snyk
nick-y-snyk merged commit 237a02c into main Aug 12, 2026
9 checks passed
@nick-y-snyk
nick-y-snyk deleted the chore/CLI-1696_trigger_axi_publish branch August 12, 2026 16:44
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.

2 participants