@forthtilliath/eslint-config 0.9.0 → 0.10.0

package/README.md

@@ -32,24 +32,25 @@ the variant and options it actually uses. A React Native app doesn't get
 `angular-eslint` (and the Angular CLI it pulls in), nor a TypeScript library
 every React plugin.
 
-| Variant / option        | Also install                                                                                              |
-| ----------------------- | --------------------------------------------------------------------------------------------------------- |
-| `.` (base)              | —                                                                                                         |
-| `/react`                | `@eslint-react/eslint-plugin eslint-plugin-react eslint-plugin-react-hooks eslint-plugin-react-refresh`   |
-| `/react-native`         | the `/react` list + `eslint-plugin-react-native`                                                          |
-| `/nextjs`               | the `/react` list + `@next/eslint-plugin-next`                                                            |
-| `/storybook`            | the `/react` list + `eslint-plugin-storybook`                                                             |
-| `/angular`              | `angular-eslint`                                                                                          |
-| `turbo: true` (default) | `eslint-plugin-turbo` — or pass `turbo: false` outside a Turborepo                                        |
-| `a11y: true`\*          | `eslint-plugin-jsx-a11y` — \*default for `/react`, `/nextjs`, `/storybook`; pass `a11y: false` to skip it |
-| `i18n: true`            | `eslint-plugin-i18next`                                                                                   |
-| `testingLibrary: true`  | `eslint-plugin-testing-library eslint-plugin-jest-dom`                                                    |
+| Variant / option       | Also install                                                                                              |
+| ---------------------- | --------------------------------------------------------------------------------------------------------- |
+| `.` (base)             | —                                                                                                         |
+| `/react`               | `@eslint-react/eslint-plugin eslint-plugin-react eslint-plugin-react-hooks eslint-plugin-react-refresh`   |
+| `/react-native`        | the `/react` list + `eslint-plugin-react-native`                                                          |
+| `/nextjs`              | the `/react` list + `@next/eslint-plugin-next`                                                            |
+| `/storybook`           | the `/react` list + `eslint-plugin-storybook`                                                             |
+| `/angular`             | `angular-eslint`                                                                                          |
+| `turbo: true`\*\*      | `eslint-plugin-turbo` — \*\*default inside a Turborepo (a `turbo.json` in the working directory or above) |
+| `a11y: true`\*         | `eslint-plugin-jsx-a11y` — \*default for `/react`, `/nextjs`, `/storybook`; pass `a11y: false` to skip it |
+| `i18n: true`           | `eslint-plugin-i18next`                                                                                   |
+| `testingLibrary: true` | `eslint-plugin-testing-library eslint-plugin-jest-dom`                                                    |
+| `expo: true`           | `eslint-plugin-expo` — `/react-native` only                                                               |
 
 ```bash
-# e.g. a standalone React Native / Expo app, with createReactNativeConfig({ turbo: false })
+# e.g. a standalone React Native / Expo app, with createReactNativeConfig({ expo: true })
 npm install --save-dev @forthtilliath/eslint-config eslint typescript \
   @eslint-react/eslint-plugin eslint-plugin-react eslint-plugin-react-hooks \
-  eslint-plugin-react-refresh eslint-plugin-react-native
+  eslint-plugin-react-refresh eslint-plugin-react-native eslint-plugin-expo
 ```
 
 A missing plugin fails with an explicit error naming the package to install
@@ -123,9 +124,21 @@ export default reactNativeConfig;
 
 All six are **named** exports — a default import (`import config from "..."`)
 resolves to the whole module namespace object instead of the config array and
-crashes ESLint's flat-config loader outright. Two of the five variants had
+crashes ESLint's flat-config loader outright. Two of the variants had
 exactly this bug at one point; use the named-import form above.
 
+The config loads from a CommonJS `eslint.config.js` too (the one Expo's
+templates generate), through `require()` — no top-level `await` anywhere in
+the package. That needs Node 20.19+ or 22.12+ (`require()` of an ES module):
+
+```js
+// eslint.config.js (CommonJS)
+const {
+  createReactNativeConfig,
+} = require("@forthtilliath/eslint-config/react-native");
+module.exports = createReactNativeConfig({ expo: true });
+```
+
 Each variant is additive: `reactConfig`/`angularConfig` extend `baseConfig`,
 `nextJsConfig`/`storybookConfig`/`reactNativeConfig` extend `reactConfig`.
 
