@eslint-umbrella/presets 1.1.0 → 1.1.1

package/CHANGELOG.md

@@ -1,5 +1,11 @@
 # @eslint-umbrella/presets
 
+## 1.1.1
+
+### Patch Changes
+
+- 4694128: chore(release): version packages
+
 ## 1.1.0
 
 ### Minor Changes

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@eslint-umbrella/presets",
-  "version": "1.1.0",
+  "version": "1.1.1",
   "type": "module",
   "description": "Pinned ESLint flat config presets for React, Next.js, Express, NestJS, React Native, React-PDF, and base/test",
   "publishConfig": {
@@ -9,15 +9,15 @@
   "exports": {
     ".": "./src/index.js",
     "./base": "./src/base.js",
+    "./base-typechecked": "./src/base-typechecked.js",
     "./react": "./src/react.js",
     "./next": "./src/next.js",
     "./express": "./src/express.js",
     "./nest": "./src/nest.js",
     "./react-native": "./src/react-native.js",
-    "./react-pdf": "./src/react-pdf.js",
-    "./test": "./src/test.js",
-    "./base-typechecked": "./src/base-typechecked.js"
+    "./react-pdf": "./src/react-pdf.js"
   },
+  "sideEffects": false,
   "scripts": {
     "lint": "eslint .",
     "lint:fix": "eslint . --fix",
@@ -28,21 +28,24 @@
   },
   "license": "MIT",
   "dependencies": {
-    "@eslint/js": "9.11.0",
     "@next/eslint-plugin-next": "14.2.14",
-    "eslint": "9.11.0",
     "eslint-plugin-import": "2.31.0",
-    "eslint-plugin-promise": "6.5.1",
+    "eslint-plugin-react": "7.37.1",
     "eslint-plugin-react-hooks": "5.1.0",
     "eslint-plugin-react-refresh": "0.4.12",
     "eslint-plugin-security": "3.0.1",
     "eslint-plugin-unused-imports": "4.1.4",
-    "eslint-plugin-react": "7.37.1",
+    "globals": "^16.4.0",
     "typescript": "5.6.3",
     "typescript-eslint": "8.8.1"
   },
   "devDependencies": {
     "@changesets/cli": "^2.29.7",
-    "eslint-plugin-react-native": "^5.0.0"
+    "@eslint/js": "^9.35.0",
+    "@stylistic/eslint-plugin": "^5.3.1",
+    "eslint": "^9.35.0",
+    "eslint-plugin-promise": "^7.2.1",
+    "eslint-plugin-react-native": "^5.0.0",
+    "nodemon": "^3.1.10"
   }
 }

package/src/base-typechecked.js

@@ -1,52 +1,173 @@
-// src/base-typechecked.js
 import js from "@eslint/js";
 import importPlugin from "eslint-plugin-import";
+import promise from "eslint-plugin-promise";
 import unused from "eslint-plugin-unused-imports";
