refactor(ast): remove JsImportNamespaceClause

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

@@ -105,7 +105,6 @@ impl Rule for OrganizeImports {
                 AnyJsImportClause::JsImportBareClause(clause) => clause.source().ok()?,
                 AnyJsImportClause::JsImportDefaultClause(clause) => clause.source().ok()?,
                 AnyJsImportClause::JsImportNamedClause(clause) => clause.source().ok()?,
-                AnyJsImportClause::JsImportNamespaceClause(clause) => clause.source().ok()?,
             };
 
             let key = source.inner_string_text().ok()?;
@@ -338,8 +337,7 @@ impl From<JsImport> for ImportNode {
         let specifiers = import_clause.and_then(|import_clause| {
             let import_named_clause = match import_clause {
                 AnyJsImportClause::JsImportBareClause(_)
-                | AnyJsImportClause::JsImportDefaultClause(_)
-                | AnyJsImportClause::JsImportNamespaceClause(_) => return None,
+                | AnyJsImportClause::JsImportDefaultClause(_) => return None,
                 AnyJsImportClause::JsImportNamedClause(import_clause) => import_clause,
             };
 

crates/biome_js_analyze/src/semantic_analyzers/correctness/no_unused_variables.rs

@@ -192,7 +192,6 @@ fn suggested_fix_if_unused(binding: &AnyJsIdentifierBinding) -> Option<Suggested
         AnyJsBindingDeclaration::JsBogusParameter(_)
         // Imports are never ok to be unused
         | AnyJsBindingDeclaration::JsImportDefaultClause(_)
-        | AnyJsBindingDeclaration::JsImportNamespaceClause(_)
         | AnyJsBindingDeclaration::JsShorthandNamedImportSpecifier(_)
         | AnyJsBindingDeclaration::JsNamedImportSpecifier(_)
         | AnyJsBindingDeclaration::JsBogusNamedImportSpecifier(_)

crates/biome_js_analyze/src/semantic_analyzers/nursery/use_exhaustive_dependencies.rs

@@ -482,7 +482,6 @@ fn capture_needs_to_be_in_the_dependency_list(
         // This should not be unreachable because of the test
         // if the capture is imported
         AnyJsBindingDeclaration::JsImportDefaultClause(_)
-        | AnyJsBindingDeclaration::JsImportNamespaceClause(_)
         | AnyJsBindingDeclaration::JsShorthandNamedImportSpecifier(_)
         | AnyJsBindingDeclaration::JsNamedImportSpecifier(_)
         | AnyJsBindingDeclaration::JsBogusNamedImportSpecifier(_)

crates/biome_js_analyze/src/semantic_analyzers/style/use_naming_convention.rs

@@ -794,8 +794,7 @@ impl Named {
             AnyJsBindingDeclaration::JsCatchDeclaration(_) => Some(Named::CatchParameter),
             AnyJsBindingDeclaration::TsPropertyParameter(_) => Some(Named::ParameterProperty),
             AnyJsBindingDeclaration::TsIndexSignatureParameter(_) => Some(Named::IndexParameter),
-            AnyJsBindingDeclaration::JsNamespaceImportSpecifier(_)
-            | AnyJsBindingDeclaration::JsImportNamespaceClause(_) => Some(Named::ImportNamespace),
+            AnyJsBindingDeclaration::JsNamespaceImportSpecifier(_) => Some(Named::ImportNamespace),
             AnyJsBindingDeclaration::JsFunctionDeclaration(_)
             | AnyJsBindingDeclaration::JsFunctionExpression(_)
             | AnyJsBindingDeclaration::JsFunctionExportDefaultDeclaration(_)

crates/biome_js_analyze/src/semantic_analyzers/suspicious/no_import_assign.rs

@@ -4,8 +4,7 @@ use biome_console::markup;
 use biome_js_semantic::ReferencesExtensions;
 use biome_js_syntax::{
     JsDefaultImportSpecifier, JsIdentifierAssignment, JsIdentifierBinding, JsImportDefaultClause,
-    JsImportNamespaceClause, JsNamedImportSpecifier, JsNamespaceImportSpecifier,
-    JsShorthandNamedImportSpecifier,
+    JsNamedImportSpecifier, JsNamespaceImportSpecifier, JsShorthandNamedImportSpecifier,
 };
 
 use biome_rowan::{declare_node_union, AstNode};
@@ -69,8 +68,6 @@ impl Rule for NoImportAssign {
         let local_name_binding = match label_statement {
             // `import xx from 'y'`
             AnyJsImportLike::JsImportDefaultClause(clause) => clause.local_name().ok(),
-            // `import * as xxx from 'y'`
-            AnyJsImportLike::JsImportNamespaceClause(clause) => clause.local_name().ok(),
             // `import {x as xx} from 'y'`
             //          ^^^^^^^
             AnyJsImportLike::JsNamedImportSpecifier(specifier) => specifier.local_name().ok(),
@@ -125,5 +122,5 @@ impl Rule for NoImportAssign {
 }
 
 declare_node_union! {
-    pub(crate) AnyJsImportLike = JsImportDefaultClause | JsImportNamespaceClause | JsNamedImportSpecifier | JsShorthandNamedImportSpecifier | JsNamespaceImportSpecifier | JsDefaultImportSpecifier
+    pub(crate) AnyJsImportLike = JsImportDefaultClause | JsNamedImportSpecifier | JsShorthandNamedImportSpecifier | JsNamespaceImportSpecifier | JsDefaultImportSpecifier
 }

crates/biome_js_formatter/src/generated.rs

@@ -6321,46 +6321,6 @@ impl IntoFormat<JsFormatContext> for biome_js_syntax::JsImportDefaultClause {
         )
     }
 }
-impl FormatRule<biome_js_syntax::JsImportNamespaceClause>
-    for crate::js::module::import_namespace_clause::FormatJsImportNamespaceClause
-{
-    type Context = JsFormatContext;
-    #[inline(always)]
-    fn fmt(
-        &self,
-        node: &biome_js_syntax::JsImportNamespaceClause,
-        f: &mut JsFormatter,
-    ) -> FormatResult<()> {
-        FormatNodeRule::<biome_js_syntax::JsImportNamespaceClause>::fmt(self, node, f)
-    }
-}
-impl AsFormat<JsFormatContext> for biome_js_syntax::JsImportNamespaceClause {
-    type Format<'a> = FormatRefWithRule<
-        'a,
-        biome_js_syntax::JsImportNamespaceClause,
-        crate::js::module::import_namespace_clause::FormatJsImportNamespaceClause,
-    >;
-    fn format(&self) -> Self::Format<'_> {
-        #![allow(clippy::default_constructed_unit_structs)]
-        FormatRefWithRule::new(
-            self,
-            crate::js::module::import_namespace_clause::FormatJsImportNamespaceClause::default(),
-        )
-    }
-}
-impl IntoFormat<JsFormatContext> for biome_js_syntax::JsImportNamespaceClause {
-    type Format = FormatOwnedWithRule<
-        biome_js_syntax::JsImportNamespaceClause,
-        crate::js::module::import_namespace_clause::FormatJsImportNamespaceClause,
-    >;
-    fn into_format(self) -> Self::Format {
-        #![allow(clippy::default_constructed_unit_structs)]
-        FormatOwnedWithRule::new(
-            self,
-            crate::js::module::import_namespace_clause::FormatJsImportNamespaceClause::default(),
-        )
-    }
-}
 impl FormatRule<biome_js_syntax::JsModuleSource>
     for crate::js::module::module_source::FormatJsModuleSource
 {

crates/biome_js_formatter/src/js/any/import_clause.rs

@@ -11,7 +11,6 @@ impl FormatRule<AnyJsImportClause> for FormatAnyJsImportClause {
             AnyJsImportClause::JsImportBareClause(node) => node.format().fmt(f),
             AnyJsImportClause::JsImportNamedClause(node) => node.format().fmt(f),
             AnyJsImportClause::JsImportDefaultClause(node) => node.format().fmt(f),
-            AnyJsImportClause::JsImportNamespaceClause(node) => node.format().fmt(f),
         }
     }
 }

crates/biome_js_formatter/src/js/module/mod.rs

@@ -17,7 +17,6 @@ pub(crate) mod import_assertion_entry;
 pub(crate) mod import_bare_clause;
 pub(crate) mod import_default_clause;
 pub(crate) mod import_named_clause;
-pub(crate) mod import_namespace_clause;
 pub(crate) mod literal_export_name;
 pub(crate) mod module_source;
 pub(crate) mod named_import_specifier;

crates/biome_js_parser/src/syntax/module.rs

@@ -280,8 +280,7 @@ fn parse_import_clause(p: &mut JsParser) -> ParsedSyntax {
     }
 
     let clause = match p.cur() {
-        T![*] => parse_import_namespace_clause_rest(p, m),
-        T!['{'] => parse_import_named_clause_rest(p, m),
+        T![*] | T!['{'] => parse_import_named_clause_rest(p, m),
         _ if is_at_identifier_binding(p) => {
             parse_identifier_binding(p).unwrap();
             parse_import_default_or_named_clause_rest(p, m, is_typed)
@@ -358,20 +357,6 @@ fn parse_import_bare_clause(p: &mut JsParser) -> ParsedSyntax {
     })
 }
 
-// test js import_decl
-// import * as foo from "bla";
-fn parse_import_namespace_clause_rest(p: &mut JsParser, m: Marker) -> CompletedMarker {
-    p.expect(T![*]);
-
-    p.expect(T![as]);
-    parse_binding(p).or_add_diagnostic(p, expected_binding);
-    p.expect(T![from]);
-    parse_module_source(p).or_add_diagnostic(p, expected_module_source);
-    parse_import_assertion(p).ok();
-
-    m.complete(p, JS_IMPORT_NAMESPACE_CLAUSE)
-}
-
 // test js import_named_clause
 // import {} from "a";
 // import { a, b, c, } from "b";