@@ -146,20 +159,21 @@ export default createReactConfig({ prettier: false });
 ```ts
 // Fall back to typescript-eslint's `recommended` preset instead of
 // `strictTypeChecked`/`stylisticTypeChecked` (e.g. while migrating an
-// existing codebase), and skip the Turborepo-only rule outside a Turborepo.
+// existing codebase).
 import { createBaseConfig } from "@forthtilliath/eslint-config";
-export default createBaseConfig({ strict: false, turbo: false });
+export default createBaseConfig({ strict: false });
 ```
 
 | Option           | Default  | Effect                                                                                                                                                                                                   |
 | ---------------- | -------- | -------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------- |
 | `prettier`       | `true`   | Append `eslint-config-prettier` at the end.                                                                                                                                                              |
 | `strict`         | `true`   | Use `strictTypeChecked`/`stylisticTypeChecked` instead of plain `recommended`.                                                                                                                           |
-| `turbo`          | `true`   | Enable `eslint-plugin-turbo`'s `no-undeclared-env-vars` rule.                                                                                                                                            |
+| `turbo`          | auto     | Enable `eslint-plugin-turbo`'s `no-undeclared-env-vars` rule. Defaults to `true` when a `turbo.json` sits in ESLint's working directory or one of its parents, `false` otherwise.                        |
 | `snakeCase`      | `false`  | Also accept `snake_case` variable names in `@typescript-eslint/naming-convention` (e.g. rows destructured straight from a Supabase/SQL database).                                                        |
 | `a11y`           | `true`\* | _(react/nextjs/storybook/react-native)_ Enable `eslint-plugin-jsx-a11y`'s recommended rules. \*Defaults to `false` in `createReactNativeConfig` (jsx-a11y targets DOM semantics, which RN doesn't have). |
 | `i18n`           | `false`  | _(react/nextjs/storybook/react-native)_ Enable `eslint-plugin-i18next`'s `no-literal-string` rule.                                                                                                       |
 | `testingLibrary` | `false`  | _(react/nextjs/storybook/react-native)_ Enable `eslint-plugin-testing-library` + `eslint-plugin-jest-dom` on test files.                                                                                 |
+| `expo`           | `false`  | _(react-native)_ Enable the `eslint-plugin-expo` rules `eslint-config-expo` turns on: `use-dom-exports`, `no-env-var-destructuring`, `no-dynamic-env-var`.                                               |
 
 Options are forwarded down the chain, so `createNextJsConfig({ prettier: false })`
 also disables Prettier in the `reactConfig`/`baseConfig` layers it builds on.

package/dist/base.d.ts

