feat: Named imports and exports are collapsed by default.

Author: dsherret

Cargo.lock

@@ -24,7 +24,7 @@ panic = "abort"
 wasm = ["serde_json", "dprint-core/wasm"]
 
 [dependencies]
-dprint-core = { version = "0.28.0", features = ["formatting"] }
+dprint-core = { version = "0.31.0", features = ["formatting"] }
 swc_common = "=0.10.2"
 swc_ecmascript = { version = "=0.7.7", features = ["parser"] }
 serde = { version = "1.0.88", features = ["derive"] }

Cargo.toml

@@ -40,7 +40,8 @@ pub fn resolve_config(config: ConfigKeyMap, global_config: &GlobalConfiguration)
     let trailing_commas = get_value(&mut config, "trailingCommas", TrailingCommas::OnlyMultiLine, &mut diagnostics);
     let use_braces = get_value(&mut config, "useBraces", UseBraces::WhenNotSingleLine, &mut diagnostics);
     let prefer_hanging = get_value(&mut config, "preferHanging", false, &mut diagnostics);
-    let prefer_single_line = get_value(&mut config, "preferSingleLine", false, &mut diagnostics);
+    let prefer_single_line_nullable = get_nullable_value(&mut config, "preferSingleLine", &mut diagnostics);
+    let prefer_single_line = prefer_single_line_nullable.unwrap_or(false);
     let type_literal_separator_kind = get_value(&mut config, "typeLiteral.separatorKind", SemiColonOrComma::SemiColon, &mut diagnostics);
 
     let resolved_config = Configuration {
@@ -151,9 +152,9 @@ pub fn resolve_config(config: ConfigKeyMap, global_config: &GlobalConfiguration)
         conditional_expression_prefer_single_line: get_value(&mut config, "conditionalExpression.preferSingleLine", prefer_single_line, &mut diagnostics),
         conditional_type_prefer_single_line: get_value(&mut config, "conditionalType.preferSingleLine", prefer_single_line, &mut diagnostics),
         decorators_prefer_single_line: get_value(&mut config, "decorators.preferSingleLine", prefer_single_line, &mut diagnostics),
-        export_declaration_prefer_single_line: get_value(&mut config, "exportDeclaration.preferSingleLine", prefer_single_line, &mut diagnostics),
+        export_declaration_prefer_single_line: get_value(&mut config, "exportDeclaration.preferSingleLine", prefer_single_line_nullable.unwrap_or(true), &mut diagnostics),
         for_statement_prefer_single_line: get_value(&mut config, "forStatement.preferSingleLine", prefer_single_line, &mut diagnostics),
-        import_declaration_prefer_single_line: get_value(&mut config, "importDeclaration.preferSingleLine", prefer_single_line, &mut diagnostics),
+        import_declaration_prefer_single_line: get_value(&mut config, "importDeclaration.preferSingleLine", prefer_single_line_nullable.unwrap_or(true), &mut diagnostics),
         jsx_attributes_prefer_single_line: get_value(&mut config, "jsxAttributes.preferSingleLine", prefer_single_line, &mut diagnostics),
         jsx_element_prefer_single_line: get_value(&mut config, "jsxElement.preferSingleLine", prefer_single_line, &mut diagnostics),
         mapped_type_prefer_single_line: get_value(&mut config, "mappedType.preferSingleLine", prefer_single_line, &mut diagnostics),

src/configuration/resolve_config.rs

@@ -63,3 +63,17 @@ export {
     test,
     testingThisOut,
 } from "testingtttttttttttttttttttttttttttttt";
+
+== should prefer single line by default ==
+export {
+    a
+} from "test";
+export {
+    b // test
+} from "test";
+
+[expect]
+export { a } from "test";
+export {
+    b, // test
+} from "test";

tests/specs/declarations/export/ExportNamedDeclaration_All.txt

@@ -12,19 +12,13 @@ export { n1, n2,   n3,   n4  , n5, n6} from   "test";
 export { n1, n2, n3, n4, n5,
     n6 } from "test";
 
-== should format with newlines if the first is on a different line ==
+== should not format with newlines if the first is on a different line since preferSingleLine is true by default for export decls ==
 export {
     n1, n2,   n3,   n4  , n5, n6} from   "test";
 
 [expect]
-export {
-    n1,
-    n2,
-    n3,
-    n4,
-    n5,
-    n6,
-} from "test";
+export { n1, n2, n3, n4, n5,
+    n6 } from "test";
 
 == should always do a newline as a unit ==
 export { test1, test2, test3 as alias } from "test";

tests/specs/declarations/export/NamedExports_PreferHanging_True.txt

@@ -1,24 +1,24 @@
 ~~ lineWidth: 40, exportDeclaration.trailingCommas: never ~~
 == should not use trailing commas when multi-line ==
 export {
-    n1, n2,
-    n3 } from "test";
+    testing1, testing2,
+    testing3 } from "test";
 
 [expect]
 export {
-    n1,
-    n2,
-    n3
+    testing1,
+    testing2,
+    testing3
 } from "test";
 
 == should sort the exports in alphabetical order taking into account the comma ==
 export {
-    n3, n2, n1
+    testing3, testing2, testing1
 } from "test";
 
 [expect]
 export {
-    n1,
-    n2,
-    n3
+    testing1,
+    testing2,
+    testing3
 } from "test";

tests/specs/declarations/export/NamedExports_TrailingCommas_Never.txt

@@ -92,3 +92,17 @@ import { d, c, B, a, E, f2, f, f1} from "test";
 
 [expect]
 import { a, B, c, d, E, f, f1, f2 } from "test";
+
+== should prefer single line by default ==
+import {
+    a
+} from "test";
+import {
+    b // test
+} from "test";
+
+[expect]
+import { a } from "test";
+import {
+    b, // test
+} from "test";

tests/specs/declarations/import/ImportDeclaration_All.txt

@@ -1,24 +1,24 @@
 ~~ lineWidth: 40, importDeclaration.trailingCommas: never ~~
 == should not use trailing commas when multi-line ==
 import {
-    n1, n2,
-    n3 } from "test";
+    testing1, testing2,
+    testing3 } from "test";
 
 [expect]
 import {
-    n1,
-    n2,
-    n3
+    testing1,
+    testing2,
+    testing3
 } from "test";
 
 == should sort the imports in alphabetical order taking into account the comma ==
 import {
-    n3, n2, n1
+    testing3, testing2, testing1
 } from "test";
 
 [expect]
 import {
-    n1,
-    n2,
-    n3
+    testing1,
+    testing2,
+    testing3
 } from "test";

tests/specs/declarations/import/NamedImports_TrailingCommas_Never.txt

@@ -0,0 +1,14 @@
+~~ deno: true ~~
+== should collapse imports and exports when able ==
+import {
+    testing,
+    thisOut,
+} from "test.ts";
+export {
+    testing,
+    thisOut,
+} from "test.ts";
+
+[expect]
+import { testing, thisOut } from "test.ts";
+export { testing, thisOut } from "test.ts";