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

🐛 processing panicked: internal error: entered unreachable code: you should map the correct references #548

Closed
1 task done
lgarron opened this issue Oct 19, 2023 · 2 comments · Fixed by #549
Closed
1 task done
Assignees

Comments

@lgarron
Copy link

lgarron commented Oct 19, 2023

Environment information

Version: 1.3.0
macOS Sonoma 14.0

What happened?

Run:

git clone https://github.com/cubing/cubing.js && cd cubing.js
git checkout https://github.com/cubing/cubing.js
make lint

This fails with:

./src/cubing/puzzle-geometry/PuzzleGeometry.ts internalError/panic  INTERNAL  ━━━━━━━━━━━━━━━━━━━━━━

  ✖ processing panicked: internal error: entered unreachable code: you should map the correct references
  
  ⚠ This diagnostic was derived from an internal Biome error. Potential bug, please report it if necessary.

Expected result

No internal error?

Code of Conduct

  • I agree to follow Biome's Code of Conduct
@lgarron
Copy link
Author

lgarron commented Oct 19, 2023

I was able to reduce this to the following repro (which is valid JS and can be run successfully):

const a = [3];
const b = [4];
const c = 0;
a[c] = b[0];
console.log(a);

Pasting this into the playground gives:

panicked at crates/biome_js_analyze/src/analyzers/correctness/no_self_assign.rs:716:18:

Which does seem to correspond to the error message:

_ => unreachable!("you should map the correct references"),

lgarron added a commit to cubing/cubing.js that referenced this issue Oct 19, 2023
This reverts commit 1afd544.

Biome currently crashes on our codebase, due to: biomejs/biome#548
@Conaclos Conaclos self-assigned this Oct 19, 2023
@Hawkeye66
Copy link

Also getting the same error on following two lines:

let i = 0;
$myVar.TEMPLATE[i].LOCAL.test = $myVar.TEMPLATE[i].test;

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 a pull request may close this issue.

3 participants