@wise/wds-codemods 0.0.1-experimental-4bee5c0 → 0.0.1-experimental-b974fa6

package/babel.config.js

@@ -1,28 +0,0 @@
-export default {
-  presets: [
-    [
-      '@babel/preset-env',
-      {
-        targets: { node: 'current' },
-        modules: false,
-      },
-    ],
-    '@babel/preset-typescript',
-  ],
-  plugins: ['@babel/plugin-syntax-import-meta', 'babel-plugin-transform-import-meta'],
-  env: {
-    test: {
-      presets: [
-        [
-          '@babel/preset-env',
-          {
-            targets: { node: 'current' },
-            modules: 'commonjs',
-          },
-        ],
-        '@babel/preset-typescript',
-      ],
-      plugins: ['@babel/plugin-syntax-import-meta', 'babel-plugin-transform-import-meta'],
-    },
-  },
-};

package/commitlint.config.js

@@ -1,3 +0,0 @@
-export default {
-  extends: ['@commitlint/config-conventional'],
-};

package/eslint.config.js

@@ -1,15 +0,0 @@
-import baseStrict from '@wise/eslint-config/base-strict';
-import jest from '@wise/eslint-config/jest';
-
-export default [
-  ...baseStrict,
-  ...jest,
-  {
-    ignores: ['dist', 'test-button.tsx'],
-  },
-  {
-    rules: {
-      'functional/immutable-data': 'off',
-    },
-  },
-];

package/jest.config.js

@@ -1,9 +0,0 @@
-export default {
-  testEnvironment: 'node',
-  testPathIgnorePatterns: ['<rootDir>/dist/'],
-  extensionsToTreatAsEsm: ['.ts', '.tsx'],
-  transform: {
-    '^.+\\.tsx?$': ['babel-jest', { configFile: './babel.config.js' }],
-  },
-  transformIgnorePatterns: ['/node_modules/'],
-};

package/mkdocs.yml

@@ -1,4 +0,0 @@
-site_name: Repository docs
-
-nav:
-  - Introduction: index.md

package/renovate.json

@@ -1,9 +0,0 @@
-{
-  "extends": ["github>transferwise/renovate-config:node-lib", "schedule:weekly"],
-  "packageRules": [
-    {
-      "matchDatasources": ["npm"],
-      "registryUrls": ["https://registry.npmjs.org"]
-    }
-  ]
-}

package/src/__tests__/runCodemod.test.ts

@@ -1,96 +0,0 @@
-import { execSync } from 'child_process';
-import path from 'path';
-import { fileURLToPath } from 'url';
-
-import { runCodemod } from '../runCodemod';
-import * as utils from '../utils';
-
-Object.defineProperty(globalThis, 'import', {
-  value: {
-    meta: {
-      url: 'file:///current/file/path.js',
-    },
-  },
-  writable: true,
-});
-
-jest.mock('node:fs/promises');
-jest.mock('child_process', () => ({
-  execSync: jest.fn(),
-}));
-
-jest.mock('path', () => ({
-  resolve: jest.fn((...args: string[]) => args.join('/')),
-  dirname: jest.fn((filePath: string) => {
-    // Simple implementation that mimics path.dirname
-    const lastSlashIndex = filePath.lastIndexOf('/');
-    return lastSlashIndex !== -1 ? filePath.substring(0, lastSlashIndex) : '.';
-  }),
-}));
-
-jest.mock('../utils', () => ({
-  loadTransformModules: jest.fn(),
-  getOptions: jest.fn(),
-  handleError: jest.fn(),
-}));
-
-describe('runCodemod', () => {
-  const consoleDebugSpy = jest.spyOn(console, 'debug').mockImplementation(() => {});
-  const consoleLogSpy = jest.spyOn(console, 'log').mockImplementation(() => {});
-  const consoleErrorSpy = jest.spyOn(console, 'error').mockImplementation(() => {});
-
-  afterEach(() => {
-    jest.clearAllMocks();
-  });
-
-  it('should run jscodeshift with correct command using mock transformsDir', async () => {
-    const mockTransformsDir = '/mock/dist/transforms';
-
-    (path.resolve as jest.Mock).mockImplementation((...args: string[]) => args.join('/'));
-    (utils.loadTransformModules as jest.Mock).mockResolvedValue({
-      transformFiles: [Promise.resolve('transformA'), Promise.resolve('transformB')],
-    });
-    (utils.getOptions as jest.Mock).mockResolvedValue({
-      transformFile: 'transformA',
-      targetPath: 'src/',
-      dry: true,
-      print: false,
-    });
-
-    await runCodemod(mockTransformsDir);
-
-    expect(utils.loadTransformModules).toHaveBeenCalledWith(mockTransformsDir);
-    expect(utils.getOptions).toHaveBeenCalledWith(['transformA', 'transformB']);
-    expect(execSync).toHaveBeenCalledWith(
-      'npx jscodeshift -t /mock/dist/transforms/transformA.js src/ --dry',
-      { stdio: 'inherit' },
-    );
-  });
-
-  it('should throw error if no transform scripts found', async () => {
-    (path.resolve as jest.Mock).mockReturnValue('/mock/dist/transforms');
-    (utils.loadTransformModules as jest.Mock).mockResolvedValue({
-      transformFiles: [],
-    });
-
-    await runCodemod('/mock/dist/transforms');
-
-    expect(execSync).not.toHaveBeenCalled();
-  });
-
-  it('should log error if exception thrown', async () => {
-    const testError = new Error('Test error');
-
-    (utils.loadTransformModules as jest.Mock).mockRejectedValue(testError);
-
-    const mockExit = jest.spyOn(process, 'exit').mockImplementation(((code?: number) => {
-      throw new Error(`Process exited with code ${code}`);
-    }) as never);
-
-    await runCodemod('/mock/dist/transforms').catch(() => {});
-
-    expect(consoleErrorSpy).toHaveBeenCalledWith('Error running codemod:', testError.message);
-
-    mockExit.mockRestore();
-  });
-});

