@himynameisdave/oxlint-config 1.0.0 → 1.2.0

package/README.md

@@ -29,7 +29,7 @@ pnpm add -D oxlint @himynameisdave/oxlint-config
 yarn add -D oxlint @himynameisdave/oxlint-config
 ```
 
-- Requires `oxlint >=1.76.0 <2`. Why that range, and how it moves: [Versioning & compatibility](#versioning--compatibility).
+- Requires `oxlint >=1.78.0 <2`. Why that range, and how it moves: [Versioning & compatibility](#versioning--compatibility).
 - Type-aware linting requires TypeScript 7+ and a `strict` tsconfig.
 
 ## Configurations
@@ -99,7 +99,7 @@ Rule churn is deliberately _not_ a major bump. A newly-decided rule and a rule f
 
 `^` accepts new errors on update. Don't want that? Use `~` (patch only) with a committed lockfile, and upgrade deliberately.
 
-**Supported oxlint: `>=1.76.0 <2`.** The floor is the version this release's rule inventory was certified against, so it moves whenever new rules are decided. Older oxlint skips rules it doesn't know instead of erroring, which means a stale binary quietly under-lints. The `<2` ceiling is there because an oxlint 2.0 needs a release here anyway.
+**Supported oxlint: `>=1.78.0 <2`.** The floor is the version this release's rule inventory was certified against, so it moves whenever new rules are decided. Older oxlint skips rules it doesn't know instead of erroring, which means a stale binary quietly under-lints. The `<2` ceiling is there because an oxlint 2.0 needs a release here anyway.
 
 **Type-aware assumes a strict tsconfig.** The `type-aware` config expects `"strict": true`, and does its best work with `"noUncheckedIndexedAccess"`. Without them, rules like `typescript/no-unnecessary-condition` both over- and under-report.
 

package/dist/base.d.ts

@@ -169,6 +169,7 @@ declare const _default: {
         'eslint/no-ternary': "off";
         'eslint/no-useless-computed-key': "error";
         'eslint/object-shorthand': "error";
+        'eslint/one-var': ["error", string];
         'eslint/operator-assignment': "error";
         'eslint/prefer-arrow-callback': "error";
         'eslint/prefer-const': "error";
@@ -452,7 +453,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";
@@ -506,6 +507,9 @@ declare const _default: {
         'jsdoc/check-property-names': "error";
         'jsdoc/check-tag-names': "error";
         'jsdoc/implements-on-classes': "error";
+        'jsdoc/no-blank-blocks': ["error", {
+            enableFixer: true;
+        }];
         'jsdoc/no-defaults': "error";
         'jsdoc/require-property': "off";
         'jsdoc/require-property-description': "error";

package/dist/base.js

@@ -328,6 +328,9 @@ export default defineConfig({
         'eslint/no-useless-computed-key': 'error',
         // {x: x} has a shorthand for a reason.
         'eslint/object-shorthand': 'error',
+        // "never" (the default is the opposite): `let a, b = 1` reads as if both were
+        // initialized, and one declarator per statement keeps diffs one binding wide.
+        'eslint/one-var': ['error', 'never'],
         // x = x + y has a += spelling.
         'eslint/operator-assignment': 'error',
         // Arrow callbacks keep lexical this; function callbacks invite this-bugs.
@@ -494,8 +497,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 +952,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.
@@ -1092,6 +1098,9 @@ export default defineConfig({
         'jsdoc/check-tag-names': 'error',
         // @implements on a non-class is meaningless.
         'jsdoc/implements-on-classes': 'error',
+        // A `/** */` documents nothing — the require-*-description bar applied to the
+        // whole block. Fixer on: oxfmt reflows comments but never deletes them.
+        'jsdoc/no-blank-blocks': ['error', { enableFixer: true }],
         // Defaults documented in JSDoc drift from the code's actual defaults.
         'jsdoc/no-defaults': 'error',
         // Demanding @property blocks on every typedef is ceremony; completeness

package/dist/index.d.ts

@@ -172,6 +172,7 @@ declare const _default: {
             'eslint/no-ternary': "off";
             'eslint/no-useless-computed-key': "error";
             'eslint/object-shorthand': "error";
+            'eslint/one-var': ["error", string];
             'eslint/operator-assignment': "error";
             'eslint/prefer-arrow-callback': "error";
             'eslint/prefer-const': "error";
@@ -455,7 +456,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";
@@ -509,6 +510,9 @@ declare const _default: {
             'jsdoc/check-property-names': "error";
             'jsdoc/check-tag-names': "error";
             'jsdoc/implements-on-classes': "error";
+            'jsdoc/no-blank-blocks': ["error", {
+                enableFixer: true;
+            }];
             'jsdoc/no-defaults': "error";
             'jsdoc/require-property': "off";
             'jsdoc/require-property-description': "error";
@@ -658,7 +662,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.2.0",
 	"description": "An opinionated oxlint config, by and for himynameisdave.",
 	"keywords": [
 		"config",
@@ -61,19 +61,17 @@
 		"prepublishOnly": "npm run build",
 		"format": "oxfmt --write",
 		"format:check": "oxfmt --check",
-		"test": "bun run lint && bun run format:check && bun run check-coverage && bun run test:consumer",
-		"release": "np"
+		"test": "bun run lint && bun run format:check && bun run check-coverage && bun run test:consumer"
 	},
 	"devDependencies": {
 		"@types/bun": "^1.3.14",
-		"np": "^12.0.0",
-		"oxfmt": "0.61.0",
-		"oxlint": "1.76.0",
+		"oxfmt": "0.63.0",
+		"oxlint": "1.78.0",
 		"oxlint-tsgolint": "^7.0.2001",
 		"typescript": "^7.0.0"
 	},
 	"peerDependencies": {
-		"oxlint": ">=1.76.0 <2",
+		"oxlint": ">=1.78.0 <2",
 		"oxlint-tsgolint": ">=7.0.2001"
 	},
 	"peerDependenciesMeta": {