ESLint 9

.github/workflows/ci.yml

@@ -10,35 +10,28 @@ jobs:
     runs-on: ubuntu-latest
     strategy:
       matrix:
-        node-version: [18.x]
+        node-version: [22]
 
     steps:
-    - uses: actions/checkout@v4
+      - uses: actions/checkout@v4
 
-    - name: Use Node.js ${{ matrix.node-version }}
-      uses: actions/setup-node@v4
-      with:
-        node-version: ${{ matrix.node-version }}
+      - name: Install pnpm
+        uses: pnpm/action-setup@v4
 
-    - name: Get Yarn cache directory
-      id: yarn-cache-dir-path
-      run: echo "dir=$(yarn cache dir)" >> $GITHUB_OUTPUT
+      - name: Use Node.js ${{ matrix.node-version }}
+        uses: actions/setup-node@v4
+        with:
+          node-version: ${{ matrix.node-version }}
+          cache: 'pnpm'
 
-    - name: Use Yarn cache
-      uses: actions/cache@v4
-      id: yarn-cache
-      with:
-        path: ${{ steps.yarn-cache-dir-path.outputs.dir }}
-        key: ${{ runner.os }}-yarn-${{ matrix.node-version }}-${{ hashFiles('**/yarn.lock') }}
+      - name: Install dependencies
+        run: pnpm i --frozen-lockfile
 
-    - name: Install dependencies
-      run: yarn install --prefer-offline --frozen-lockfile
+      - name: Test
+        run: pnpm test
 
-    - name: Test
-      run: yarn run test
+      - name: Lint
+        run: pnpm lint
 
-    - name: Lint
-      run: yarn run lint
-
-    - name: Build
-      run: yarn run build
+      - name: Build
+        run: pnpm build

.github/workflows/release.yml

@@ -22,22 +22,22 @@ jobs:
     steps:
       - name: Checkout Repo
         uses: actions/checkout@v4
-      
+
       - name: Use Node.js
         uses: actions/setup-node@v4
 
       - name: Install dependencies
-        run: yarn install --frozen-lockfile
+        run: pnpm install --frozen-lockfile
 
       - name: Build packages
-        run: yarn build
+        run: pnpm build
 
       - name: PR or Publish
         id: changesets
         uses: changesets/action@v1
         with:
-          version: yarn run changeset version
-          publish: yarn run changeset publish
+          version: pnpm changeset version
+          publish: pnpm changeset publish
         env:
           GITHUB_TOKEN: ${{ secrets.GITHUB_TOKEN }}
           NPM_TOKEN: ${{ secrets.NPM_TOKEN }}

.gitignore

@@ -1,8 +1,24 @@
-.DS_Store
-coverage
-lib
+# Logs
+logs
+*.log
+npm-debug.log*
+yarn-debug.log*
+yarn-error.log*
+pnpm-debug.log*
+lerna-debug.log*
+
 node_modules
-npm-debug.log
-package-lock.json
-reports
-yarn-error.log
+dist
+dist-ssr
+*.local
+
+# Editor directories and files
+.vscode/*
+!.vscode/extensions.json
+.idea
+.DS_Store
+*.suo
+*.ntvs*
+*.njsproj
+*.sln
+*.sw?

.npmignore

@@ -7,7 +7,6 @@
 npm-debug.log
 /src
 /reports
-yarn-error.log
 /.vscode
 /docs
 /.github

.nvmrc

@@ -0,0 +1 @@
+v23.10.0

__tests__/__util__/languageOptionsMapper.ts

@@ -0,0 +1,25 @@
+import type { RuleTester } from 'eslint';
+import deepmerge from 'deepmerge';
+
+const defaultLanguageOptions = {
+  ecmaVersion: 2020,
+  parserOptions: {
+    ecmaFeatures: {
+      jsx: true,
+    },
+  },
+};
+
+const languageOptionsMapper = (
+  testCase: RuleTester.ValidTestCase | RuleTester.InvalidTestCase,
+) => ({
+  ...testCase,
+  errors: (<RuleTester.InvalidTestCase>testCase).errors,
+  options: testCase.options ?? [],
+  languageOptions: deepmerge(
+    defaultLanguageOptions,
+    testCase.languageOptions ?? {},
+  ),
+});
+
+export default languageOptionsMapper;

__tests__/index.test.ts

@@ -0,0 +1,46 @@
+import plugin from '../src';
+import type { Rule } from 'eslint';
+
+const rules = import.meta.glob<true, string, { default: Rule.RuleModule }>(
+  '../src/rules/*.ts',
+  { eager: true },
+);
+
+describe('all rule files should be exported by the plugin', () => {
+  it.each(Object.entries(rules))(
+    `exports %s`,
+    (rulePath, { default: ruleModule }) => {
+      const ruleName = rulePath.split('/').at(-1)?.replace('.ts', '');
+      expect(plugin.rules[ruleName as keyof typeof plugin.rules]).toEqual(
+        ruleModule,
+      );
+    },
+  );
+});
+
+describe('configs', () => {
+  it.each(['basic', 'ios', 'android', 'all'])(
+    `exports a %s config`,
+    (config) => {
+      expect(
+        plugin.configs[config as keyof typeof plugin.configs],
+      ).toBeTruthy();
+    },
+  );
+});
+
+describe('schemas', () => {
+  it.each(
+    Object.entries(rules).filter(([filename]) => !filename.includes('index')),
+  )(
+    `%s exports a schema with type object`,
+    async (_, { default: ruleModule }) => {
+      const schema =
+        ruleModule.meta &&
+        ruleModule.meta.schema &&
+        (ruleModule.meta.schema as { type: string }[])[0];
+
+      expect((schema as { type: string }).type).toBe('object');
+    },
+  );
+});

__tests__/src/rules/has-accessibility-hint-test.js → __tests__/src/rules/has-accessibility-hint.test.ts

@@ -9,7 +9,7 @@
 // -----------------------------------------------------------------------------
 
 import { RuleTester } from 'eslint';
-import parserOptionsMapper from '../../__util__/parserOptionsMapper';
+import languageOptionsMapper from '../../__util__/languageOptionsMapper';
 import rule from '../../../src/rules/has-accessibility-hint';
 
 // -----------------------------------------------------------------------------
@@ -39,13 +39,13 @@ ruleTester.run('has-accessibility-hint', rule, {
                 accessibilityHint="Navigates to the previous screen"
               />`,
     },
-  ].map(parserOptionsMapper),
+  ].map(languageOptionsMapper),
   invalid: [
     {
       code: `<TouchableOpacity
                 accessibilityLabel="Go back"
               />`,
       errors: [expectedError],
     },
-  ].map(parserOptionsMapper),
+  ].map(languageOptionsMapper),
 });