-import tseslint from "typescript-eslint";             // ← default import
+import tseslint from "typescript-eslint";
 
 /**
- * Type-aware base preset (requires a tsconfig).
+ * Type-aware base preset (requires tsconfig).
  * @param {string[]} project - tsconfig paths (default: ["tsconfig.json"])
  * @returns {import('eslint').Linter.FlatConfig[]}
  */
 export default function baseTypeChecked(project = ["tsconfig.json"]) {
 	return tseslint.config(
-		// 1) general ignores
-		{ ignores: ["dist", "build", "coverage", "**/*.min.*"] },
+		// 0) general ignores
+		{ ignores: [
+		// package managers
+			"node_modules",
+			".pnpm-store", ".turbo", ".npm", ".yarn", ".yarn/*", "pnpm-lock.yaml", "yarn.lock",
 
-		// 2) core JS rules
+			// builds / caches / coverage
+			"dist", "build", "coverage", ".cache", ".eslintcache",
+
+			// env / local
+			".env", ".env.*", "*.local.*",
+
+			// logs & misc
+			"*.log", "logs", "tmp", "temp",
+
+			// assets & generated
+			"**/*.min.*", "**/*.map", "**/*.snap", "public", "static", "storybook-static",
+
+			// tests/artifacts
+			"cypress/videos", "cypress/screenshots", "playwright-report", "test-results",
+
+			// monorepo tool outputs
+			".nx", ".rush", ".changeset/*-changeset/*.md", // keep `.changeset/*.md` itself included
+
+			// IDE
+			".idea", ".vscode/*.tsbuildinfo"
+		] },
+
+		// 1) core JS recommendations
 		js.configs.recommended,
 
-		// 3) TS type-aware recommended presets (this sets the TS parser & plugin)
+		// 2) TS type-aware recommendations
 		...tseslint.configs.recommendedTypeChecked,
 
-		// 4) your extra plugins/rules + the project's tsconfig paths
+		// 3) opinionated, broadly accepted best practices
 		{
-			plugins: { import: importPlugin, "unused-imports": unused },
-			rules: {
-				"no-console": ["warn", { allow: ["warn", "error"] }],
-				"no-debugger": "warn",
-
-				"import/order": ["warn", {
-					alphabetize: { order: "asc", caseInsensitive: true },
-					"newlines-between": "always"
-				}],
-
-				"unused-imports/no-unused-imports": "warn",
-				"unused-imports/no-unused-vars": ["warn", {
-					vars: "all",
-					varsIgnorePattern: "^_",
-					args: "after-used",
-					argsIgnorePattern: "^_"
-				}]
+			plugins: {
+				import: importPlugin,
+				"unused-imports": unused,
+				promise
 			},
+
 			languageOptions: {
 				ecmaVersion: "latest",
 				sourceType: "module",
 				parserOptions: {
-					project,                  // ← pass your tsconfig paths here
-					// tsconfigRootDir: new URL(".", import.meta.url) // usually NOT needed; consumer root is better
+					project
+					// tsconfigRootDir: new URL(".", import.meta.url) // usually not needed
 				}
+			},
+
+			rules: {
+				/* ---------- General safety / clarity ---------- */
+				"no-console": ["warn", { allow: ["warn", "error"] }],
+				"no-debugger": "warn",
+				eqeqeq: ["error", "always", { null: "ignore" }],
+				"no-var": "error",
+				"prefer-const": ["error", { destructuring: "all" }],
+				"object-shorthand": ["error", "always"],
+				"prefer-template": "warn",
+				"no-implicit-coercion": "warn",
+				"no-restricted-syntax": [
+					"error",
+					{
+						selector: "ForInStatement",
+						message:
+							"for..in iterates over prototype chain; use Object.keys()/Object.entries() or for..of."
+					},
+					{
+						selector: "LabeledStatement",
+						message: "Labels make code confusing and hard to maintain."
+					},
+					{
+						selector: "WithStatement",
+						message: "`with` is disallowed in strict mode."
+					}
+				],
+
+				/* ---------- Import hygiene ---------- */
+				"import/order": [
+					"warn",
+					{
+						groups: [
+							"builtin",
+							"external",
+							"internal",
+							["parent", "sibling", "index"],
+							"type",
+							"object",
+							"unknown"
+						],
+						alphabetize: { order: "asc", caseInsensitive: true },
+						"newlines-between": "always"
+					}
+				],
+				// Avoid noisy false-positives without a TS resolver:
+				// "import/no-unresolved": "off",
+
+				/* ---------- Unused code ---------- */
+				"unused-imports/no-unused-imports": "warn",
+				"unused-imports/no-unused-vars": [
+					"warn",
+					{
+						vars: "all",
+						varsIgnorePattern: "^_",
+						args: "after-used",
+						argsIgnorePattern: "^_",
+						ignoreRestSiblings: true
+					}
+				],
+
+				/* ---------- Promise discipline ---------- */
+				"promise/no-return-wrap": "error",
+				"promise/no-new-statics": "error",
+				"promise/no-nesting": "warn",
+				"promise/no-promise-in-callback": "warn",
+				"promise/no-callback-in-promise": "warn",
+				"promise/valid-params": "error",
+
+				/* ---------- TypeScript-specific best practices ---------- */
+				// Prefer `import type` for types (tree-shaking & clarity)
+				"@typescript-eslint/consistent-type-imports": [
+					"error",
+					{ prefer: "type-imports", disallowTypeAnnotations: false, fixStyle: "inline-type-imports" }
+				],
+				// Common foot-guns
+				"@typescript-eslint/no-floating-promises": ["error", { ignoreVoid: true }],
+				"@typescript-eslint/no-misused-promises": [
+					"error",
+					{ checksVoidReturn: { attributes: false } }
+				],
+				"@typescript-eslint/no-unnecessary-condition": [
+					"warn",
+					{ allowConstantLoopConditions: true }
+				],
+				"@typescript-eslint/no-unnecessary-type-assertion": "warn",
+				"@typescript-eslint/prefer-nullish-coalescing": [
+					"warn",
+					{
+						ignoreConditionalTests: true,
+						ignoreMixedLogicalExpressions: true
+					}
+				],
+				"@typescript-eslint/prefer-optional-chain": "warn",
+				"@typescript-eslint/consistent-type-definitions": ["warn", "type"],
+				"@typescript-eslint/method-signature-style": ["warn", "property"],
+				// Allow top-level void for fire-and-forget promises
+				"@typescript-eslint/no-confusing-void-expression": [
+					"error",
+					{ ignoreVoidOperator: true, ignoreArrowShorthand: true }
+				],
+				// Avoid `async` without `await`
+				"@typescript-eslint/require-await": "error",
+
+				/* ---------- Async/await clarity ---------- */
+				"no-return-await": "error",
+				"no-await-in-loop": "warn"
 			}
 		}
 	);

package/src/base.js

@@ -1,53 +1,143 @@
+// src/base.js
 import js from "@eslint/js";
+import stylistic from "@stylistic/eslint-plugin";
 import importPlugin from "eslint-plugin-import";
+import promise from "eslint-plugin-promise";
 import unused from "eslint-plugin-unused-imports";
-import tseslint from "typescript-eslint";           // default import
+import globals from "globals";
+import tseslint from "typescript-eslint"; // just the config composer
 
 /** @returns {import('eslint').Linter.FlatConfig[]} */
 export default function base() {
 	return tseslint.config(
-		{ ignores: ["dist", "build", "coverage", "**/*.min.*"] },
+		/* 0) ignores */
+		{
+			ignores: [
+				// package managers
+				"node_modules",
+				".pnpm-store", ".turbo", ".npm", ".yarn", ".yarn/*", "pnpm-lock.yaml", "yarn.lock",
+
+				// builds / caches / coverage
+				"dist", "build", "coverage", ".cache", ".eslintcache",
+
+				// env / local
+				".env", ".env.*", "*.local.*",
+
+				// logs & misc
+				"*.log", "logs", "tmp", "temp",
+
+				// assets & generated
+				"**/*.min.*", "**/*.map", "**/*.snap", "public", "static", "storybook-static",
+
+				// tests/artifacts
+				"cypress/videos", "cypress/screenshots", "playwright-report", "test-results",
+
+				// monorepo tool outputs
+				".nx", ".rush", ".changeset/*-changeset/*.md",
+
+				// IDE
+				".idea", ".vscode/*.tsbuildinfo",
+			],
+		},
+
+		/* 1) core JS recommendations */
 		js.configs.recommended,
-		...tseslint.configs.recommended,
+
+		/* 2) extras */
 		{
-			plugins: { import: importPlugin, "unused-imports": unused },
+			plugins: {
+				import: importPlugin,
+				"unused-imports": unused,
+				promise,
+				"@stylistic": stylistic,
+			},
+
+			languageOptions: {
+				ecmaVersion: "latest",
+				sourceType: "module",
+				globals: {
+					...globals.es2024,
+					...globals.node,
+				},
+			},
+
 			rules: {
+				/* ---------- General safety / clarity ---------- */
 				"no-console": ["warn", { allow: ["warn", "error"] }],
 				"no-debugger": "warn",
+				eqeqeq: ["error", "always", { null: "ignore" }],
+				"no-var": "error",
+				"prefer-const": ["error", { destructuring: "all" }],
+				"object-shorthand": ["error", "always"],
+				"prefer-template": "warn",
+				"no-implicit-coercion": "warn",
+				curly: ["error", "multi-line", "consistent"],
+				"no-use-before-define": ["error", { functions: false, classes: true, variables: false }],
+
+				"@stylistic/max-len": ["warn", {
+					code: 120,
+					tabWidth: 4,
+					ignoreUrls: true,
+					ignoreStrings: true,
+					ignoreTemplateLiterals: true,
+					ignoreComments: true,
+				}],
+
+				/* ---------- Import hygiene ---------- */
 				"import/order": ["warn", {
+					groups: [
+						"builtin",
+						"external",
+						"internal",
+						["parent", "sibling", "index"],
+						"type",
+						"object",
+						"unknown",
+					],
 					alphabetize: { order: "asc", caseInsensitive: true },
-					"newlines-between": "always"
+					"newlines-between": "always",
 				}],
-				"unused-imports/no-unused-imports": "warn",
-				"unused-imports/no-unused-vars": ["warn", {
+				// "import/no-unresolved": "off", // enable if you add a resolver
+
+				/* ---------- Unused code ---------- */
+				"no-unused-vars": "off",
+				"unused-imports/no-unused-imports": "error",
+				"unused-imports/no-unused-vars": ["error", {
 					vars: "all",
 					varsIgnorePattern: "^_",
 					args: "after-used",
-					argsIgnorePattern: "^_"
-				}],
-				"indent": ["warn", "tab", {
-					"SwitchCase": 1,
-					"VariableDeclarator": 1,
-					"outerIIFEBody": 1,
-					"MemberExpression": 1,
-					"FunctionDeclaration": { "body": 1, "parameters": 1 },
-					"FunctionExpression":  { "body": 1, "parameters": 1 },
-					"CallExpression":      { "arguments": 1 },
-					"ArrayExpression": 1,
-					"ObjectExpression": 1,
-					"ImportDeclaration": 1,
-					"flatTernaryExpressions": false,
-					"ignoreComments": false
+					argsIgnorePattern: "^_",
+					ignoreRestSiblings: true,
 				}],
-				"no-mixed-spaces-and-tabs": ["warn", "smart-tabs"], // avoid mixing, allow smart alignment
-				"no-tabs": "off", // make sure tabs are allowed
 
+				/* ---------- Promise discipline ---------- */
+				"promise/no-return-wrap": "error",
+				"promise/no-new-statics": "error",
+				"promise/no-nesting": "warn",
+				"promise/no-promise-in-callback": "warn",
+				"promise/no-callback-in-promise": "warn",
+				"promise/valid-params": "error",
+
+				/* ---------- Stylistic: tabs, no JSX conflict ---------- */
+				"@stylistic/indent": ["error", "tab", {
+					SwitchCase: 1,
+					VariableDeclarator: 1,
+					outerIIFEBody: 1,
+					MemberExpression: 1,
+					FunctionDeclaration: { body: 1, parameters: 1 },
+					FunctionExpression: { body: 1, parameters: 1 },
+					CallExpression: { arguments: 1 },
+					ArrayExpression: 1,
+					ObjectExpression: 1,
+					ImportDeclaration: 1,
+					flatTernaryExpressions: false,
+					ignoreComments: false,
+					// avoid conflict with JSX rules handled in the react preset
+					ignoredNodes: ["JSXElement", "JSXElement *"],
+				}],
+				"@stylistic/no-mixed-spaces-and-tabs": ["warn", "smart-tabs"],
+				"no-tabs": "off",
 			},
-			languageOptions: {
-				ecmaVersion: "latest",
-				sourceType: "module"
-				// no parserOptions.project here (non type-aware preset)
-			}
 		}
 	);
 }

package/src/express.js

@@ -1,16 +1,55 @@
-import promise from "eslint-plugin-promise";
 import security from "eslint-plugin-security";
 
 import base from "./base.js";
+
+/** @returns {import('eslint').Linter.FlatConfig[]} */
 export default function express() {
 	return [
 		...base(),
-		{ plugins: { security, promise },
-			languageOptions: { globals: { process: "readonly", __dirname: "readonly", module: "readonly" } },
+		{ ignores: ["dist", "build", "coverage"] },	
+		{
+			plugins: { security },
 			rules: {
-				"no-restricted-syntax": ["warn", { selector: "CallExpression[callee.name='eval']", message: "Avoid eval() in Node." }],
-				"promise/catch-or-return": "warn",
-				"security/detect-object-injection": "off"
-			} }
+				/* ---------- Server-friendly overrides ---------- */
+				// Servers usually allow logging:
+				"no-console": "off",
+
+				/* ---------- Import hygiene ---------- */
+				// Disallow requiring deps not listed in package.json
+				"import/no-extraneous-dependencies": ["error", {
+					devDependencies: [
+						"**/*.test.*",
+						"**/*.spec.*",
+						"**/tests/**",
+						"**/scripts/**",
+						"**/*.config.*",
+						"**/eslint.config.*"
+					],
+					optionalDependencies: false,
+					peerDependencies: true
+				}],
+
+				/* ---------- Security checks (pragmatic set) ---------- */
+				// Avoid eval and similar dynamic code execution
+				"no-restricted-syntax": [
+					"error",
+					{ selector: "CallExpression[callee.name='eval']", message: "Avoid eval() in Node." },
+					{ selector: "NewExpression[callee.name='Function']", message: "Avoid new Function() in Node." }
+				],
+				"security/detect-child-process": "warn",
+				"security/detect-non-literal-fs-filename": "warn",
+				"security/detect-non-literal-regexp": "off",
+				"security/detect-possible-timing-attacks": "warn",
+				"security/detect-unsafe-regex": "warn",
+				// Too noisy for route objects & query builders:
+				"security/detect-object-injection": "off",
+
+				/* ---------- Promise discipline (adds to base) ---------- */
+				"promise/catch-or-return": ["warn", { allowThen: true, terminationMethod: ["catch", "finally"] }],
+				"promise/no-return-in-finally": "error",
+				"promise/prefer-await-to-then": "warn",
+				"promise/prefer-await-to-callbacks": "off"
+			}
+		}
 	];
 }

package/src/index.js

@@ -1,9 +1,39 @@
-export { default as base } from "./base.js";
-export { default as react } from "./react.js";
-export { default as next } from "./next.js";
-export { default as express } from "./express.js";
-export { default as nest } from "./nest.js";
-export { default as reactNative } from "./react-native.js";
-export { default as reactPdf } from "./react-pdf.js";
-export { default as test } from "./test.js";
-export { default as baseTypeChecked } from "./base-typechecked.js";
+// src/index.js
+
+// Import each preset’s default export:
+import baseTypeChecked from "./base-typechecked.js";
+import base from "./base.js";
+import express from "./express.js";
+import nestTypeChecked from "./nest-typechecked.js";
+import nest from "./nest.js";
+import next from "./next.js";
+import reactNative from "./react-native.js";
+import reactPdf from "./react-pdf.js";
+import react from "./react.js";
+
+// Re-export as **named** exports:
+export {
+	base,
+	baseTypeChecked,
+	react,
+	next,
+	express,
+	nest,
+	reactNative,
+	reactPdf,
+	nestTypeChecked,
+};
+
+// Optional: also provide a default object for people who prefer it.
+// (Named imports are better for tree-shaking.)
+export default {
+	base,
+	baseTypeChecked,
+	react,
+	next,
+	express,
+	nest,
+	reactNative,
+	reactPdf,
+	nestTypeChecked,
+};

package/src/nest-typechecked.js

@@ -0,0 +1,49 @@
+import baseTypeChecked from "./base-typechecked.js";
+
+/**
+ * Type-aware NestJS preset (requires tsconfig).
+ * @param {string[]} project - tsconfig paths (default: ["tsconfig.json"])
+ * @returns {import('eslint').Linter.FlatConfig[]}
+ */
+export default function nestTypeChecked(project = ["tsconfig.json"]) {
+	return [
+		...baseTypeChecked(project),
+
+		{
+			languageOptions: {
+				globals: {
+					process: "readonly",
+					__dirname: "readonly",
+					__filename: "readonly",
+					module: "readonly",
+					require: "readonly"
+				},
+				parserOptions: {
+					project
+				}
+			},
+
+			rules: {
+				"no-useless-constructor": "off",
+				"no-empty-function": ["warn", { allow: ["constructors", "methods"] }],
+				"class-methods-use-this": "off",
+
+				// TS-specific niceties for Nest
+				"@typescript-eslint/no-floating-promises": ["error", { ignoreVoid: true }],
+				"@typescript-eslint/no-misused-promises": ["error", { checksVoidReturn: { attributes: false } }],
+				"@typescript-eslint/consistent-type-imports": [
+					"error",
+					{ prefer: "type-imports", fixStyle: "inline-type-imports" }
+				],
+
+				"no-restricted-syntax": [
+					"error",
+					{ selector: "CallExpression[callee.name='eval']", message: "Avoid eval() in Node." },
+					{ selector: "NewExpression[callee.name='Function']", message: "Avoid new Function() in Node." }
+				],
+
+				"promise/catch-or-return": ["warn", { allowThen: true, terminationMethod: ["catch", "finally"] }]
+			}
+		}
+	];
+}

package/src/nest.js

@@ -1,15 +1,44 @@
-import promise from "eslint-plugin-promise";
-
 import base from "./base.js";
+
+/** @returns {import('eslint').Linter.FlatConfig[]} */
 export default function nest() {
 	return [
 		...base(),
-		{ plugins: { promise },
-			languageOptions: { globals: { process: "readonly" } },
+		{ ignores: ["dist", "build", "coverage"] },	
+		{
+			// Node-ish globals commonly used in Nest environments
+			languageOptions: {
+				globals: {
+					process: "readonly",
+					__dirname: "readonly",
+					__filename: "readonly",
+					module: "readonly",
+					require: "readonly"
+				}
+			},
+
 			rules: {
-				"no-restricted-syntax": ["warn", { selector: "CallExpression[callee.name='eval']", message: "Avoid eval() in Node." }],
-				"promise/catch-or-return": "warn",
-				"no-useless-constructor": "off"
-			} }
+				/* ----- Server-friendly relaxations for Nest patterns ----- */
+				// Constructors with only DI are fine
+				"no-useless-constructor": "off",
+
+				// Allow empty constructors/handlers when required by decorators/DI
+				"no-empty-function": ["warn", { allow: ["constructors", "methods"] }],
+
+				// Nest handlers/providers often don't need `this`
+				"class-methods-use-this": "off",
+
+				/* ----- General safety ----- */
+				"no-restricted-syntax": [
+					"error",
+					{ selector: "CallExpression[callee.name='eval']", message: "Avoid eval() in Node." },
+					{ selector: "NewExpression[callee.name='Function']", message: "Avoid new Function() in Node." }
+				],
+
+				/* ----- Promise discipline (base already enables most good ones) ----- */
+				// Keep this mild guard for route handlers/services
+				"promise/catch-or-return": ["warn", { allowThen: true, terminationMethod: ["catch", "finally"] }]
+			}
+		}
 	];
 }

package/src/next.js

@@ -1,9 +1,41 @@
 import nextPlugin from "@next/eslint-plugin-next";
 
 import react from "./react.js";
+
+/** @returns {import('eslint').Linter.FlatConfig[]} */
 export default function next() {
 	return [
 		...react(),
-		{ plugins: { "@next/next": nextPlugin }, rules: { "@next/next/no-img-element": "off" } }
+		{ ignores: [".next", ".vercel", "out"] },
+		{
+			plugins: { "@next/next": nextPlugin },
+
+			// Browser + Node globals commonly used in Next apps
+			languageOptions: {
+				globals: {
+					window: "readonly",
+					document: "readonly",
+					navigator: "readonly",
+					fetch: "readonly",
+					URL: "readonly",
+					Request: "readonly",
+					Response: "readonly",
+					Headers: "readonly",
+					process: "readonly"
+				}
+			},
+
+			rules: {
+				"@next/next/no-img-element": "off",
+				"@next/next/no-sync-scripts": "error",
+				"@next/next/inline-script-id": "error",
+				"@next/next/no-css-tags": "error",
+				"@next/next/no-document-import-in-page": "error",
+				"@next/next/no-unwanted-polyfillio": "error",
+				"@next/next/no-typos": "error",
+				"@next/next/next-script-for-ga": "warn",
+				"@next/next/no-html-link-for-pages": "off"
+			}
+		}
 	];
 }

package/src/react-native.js

@@ -1,15 +1,43 @@
 import rn from "eslint-plugin-react-native";
 
 import react from "./react.js";
+
+/** @returns {import('eslint').Linter.FlatConfig[]} */
 export default function reactNative() {
 	return [
 		...react(),
-		{ plugins: { "react-native": rn },
-			languageOptions: { globals: { __DEV__: "readonly" } },
+		{ ignores: ["android", "ios", ".expo", ".expo-shared", "expo-shared"] },
+		{
+			plugins: { "react-native": rn },
+
+			// Common RN globals
+			languageOptions: {
+				globals: {
+					__DEV__: "readonly",
+					fetch: "readonly",
+					FormData: "readonly",
+					alert: "readonly",
+					Request: "readonly",
+					Response: "readonly",
+					Headers: "readonly"
+				}
+			},
+
 			rules: {
+				/* --- Core RN best practices --- */
 				"react-native/no-inline-styles": "warn",
 				"react-native/no-unused-styles": "warn",
-				"react-native/split-platform-components": "warn"
-			} }
+				"react-native/split-platform-components": "warn",
+
+				// Useful extras (less noise than errors):
+				"react-native/no-raw-text": "warn",                // text must be inside <Text>
+				"react-native/no-color-literals": "warn",          // encourage theme/tokens
+				"react-native/no-single-element-style-arrays": "warn"
+
+				// If you hit unresolved imports with .ios/.android extensions,
+				// either add a TS resolver or disable the rule in this preset:
+				// "import/no-unresolved": "off"
+			}
+		}
 	];
 }

package/src/react-pdf.js

@@ -1,14 +1,42 @@
 import reactHooks from "eslint-plugin-react-hooks";
 
 import base from "./base.js";
+
+/** @returns {import('eslint').Linter.FlatConfig[]} */
 export default function reactPdf() {
 	return [
 		...base(),
-		{ plugins: { "react-hooks": reactHooks },
-			languageOptions: { globals: { process: "readonly", Buffer: "readonly" } },
+		{ ignores: ["*.pdf"] },
+		{
+			plugins: { "react-hooks": reactHooks },
+
+			// React-PDF usually runs in Node, not the browser
+			languageOptions: {
+				globals: {
+					process: "readonly",
+					Buffer: "readonly",
+					__dirname: "readonly",
+					__filename: "readonly",
+					module: "readonly",
+					require: "readonly"
+				}
+			},
+
 			rules: {
+				/* React Hooks basics */
 				"react-hooks/rules-of-hooks": "error",
-				"react-hooks/exhaustive-deps": "warn"
-			} }
+				"react-hooks/exhaustive-deps": "warn",
+
+				/* Node-side safety */
+				"no-restricted-syntax": [
+					"error",
+					{ selector: "CallExpression[callee.name='eval']", message: "Avoid eval() in Node." },
+					{ selector: "NewExpression[callee.name='Function']", message: "Avoid new Function() in Node." }
+				],
+
+				/* Servers can log while generating PDFs */
+				"no-console": "off"
+			}
+		}
 	];
 }

