@himynameisdave/oxlint-config 1.0.0 → 1.1.0

package/dist/base.d.ts

@@ -452,7 +452,7 @@ declare const _default: {
         'import/max-dependencies': ["error", {
             max: number;
         }];
-        'import/consistent-type-specifier-style': ["error", "prefer-inline"];
+        'import/consistent-type-specifier-style': ["error", "prefer-top-level-if-only-type-imports"];
         'import/exports-last': "off";
         'import/first': "error";
         'import/group-exports': "off";

package/dist/base.js

@@ -494,8 +494,9 @@ export default defineConfig({
         'typescript/consistent-type-assertions': 'error',
         // type over interface: no declaration merging surprises, works for unions too.
         'typescript/consistent-type-definitions': ['error', 'type'],
-        // Inline `import { type X }` keeps one import per module; pairs with
-        // import/consistent-type-specifier-style below.
+        // Inline `import { x, type Y }` keeps mixed value+type imports on one line; the
+        // all-type case is handled by import/consistent-type-specifier-style below, which
+        // sends it top-level so erasure drops it (issue #33).
         'typescript/consistent-type-imports': ['error', { fixStyle: 'inline-type-imports' }],
         // Property signatures (fn: () => void) get strict variance checking; methods don't.
         'typescript/method-signature-style': 'error',
@@ -948,8 +949,10 @@ export default defineConfig({
         /* ================================================================== *
          * import — style
          * ================================================================== */
-        // import { type X, y } inline — one import line per module, types marked in place.
-        'import/consistent-type-specifier-style': ['error', 'prefer-inline'],
+        // Top-level `import type { X }` only when every specifier is a type — that shape
+        // fully erases, so this agrees with typescript/no-import-type-side-effects and
+        // --fix converges; mixed imports keep inline `type` on one line (see issue #33).
+        'import/consistent-type-specifier-style': ['error', 'prefer-top-level-if-only-type-imports'],
         // Export placement is layout preference, not correctness (off per cacographer).
         'import/exports-last': 'off',
         // Imports scattered below code hide the dependency list.

package/dist/index.d.ts

@@ -455,7 +455,7 @@ declare const _default: {
             'import/max-dependencies': ["error", {
                 max: number;
             }];
-            'import/consistent-type-specifier-style': ["error", "prefer-inline"];
+            'import/consistent-type-specifier-style': ["error", "prefer-top-level-if-only-type-imports"];
             'import/exports-last': "off";
             'import/first': "error";
             'import/group-exports': "off";
@@ -658,7 +658,9 @@ declare const _default: {
             'vitest/valid-describe-callback': "error";
             'vitest/valid-expect': "error";
             'vitest/valid-expect-in-promise': "error";
-            'vitest/valid-title': "error";
+            'vitest/valid-title': ["error", {
+                allowArguments: boolean;
+            }];
             'vitest/warn-todo': "off";
             'vitest/no-commented-out-tests': "error";
             'vitest/no-conditional-in-test': "off";

package/dist/vitest.d.ts

@@ -32,7 +32,9 @@ declare const _default: {
         'vitest/valid-describe-callback': "error";
         'vitest/valid-expect': "error";
         'vitest/valid-expect-in-promise': "error";
-        'vitest/valid-title': "error";
+        'vitest/valid-title': ["error", {
+            allowArguments: boolean;
+        }];
         'vitest/warn-todo': "off";
         'vitest/no-commented-out-tests': "error";
         'vitest/no-conditional-in-test': "off";

package/dist/vitest.js

@@ -64,7 +64,10 @@ export default defineConfig({
         // the test has already passed.
         'vitest/valid-expect-in-promise': 'error',
         // Empty or dynamic-garbage titles break -t filtering and failure reports.
-        'vitest/valid-title': 'error',
+        // allowArguments: true lets a title be the imported function reference
+        // itself, which is what prefer-describe-function-title (below) demands —
+        // without it the two rules deadlock (oxc-project/oxc#18591).
+        'vitest/valid-title': ['error', { allowArguments: true }],
         // .todo is the sanctioned marker for planned tests; banning it would push
         // people back to .skip, which no-disabled-tests rightly forbids.
         'vitest/warn-todo': 'off',

package/package.json

@@ -1,6 +1,6 @@
 {
 	"name": "@himynameisdave/oxlint-config",
-	"version": "1.0.0",
+	"version": "1.1.0",
 	"description": "An opinionated oxlint config, by and for himynameisdave.",
 	"keywords": [
 		"config",