Fix last remaining test issues

Author: shanshrew

packages/swc-plugin/src/lib.rs

@@ -415,7 +415,7 @@ impl CompiledTransform {
 
         // Check what types of React imports already exist
         let mut has_react_namespace_or_default = false;
-        let mut has_react_named_imports = false;
+        let mut _has_react_named_imports = false;
         let mut has_forwardref_import = false;
 
         for item in &module.body {
@@ -427,7 +427,7 @@ impl CompiledTransform {
                                 has_react_namespace_or_default = true;
                             }
                             ImportSpecifier::Named(named) => {
-                                has_react_named_imports = true;
+                                _has_react_named_imports = true;
                                 if named.local.sym.as_ref() == "forwardRef" {
                                     has_forwardref_import = true;
                                 }

packages/swc-plugin/src/utils/module_resolver.rs

@@ -133,6 +133,7 @@ impl ModuleResolver {
     /// # Examples
     /// 
     /// ```rust
+    /// use compiled_swc::utils::module_resolver::ModuleResolver;
     /// let resolver = ModuleResolver::new("src/components");
     /// // Relative imports like "./colors" will be resolved as "src/components/colors"
     /// ```
@@ -161,6 +162,7 @@ impl ModuleResolver {
     /// # Examples
     /// 
     /// ```rust
+    /// use compiled_swc::utils::module_resolver::ModuleResolver;
     /// let mut resolver = ModuleResolver::new("src");
     /// 
     /// // Resolve a relative import
@@ -409,6 +411,7 @@ impl ModuleResolver {
     /// # Examples
     /// 
     /// ```rust
+    /// use compiled_swc::utils::module_resolver::{ModuleResolver, ExportValue};
     /// let mut resolver = ModuleResolver::new("src");
     /// 
     /// // Get a named export: export const primaryColor = "red"
@@ -445,6 +448,7 @@ impl ModuleResolver {
     /// # Examples
     /// 
     /// ```rust
+    /// use compiled_swc::utils::module_resolver::{ModuleResolver, ExportValue};
     /// let mut resolver = ModuleResolver::new("src");
     /// 
     /// // Get default export: export default { theme: "dark" }

packages/swc-plugin/src/utils/variable_context.rs

@@ -47,11 +47,6 @@ pub fn build_variable_context_from_module_with_mutations(module: &Module, mutate
     context
 }
 
-/// Extract variable declarations from a statement
-fn extract_variable_declarations_from_stmt(stmt: &ModuleItem, context: &mut VariableContext) {
-    extract_variable_declarations_from_stmt_with_mutations(stmt, context, &std::collections::HashSet::new())
-}
-
 /// Extract variable declarations from a statement, excluding mutated variables
 fn extract_variable_declarations_from_stmt_with_mutations(stmt: &ModuleItem, context: &mut VariableContext, mutated_variables: &std::collections::HashSet<String>) {
     match stmt {
@@ -62,11 +57,6 @@ fn extract_variable_declarations_from_stmt_with_mutations(stmt: &ModuleItem, con
     }
 }
 
-/// Extract variable declarations from a statement (inner)
-fn extract_variable_declarations_from_stmt_inner(stmt: &Stmt, context: &mut VariableContext) {
-    extract_variable_declarations_from_stmt_inner_with_mutations(stmt, context, &std::collections::HashSet::new())
-}
-
 /// Extract variable declarations from a statement (inner), excluding mutated variables
 fn extract_variable_declarations_from_stmt_inner_with_mutations(stmt: &Stmt, context: &mut VariableContext, mutated_variables: &std::collections::HashSet<String>) {
     match stmt {
@@ -97,7 +87,6 @@ fn extract_variable_declarations_from_stmt_inner_with_mutations(stmt: &Stmt, con
 #[cfg(test)]
 mod tests {
     use super::*;
-    use swc_core::ecma::ast::*;
     use swc_core::common::DUMMY_SP;
 
     #[test]

packages/swc-plugin/tests/class_names_test.rs

@@ -1,4 +1,4 @@
-use compiled_swc::{test_utils::{transform_with_compiled, TestTransformOptions, TestAssertions}, assert_includes, assert_includes_multiple};
+use compiled_swc::{test_utils::{transform_with_compiled, TestTransformOptions, TestAssertions}, assert_includes_multiple};
 
 #[cfg(test)]
 mod class_names_tests {

packages/swc-plugin/tests/css_map_test.rs

@@ -1,4 +1,4 @@
-use compiled_swc::{test_utils::{transform_with_compiled, TestTransformOptions, TestAssertions}, assert_includes, assert_includes_multiple};
+use compiled_swc::{test_utils::{transform_with_compiled, TestTransformOptions, TestAssertions}, assert_includes_multiple};
 
 #[cfg(test)]
 mod css_map_tests {

packages/swc-plugin/tests/custom_import_test.rs

@@ -3,7 +3,7 @@
 // exactly like the babel plugin, including @atlaskit/css automatic detection
 // and proper error handling for misconfigured sources.
 
-use compiled_swc::{test_utils::{transform_with_compiled, TestTransformOptions, TestAssertions}, assert_includes, assert_includes_multiple};
+use compiled_swc::{test_utils::{transform_with_compiled, TestTransformOptions, TestAssertions}, assert_includes};
 
 #[cfg(test)]
 mod custom_import_tests {

packages/swc-plugin/tests/jsx_automatic_test.rs

@@ -1,4 +1,4 @@
-use compiled_swc::{test_utils::{transform_with_compiled, TestTransformOptions, TestAssertions}, assert_includes, assert_includes_multiple};
+use compiled_swc::{test_utils::{transform_with_compiled, TestTransformOptions, TestAssertions}, assert_includes_multiple};
 
 #[cfg(test)]
 mod jsx_automatic_tests {

packages/swc-plugin/tests/keyframes_test.rs

@@ -1,4 +1,4 @@
-use compiled_swc::{test_utils::{transform_with_compiled, TestTransformOptions, TestAssertions}, assert_includes, assert_includes_multiple};
+use compiled_swc::{test_utils::{transform_with_compiled, TestTransformOptions, TestAssertions}, assert_includes_multiple};
 
 #[cfg(test)]
 mod keyframes_tests {

packages/swc-plugin/tests/styled_components_test.rs

@@ -1,4 +1,4 @@
-use compiled_swc::{test_utils::{transform_with_compiled, TestTransformOptions, TestAssertions}, assert_includes, assert_includes_multiple};
+use compiled_swc::{test_utils::{transform_with_compiled, TestTransformOptions, TestAssertions}, assert_includes_multiple};
 
 #[cfg(test)]
 mod styled_components_tests {