Skip to content

feat(ci): validate registry plugins - #124

Merged
caterryan merged 1 commit into
mainfrom
ci/validate-registry-plugins
Jul 15, 2026
Merged

feat(ci): validate registry plugins#124
caterryan merged 1 commit into
mainfrom
ci/validate-registry-plugins

Conversation

@caterryan

@caterryan caterryan commented Jul 14, 2026

Copy link
Copy Markdown
Collaborator

Summary

Adds a validate-registry-plugins workflow that runs the influxdb3-plugin CLI against every packageable plugin directory on pull requests and pushes to main.

  • Stable required check: changesvalidateresult job split gives branch protection a single registry-plugin-validate check name that passes on unrelated PRs (validation skipped) without blocking them.
  • No silent passes: plugin discovery fails the build if it finds zero directories, so a broken glob or renamed root can't produce a green build that validated nothing (also applied to publish-registry).
  • Single SDK version source: both workflows read .github/plugin-sdk-version (now v0.6.0, required for manifest schema 1.3), so validate and publish can't drift.
  • Hardening: CLI installed to $RUNNER_TEMP/bin instead of /usr/local/bin; actions/checkout pinned to a commit SHA; SHA256-verified CLI download; persist-credentials: false.
  • Accurate PR filtering: diffs base...head (not the merge ref) so base-branch drift doesn't trigger validation, and excluded directories (library) don't count as plugin changes.

Run the influxdb3-plugin CLI against every packageable plugin directory
on pull requests and pushes to main.

- Split into changes/validate/result jobs so a single stable
  registry-plugin-validate check can be required in branch protection
  without blocking unrelated PRs.
- Fail when plugin discovery finds no directories so a broken glob
  cannot pass silently (also applied to publish-registry).
- Read the SDK version from .github/plugin-sdk-version, shared with
  publish-registry, so validate and publish cannot drift; bump the SDK
  to v0.6.0 for manifest schema 1.3 support.
- Install the CLI into $RUNNER_TEMP/bin instead of /usr/local/bin and
  pin actions/checkout to a commit SHA.

Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
@caterryan caterryan changed the title ci: validate registry plugins feat(ci): validate registry plugins Jul 14, 2026
@caterryan
caterryan merged commit 7ca950e into main Jul 15, 2026
7 checks passed
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