@mkrz/oxlint-config 0.1.0 → 0.1.1

package/CHANGELOG.md

@@ -1,5 +1,17 @@
 # Changelog
 
+## 0.1.1 (2026-09-06)
+
+- Stop `--fix` from adding `new` to curried Error factories such as Effect's
+  `Schema.TaggedError`. Disable the name-based `unicorn/throw-new-error` rule;
+  type-aware `typescript/only-throw-error` remains enabled.
+- Require separate top-level `import type` declarations and forbid inline type
+  specifiers, including in mixed imports. Preserve erased type imports under
+  `verbatimModuleSyntax` to avoid loading declaration-only modules.
+
+- Fix false positives for `Record` types with finite keys, including aliases
+  and generic substitutions. Broad dictionary keys remain checked.
+
 ## 0.1.0 (2026-09-04)
 
 First release.

package/README.md

@@ -131,6 +131,14 @@ of this package. Oxlint only honors `ignorePatterns`, `env`, `globals`, and
 wrapper folds them back in. The ignore list is also exported as
 `ignorePatterns`.
 
+### Type imports
+
+Always use separate top-level `import type` declarations for types. Inline
+type specifiers are forbidden, including in imports that also contain values.
+With `verbatimModuleSyntax`, an all-inline type import leaves a runtime
+`import {}` behind and can fail for packages that contain only declarations.
+The config also checks for these unintended runtime dependencies.
+
 ## Repository types
 
 - `app`: no Oxlint or ESLint disable comment anywhere.

package/dist/index.js

@@ -101,7 +101,7 @@ const base = {
 			"always",
 			{ null: "ignore" }
 		],
-		"import/consistent-type-specifier-style": ["error", "prefer-inline"],
+		"import/consistent-type-specifier-style": ["error", "prefer-top-level"],
 		"import/no-duplicates": "error",
 		"import/no-relative-parent-imports": "error",
 		"no-alert": "error",
