Initial implementation of convertInputSelectorsToArray codemod

Author: aryaemami59

codemods/.gitignore

@@ -0,0 +1,37 @@
+# Dependencies
+/node_modules
+
+# Production
+/build
+
+# Generated files
+.docusaurus
+.cache-loader
+
+# Misc
+.DS_Store
+.env.local
+.env.development.local
+.env.test.local
+.env.production.local
+
+npm-debug.log*
+yarn-debug.log*
+yarn-error.log*
+.cache
+.yarnrc
+.yarn/*
+!.yarn/patches
+!.yarn/releases
+!.yarn/plugins
+!.yarn/sdks
+!.yarn/versions
+.pnp.*
+*.tgz
+
+tsconfig.vitest-temp.json
+.eslintcache
+
+.yalc
+.yalc.lock
+.vscode/

codemods/.prettierignore

@@ -0,0 +1 @@
+**/__testfixtures__/

codemods/.release-it.json

@@ -0,0 +1,9 @@
+{
+  "hooks": {
+    "after:bump": "yarn && git add -u"
+  },
+  "git": {
+    "commitMessage": "Release reselect-codemods ${version}",
+    "tagName": "reselect-codemods@${version}"
+  }
+}

codemods/README.md

@@ -0,0 +1,42 @@
+# Reselect Codemods
+
+A collection of codemods for updating legacy Reselect API usage patterns to modern patterns.
+
+## Usage
+
+To run a specific codemod from this project, you would run the following:
+
+```bash
+npx reselect-codemods <TRANSFORM NAME> path/of/files/ or/some**/*glob.js
+
+# or
+
+yarn global add reselect-codemods
+reselect-codemods <TRANSFORM NAME> path/of/files/ or/some**/*glob.js
+```
+
+## Local Usage
+
+```
+node ./bin/cli.mjs <TRANSFORM NAME> path/of/files/ or/some**/*glob.js
+```
+
+## Transforms
+
+<!--TRANSFORMS_START-->
+
+- [convertInputSelectorsToArray](transforms/convertInputSelectorsToArray/README.md)
+
+<!--TRANSFORMS_END-->
+
+## Contributing
+
+### Installation
+
+- clone the repo
+- change into the repo directory
+- `yarn`
+
+### Running tests
+
+- `yarn test`

codemods/bin/cli.mjs

@@ -0,0 +1,39 @@
+#!/usr/bin/env node
+import { execaSync } from 'execa'
+import { globbySync } from 'globby'
+import { createRequire } from 'node:module'
+import path from 'node:path'
+import { fileURLToPath } from 'node:url'
+
+const require = createRequire(import.meta.url)
+
+const __filename = fileURLToPath(import.meta.url)
+const __dirname = path.dirname(__filename)
+
+const transformerDirectory = path.join(
+  __dirname,
+  '..',
+  'transforms',
+  `${process.argv[2]}/index.ts`,
+)
+
+const jscodeshiftExecutable = require.resolve('.bin/jscodeshift')
+
+const extensions = 'ts,js,tsx,jsx'
+
+execaSync(
+  jscodeshiftExecutable,
+  [
+    '-t',
+    transformerDirectory,
+    '--extensions',
+    extensions,
+    ...(process.argv.slice(3).length === 1
+      ? globbySync(process.argv[3])
+      : globbySync(process.argv.slice(3))),
+  ],
+  {
+    stdio: 'inherit',
+    stripFinalNewline: false,
+  },
+)

codemods/eslint.config.mts

@@ -0,0 +1,63 @@
+import js from '@eslint/js'
+import prettierConfig from 'eslint-config-prettier'
+import { config, configs, parser } from 'typescript-eslint'
+
+const ESLintConfig = config(
+  { name: 'ignores', ignores: ['**/dist/', '**/__testfixtures__/'] },
+  { name: 'javascript', ...js.configs.recommended },
+  ...configs.recommended,
+  ...configs.stylistic,
+  { name: 'prettier-config', ...prettierConfig },
+  {
+    name: 'main',
+    languageOptions: {
+      parser,
+      parserOptions: {
+        projectService: {
+          defaultProject: './tsconfig.json',
+        },
+        ecmaVersion: 'latest',
+      },
+    },
+    rules: {
+      'no-undef': [0],
+      '@typescript-eslint/consistent-type-imports': [
+        2,
+        { fixStyle: 'separate-type-imports', disallowTypeAnnotations: false },
+      ],
+      '@typescript-eslint/consistent-type-exports': [2],
+      '@typescript-eslint/no-unused-vars': [0],
+      '@typescript-eslint/no-explicit-any': [0],
+      '@typescript-eslint/no-empty-object-type': [
+        2,
+        { allowInterfaces: 'with-single-extends' },
+      ],
+      '@typescript-eslint/no-namespace': [
+        2,
+        { allowDeclarations: true, allowDefinitionFiles: true },
+      ],
+      '@typescript-eslint/ban-ts-comment': [0],
+      'sort-imports': [
+        2,
+        {
+          ignoreCase: false,
+          ignoreDeclarationSort: true,
+          ignoreMemberSort: false,
+          memberSyntaxSortOrder: ['none', 'all', 'multiple', 'single'],
+          allowSeparatedGroups: true,
+        },
+      ],
+    },
+    linterOptions: { reportUnusedDisableDirectives: 2 },
+  },
+  {
+    name: 'commonjs',
+    files: ['**/*.c[jt]s'],
+    languageOptions: { sourceType: 'commonjs' },
+    rules: {
+      '@typescript-eslint/no-require-imports': [0],
+    },
+  },
+)
+
+export default ESLintConfig

