Skip to content

Fix release workflow ruleset bypass - #13

Merged
ntotten merged 2 commits into
mainfrom
nate/fix-release-bypass
Aug 28, 2026
Merged

Fix release workflow ruleset bypass#13
ntotten merged 2 commits into
mainfrom
nate/fix-release-bypass

Conversation

@ntotten

@ntotten ntotten commented Aug 28, 2026

Copy link
Copy Markdown
Member

Summary

  • generate the Integrations GitHub App token before checkout
  • scope the token to both tools and www
  • use the App token for checkout so release metadata and tag pushes authenticate as the ruleset bypass actor
  • reuse the token for the existing downstream www dispatch

Fixes the protected-branch failure in release run 33181251265.

Verification

  • ./scripts/sync-skill-metadata.sh --check
  • python3 scripts/check-plugin-manifests.py
  • ./scripts/build-release.sh https://github.com/zuplo/tools/releases/download/v-test
  • parsed .github/workflows/release.yaml with Ruby YAML
  • git diff --check

Copilot AI balanced review requested due to automatic review settings August 28, 2026 15:05

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Pull request overview

Updates the release workflow so protected-branch and tag pushes authenticate through the ruleset-bypass GitHub App.

Changes:

  • Generates one App token for both tools and www.
  • Reuses it for checkout, Git pushes, and downstream dispatch.

💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.

Comment thread .github/workflows/release.yaml
Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com>
@ntotten
ntotten merged commit ef73f5f into main Aug 28, 2026
4 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