Skip to content

ci: cherry-pick release workflow fixes onto ga/1.0#16

Merged
timeleft-- merged 1 commit into
ga/1.0from
ci/release-workflow-on-ga
May 2, 2026
Merged

ci: cherry-pick release workflow fixes onto ga/1.0#16
timeleft-- merged 1 commit into
ga/1.0from
ci/release-workflow-on-ga

Conversation

@timeleft--
Copy link
Copy Markdown
Member

Cherry-pick of PR #13 (which landed on main) onto ga/1.0.

The release: published event uses the workflow file from the release's target_commitish branch — for KissClaw releases that's ga/1.0, not main. So PR #13 (which fixed permissions + tarball mv + added kc-check-imports gate) had no effect on the v1.0.1-rc.2 release run, which is why it failed at Package tarball with the same broken mv *.tgz self-move logic.

Putting the workflow file on every release-target branch (ga/**, lts/**) ensures release events pick up the corrected workflow.

Release workflow fixes:
- permissions: contents: read -> write (gh release upload requires write).
- Tarball move logic: guarded mv when src and dst names already match.
  pnpm pack already produces "<name>-<version>.tgz", which equals the
  desired "kissclaw-<version>.tgz" when package.json name is "kissclaw".
  The unguarded mv would silently fail or no-op depending on coreutils.
- Resolved release tag once via outputs, used consistently for checkout
  ref, integrity check, and upload.
- Added workflow-dispatch artifact upload for dry-run testing.

kc-check-imports CI gate (NEW):
- Triggers on PRs targeting ga/** and lts/** branches.
- Fetches MachineWisdomAI/kissclaw-tools (the repo for kc-check-imports).
- Runs both --candidates (per-cherry-pick) and --final-tree (accumulated)
  modes against the v2026.4.20 baseline.
- Uploads JSON reports as workflow artifacts.
- Final-tree always runs even if per-candidate fails so reviewers see
  both views.

This wires kc-check-imports as the authoritative CI gate per plan v4
Phase 3.5. Pre-commit hook can be bypassed; CI cannot.

(cherry picked from commit 47624bb)
@timeleft-- timeleft-- merged commit ddfc330 into ga/1.0 May 2, 2026
3 of 11 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.

1 participant