feat(codemod): improvements to s1-to-s2 codemod (agent mode, E2E tests, more) (adobe#9754)

* fixed TabList render function children kept Item instead of converting to Tab

* import cleanup: removeComponentImport was only checking the first matching import and could miss v3 imports when an @react-spectrum/s2 import appears first

* dynamic import('@react-spectrum/s2') was incorrectly flagged as a v3 dynamic import

* Fixed in codemod options parsing

* more import fixes

* add agent mode (non-interactive, run transforms only)

* add e2e tests

* have --components include related components

* yarn.lock

* update outdated links

* bump @adobe/react-spectrum version

* fix missing imports and remove unused

* fix snapshot

* yarn.lock

* map illustrations to s2 illustrations

Author: reidbarber

eslint.config.mjs

@@ -60,7 +60,8 @@ export default [{
         "packages/dev/storybook-builder-parcel/*",
         "packages/dev/storybook-react-parcel/*",
         "packages/dev/s2-docs/pages/**",
-        "packages/dev/mcp/*/dist"
+        "packages/dev/mcp/*/dist",
+        "packages/dev/codemods/src/s1-to-s2/__testfixtures__/cli/**"
     ],
 }, ...compat.extends("eslint:recommended"), {
     plugins: {
@@ -534,4 +535,4 @@ export default [{
             ...globals.browser
         }
     }
-}];
+}];

packages/dev/codemods/package.json

@@ -21,6 +21,7 @@
     "url": "https://github.com/adobe/react-spectrum"
   },
   "dependencies": {
+    "@adobe/react-spectrum": "^3.46.2",
     "@babel/parser": "^7.24.5",
     "@babel/traverse": "^7.24.5",
     "@babel/types": "^7.24.5",

packages/dev/codemods/src/index.ts

@@ -35,7 +35,14 @@ export interface S1ToS2CodemodOptions extends JSCodeshiftOptions {
    * An optional subset of components to have the s1-to-s2 codemod apply to.
    * Provide a comma-separated list of component names.
    */
-  components?: string
+  components?: string,
+  /**
+   * Whether to run the codemod in agent mode, which skips interactive prompts
+   * and package installation. This matches the shipped CLI behavior.
+   *
+   * @default false
+   */
+  agent?: boolean
 }
 
 export interface UseMonopackagesCodemodOptions extends JSCodeshiftOptions {
@@ -67,6 +74,9 @@ const options = {
   },
   'components': {
     type: 'string'
+  },
+  'agent': {
+    type: 'boolean'
   }
 };
 
@@ -80,22 +90,24 @@ if (positionals.length < 1) {
   process.exit(1);
 }
 
-const codemodName = positionals[0];
-const codemodFunction = codemods[codemodName];
+async function main() {
+  const codemodName = positionals[0];
+  const codemodFunction = codemods[codemodName];
 
-if (!codemodFunction) {
-  console.error(`Unknown codemod: ${codemodName}, available codemods: ${Object.keys(codemods).join(', ')}`);
-  process.exit(1);
-}
+  if (!codemodFunction) {
+    console.error(`Unknown codemod: ${codemodName}, available codemods: ${Object.keys(codemods).join(', ')}`);
+    process.exit(1);
+  }
 
-try {  
-  codemodFunction({
+  await Promise.resolve(codemodFunction({
     parser: 'tsx',
     ignorePattern: '**/node_modules/**',
     path: '.',
     ...values
-  });
-} catch (error) {
+  }));
+}
+
+main().catch((error) => {
   console.error(`Error running codemod: ${error}`);
   process.exit(1);
-}
+});

packages/dev/codemods/src/s1-to-s2/__testfixtures__/cli/full-project/input/package.fixture.json

@@ -0,0 +1,7 @@
+{
+  "name": "s1-to-s2-cli-fixture",
+  "private": true,
+  "devDependencies": {
+    "parcel": "^2.12.0"
+  }
+}

packages/dev/codemods/src/s1-to-s2/__testfixtures__/cli/full-project/input/src/App.tsx

@@ -0,0 +1,10 @@
+import {Button} from '@adobe/react-spectrum';
+import React from 'react';
+
+export function App() {
+  return (
+    <Button variant="cta">
+      Save
+    </Button>
+  );
+}

packages/dev/codemods/src/s1-to-s2/__testfixtures__/cli/full-project/input/src/Form.tsx

@@ -0,0 +1,6 @@
+import React from 'react';
+import {TextArea} from '@adobe/react-spectrum';
+
+export function Form() {
+  return <TextArea isQuiet />;
+}

packages/dev/codemods/src/s1-to-s2/__testfixtures__/cli/full-project/input/yarn.lock

@@ -0,0 +1 @@
+# yarn lockfile v1

packages/dev/codemods/src/s1-to-s2/__testfixtures__/cli/full-project/output/package.fixture.json

@@ -0,0 +1,7 @@
+{
+  "name": "s1-to-s2-cli-fixture",
+  "private": true,
+  "devDependencies": {
+    "parcel": "^2.12.0"
+  }
+}

packages/dev/codemods/src/s1-to-s2/__testfixtures__/cli/full-project/output/src/App.tsx

@@ -0,0 +1,6 @@
+import { Button } from "@react-spectrum/s2";
+import React from 'react';
+
+export function App() {
+  return (<Button variant="accent">Save</Button>);
+}

packages/dev/codemods/src/s1-to-s2/__testfixtures__/cli/full-project/output/src/Form.tsx

@@ -0,0 +1,6 @@
+import React from 'react';
+import { TextArea } from "@react-spectrum/s2";
+
+export function Form() {
+  return <TextArea />;
+}