fix(rome_service): correctly load recommended rules

CHANGELOG.md

@@ -10,13 +10,18 @@
 
 
 ### Configuration
+
+#### Other changes
+
+- Fix an issue where Rome was loading incorrectly recommended rule [#4479](https://github.com/rome/tools/issues/4479) [#4488](https://github.com/rome/tools/issues/4488)
+
 ### Editors
 ### Formatter
 ### Linter
 
 #### Other changes
 
-- Fixed an issue where the `noAssignInExpressions` rule replaced the operator with an invalid token, which caused other lint rules to crash. [#4464](https://github.com/rome/tools/issues/4464)
+- Fix an issue where the `noAssignInExpressions` rule replaced the operator with an invalid token, which caused other lint rules to crash. [#4464](https://github.com/rome/tools/issues/4464)
 - Fix an issue that [`noUnusedVariables`](https://docs.rome.tools/lint/rules/nounusedvariables/) rule did not correctly detect exports when a variable and an `interface` had the same name [#4468](https://github.com/rome/tools/pull/4468)
 
 ### Parser

crates/rome_cli/tests/commands/check.rs

@@ -2112,3 +2112,61 @@ fn apply_unsafe_no_assign_in_expression() {
         result,
     ));
 }
+
+#[test]
+fn should_not_enable_all_recommended_rules() {
+    let mut fs = MemoryFileSystem::default();
+    let mut console = BufferConsole::default();
+
+    let configuration = r#"	{
+		"$schema": "https://docs.rome.tools/schemas/12.1.0/schema.json",
+		"organizeImports": {
+		"enabled": false
+	},
+		"linter": {
+		"enabled": true,
+		"rules": {
+			"recommended": false,
+			"a11y": {},
+			"complexity": {},
+			"correctness": {},
+			"performance": {},
+			"security": {},
+			"style": {},
+			"suspicious": {}
+		}
+	}
+	}"#;
+
+    let configuration_path = Path::new("rome.json");
+    fs.insert(configuration_path.into(), configuration.as_bytes());
+
+    let file_path = Path::new("fix.js");
+    fs.insert(
+        file_path.into(),
+        r#"
+    		LOOP: for (const x of xs) {
+    		    if (x > 0) {
+    		        break;
+    		    }
+    		    f(x);
+    		}
+		"#,
+    );
+
+    let result = run_cli(
+        DynRef::Borrowed(&mut fs),
+        &mut console,
+        Args::from(&[("check"), file_path.as_os_str().to_str().unwrap()]),
+    );
+
+    assert!(result.is_ok(), "run_cli returned {result:?}");
+
+    assert_cli_snapshot(SnapshotPayload::new(
+        module_path!(),
+        "should_not_enable_all_recommended_rules",
+        fs,
+        console,
+        result,
+    ));
+}

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

@@ -0,0 +1,48 @@
+---
+source: crates/rome_cli/tests/snap_test.rs
+expression: content
+---
+## `rome.json`
+
+```json
+{
+  "$schema": "https://docs.rome.tools/schemas/12.1.0/schema.json",
+  "organizeImports": {
+    "enabled": false
+  },
+  "linter": {
+    "enabled": true,
+    "rules": {
+      "recommended": false,
+      "a11y": {},
+      "complexity": {},
+      "correctness": {},
+      "performance": {},
+      "security": {},
+      "style": {},
+      "suspicious": {}
+    }
+  }
+}
+```
+
+## `fix.js`
+
+```js
+
+    		LOOP: for (const x of xs) {
+    		    if (x > 0) {
+    		        break;
+    		    }
+    		    f(x);
+    		}
+
+```
+
+# Emitted Messages
+
+```block
+Checked 1 file(s) in <TIME>
+```
+
+