chore: improve performance

Author: anubra266

plugin/src/rules/no-config-function-in-source.ts

@@ -1,6 +1,6 @@
 import { isIdentifier, isVariableDeclaration } from '../utils/nodes'
 import { type Rule, createRule } from '../utils'
-import { getAncestor, getImportSpecifiers, isValidFile } from '../utils/helpers'
+import { getAncestor, getImportSpecifiers, hasPkgImport, isPandaConfigFunction, isValidFile } from '../utils/helpers'
 
 export const RULE_NAME = 'no-config-function-in-source'
 
@@ -20,11 +20,14 @@ const rule: Rule = createRule({
   },
   defaultOptions: [],
   create(context) {
+    if (!hasPkgImport(context)) return {}
+
     return {
       CallExpression(node) {
         if (!isValidFile(context)) return
         if (!isIdentifier(node.callee)) return
         if (!CONFIG_FUNCTIONS.includes(node.callee.name)) return
+        if (!isPandaConfigFunction(context, node.callee.name)) return
 
         context.report({
           node,

plugin/src/rules/no-unsafe-token-fn-usage.ts

@@ -107,8 +107,6 @@ const rule: Rule = createRule({
       },
 
       Property(node) {
-        // Ignore css variables
-        if (isLiteral(node.key) && node.key.value?.toString().startsWith('--')) return
         if (!isCallExpression(node.value) && !isLiteral(node.value) && !isTemplateLiteral(node.value)) return
         if (!isPandaAttribute(node, context)) return
 

plugin/src/utils/helpers.ts

@@ -47,41 +47,33 @@ export const getImportSpecifiers = (context: RuleContext<any, any>) => {
 
     node.specifiers.forEach((specifier) => {
       if (!isImportSpecifier(specifier)) return
+
       specifiers.push({ specifier, mod })
     })
   })
 
   return specifiers
 }
 
+export const hasPkgImport = (context: RuleContext<any, any>) => {
+  const imports = _getImports(context)
+  return imports.some(({ mod }) => mod === '@pandacss/dev')
+}
+
+export const isPandaConfigFunction = (context: RuleContext<any, any>, name: string) => {
+  const imports = _getImports(context)
+  return imports.some(({ alias, mod }) => alias === name && mod === '@pandacss/dev')
+}
+
 const _getImports = (context: RuleContext<any, any>) => {
   const specifiers = getImportSpecifiers(context)
-  specifiers.map(({ specifier, mod }) => ({
+
+  const imports: ImportResult[] = specifiers.map(({ specifier, mod }) => ({
     name: specifier.imported.name,
     alias: specifier.local.name,
     mod,
   }))
 
-  const imports: ImportResult[] = []
-
-  context.sourceCode.ast.body.forEach((node) => {
-    if (!isImportDeclaration(node)) return
-
-    const mod = node.source.value
-    if (!mod) return
-
-    node.specifiers.forEach((specifier) => {
-      if (!isImportSpecifier(specifier)) return
-
-      const name = specifier.imported.name
-      const alias = specifier.local.name
-
-      const result = { name, alias, mod }
-
-      imports.push(result)
-    })
-  })
-
   return imports
 }
 

plugin/tests/no-unsafe-token-fn-usage.test.ts

@@ -29,12 +29,6 @@ function App(){
   return <Circle _hover={{  border: 'solid 1px {colors.blue.400}' }} />;
 }`,
   },
-  {
-    code: javascript`
-import { css } from './panda/css';
-
-const styles = css({ '--my-var': '{spacing.8}' })`,
-  },
 ]
 
 const invalids = [