fix(organizeImports): add space where needed (#2602)

CHANGELOG.md

@@ -37,6 +37,9 @@ our [guidelines for writing a good changelog entry](https://github.com/biomejs/b
 
   Contributed by @Conaclos
 
+- Import sorting now adds spaces where needed ([#1665](https://github.com/biomejs/biome/issues/1665))
+  Contributed by @Conaclos
+
 ### CLI
 
 #### Bug fixes

crates/biome_js_analyze/src/assists/correctness/organize_imports.rs

@@ -448,7 +448,14 @@ impl ImportNode {
                     // If the node we're attaching this separator to has no trailing trivia, just create a simple comma token
                     let last_trailing_trivia = match node.syntax().last_trailing_trivia() {
                         Some(trivia) if !trivia.is_empty() => trivia,
-                        _ => return make::token(T![,]),
+                        _ => {
+                            let sep = make::token(T![,]);
+                            return if node.syntax().has_leading_newline() {
+                                sep
+                            } else {
+                                sep.with_trailing_trivia([(TriviaPieceKind::Whitespace, " ")])
+                            };
+                        }
                     };
 
                     // Otherwise we need to clone the trailing trivia from the node to the separator

crates/biome_js_analyze/tests/specs/correctness/organizeImports/space.ts

@@ -0,0 +1 @@
+import {type ContactOption, /* 1 */ loadStripe, /* 2 */ type StripeElementsOptions} from '@stripe/stripe-js';

crates/biome_js_analyze/tests/specs/correctness/organizeImports/space.ts.snap

@@ -0,0 +1,18 @@
+---
+source: crates/biome_js_analyze/tests/spec_tests.rs
+expression: space.ts
+---
+# Input
+```ts
+import {type ContactOption, /* 1 */ loadStripe, /* 2 */ type StripeElementsOptions} from '@stripe/stripe-js';
+```
+
+# Actions
+```diff
+@@ -1 +1 @@
+-import {type ContactOption, /* 1 */ loadStripe, /* 2 */ type StripeElementsOptions} from '@stripe/stripe-js';
+\ No newline at end of file
++import {type ContactOption, /* 1 */ type StripeElementsOptions, loadStripe /* 2 */ } from '@stripe/stripe-js';
+\ No newline at end of file
+
+```