@himynameisdave/oxlint-config 1.1.0 → 2.0.0

package/README.md

@@ -29,18 +29,21 @@ 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.85.0 <2`. Why that range, and how it moves: [Versioning & compatibility](#versioning--compatibility).
 - Type-aware linting requires TypeScript 7+ and a `strict` tsconfig.
 
 ## Configurations
 
-| Config       | Import                                     | What it is                                                            |
-| ------------ | ------------------------------------------ | --------------------------------------------------------------------- |
-| `base`       | `@himynameisdave/oxlint-config/base`       | Core JS/TS rules. No framework assumptions. Start here.               |
-| `svelte`     | `@himynameisdave/oxlint-config/svelte`     | Svelte 5 (runes) overrides for `.svelte`/`.svelte.ts` files.          |
-| `type-aware` | `@himynameisdave/oxlint-config/type-aware` | Rules needing type info. Requires `oxlint-tsgolint` + `--type-aware`. |
-| `vitest`     | `@himynameisdave/oxlint-config/vitest`     | Test-suite rules for Vitest projects (`.only` in CI, etc).            |
-| _(default)_  | `@himynameisdave/oxlint-config`            | Kitchen sink: all of the above.                                       |
+| Config       | Import                                     | What it is                                                                   |
+| ------------ | ------------------------------------------ | ---------------------------------------------------------------------------- |
+| `base`       | `@himynameisdave/oxlint-config/base`       | Core JS/TS rules. No framework assumptions. Start here.                      |
+| `bun`        | `@himynameisdave/oxlint-config/bun`        | All rules from the pinned bunisms plugin, as errors. Bun-targeted code only. |
+| `svelte`     | `@himynameisdave/oxlint-config/svelte`     | Svelte 5 (runes) overrides for `.svelte`/`.svelte.ts` files.                 |
+| `type-aware` | `@himynameisdave/oxlint-config/type-aware` | Rules needing type info. Requires `oxlint-tsgolint` + `--type-aware`.        |
+| `vitest`     | `@himynameisdave/oxlint-config/vitest`     | Test-suite rules for Vitest projects (`.only` in CI, etc).                   |
+| _(default)_  | `@himynameisdave/oxlint-config`            | Kitchen sink for Bun projects: all of the above.                             |
+
+The default assumes your code targets Bun. For Node or browser projects, compose the individual presets without `bun`.
 
 ## Usage
 
@@ -91,27 +94,48 @@ oxlint -c oxlint.config.ts --deny-warnings
 
 Version bumps describe what a release does to _your_ CI:
 
-- **major**: structural change to what this package _is_. An oxlint major bump, a new plugin enabled, an entry point renamed or removed.
-- **minor**: rule decisions. New rules decided (usually after an oxlint release adds them), an existing rule flipped between `error` and `off`, or options tightened. New errors can appear in code that passed before.
+- **major**: structural change to what this package _is_. An oxlint major bump, a new plugin enabled in an existing preset, an entry point or public rule name renamed or removed, or an incompatible runtime requirement.
+- **minor**: rule decisions. New rules decided (usually after an oxlint release adds them), an existing rule flipped between `error` and `off`, options tightened, or a plugin update changes lint findings. New errors can appear in code that passed before.
 - **patch**: docs, comments, tooling. No behavior change.
 
 Rule churn is deliberately _not_ a major bump. A newly-decided rule and a rule flipped from `off` to `error` break your build in exactly the same way, so pretending one is riskier than the other would just inflate the major number without telling you anything. New errors are the point of the package.
 
 `^` 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.85.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.
 
 **Plugins you add start off.** Every category is `"off"` by design, so adding `plugins: ['react']` to your own config enables zero react rules until you name each one. Surprising once, then greppable forever.
 
+## Bun support and upstream updates
+
+The `bun` preset enables the reviewed rules from [`eslint-plugin-bunisms`](https://github.com/himynameisdave/eslint-plugin-bunisms) as errors. The plugin is bundled as a runtime dependency, so consumers only need this config and Oxlint. Use it for code targeting Bun >=1.4.0.
+
+```ts
+import { defineConfig } from 'oxlint';
+import base from '@himynameisdave/oxlint-config/base';
+import bun from '@himynameisdave/oxlint-config/bun';
+
+export default defineConfig({
+	extends: [base, bun],
+	rules: {
+		// Keep a Node-compatible subprocess call where the application needs one.
+		'bun/prefer-bun-spawn': 'off'
+	}
+});
+```
+
+The rule set is reviewed and updated with this package; upstream updates may add new lint errors. For Node or browser projects, compose the other presets without `bun`.
+
 ## Enabled plugins
 
-`typescript` · `unicorn` · `oxc` · `import` · `promise` · `node` · `jsdoc` (plus the core `eslint` rules) · `vitest` (via the opt-in `vitest` add-on)
+- Core: `eslint`, `typescript`, `unicorn`, `oxc`, `import`, `promise`, `node`, `jsdoc`
+- Add-ons: `vitest`, `bun` (both included in the default config)
 
 The `vitest` stance: test suites deserve the same rigor as app code. The flagship rule is `no-focused-tests`: a committed `it.only` makes CI silently green while skipping every other test. The add-on's rules only fire on test-shaped syntax, so extending it is harmless for non-test files. **Not for `bun:test` suites:** oxlint recognizes test functions by import source (`vitest`, `@jest/globals`) or bare globals, and `import { it } from 'bun:test'` is invisible to it (verified empirically; see `src/vitest.ts`). Bun-native suites get no lint coverage until oxlint supports `bun:test` upstream.
 
-The `jsdoc` stance: exported symbols should be documented; internal code doesn't have to be. Any JSDoc you _do_ write must be complete and descriptive (a partial `@param` list or a bare `@returns` errors), and types never go in JSDoc (TypeScript owns them). oxlint has no `require-jsdoc` rule yet, so _existence_ of docs on exports stays a review expectation until upstream ships one (this config will adopt it with `publicOnly` when it lands).
+The `jsdoc` stance: exported symbols should be documented; internal code doesn't have to be. Any JSDoc you _do_ write must be complete and descriptive (a partial `@param` list or a bare `@returns` errors), and types never go in JSDoc (TypeScript owns them). [oxfmt](https://oxc.rs/docs/guide/usage/formatter.html) handles comment formatting. oxlint has no `require-jsdoc` rule yet, so _existence_ of docs on exports stays a review expectation until upstream ships one (this config will adopt it with `publicOnly` when it lands).
 
 ## Svelte support
 
@@ -150,7 +174,7 @@ export default defineConfig({
 });
 ```
 
