@hypersolid-dev/eslint-config 3.0.0 → 5.0.0

package/CHANGELOG.md

@@ -1,3 +1,37 @@
+## [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
+
+* **react:** effect dependency findings now come from
+react-hooks/exhaustive-effect-dependencies, so existing
+eslint-disable comments for react-hooks/exhaustive-deps no longer apply.
+* **react-native:** importing `Animated` from `react-native` is now an error in the react-native preset.
+* **react-native:** react-native reports code it accepted before.
+* **react:** react and react-native report code they accepted before.
+* **base:** base reports code it accepted before.
+
+### Features
+
+* **base:** add tier 1 rules from the ESPN Goal pilot ([68d6391](https://dev.azure.com/wearetriple/Triple%20-%20Webteam/_git/eslint-config-hypersolid/commit/68d6391ce228e6b205fd3a7abc77b6bd179569d0)), closes [AB#5179](https://dev.azure.com/wearetriple/Triple%20-%20Webteam/_workitems/edit/5179)
+* port boolean-guard and undefined-check rules ([79f50cc](https://dev.azure.com/wearetriple/Triple%20-%20Webteam/_git/eslint-config-hypersolid/commit/79f50ccf5bfb34ba1683b7d46947a423fbe56ee5))
+* port comment-style, jsdoc-for-props and regex-pattern-const rules ([d039153](https://dev.azure.com/wearetriple/Triple%20-%20Webteam/_git/eslint-config-hypersolid/commit/d03915380754f723bf875cd9874519d291f7532b))
+* port no-markup-variable, no-callback-payload, effect-takes-callback and no-unnecessary-use-callback rules ([dde97ee](https://dev.azure.com/wearetriple/Triple%20-%20Webteam/_git/eslint-config-hypersolid/commit/dde97ee3e6bf97c6337d0b660083acdaf32cc548))
+* port test-id-format, test-id-from-props, test-id-default and webview-allow-list rules ([33d2660](https://dev.azure.com/wearetriple/Triple%20-%20Webteam/_git/eslint-config-hypersolid/commit/33d266031431fa1f37961974d91645c1df553acc))
+* **react-native:** add tier 1 React Native rules from the ESPN Goal pilot ([e61ef0c](https://dev.azure.com/wearetriple/Triple%20-%20Webteam/_git/eslint-config-hypersolid/commit/e61ef0c550638e77b30128e9b776712714706bc8)), closes [AB#5179](https://dev.azure.com/wearetriple/Triple%20-%20Webteam/_workitems/edit/5179)
+* **react-native:** ban Animated from react-native in favour of Reanimated ([19b6952](https://dev.azure.com/wearetriple/Triple%20-%20Webteam/_git/eslint-config-hypersolid/commit/19b6952dde8c462be851e6dd90744167374d80ea)), closes [AB#5179](https://dev.azure.com/wearetriple/Triple%20-%20Webteam/_workitems/edit/5179)
+* **react:** add tier 1 React rules from the ESPN Goal pilot ([b9f9e58](https://dev.azure.com/wearetriple/Triple%20-%20Webteam/_git/eslint-config-hypersolid/commit/b9f9e58ebd6dc6d80454580ae3969daeafe5da2c)), closes [AB#5179](https://dev.azure.com/wearetriple/Triple%20-%20Webteam/_workitems/edit/5179)
+* **react:** check dependency arrays once, with the compiler rules ([5502049](https://dev.azure.com/wearetriple/Triple%20-%20Webteam/_git/eslint-config-hypersolid/commit/5502049745c8e732b30426697325940d11ea44b4)), closes [AB#5179](https://dev.azure.com/wearetriple/Triple%20-%20Webteam/_workitems/edit/5179)
+
+## [4.0.0](https://dev.azure.com/wearetriple/Triple%20-%20Webteam/_git/eslint-config-hypersolid/branchCompare?baseVersion=GTv3.0.0&targetVersion=GTv4.0.0) (2026-11-09)
+
+### ⚠ BREAKING CHANGES
+
+* **react-native:** the react-native preset enables three new error-level rules.
+
+### Features
+
+* **react-native:** add Expo rules and Metro asset requires ([059b8c1](https://dev.azure.com/wearetriple/Triple%20-%20Webteam/_git/eslint-config-hypersolid/commit/059b8c1cd8d9bf3772da3b24fc3ed4a91078857a)), closes [AB#5178](https://dev.azure.com/wearetriple/Triple%20-%20Webteam/_workitems/edit/5178)
+
 ## [3.0.0](https://dev.azure.com/wearetriple/Triple%20-%20Webteam/_git/eslint-config-hypersolid/branchCompare?baseVersion=GTv2.0.0&targetVersion=GTv3.0.0) (2026-11-09)
 
 ### ⚠ BREAKING CHANGES

package/README.md

@@ -92,11 +92,24 @@ export default [...base({ tsconfigRootDir: import.meta.dirname }), ...react(), .
 
 ## Custom rules
 
-This package ships its own plugin, namespaced `@hypersolid-dev`, enabled in `react` and `react-native`.
+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 | Enforces |
-| --- | --- |
-| `@hypersolid-dev/props-destructuring` | 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`). |
+| 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`. |
 
 ## Overriding rules
 
@@ -137,7 +150,7 @@ export default [...base({ tsconfigRootDir: import.meta.dirname }), { ignores: ['
 
 ### Test files
 
-Every preset relaxes a few rules for tests, stories, mocks and test setup (`TEST_FILES`, exported from `@hypersolid-dev/eslint-config/base`): `@typescript-eslint/no-unsafe-assignment`, `no-unsafe-member-access`, `no-invalid-void-type`, `unicorn/no-error-property-assignment` and `no-global-object-property-assignment` in `base`; `@eslint-react/no-unnecessary-use-prefix` in `react` and `react-native`; and the browser-globals list (`no-restricted-globals`) in `react-native`. Source code keeps them all.
+Every preset relaxes a few rules for tests, stories, mocks and test setup (`TEST_FILES`, exported from `@hypersolid-dev/eslint-config/base`): `@typescript-eslint/no-unsafe-assignment`, `no-unsafe-member-access`, `no-invalid-void-type`, `unicorn/no-error-property-assignment` and `no-global-object-property-assignment` in `base`; `@eslint-react/no-unnecessary-use-prefix` in `react` and `react-native`; and the browser-globals list (`no-restricted-globals`) in `react-native`. `unicorn/consistent-function-scoping` and the source-only custom rules marked "Off in tests" above are also off there. Source code keeps them all.
 
 ```js
 import base, { TEST_FILES } from '@hypersolid-dev/eslint-config/base';
@@ -149,7 +162,25 @@ export default [...base({ tsconfigRootDir: import.meta.dirname }), { files: TEST
 
 - `unicorn/consistent-boolean-name` skips function names in every preset; `react` and `react-native` also skip hook (`^use[A-Z]`) and jest mock (`^mock`) names, and `react-native` reads Reanimated's `SharedValue` through `.value`.
 - `no-secrets/no-secrets` ignores `data:` URIs in JSON; in `react-native` it also ignores Expo and native identifiers (`EXPO_*`, `android.settings.*`, `@xml…`, `NSPrivacy*`, i18n `Namespace/Key` paths) in code and JSON.
-- `@typescript-eslint/switch-exhaustiveness-check` accepts a `default` branch as exhaustive (`considerDefaultExhaustiveForUnions`). Set it to `false` to make every new union member fail its switches.
+- `no-restricted-syntax` bans an empty `.catch(() => {})` and `{ default as x }` specifiers in `base`; `react` adds `"use no memo"`, hooks called inside JSX and `React.useX`; `react-native` adds `pointerEvents` as a prop. Each preset restates the earlier entries, because flat config replaces options; spread them the same way when you add your own.
+- `@typescript-eslint/naming-convention`: values camelCase, PascalCase or UPPER_CASE, functions camelCase or PascalCase, types PascalCase; destructured names are exempt. `react` adds that a boolean prop doesn't start with `show`, `display` or `hide`.
+- `@typescript-eslint/no-restricted-imports` bans `InteractionManager` (deprecated in the New Architecture) and `Animated` (runs on the JS thread; use react-native-reanimated) from `react-native`. Type imports are allowed.
+- `@typescript-eslint/switch-exhaustiveness-check` requires a `default` on a switch over a non-union value (`requireDefaultForNonUnion`), and accepts a `default` branch as exhaustive (`considerDefaultExhaustiveForUnions`). Set it to `false` to make every new union member fail its switches.
+
+- `react-hooks/exhaustive-deps` is off: the compiler rules `react-hooks/memo-dependencies` and `react-hooks/exhaustive-effect-dependencies` check the same arrays, and both on would report one missing dependency twice. `eslint-plugin-react-hooks` is pinned to an exact version, so a plugin release can't change what the compiler rules report without a release of this package.
+
+### 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:
+
+```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' } },
+];
+```
 
 ## Formatting and import order
 

package/dist/base.js

@@ -5,7 +5,8 @@ import { createNodeResolver, flatConfigs as importX } from 'eslint-plugin-import
 import noSecrets from 'eslint-plugin-no-secrets';
 import unicorn from 'eslint-plugin-unicorn';
 import { configs as tsConfigs } from 'typescript-eslint';
-import { BOOLEAN_NAME_OPTIONS, CODE_FILES, NUMBER_GLOBAL_RESTRICTIONS, raiseWarnings, scopeToCode, TEST_FILES } from './shared.js';
+import hypersolid from './plugins/hypersolid.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';
 export default function base(options) {
@@ -22,9 +23,11 @@ export default function base(options) {
             js.configs.recommended,
             // Type-aware TypeScript rules, the strict set. Bad: `const n: any = data; n.foo()`, `if (alwaysTrue)`.
             ...tsConfigs.strictTypeChecked,
-            // Unicorn's recommended set minus its taste rules (file naming, `no-null`, `no-array-reduce`,
-            // abbreviations, formatting); what remains catches bugs and outdated APIs.
-            // Bad: `array.indexOf(x) !== -1` (use `includes`), `new Error()` without a message.
+            /**
+             * Unicorn's recommended set minus its taste rules (file naming, `no-null`, `no-array-reduce`,
+             * abbreviations, formatting); what remains catches bugs and outdated APIs.
+             * Bad: `array.indexOf(x) !== -1` (use `includes`), `new Error()` without a message.
+             */
             unicorn.configs.unopinionated,
             // Import mistakes. Bad: `import { missing } from './module.js'`, importing the same module twice.
             importX.recommended,
@@ -41,7 +44,7 @@ export default function base(options) {
                     tsconfigRootDir: options.tsconfigRootDir,
                 },
             },
-            plugins: { 'no-secrets': noSecrets },
+            plugins: { '@hypersolid-dev': hypersolid, 'no-secrets': noSecrets },
             settings: {
                 'import-x/resolver-next': [createTypeScriptImportResolver(), createNodeResolver()],
             },
@@ -60,6 +63,16 @@ export default function base(options) {
                 'preserve-caught-error': 'error',
                 // The coercing globals. Bad: `isNaN('x')` (use `Number.isNaN`).
                 'no-restricted-globals': ['error', ...NUMBER_GLOBAL_RESTRICTIONS],
+                // The same globals reached through an object, and `Math.pow`. Bad: `globalThis.isNaN(x)`, `Math.pow(2, 8)` (use `2 ** 8`).
+                'no-restricted-properties': [
+                    'error',
+                    ...['global', 'globalThis'].flatMap((object) => NUMBER_GLOBAL_RESTRICTIONS.map(({ name, message }) => ({ object, property: name, message }))),
+                    { object: 'Math', property: 'pow', message: 'Use the `**` operator.' },
+                ],
+                // A silent catch and renamed default specifiers. Bad: `.catch(() => {})`, `import { default as user } from './user.js'`.
+                'no-restricted-syntax': ['error', ...BASE_RESTRICTED_SYNTAX],
+                // `let` only for a reassigned binding, ignoring one read in a closure before its assignment. Bad: `let total = 1` never reassigned.
+                'prefer-const': ['error', { destructuring: 'any', ignoreReadBeforeAssign: true }],
                 // `type` over `interface`. Bad: `interface User { name: string }`.
                 '@typescript-eslint/consistent-type-definitions': ['error', 'type'],
                 // Every promise is awaited or handled; `void` does not count. Bad: `save();`, `void save();`.
@@ -79,8 +92,22 @@ export default function base(options) {
                 ],
                 // No value returned where `void` is expected. Bad: `run(() => 1)` with `run(callback: () => void)`.
                 '@typescript-eslint/strict-void-return': ['error', { allowReturnAny: true }],
-                // A `switch` on a union handles every member or has a `default`. Bad: `switch (shape) { case 'circle': … }` missing `'square'`.
-                '@typescript-eslint/switch-exhaustiveness-check': ['error', { considerDefaultExhaustiveForUnions: true }],
+                /**
+                 * A `switch` on a union handles every member or has a `default`. Bad: `switch (shape) { case 'circle': … }` missing `'square'`.
+                 * and a `switch` on a non-union value has a `default`. Bad: `switch (count) { case 0: … }` without `default`.
+                 */
+                '@typescript-eslint/switch-exhaustiveness-check': [
+                    'error',
+                    { considerDefaultExhaustiveForUnions: true, requireDefaultForNonUnion: true },
+                ],
+                // Rest siblings omit a key, and `_` skips an argument. Bad: `const unused = 1`.
+                '@typescript-eslint/no-unused-vars': ['error', { ignoreRestSiblings: true, argsIgnorePattern: '^_+$' }],
+                // A type the literal already says. Bad: `const count: number = 5`.
+                '@typescript-eslint/no-inferrable-types': 'error',
+                // `startsWith`/`endsWith` state the intent. Bad: `name.indexOf('use') === 0`.
+                '@typescript-eslint/prefer-string-starts-ends-with': 'error',
+                // Values camelCase, PascalCase or UPPER_CASE; functions camelCase or PascalCase; types PascalCase. Bad: `const user_name = ''`, `type props = {}`.
+                '@typescript-eslint/naming-convention': ['error', ...NAMING_CONVENTION],
                 // The caught value is named `error`. Bad: `catch (e)`.
                 'unicorn/catch-error-name': 'error',
                 // A boolean reads as a claim. Bad: `const visible = true` (use `isVisible`).
@@ -108,6 +135,20 @@ export default function base(options) {
                 'unicorn/no-unsafe-promise-all-settled-values': 'error',
                 // An async function already wraps its return value. Bad: `async () => Promise.resolve(1)`.
                 'unicorn/no-useless-promise-resolve-reject': 'error',
+                // No cleartext URLs. Bad: a cleartext `http` URL in a `fetch`.
+                'unicorn/prefer-https': 'error',
+                // `.catch()` over a second `.then()` argument. Bad: `promise.then(render, report)`.
+                'unicorn/prefer-then-catch': 'error',
+                // A helper that closes over nothing belongs at module level. Bad: `function outer() { const double = (x) => x * 2; }`.
+                'unicorn/consistent-function-scoping': 'error',
+                // 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',
+                // 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',
+                // `//` and `/* */` comments stay one row; longer prose is a `/** */` block. Bad: three `//` lines in a row.
+                '@hypersolid-dev/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.
@@ -158,6 +199,10 @@ export default function base(options) {
                 'unicorn/no-error-property-assignment': 'off',
                 // Off: test setup stubs globals. Bad: `globalThis.fetch = mockFetch` reported.
                 'unicorn/no-global-object-property-assignment': 'off',
+                // 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',
             },
         },
         {

package/dist/plugins/hypersolid.d.ts

@@ -0,0 +1,3 @@
+import type { ESLint } from 'eslint';
+declare const plugin: ESLint.Plugin;
+export default plugin;

package/dist/plugins/hypersolid.js

@@ -0,0 +1,38 @@
+import propsDestructuring from './props-destructuring.js';
+import booleanGuard from './rules/boolean-guard.js';
+import commentStyle from './rules/comment-style.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 regexPatternConst from './rules/regex-pattern-const.js';
+import testIdDefault from './rules/test-id-default.js';
+import testIdFormat from './rules/test-id-format.js';
+import testIdFromProps from './rules/test-id-from-props.js';
+import undefinedCheck from './rules/undefined-check.js';
+import webviewAllowList from './rules/webview-allow-list.js';
+// One plugin object for every preset: flat config rejects two different objects under the same plugin name.
+const plugin = {
+    meta: { name: '@hypersolid-dev/eslint-config' },
+    rules: {
+        ...propsDestructuring.rules,
+        // typescript-eslint's `RuleModule` and ESLint's `Rule.RuleModule` type the same runtime object differently.
+        ...Object.fromEntries(Object.entries({
+            'boolean-guard': booleanGuard,
+            'comment-style': commentStyle,
+            'effect-takes-callback': effectTakesCallback,
+            'jsdoc-for-props': jsdocForProps,
+            'no-callback-payload': noCallbackPayload,
+            'no-markup-variable': noMarkupVariable,
+            'no-unnecessary-use-callback': noUnnecessaryUseCallback,
+            'regex-pattern-const': regexPatternConst,
+            'test-id-default': testIdDefault,
+            'test-id-format': testIdFormat,
+            'test-id-from-props': testIdFromProps,
+            'undefined-check': undefinedCheck,
+            'webview-allow-list': webviewAllowList,
+        })),
+    },
+};
+export default plugin;

package/dist/plugins/props-destructuring.js

@@ -1,7 +1,8 @@
 const FUNCTION_TYPES = new Set(['ArrowFunctionExpression', 'FunctionExpression', 'FunctionDeclaration']);
 const WRAPPERS = new Set(['memo', 'forwardRef']);
 const isFunction = (node) => typeof node === 'object' && node !== null && FUNCTION_TYPES.has(node.type);
-const isPascalCase = (name) => typeof name === 'string' && /^[A-Z]/.test(name);
+const PASCAL_CASE_PATTERN = /^[A-Z]/;
+const isPascalCase = (name) => typeof name === 'string' && PASCAL_CASE_PATTERN.test(name);
 function isJsx(node) {
     if (typeof node !== 'object' || node === null) {
         return false;

package/dist/plugins/react-native-a11y/index.js

@@ -38,8 +38,10 @@ const rules = {
     "require-invert-colors-opt-out": requireInvertColorsOptOut,
     "require-modal-a11y-on-sheets": requireModalA11yOnSheets,
 };
-// React Native accessibility rules, maintained in-house. Originally @hypersolid-dev/eslint-config (react-native preset) v2.2.0 (MIT), from
-// before Triple became Hypersolid. Rule docs live in ./docs.
+/**
+ * React Native accessibility rules, maintained in-house. Originally @hypersolid-dev/eslint-config (react-native preset) v2.2.0 (MIT), from
+ * before Triple became Hypersolid. Rule docs live in ./docs.
+ */
 const plugin = {
     meta: { name: "@hypersolid-dev/react-native-a11y" },
     rules,

package/dist/plugins/react-native-a11y/rules/accessibility-hint.js

@@ -54,8 +54,10 @@ const accessibilityHint = createRule({
                     isInsideFocusStop(node)) {
                     return;
                 }
-                // Visible text already names the control, so the hint has a name to
-                // hang off even without a label
+                /**
+                 * Visible text already names the control, so the hint has a name to
+                 * hang off even without a label
+                 */
                 const { children } = node.parent;
                 if (hasLabelProperty(node) || getAmountOfTextChildren(children) > 0) {
                     return;

package/dist/plugins/react-native-a11y/rules/accessibility-label.js

@@ -49,8 +49,10 @@ const accessibilityLabel = createRule({
                 if (!components.includes(componentName)) {
                     return;
                 }
-                // Decorative content is hidden rather than labelled
-                // The surface above answers for its children
+                /**
+                 * Decorative content is hidden rather than labelled
+                 * The surface above answers for its children
+                 */
                 if (isHiddenFromAccessibility(node) ||
                     isNotAFocusStop(node) ||
                     isInsideFocusStop(node)) {
@@ -73,8 +75,10 @@ const accessibilityLabel = createRule({
                 // Visible text already names the control
                 const { children } = node.parent;
                 const amountOfTextChildren = getAmountOfTextChildren(children);
-                // 2.5.3: a spoken command matches the accessible name, so the name
-                // has to contain what the user can read
+                /**
+                 * 2.5.3: a spoken command matches the accessible name, so the name
+                 * has to contain what the user can read
+                 */
                 const label = getLiteralLabel(node);
                 const visible = getLiteralTextChild(children);
                 if (label !== undefined &&

package/dist/plugins/react-native-a11y/rules/accessibility-role.js

@@ -45,8 +45,10 @@ const accessibilityRole = createRule({
                 if (!components.includes(componentName)) {
                     return;
                 }
-                // Decorative content is hidden rather than given a role
-                // The surface above answers for its children
+                /**
+                 * Decorative content is hidden rather than given a role
+                 * The surface above answers for its children
+                 */
                 if (isHiddenFromAccessibility(node) ||
                     isNotAFocusStop(node) ||
                     isInsideFocusStop(node)) {

package/dist/plugins/react-native-a11y/rules/gate-announce-for-accessibility.js

@@ -29,9 +29,11 @@ const gateAnnounceForAccessibility = createRule({
         schema: [],
     },
     create(context) {
-        // A bare announce is the legitimate cross-platform pattern. The double
-        // announcement only happens where a live region covers Android already, so
-        // that pairing is the trigger rather than the call on its own.
+        /**
+         * A bare announce is the legitimate cross-platform pattern. The double
+         * announcement only happens where a live region covers Android already, so
+         * that pairing is the trigger rather than the call on its own.
+         */
         let hasLiveRegion = false;
         const announces = [];
         const platformChecks = [];
@@ -60,9 +62,11 @@ const gateAnnounceForAccessibility = createRule({
                 if (!hasLiveRegion) {
                     return;
                 }
-                // ponytail: any Platform.OS in the same function counts as the gate.
-                // Lenient on purpose — a guard clause, a switch and a named boolean are
-                // all real gates, and none of them is an ancestor of the call.
+                /**
+                 * ponytail: any Platform.OS in the same function counts as the gate.
+                 * Lenient on purpose — a guard clause, a switch and a named boolean are
+                 * all real gates, and none of them is an ancestor of the call.
+                 */
                 const gatedScopes = new Set(platformChecks.map((check) => enclosingScope(check)));
                 for (const announce of announces) {
                     if (gatedScopes.has(enclosingScope(announce))) {

package/dist/plugins/react-native-a11y/rules/input-purpose.js

@@ -56,12 +56,16 @@ const inputPurpose = createRule({
                 if (!components.includes(componentName)) {
                     return;
                 }
-                // The missing token may arrive through the spread; an explicit one is
-                // already wrong whatever the spread carries
+                /**
+                 * The missing token may arrive through the spread; an explicit one is
+                 * already wrong whatever the spread carries
+                 */
                 const autoComplete = getProperty(node, "autoComplete");
                 const textContentType = getProperty(node, "textContentType");
-                // One fix either way, so one finding: the sharper message where only
-                // the iOS half is already set
+                /**
+                 * One fix either way, so one finding: the sharper message where only
+                 * the iOS half is already set
+                 */
                 if (!autoComplete && !hasSpreadProps(node)) {
                     context.report(textContentType
                         ? { node: textContentType, messageId: "missingAutoComplete" }

package/dist/plugins/react-native-a11y/rules/no-unnamed-interactive.js

@@ -46,8 +46,10 @@ const noUnnamedInteractive = createRule({
                 if (componentName === undefined) {
                     return;
                 }
-                // accessibility-label owns the components it knows by name, so a
-                // control does not collect a finding from both rules
+                /**
+                 * accessibility-label owns the components it knows by name, so a
+                 * control does not collect a finding from both rules
+                 */
                 if (ALREADY_COVERED.has(componentName) &&
                     !components.includes(componentName)) {
                     return;

package/dist/plugins/react-native-a11y/rules/require-invert-colors-opt-out.js

@@ -30,10 +30,12 @@ const requireInvertColorsOptOut = createRule({
         ],
     },
     create(context) {
-        // Matched on the element name, so an image from expo-image or a wrapper
-        // named through additionalComponents is checked the same way. Deliberately
-        // not gated on the import statement: that reads the source text and misses
-        // a codebase whose quote style differs.
+        /**
+         * Matched on the element name, so an image from expo-image or a wrapper
+         * named through additionalComponents is checked the same way. Deliberately
+         * not gated on the import statement: that reads the source text and misses
+         * a codebase whose quote style differs.
+         */
         const components = getComponents({
             context,
             defaultComponents: IMAGES,

package/dist/plugins/react-native-a11y/rules/require-modal-a11y-on-sheets.js

@@ -28,9 +28,11 @@ const requireModalA11yOnSheets = createRule({
         ],
     },
     create(context) {
-        // No default list: a sheet is whatever this project calls one, and
-        // hardcoding one library's component names would fire on a codebase that
-        // happens to reuse them. Configure through additionalComponents.
+        /**
+         * No default list: a sheet is whatever this project calls one, and
+         * hardcoding one library's component names would fire on a codebase that
+         * happens to reuse them. Configure through additionalComponents.
+         */
         const components = getComponents({ context });
         return {
             JSXOpeningElement(node) {

package/dist/plugins/react-native-a11y/utils/get-amount-of-text-children.js

@@ -1,7 +1,9 @@
 import { AST_NODE_TYPES } from "@typescript-eslint/types";
 import { TEXTS } from "../constants/react-native-components.js";
-// ponytail: `Text` by name only. A renamed import or a differently named text
-// atom reads as no text; pass it through `additionalComponents` if that bites.
+/**
+ * ponytail: `Text` by name only. A renamed import or a differently named text
+ * atom reads as no text; pass it through `additionalComponents` if that bites.
+ */
 const isTextElement = (node) => node.openingElement.name.type === AST_NODE_TYPES.JSXIdentifier &&
     TEXTS.includes(node.openingElement.name.name);
 /** `<Text>{null}</Text>` and `<Text> </Text>` render nothing a user can read. */
@@ -45,8 +47,10 @@ const getAmountOfTextChildren = (children, oldAmount = 0) => children.reduce((am
             expression.type === AST_NODE_TYPES.TemplateLiteral) {
             return amount + 1;
         }
-        // JSX inside an expression renders like any other child, and a branch
-        // that renders text still names the control
+        /**
+         * JSX inside an expression renders like any other child, and a branch
+         * that renders text still names the control
+         */
         const rendered = (isJsxChild(expression) ? [expression] : branchesOf(expression)).filter(isJsxChild);
         return rendered.length > 0
             ? getAmountOfTextChildren(rendered, amount)

package/dist/plugins/react-native-a11y/utils/get-element-name.js

@@ -41,7 +41,7 @@ const collectAliases = (program) => {
 const resolveAlias = (node, name) => {
     const program = getProgram(node);
     let aliases = aliasesByProgram.get(program);
-    if (aliases === undefined) {
+    if (!aliases) {
         aliases = collectAliases(program);
         aliasesByProgram.set(program, aliases);
     }

package/dist/plugins/rules/boolean-guard.d.ts

@@ -0,0 +1,30 @@
+import { ESLintUtils } from "@typescript-eslint/utils";
+/**
+ * Render a conditional child with `&&`, and coerce the condition when it can be
+ * a falsy value React draws.
+ *
+ * React skips `null`, `undefined`, `true` and `false`, but renders `0` and `""`.
+ * On React Native a rendered `0` outside a `<Text>` throws, so
+ * `{items.length && <List />}` is a crash on an empty list.
+ *
+ * - `{cond ? <X /> : null}` is a plain `&&` (`preferLogical`), or `!!cond && …`
+ *   when the condition can leak (`preferLogicalCoerced`). `{cond ? null : <X />}`
+ *   is advised as `!cond && …`.
+ * - `{count && <X />}` leaks when the type admits `0`, `""` or `0n` (`coerce`).
+ *   For `a && b && <X />` only `b` is judged.
+ * - `{x !== undefined && <X />}` is `!!x` when `undefined` is the type's only
+ *   falsy value (`preferBangBang`).
+ *
+ * Leaking is read from the type: string, number, bigint, branded primitives,
+ * `any` and `unknown` leak; literal unions leak only if they hold `""`/`0`/`0n`.
+ * Attributes and element-vs-element ternaries are left alone. A guard held in a
+ * `const` is checked when every read of that name is drawn as a child.
+ *
+ * Limits: guards returned from functions or render props are not followed, and
+ * `||`/`??` guards are not examined. Silent without type information.
+ */
+type MessageIds = "preferLogical" | "preferLogicalCoerced" | "coerce" | "preferBangBang";
+declare const booleanGuard: ESLintUtils.RuleModule<MessageIds, [], unknown, ESLintUtils.RuleListener> & {
+    name: string;
+};
+export default booleanGuard;