package/src/index.ts

@@ -1,4 +0,0 @@
-#!/usr/bin/env node
-import { runCodemod } from './runCodemod';
-
-void runCodemod();

package/src/runCodemod.ts

@@ -1,88 +0,0 @@
-#!/usr/bin/env node
-
-import fs from 'node:fs/promises';
-
-import { execSync } from 'child_process';
-import path from 'path';
-import { fileURLToPath } from 'url';
-
-import { getOptions, handleError, loadTransformModules } from './utils';
-
-const currentFilePath = fileURLToPath(import.meta.url);
-const currentDirPath = path.dirname(currentFilePath);
-
-async function runCodemod(transformsDir?: string) {
-  try {
-    const resolvedTransformsDir =
-      transformsDir ?? path.resolve(currentDirPath, '../dist/transforms');
-    console.debug(`Resolved transforms directory: ${resolvedTransformsDir}`);
-
-    const { transformFiles } = await loadTransformModules(resolvedTransformsDir);
-
-    if (transformFiles.length === 0) {
-      throw new Error(`No transform scripts found in directory: ${resolvedTransformsDir}`);
-    }
-
-    const resolvedTransformFiles = await Promise.all(transformFiles);
-    const options = await getOptions(resolvedTransformFiles);
-
-    const codemodPath = path.resolve(resolvedTransformsDir, `${options.transformFile}.js`);
-    console.debug(`Resolved codemod path: ${codemodPath}`);
-
-    const args = [
-      '-t',
-      codemodPath,
-      options.targetPath,
-      options.dry ? '--dry' : '',
-      options.print ? '--print' : '',
-      options.ignorePattern
-        ? options.ignorePattern
-            .split(',')
-            .map((pattern) => `--ignore-pattern=${pattern.trim()}`)
-            .join(' ')
-        : '',
-      options.gitignore ? '--gitignore' : '',
-    ].filter(Boolean);
-
-    const command = `npx jscodeshift ${args.join(' ')}`;
-
-    console.debug(`Running: ${command}`);
-
-    const reportPath = path.resolve(process.cwd(), 'codemod-report.txt');
-
-    try {
-      await fs.access(reportPath);
-      await fs.rm(reportPath);
-      console.debug(`Removed existing report file: ${reportPath}`);
-    } catch {
-      console.debug(`No existing report file to remove: ${reportPath}`);
-    }
-
-    execSync(command, { stdio: 'inherit' });
-
-    try {
-      const reportContent = await fs.readFile(reportPath, 'utf8');
-      const lines = reportContent.split('\n').filter(Boolean);
-      if (lines.length) {
-        console.log(
-          `\n⚠️  ${lines.length} manual review${lines.length > 1 ? 's are' : ' is'} required. See ${reportPath} for details.`,
-        );
-      } else {
-        console.debug(`Report file exists but is empty: ${reportPath}`);
-      }
-    } catch {
-      console.debug(`No report file generated - no manual reviews needed`);
-    }
-  } catch (error: unknown) {
-    if (error instanceof Error) {
-      console.error('Error running codemod:', error.message);
-    } else {
-      console.error('Error running codemod:', error);
-    }
-    if (process.env.NODE_ENV !== 'test') {
-      process.exit(1);
-    }
-  }
-}
-
-export { runCodemod };

package/src/transforms/button/__tests__/button.test.tsx