-`eslint/no-unused-labels` is _not_ in that list: as of oxlint 1.76 it doesn't fire inside `.svelte` files at all (it does in `.ts`). Add it if a future oxlint starts flagging `$:`.
+`eslint/no-unused-labels` is _not_ in that list: as of oxlint 1.85 it doesn't fire inside `.svelte` files at all (it does in `.ts`). Add it if a future oxlint starts flagging `$:`.
 
 SvelteKit route files need no override. `unicorn/filename-case` skips the leading `+` and checks the rest, so `+page.svelte`, `+layout.server.ts` and friends all pass. Only genuinely bad casing after the `+` (`+Bad_Name.ts`) errors.
 

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", "never"];
         'eslint/operator-assignment': "error";
         'eslint/prefer-arrow-callback': "error";
         'eslint/prefer-const': "error";
@@ -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.
@@ -1095,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/bun.d.ts

@@ -0,0 +1,16 @@
+/**
+ * Bun-targeted code: every rule in the pinned bunisms version is an error.
+ * Resolve from this package so consumers need no separate plugin installation.
+ */
+declare const _default: {
+    jsPlugins: {
+        name: string;
+        specifier: string;
+    }[];
+    rules: {
+        'bun/prefer-bun-file': "error";
+        'bun/prefer-bun-write': "error";
+        'bun/prefer-bun-spawn': "error";
+    };
+};
+export default _default;

package/dist/bun.js

@@ -0,0 +1,25 @@
+import { fileURLToPath } from 'node:url';
+import { defineConfig } from 'oxlint';
+/**
+ * Bun-targeted code: every rule in the pinned bunisms version is an error.
+ * Resolve from this package so consumers need no separate plugin installation.
+ */
+export default defineConfig({
+    jsPlugins: [
+        {
+            name: 'bun',
+            specifier: fileURLToPath(import.meta.resolve('eslint-plugin-bunisms'))
+        }
+    ],
+    rules: {
+        // Bun.file keeps file reads on Bun's native file API.
+        // https://github.com/himynameisdave/eslint-plugin-bunisms/blob/main/docs/rules/prefer-bun-file.md
+        'bun/prefer-bun-file': 'error',
+        // Bun.write provides the native write path for Bun-targeted applications.
+        // https://github.com/himynameisdave/eslint-plugin-bunisms/blob/main/docs/rules/prefer-bun-write.md
+        'bun/prefer-bun-write': 'error',
+        // Bun's subprocess APIs integrate directly with its runtime and streams.
+        // https://github.com/himynameisdave/eslint-plugin-bunisms/blob/main/docs/rules/prefer-bun-spawn.md
+        'bun/prefer-bun-spawn': 'error'
+    }
+});

package/dist/index.d.ts

@@ -1,9 +1,10 @@
 export { default as base } from './base.js';
