rome · ematipico · Jan 10, 2023 · Jan 3, 2023 · Jan 3, 2023 · Jan 3, 2023
diff --git a/Cargo.lock b/Cargo.lock
diff --git a/crates/rome_cli/tests/configs.rs b/crates/rome_cli/tests/configs.rs
@@ -9,23 +9,25 @@ pub const CONFIG_FORMAT: &str = r#"{
 "#;
 
 pub const CONFIG_INIT_DEFAULT: &str = r#"{
-  "linter": {
-    "enabled": true,
-    "rules": {
-      "recommended": true
-    }
-  }
-}"#;
+	"linter": {
+		"enabled": true,
+		"rules": {
+			"recommended": true
+		}
+	}
+}
+"#;
 
 pub const CONFIG_INIT_DEFAULT_WHEN_INSTALLED: &str = r#"{
-  "$schema": "./node_modules/rome/configuration_schema.json",
-  "linter": {
-    "enabled": true,
-    "rules": {
-      "recommended": true
-    }
-  }
-}"#;
+	"$schema": "./node_modules/rome/configuration_schema.json",
+	"linter": {
+		"enabled": true,
+		"rules": {
+			"recommended": true
+		}
+	}
+}
+"#;
 
 pub const CONFIG_DISABLED_FORMATTER: &str = r#"{
   "formatter": {

diff --git a/crates/rome_cli/tests/snapshots/main_commands_init/creates_config_file.snap b/crates/rome_cli/tests/snapshots/main_commands_init/creates_config_file.snap
@@ -6,13 +6,14 @@ expression: content
 
 ```json
 {
-  "linter": {
-    "enabled": true,
-    "rules": {
-      "recommended": true
-    }
-  }
+	"linter": {
+		"enabled": true,
+		"rules": {
+			"recommended": true
+		}
+	}
 }
+
 ```
 
 # Emitted Messages

diff --git a/...shots/main_commands_init/creates_config_file_when_rome_installed_via_package_manager.snap b/...shots/main_commands_init/creates_config_file_when_rome_installed_via_package_manager.snap
@@ -6,14 +6,15 @@ expression: content
 
 ```json
 {
-  "$schema": "./node_modules/rome/configuration_schema.json",
-  "linter": {
-    "enabled": true,
-    "rules": {
-      "recommended": true
-    }
-  }
+	"$schema": "./node_modules/rome/configuration_schema.json",
+	"linter": {
+		"enabled": true,
+		"rules": {
+			"recommended": true
+		}
+	}
 }
+
 ```
 
 ## `./node_modules/rome/configuration_schema.json`

diff --git a/crates/rome_fs/src/fs/os.rs b/crates/rome_fs/src/fs/os.rs
@@ -257,6 +257,13 @@ fn handle_dir_entry<'scope>(
     }
 
     if file_type.is_file() {
+        if matches!(
+            path.file_name().and_then(OsStr::to_str),
+            Some("package.json" | "package-lock.json" | "tsconfig.json")
+        ) {
+            return;
+        }
+
         // Performing this check here let's us skip skip unsupported
         // files entirely, as well as silently ignore unsupported files when
         // doing a directory traversal, but printing an error message if the

diff --git a/crates/rome_service/src/configuration/mod.rs b/crates/rome_service/src/configuration/mod.rs
@@ -32,6 +32,8 @@ pub use javascript::{JavascriptConfiguration, JavascriptFormatter};
 pub use linter::{LinterConfiguration, RuleConfiguration, Rules};
 use rome_analyze::{AnalyzerConfiguration, AnalyzerRules, MetadataRegistry};
 use rome_js_analyze::metadata;
+use rome_json_formatter::context::JsonFormatOptions;
+use rome_json_parser::parse_json;
 use rome_json_syntax::JsonRoot;
 
 /// The configuration that is contained inside the file `rome.json`
@@ -209,8 +211,14 @@ pub fn create_config(
         WorkspaceError::Configuration(ConfigurationDiagnostic::new_serialization_error())
     })?;
 
+    let parsed = parse_json(&contents);
+    let formatted =
+        rome_json_formatter::format_node(JsonFormatOptions::default(), &parsed.syntax())?
+            .print()
+            .expect("valid format document");
+
     config_file
-        .set_content(contents.as_bytes())
+        .set_content(formatted.as_code().as_bytes())
         .map_err(|_| WorkspaceError::cant_read_file(format!("{}", path.display())))?;
 
     Ok(())

diff --git a/crates/rome_service/src/file_handlers/json.rs b/crates/rome_service/src/file_handlers/json.rs
@@ -6,14 +6,12 @@ use crate::settings::{
 };
 use crate::workspace::GetSyntaxTreeResult;
 use crate::WorkspaceError;
-#[cfg(any(debug_assertions, target_family = "wasm"))]
 use rome_formatter::{FormatError, Printed};
 use rome_fs::RomePath;
 use rome_json_formatter::context::JsonFormatOptions;
 use rome_json_formatter::format_node;
 use rome_json_syntax::{JsonLanguage, JsonRoot, JsonSyntaxNode};
 use rome_parser::AnyParse;
-#[cfg(any(debug_assertions, target_family = "wasm"))]
 use rome_rowan::{TextRange, TextSize, TokenAtOffset};
 
 impl Language for JsonLanguage {
@@ -61,25 +59,15 @@ impl ExtensionHandler for JsonFileHandler {
                 debug_formatter_ir: Some(debug_formatter_ir),
             },
             analyzer: Default::default(),
-            formatter: formatter_capabilities(),
+            formatter: FormatterCapabilities {
+                format: Some(format),
+                format_range: Some(format_range),
+                format_on_type: Some(format_on_type),
+            },
         }
     }
 }
 
-#[cfg(any(debug_assertions, target_family = "wasm"))]
-fn formatter_capabilities() -> FormatterCapabilities {
-    FormatterCapabilities {
-        format: Some(format),
-        format_range: Some(format_range),
-        format_on_type: Some(format_on_type),
-    }
-}
-
-#[cfg(all(not(debug_assertions), not(target_family = "wasm")))]
-fn formatter_capabilities() -> FormatterCapabilities {
-    FormatterCapabilities::default()
-}
-
 fn parse(_: &RomePath, _: LanguageId, text: &str) -> AnyParse {
     let parse = rome_json_parser::parse_json(text);
     AnyParse::from(parse)
@@ -108,7 +96,6 @@ fn debug_formatter_ir(
     Ok(root_element.to_string())
 }
 
-#[cfg(any(debug_assertions, target_family = "wasm"))]
 #[tracing::instrument(level = "debug", skip(parse))]
 fn format(
     rome_path: &RomePath,
@@ -128,7 +115,6 @@ fn format(
     }
 }
 
-#[cfg(any(debug_assertions, target_family = "wasm"))]
 fn format_range(
     rome_path: &RomePath,
     parse: AnyParse,
@@ -142,7 +128,6 @@ fn format_range(
     Ok(printed)
 }
 
-#[cfg(any(debug_assertions, target_family = "wasm"))]
 fn format_on_type(
     rome_path: &RomePath,
     parse: AnyParse,