Skip to content

fix(ci): publish on releases_created (manifest mode)#74

Merged
zhawtof merged 1 commit into
mainfrom
fix/release-please-publish-conditional
May 20, 2026
Merged

fix(ci): publish on releases_created (manifest mode)#74
zhawtof merged 1 commit into
mainfrom
fix/release-please-publish-conditional

Conversation

@zhawtof
Copy link
Copy Markdown
Contributor

@zhawtof zhawtof commented May 20, 2026

Summary

  • Switch every publish step's if: from release_created (singular) to releases_created (plural).
  • This matches what release-please-action actually sets when running in manifest mode (which we use via manifest-file: .release-please-manifest.json).

Why

The publish steps have been silently skipping on every release because steps.release.outputs.release_created is empty in manifest mode — only releases_created (plural) is set, plus per-path .--release_created. A skipped step counts as success, so the job stays green and the failure is invisible.

Proof: the v0.8.0 merge run https://github.com/TightknitAI/block-kitchen/actions/runs/26129585575 shows googleapis/release-please-action@v5 ✅ then every step after it ⏭ skipped (checkout, setup, lint, typecheck, build:clean, pnpm publish).

Impact: npm `@tightknitai/block-kitchen` has been stuck at `0.6.1-alpha.0` (published May 17) while GitHub releases continued through `0.6.2-alpha.0`, `0.6.3-alpha.0`, `0.7.0`, and `0.8.0` (today). Anyone installing from npm gets a version the README no longer describes.

Test plan

  • Merge this PR
  • Merge the next release-please PR (likely `chore(main): release block-kitchen 0.8.1` or similar)
  • Verify the Release Please run shows the publish steps running (not skipped) on the release-merge commit
  • Verify `npm view @tightknitai/block-kitchen version` reports the new version

🤖 Generated with Claude Code

The release-please-action in manifest mode (which we use via
`manifest-file: .release-please-manifest.json`) emits `releases_created`
(plural) and per-path `<path>--release_created`. There is no top-level
`release_created` singular, so the publish steps were silently skipped
on every release — including v0.8.0 today.

Proof: run https://github.com/TightknitAI/block-kitchen/actions/runs/26129585575
(v0.8.0 merge) shows release-please-action ✅ but every step gated on
release_created ⏭ skipped. As a result npm has been stuck at
0.6.1-alpha.0 while GitHub releases continued through 0.7.x and 0.8.0.

Switching the conditional to `releases_created` matches what the action
actually sets in manifest mode.

Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
@cloudflare-workers-and-pages
Copy link
Copy Markdown
Contributor

Deploying with  Cloudflare Workers  Cloudflare Workers

The latest updates on your project. Learn more about integrating Git with Workers.

Status Name Latest Commit Preview URL Updated (UTC)
✅ Deployment successful!
View logs
block-kitchen 800ddb0 Commit Preview URL

Branch Preview URL
May 20 2026, 04:58 AM

@zhawtof zhawtof merged commit c29a2ce into main May 20, 2026
13 checks passed
@zhawtof zhawtof deleted the fix/release-please-publish-conditional branch May 20, 2026 05:02
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