@@ -8,10 +8,11 @@
  *   `strictTypeChecked` + `stylisticTypeChecked` presets. Set to `false` to
  *   fall back to the plain `recommended` preset instead (useful while
  *   migrating an existing codebase that isn't ready for the strict rules).
- * @property {boolean} [turbo=true] - Enable eslint-plugin-turbo's
+ * @property {boolean} [turbo] - Enable eslint-plugin-turbo's
  *   `no-undeclared-env-vars` rule (needs `eslint-plugin-turbo` installed).
- *   Only relevant inside a Turborepo; set to `false` for a standalone
- *   project.
+ *   Only relevant inside a Turborepo: defaults to `true` when a `turbo.json`
+ *   sits in ESLint's working directory or one of its parents, `false`
+ *   otherwise.
  * @property {boolean} [snakeCase=false] - Also accept `snake_case` variable
  *   names in `@typescript-eslint/naming-convention` — for code that
  *   destructures rows straight from a SQL database (e.g. Supabase), whose
@@ -72,8 +73,9 @@ export type BaseConfigOptions = {
     /**
      * - Enable eslint-plugin-turbo's
      * `no-undeclared-env-vars` rule (needs `eslint-plugin-turbo` installed).
-     * Only relevant inside a Turborepo; set to `false` for a standalone
-     * project.
+     * Only relevant inside a Turborepo: defaults to `true` when a `turbo.json`
+     * sits in ESLint's working directory or one of its parents, `false`
+     * otherwise.
      */
     turbo?: boolean | undefined;
     /**

package/dist/base.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"base.d.ts","sourceRoot":"","sources":["../src/base.js"],"names":[],"mappings":"AAgCA;;;;;;;;;;;;;;;;;;GAkBG;AAEH;;;;;;;;;;GAUG;AACH,mEALG;IAA0B,SAAS;IACT,KAAK;CAE/B,GAAU,OAAO,QAAQ,EAAE,MAAM,CAAC,SAAS,CA+B7C;AAED;;;;;GAKG;AACH,2EAHW,iBAAiB,GACf,OAAO,QAAQ,EAAE,MAAM,CAAC,MAAM,EAAE,CAiH5C;AAlMD;;;;;;GAMG;AACH,wCAFU,MAAM,EAAE,CAQhB;AAuLF;;;;;KAKK;AACL,yBAFU,OAAO,QAAQ,EAAE,MAAM,CAAC,MAAM,EAAE,CAEqB"}
+{"version":3,"file":"base.d.ts","sourceRoot":"","sources":["../src/base.js"],"names":[],"mappings":"AA0BA;;;;;;;;;;;;;;;;;;;GAmBG;AAEH;;;;;;;;;;GAUG;AACH,mEALG;IAA0B,SAAS;IACT,KAAK;CAE/B,GAAU,OAAO,QAAQ,EAAE,MAAM,CAAC,SAAS,CA+B7C;AAED;;;;;GAKG;AACH,2EAHW,iBAAiB,GACf,OAAO,QAAQ,EAAE,MAAM,CAAC,MAAM,EAAE,CAmH5C;AArMD;;;;;;GAMG;AACH,wCAFU,MAAM,EAAE,CAQhB;AA0LF;;;;;KAKK;AACL,yBAFU,OAAO,QAAQ,EAAE,MAAM,CAAC,MAAM,EAAE,CAEqB"}

package/dist/base.js

@@ -5,10 +5,8 @@ import simpleImportSortPlugin from "eslint-plugin-simple-import-sort";
 import globals from "globals";
 import tseslint from "typescript-eslint";
 import { lazyConfig } from "./internal/lazy-config.js";
-import { importOptional, requirePlugin } from "./internal/optional-plugin.js";
-// Optional peer dependency: only a Turborepo needs it (and its own `turbo`
-// peer dependency, the whole Turborepo binary).
-const turboPlugin = await importOptional("eslint-plugin-turbo", () => import("eslint-plugin-turbo"));
+import { requireOptional } from "./internal/optional-plugin.js";
+import { isInsideTurborepo } from "./internal/turborepo.js";
 /**
  * Node.js builtins without the `node:` prefix, as the first
  * `simple-import-sort/imports` group. Exported so other variants (e.g.
@@ -33,10 +31,11 @@ export const NODE_BUILTIN_IMPORT_GROUP = [
  *   `strictTypeChecked` + `stylisticTypeChecked` presets. Set to `false` to
  *   fall back to the plain `recommended` preset instead (useful while
  *   migrating an existing codebase that isn't ready for the strict rules).
- * @property {boolean} [turbo=true] - Enable eslint-plugin-turbo's
+ * @property {boolean} [turbo] - Enable eslint-plugin-turbo's
  *   `no-undeclared-env-vars` rule (needs `eslint-plugin-turbo` installed).
- *   Only relevant inside a Turborepo; set to `false` for a standalone
- *   project.
+ *   Only relevant inside a Turborepo: defaults to `true` when a `turbo.json`
+ *   sits in ESLint's working directory or one of its parents, `false`
+ *   otherwise.
  * @property {boolean} [snakeCase=false] - Also accept `snake_case` variable
  *   names in `@typescript-eslint/naming-convention` — for code that
  *   destructures rows straight from a SQL database (e.g. Supabase), whose
@@ -86,7 +85,7 @@ export function createNamingConventionRule({ snakeCase = false, react = false, }
  * @param {BaseConfigOptions} [options]
  * @returns {import("eslint").Linter.Config[]}
  */
-export function createBaseConfig({ prettier = true, strict = true, turbo = true, snakeCase = false, } = {}) {
+export function createBaseConfig({ prettier = true, strict = true, turbo = isInsideTurborepo(), snakeCase = false, } = {}) {
     return defineConfig([
         eslint.configs.recommended,
         strict ? tseslint.configs.strictTypeChecked : tseslint.configs.recommended,
@@ -157,7 +156,7 @@ export function createBaseConfig({ prettier = true, strict = true, turbo = true,
         turbo
             ? {
                 plugins: {
-                    turbo: requirePlugin(turboPlugin, "eslint-plugin-turbo", "turbo"),
+                    turbo: /** @type {typeof import("eslint-plugin-turbo").default} */ (requireOptional("eslint-plugin-turbo", "turbo")),
                 },
                 rules: {
                     "turbo/no-undeclared-env-vars": "error",

package/dist/internal/optional-plugin.d.ts

@@ -1,36 +1,16 @@
 /**
- * Plugins that only some variants or options use are optional peer
- * dependencies (see package.json's `peerDependenciesMeta`): a React Native app
- * must not end up installing angular-eslint (and the whole Angular CLI it
- * pulls in), nor a plain TypeScript library every React plugin. Plugins a
- * variant always needs are imported statically by that variant's module;
- * plugins behind an option (`a11y`, `i18n`, `testingLibrary`, `turbo`) go
- * through these helpers instead, so a project that leaves the option off
- * doesn't have to install them.
- */
-/**
- * Imports an optional plugin, resolving to `undefined` when the package isn't
- * installed. Any other failure (a broken install, a missing transitive
- * dependency of the plugin itself) is rethrown rather than hidden.
+ * Loads an optional plugin, or throws an error explaining which package to
+ * install (or which option to turn off) when it isn't installed. Any other
+ * failure (a broken install, a missing transitive dependency of the plugin
+ * itself) is rethrown rather than hidden.
  *
- * @template T
- * @param {string} name - The package name, as passed to `import()`.
- * @param {() => Promise<{ default: T }>} load - `() => import(name)`, written
- *   out by the caller so the plugin keeps its type.
- * @returns {Promise<T | undefined>}
- */
-export function importOptional<T>(name: string, load: () => Promise<{
-    default: T;
-}>): Promise<T | undefined>;
-/**
- * Returns `plugin`, or throws an error explaining which package to install
- * (or which option to turn off) when it's missing.
+ * Callers cast the result to the package's own type (its `import()`
+ * default export) to keep the plugin's type, which `require()` loses.
  *
- * @template T
- * @param {T | undefined} plugin
  * @param {string} name - The package name.
  * @param {string} option - The option that needs it, e.g. `a11y`.
- * @returns {T}
+ * @param {(name: string) => unknown} [load] - Stands in for `require` in tests.
+ * @returns {unknown}
  */
-export function requirePlugin<T>(plugin: T | undefined, name: string, option: string): T;
+export function requireOptional(name: string, option: string, load?: (name: string) => unknown): unknown;
 //# sourceMappingURL=optional-plugin.d.ts.map

package/dist/internal/optional-plugin.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"optional-plugin.d.ts","sourceRoot":"","sources":["../../src/internal/optional-plugin.js"],"names":[],"mappings":"AAAA;;;;;;;;;GASG;AAEH;;;;;;;;;;GAUG;AACH,+BANa,CAAC,QACH,MAAM,QACN,MAAM,OAAO,CAAC;IAAE,OAAO,EAAE,CAAC,CAAA;CAAE,CAAC,GAE3B,OAAO,CAAC,CAAC,GAAG,SAAS,CAAC,CASlC;AAmBD;;;;;;;;;GASG;AACH,8BANa,CAAC,UACH,CAAC,GAAG,SAAS,QACb,MAAM,UACN,MAAM,GACJ,CAAC,CAWb"}
+{"version":3,"file":"optional-plugin.d.ts","sourceRoot":"","sources":["../../src/internal/optional-plugin.js"],"names":[],"mappings":"AAmBA;;;;;;;;;;;;;GAaG;AACH,sCALW,MAAM,UACN,MAAM,SACN,CAAC,IAAI,EAAE,MAAM,KAAK,OAAO,GACvB,OAAO,CAmBnB"}

package/dist/internal/optional-plugin.js

@@ -1,33 +1,62 @@
+import { createRequire } from "node:module";
 /**
  * Plugins that only some variants or options use are optional peer
  * dependencies (see package.json's `peerDependenciesMeta`): a React Native app
  * must not end up installing angular-eslint (and the whole Angular CLI it
  * pulls in), nor a plain TypeScript library every React plugin. Plugins a
  * variant always needs are imported statically by that variant's module;
- * plugins behind an option (`a11y`, `i18n`, `testingLibrary`, `turbo`) go
- * through these helpers instead, so a project that leaves the option off
- * doesn't have to install them.
+ * plugins behind an option (`a11y`, `expo`, `i18n`, `testingLibrary`,
+ * `turbo`) go through {@link requireOptional} instead, only when the option is
+ * on, so a project that leaves it off doesn't have to install them.
+ *
+ * Loaded with `require()` rather than `import()`: an `await import()` at the
+ * top of a variant module would make the whole package an async module, which
+ * Node refuses to `require()` (`ERR_REQUIRE_ASYNC_MODULE`) — and the CommonJS
+ * `eslint.config.js` generated by Expo's templates does exactly that.
  */
+const require = createRequire(import.meta.url);
 /**
- * Imports an optional plugin, resolving to `undefined` when the package isn't
- * installed. Any other failure (a broken install, a missing transitive
- * dependency of the plugin itself) is rethrown rather than hidden.
+ * Loads an optional plugin, or throws an error explaining which package to
+ * install (or which option to turn off) when it isn't installed. Any other
+ * failure (a broken install, a missing transitive dependency of the plugin
+ * itself) is rethrown rather than hidden.
+ *
+ * Callers cast the result to the package's own type (its `import()`
+ * default export) to keep the plugin's type, which `require()` loses.
  *
- * @template T
- * @param {string} name - The package name, as passed to `import()`.
- * @param {() => Promise<{ default: T }>} load - `() => import(name)`, written
- *   out by the caller so the plugin keeps its type.
- * @returns {Promise<T | undefined>}
+ * @param {string} name - The package name.
+ * @param {string} option - The option that needs it, e.g. `a11y`.
+ * @param {(name: string) => unknown} [load] - Stands in for `require` in tests.
+ * @returns {unknown}
  */
-export async function importOptional(name, load) {
+export function requireOptional(name, option, load = require) {
+    /** @type {unknown} */
+    let loaded;
     try {
-        return (await load()).default;
+        loaded = load(name);
     }
     catch (error) {
-        if (isPackageNotFound(error, name))
-            return undefined;
+        if (isPackageNotFound(error, name)) {
+            throw new Error(`@forthtilliath/eslint-config: the \`${option}\` option needs ${name}, ` +
+                `which isn't installed. Install it as a devDependency ` +
+                `(npm install --save-dev ${name}), or pass \`${option}: false\`.`, { cause: error });
+        }
         throw error;
     }
+    return isModuleNamespace(loaded) ? loaded.default : loaded;
+}
+/**
+ * `require()` of an ES module returns its namespace object, where `import()`'s
+ * `.default` is the plugin; a CommonJS module is returned as `module.exports`,
+ * which is already what `import()` puts in `.default`.
+ *
+ * @param {unknown} value
+ * @returns {value is { default: unknown }}
+ */
+function isModuleNamespace(value) {
+    return (typeof value === "object" &&
+        value !== null &&
+        Object.prototype.toString.call(value) === "[object Module]");
 }
 /**
  * @param {unknown} error
@@ -37,27 +66,11 @@ export async function importOptional(name, load) {
 function isPackageNotFound(error, name) {
     return (error instanceof Error &&
         "code" in error &&
-        error.code === "ERR_MODULE_NOT_FOUND" &&
-        // Node names the package it couldn't find: checking it tells "the
-        // plugin isn't installed" apart from "the plugin is installed but one of
-        // its own dependencies is missing", which must not be swallowed.
-        error.message.includes(`'${name}'`));
-}
-/**
- * Returns `plugin`, or throws an error explaining which package to install
- * (or which option to turn off) when it's missing.
- *
- * @template T
- * @param {T | undefined} plugin
- * @param {string} name - The package name.
- * @param {string} option - The option that needs it, e.g. `a11y`.
- * @returns {T}
- */
-export function requirePlugin(plugin, name, option) {
-    if (plugin === undefined) {
-        throw new Error(`@forthtilliath/eslint-config: the \`${option}\` option needs ${name}, ` +
-            `which isn't installed. Install it as a devDependency ` +
-            `(npm install --save-dev ${name}), or pass \`${option}: false\`.`);
-    }
-    return plugin;
+        (error.code === "MODULE_NOT_FOUND" ||
+            error.code === "ERR_MODULE_NOT_FOUND") &&
+        // Node names the package it couldn't find on the first line (the
+        // require stack follows): checking it tells "the plugin isn't installed"
+        // apart from "the plugin is installed but one of its own dependencies is
+        // missing", which must not be swallowed.
+        (error.message.split("\n")[0] ?? "").includes(`'${name}'`));
 }

package/dist/internal/turborepo.d.ts

@@ -0,0 +1,12 @@
+/**
+ * Whether `dir` or one of its parents holds a `turbo.json` — i.e. ESLint runs
+ * inside a Turborepo, where eslint-plugin-turbo's rule makes sense. Used as
+ * the default of the `turbo` option, so a standalone app doesn't have to pass
+ * `turbo: false` (nor install the plugin) to load the config.
+ *
+ * @param {string} [dir] - Where to start looking; ESLint's working directory
+ *   by default.
+ * @returns {boolean}
+ */
+export function isInsideTurborepo(dir?: string): boolean;
+//# sourceMappingURL=turborepo.d.ts.map

package/dist/internal/turborepo.d.ts.map

@@ -0,0 +1 @@
+{"version":3,"file":"turborepo.d.ts","sourceRoot":"","sources":["../../src/internal/turborepo.js"],"names":[],"mappings":"AAGA;;;;;;;;;GASG;AACH,wCAJW,MAAM,GAEJ,OAAO,CAOnB"}

package/dist/internal/turborepo.js

@@ -0,0 +1,20 @@
+import { existsSync } from "node:fs";
+import { dirname, join } from "node:path";
+/**
+ * Whether `dir` or one of its parents holds a `turbo.json` — i.e. ESLint runs
+ * inside a Turborepo, where eslint-plugin-turbo's rule makes sense. Used as
+ * the default of the `turbo` option, so a standalone app doesn't have to pass
+ * `turbo: false` (nor install the plugin) to load the config.
+ *
+ * @param {string} [dir] - Where to start looking; ESLint's working directory
+ *   by default.
+ * @returns {boolean}
+ */
+export function isInsideTurborepo(dir = process.cwd()) {
+    for (let current = dir;; current = dirname(current)) {
+        if (existsSync(join(current, "turbo.json")))
+            return true;
+        if (dirname(current) === current)
+            return false;
+    }
+}

package/dist/react-native.d.ts

@@ -4,12 +4,20 @@
  * hand here instead of spreading a ready-made flat config like storybook.js
  * does.
  */
+/**
+ * @typedef {object} ReactNativeConfigOptions
+ * @property {boolean} [expo=false] - Enable the eslint-plugin-expo rules that
+ *   eslint-config-expo turns on (`use-dom-exports`,
+ *   `no-env-var-destructuring`, `no-dynamic-env-var`), so an Expo app
+ *   migrating from eslint-config-expo keeps them (needs `eslint-plugin-expo`
+ *   installed).
+ */
 /**
  * A custom ESLint configuration for React Native / Expo apps.
  *
- * @param {import("./base.js").BaseConfigOptions & import("./react.js").ReactConfigOptions} [options]
+ * @param {import("./base.js").BaseConfigOptions & import("./react.js").ReactConfigOptions & ReactNativeConfigOptions} [options]
  * @returns {import("eslint").Linter.Config[]} */
-export function createReactNativeConfig({ a11y, ...options }?: import("./base.js").BaseConfigOptions & import("./react.js").ReactConfigOptions): import("eslint").Linter.Config[];
+export function createReactNativeConfig({ a11y, expo, ...options }?: import("./base.js").BaseConfigOptions & import("./react.js").ReactConfigOptions & ReactNativeConfigOptions): import("eslint").Linter.Config[];
 /**
  * A custom ESLint configuration for React Native / Expo apps, with the
  * default options. Use `createReactNativeConfig(options)` instead to
@@ -17,4 +25,14 @@ export function createReactNativeConfig({ a11y, ...options }?: import("./base.js
  *
  * @type {import("eslint").Linter.Config[]} */
 export const reactNativeConfig: import("eslint").Linter.Config[];
+export type ReactNativeConfigOptions = {
+    /**
+     * - Enable the eslint-plugin-expo rules that
+     * eslint-config-expo turns on (`use-dom-exports`,
+     * `no-env-var-destructuring`, `no-dynamic-env-var`), so an Expo app
+     * migrating from eslint-config-expo keeps them (needs `eslint-plugin-expo`
+     * installed).
+     */
+    expo?: boolean | undefined;
+};
 //# sourceMappingURL=react-native.d.ts.map

package/dist/react-native.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"react-native.d.ts","sourceRoot":"","sources":["../src/react-native.js"],"names":[],"mappings":"AAMA;;;;;GAKG;AAEH;;;;iDAIiD;AACjD,+DAFW,OAAO,WAAW,EAAE,iBAAiB,GAAG,OAAO,YAAY,EAAE,kBAAkB,GAC7E,OAAO,QAAQ,EAAE,MAAM,CAAC,MAAM,EAAE,CAwC5C;AAED;;;;;8CAK8C;AAC9C,gCADU,OAAO,QAAQ,EAAE,MAAM,CAAC,MAAM,EAAE,CACmC"}
+{"version":3,"file":"react-native.d.ts","sourceRoot":"","sources":["../src/react-native.js"],"names":[],"mappings":"AAOA;;;;;GAKG;AAEH;;;;;;;GAOG;AAEH;;;;iDAIiD;AACjD,qEAFW,OAAO,WAAW,EAAE,iBAAiB,GAAG,OAAO,YAAY,EAAE,kBAAkB,GAAG,wBAAwB,GACxG,OAAO,QAAQ,EAAE,MAAM,CAAC,MAAM,EAAE,CA6D5C;AAED;;;;;8CAK8C;AAC9C,gCADU,OAAO,QAAQ,EAAE,MAAM,CAAC,MAAM,EAAE,CACmC"}

package/dist/react-native.js

@@ -1,6 +1,7 @@
 import { defineConfig } from "eslint/config";
 import pluginReactNative from "eslint-plugin-react-native";
 import { lazyConfig } from "./internal/lazy-config.js";
+import { requireOptional } from "./internal/optional-plugin.js";
 import { createReactConfig } from "./react.js";
 /**
  * eslint-plugin-react-native only ships a legacy (eslintrc) `configs.all`
@@ -8,14 +9,37 @@ import { createReactConfig } from "./react.js";
  * hand here instead of spreading a ready-made flat config like storybook.js
  * does.
  */
+/**
+ * @typedef {object} ReactNativeConfigOptions
+ * @property {boolean} [expo=false] - Enable the eslint-plugin-expo rules that
+ *   eslint-config-expo turns on (`use-dom-exports`,
+ *   `no-env-var-destructuring`, `no-dynamic-env-var`), so an Expo app
+ *   migrating from eslint-config-expo keeps them (needs `eslint-plugin-expo`
+ *   installed).
+ */
 /**
  * A custom ESLint configuration for React Native / Expo apps.
  *
- * @param {import("./base.js").BaseConfigOptions & import("./react.js").ReactConfigOptions} [options]
+ * @param {import("./base.js").BaseConfigOptions & import("./react.js").ReactConfigOptions & ReactNativeConfigOptions} [options]
  * @returns {import("eslint").Linter.Config[]} */
-export function createReactNativeConfig({ a11y = false, ...options } = {}) {
+export function createReactNativeConfig({ a11y = false, expo = false, ...options } = {}) {
     return defineConfig([
         ...createReactConfig({ a11y, ...options }),
+        expo
+            ? {
+                plugins: {
+                    expo: /** @type {typeof import("eslint-plugin-expo")} */ (requireOptional("eslint-plugin-expo", "expo")),
+                },
+                rules: {
+                    // Every `EXPO_PUBLIC_*` variable is inlined at build time by
+                    // reading `process.env.EXPO_PUBLIC_X` literally: a destructured or
+                    // computed access stays `undefined` in the bundle.
+                    "expo/no-env-var-destructuring": "error",
+                    "expo/no-dynamic-env-var": "error",
+                    "expo/use-dom-exports": "error",
+                },
+            }
+            : [],
         {
             languageOptions: {
                 globals: {

package/dist/react.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"react.d.ts","sourceRoot":"","sources":["../src/react.js"],"names":[],"mappings":"AA2DA;;;;;;;;;;;;;;;;;;GAkBG;AAEH;;;;;GAKG;AACH,0FAHW,OAAO,WAAW,EAAE,iBAAiB,GAAG,kBAAkB,GACxD,OAAO,QAAQ,EAAE,MAAM,CAAC,MAAM,EAAE,CAyG5C;AAED;;;;8CAI8C;AAC9C,0BADU,OAAO,QAAQ,EAAE,MAAM,CAAC,MAAM,EAAE,CACuB"}
+{"version":3,"file":"react.d.ts","sourceRoot":"","sources":["../src/react.js"],"names":[],"mappings":"AAqCA;;;;;;;;;;;;;;;;;;GAkBG;AAEH;;;;;GAKG;AACH,0FAHW,OAAO,WAAW,EAAE,iBAAiB,GAAG,kBAAkB,GACxD,OAAO,QAAQ,EAAE,MAAM,CAAC,MAAM,EAAE,CAqG5C;AAED;;;;8CAI8C;AAC9C,0BADU,OAAO,QAAQ,EAAE,MAAM,CAAC,MAAM,EAAE,CACuB"}

package/dist/react.js

@@ -5,16 +5,8 @@ import pluginReactHooks from "eslint-plugin-react-hooks";
 import pluginReactRefresh from "eslint-plugin-react-refresh";
 import globals from "globals";
 import { lazyConfig } from "./internal/lazy-config.js";
-import { importOptional, requirePlugin } from "./internal/optional-plugin.js";
+import { requireOptional } from "./internal/optional-plugin.js";
 import { createBaseConfig, createNamingConventionRule } from "./base.js";
-// Optional peer dependencies, each behind an option (see
-// ReactConfigOptions): a project leaving the option off doesn't install them.
-const [pluginJsxA11y, pluginI18next, pluginTestingLibrary, pluginJestDom] = await Promise.all([
-    importOptional("eslint-plugin-jsx-a11y", () => import("eslint-plugin-jsx-a11y")),
-    importOptional("eslint-plugin-i18next", () => import("eslint-plugin-i18next")),
-    importOptional("eslint-plugin-testing-library", () => import("eslint-plugin-testing-library")),
-    importOptional("eslint-plugin-jest-dom", () => import("eslint-plugin-jest-dom")),
-]);
 /**
  * Any file matching this is a test file, whatever runner or naming
  * convention is used to suffix it (`.test.`, `.spec.`, `.dom.test.`...) —
@@ -68,13 +60,13 @@ export function createReactConfig({ a11y = true, i18n = false, testingLibrary =
     return defineConfig([
         ...createBaseConfig({ snakeCase, ...options }),
         a11y
-            ? requirePlugin(pluginJsxA11y, "eslint-plugin-jsx-a11y", "a11y")
-                .flatConfigs.recommended
+            ? /** @type {typeof import("eslint-plugin-jsx-a11y").default} */ (requireOptional("eslint-plugin-jsx-a11y", "a11y")).flatConfigs.recommended
             : [],
         i18n
             ? {
                 plugins: {
-                    i18next: requirePlugin(pluginI18next, "eslint-plugin-i18next", "i18n"),
+                    i18next: 
+                    /** @type {typeof import("eslint-plugin-i18next").default} */ (requireOptional("eslint-plugin-i18next", "i18n")),
                 },
                 rules: { "i18next/no-literal-string": "error" },
             }
@@ -82,11 +74,11 @@ export function createReactConfig({ a11y = true, i18n = false, testingLibrary =
         testingLibrary
             ? [
                 {
-                    ...requirePlugin(pluginTestingLibrary, "eslint-plugin-testing-library", "testingLibrary").configs["flat/react"],
+                    ... /** @type {typeof import("eslint-plugin-testing-library").default} */(requireOptional("eslint-plugin-testing-library", "testingLibrary")).configs["flat/react"],
                     files: TEST_FILE_GLOBS,
                 },
                 {
-                    ...requirePlugin(pluginJestDom, "eslint-plugin-jest-dom", "testingLibrary").configs["flat/recommended"],
+                    ... /** @type {typeof import("eslint-plugin-jest-dom").default} */(requireOptional("eslint-plugin-jest-dom", "testingLibrary")).configs["flat/recommended"],
                     files: TEST_FILE_GLOBS,
                 },
             ]

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@forthtilliath/eslint-config",
-  "version": "0.9.0",
+  "version": "0.10.0",
   "type": "module",
   "license": "MIT",
   "publishConfig": {
@@ -33,6 +33,7 @@
     "@next/eslint-plugin-next": "^16.3.0",
     "angular-eslint": "^21.4.0",
     "eslint": "^9.0.0 || ^10.0.0",
+    "eslint-plugin-expo": "^1.2.1",
     "eslint-plugin-i18next": "^6.1.5",
     "eslint-plugin-jest-dom": "^5.10.1",
     "eslint-plugin-jsx-a11y": "^6.10.2",
@@ -55,6 +56,9 @@
     "angular-eslint": {
       "optional": true
     },
+    "eslint-plugin-expo": {
+      "optional": true
+    },
     "eslint-plugin-i18next": {
       "optional": true
     },
@@ -87,11 +91,12 @@
     }
   },
   "devDependencies": {
-    "@angular/core": "^21.0.0",
+    "@angular/core": "^21.2.25",
     "@eslint-react/eslint-plugin": "^5.18.3",
     "@next/eslint-plugin-next": "^16.3.0",
     "angular-eslint": "^21.4.0",
     "eslint": "10.7.0",
+    "eslint-plugin-expo": "^1.2.1",
     "eslint-plugin-i18next": "^6.1.5",
     "eslint-plugin-jest-dom": "^5.10.1",
     "eslint-plugin-jsx-a11y": "^6.10.2",
@@ -104,7 +109,7 @@
     "eslint-plugin-turbo": "^2.10.9",
     "rxjs": "^7.8.0",
     "typescript": "^5.9.3",
-    "vitest": "^5.0.2",
+    "vitest": "^5.0.5",
     "@forthtilliath/typescript-config": "0.1.1"
   },
   "scripts": {