Skip to content

fix(release): correct repo URLs to platform_services + Node 24 - #31

Merged
ABujalance merged 2 commits into
mainfrom
fix/release-repo-url-node24
Jul 26, 2026
Merged

fix(release): correct repo URLs to platform_services + Node 24#31
ABujalance merged 2 commits into
mainfrom
fix/release-repo-url-node24

Conversation

@ABujalance

Copy link
Copy Markdown
Contributor

Recovers the partial 0.4.0 release and unblocks publishing.

What happened

Merging #30 ran the release: OIDC trusted publishing worked (authenticated, built, signed provenance), but the final publish hit npm error 422 - Error verifying sigstore provenance bundle: package.json repository.url still said ThatOpen/engine_services, which no longer matches the platform_services provenance from Actions. So main got bumped to 0.4.0 and the changeset was consumed, but nothing published (npm still 0.3.16).

This PR

  • Revert the partial version bump (f357456) so the get-avatar changeset is restored and the pipeline can re-cut 0.4.0 cleanly.
  • Fix the three engine_servicesplatform_services URLs in package.json (repository, homepage, bugs) — the actual publish blocker.
  • Bump both workflows to Node 24 (Node 20 is EOL). ci.yml moving too means this PR's own build check validates the vite-5 build on Node 24 before any release depends on it.

On merge

Pipeline finds the changeset, versions to 0.4.0, builds on Node 24, and publishes with matching provenance → getAvatar ships as 0.4.0.

Depends on this PR's CI (Node 24 build) passing first — if it fails, the Node 24 bump comes out and we ship on 20.

…e to Node 24

The 0.4.0 publish failed npm provenance verification: package.json repository.url (and homepage/bugs) still pointed at the old engine_services repo name, which no longer matches the platform_services provenance from Actions. Point them at the current repo. Also bumps both workflows to Node 24 (Node 20 is EOL); the paired ci.yml change validates the vite build on Node 24 in this PR before the release relies on it. The preceding revert restores the get-avatar changeset so the pipeline re-cuts 0.4.0 cleanly on merge.
@ABujalance
ABujalance merged commit e4452e7 into main Jul 26, 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