Skip to content

ci: restore npm CI and OIDC publishing - #125

Merged
mulhamna merged 2 commits into
mainfrom
fix/npm-ci-oidc
Sep 14, 2026
Merged

mulhamna merged 2 commits into
mainfrom
fix/npm-ci-oidc

Conversation

@mulhamna

Copy link
Copy Markdown
Owner

Summary

  • run CI validation with Node.js/npm and the committed package-lock.json
  • keep Bun for the runtime smoke test
  • preserve npm trusted publishing with id-token: write and npm provenance
  • keep Bun available in the publish job for the Bun-based release check

Root cause

The merged Bun migration caused the existing npm publish workflow to fail with exit code 127 because it still expected the Node/npm workflow assumptions while the migrated scripts use Bun.

Validation

  • workflow commands mirror the committed package scripts
  • CI and publish workflow use the npm lockfile for dependency installation

@mulhamna
mulhamna merged commit 3c35307 into main Sep 14, 2026
4 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