Skip to content

feat(core): harden plugin and release delivery - #242

Open
mwaldheim wants to merge 6 commits into
mainfrom
copilot/production-readiness-2026-07-24
Open

feat(core): harden plugin and release delivery#242
mwaldheim wants to merge 6 commits into
mainfrom
copilot/production-readiness-2026-07-24

Conversation

@mwaldheim

@mwaldheim mwaldheim commented Jul 24, 2026

Copy link
Copy Markdown
Contributor

Summary

  • add canonical first-party plugin identity, migration warnings, lock handling, doctor coverage, and action-image ownership checks
  • add SDK contract/process helpers and registry release-path E2E coverage
  • harden release workflows and extend the central verifier to all 43 plugins, including publisher-docker
  • compare generated workflow architecture independently from valid setup-go patch pin advances

Validation

  • actionlint -no-color
  • 43-repository scripts/verify-release-workflows.ps1
  • go test -count=1 ./internal/... ./pkg/... ./cmd/...
  • integration contract/release-path tests
  • go build ./cmd/semrel
  • git diff --check

Linux/Docker runtime caveat

Local validation ran on Windows without a Docker daemon. Linux action-shell and Docker build/scan/promotion paths remain covered by CI.

@mwaldheim
mwaldheim requested a review from tboerger as a code owner July 24, 2026 12:28
Comment thread pkg/sdk/sdktest/process.go Fixed
@mwaldheim
mwaldheim force-pushed the copilot/production-readiness-2026-07-24 branch 2 times, most recently from 47514c0 to 75ef54f Compare July 24, 2026 12:33
Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Signed-off-by: mwaldheim <mwaldheim@users.noreply.github.com>
mwaldheim and others added 2 commits August 3, 2026 12:33
Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Signed-off-by: Markus Waldheim <mawa@if.de>
Reject spoofed first-party metadata and unsafe cache or executable path components before resolving or installing plugin binaries.

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Signed-off-by: Markus Waldheim <mawa@if.de>
mwaldheim and others added 2 commits August 3, 2026 12:53
Keep namespace-isolated paths for community plugins while retaining stable artifact cache directories for official plugins.

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Signed-off-by: Markus Waldheim <mawa@if.de>
Carry the open Renovate and CodeQL pin updates in the production-readiness branch so it does not downgrade the dependency PRs when merged.

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Signed-off-by: Markus Waldheim <mawa@if.de>
Comment thread .github/workflows/actionlint.yaml Outdated
Comment thread .github/workflows/ci.yaml Outdated
Comment thread .github/workflows/ci.yaml Outdated
Comment thread .github/workflows/ci.yaml Outdated
Comment thread .github/workflows/core-registry-e2e.yml Outdated
Comment thread .github/workflows/fuzzing.yaml Outdated
Comment thread .github/workflows/release.yaml Outdated
Comment thread .github/workflows/reuse.yaml Outdated
Comment thread .github/workflows/sast.yaml Outdated
Comment thread .github/workflows/scorecard.yaml Outdated
Use the workflow name directly in concurrency groups, centralize read permissions, and remove the duplicate release setup-go step.

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Signed-off-by: Markus Waldheim <mawa@if.de>
@mwaldheim

Copy link
Copy Markdown
Contributor Author

Thomas’ workflow feedback is addressed in 874ddab: redundant concurrency-name prefixes and duplicate job permissions were removed, and the duplicate setup-go step in the release workflow was deleted. All Go tests/builds pass; the workflow review threads are resolved.

@mwaldheim
mwaldheim requested a review from tboerger August 6, 2026 07:39
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.

3 participants