Skip to content

ci: fix Release workflow (Node 22) + add missing changeset - #5

Merged
chiliec merged 1 commit into
mainfrom
ci/fix-release-node-and-changeset
Aug 14, 2026
Merged

ci: fix Release workflow (Node 22) + add missing changeset#5
chiliec merged 1 commit into
mainfrom
ci/fix-release-node-and-changeset

Conversation

@chiliec

@chiliec chiliec commented Aug 14, 2026

Copy link
Copy Markdown
Member

Problem

The Release workflow has been failing on every push to main, so merged changes never publish to npm. Root cause in the run logs:

npm error code EBADENGINE
npm error notsup Required: {"node":"^22.22.2 || ^24.15.0 || >=26.0.0"}
npm error notsup Actual:   {"npm":"10.8.2","node":"v20.20.2"}

The job runs npm install -g npm@latest (for OIDC Trusted Publisher support) but pins Node 20. npm@latest is now v12, which requires Node ≥22 → the step exits 1 before build/publish.

Fix

  1. Bump the Release job to Node 22 so npm@latest installs cleanly and the OIDC --provenance publish path works. (CI workflow is unaffected — it does not upgrade npm; its 18/20/22 matrix stays.)
  2. Add the changeset that was missing when the witness→validator read-type fix merged, so Changesets bumps the package and actually cuts the release.

Result

On merge, Changesets opens/lands the version bump and the OIDC publish step ships the patch to npm — no NPM_TOKEN needed (Trusted Publisher).

…types

The Release workflow failed on every push to main: it runs
`npm install -g npm@latest`, which now resolves npm@12 requiring Node >=22,
while the job pinned Node 20 (EBADENGINE, exit 1) — so the merged type fix
never published. Bump the release job to Node 22 so npm@latest installs and
the OIDC Trusted Publisher path works.

Also add the missing changeset so Changesets bumps @viz-cx/api and the
already-merged read-path type fix actually gets released.
@chiliec
chiliec merged commit f75d54d into main Aug 14, 2026
4 checks passed
@chiliec
chiliec deleted the ci/fix-release-node-and-changeset branch August 14, 2026 02:11
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