This repository was archived by the owner on Aug 31, 2023. It is now read-only.
-
Notifications
You must be signed in to change notification settings - Fork 649
This repository was archived by the owner on Aug 31, 2023. It is now read-only.
🐛 Big pack of js/ts files that crashes rome #4323
Copy link
Copy link
Open
Labels
S-Bug: confirmedStatus: report has been confirmed as a valid bugStatus: report has been confirmed as a valid bug
Description
Environment information
Rome 763fd98
CLI:
Version: 0.0.0
Color support: true
Platform:
CPU Architecture: x86_64
OS: linux
Environment:
ROME_LOG_DIR: unset
NO_COLOR: unset
TERM: "xterm-256color"
JS_RUNTIME_VERSION: unset
JS_RUNTIME_NAME: unset
NODE_PACKAGE_MANAGER: unset
Rome Configuration:
Status: unset
Workspace:
Open Documents: 0
Checking multiple broken javascript/typescript files(similar to astral-sh/ruff#3721) with rome check file, crashes app in several ways e.g.
Message: byte index 38 is not a char boundary; it is inside '\u{ac257}' (bytes 35..39) of `/ Copyright (c) 212 EcmInternatonal. "true" All rights eserved.`
Source Location: /home/rafal/.cargo/git/checkouts/tools-29b78d514589893f/763fd98/crates/rome_text_size/src/range.rs:48:9
Thread Name: rome::console
Message: assertion failed: start <= end
Source Location: /home/rafal/.cargo/git/checkouts/tools-29b78d514589893f/763fd98/crates/rome_js_semantic/src/semantic_model/model.rs:332:19
Source Location: crates/rome_js_analyze/src/analyzers/complexity/no_multiple_spaces_in_regular_expression_literals.rs:129:41
Thread Name: rome::worker_1
Message: byte index 2 is not a char boundary; it is inside '\u{b8a08}' (bytes 1..5) of `/ from this software without specific prior written permission.`
List of files(I can easily produce more errors if needed):
- 763fd98 - 335 files - Rome_763fd9_335.zip
- e68674c - 2103 files - Broken_e68674c_2103.zip
Most of files is minimized, so contains max few lines of code
dayblox and denbezrukov
Metadata
Metadata
Assignees
Labels
S-Bug: confirmedStatus: report has been confirmed as a valid bugStatus: report has been confirmed as a valid bug