js icon indicating copy to clipboard operation
js copied to clipboard

[Bug]: Protobufs Library 404

Open agieocean opened this issue 1 year ago • 5 comments

Hardware

Not Applicable

Connection Type

HTTP

Firmware Version

n/a

Description

Protobufs library seems to be missing when trying to install with pnpm install instructions and then the provided instructions for this repository

Relevant console output

❯ curl -fsSL https://get.pnpm.io/install.sh | sh -
==> Downloading pnpm binaries 9.13.0
 WARN  using --force I sure hope you know what you are doing          Copying pnpm CLI from /tmp/tmp.1LVuBxDmRV/pnpm to /home/ag/.local/share/pnpm/pnpm                                                           Appended new lines to /home/ag/.config/fish/config.fish               
Next configuration changes were made:                                 set -gx PNPM_HOME "/home/ag/.local/share/pnpm"                        if not string match -q -- $PNPM_HOME $PATH
  set -gx PATH "$PNPM_HOME" $PATH
end

To start using pnpm, run:                                             source /home/ag/.config/fish/config.fish

❯ source /home/ag/.config/fish/config.fish
                                                               
❯ pnpm config set @buf:registry https://buf.build/gen/npm/v1

❯ pnpm add @meshtastic/js
 ERR_PNPM_FETCH_404  GET https://registry.npmjs.org/@jsr%2Fmeshtastic__protobufs: Not Found - 404

This error happened while installing the dependencies of @meshtastic/[email protected]

@jsr/meshtastic__protobufs is not in the npm registry, or you have no permission to fetch it.

No authorization header was set for the request.
Progress: resolved 5, reused 0, downloaded 0, added 0

agieocean avatar Nov 13 '24 14:11 agieocean

Further, it seems to build fine off the repository:

❯ pnpm build

> @meshtastic/[email protected] build /home/ag/Documents/mesh_js_test/js
> tsup && pnpm biome format .

CLI Building entry: src/index.ts
CLI Using tsconfig: tsconfig.json                                     CLI tsup v8.3.0                                                       CLI Using tsup config: /home/ag/Documents/mesh_js_test/js/tsup.config.ts
CLI Target: esnext                                                    CLI Cleaning output folder                                            ESM Build start                                                       ESM dist/index.js     68.75 KB
ESM dist/index.js.map 121.62 KB
ESM ⚡️ Build success in 49ms
DTS Build start                                                       DTS ⚡️ Build success in 3828ms
DTS dist/index.d.ts 28.20 KB
Checked 25 files in 17ms. No fixes applied.

Could this be an issue with my pnpm install?

agieocean avatar Nov 13 '24 16:11 agieocean

Got the same error here using npm instead of pnpn....

npm error code E404
npm error 404 Not Found - GET https://registry.npmjs.org/@jsr%2fmeshtastic__protobufs - Not found
npm error 404
npm error 404  '@jsr/meshtastic__protobufs@^2.5.9' is not in this registry.
npm error 404
npm error 404 Note that you can also install from a
npm error 404 tarball, folder, http url, or git url.

danpeig avatar Nov 13 '24 19:11 danpeig

Found the problem.... installation instructions are still for the old .npmrc. They should be updated to @jsr:registry=https://npm.jsr.io

danpeig avatar Nov 13 '24 20:11 danpeig

❯ pnpm add @meshtastic/js                                             Packages: +1
+                                                                     Progress: resolved 210, reused 166, downloaded 0, added 1, done

dependencies:                                                         + @meshtastic/js 2.5.9-2                                              
Done in 2.6s

Yup that did it

agieocean avatar Nov 13 '24 20:11 agieocean

Just FYI - I see you fixed this in the repo but the public README still has the old instructions

bkimmel avatar Feb 16 '25 18:02 bkimmel