Skip to content

Dual-publish to GitHub Packages alongside npmjs.org - #13

Merged
miguelcolmenares merged 1 commit into
mainfrom
feature/dual-registry-publish
Sep 2, 2026
Merged

Dual-publish to GitHub Packages alongside npmjs.org#13
miguelcolmenares merged 1 commit into
mainfrom
feature/dual-registry-publish

Conversation

@miguelcolmenares

Copy link
Copy Markdown
Member

Summary

  • Adds a second publish step to GitHub Packages, auth'd with the workflow's own GITHUB_TOKEN.
  • Backports the already-published-version idempotency guard this workflow never had (unlike icons/recaptcha/consent-banner), for both registries.

Why

Same rollout as the rest of the fleet: @silverassist/nextjs-core is private, GitHub-Packages-only, and a consumer scoping @silverassist there needs every package in the scope reachable, or npm's allow-remote hardening blocks the mismatched ones on npm ci.

Test plan

  • npm run check passes locally via the pre-push gate
  • actionlint on the changed workflow, no errors
  • Cut a release (or workflow_dispatch) after merge and confirm the package appears on GitHub Packages

Keeps the whole @SilverAssist scope resolvable from a single registry
for consumers whose .npmrc scopes @SilverAssist to GitHub Packages --
needed because @silverassist/nextjs-core is GitHub-Packages-only
(private), and mixing registries within one scope otherwise makes npm
treat every other package's lockfile-resolved URL as an untrusted
"remote" dependency.

Also backports the already-published-version idempotency guard this
package never had, so a re-run of this workflow doesn't fail trying to
republish an already-current version on either registry.
@miguelcolmenares
miguelcolmenares merged commit f625b66 into main Sep 2, 2026
3 checks passed
@miguelcolmenares
miguelcolmenares deleted the feature/dual-registry-publish branch September 2, 2026 17:23
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