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

Bun crashes when migrating from package-lock with a git dependency #12205

Open
wundersolutions-juanjo opened this issue Jun 27, 2024 · 3 comments
Assignees
Labels
crash An issue that could cause a crash npm Something that relates to the npm-compatible client windows An issue that only occurs on Windows

Comments

@wundersolutions-juanjo
Copy link

wundersolutions-juanjo commented Jun 27, 2024

How can we reproduce the crash?

Tested with the following package.json, no other files on folder, running on a Powershell windows terminal

{
  "dependencies": {
    "sass": "^1.35.1",
    "@hasanm95/cogo-toast": "https://github.com/hasanm95/cogo-toast"
  }
}
  • npm i
  • Remove-Item -r .\node_modules\
  • bun i

Using NPM 10.2.4

Relevant log output

Bun v1.1.13 (bd6a605) Windows x64
Args: "C:\Users\Juanjo.bun\bin\bun.exe" "i"
Elapsed: 2033ms | User: 15ms | Sys: 93ms
RSS: 97.37MB | Peak: 97.64MB | Commit: 0.15GB | Faults: 25605

Stack Trace (bun.report)

Bun v1.1.13 (bd6a605) on windows x86_64 [InstallCommand]

panic: Internal assertion failure

@wundersolutions-juanjo wundersolutions-juanjo added the npm Something that relates to the npm-compatible client label Jun 27, 2024
@Jarred-Sumner Jarred-Sumner added the crash An issue that could cause a crash label Jun 27, 2024
@github-actions github-actions bot added the windows An issue that only occurs on Windows label Jun 27, 2024
Copy link
Contributor

@wundersolutions-juanjo, the latest version of Bun is v1.1.16, but this crash was reported on Bun v1.1.13.

Are you able to reproduce this crash on the latest version of Bun?

bun upgrade

@wundersolutions-juanjo
Copy link
Author

Yup, still reproducible;

Bun v1.1.17 (bb66bba) on windows x86_64 [InstallCommand]

panic: Internal assertion failure

@wundersolutions-juanjo
Copy link
Author

I should also mention that after that crash, running the command again works fine, and that i'm pretty sure it's also reproducible on mac M1

@dylan-conway dylan-conway self-assigned this Jun 28, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
crash An issue that could cause a crash npm Something that relates to the npm-compatible client windows An issue that only occurs on Windows
Projects
None yet
Development

No branches or pull requests

3 participants