+export { default as bun } from './bun.js';
 export { default as svelte } from './svelte.js';
 export { default as typeAware } from './type-aware.js';
 export { default as vitest } from './vitest.js';
 /**
- * Kitchen sink: base + svelte + vitest + type-aware, in override-safe order.
+ * Kitchen sink for Bun: base + bun + svelte + vitest + type-aware, in override-safe order.
  * Requires `oxlint-tsgolint` installed (type-aware rules are on).
  *
  * For à la carte composition, extend the named exports instead:
@@ -172,6 +173,7 @@ declare const _default: {
             'eslint/no-ternary': "off";
             'eslint/no-useless-computed-key': "error";
             'eslint/object-shorthand': "error";
+            'eslint/one-var': ["error", "never"];
             'eslint/operator-assignment': "error";
             'eslint/prefer-arrow-callback': "error";
             'eslint/prefer-const': "error";
@@ -509,6 +511,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";
@@ -553,6 +558,16 @@ declare const _default: {
             };
         })[];
         ignorePatterns: string[];
+    } | {
+        jsPlugins: {
+            name: string;
+            specifier: string;
+        }[];
+        rules: {
+            'bun/prefer-bun-file': "error";
+            'bun/prefer-bun-write': "error";
+            'bun/prefer-bun-spawn': "error";
+        };
     } | {
         overrides: {
             files: string[];

package/dist/index.js

@@ -1,14 +1,16 @@
 import { defineConfig } from 'oxlint';
 import base from './base.js';
+import bun from './bun.js';
 import svelte from './svelte.js';
 import typeAware from './type-aware.js';
 import vitest from './vitest.js';
 export { default as base } from './base.js';
+export { default as bun } from './bun.js';
 export { default as svelte } from './svelte.js';
 export { default as typeAware } from './type-aware.js';
 export { default as vitest } from './vitest.js';
 /**
- * Kitchen sink: base + svelte + vitest + type-aware, in override-safe order.
+ * Kitchen sink for Bun: base + bun + svelte + vitest + type-aware, in override-safe order.
  * Requires `oxlint-tsgolint` installed (type-aware rules are on).
  *
  * For à la carte composition, extend the named exports instead:
@@ -19,5 +21,5 @@ export { default as vitest } from './vitest.js';
  * ```
  */
 export default defineConfig({
-    extends: [base, svelte, vitest, typeAware]
+    extends: [base, bun, svelte, vitest, typeAware]
 });

package/dist/type-aware.js

@@ -22,7 +22,7 @@ export default defineConfig({
         // obj["key"] for a statically-known key is obj.key.
         'typescript/dot-notation': 'error',
         // NOTE: tsgolint also implements naming-convention and prefer-destructuring,
-        // but oxlint 1.75 doesn't register those rule names yet — add them here once
+        // but oxlint 1.85 doesn't register those rule names yet — add them here once
         // they land upstream.
         // delete arr[i] leaves a hole, not a shorter array — splice/filter instead.
         'typescript/no-array-delete': 'error',

package/package.json

@@ -1,6 +1,6 @@
 {
 	"name": "@himynameisdave/oxlint-config",
-	"version": "1.1.0",
+	"version": "2.0.0",
 	"description": "An opinionated oxlint config, by and for himynameisdave.",
 	"keywords": [
 		"config",
@@ -36,6 +36,10 @@
 			"types": "./dist/base.d.ts",
 			"default": "./dist/base.js"
 		},
+		"./bun": {
+			"types": "./dist/bun.d.ts",
+			"default": "./dist/bun.js"
+		},
 		"./svelte": {
 			"types": "./dist/svelte.d.ts",
 			"default": "./dist/svelte.js"
@@ -61,19 +65,21 @@
 		"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"
+	},
+	"dependencies": {
+		"eslint-plugin-bunisms": "0.1.0"
 	},
 	"devDependencies": {
-		"@types/bun": "^1.3.14",
-		"np": "^12.0.0",
-		"oxfmt": "0.61.0",
-		"oxlint": "1.76.0",
-		"oxlint-tsgolint": "^7.0.2001",
+		"@types/bun": "^1.4.2",
+		"@types/node": "^26.1.1",
+		"oxfmt": "0.70.0",
+		"oxlint": "1.85.0",
+		"oxlint-tsgolint": "7.0.2003",
 		"typescript": "^7.0.0"
 	},
 	"peerDependencies": {
-		"oxlint": ">=1.76.0 <2",
+		"oxlint": ">=1.85.0 <2",
 		"oxlint-tsgolint": ">=7.0.2001"
 	},
 	"peerDependenciesMeta": {
@@ -81,5 +87,5 @@
 			"optional": true
 		}
 	},
-	"packageManager": "bun@1.3.13"
+	"packageManager": "bun@1.4.2"
 }