@configg/oxc 0.1.6 → 0.1.8

package/README.md

@@ -40,16 +40,26 @@ export default defineConfig(configs.client)
 - `client`: browser env, type-aware, with rules relaxed for component code. Its `**/*.spec.{ts,tsx}` override enables the Vitest env and Testing Library rules, so it needs `eslint-plugin-testing-library`.
 - `scripts`: Node env, type-aware, for tooling and config files.
 
-Both Oxlint presets include all 18 rules from the vendored [anti-slop](https://github.com/dmmulroy/anti-slop) plugin.
-See [UPSTREAM.md](./src/anti-slop/UPSTREAM.md) for its source and update policy.
-`unicorn/prefer-reflect-apply` is off because anti-slop forbids `Reflect.apply`.
-The plugin ships as TypeScript, so run oxlint under Bun (`bunx --bun oxlint`): Node refuses to strip types from files in `node_modules`.
+The Oxlint presets don't enable anti-slop. `antiSlop` from `@configg/oxc/lint` is an opt-in set with the plugin and all 18 rules as errors; `unicorn/prefer-reflect-apply` is off. See [UPSTREAM.md](./src/anti-slop/UPSTREAM.md) for its source and update policy.
+
+```ts
+import { antiSlop, configs } from '@configg/oxc/lint'
+
+export default defineConfig({
+	...configs.client,
+	jsPlugins: [...antiSlop.jsPlugins],
+	rules: { ...configs.client.rules, ...antiSlop.rules },
+})
+```
+
+If your config has its own `jsPlugins`, keep the anti-slop entry in that array. The plugin ships as TypeScript, so run oxlint under Bun (`bunx --bun oxlint`): Node refuses to strip types from files in `node_modules`.
 
 Both presets allow up to two classes per file, allow `reduce()`, allow `process.exit()` (CLI scripts and
 shutdown handlers exit with a status code on purpose) and allow short identifiers (`eslint/id-length` is off). They turn `import/prefer-default-export` off, since named
 exports are the convention (it matters once a config adds the `import` plugin), and `unicorn/number-literal-case`
 off, since oxfmt and Prettier write hex digits in lowercase. `async`/`await` is allowed (`oxc/no-async-await` is
-off), and functions may run to 100 lines (blank lines and comments not counted) and 30 statements. Template expressions
+off), `continue` is allowed (`eslint/no-continue` is off), and functions may run to 100 lines (blank lines and comments
+not counted) and 30 statements. Template expressions
 allow booleans, numbers, nullish values and `never`; other types follow Oxlint's defaults.
 
 ## Peer dependencies

package/package.json

@@ -1,6 +1,6 @@
 {
 	"name": "@configg/oxc",
-	"version": "0.1.6",
+	"version": "0.1.8",
 	"description": "oxfmt and oxlint presets",
 	"keywords": [
 		"config",

package/src/lint.ts

@@ -1,30 +1,13 @@
 import type { OxlintConfig } from 'oxlint'
 
-const antiSlop = { name: 'anti-slop', specifier: `${import.meta.dirname}/anti-slop/index.ts` }
+const antiSlopPlugin = { name: 'anti-slop', specifier: `${import.meta.dirname}/anti-slop/index.ts` }
 
 const sharedRules = {
-	'anti-slop/no-array-filter-map': 'error',
-	'anti-slop/no-chained-type-assertions': 'error',
-	'anti-slop/no-conditional-empty-object-spread': 'error',
-	'anti-slop/no-known-value-widening': 'error',
-	'anti-slop/no-module-mocking': 'error',
-	'anti-slop/no-object-parameters': 'error',
-	'anti-slop/no-reduce-accumulator-copy': 'error',
-	'anti-slop/no-reflect-apply': 'error',
-	'anti-slop/no-reflect-get': 'error',
-	'anti-slop/no-runtime-typeof': 'error',
-	'anti-slop/no-shape-in-symbol-names': 'error',
-	'anti-slop/no-unknown-parameters': 'error',
-	'anti-slop/no-unknown-returns': 'error',
-	'anti-slop/no-unknown-type-aliases': 'error',
-	'anti-slop/no-unsafe-dictionary-type': 'error',
-	'anti-slop/no-widen-then-assert': 'error',
-	'anti-slop/require-readable-spacing': 'error',
-	'anti-slop/require-safety-comment-for-type-assertion': 'error',
 	'eslint/id-length': 'off',
 	'eslint/max-classes-per-file': ['warn', { max: 2 }],
 	'eslint/max-lines-per-function': ['warn', { max: 100, skipBlankLines: true, skipComments: true }],
 	'eslint/max-statements': ['warn', { max: 30 }],
+	'eslint/no-continue': 'off',
 	'import/prefer-default-export': 'off',
 	'oxc/no-async-await': 'off',
 	'typescript/restrict-template-expressions': [
@@ -40,10 +23,35 @@ const sharedRules = {
 	'unicorn/no-process-exit': 'off',
 	// Oxfmt and Prettier write hex digits in lowercase; this rule wants uppercase and takes no option.
 	'unicorn/number-literal-case': 'off',
-	// This pushes toward Reflect.apply, which anti-slop/no-reflect-apply forbids.
-	'unicorn/prefer-reflect-apply': 'off',
 } as const satisfies OxlintConfig['rules']
 
+/** Opt in by spreading into a preset. The plugin ships as TypeScript, so run oxlint under Bun. */
+export const antiSlop = {
+	jsPlugins: [antiSlopPlugin],
+	rules: {
+		'anti-slop/no-array-filter-map': 'error',
+		'anti-slop/no-chained-type-assertions': 'error',
+		'anti-slop/no-conditional-empty-object-spread': 'error',
+		'anti-slop/no-known-value-widening': 'error',
+		'anti-slop/no-module-mocking': 'error',
+		'anti-slop/no-object-parameters': 'error',
+		'anti-slop/no-reduce-accumulator-copy': 'error',
+		'anti-slop/no-reflect-apply': 'error',
+		'anti-slop/no-reflect-get': 'error',
+		'anti-slop/no-runtime-typeof': 'error',
+		'anti-slop/no-shape-in-symbol-names': 'error',
+		'anti-slop/no-unknown-parameters': 'error',
+		'anti-slop/no-unknown-returns': 'error',
+		'anti-slop/no-unknown-type-aliases': 'error',
+		'anti-slop/no-unsafe-dictionary-type': 'error',
+		'anti-slop/no-widen-then-assert': 'error',
+		'anti-slop/require-readable-spacing': 'error',
+		'anti-slop/require-safety-comment-for-type-assertion': 'error',
+		// This pushes toward Reflect.apply, which anti-slop/no-reflect-apply forbids.
+		'unicorn/prefer-reflect-apply': 'off',
+	},
+} as const satisfies OxlintConfig
+
 export const configs = {
 	client: {
 		categories: {
@@ -59,7 +67,6 @@ export const configs = {
 			browser: true,
 			builtin: true,
 		},
-		jsPlugins: [antiSlop],
 		options: {
 			typeAware: true,
 			typeCheck: true,
@@ -167,7 +174,6 @@ export const configs = {
 			builtin: true,
 			node: true,
 		},
-		jsPlugins: [antiSlop],
 		options: {
 			typeAware: true,
 			typeCheck: true,