@hypersolid-dev/eslint-config 5.0.0 → 7.0.0

package/CHANGELOG.md

@@ -1,3 +1,42 @@
+## [7.0.0](https://dev.azure.com/wearetriple/Triple%20-%20Webteam/_git/eslint-config-hypersolid/branchCompare?baseVersion=GTv6.0.0&targetVersion=GTv7.0.0) (2026-11-10)
+
+### ⚠ BREAKING CHANGES
+
+* **plugins:** the base-preset rules are @hypersolid-dev/base/<rule>
+(comment-style, regex-pattern-const, undefined-check), not
+@hypersolid-dev/core/<rule> as the previous commit stated.
+* **plugins:** every in-house rule ID gains its domain. Update
+eslint-disable comments and rule overrides:
+- @hypersolid-dev/{comment-style,regex-pattern-const,undefined-check}
+  -> @hypersolid-dev/core/<rule>
+- @hypersolid-dev/{boolean-guard,effect-takes-callback,jsdoc-for-props,
+  no-callback-payload,no-markup-variable,no-unnecessary-use-callback,
+  props-destructuring} -> @hypersolid-dev/react/<rule>
+- @hypersolid-dev/{test-id-default,test-id-format,test-id-from-props,
+  webview-allow-list} -> @hypersolid-dev/react-native/<rule>
+- @hypersolid-dev/react-native-a11y/<rule>
+  -> @hypersolid-dev/react-native/a11y/<rule>
+
+### Code Refactoring
+
+* **plugins:** group in-house rules by domain with prefixed rule IDs ([5c4917c](https://dev.azure.com/wearetriple/Triple%20-%20Webteam/_git/eslint-config-hypersolid/commit/5c4917c3714a51aeea907ad4e453300ee689d722)), closes [AB#5181](https://dev.azure.com/wearetriple/Triple%20-%20Webteam/_workitems/edit/5181)
+* **plugins:** rename the core rule domain to base ([f13bfc4](https://dev.azure.com/wearetriple/Triple%20-%20Webteam/_git/eslint-config-hypersolid/commit/f13bfc492fa035c7034b4bcfc0a2839b41fcd1f4)), closes [AB#5181](https://dev.azure.com/wearetriple/Triple%20-%20Webteam/_workitems/edit/5181)
+
+## [6.0.0](https://dev.azure.com/wearetriple/Triple%20-%20Webteam/_git/eslint-config-hypersolid/branchCompare?baseVersion=GTv5.0.0&targetVersion=GTv6.0.0) (2026-11-10)
+
+### ⚠ BREAKING CHANGES
+
+* **react:** react and react-native report new React Doctor errors.
+* **base:** function declarations and expressions now fail lint.
+* **base:** functions above a cognitive complexity of 15 now fail lint.
+
+### Features
+
+* **base:** enable prefer-arrow-functions ([a1fcf80](https://dev.azure.com/wearetriple/Triple%20-%20Webteam/_git/eslint-config-hypersolid/commit/a1fcf801956aeb42ecd0d123e454bc17f376cbd3)), closes [AB#5180](https://dev.azure.com/wearetriple/Triple%20-%20Webteam/_workitems/edit/5180)
+* **base:** enable sonarjs/cognitive-complexity ([8ee0e2a](https://dev.azure.com/wearetriple/Triple%20-%20Webteam/_git/eslint-config-hypersolid/commit/8ee0e2aa70053e513c6df6fbff63cd726d38bfde)), closes [AB#5180](https://dev.azure.com/wearetriple/Triple%20-%20Webteam/_workitems/edit/5180)
+* **jest:** add opt-in jest preset ([6f26b68](https://dev.azure.com/wearetriple/Triple%20-%20Webteam/_git/eslint-config-hypersolid/commit/6f26b68552cd9dc0f0651e654d621b539b5d6836)), closes [AB#5180](https://dev.azure.com/wearetriple/Triple%20-%20Webteam/_workitems/edit/5180)
+* **react:** add React Doctor rules ([026a2f3](https://dev.azure.com/wearetriple/Triple%20-%20Webteam/_git/eslint-config-hypersolid/commit/026a2f3f8cce13ff74c032cfaa78d9081ddc3fe8)), closes [AB#5180](https://dev.azure.com/wearetriple/Triple%20-%20Webteam/_workitems/edit/5180)
+
 ## [5.0.0](https://dev.azure.com/wearetriple/Triple%20-%20Webteam/_git/eslint-config-hypersolid/branchCompare?baseVersion=GTv4.0.0&targetVersion=GTv5.0.0) (2026-11-10)
 
 ### ⚠ BREAKING CHANGES

package/README.md

@@ -26,6 +26,7 @@ The plugins are regular dependencies of this package, so you don't install them
 | `react-native` | `@hypersolid-dev/eslint-config/react-native` | React Native and Expo. Use instead of `react`. |
 | `node` | `@hypersolid-dev/eslint-config/node` | Node.js services, scripts and CLIs. |
 | `next` | `@hypersolid-dev/eslint-config/next` | Next.js. Stack on top of `react`. |
+| `jest` | `@hypersolid-dev/eslint-config/jest` | Jest and Testing Library suites. Opt-in, on top of the others. |
 
 Each preset is a function that returns an array of config objects. Presets don't include `base`: spread `base` first, then the presets you need, then your own overrides.
 
@@ -58,6 +59,8 @@ import react from '@hypersolid-dev/eslint-config/react';
 export default [...base({ tsconfigRootDir: import.meta.dirname }), ...react()];
 ```
 
+`react` and `react-native` include rules from [React Doctor](https://react.doctor) (`react-doctor/*`); `react-native` adds its `rn-*` rules. They are off in test files.
+
 ### React Native / Expo
 
 ```js
@@ -69,13 +72,13 @@ export default [...base({ tsconfigRootDir: import.meta.dirname }), ...reactNativ
 
 `react-native` bans browser globals that don't exist on React Native (`document`, `localStorage`, …), and ignores `.expo`, `android`, `ios` and `expo-env.d.ts`.
 
-It also enables the bundled `@hypersolid-dev/react-native-a11y` rules, each citing the WCAG criterion it enforces ([rule docs](src/plugins/react-native-a11y/docs)). Custom components are matched by name, so tell a rule about your wrappers with `additionalComponents`. `require-modal-a11y-on-sheets` has no defaults and stays off until you list your sheet components:
+It also enables the bundled `@hypersolid-dev/react-native/a11y` rules, each citing the WCAG criterion it enforces ([rule docs](src/plugins/react-native/a11y/docs)). Custom components are matched by name, so tell a rule about your wrappers with `additionalComponents`. `require-modal-a11y-on-sheets` has no defaults and stays off until you list your sheet components:
 
 ```js
 {
   rules: {
-    '@hypersolid-dev/react-native-a11y/accessibility-label': ['error', { additionalComponents: ['IconButton'] }],
-    '@hypersolid-dev/react-native-a11y/require-modal-a11y-on-sheets': ['error', { additionalComponents: ['BottomSheet'] }],
+    '@hypersolid-dev/react-native/a11y/accessibility-label': ['error', { additionalComponents: ['IconButton'] }],
+    '@hypersolid-dev/react-native/a11y/require-modal-a11y-on-sheets': ['error', { additionalComponents: ['BottomSheet'] }],
   },
 }
 ```
@@ -90,26 +93,38 @@ import react from '@hypersolid-dev/eslint-config/react';
 export default [...base({ tsconfigRootDir: import.meta.dirname }), ...react(), ...next()];
 ```
 
-## Custom rules
+### Jest and Testing Library
+
+```js
+import base from '@hypersolid-dev/eslint-config/base';
+import jest from '@hypersolid-dev/eslint-config/jest';
+import reactNative from '@hypersolid-dev/eslint-config/react-native';
+
+export default [...base({ tsconfigRootDir: import.meta.dirname }), ...reactNative(), ...jest()];
+```
+
+`jest` applies `eslint-plugin-jest`'s recommended set plus a few stricter rules (`it` over `test`, no conditionals in tests, hooks on top) to `*.test.*`, `*.spec.*` and `jest.setup.*` (`JEST_FILES`, exported from `@hypersolid-dev/eslint-config/jest`), and Testing Library's React rules to the suites. The React set also covers React Native Testing Library; `no-await-sync-events` is off because its `fireEvent` returns a promise. `eslint-plugin-jest` reads your installed Jest version; set `settings.jest.version` if Jest isn't resolvable from the config.
+
+
 
 This package ships its own plugin, namespaced `@hypersolid-dev`. The preset column says where each rule is on; `react` rules are on in `react-native` too.
 
 | Rule | Preset | Enforces |
 | --- | --- | --- |
-| `@hypersolid-dev/props-destructuring` | react | A component destructures its props in the signature, so its props are readable there. Replaces `react/destructuring-assignment` (`always`, `destructureInSignature: always`), as `eslint-plugin-react` doesn't support ESLint 10. Checks PascalCase functions that return JSX, including those wrapped in `memo` and `forwardRef`. Bad: `const Card = (props: Props) => <h1>{props.title}</h1>`, or `const { title } = props` in the body. Good: `const Card = ({ title }: Props) => …`. When the props type is a union, such as a discriminated union, `props` may stay whole, because destructuring breaks narrowing (`if (props.kind === 'a') props.a`). |
-| `@hypersolid-dev/undefined-check` | base | No `x !== undefined` (or `=== undefined`) where `undefined` is the type's only falsy value, so truthiness already decides. Bad: `if (user !== undefined)` with `user: User \| undefined`. Good: `if (user)`. Needs type information. |
-| `@hypersolid-dev/regex-pattern-const` | base | A regex is a module-level `const <NAME>_PATTERN`, so it's read by its name. Bad: `/^\d+$/.test(id)` in a function. Off in tests. |
-| `@hypersolid-dev/comment-style` | base | `//` and `/* */` comments are one row; longer prose is a `/** */` JSDoc block. Directive-only runs (`eslint-disable…`, `@ts-…`) are fine. Option `maxLength` caps each row (unset by default). |
-| `@hypersolid-dev/jsdoc-for-props` | react | A comment on a type member is JSDoc, the form editors show at the call site. Bad: `// The title` above `title: string`. |
-| `@hypersolid-dev/boolean-guard` | react | A JSX `&&` guard can't leak `0` or `''`, which React renders (and which crashes React Native outside `<Text>`). Bad: `{count && <Badge />}`. Good: `{count > 0 && <Badge />}`. Needs type information. Off in tests. |
-| `@hypersolid-dev/no-markup-variable` | react | Styled JSX isn't stored in a variable in a component body; it's its own component, or written where it renders. Off in tests. |
-| `@hypersolid-dev/effect-takes-callback` | react | A `use*Effect` hook takes the callback it runs as its first parameter, like `useEffect`. Off in tests. |
-| `@hypersolid-dev/no-unnecessary-use-callback` | react | A `useCallback` with empty deps that captures nothing from render is a module-level function. Off in tests. |
-| `@hypersolid-dev/no-callback-payload` | react | A component passes a callback prop straight through instead of calling it with a payload built from its own props; the caller binds the payload. Bad: `onPress={() => onSelect(id)}` where `id` is a prop. Off in tests. |
-| `@hypersolid-dev/test-id-format` | react-native | A `testID` is lowercase kebab-case. Bad: `testID="Header.Title"`. |
-| `@hypersolid-dev/test-id-from-props` | react-native | In `**/components/**/*.tsx`, a `testID` comes from the `testID` prop (suffixed for inner nodes), never a fixed string. Off in screens and tests. |
-| `@hypersolid-dev/test-id-default` | react-native | In `**/components/**/*.tsx`, a `testID` prop defaults to the component's kebab-case name. Bad: `({ testID }: Props)` in `Card`. Good: `({ testID = 'card' }: Props)`. |
-| `@hypersolid-dev/webview-allow-list` | react-native | A `react-native-webview` WebView sets `originWhitelist={['*']}` and filters every navigation in an always-present `onShouldStartLoadWithRequest`. |
+| `@hypersolid-dev/react/props-destructuring` | react | A component destructures its props in the signature, so its props are readable there. Replaces `react/destructuring-assignment` (`always`, `destructureInSignature: always`), as `eslint-plugin-react` doesn't support ESLint 10. Checks PascalCase functions that return JSX, including those wrapped in `memo` and `forwardRef`. Bad: `const Card = (props: Props) => <h1>{props.title}</h1>`, or `const { title } = props` in the body. Good: `const Card = ({ title }: Props) => …`. When the props type is a union, such as a discriminated union, `props` may stay whole, because destructuring breaks narrowing (`if (props.kind === 'a') props.a`). |
+| `@hypersolid-dev/base/undefined-check` | base | No `x !== undefined` (or `=== undefined`) where `undefined` is the type's only falsy value, so truthiness already decides. Bad: `if (user !== undefined)` with `user: User \| undefined`. Good: `if (user)`. Needs type information. |
+| `@hypersolid-dev/base/regex-pattern-const` | base | A regex is a module-level `const <NAME>_PATTERN`, so it's read by its name. Bad: `/^\d+$/.test(id)` in a function. Off in tests. |
+| `@hypersolid-dev/base/comment-style` | base | `//` and `/* */` comments are one row; longer prose is a `/** */` JSDoc block. Directive-only runs (`eslint-disable…`, `@ts-…`) are fine. Option `maxLength` caps each row (unset by default). |
+| `@hypersolid-dev/react/jsdoc-for-props` | react | A comment on a type member is JSDoc, the form editors show at the call site. Bad: `// The title` above `title: string`. |
+| `@hypersolid-dev/react/boolean-guard` | react | A JSX `&&` guard can't leak `0` or `''`, which React renders (and which crashes React Native outside `<Text>`). Bad: `{count && <Badge />}`. Good: `{count > 0 && <Badge />}`. Needs type information. Off in tests. |
+| `@hypersolid-dev/react/no-markup-variable` | react | Styled JSX isn't stored in a variable in a component body; it's its own component, or written where it renders. Off in tests. |
+| `@hypersolid-dev/react/effect-takes-callback` | react | A `use*Effect` hook takes the callback it runs as its first parameter, like `useEffect`. Off in tests. |
+| `@hypersolid-dev/react/no-unnecessary-use-callback` | react | A `useCallback` with empty deps that captures nothing from render is a module-level function. Off in tests. |
+| `@hypersolid-dev/react/no-callback-payload` | react | A component passes a callback prop straight through instead of calling it with a payload built from its own props; the caller binds the payload. Bad: `onPress={() => onSelect(id)}` where `id` is a prop. Off in tests. |
+| `@hypersolid-dev/react-native/test-id-format` | react-native | A `testID` is lowercase kebab-case. Bad: `testID="Header.Title"`. |
+| `@hypersolid-dev/react-native/test-id-from-props` | react-native | In `**/components/**/*.tsx`, a `testID` comes from the `testID` prop (suffixed for inner nodes), never a fixed string. Off in screens and tests. |
+| `@hypersolid-dev/react-native/test-id-default` | react-native | In `**/components/**/*.tsx`, a `testID` prop defaults to the component's kebab-case name. Bad: `({ testID }: Props)` in `Card`. Good: `({ testID = 'card' }: Props)`. |
+| `@hypersolid-dev/react-native/webview-allow-list` | react-native | A `react-native-webview` WebView sets `originWhitelist={['*']}` and filters every navigation in an always-present `onShouldStartLoadWithRequest`. |
 
 ## Overriding rules
 
@@ -171,14 +186,14 @@ export default [...base({ tsconfigRootDir: import.meta.dirname }), { files: TEST
 
 ### Component folders
 
-`@hypersolid-dev/no-callback-payload` applies to every component, and the `test-id-*` rules to `**/components/**/*.tsx`. If your components live elsewhere, scope them to your own folders:
+`@hypersolid-dev/react/no-callback-payload` applies to every component, and the `test-id-*` rules to `**/components/**/*.tsx`. If your components live elsewhere, scope them to your own folders:
 
 ```js
 export default [
   ...base({ tsconfigRootDir: import.meta.dirname }),
   ...reactNative(),
-  { files: ['**/components/**/*.tsx'], rules: { '@hypersolid-dev/test-id-default': 'off' } },
-  { files: ['src/ui/**/*.tsx'], rules: { '@hypersolid-dev/test-id-default': 'error' } },
+  { files: ['**/components/**/*.tsx'], rules: { '@hypersolid-dev/react-native/test-id-default': 'off' } },
+  { files: ['src/ui/**/*.tsx'], rules: { '@hypersolid-dev/react-native/test-id-default': 'error' } },
 ];
 ```
 
@@ -193,6 +208,7 @@ Root files such as `eslint.config.js` or `vite.config.ts` are often outside the
 ## Known issues
 
 - `eslint-plugin-jsx-a11y` (used by `react`) declares support for ESLint 9 only. It works on ESLint 10, but installing prints a peer-dependency warning, and installs with `strict-peer-dependencies` fail.
+- `eslint-plugin-react-doctor` declares no `eslint` peer dependency. It reads the nearest `package.json` to decide which rules apply (the `rn-*` rules need `react-native` there), and skips some rules for files under `test/`, `__tests__/` and similar paths.
 - In `next`, `@next/next/no-html-link-for-pages` prints "Pages directory cannot be found" on App Router projects. Turn it off if you don't use the `pages` directory:
 
   ```js

package/dist/base.js

@@ -3,9 +3,11 @@ import json from '@eslint/json';
 import { createTypeScriptImportResolver } from 'eslint-import-resolver-typescript';
 import { createNodeResolver, flatConfigs as importX } from 'eslint-plugin-import-x';
 import noSecrets from 'eslint-plugin-no-secrets';
+import preferArrowFunctions from 'eslint-plugin-prefer-arrow-functions';
+import sonarjs from 'eslint-plugin-sonarjs';
 import unicorn from 'eslint-plugin-unicorn';
 import { configs as tsConfigs } from 'typescript-eslint';
-import hypersolid from './plugins/hypersolid.js';
+import basePlugin from './plugins/base/index.js';
 import { BASE_RESTRICTED_SYNTAX, BOOLEAN_NAME_OPTIONS, CODE_FILES, NAMING_CONVENTION, NUMBER_GLOBAL_RESTRICTIONS, raiseWarnings, scopeToCode, TEST_FILES, } from './shared.js';
 // The glob the presets relax test-only rules for; reuse it for your own test overrides.
 export { TEST_FILES } from './shared.js';
@@ -44,7 +46,12 @@ export default function base(options) {
                     tsconfigRootDir: options.tsconfigRootDir,
                 },
             },
-            plugins: { '@hypersolid-dev': hypersolid, 'no-secrets': noSecrets },
+            plugins: {
+                '@hypersolid-dev/base': basePlugin,
+                'no-secrets': noSecrets,
+                'prefer-arrow-functions': preferArrowFunctions,
+                sonarjs,
+            },
             settings: {
                 'import-x/resolver-next': [createTypeScriptImportResolver(), createNodeResolver()],
             },
@@ -59,6 +66,8 @@ export default function base(options) {
                 'no-nested-ternary': 'error',
                 // Arrow functions as callbacks. Bad: `list.map(function (x) { return x; })`.
                 'prefer-arrow-callback': 'error',
+                // One function form throughout, and no `this` to reason about. Bad: `function load() {}`, `export default function base() {}`.
+                'prefer-arrow-functions/prefer-arrow-functions': ['error', { disallowPrototype: true }],
                 // A rethrow keeps the original error as `cause`. Bad: `catch (error) { throw new Error('failed'); }`.
                 'preserve-caught-error': 'error',
                 // The coercing globals. Bad: `isNaN('x')` (use `Number.isNaN`).
@@ -144,11 +153,11 @@ export default function base(options) {
                 // One escape style for `${` in template literals. Bad: `` `\${a} $\{b}` ``.
                 'unicorn/consistent-template-literal-escape': 'error',
                 // `x !== undefined` where truthiness already decides. Bad: `if (user !== undefined)` with `user: User | undefined`.
-                '@hypersolid-dev/undefined-check': 'error',
+                '@hypersolid-dev/base/undefined-check': 'error',
                 // A regex is a module-level `*_PATTERN` constant, named for what it matches. Bad: `if (/^\d+$/.test(id))` in a function.
-                '@hypersolid-dev/regex-pattern-const': 'error',
+                '@hypersolid-dev/base/regex-pattern-const': 'error',
                 // `//` and `/* */` comments stay one row; longer prose is a `/** */` block. Bad: three `//` lines in a row.
-                '@hypersolid-dev/comment-style': 'error',
+                '@hypersolid-dev/base/comment-style': 'error',
                 // `undefined` as an argument is often required by the callee's signature. Bad: `let value = undefined`.
                 'unicorn/no-useless-undefined': ['error', { checkArguments: false }],
                 // Off: `.then()` chains are fine. Bad: `fetchUser().then(render)` reported.
@@ -183,12 +192,16 @@ export default function base(options) {
                 'import-x/namespace': 'off',
                 // High-entropy strings that look like credentials. Bad: `const token = 'ghp_Zx8kQ2vLm9…'`.
                 'no-secrets/no-secrets': 'error',
+                // A function too tangled to follow, nesting weighted over length (limit 15). Bad: a loop holding an `if` holding a `try`.
+                'sonarjs/cognitive-complexity': 'error',
             },
         },
         {
             name: '@hypersolid-dev/base/tests',
             files: TEST_FILES,
             rules: {
+                // A test's setup and table of cases are read once, not maintained as logic.
+                'sonarjs/cognitive-complexity': 'off',
                 // Off: mocks and fixtures are loosely typed by design. Bad: `const user = jest.fn()()` reported.
                 '@typescript-eslint/no-unsafe-assignment': 'off',
                 // Off: mocks and fixtures are loosely typed by design. Bad: `mock.calls[0].user` reported.
@@ -202,7 +215,7 @@ export default function base(options) {
                 // Off: helpers inside a `describe` keep each suite readable on its own. Bad: `const render = () => …` in a `describe` reported.
                 'unicorn/consistent-function-scoping': 'off',
                 // Off: a test keeps its inline `toMatch(/…/)` literal. Bad: `expect(id).toMatch(/^\d+$/)` reported.
-                '@hypersolid-dev/regex-pattern-const': 'off',
+                '@hypersolid-dev/base/regex-pattern-const': 'off',
             },
         },
         {

package/dist/jest.d.ts

@@ -0,0 +1,3 @@
+import type { Linter } from 'eslint';
+export declare const JEST_FILES: string[];
+export default function jest(): Linter.Config[];

package/dist/jest.js

@@ -0,0 +1,56 @@
+import jestPlugin from 'eslint-plugin-jest';
+import testingLibrary from 'eslint-plugin-testing-library';
+import { raiseWarnings } from './shared.js';
+// Test suites and Jest setup. Mocks and stories are left out: `jest/no-export` would reject what they exist to export.
+export const JEST_FILES = ['**/*.{test,spec}.{js,mjs,cjs,jsx,ts,mts,cts,tsx}', '**/jest.setup.*'];
+const SUITE_FILES = ['**/*.{test,spec}.{js,mjs,cjs,jsx,ts,mts,cts,tsx}'];
+export default function jest() {
+    return raiseWarnings([
+        {
+            // Jest's recommended set and its globals; the plugin ships it unscoped. Bad: `it.only(…)`, `expect` inside a `catch`.
+            ...jestPlugin.configs['flat/recommended'],
+            name: '@hypersolid-dev/jest',
+            files: JEST_FILES,
+            rules: {
+                ...jestPlugin.configs['flat/recommended'].rules,
+                // Off: jest/unbound-method replaces it and knows `expect(service.method)` does not call it. Bad: `expect(Alert.alert)` reported.
+                '@typescript-eslint/unbound-method': 'off',
+                // A method passed on loses its `this`, except into `expect`. Bad: `const { save } = service; save()`.
+                'jest/unbound-method': 'error',
+                // A test must assert; `expectX()` helpers count as assertions. Bad: `it('renders', () => { render(<A />); })`.
+                'jest/expect-expect': ['error', { assertFunctionNames: ['expect', 'expect*'] }],
+                // One spelling for a test case. Bad: `test('…')` next to `it('…')`.
+                'jest/consistent-test-it': ['error', { fn: 'it' }],
+                // A conditional in a test means it does not know what it asserts. Bad: `if (items.length) expect(…)`.
+                'jest/no-conditional-in-test': 'error',
+                // Two hooks of a kind in one block run in an order nobody chose. Bad: two `beforeEach` in one `describe`.
+                'jest/no-duplicate-hooks': 'error',
+                // Returning from a test hides an unawaited promise. Bad: `it('loads', () => { return load(); })`.
+                'jest/no-test-return-statement': 'error',
+                // Setup is read before what it sets up. Bad: `beforeEach` below the first `it`.
+                'jest/prefer-hooks-on-top': 'error',
+                // A blank line around each block, hook and assertion group. Bad: `it(…)` directly under `beforeEach(…)`.
+                'jest/padding-around-all': 'error',
+                // `jest.setTimeout` below the top level applies to nothing. Bad: `jest.setTimeout(10_000)` inside a `describe`.
+                'jest/no-confusing-set-timeout': 'error',
+            },
+        },
+        {
+            // Testing Library's React set; it covers React Native Testing Library, which has no flavour of its own. Bad: `container.querySelector(…)`, an unawaited `findBy…`.
+            ...testingLibrary.configs['flat/react'],
+            name: '@hypersolid-dev/jest/testing-library',
+            files: SUITE_FILES,
+            rules: {
+                ...testingLibrary.configs['flat/react'].rules,
+                // Events are awaited. Bad: `userEvent.press(button)` without `await`.
+                'testing-library/await-async-events': 'error',
+                // Testing Library already wraps its utilities in `act`. Bad: `act(() => fireEvent.press(button))`.
+                'testing-library/no-unnecessary-act': 'error',
+                // Off: React Native Testing Library 14's `fireEvent` returns a promise to await. Bad: `await fireEvent.press(button)` reported.
+                'testing-library/no-await-sync-events': 'off',
+                // Off: suites query through `screen`, so it only reaches helpers that return a mock. Bad: `const view = render(<A />)` reported.
+                'testing-library/render-result-naming-convention': 'off',
+            },
+        },
+    ]);
+}

package/dist/plugins/{hypersolid.d.ts → base/index.d.ts}

@@ -1,3 +1,4 @@
 import type { ESLint } from 'eslint';
+/** ESLint reads a scoped rule ID up to its last slash as the plugin name. */
 declare const plugin: ESLint.Plugin;
 export default plugin;

package/dist/plugins/base/index.js

@@ -0,0 +1,15 @@
+import commentStyle from './rules/comment-style.js';
+import regexPatternConst from './rules/regex-pattern-const.js';
+import undefinedCheck from './rules/undefined-check.js';
+/** ESLint reads a scoped rule ID up to its last slash as the plugin name. */
+// One object per plugin name: flat config rejects two different objects under the same name.
+const plugin = {
+    meta: { name: '@hypersolid-dev/base' },
+    // typescript-eslint's `RuleModule` and ESLint's `Rule.RuleModule` type the same runtime object differently.
+    rules: {
+        'comment-style': commentStyle,
+        'regex-pattern-const': regexPatternConst,
+        'undefined-check': undefinedCheck,
+    },
+};
+export default plugin;

package/dist/plugins/{rules → base/rules}/comment-style.js

@@ -1,6 +1,6 @@
 import { AST_NODE_TYPES, AST_TOKEN_TYPES } from "@typescript-eslint/utils";
-import createRule from "../utils/create-rule.js";
-import { DIRECTIVE_PATTERN, isJsdoc, isRowStart } from "../utils/comments.js";
+import createRule from "../../shared/create-rule.js";
+import { DIRECTIVE_PATTERN, isJsdoc, isRowStart } from "../../shared/comments.js";
 const REASON_PATTERN = /\s--\s([\s\S]*)$/;
 const LINK_PATTERN = /\S*https?:\/\/\S*/g;
 const JSDOC_ROW_PATTERN = /^\s*\*?\s?/;
@@ -74,7 +74,7 @@ export default createRule({
             tooLong: "This comment line is {{length}} characters. Keep it to {{max}}.",
         },
     },
-    create(context) {
+    create: (context) => {
         const { sourceCode } = context;
         const maxLength = context.options[0]?.maxLength;
         const isTooLong = (comment) => Boolean(maxLength) && lineLength(comment) > (maxLength ?? 0);
@@ -86,7 +86,7 @@ export default createRule({
             });
         };
         return {
-            "Program:exit"() {
+            "Program:exit": () => {
                 let run = [];
                 const doesContinue = (comment) => {
                     const last = run.at(-1);
@@ -108,48 +108,57 @@ export default createRule({
                     }
                     run = [];
                 };
-                for (const comment of sourceCode.getAllComments()) {
-                    if (comment.type === AST_TOKEN_TYPES.Block) {
-                        const rows = comment.loc.end.line - comment.loc.start.line + 1;
-                        // A JSX comment's `{` is syntax, so a row holding only `{/* … */}` starts its row.
-                        const isOwnRow = isRowStart(sourceCode, comment) ||
-                            (isJsxComment(sourceCode, comment) &&
-                                !sourceCode.lines[comment.loc.start.line - 1]
-                                    ?.slice(0, Math.max(0, comment.loc.start.column - 1))
-                                    .trim());
-                        if (!isJsdoc(comment) && rows === 1 && isOwnRow) {
-                            if (!doesContinue(comment))
-                                flush();
-                            run.push(comment);
-                            continue;
-                        }
-                        flush();
-                        if (isJsdoc(comment)) {
-                            if (isTooLong(comment))
-                                reportTooLong(comment);
-                            continue;
-                        }
-                        if (rows > 1) {
-                            context.report({
-                                loc: comment.loc,
-                                messageId: isJsxComment(sourceCode, comment) ? "jsxBlock" : "blockRun",
-                                data: { rows: String(rows) },
-                            });
-                        }
-                        else if (isTooLong(comment)) {
+                /** A JSX comment's `{` is syntax, so a row holding only a braced JSX comment starts its row. */
+                const isOwnRow = (comment) => isRowStart(sourceCode, comment) ||
+                    (isJsxComment(sourceCode, comment) &&
+                        !sourceCode.lines[comment.loc.start.line - 1]
+                            ?.slice(0, Math.max(0, comment.loc.start.column - 1))
+                            .trim());
+                /** A block comment that is not part of a run: JSDoc, multi-row, or sharing its row with code. */
+                const checkLoneBlock = (comment, rows) => {
+                    if (isJsdoc(comment)) {
+                        if (isTooLong(comment))
                             reportTooLong(comment);
-                        }
-                        continue;
+                        return;
+                    }
+                    if (rows > 1) {
+                        context.report({
+                            loc: comment.loc,
+                            messageId: isJsxComment(sourceCode, comment) ? "jsxBlock" : "blockRun",
+                            data: { rows: String(rows) },
+                        });
                     }
+                    else if (isTooLong(comment)) {
+                        reportTooLong(comment);
+                    }
+                };
+                const visitBlock = (comment) => {
+                    const rows = comment.loc.end.line - comment.loc.start.line + 1;
+                    if (!isJsdoc(comment) && rows === 1 && isOwnRow(comment)) {
+                        if (!doesContinue(comment))
+                            flush();
+                        run.push(comment);
+                        return;
+                    }
+                    flush();
+                    checkLoneBlock(comment, rows);
+                };
+                const visitLine = (comment) => {
                     if (!doesContinue(comment))
                         flush();
                     if (!isRowStart(sourceCode, comment)) {
                         flush();
                         if (isTooLong(comment))
                             reportTooLong(comment);
-                        continue;
+                        return;
                     }
                     run.push(comment);
+                };
+                for (const comment of sourceCode.getAllComments()) {
+                    if (comment.type === AST_TOKEN_TYPES.Block)
+                        visitBlock(comment);
+                    else
+                        visitLine(comment);
                 }
                 flush();
             },

package/dist/plugins/{rules → base/rules}/regex-pattern-const.js

@@ -1,5 +1,5 @@
 import { AST_NODE_TYPES } from "@typescript-eslint/utils";
-import createRule from "../utils/create-rule.js";
+import createRule from "../../shared/create-rule.js";
 const NAME_PATTERN = /^[A-Z][A-Z0-9_]*_PATTERN$/;
 /** A string literal, or a template (plain or `String.raw`) with nothing interpolated. */
 const isStatic = (node) => (node.type === AST_NODE_TYPES.Literal && typeof node.value === "string") ||
@@ -64,7 +64,7 @@ export default createRule({
             rename: "Rename `{{name}}` to end in `_PATTERN` (SCREAMING_SNAKE_CASE), so the name says it is a regular expression.",
         },
     },
-    create(context) {
+    create: (context) => {
         const check = (node) => {
             if (!isRegex(node))
                 return;

package/dist/plugins/{rules → base/rules}/undefined-check.js

@@ -1,6 +1,6 @@
 import { AST_NODE_TYPES, ESLintUtils } from "@typescript-eslint/utils";
-import createRule from "../utils/create-rule.js";
-import { isSameAsBangBang, undefinedComparison } from "../utils/types.js";
+import createRule from "../../shared/create-rule.js";
+import { isSameAsBangBang, undefinedComparison } from "../../shared/types.js";
 /** The slots where a value is being asked whether it is there. */
 const isCondition = (node) => {
     const { parent } = node;
@@ -35,13 +35,13 @@ const undefinedCheck = createRule({
             redundant: "`{{ text }}` is `{{ suggestion }}` for this type: `undefined` is the only falsy value it admits, so the comparison draws no line. Write the value.",
         },
     },
-    create(context) {
+    create: (context) => {
         const services = ESLintUtils.getParserServices(context, true);
         if (!services.program)
             return {};
         const checker = services.program.getTypeChecker();
         return {
-            BinaryExpression(node) {
+            BinaryExpression: (node) => {
                 if (!isCondition(node))
                     return;
                 const compared = undefinedComparison(node);

package/dist/plugins/react/index.d.ts

@@ -0,0 +1,4 @@
+import type { ESLint } from 'eslint';
+/** ESLint reads a scoped rule ID up to its last slash as the plugin name. */
+declare const plugin: ESLint.Plugin;
+export default plugin;

package/dist/plugins/react/index.js

@@ -0,0 +1,23 @@
+import booleanGuard from './rules/boolean-guard.js';
+import effectTakesCallback from './rules/effect-takes-callback.js';
+import jsdocForProps from './rules/jsdoc-for-props.js';
+import noCallbackPayload from './rules/no-callback-payload.js';
+import noMarkupVariable from './rules/no-markup-variable.js';
+import noUnnecessaryUseCallback from './rules/no-unnecessary-use-callback.js';
+import propsDestructuring from './rules/props-destructuring.js';
+/** ESLint reads a scoped rule ID up to its last slash as the plugin name. */
+// One object per plugin name: flat config rejects two different objects under the same name.
+const plugin = {
+    meta: { name: '@hypersolid-dev/react' },
+    // typescript-eslint's `RuleModule` and ESLint's `Rule.RuleModule` type the same runtime object differently.
+    rules: {
+        'boolean-guard': booleanGuard,
+        'effect-takes-callback': effectTakesCallback,
+        'jsdoc-for-props': jsdocForProps,
+        'no-callback-payload': noCallbackPayload,
+        'no-markup-variable': noMarkupVariable,
+        'no-unnecessary-use-callback': noUnnecessaryUseCallback,
+        'props-destructuring': propsDestructuring,
+    },
+};
+export default plugin;

package/dist/plugins/{rules → react/rules}/boolean-guard.js

@@ -1,7 +1,7 @@
 import { AST_NODE_TYPES, ESLintUtils } from "@typescript-eslint/utils";
 import ts, { TypeFlags } from "typescript";
-import createRule from "../utils/create-rule.js";
-import { isSameAsBangBang, undefinedComparison } from "../utils/types.js";
+import createRule from "../../shared/create-rule.js";
+import { isSameAsBangBang, undefinedComparison } from "../../shared/types.js";
 /** React draws these; every other falsy value renders as nothing. */
 const RENDERS_WHEN_FALSY = TypeFlags.NumberLike | TypeFlags.StringLike | TypeFlags.BigIntLike;
 /** `any` and `unknown` could be anything, so they are treated as leaking. */
@@ -69,7 +69,7 @@ const booleanGuard = createRule({
             preferBangBang: "`!== undefined` is `!!` for this type. Write `!!{{condition}} && …`.",
         },
     },
-    create(context) {
+    create: (context) => {
         const services = ESLintUtils.getParserServices(context, true);
         if (!services.program)
             return {};
@@ -99,7 +99,7 @@ const booleanGuard = createRule({
             return reads.every((reference) => isInRenderPosition(reference.identifier));
         };
         return {
-            ConditionalExpression(node) {
+            ConditionalExpression: (node) => {
                 // The shape is what proves a `const` holding this is a guard.
                 const isConsequentGuarded = isRenderable(node.consequent) && isNothing(node.alternate);
                 const isAlternateGuarded = isRenderable(node.alternate) && isNothing(node.consequent);
@@ -122,7 +122,7 @@ const booleanGuard = createRule({
                     data: { condition: isAlternateGuarded ? `!${nameOf(node.test)}` : nameOf(node.test) },
                 });
             },
-            LogicalExpression(node) {
+            LogicalExpression: (node) => {
                 if (node.operator !== "&&")
                     return;
                 if (!isInRenderPosition(node))

package/dist/plugins/{rules → react/rules}/effect-takes-callback.js

@@ -1,6 +1,6 @@
 import { AST_NODE_TYPES } from "@typescript-eslint/utils";
 import ts, { TypeFlags } from "typescript";
-import createRule from "../utils/create-rule.js";
+import createRule from "../../shared/create-rule.js";
 const EFFECT_HOOK_PATTERN = /^use[A-Z]\w*Effect$/;
 const NOTHING = TypeFlags.Undefined | TypeFlags.Null | TypeFlags.Void;
 const ABSENT = new Set(["TSUndefinedKeyword", "TSNullKeyword", "TSVoidKeyword"]);
@@ -48,7 +48,7 @@ const effectTakesCallback = createRule({
             callbackFirst: "`{{name}}` ends in Effect but its first parameter is not a callback. Take the callback first, or drop the Effect suffix.",
         },
     },
-    create(context) {
+    create: (context) => {
         const services = context.sourceCode.parserServices;
         const checker = services?.program ? services.program.getTypeChecker() : undefined;
         /** Whether a value of this type can be called, once `undefined` is set aside. */

package/dist/plugins/{rules → react/rules}/jsdoc-for-props.js

@@ -1,6 +1,6 @@
 import { AST_TOKEN_TYPES } from "@typescript-eslint/utils";
-import createRule from "../utils/create-rule.js";
-import { DIRECTIVE_PATTERN, isJsdoc, isRowStart } from "../utils/comments.js";
+import createRule from "../../shared/create-rule.js";
+import { DIRECTIVE_PATTERN, isJsdoc, isRowStart } from "../../shared/comments.js";
 /**
  * A comment on a type member is a JSDoc block, the only form an editor shows when the member is hovered elsewhere.
  *
@@ -28,7 +28,7 @@ export default createRule({
             form: "Write this as a JSDoc block, not `{{ form }}`. A member's comment is what an editor shows when someone hovers it elsewhere, and only JSDoc reaches there.",
         },
     },
-    create(context) {
+    create: (context) => {
         const { sourceCode } = context;
         const check = (node) => {
             // A directive neither documents the member nor separates it from the comment above.