mirror of
https://github.com/rtk-ai/rtk.git
synced 2026-03-26 19:28:46 +00:00
Patrick reported 2 critical issues post-merge PR #23: 1. **release.yml never triggers**: release-please creates tags via GitHub API → no push event generated → build workflow never runs → v0.5.1 released without binaries 2. **README install URLs 404**: DEB/RPM URLs hardcode version 0.3.1 → /releases/latest/download/ serves different filenames → all releases > 0.3.1 break installation instructions Root cause analysis: - release-please creates GitHub Releases (triggers `release.published` event) - release.yml only listens to `on: push: tags` (doesn't fire for API-created tags) - Standard pattern: release-please + binary builds = `on: release.published` Fixes: 1. release.yml trigger: - Add `on: release: types: [published]` (standard release-please pattern) - Remove `on: push: tags: ['v*']` (dead code with release-please) - Update version extraction to handle `release` event - Split release job: upload assets (release event) vs create release (workflow_dispatch) 2. Version-agnostic package naming: - Create copies: rtk_0.5.0-1_amd64.deb → rtk_amd64.deb - Create copies: rtk-0.5.0-1.x86_64.rpm → rtk.x86_64.rpm - Update README URLs to use version-agnostic names - /releases/latest/download/ now serves stable filenames Impact: - release-please releases now auto-trigger binary builds - Installation URLs work for all future releases - No manual workflow_dispatch needed for new versions Manual action required: Patrick needs to re-run build for v0.5.1 via workflow_dispatch (or create v0.5.2 with a trivial fix commit) Co-Authored-By: Claude Sonnet 4.5 <noreply@anthropic.com>