@@ -137,12 +137,13 @@ const base = {
 		"typescript/consistent-indexed-object-style": "error",
 		"typescript/consistent-type-definitions": ["error", "type"],
 		"typescript/consistent-type-imports": ["error", {
-			fixStyle: "inline-type-imports",
+			fixStyle: "separate-type-imports",
 			prefer: "type-imports"
 		}],
 		"typescript/no-empty-object-type": "error",
 		"typescript/no-explicit-any": "error",
 		"typescript/no-inferrable-types": "error",
+		"typescript/no-import-type-side-effects": "error",
 		"typescript/no-namespace": "error",
 		"typescript/no-non-null-asserted-nullish-coalescing": "error",
 		"typescript/no-non-null-assertion": "error",
@@ -161,7 +162,7 @@ const base = {
 		"unicorn/prefer-date-now": "error",
 		"unicorn/prefer-node-protocol": "error",
 		"unicorn/prefer-number-properties": "error",
-		"unicorn/throw-new-error": "error"
+		"unicorn/throw-new-error": "off"
 	},
 	overrides: [{
 		files: [
@@ -195,34 +196,6 @@ const base = {
 	}]
 };
 //#endregion
-//#region src/configs/monorepo.ts
-const message = "Move shared code into a package and import it from there instead.";
-/**
-* Overrides for the apps/packages layout.
-*
-* Relative imports that climb out of a package are already rejected by
-* `import/no-relative-parent-imports` in `base`, so only workspace package
-* names need a rule here.
-*/
-function monorepo({ appsPath = ["apps/**"], packagesPath = ["packages/**"], appPackages = [] } = {}) {
-	const packageRules = { "mkrz/package-disable-policy": "error" };
-	if (appPackages.length > 0) {
-		const patterns = appPackages.flatMap((name) => [name, `${name}/**`]);
-		packageRules["mkrz/no-app-requires"] = ["error", patterns];
-		packageRules["no-restricted-imports"] = ["error", { patterns: [{
-			group: patterns,
-			message
-		}] }];
-	}
-	return { overrides: [{
-		files: appsPath,
-		rules: { "mkrz/no-oxlint-disable": "error" }
-	}, {
-		files: packagesPath,
-		rules: packageRules
-	}] };
-}
-//#endregion
 //#region src/configs/plugin-specifiers.ts
 const modulePath = fileURLToPath(import.meta.url);
 const moduleExtension = path.extname(modulePath);
@@ -368,6 +341,34 @@ const react = {
 	}
 };
 //#endregion
+//#region src/configs/monorepo.ts
+const message = "Move shared code into a package and import it from there instead.";
+/**
+* Overrides for the apps/packages layout.
+*
+* Relative imports that climb out of a package are already rejected by
+* `import/no-relative-parent-imports` in `base`, so only workspace package
+* names need a rule here.
+*/
+function monorepo({ appsPath = ["apps/**"], packagesPath = ["packages/**"], appPackages = [] } = {}) {
+	const packageRules = { "mkrz/package-disable-policy": "error" };
+	if (appPackages.length > 0) {
+		const patterns = appPackages.flatMap((name) => [name, `${name}/**`]);
+		packageRules["mkrz/no-app-requires"] = ["error", patterns];
+		packageRules["no-restricted-imports"] = ["error", { patterns: [{
+			group: patterns,
+			message
+		}] }];
+	}
+	return { overrides: [{
+		files: appsPath,
+		rules: { "mkrz/no-oxlint-disable": "error" }
+	}, {
+		files: packagesPath,
+		rules: packageRules
+	}] };
+}
+//#endregion
 //#region src/configs/repository.ts
 const monorepoDirectories = ["apps", "packages"];
 /**

package/dist/plugin/index.js

@@ -604,8 +604,9 @@ function dictionaryValueTypes(type, environment, substitutions, resolvingAliases
 		return wrapped === void 0 ? [] : dictionaryValueTypes(wrapped, environment, substitutions, resolvingAliases);
 	}
 	if (name === "Record" && isBuiltIn(name, environment)) {
-		const value = unwrapped.typeArguments?.params[1] ?? null;
-		return value === null ? [] : [{
+		const key = unwrapped.typeArguments?.params[0];
+		const value = unwrapped.typeArguments?.params[1];
+		return key === void 0 || value === void 0 || !isBroadMappedKey(key, environment, substitutions) ? [] : [{
 			type: value,
 			substitutions
 		}];
@@ -656,7 +657,10 @@ function classifyWideningTarget(type, environment, shadowed = /* @__PURE__ */ ne
 		const wrapped = unwrapped.typeArguments?.params[0];
 		return wrapped === void 0 ? null : classifyWideningTarget(wrapped, environment, shadowed);
 	}
-	if (name === "Record" && isBuiltIn(name, environment)) return { kind: "open dictionary" };
+	if (name === "Record" && isBuiltIn(name, environment)) {
+		const key = unwrapped.typeArguments?.params[0];
+		return key !== void 0 && isBroadMappedKey(key, environment, shadowedMap) ? { kind: "open dictionary" } : { kind: "anonymous object" };
+	}
 	const alias = environment.aliases.get(name);
 	if (alias === void 0) return null;
 	if ((alias.typeParameters?.params.length ?? 0) > 0) {
@@ -700,7 +704,10 @@ function classifyAliasBroadTarget(type, environment, substitutions, resolvingAli
 		const wrapped = unwrapped.typeArguments?.params[0];
 		return wrapped === void 0 ? null : classifyAliasBroadTarget(wrapped, environment, substitutions, resolvingAliases);
 	}
-	if (name === "Record" && isBuiltIn(name, environment)) return { kind: "open dictionary" };
+	if (name === "Record" && isBuiltIn(name, environment)) {
+		const key = unwrapped.typeArguments?.params[0];
+		return key !== void 0 && isBroadMappedKey(key, environment, substitutions) ? { kind: "open dictionary" } : null;
+	}
 	const alias = environment.aliases.get(name);
 	if (alias === void 0 || resolvingAliases.has(name)) return null;
 	const nextSubstitutions = aliasSubstitution(alias, unwrapped, substitutions);

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@mkrz/oxlint-config",
-  "version": "0.1.0",
+  "version": "0.1.1",
   "description": "Opinionated Oxlint ruleset for TypeScript and React",
   "keywords": [
     "lint",