@@ -1,153 +0,0 @@
-import fs from 'node:fs/promises';
-
-import createTestTransform from '../../helpers/createTestTransform';
-import transformer from '../transformer';
-
-jest.mock(
-  '../../../utils',
-  (): Record<string, unknown> => ({
-    ...jest.requireActual('../../../utils'),
-    reportManualReview: jest.fn(),
-  }),
-);
-
-describe('button codemod', () => {
-  const transform = createTestTransform(transformer);
-  it('renames ActionButton to Button and adds v2 and size', () => {
-    const input = `
-    import { ActionButton } from '@transferwise/components';
-    export default function Test() {
-      return <ActionButton>Click</ActionButton>;
-    }
-            `;
-    const output = transform({ source: input });
-    expect(output).toContain('<Button v2 size="sm">Click</Button>');
-    expect(output).not.toContain('ActionButton');
-  });
-
-  it('adds v2 to Button if missing', () => {
-    const input = `
-      import { Button } from '@transferwise/components';
-      export default function Test() {
-        return <Button htmlType="button">Click</Button>;
-      }
-    `;
-    const output = transform({ source: input });
-    expect(output).toContain('<Button v2 type="button">Click</Button>');
-  });
-
-  it('removes legacy props and adds new ones', () => {
-    const input = `
-      import { Button } from '@transferwise/components';
-      export default function Test() {
-        return (
-          <Button priority="primary" size="xs" type="accent" htmlType="submit" sentiment="positive">Test</Button>
-        );
-      }
-    `;
-    const output = transform({ source: input });
-    expect(output).toContain('size="sm"');
-    expect(output).toContain('priority="primary"');
-    expect(output).toContain('type="submit"');
-    expect(output).not.toContain(
-      'priority="primary" size="xs" type="accent" htmlType="submit" sentiment="positive"',
-    );
-  });
-
-  it('converts enum props to string values', () => {
-    const input = `
-      import { Button } from '@transferwise/components';
-      export default function Test() {
-        return (
-          <Button priority={Priority.SECONDARY} type={ControlType.NEGATIVE}>Secondary Negative</Button>
-        );
-      }
-    `;
-    const output = transform({ source: input });
-    expect(output).toContain('priority="secondary"');
-    expect(output).toContain('sentiment="negative"');
-  });
-
-  it('handles htmlType="submit" and legacy type/priority enums', () => {
-    const input = `
-      import { Button } from '@transferwise/components';
-      export default function Test() {
-        return (
-          <Button htmlType="submit" type={ControlType.ACCENT} priority={Priority.PRIMARY} disabled>
-            Button Disabled
-          </Button>
-        );
-      }
-    `;
-    const output = transform({ source: input });
-    expect(output).toContain('type="submit"');
-    expect(output).toContain('priority="primary"');
-    expect(output).toContain('disabled');
-    expect(output).not.toContain('htmlType');
-  });
-
-  it('removes legacy props and keeps style and other non-legacy props', () => {
-    const input = `
-      import { Button } from '@transferwise/components';
-      export default function Test() {
-        return (
-          <Button type={ControlType.NEGATIVE} style={{ color: 'red' }} priority={Priority.PRIMARY}>
-            Primary Negative
-          </Button>
-        );
-      }
-    `;
-    const output = transform({ source: input });
-    expect(output).toContain('sentiment="negative"');
-    expect(output).toContain('priority="primary"');
-    expect(output).toContain("style={{ color: 'red' }}");
-    expect(output).not.toContain('ControlType');
-    expect(output).not.toContain('Priority');
-  });
-
-  it('adds v2 to Button if missing and keeps block/as/href', () => {
-    const input = `
-      import { Button } from '@transferwise/components';
-      export default function Test() {
-        return (
-          <Button as="a" href="/account" block>
-            Go to account
-          </Button>
-        );
-      }
-    `;
-    const output = transform({ source: input });
-    expect(output).toContain('v2');
-    expect(output).not.toContain('as="a"');
-    expect(output).toContain('href="/account"');
-    expect(output).toContain('block');
-  });
-
-  it('removes as="a" and adds href="#" if missing', () => {
-    const input = `
-      import { Button } from '@transferwise/components';
-      export default function Test() {
-        return (
-          <Button as="a">Link</Button>
-        );
-      }
-    `;
-    const output = transform({ source: input });
-    expect(output).not.toContain('as="a"');
-    expect(output).toContain('href="#"');
-  });
-
-  it('reports ambiguous size prop (identifier)', async () => {
-    const input = `
-      import { Button } from '@transferwise/components';
-      export default function Test() {
-        return <Button size={dynamicSize}>Test</Button>;
-      }
-    `;
-    transform({ source: input });
-    const report = await fs.readFile('codemod-report.txt', 'utf8');
-    expect(report).toContain(
-      'Manual review required: prop "size" on <Button> has unsupported value "dynamicSize".',
-    );
-  });
-});