Skip to content

fix: install nRF Cloud libraries from NPM instead of JSR - #68

Merged
coderbyheart merged 1 commit into
mainfrom
fix/deps-from-npm
Aug 5, 2026
Merged

fix: install nRF Cloud libraries from NPM instead of JSR#68
coderbyheart merged 1 commit into
mainfrom
fix/deps-from-npm

Conversation

@coderbyheart

@coderbyheart coderbyheart commented Aug 5, 2026

Copy link
Copy Markdown
Member

Consumes the nRF Cloud helper libraries from NPM instead of JSR, now that they
are published to NPM.

What changed

  • npm:@jsr/nrfcloud__… aliases are replaced with plain NPM dependencies,
    pinned to explicit versions without a caret.
  • @jsr:registry=https://npm.jsr.io is removed from .npmrc — nothing is
    installed from JSR anymore.
  • allow-remote is removed from .npmrc as well. It only existed to permit
    JSR's remote tarball URLs, which NPM v12 blocks by default. Leaving it in
    place would keep remote tarball fetches enabled for every dependency in the
    tree for no reason. There are no remaining dependencies on remote URLs — the
    only other npm: aliases are the TypeScript ones, which resolve from the
    registry and do not need it. With both settings gone the file has no content
    left, so it is deleted.

Verified locally

npm install from a clean slate (no node_modules, no lock file, no .npmrc)
resolves everything from registry.npmjs.org, npm ci then reinstalls from the
regenerated lock file, and the type check, tests and build all pass.

Release order

This has to be released before the applications that consume this package
can drop @jsr:registry from their own .npmrc. The currently published
versions still carry the JSR aliases in their metadata, which forces every
consumer to keep a @jsr registry mapping configured.

@coderbyheart
coderbyheart requested a review from a team as a code owner August 5, 2026 12:30
These libraries are published to NPM now, so consume them as ordinary registry
dependencies instead of through JSR's NPM compatibility layer
(`npm:@jsr/nrfcloud__…`). The versions are pinned explicitly, without a caret.

This also removes `@jsr:registry` and `allow-remote` from `.npmrc`. Nothing is
installed from JSR anymore, and `allow-remote` only existed to permit JSR's
remote tarball URLs, which NPM v12 blocks by default. Keeping it would leave
remote tarball fetches enabled for every dependency in the tree for no reason.
@coderbyheart

Copy link
Copy Markdown
Member Author

Security fix, no review required.

@coderbyheart
coderbyheart merged commit 8b04006 into main Aug 5, 2026
2 checks passed
@coderbyheart
coderbyheart deleted the fix/deps-from-npm branch August 5, 2026 13: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