Merge branch 'main' into don/07-12-feat_linter_support_suggestions_an…

…d_dangerous_fixes

Cargo.toml

@@ -192,6 +192,11 @@ ignored = ["napi", "oxc_transform_napi", "prettyplease"]
 # and we don't rely on it for debugging that much.
 debug = false
 
+[profile.dev.package]
+# Compile macros with some optimizations to make consuming crates build faster
+oxc_macros.opt-level     = 1
+oxc_ast_macros.opt-level = 1
+
 [profile.release.package.oxc_wasm]
 opt-level = 'z'
 

crates/oxc_ast/src/generated/visit.rs

@@ -27,10 +27,14 @@ use walk::*;
 
 /// Syntax tree traversal
 pub trait Visit<'a>: Sized {
+    #[inline]
     fn enter_node(&mut self, kind: AstKind<'a>) {}
+    #[inline]
     fn leave_node(&mut self, kind: AstKind<'a>) {}
 
+    #[inline]
     fn enter_scope(&mut self, flags: ScopeFlags, scope_id: &Cell<Option<ScopeId>>) {}
+    #[inline]
     fn leave_scope(&mut self) {}
 
     #[inline]

crates/oxc_ast/src/generated/visit_mut.rs

@@ -27,10 +27,14 @@ use walk_mut::*;
 
 /// Syntax tree traversal
 pub trait VisitMut<'a>: Sized {
+    #[inline]
     fn enter_node(&mut self, kind: AstType) {}
+    #[inline]
     fn leave_node(&mut self, kind: AstType) {}
 
+    #[inline]
     fn enter_scope(&mut self, flags: ScopeFlags, scope_id: &Cell<Option<ScopeId>>) {}
+    #[inline]
     fn leave_scope(&mut self) {}
 
     #[inline]

crates/oxc_linter/src/rules.rs

@@ -21,6 +21,7 @@ mod import {
     pub mod no_named_as_default_member;
     pub mod no_self_import;
     // pub mod no_unused_modules;
+    pub mod no_webpack_loader_syntax;
 }
 
 mod eslint {
@@ -741,6 +742,7 @@ oxc_macros::declare_all_lint_rules! {
     // import::no_unused_modules,
     import::no_duplicates,
     import::no_default_export,
+    import::no_webpack_loader_syntax,
     jsx_a11y::alt_text,
     jsx_a11y::anchor_has_content,
     jsx_a11y::anchor_is_valid,

crates/oxc_linter/src/rules/import/no_webpack_loader_syntax.rs

@@ -0,0 +1,118 @@
+use oxc_ast::{
+    ast::{Argument, Expression},
+    AstKind,
+};
+use oxc_diagnostics::OxcDiagnostic;
+use oxc_macros::declare_oxc_lint;
+use oxc_semantic::AstNode;
+use oxc_span::Span;
+
+use crate::{context::LintContext, rule::Rule};
+
+fn no_named_as_default_diagnostic(x0: &str, span0: Span) -> OxcDiagnostic {
+    OxcDiagnostic::warn(format!(
+        "eslint-plugin-import(no-webpack-loader-syntax): Unexpected `!` in `{x0}`."
+    ))
+    .with_help("Do not use import syntax to configure webpack loaders")
+    .with_label(span0)
+}
+
+#[derive(Debug, Default, Clone)]
+pub struct NoWebpackLoaderSyntax;
+
+declare_oxc_lint!(
+    /// ### What it does
+    ///
+    /// Forbid Webpack loader syntax in imports.
+    ///
+    /// ### Why is this bad?
+    ///
+    /// This loader syntax is non-standard, so it couples the code to Webpack. The recommended way to
+    /// specify Webpack loader configuration is in a [Webpack configuration file](https://webpack.js.org/concepts/loaders/#configuration).
+    ///
+    /// ### Example
+    /// ```javascript
+    /// import myModule from 'my-loader!my-module';
+    /// import theme from 'style!css!./theme.css';
+    ///
+    /// var myModule = require('my-loader!./my-module');
+    /// var theme = require('style!css!./theme.css');
+    /// ```
+    NoWebpackLoaderSyntax,
+    restriction,
+);
+
+impl Rule for NoWebpackLoaderSyntax {
+    fn run<'a>(&self, node: &AstNode<'a>, ctx: &LintContext<'a>) {
+        // not in top level
+        if node.scope_id() != ctx.scopes().root_scope_id() {
+            return;
+        }
+
+        match node.kind() {
+            AstKind::CallExpression(call_expr) => {
+                if let Expression::Identifier(identifier) = &call_expr.callee {
+                    if identifier.name != "require" {
+                        return;
+                    }
+
+                    if call_expr.arguments.len() != 1 {
+                        return;
+                    }
+
+                    let Argument::StringLiteral(ident) = &call_expr.arguments[0] else {
+                        return;
+                    };
+
+                    if ident.value.contains('!') {
+                        ctx.diagnostic(no_named_as_default_diagnostic(
+                            ident.value.as_str(),
+                            ident.span,
+                        ));
+                    }
+                }
+            }
+            AstKind::ImportDeclaration(import_decl) => {
+                if import_decl.source.value.contains('!') {
+                    ctx.diagnostic(no_named_as_default_diagnostic(
+                        &import_decl.source.value,
+                        import_decl.source.span,
+                    ));
+                }
+            }
+            _ => {}
+        }
+    }
+}
+
+#[test]
+fn test() {
+    use crate::tester::Tester;
+
+    let pass = vec![
+        "import _ from 'lodash'",
+        "import find from 'lodash.find'",
+        "import foo from './foo.css'",
+        "import data from '@scope/my-package/data.json'",
+        "var _ = require('lodash')",
+        "var find = require('lodash.find')",
+        "var foo = require('./foo')",
+        "var foo = require('../foo')",
+        "var foo = require('foo')",
+        "var foo = require('./')",
+        "var foo = require('@scope/foo')",
+    ];
+
+    let fail = vec![
+        "import _ from 'babel!lodash'",
+        "import find from '-babel-loader!lodash.find'",
+        "import foo from 'style!css!./foo.css'",
+        "import data from 'json!@scope/my-package/data.json'",
+        "var _ = require('babel!lodash')",
+        "var find = require('-babel-loader!lodash.find')",
+        "var foo = require('style!css!./foo.css')",
+        "var data = require('json!@scope/my-package/data.json')",
+    ];
+
+    Tester::new(NoWebpackLoaderSyntax::NAME, pass, fail).test_and_snapshot();
+}