oxlint-config-awesomeness 4.2.0 → 4.4.0

package/README.md

@@ -2,7 +2,7 @@
 
 Opinionated Oxlint config for software houses that want all their apps to feel the same.
 
-**454 rules** across **15 plugins**. Built for full-stack TypeScript monorepos with React, Next.js, Hono, Prisma, and more.
+**458 rules** across **15 plugins**. Built for full-stack TypeScript monorepos with React, Next.js, Hono, Prisma, and more.
 
 ## Installation
 
@@ -2307,20 +2307,6 @@ with (obj) {
 obj.foo = 1;
 ```
 
-### one-var
-
-Require one declaration per variable; never comma-combine declarations.
-
-```js
-// bad
-const a = 1,
-  b = 2;
-
-// good
-const a = 1;
-const b = 2;
-```
-
 ### operator-assignment
 
 Require shorthand operators where possible.
@@ -7969,6 +7955,22 @@ Sort object keys alphabetically, with partition-by-comment support.
 
 ## Other JS Plugin Rules
 
+### @nkzw/require-use-effect-arguments
+
+Require an explicit dependency array on `useEffect`. Omitting it runs the effect after every render — always a deliberate choice, never an omission; `react-hooks/exhaustive-deps` validates the array's contents but cannot require its presence.
+
+```tsx
+// bad
+useEffect(() => {
+  subscribe(id);
+});
+
+// good
+useEffect(() => {
+  subscribe(id);
+}, [id]);
+```
+
 ### react-hooks/exhaustive-deps
 
 Verify the dependency array for React Hooks like `useEffect`, `useMemo`, and `useCallback` includes all referenced reactive values.

package/awesomeness/index.js

@@ -1,3 +1,4 @@
+// @generated from src/awesomeness.ts — do not edit. Regenerate with `pnpm build`.
 import { definePlugin, defineRule } from "@oxlint/plugins";
 
 const MAX_LINES = 5;

package/bin/init.js

@@ -1,4 +1,5 @@
 #!/usr/bin/env node
+// @generated from src/init.ts — do not edit. Regenerate with `pnpm build`.
 import { constants, copyFileSync } from "node:fs";
 import { resolve } from "node:path";
 import { parseArgs } from "node:util";

package/index.d.ts

@@ -1,3 +1,4 @@
+// @generated from src/index.ts — do not edit. Regenerate with `pnpm build`.
 import type { OxlintConfig } from "oxlint";
 
 declare const config: OxlintConfig;

package/index.js

@@ -1,3 +1,4 @@
+// @generated from src/index.ts — do not edit. Regenerate with `pnpm build`.
 import { defineConfig } from "oxlint";
 
 const UNSAFE_ANY_OFF = {
@@ -58,6 +59,7 @@ const config = defineConfig({
   },
   // JS-plugin bridge for ESLint plugins oxlint doesn't yet implement natively.
   jsPlugins: [
+    "@nkzw/eslint-plugin",
     "eslint-plugin-no-only-tests",
     "eslint-plugin-perfectionist",
     "eslint-plugin-unused-imports",
@@ -274,14 +276,17 @@ const config = defineConfig({
     // Prevent duplicate import statements but still allow separating type-only imports
     // from value imports (`import type { X } from 'x'; import { y } from 'x';`).
     "eslint/no-duplicate-imports": ["error", { allowSeparateTypeImports: true }],
+    // `_`-prefixed names are the convention for deliberately unused bindings
+    // (ignored destructure slots, unused callback params).
+    "no-unused-vars": ["error", { argsIgnorePattern: "^_", varsIgnorePattern: "^_" }],
     // Require the `v` flag on regex literals — opts into Unicode-aware matching with set notation.
     "eslint/require-unicode-regexp": ["error", { requireFlag: "v" }],
     // Block `@nocommit` markers from reaching main — a hard stop for WIP code.
     "no-warning-comments": ["error", { terms: ["@nocommit"] }],
-    // New in oxlint 1.78 under `style`, where its upstream default ("always")
-    // demands comma-combined declarations. Pin to "never": one declaration per
-    // variable, matching how the fleet already writes.
-    "one-var": ["error", "never"],
+    // Lives in `style`, which we bulk-enable, so it needs an explicit "off" —
+    // deleting this line would restore the upstream default ("always"), which
+    // demands comma-combined declarations.
+    "one-var": "off",
     // Restriction — core eslint (cherry-picked from the 90 rules in `restriction`)
     // Always use braces — prevents the dangling-else bug and ambiguous single-line bodies.
     curly: "error",
@@ -389,6 +394,10 @@ const config = defineConfig({
     // Catches typos like `class=` or `tabindex=` in JSX.
     "react/no-unknown-property": "error",
     // Restriction — import graph
+    // Duplicate named exports and re-exports of names that don't exist — invalid
+    // module shapes that only surface at import time. Nursery-classified in
+    // oxlint, so no bulk-enabled category covers it; enabled here by name.
+    "import/export": "error",
     // Circular imports produce `undefined` exports at runtime — a real-world crash source.
     "import/no-cycle": "error",
     // Restriction — unicorn (modern JS + anti-escape-hatch)
@@ -396,6 +405,8 @@ const config = defineConfig({
     "unicorn/no-abusive-eslint-disable": "error",
     // Direct `document.cookie` access — use a cookie library for encoding/security.
     "unicorn/no-document-cookie": "error",
+    // A bare number in `.flat(2)` hides intent — use a named constant or `Infinity`.
+    "unicorn/no-magic-array-flat-depth": "error",
     // `process.exit()` skips `finally` blocks and async flushes. CLI override above handles CLIs.
     "unicorn/no-process-exit": "error",
     // `Math.trunc(x)` over `x | 0` — clearer intent, correct for numbers outside Int32 range.
@@ -544,6 +555,9 @@ const config = defineConfig({
     "perfectionist/sort-enums": ["error", { partitionByComment: true, sortByValue: "always" }],
     // `class Foo extends A, B` — alphabetical for stable diffs.
     "perfectionist/sort-heritage-clauses": "error",
+    // Interfaces are banned by consistent-type-definitions, but `.d.ts` and
+    // declaration-merging files still carry them — keep those sorted too.
+    "perfectionist/sort-interfaces": "error",
     // JSX prop order is a huge diff-noise source — sort it.
     "perfectionist/sort-jsx-props": "error",
     // Type fields sorted alphabetically for scannability.
@@ -598,6 +612,14 @@ const config = defineConfig({
     "react-hooks/exhaustive-deps": "error",
     // Hooks only at the top level, only from React functions — never in conditions/loops.
     "react-hooks/rules-of-hooks": "error",
+    // @nkzw — only require-use-effect-arguments from the plugin's three rules.
+    // ensure-relay-types is Relay-specific (unused in the fleet) and no-instanceof
+    // conflicts with the `err instanceof Error` narrowing our unknown-first
+    // catch-handling rules push toward.
+    // `useEffect(fn)` with no deps array runs after every render — always an
+    // explicit choice, never an omission. exhaustive-deps validates the array's
+    // contents but cannot require its presence.
+    "@nkzw/require-use-effect-arguments": "error",
     // Testing
     // `.only()` in a test file skips every other test — must never land on main.
     "no-only-tests/no-only-tests": "error",

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "oxlint-config-awesomeness",
-  "version": "4.2.0",
+  "version": "4.4.0",
   "description": "Oxlint config with sensible defaults.",
   "keywords": [
     "autofix",
@@ -45,6 +45,7 @@
   },
   "devDependencies": {
     "@changesets/cli": "^3.0.0",
+    "@nkzw/eslint-plugin": "^2.0.0",
     "@types/node": "^26.2.0",
     "eslint-plugin-no-only-tests": "^3.4.0",
     "eslint-plugin-perfectionist": "^5.10.1",
@@ -60,6 +61,7 @@
     "vitest": "^4.1.10"
   },
   "peerDependencies": {
+    "@nkzw/eslint-plugin": "^2.0.0",
     "eslint-plugin-no-only-tests": "^3.4.0",
     "eslint-plugin-perfectionist": "^5.10.1",
     "eslint-plugin-unused-imports": "~4.4.1",