Skip to content

ci: use shared npm-publish action#145

Merged
dev-jodee merged 1 commit into
mainfrom
ci/use-shared-npm-publish-action
Jun 1, 2026
Merged

ci: use shared npm-publish action#145
dev-jodee merged 1 commit into
mainfrom
ci/use-shared-npm-publish-action

Conversation

@dev-jodee

@dev-jodee dev-jodee commented Jun 1, 2026

Copy link
Copy Markdown
Collaborator

Summary

  • Replace the inline pack / tarball-guard / npm publish steps in publish-typescript.yml with the shared npm-publish composite action, mirroring how publish-rust.yml uses the shared cargo-publish action
  • The workflow still checks out, installs, generates clients, and builds; the action handles resolve → pack → validate → version-check → publish, and feeds version/tag back to the tag/release steps
  • Net: -93/+18 lines in the workflow, no behavior change to tagging or release creation

Test Plan

  • Dry-run dispatched from a hotfix/* branch (temporary 0.3.1 bump, since reverted) passed end-to-end: resolve @solana/subscriptions@0.3.1 (tag: latest) → pack → tarball validation → npm publish --dry-run listing dist/index.{js,cjs,d.ts,d.cts} + maps + package.json + LICENSE + README.md; tag/release correctly skipped on dry-run
  • Caught and fixed a pipefail bug in the action's tarball validation during testing

Action ref

@vercel

vercel Bot commented Jun 1, 2026

Copy link
Copy Markdown

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
solana-subscriptions-program Ready Ready Preview, Comment Jun 1, 2026 9:11pm

Request Review

@github-actions

github-actions Bot commented Jun 1, 2026

Copy link
Copy Markdown

Compute Unit Report

Instruction Samples Min CUs Max CUs Avg CUs Est Cost (Low) [SOL] Est Cost (Med) [SOL] Est Cost (High) [SOL]
cancel_subscription 22 1720 2030 1919 0.000005000 0.000005076 0.000005959
close_subscription_authority 10 1803 1833 1806 0.000005000 0.000005072 0.000005903
create_fixed_delegation 41 3517 13290 5302 0.000005001 0.000005212 0.000007651
create_plan 97 3436 16936 5464 0.000005001 0.000005218 0.000007732
create_recurring_delegation 29 3553 11055 5160 0.000005001 0.000005206 0.000007580
delete_plan 9 359 359 359 0.000005000 0.000005014 0.000005179
init_subscription_authority 173 6226 21226 9421 0.000005002 0.000005376 0.000009710
resume_subscription 3 1723 1723 1723 0.000005000 0.000005068 0.000005861
revoke_delegation 19 255 519 353 0.000005000 0.000005014 0.000005176
subscribe 32 6485 14009 7944 0.000005002 0.000005317 0.000008972
transfer_fixed 9 5479 9966 7364 0.000005002 0.000005294 0.000008682
transfer_recurring 19 5591 11684 8329 0.000005002 0.000005333 0.000009164
transfer_subscription 10 5838 16299 7493 0.000005002 0.000005299 0.000008746
update_plan 22 424 503 477 0.000005000 0.000005019 0.000005238

Generated: 2026-06-01

@dev-jodee dev-jodee marked this pull request as ready for review June 1, 2026 19:25
Replace the inline pack, tarball guards, and npm publish steps with the
shared npm-publish composite action, mirroring the rust client workflow.
@dev-jodee dev-jodee force-pushed the ci/use-shared-npm-publish-action branch from d935630 to 02afc37 Compare June 1, 2026 21:10
@dev-jodee dev-jodee merged commit cef166f into main Jun 1, 2026
13 checks passed
@dev-jodee dev-jodee deleted the ci/use-shared-npm-publish-action branch June 1, 2026 21:18
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