chore: merge

crates/rome_cli/tests/commands/check.rs

@@ -512,7 +512,7 @@ fn downgrade_severity() {
             .filter(|m| m.level == LogLevel::Error)
             .filter(|m| {
                 let content = format!("{:#?}", m.content);
-                content.contains("security/noDebugger")
+                content.contains("correctness/noDebugger")
             })
             .count(),
         1

crates/rome_cli/tests/configs.rs

@@ -100,7 +100,7 @@ pub const CONFIG_LINTER_SUPPRESSED_RULE: &str = r#"{
   "linter": {
     "rules": {
         "recommended": true,
-        "security": {
+        "correctness": {
             "noDebugger": "off"
         }
     }
@@ -122,8 +122,7 @@ pub const CONFIG_LINTER_DOWNGRADE_DIAGNOSTIC: &str = r#"{
   "linter": {
     "rules": {
         "recommended": true,
-        "security": {
-            "recommended": true,      
+        "correctness": {
             "noDebugger": "warn"
         }
     }

crates/rome_cli/tests/snapshots/main_commands_check/downgrade_severity.snap

@@ -9,8 +9,7 @@ expression: content
   "linter": {
     "rules": {
         "recommended": true,
-        "security": {
-            "recommended": true,      
+        "correctness": {
             "noDebugger": "warn"
         }
     }
@@ -27,7 +26,7 @@ debugger;
 # Emitted Messages
 
 ```block
-file.js:1:1 lint/security/noDebugger  FIXABLE  ━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━
+file.js:1:1 lint/correctness/noDebugger  FIXABLE  ━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━
 
   ! This is an unexpected use of the debugger statement.
   

crates/rome_cli/tests/snapshots/main_commands_check/should_disable_a_rule.snap

@@ -9,7 +9,7 @@ expression: content
   "linter": {
     "rules": {
         "recommended": true,
-        "security": {
+        "correctness": {
             "noDebugger": "off"
         }
     }

crates/rome_cli/tests/snapshots/main_commands_check/upgrade_severity.snap

@@ -1,6 +1,5 @@
 ---
 source: crates/rome_cli/tests/snap_test.rs
-assertion_line: 223
 expression: content
 ---
 ## `rome.json`
@@ -11,8 +10,7 @@ expression: content
     "rules": {
         "recommended": true,
         "nursery": {
-            "noUnreachable": "error",
-            "noUnusedVariables": "error"
+            "noInvalidConstructorSuper": "error"
         }
     }
   }
@@ -22,13 +20,9 @@ expression: content
 ## `file.js`
 
 ```js
-function f() {
-    for (;;) {
-        continue;
-        break;
-    }
+class A extends B {
+    constructor() {}
 }
-
 ```
 
 # Termination Message
@@ -40,40 +34,14 @@ errors where emitted while running checks
 # Emitted Messages
 
 ```block
-file.js:4:9 lint/nursery/noUnreachable ━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━
+file.js:1:9 lint/nursery/noInvalidConstructorSuper ━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━
 
-  × This code will never be reached ...
-  
-    2 │     for (;;) {
-    3 │         continue;
-  > 4 │         break;
-      │         ^^^^^^
-    5 │     }
-    6 │ }
+  × This class extends another class and a super() call is expected.
   
-  i ... because this statement will continue the loop beforehand
-  
-    1 │ function f() {
-    2 │     for (;;) {
-  > 3 │         continue;
+  > 1 │ class A extends B {
       │         ^^^^^^^^^
-    4 │         break;
-    5 │     }
-  
-
-```
-
-```block
-file.js:1:10 lint/nursery/noUnusedVariables ━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━
-
-  × This function is unused.
-  
-  > 1 │ function f() {
-      │          ^
-    2 │     for (;;) {
-    3 │         continue;
-  
-  i Unused variables usually are result of incomplete refactoring, typos and other source of bugs.
+    2 │     constructor() {}
+    3 │ }
   
 
 ```