Skip to content

Conversation

@ichoosetoaccept
Copy link
Member

@ichoosetoaccept ichoosetoaccept commented Dec 30, 2025

Summary

Gates releases on CI passing to prevent broken commits from being published to npm/MCP Registry.

Problem

Previously, release.yml and ci.yml both triggered on push to main in parallel. This meant a broken commit could publish to npm/MCP Registry before CI caught the failure.

Solution

Changed release.yml to use workflow_run trigger:

  • Waits for CI workflow to complete on main branch
  • Only proceeds if CI passed (or if manually triggered via workflow_dispatch)
  • Follows the pattern documented in ci-components README

Changes

  • release.yml: Changed trigger from push to workflow_run on CI completion
  • Added if condition to check workflow_run.conclusion == 'success'

Copy link
Member Author

This stack of pull requests is managed by Graphite. Learn more about stacking.

@ichoosetoaccept ichoosetoaccept marked this pull request as ready for review December 30, 2025 09:17
Copy link
Member Author

ichoosetoaccept commented Dec 30, 2025

Merge activity

  • Dec 30, 9:17 AM UTC: A user started a stack merge that includes this pull request via Graphite.
  • Dec 30, 9:18 AM UTC: @ichoosetoaccept merged this pull request with Graphite.

@ichoosetoaccept ichoosetoaccept merged commit 06a7108 into main Dec 30, 2025
3 checks passed
@github-actions
Copy link

🎉 This PR is included in version 0.3.1 🎉

The release is available on GitHub release

Your semantic-release bot 📦🚀

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants