@pyreon/lint 0.11.4 → 0.11.6

package/src/rules/ssr/no-mismatch-risk.ts

@@ -1,13 +1,13 @@
-import type { Rule, VisitorCallbacks } from "../../types"
-import { getSpan, isMemberCallTo } from "../../utils/ast"
+import type { Rule, VisitorCallbacks } from '../../types'
+import { getSpan, isMemberCallTo } from '../../utils/ast'
 
 export const noMismatchRisk: Rule = {
   meta: {
-    id: "pyreon/no-mismatch-risk",
-    category: "ssr",
+    id: 'pyreon/no-mismatch-risk',
+    category: 'ssr',
     description:
-      "Warn about non-deterministic calls (Date.now, Math.random, crypto.randomUUID) in JSX context that cause hydration mismatches.",
-    severity: "warn",
+      'Warn about non-deterministic calls (Date.now, Math.random, crypto.randomUUID) in JSX context that cause hydration mismatches.',
+    severity: 'warn',
     fixable: false,
   },
   create(context) {
@@ -16,22 +16,22 @@ export const noMismatchRisk: Rule = {
       JSXElement() {
         jsxDepth++
       },
-      "JSXElement:exit"() {
+      'JSXElement:exit'() {
         jsxDepth--
       },
       JSXFragment() {
         jsxDepth++
       },
-      "JSXFragment:exit"() {
+      'JSXFragment:exit'() {
         jsxDepth--
       },
       CallExpression(node: any) {
         if (jsxDepth === 0) return
 
         if (
-          isMemberCallTo(node, "Date", "now") ||
-          isMemberCallTo(node, "Math", "random") ||
-          isMemberCallTo(node, "crypto", "randomUUID")
+          isMemberCallTo(node, 'Date', 'now') ||
+          isMemberCallTo(node, 'Math', 'random') ||
+          isMemberCallTo(node, 'crypto', 'randomUUID')
         ) {
           const callee = node.callee
           const name = `${callee.object.name}.${callee.property.name}`

package/src/rules/ssr/no-window-in-ssr.ts

@@ -1,13 +1,13 @@
-import type { Rule, VisitorCallbacks } from "../../types"
-import { getSpan, isCallTo } from "../../utils/ast"
-import { BROWSER_GLOBALS } from "../../utils/imports"
+import type { Rule, VisitorCallbacks } from '../../types'
+import { getSpan, isCallTo } from '../../utils/ast'
+import { BROWSER_GLOBALS } from '../../utils/imports'
 
 export const noWindowInSsr: Rule = {
   meta: {
-    id: "pyreon/no-window-in-ssr",
-    category: "ssr",
-    description: "Disallow browser globals outside onMount/effect/typeof guards — they break SSR.",
-    severity: "error",
+    id: 'pyreon/no-window-in-ssr',
+    category: 'ssr',
+    description: 'Disallow browser globals outside onMount/effect/typeof guards — they break SSR.',
+    severity: 'error',
     fixable: false,
   },
   create(context) {
@@ -16,12 +16,12 @@ export const noWindowInSsr: Rule = {
 
     const callbacks: VisitorCallbacks = {
       CallExpression(node: any) {
-        if (isCallTo(node, "onMount") || isCallTo(node, "effect")) {
+        if (isCallTo(node, 'onMount') || isCallTo(node, 'effect')) {
           safeDepth++
         }
       },
-      "CallExpression:exit"(node: any) {
-        if (isCallTo(node, "onMount") || isCallTo(node, "effect")) {
+      'CallExpression:exit'(node: any) {
+        if (isCallTo(node, 'onMount') || isCallTo(node, 'effect')) {
           safeDepth--
         }
       },
@@ -29,19 +29,19 @@ export const noWindowInSsr: Rule = {
         // typeof window !== "undefined"
         const test = node.test
         if (
-          test?.type === "BinaryExpression" &&
-          test.left?.type === "UnaryExpression" &&
-          test.left.operator === "typeof"
+          test?.type === 'BinaryExpression' &&
+          test.left?.type === 'UnaryExpression' &&
+          test.left.operator === 'typeof'
         ) {
           typeofGuardDepth++
         }
       },
-      "IfStatement:exit"(node: any) {
+      'IfStatement:exit'(node: any) {
         const test = node.test
         if (
-          test?.type === "BinaryExpression" &&
-          test.left?.type === "UnaryExpression" &&
-          test.left.operator === "typeof"
+          test?.type === 'BinaryExpression' &&
+          test.left?.type === 'UnaryExpression' &&
+          test.left.operator === 'typeof'
         ) {
           typeofGuardDepth--
         }
@@ -51,18 +51,18 @@ export const noWindowInSsr: Rule = {
         if (!BROWSER_GLOBALS.has(node.name)) return
 
         // Skip typeof expressions: typeof window
-        if (parent?.type === "UnaryExpression" && parent.operator === "typeof") return
+        if (parent?.type === 'UnaryExpression' && parent.operator === 'typeof') return
 
         // Skip import specifiers
         if (
-          parent?.type === "ImportSpecifier" ||
-          parent?.type === "ImportDefaultSpecifier" ||
-          parent?.type === "ImportNamespaceSpecifier"
+          parent?.type === 'ImportSpecifier' ||
+          parent?.type === 'ImportDefaultSpecifier' ||
+          parent?.type === 'ImportNamespaceSpecifier'
         )
           return
 
         // Skip property access on member expressions (only flag when used as the object)
-        if (parent?.type === "MemberExpression" && parent.property === node && !parent.computed)
+        if (parent?.type === 'MemberExpression' && parent.property === node && !parent.computed)
           return
 
         context.report({

package/src/rules/ssr/prefer-request-context.ts

@@ -1,22 +1,22 @@
-import type { Rule, VisitorCallbacks } from "../../types"
-import { getSpan, isCallTo } from "../../utils/ast"
+import type { Rule, VisitorCallbacks } from '../../types'
+import { getSpan, isCallTo } from '../../utils/ast'
 
 export const preferRequestContext: Rule = {
   meta: {
-    id: "pyreon/prefer-request-context",
-    category: "ssr",
+    id: 'pyreon/prefer-request-context',
+    category: 'ssr',
     description:
-      "Warn about module-level signal()/createStore() in server files — use request context instead.",
-    severity: "warn",
+      'Warn about module-level signal()/createStore() in server files — use request context instead.',
+    severity: 'warn',
     fixable: false,
   },
   create(context) {
     const filePath = context.getFilePath()
     const isServerFile =
-      filePath.includes("server") ||
-      filePath.includes(".server.") ||
-      filePath.endsWith("server.ts") ||
-      filePath.endsWith("server.tsx")
+      filePath.includes('server') ||
+      filePath.includes('.server.') ||
+      filePath.endsWith('server.ts') ||
+      filePath.endsWith('server.tsx')
 
     if (!isServerFile) return {}
 
@@ -25,24 +25,24 @@ export const preferRequestContext: Rule = {
       FunctionDeclaration() {
         functionDepth++
       },
-      "FunctionDeclaration:exit"() {
+      'FunctionDeclaration:exit'() {
         functionDepth--
       },
       FunctionExpression() {
         functionDepth++
       },
-      "FunctionExpression:exit"() {
+      'FunctionExpression:exit'() {
         functionDepth--
       },
       ArrowFunctionExpression() {
         functionDepth++
       },
-      "ArrowFunctionExpression:exit"() {
+      'ArrowFunctionExpression:exit'() {
         functionDepth--
       },
       CallExpression(node: any) {
         if (functionDepth > 0) return // only flag module-level calls
-        if (isCallTo(node, "signal") || isCallTo(node, "createStore")) {
+        if (isCallTo(node, 'signal') || isCallTo(node, 'createStore')) {
           const name = node.callee.name
           context.report({
             message: `Module-level \`${name}()\` in a server file — this state is shared across all requests. Use \`runWithRequestContext()\` for per-request isolation.`,

package/src/rules/store/no-duplicate-store-id.ts

@@ -1,12 +1,12 @@
-import type { Rule, VisitorCallbacks } from "../../types"
-import { getSpan, isCallTo } from "../../utils/ast"
+import type { Rule, VisitorCallbacks } from '../../types'
+import { getSpan, isCallTo } from '../../utils/ast'
 
 export const noDuplicateStoreId: Rule = {
   meta: {
-    id: "pyreon/no-duplicate-store-id",
-    category: "store",
-    description: "Disallow duplicate defineStore() IDs in the same file.",
-    severity: "error",
+    id: 'pyreon/no-duplicate-store-id',
+    category: 'store',
+    description: 'Disallow duplicate defineStore() IDs in the same file.',
+    severity: 'error',
     fixable: false,
   },
   create(context) {
@@ -14,7 +14,7 @@ export const noDuplicateStoreId: Rule = {
 
     const callbacks: VisitorCallbacks = {
       CallExpression(node: any) {
-        if (!isCallTo(node, "defineStore")) return
+        if (!isCallTo(node, 'defineStore')) return
         const args = node.arguments
         if (!args || args.length === 0) return
 
@@ -22,11 +22,11 @@ export const noDuplicateStoreId: Rule = {
         if (!firstArg) return
 
         let id: string | null = null
-        if (firstArg.type === "Literal" || firstArg.type === "StringLiteral") {
+        if (firstArg.type === 'Literal' || firstArg.type === 'StringLiteral') {
           id = firstArg.value as string
         }
 
-        if (typeof id !== "string") return
+        if (typeof id !== 'string') return
 
         if (storeIds.has(id)) {
           context.report({

package/src/rules/store/no-mutate-store-state.ts

@@ -1,30 +1,30 @@
-import type { Rule, VisitorCallbacks } from "../../types"
-import { getSpan } from "../../utils/ast"
+import type { Rule, VisitorCallbacks } from '../../types'
+import { getSpan } from '../../utils/ast'
 
 export const noMutateStoreState: Rule = {
   meta: {
-    id: "pyreon/no-mutate-store-state",
-    category: "store",
-    description: "Warn when directly calling .set() on store signals — use store actions instead.",
-    severity: "warn",
+    id: 'pyreon/no-mutate-store-state',
+    category: 'store',
+    description: 'Warn when directly calling .set() on store signals — use store actions instead.',
+    severity: 'warn',
     fixable: false,
   },
   create(context) {
     const callbacks: VisitorCallbacks = {
       CallExpression(node: any) {
         const callee = node.callee
-        if (!callee || callee.type !== "MemberExpression") return
-        if (callee.property?.type !== "Identifier" || callee.property.name !== "set") return
+        if (!callee || callee.type !== 'MemberExpression') return
+        if (callee.property?.type !== 'Identifier' || callee.property.name !== 'set') return
 
         // Check for store.signal.set() pattern — member.member.set()
         const obj = callee.object
-        if (!obj || obj.type !== "MemberExpression") return
+        if (!obj || obj.type !== 'MemberExpression') return
         const outerObj = obj.object
-        if (!outerObj || outerObj.type !== "Identifier") return
+        if (!outerObj || outerObj.type !== 'Identifier') return
 
         const name: string = outerObj.name
         // Heuristic: if the outer object name contains "store" (case-insensitive)
-        if (name.toLowerCase().includes("store")) {
+        if (name.toLowerCase().includes('store')) {
           context.report({
             message: `Direct \`.set()\` on store state \`${name}\` — use store actions to mutate state for better traceability.`,
             span: getSpan(node),

package/src/rules/store/no-store-outside-provider.ts

@@ -1,22 +1,22 @@
-import type { Rule, VisitorCallbacks } from "../../types"
-import { getSpan } from "../../utils/ast"
-import { extractImportInfo } from "../../utils/imports"
+import type { Rule, VisitorCallbacks } from '../../types'
+import { getSpan } from '../../utils/ast'
+import { extractImportInfo } from '../../utils/imports'
 
 export const noStoreOutsideProvider: Rule = {
   meta: {
-    id: "pyreon/no-store-outside-provider",
-    category: "store",
-    description: "Warn when store hooks are used in SSR files without a provider import.",
-    severity: "warn",
+    id: 'pyreon/no-store-outside-provider',
+    category: 'store',
+    description: 'Warn when store hooks are used in SSR files without a provider import.',
+    severity: 'warn',
     fixable: false,
   },
   create(context) {
     const filePath = context.getFilePath()
     const isServerFile =
-      filePath.includes("server") ||
-      filePath.includes(".server.") ||
-      filePath.endsWith("server.ts") ||
-      filePath.endsWith("server.tsx")
+      filePath.includes('server') ||
+      filePath.includes('.server.') ||
+      filePath.endsWith('server.ts') ||
+      filePath.endsWith('server.tsx')
 
     if (!isServerFile) return {}
 
@@ -30,7 +30,7 @@ export const noStoreOutsideProvider: Rule = {
         if (
           info.specifiers.some(
             (s) =>
-              s.imported === "setStoreRegistryProvider" || s.imported === "runWithRequestContext",
+              s.imported === 'setStoreRegistryProvider' || s.imported === 'runWithRequestContext',
           )
         ) {
           hasProviderImport = true
@@ -38,13 +38,13 @@ export const noStoreOutsideProvider: Rule = {
       },
       CallExpression(node: any) {
         const callee = node.callee
-        if (!callee || callee.type !== "Identifier") return
+        if (!callee || callee.type !== 'Identifier') return
         const name: string = callee.name
-        if (name.endsWith("Store") && name.startsWith("use")) {
+        if (name.endsWith('Store') && name.startsWith('use')) {
           storeHookCalls.push({ name, span: getSpan(node) })
         }
       },
-      "Program:exit"() {
+      'Program:exit'() {
         if (hasProviderImport) return
         for (const call of storeHookCalls) {
           context.report({

package/src/rules/styling/no-dynamic-styled.ts

@@ -1,13 +1,13 @@
-import type { Rule, VisitorCallbacks } from "../../types"
-import { getSpan, isCallTo } from "../../utils/ast"
+import type { Rule, VisitorCallbacks } from '../../types'
+import { getSpan, isCallTo } from '../../utils/ast'
 
 export const noDynamicStyled: Rule = {
   meta: {
-    id: "pyreon/no-dynamic-styled",
-    category: "styling",
+    id: 'pyreon/no-dynamic-styled',
+    category: 'styling',
     description:
-      "Warn when styled() is called inside a function — it creates new CSS on every render.",
-    severity: "warn",
+      'Warn when styled() is called inside a function — it creates new CSS on every render.',
+    severity: 'warn',
     fixable: false,
   },
   create(context) {
@@ -16,27 +16,27 @@ export const noDynamicStyled: Rule = {
       FunctionDeclaration() {
         functionDepth++
       },
-      "FunctionDeclaration:exit"() {
+      'FunctionDeclaration:exit'() {
         functionDepth--
       },
       FunctionExpression() {
         functionDepth++
       },
-      "FunctionExpression:exit"() {
+      'FunctionExpression:exit'() {
         functionDepth--
       },
       ArrowFunctionExpression() {
         functionDepth++
       },
-      "ArrowFunctionExpression:exit"() {
+      'ArrowFunctionExpression:exit'() {
         functionDepth--
       },
       CallExpression(node: any) {
         if (functionDepth === 0) return
-        if (isCallTo(node, "styled")) {
+        if (isCallTo(node, 'styled')) {
           context.report({
             message:
-              "`styled()` inside a function — this creates new CSS rules on every render. Move `styled()` to module scope.",
+              '`styled()` inside a function — this creates new CSS rules on every render. Move `styled()` to module scope.',
             span: getSpan(node),
           })
         }
@@ -46,10 +46,10 @@ export const noDynamicStyled: Rule = {
         const tag = node.tag
         if (!tag) return
         // styled('div')`...` — tag is a CallExpression of styled
-        if (tag.type === "CallExpression" && isCallTo(tag, "styled")) {
+        if (tag.type === 'CallExpression' && isCallTo(tag, 'styled')) {
           context.report({
             message:
-              "`styled()` tagged template inside a function — this creates new CSS rules on every render. Move to module scope.",
+              '`styled()` tagged template inside a function — this creates new CSS rules on every render. Move to module scope.',
             span: getSpan(node),
           })
         }

package/src/rules/styling/no-inline-style-object.ts

@@ -1,25 +1,25 @@
-import type { Rule, VisitorCallbacks } from "../../types"
-import { getSpan } from "../../utils/ast"
+import type { Rule, VisitorCallbacks } from '../../types'
+import { getSpan } from '../../utils/ast'
 
 export const noInlineStyleObject: Rule = {
   meta: {
-    id: "pyreon/no-inline-style-object",
-    category: "styling",
-    description: "Warn against inline style objects in JSX — prefer styled() or css``.",
-    severity: "warn",
+    id: 'pyreon/no-inline-style-object',
+    category: 'styling',
+    description: 'Warn against inline style objects in JSX — prefer styled() or css``.',
+    severity: 'warn',
     fixable: false,
   },
   create(context) {
     const callbacks: VisitorCallbacks = {
       JSXAttribute(node: any) {
-        if (node.name?.type !== "JSXIdentifier" || node.name.name !== "style") return
+        if (node.name?.type !== 'JSXIdentifier' || node.name.name !== 'style') return
         const value = node.value
-        if (!value || value.type !== "JSXExpressionContainer") return
+        if (!value || value.type !== 'JSXExpressionContainer') return
         const expr = value.expression
-        if (expr?.type === "ObjectExpression") {
+        if (expr?.type === 'ObjectExpression') {
           context.report({
             message:
-              "Inline style object in JSX — consider using `styled()` or `css\\`...\\`` for better performance and caching.",
+              'Inline style object in JSX — consider using `styled()` or `css\\`...\\`` for better performance and caching.',
             span: getSpan(node),
           })
         }

package/src/rules/styling/no-theme-outside-provider.ts

@@ -1,13 +1,13 @@
-import type { Rule, VisitorCallbacks } from "../../types"
-import { getSpan, isCallTo } from "../../utils/ast"
-import { extractImportInfo } from "../../utils/imports"
+import type { Rule, VisitorCallbacks } from '../../types'
+import { getSpan, isCallTo } from '../../utils/ast'
+import { extractImportInfo } from '../../utils/imports'
 
 export const noThemeOutsideProvider: Rule = {
   meta: {
-    id: "pyreon/no-theme-outside-provider",
-    category: "styling",
-    description: "Warn when useTheme() is used without PyreonUI or ThemeProvider in the same file.",
-    severity: "warn",
+    id: 'pyreon/no-theme-outside-provider',
+    category: 'styling',
+    description: 'Warn when useTheme() is used without PyreonUI or ThemeProvider in the same file.',
+    severity: 'warn',
     fixable: false,
   },
   create(context) {
@@ -19,22 +19,22 @@ export const noThemeOutsideProvider: Rule = {
         const info = extractImportInfo(node)
         if (!info) return
         if (
-          info.specifiers.some((s) => s.imported === "PyreonUI" || s.imported === "ThemeProvider")
+          info.specifiers.some((s) => s.imported === 'PyreonUI' || s.imported === 'ThemeProvider')
         ) {
           hasProviderImport = true
         }
       },
       CallExpression(node: any) {
-        if (isCallTo(node, "useTheme")) {
+        if (isCallTo(node, 'useTheme')) {
           themeCalls.push({ span: getSpan(node) })
         }
       },
-      "Program:exit"() {
+      'Program:exit'() {
         if (hasProviderImport) return
         for (const call of themeCalls) {
           context.report({
             message:
-              "`useTheme()` without a `PyreonUI` or `ThemeProvider` import — the theme context may not be available.",
+              '`useTheme()` without a `PyreonUI` or `ThemeProvider` import — the theme context may not be available.',
             span: call.span,
           })
         }

package/src/rules/styling/prefer-cx.ts

@@ -1,39 +1,39 @@
-import type { Rule, VisitorCallbacks } from "../../types"
-import { getSpan } from "../../utils/ast"
+import type { Rule, VisitorCallbacks } from '../../types'
+import { getSpan } from '../../utils/ast'
 
 export const preferCx: Rule = {
   meta: {
-    id: "pyreon/prefer-cx",
-    category: "styling",
+    id: 'pyreon/prefer-cx',
+    category: 'styling',
     description:
-      "Suggest cx() for class composition instead of string concatenation or template literals.",
-    severity: "info",
+      'Suggest cx() for class composition instead of string concatenation or template literals.',
+    severity: 'info',
     fixable: false,
   },
   create(context) {
     const callbacks: VisitorCallbacks = {
       JSXAttribute(node: any) {
-        if (node.name?.type !== "JSXIdentifier" || node.name.name !== "class") return
+        if (node.name?.type !== 'JSXIdentifier' || node.name.name !== 'class') return
         const value = node.value
-        if (!value || value.type !== "JSXExpressionContainer") return
+        if (!value || value.type !== 'JSXExpressionContainer') return
         const expr = value.expression
         if (!expr) return
 
         // String concatenation: "foo " + bar
-        if (expr.type === "BinaryExpression" && expr.operator === "+") {
+        if (expr.type === 'BinaryExpression' && expr.operator === '+') {
           context.report({
             message:
-              "String concatenation in `class` attribute — use `cx()` for cleaner class composition.",
+              'String concatenation in `class` attribute — use `cx()` for cleaner class composition.',
             span: getSpan(expr),
           })
           return
         }
 
         // Template literal: `foo ${bar}`
-        if (expr.type === "TemplateLiteral" && expr.expressions?.length > 0) {
+        if (expr.type === 'TemplateLiteral' && expr.expressions?.length > 0) {
           context.report({
             message:
-              "Template literal in `class` attribute — use `cx()` for cleaner class composition.",
+              'Template literal in `class` attribute — use `cx()` for cleaner class composition.',
             span: getSpan(expr),
           })
         }

package/src/runner.ts

@@ -1,5 +1,5 @@
-import { parseSync, Visitor } from "oxc-parser"
-import type { AstCache } from "./cache"
+import { parseSync, Visitor } from 'oxc-parser'
+import type { AstCache } from './cache'
 import type {
   Diagnostic,
   LintConfig,
@@ -8,22 +8,21 @@ import type {
   RuleContext,
   Severity,
   VisitorCallbacks,
-} from "./types"
-import { LineIndex } from "./utils/source"
-
-const JS_EXTENSIONS = new Set([".ts", ".tsx", ".js", ".jsx", ".mts", ".mjs"])
+} from './types'
+import { JS_EXTENSIONS } from './utils/index'
+import { LineIndex } from './utils/source'
 
 function getExtension(filePath: string): string {
-  const lastDot = filePath.lastIndexOf(".")
-  return lastDot === -1 ? "" : filePath.slice(lastDot)
+  const lastDot = filePath.lastIndexOf('.')
+  return lastDot === -1 ? '' : filePath.slice(lastDot)
 }
 
-type OxcLang = "jsx" | "tsx" | "ts" | "js" | "dts"
+type OxcLang = 'jsx' | 'tsx' | 'ts' | 'js' | 'dts'
 
 function getLang(ext: string): OxcLang {
-  if (ext === ".tsx" || ext === ".jsx") return "tsx"
-  if (ext === ".ts" || ext === ".mts") return "ts"
-  return "js"
+  if (ext === '.tsx' || ext === '.jsx') return 'tsx'
+  if (ext === '.ts' || ext === '.mts') return 'ts'
+  return 'js'
 }
 
 function createRuleContext(
@@ -114,7 +113,7 @@ export function lintFile(
     lineIndex = new LineIndex(sourceText)
     try {
       const result = parseSync(filePath, sourceText, {
-        sourceType: "module",
+        sourceType: 'module',
         lang: getLang(ext),
       })
       program = result.program
@@ -130,7 +129,7 @@ export function lintFile(
   const allCallbacks: VisitorCallbacks[] = []
   for (const rule of rules) {
     const severity = config.rules[rule.meta.id]
-    if (severity === undefined || severity === "off") continue
+    if (severity === undefined || severity === 'off') continue
     const ctx = createRuleContext(rule, severity, diagnostics, lineIndex, sourceText, filePath)
     allCallbacks.push(rule.create(ctx))
   }