@trigen/oxlint-config 9.0.1 → 9.0.3

package/package.json

@@ -1,7 +1,7 @@
 {
   "name": "@trigen/oxlint-config",
   "type": "module",
-  "version": "9.0.1",
+  "version": "9.0.3",
   "description": "Trigen's Oxlint config.",
   "author": "dangreen",
   "license": "MIT",

package/src/module.js

@@ -9,13 +9,14 @@ import {
 } from './subconfigs/files.js'
 
 export default {
+  jsPlugins: ['@trigen/oxlint-config/plugin'],
   overrides: [
     ...bundlerConfig.overrides,
     {
       files: not(commonjsFiles),
       plugins: ['import'],
       rules: {
-        'import/extensions': ['error', 'ignorePackages']
+        'trigen/extensions': ['error', 'ignorePackages']
       }
     }
   ]

package/src/plugin/extensions.js

@@ -0,0 +1,72 @@
+import { posix as path } from 'node:path'
+
+function getOptions(context) {
+  return {
+    ignorePackages: context.options[0] === 'ignorePackages'
+  }
+}
+
+function isPackageImport(source) {
+  return !source.startsWith('.')
+    && !source.startsWith('/')
+}
+
+function hasExtension(source) {
+  const cleanSource = source.split(/[?#]/)[0]
+  const basename = path.basename(cleanSource)
+
+  return path.extname(basename) !== ''
+}
+
+function checkSource(context, options, node) {
+  const source = node.source?.value
+
+  if (typeof source !== 'string') {
+    return
+  }
+
+  if (options.ignorePackages && isPackageImport(source)) {
+    return
+  }
+
+  if (hasExtension(source)) {
+    return
+  }
+
+  context.report({
+    node: node.source,
+    message: `Missing file extension for "${source}".`
+  })
+}
+
+export default {
+  meta: {
+    type: 'problem',
+    docs: {
+      description: 'Require import and export paths to include any file extension.'
+    },
+    schema: [
+      {
+        enum: [
+          'always',
+          'ignorePackages'
+        ]
+      }
+    ]
+  },
+  create(context) {
+    const options = getOptions(context)
+
+    return {
+      ExportAllDeclaration(node) {
+        checkSource(context, options, node)
+      },
+      ExportNamedDeclaration(node) {
+        checkSource(context, options, node)
+      },
+      ImportDeclaration(node) {
+        checkSource(context, options, node)
+      }
+    }
+  }
+}

package/src/plugin/index.js

@@ -1,3 +1,4 @@
+import extensionsRule from './extensions.js'
 import importOrderRule from './import-order.js'
 import memberOrderingRule from './member-ordering.js'
 import namedImportOrderRule from './named-import-order.js'
@@ -5,9 +6,10 @@ import namingConventionRule from './naming-convention.js'
 
 export default {
   meta: {
-    name: '@trigen/oxlint-config/plugin'
+    name: 'trigen'
   },
   rules: {
+    'extensions': extensionsRule,
     'import-order': importOrderRule,
     'member-ordering': memberOrderingRule,
     'named-import-order': namedImportOrderRule,

package/src/plugin/naming-convention.js

@@ -33,11 +33,21 @@ function getSelectorOptions(options, selector, modifiers = []) {
 }
 
 function getNormalizedName(name, option) {
+  let normalizedName = name
+
   if (option.leadingUnderscore === 'allow') {
-    return name.replace(/^_+/, '')
+    normalizedName = normalizedName.replace(/^_+/, '')
+  }
+
+  if (option.leadingDollar === 'allow') {
+    normalizedName = normalizedName.replace(/^\$+/, '')
+  }
+
+  if (option.trailingDollar === 'allow') {
+    normalizedName = normalizedName.replace(/\$+$/, '')
   }
 
-  return name
+  return normalizedName
 }
 
 function matchesFormat(name, format) {
@@ -194,6 +204,12 @@ export default {
           leadingUnderscore: {
             type: 'string'
           },
+          leadingDollar: {
+            type: 'string'
+          },
+          trailingDollar: {
+            type: 'string'
+          },
           modifiers: {
             type: 'array',
             items: {

package/src/subconfigs/basic.js

@@ -3,6 +3,7 @@
  */
 
 export default {
+  jsPlugins: ['@trigen/oxlint-config/plugin'],
   rules: {
     'eslint/constructor-super': 'error',
     'eslint/for-direction': 'error',
@@ -237,6 +238,70 @@ export default {
     'eslint/no-unneeded-ternary': 'error',
     'eslint/operator-assignment': ['error', 'always'],
     'eslint/prefer-object-spread': 'error',
-    'eslint/unicode-bom': 'error'
+    'eslint/unicode-bom': 'error',
+    'trigen/naming-convention': [
+      'error',
+      {
+        selector: 'default',
+        format: [
+          'camelCase',
+          'PascalCase',
+          'UPPER_CASE'
+        ],
+        leadingDollar: 'allow',
+        trailingDollar: 'allow'
+      },
+      {
+        selector: 'variable',
+        format: [
+          'camelCase',
+          'UPPER_CASE',
+          'PascalCase'
+        ],
+        leadingDollar: 'allow',
+        trailingDollar: 'allow'
+      },
+      {
+        selector: 'function',
+        format: ['camelCase', 'PascalCase'],
+        leadingDollar: 'allow',
+        trailingDollar: 'allow'
+      },
+      {
+        selector: 'parameter',
+        format: ['camelCase', 'PascalCase'],
+        leadingUnderscore: 'allow',
+        leadingDollar: 'allow',
+        trailingDollar: 'allow'
+      },
+      {
+        selector: 'typeLike',
+        format: ['PascalCase']
+      },
+      {
+        selector: 'interface',
+        format: ['PascalCase']
+      },
+      {
+        selector: 'enumMember',
+        format: ['PascalCase']
+      },
+      {
+        selector: 'classProperty',
+        format: [
+          'camelCase',
+          'UPPER_CASE',
+          'PascalCase'
+        ],
+        modifiers: ['static'],
+        leadingDollar: 'allow',
+        trailingDollar: 'allow'
+      },
+      {
+        selector: ['objectLiteralProperty', 'objectLiteralMethod'],
+        format: null,
+        modifiers: ['requiresQuotes']
+      }
+    ]
   }
 }

package/src/subconfigs/import.js

@@ -4,12 +4,7 @@
 
 export default {
   plugins: ['import'],
-  jsPlugins: [
-    {
-      name: 'trigen',
-      specifier: '@trigen/oxlint-config/plugin'
-    }
-  ],
+  jsPlugins: ['@trigen/oxlint-config/plugin'],
   rules: {
     'import/no-absolute-path': 'error',
     'import/no-dynamic-require': 'error',

package/src/subconfigs/typescript-type-checked.js

@@ -54,8 +54,6 @@ export default {
         'typescript/no-unnecessary-template-expression': 'error',
         'typescript/no-unnecessary-type-arguments': 'error',
         'typescript/consistent-type-imports': 'error',
-        'eslint/dot-notation': 'off',
-        'typescript/dot-notation': 'error',
         'typescript/prefer-includes': 'error',
         'typescript/prefer-nullish-coalescing': 'off',
         'typescript/prefer-optional-chain': 'error',

package/src/subconfigs/typescript.js

@@ -12,12 +12,7 @@ export default {
     {
       files: tsFiles,
       plugins: ['typescript'],
-      jsPlugins: [
-        {
-          name: 'trigen',
-          specifier: '@trigen/oxlint-config/plugin'
-        }
-      ],
+      jsPlugins: ['@trigen/oxlint-config/plugin'],
       rules: {
         // Recommended
         'typescript/ban-ts-comment': [
@@ -108,60 +103,6 @@ export default {
               ]
             }
           }
-        ],
-        'trigen/naming-convention': [
-          'error',
-          {
-            selector: 'default',
-            format: [
-              'camelCase',
-              'PascalCase',
-              'UPPER_CASE'
-            ]
-          },
-          {
-            selector: 'variable',
-            format: [
-              'camelCase',
-              'UPPER_CASE',
-              'PascalCase'
-            ]
-          },
-          {
-            selector: 'function',
-            format: ['camelCase', 'PascalCase']
-          },
-          {
-            selector: 'parameter',
-            format: ['camelCase', 'PascalCase'],
-            leadingUnderscore: 'allow'
-          },
-          {
-            selector: 'typeLike',
-            format: ['PascalCase']
-          },
-          {
-            selector: 'interface',
-            format: ['PascalCase']
-          },
-          {
-            selector: 'enumMember',
-            format: ['PascalCase']
-          },
-          {
-            selector: 'classProperty',
-            format: [
-              'camelCase',
-              'UPPER_CASE',
-              'PascalCase'
-            ],
-            modifiers: ['static']
-          },
-          {
-            selector: ['objectLiteralProperty', 'objectLiteralMethod'],
-            format: null,
-            modifiers: ['requiresQuotes']
-          }
         ]
       }
     },