package/src/react.js

@@ -1,30 +1,57 @@
+// src/react.js
+import stylistic from "@stylistic/eslint-plugin";
 import reactPlugin from "eslint-plugin-react";
 import reactHooks from "eslint-plugin-react-hooks";
 import reactRefresh from "eslint-plugin-react-refresh";
+import globals from "globals"; 
 
 import base from "./base.js";
 
-export default function react() {
+/** @param {{ files?: string[] }=} options
+ *  @returns {import('eslint').Linter.FlatConfig[]} */
+export default function react(options = {}) {
+	const files = options.files ?? ["**/*.{js,jsx,ts,tsx}"];
+
 	return [
 		...base(),
+
 		{
 			plugins: {
+				react: reactPlugin,
 				"react-hooks": reactHooks,
 				"react-refresh": reactRefresh,
-				"react": reactPlugin
+				"@stylistic": stylistic,
+			},
+
+			languageOptions: {
+				parserOptions: { ecmaFeatures: { jsx: true } },
+				globals: { ...globals.browser },
 			},
+
+			settings: { react: { version: "detect" } },
+
 			rules: {
+				/* ---------- React hooks & fast refresh ---------- */
 				"react-hooks/rules-of-hooks": "error",
 				"react-hooks/exhaustive-deps": "warn",
-				"react-refresh/only-export-components": ["warn", { allowConstantExport: true }],
+				"react-refresh/only-export-components": "off",
+
+				/* ---------- Core React sanity ---------- */
+				"react/react-in-jsx-scope": "off",
+				"react/prop-types": "off",
+				"react/jsx-key": ["error", { checkFragmentShorthand: true }],
+				"react/jsx-no-duplicate-props": "error",
+				"react/jsx-no-comment-textnodes": "warn",
+				"react/no-unstable-nested-components": "off",
+				"react/no-danger": "warn",
+				"react/self-closing-comp": "warn",
+				"react/jsx-boolean-value": ["warn", "never"],
+				"react/jsx-uses-vars": "error", // ensure JSX identifiers count as "used"
 
-				// JSX indentation (2 spaces)
-				"react/jsx-indent": ["warn", "tab"],
-				"react/jsx-indent-props": ["warn", "tab"]
+				/* ---------- JSX indentation via @stylistic (tabs) ---------- */
+				"@stylistic/jsx-indent-props": ["error", "tab"],
+				// "@stylistic/jsx-quotes": ["warn", "prefer-double"],
 			},
-			settings: {
-				react: { version: "detect" }
-			}
-		}
-	];
-}
+		},
+	].map(cfg => ("ignores" in cfg ? cfg : { ...cfg, files }));
+}

package/src/test.js

@@ -1,11 +0,0 @@
-import base from "./base.js";
-export default function test() {
-	return [
-		...base(),
-		{ languageOptions: { globals: {
-			describe: "readonly", it: "readonly", test: "readonly", expect: "readonly",
-			beforeAll: "readonly", afterAll: "readonly", beforeEach: "readonly", afterEach: "readonly"
-		} },
-		rules: { "no-undef": "off" } }
-	];
-}