Skip to content

ci: separate npm OIDC publishing from changepacks - #11

Merged
owjs3901 merged 1 commit into
mainfrom
fix/separate-npm-oidc-publish
Sep 10, 2026
Merged

owjs3901 merged 1 commit into
mainfrom
fix/separate-npm-oidc-publish

Conversation

@owjs3901

Copy link
Copy Markdown
Contributor

The npm OIDC configuration was rolled back when changepacks checked out the earlier release source, so its built-in dry-run invoked Bun and failed before npm authentication.

Follow the working csstype-extra publication pattern: run checks, let changepacks select package targets with publish: false, then build and publish from a fresh workflow checkout using npm publish --access public --ignore-scripts. Keep publish.yml unchanged as the trusted-publisher filename, retain Node 24/npm >=11.5.1 and job-scoped OIDC permission, and remove unused custom changepacks publish commands.

A separate finalization job runs only after npm succeeds and handles the existing 1.0.4 draft release. This workflow does not depend on changepacks/action#8.

Validation:

  • actionlint 1.7.12: pass.
  • 28 tests, 2 snapshots, lint, TypeScript/build: pass.
  • Exact npm publication command with --dry-run: pass; includes all dist entry points and declarations.
  • Independent review completed.

Recovery: if npm fails, use Re-run failed jobs to preserve changepacks target outputs. With the reference workflow's publish: false behavior, new GitHub releases are public before npm succeeds, so a new push or rerunning all jobs can skip an already-released version. If only finalization fails, retry that job without republishing npm.

Reference: https://github.com/dev-five-git/csstype-extra/blob/main/.github/workflows/publish.yml
Successful reference run: https://github.com/dev-five-git/csstype-extra/actions/runs/29142929643
Incident: https://github.com/dev-five-git/bun-test-env-dom/actions/runs/34464132018/job/102828597908

Actual OIDC publication has not been triggered; merging this workflow change will run it on main.

@owjs3901
owjs3901 marked this pull request as ready for review September 10, 2026 10:21
@owjs3901
owjs3901 merged commit 1b664f8 into main Sep 10, 2026
1 check 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