Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[Bug]: Protobufs Library 404 #112

Open
agieocean opened this issue Nov 13, 2024 · 4 comments · May be fixed by #120
Open

[Bug]: Protobufs Library 404 #112

agieocean opened this issue Nov 13, 2024 · 4 comments · May be fixed by #120
Labels
bug Something isn't working

Comments

@agieocean
Copy link

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 agieocean added the bug Something isn't working label Nov 13, 2024
@agieocean
Copy link
Author

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?

@danpeig
Copy link

danpeig commented Nov 13, 2024

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
Copy link

danpeig commented Nov 13, 2024

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

@agieocean
Copy link
Author

❯ 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

CreepySunny added a commit to CreepySunny/Fix-Install-Doc that referenced this issue Nov 25, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants