revert(imports): drop module.mergeImports (was never wired, will revisit when actually implemented)

Author: todor-a

README.md

@@ -46,7 +46,6 @@ This reorders imports across the import block into the listed groups and inserts
 |---|---|---|---|
 | `module.importGroups` | array | `[]` (off) | Ordered list of groups. Empty disables the feature. |
 | `module.typeImports` | `"separate"` \| `"interleave"` | `"separate"` | Whether `import type` lines form their own category. |
-| `module.mergeImports` | bool | `false` | Merge multiple imports from the same source (Biome-style). Currently detection only; emission TBD. |
 | `module.builtinsRuntime` | `"node"` \| `"deno"` \| `"bun"` \| `"none"` | `"node"` | Which runtime's built-in module list classifies as `builtin`. |
 
 ### Built-in categories
@@ -85,5 +84,4 @@ First-match-wins across the list, so position determines precedence.
 - Descending sort not supported.
 - TS `import X = require(...)` not reordered.
 - Imports inside nested `declare module "..."` bodies are not classified.
-- `module.mergeImports` is currently detection-only; merged emission TBD in a follow-up.
 - Currently, an import with `// dprint-ignore` is reordered like any other; barrier treatment is planned for a follow-up.

deployment/schema.json

@@ -1067,11 +1067,6 @@
       "default": "separate",
       "enum": ["separate", "interleave"]
     },
-    "module.mergeImports": {
-      "description": "Merge multiple imports from the same source into one declaration.",
-      "type": "boolean",
-      "default": false
-    },
     "module.builtinsRuntime": {
       "description": "Which runtime's built-in modules count as `builtin`.",
       "type": "string",

src/configuration/builder.rs

@@ -592,13 +592,6 @@ impl ConfigurationBuilder {
     self.insert("module.typeImports", value.to_string().into())
   }
 
-  /// Merge multiple imports from the same source into one declaration.
-  ///
-  /// Default: `false`
-  pub fn module_merge_imports(&mut self, value: bool) -> &mut Self {
-    self.insert("module.mergeImports", value.into())
-  }
-
   /// Which runtime's built-in modules count as `builtin`.
   ///
   /// Default: `Node`

src/configuration/resolve_config.rs

@@ -134,7 +134,6 @@ pub fn resolve_config(config: ConfigKeyMap, global_config: &GlobalConfiguration)
     ),
     module_import_groups: parse_import_groups(&mut config, &mut diagnostics),
     module_type_imports: get_value(&mut config, "module.typeImports", TypeImportsMode::Separate, &mut diagnostics),
-    module_merge_imports: get_value(&mut config, "module.mergeImports", false, &mut diagnostics),
     module_builtins_runtime: get_value(&mut config, "module.builtinsRuntime", BuiltinsRuntime::Node, &mut diagnostics),
     /* ignore comments */
     ignore_node_comment_text: get_value(&mut config, "ignoreNodeCommentText", String::from("dprint-ignore"), &mut diagnostics),
@@ -354,13 +353,6 @@ pub fn resolve_config(config: ConfigKeyMap, global_config: &GlobalConfiguration)
     }
   }
 
-  if resolved_config.module_merge_imports {
-    diagnostics.push(ConfigurationDiagnostic {
-      property_name: "module.mergeImports".to_string(),
-      message: "module.mergeImports is currently not implemented; setting it to true has no effect. Tracked for a future release.".to_string(),
-    });
-  }
-
   return ResolveConfigurationResult {
     config: resolved_config,
     diagnostics,
@@ -480,7 +472,6 @@ mod import_groups_resolution_tests {
     let r = resolve(serde_json::json!({}));
     assert!(r.config.module_import_groups.is_empty());
     assert!(matches!(r.config.module_type_imports, TypeImportsMode::Separate));
-    assert!(!r.config.module_merge_imports);
     assert!(matches!(r.config.module_builtins_runtime, BuiltinsRuntime::Node));
     assert!(r.diagnostics.is_empty());
   }

src/configuration/types.rs

@@ -440,8 +440,6 @@ pub struct Configuration {
   pub module_import_groups: Vec<ImportGroup>,
   #[serde(rename = "module.typeImports", default = "default_type_imports_mode")]
   pub module_type_imports: TypeImportsMode,
-  #[serde(rename = "module.mergeImports", default)]
-  pub module_merge_imports: bool,
   #[serde(rename = "module.builtinsRuntime", default = "default_builtins_runtime")]
   pub module_builtins_runtime: BuiltinsRuntime,
   /* ignore comments */

src/generation/imports/merge.rs

@@ -1,4 +1,3 @@
 pub mod classify;
 pub mod partition;
-pub mod merge;
 pub mod resolved;