ci: route telegram-ts publish + release through buildspace blocks#2
Conversation
Keep the spec-driven, self-updating release model intact (cron spec regen -> auto-merge -> publish on merge; version mirrors the Telegram Bot API version via scripts/compute-version.ts) and only swap the final two steps to buildspace's shared blocks: - npm publish -> photon-hq/buildspace/.github/blocks/publish-npm - tag + release -> photon-hq/buildspace/.github/blocks/create-github-release This shares the same OIDC Trusted Publishing path and registry-landed verification as every other photon-hq package without adopting buildspace's label-driven version/trigger model. Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
|
Caution Review failedPull request was closed or merged during review No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configurationConfiguration used: Organization UI Review profile: CHILL Plan: Pro Run ID: 📒 Files selected for processing (1)
📜 Recent review details🧰 Additional context used🪛 zizmor (1.25.2).github/workflows/release.yaml[error] 70-70: unpinned action reference (unpinned-uses): action is not pinned to a hash (required by blanket policy) (unpinned-uses) [error] 82-82: unpinned action reference (unpinned-uses): action is not pinned to a hash (required by blanket policy) (unpinned-uses) 🔇 Additional comments (1)
📝 WalkthroughWalkthroughThe release workflow is updated to use reusable ChangesRelease Workflow Migration
Estimated code review effort🎯 2 (Simple) | ⏱️ ~10 minutes Poem
🚥 Pre-merge checks | ✅ 5✅ Passed checks (5 passed)
✏️ Tip: You can configure your own custom pre-merge checks in the settings. ✨ Finishing Touches🧪 Generate unit tests (beta)
Comment |
Keeps the existing spec-driven, self-updating release model and only swaps the final two steps to buildspace's shared blocks — so the publish path matches every other photon-hq package without changing how this package versions or triggers.
Unchanged: the spec-change trigger,
compute:version(npm version mirrors the Telegram Bot API version), generate/build/typecheck/test, and the cron → auto-merge → publish-on-merge flow.Swapped:
npm publish --provenance→photon-hq/buildspace/.github/blocks/publish-npm@main(OIDC Trusted Publishing + provenance, NPM_TOKEN fallback, post-publish registry verification)git tag+gh release create→photon-hq/buildspace/.github/blocks/create-github-release@mainAlso drops the job-level
setup-nodestep — thepublish-npmblock sets up Node/npm internally.Supersedes #1 (the full label-driven swap), which would have broken the self-update flow.
🤖 Generated with Claude Code
Need help on this PR? Tag
/codesmithwith what you need. Autofix is disabled.Summary by CodeRabbit