codemods/package.json

@@ -0,0 +1,58 @@
+{
+  "name": "reselect-codemods",
+  "version": "0.1.0",
+  "scripts": {
+    "format": "prettier -w . --config prettier.config.mjs",
+    "format:check": "prettier -c . --config prettier.config.mjs",
+    "lint": "eslint --flag unstable_ts_config -c eslint.config.mts",
+    "lint:fix": "eslint --fix --flag unstable_ts_config -c eslint.config.mts",
+    "test": "vitest --run",
+    "test:watch": "vitest --watch"
+  },
+  "bin": "./bin/cli.mjs",
+  "files": [
+    "bin/*",
+    "transforms/**/index.ts",
+    "transforms/**/README.md",
+    "package.json",
+    "README.md"
+  ],
+  "keywords": [
+    "redux",
+    "redux-toolkit",
+    "reselect",
+    "codemod"
+  ],
+  "dependencies": {
+    "execa": "^9.3.1",
+    "globby": "^14.0.2",
+    "jscodeshift": "^17.0.0",
+    "ts-node": "^10.9.2",
+    "typescript": "^5.5.4"
+  },
+  "devDependencies": {
+    "@types/eslint": "^9.6.1",
+    "@types/eslint-config-prettier": "^6.11.3",
+    "@types/eslint__js": "^8.42.3",
+    "@types/jscodeshift": "^0.11.11",
+    "@types/node": "^22.5.0",
+    "eslint": "^9.9.1",
+    "eslint-config-prettier": "^9.1.0",
+    "jiti": "^2.0.0-beta.3",
+    "prettier": "^3.3.3",
+    "typescript-eslint": "^8.3.0",
+    "vitest": "^2.0.5"
+  },
+  "engines": {
+    "node": ">= 16"
+  },
+  "publishConfig": {
+    "access": "public"
+  },
+  "repository": {
+    "directory": "codemods",
+    "type": "git",
+    "url": "git+https://github.com/reduxjs/reselect.git"
+  },
+  "sideEffects": false
+}

codemods/prettier.config.mjs

@@ -0,0 +1,10 @@
+/**
+ * @satisfies {import('prettier').Config}
+ */
+const prettierConfig = {
+  arrowParens: 'avoid',
+  semi: false,
+  singleQuote: true,
+}
+
+export default prettierConfig

codemods/transformTestUtils.ts

@@ -0,0 +1,71 @@
+import { globbySync } from 'globby'
+import type { Transform } from 'jscodeshift'
+import type { TestOptions } from 'jscodeshift/src/testUtils'
+import { applyTransform } from 'jscodeshift/src/testUtils'
+import fs from 'node:fs'
+import path from 'node:path'
+
+export const runTransformTest = (
+  name: string,
+  transform: Transform,
+  parser: TestOptions['parser'],
+  fixturePath: string,
+) => {
+  describe(name, () => {
+    globbySync('**/*.input.*', {
+      cwd: fixturePath,
+      absolute: true,
+      objectMode: true,
+    })
+      .map(entry => entry.name)
+      .forEach(filename => {
+        const extension = path.extname(filename)
+
+        const testName = filename.replace(`.input${extension}`, '')
+
+        const testInputPath = path.join(fixturePath, `${testName}${extension}`)
+
+        const inputPath = path.join(
+          fixturePath,
+          `${testName}.input${extension}`,
+        )
+
+        const outputPath = path.join(
+          fixturePath,
+          `${testName}.output${extension}`,
+        )
+
+        const inputFileContent = fs.readFileSync(inputPath, 'utf8')
+
+        const expectedOutput = fs.readFileSync(outputPath, 'utf8')
+
+        describe(`${testName}${extension}`, () => {
+          it('transforms correctly', () => {
+            const output = applyTransform(
+              transform,
+              {},
+              {
+                path: testInputPath,
+                source: inputFileContent,
+              },
+              { parser },
+            )
+            expect(output).toBe(expectedOutput.trim().replace('\r\n', '\n'))
+          })
+
+          it('is idempotent', () => {
+            const output = applyTransform(
+              transform,
+              {},
+              {
+                path: testInputPath,
+                source: inputFileContent,
+              },
+              { parser },
+            )
+            expect(output).toBe(expectedOutput.trim().replace('\r\n', '\n'))
+          })
+        })
+      })
+  })
+}